Skip to content

Commit

Permalink
Merge branch 'master' into ddavydov/#1047-oncall-source-notion-fix-sc…
Browse files Browse the repository at this point in the history
…hema
  • Loading branch information
davydov-d committed Jan 10, 2023
2 parents fb1c52d + f921d8c commit e78514d
Show file tree
Hide file tree
Showing 1,569 changed files with 48,105 additions and 28,145 deletions.
2 changes: 1 addition & 1 deletion .bumpversion.cfg
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[bumpversion]
current_version = 0.40.26
current_version = 0.40.27
commit = False
tag = False
parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)(\-[a-z]+)?
Expand Down
7 changes: 3 additions & 4 deletions .env
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@


### SHARED ###
VERSION=0.40.26
VERSION=0.40.27

# When using the airbyte-db via default docker image
CONFIG_ROOT=/data
Expand Down Expand Up @@ -47,13 +47,13 @@ DATABASE_PORT=5432
DATABASE_DB=airbyte
# translate manually DATABASE_URL=jdbc:postgresql://${DATABASE_HOST}:${DATABASE_PORT}/${DATABASE_DB} (do not include the username or password here)
DATABASE_URL=jdbc:postgresql://db:5432/airbyte
JOBS_DATABASE_MINIMUM_FLYWAY_MIGRATION_VERSION=0.29.15.001
JOBS_DATABASE_MINIMUM_FLYWAY_MIGRATION_VERSION=0.40.26.001

# Airbyte Internal Config Database, defaults to Job Database if empty. Explicitly left empty to mute docker compose warnings.
CONFIG_DATABASE_USER=
CONFIG_DATABASE_PASSWORD=
CONFIG_DATABASE_URL=
CONFIGS_DATABASE_MINIMUM_FLYWAY_MIGRATION_VERSION=0.35.15.001
CONFIGS_DATABASE_MINIMUM_FLYWAY_MIGRATION_VERSION=0.40.23.002

### AIRBYTE SERVICES ###
TEMPORAL_HOST=airbyte-temporal:7233
Expand Down Expand Up @@ -115,4 +115,3 @@ OTEL_COLLECTOR_ENDPOINT="http://host.docker.internal:4317"

USE_STREAM_CAPABLE_STATE=true
AUTO_DETECT_SCHEMA=false

15 changes: 5 additions & 10 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
@@ -1,18 +1,13 @@
# Frontend code
/airbyte-webapp-e2e-tests/ @airbytehq/frontend
/airbyte-webapp/ @airbytehq/frontend
## Exclude the package(-lock).json from code ownership to prevent version bump PRs from triggering codeowners review
/airbyte-webapp/package.json
/airbyte-webapp/package-lock.json

# Design
/airbyte-config/init/src/main/resources/icons/*.svg @airbytehq/design

# CDK and SAT
/airbyte-cdk/ @airbytehq/connector-extensibility
/airbyte-integrations/bases/source-acceptance-tests/ @airbytehq/connector-extensibility
/airbyte-integrations/connector-templates/ @airbytehq/connector-extensibility

# Oauth
/airbyte-oauth/ @airbytehq/connector-operations
/airbyte-server/src/main/java/io/airbyte/server/handlers/OAuthHandler.java @airbytehq/connector-operations
/airbyte-server/src/test/java/io/airbyte/server/handlers/OAuthHandlerTest.java @airbytehq/connector-operations

# Protocol related items
/airbyte-protocol/ @airbytehq/protocol-reviewers
/docs/understanding-airbyte/airbyte-protocol.md @airbytehq/protocol-reviewers
Expand Down
23 changes: 0 additions & 23 deletions .github/PULL_REQUEST_TEMPLATE/bug-fix_template.md

This file was deleted.

1 change: 1 addition & 0 deletions .github/actions/start-aws-runner/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ inputs:
required: false
ec2-instance-id:
required: false

outputs:
label:
value: ${{ steps.start-ec2-runner.outputs.label }}
Expand Down
6 changes: 3 additions & 3 deletions .github/labeler.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
# union of frontend, api, server, scheduler, protocol, worker, kubernetes
area/platform:
- airbyte-webapp/*
- airbyte-webapp/**/*
- airbyte-api/*
- airbyte-api/**/*
- airbyte-persistence/*
Expand All @@ -17,9 +15,11 @@ area/platform:
- charts/*
- charts/**/*

team/frontend:
area/frontend:
- airbyte-webapp/*
- airbyte-webapp/**/*
- airbyte-webapp-e2e-tests/*
- airbyte-webapp-e2e-tests/**/*

area/api:
- airbyte-api/*
Expand Down
28 changes: 20 additions & 8 deletions .github/workflows/gradle.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,11 @@ on:
inputs:
debug_mode:
description: "Enable or disable tmate session for debug during helm ac tests"
type: choice
default: 'false'
options:
- 'true'
- 'false'
required: false
schedule:
- cron: "0 */1 * * *"
Expand Down Expand Up @@ -67,6 +71,7 @@ jobs:
- 'buildSrc/**'
- 'tools/**'
- '*.gradle'
- 'deps.toml'
cli:
- 'airbyte-api/**'
- 'octavia-cli/**'
Expand Down Expand Up @@ -811,6 +816,16 @@ jobs:
with:
python-version: "3.9"

- name: Install unzip
shell: bash
run: sudo apt-get update && sudo apt-get install -y unzip

- uses: azure/setup-helm@v3
with:
version: 'latest'
token: ${{ secrets.GITHUB_TOKEN }}
id: install

- name: Fix EC-2 Runner
run: |
mkdir -p /home/runner
Expand Down Expand Up @@ -862,6 +877,8 @@ jobs:
HOME: /home/runner
# AWS_S3_INTEGRATION_TEST_CREDS can be found in LastPass as AWS_S3_INTEGRATION_TEST_CREDS
AWS_S3_INTEGRATION_TEST_CREDS: ${{ secrets.AWS_S3_INTEGRATION_TEST_CREDS }}
WORKFLOW_RUN_ID: ${{ github.run_id }}
AWS_S3_BUCKET: airbyte-ci-ac-tests-logs
SECRET_STORE_GCP_CREDENTIALS: ${{ secrets.SECRET_STORE_GCP_CREDENTIALS }}
SECRET_STORE_GCP_PROJECT_ID: ${{ secrets.SECRET_STORE_GCP_PROJECT_ID }}
uses: Wandalen/wretry.action@master
Expand Down Expand Up @@ -998,6 +1015,7 @@ jobs:
aws-access-key-id: ${{ secrets.SELF_RUNNER_AWS_ACCESS_KEY_ID }}
aws-secret-access-key: ${{ secrets.SELF_RUNNER_AWS_SECRET_ACCESS_KEY }}
github-token: ${{ env.PAT }}
iam-role-name: ${{ secrets.AWS_ASSUME_ROLE_NAME }}

helm-acceptance-test:
name: "Platform: Acceptance Tests (Helm)"
Expand Down Expand Up @@ -1028,12 +1046,8 @@ jobs:
uses: Wandalen/wretry.action@master
with:
command: |
sudo rm /var/lib/dpkg/lock
sudo rm /var/lib/dpkg/lock-frontend
sudo apt-get remove docker.io || sudo apt-get remove docker
curl -fsSL https://get.docker.com | bash -
sudo rm -f /var/lib/dpkg/lock
sudo rm -f /var/lib/dpkg/lock-frontend
attempt_limit: 3
attempt_delay: 2000 # in ms

Expand Down Expand Up @@ -1070,11 +1084,7 @@ jobs:
uses: Wandalen/wretry.action@master
with:
command: |
sudo rm /var/lib/dpkg/lock
sudo rm /var/lib/dpkg/lock-frontend
sudo apt-get -y install tmate
sudo rm -f /var/lib/dpkg/lock
sudo rm -f /var/lib/dpkg/lock-frontend
attempt_limit: 3
attempt_delay: 2000 # in ms

Expand Down Expand Up @@ -1129,6 +1139,8 @@ jobs:
# HOME: /home/runner
# AWS_S3_INTEGRATION_TEST_CREDS can be found in LastPass as AWS_S3_INTEGRATION_TEST_CREDS
AWS_S3_INTEGRATION_TEST_CREDS: ${{ secrets.AWS_S3_INTEGRATION_TEST_CREDS }}
WORKFLOW_RUN_ID: ${{ github.run_id }}
AWS_S3_BUCKET: airbyte-ci-ac-tests-logs
SECRET_STORE_GCP_CREDENTIALS: ${{ secrets.SECRET_STORE_GCP_CREDENTIALS }}
SECRET_STORE_GCP_PROJECT_ID: ${{ secrets.SECRET_STORE_GCP_PROJECT_ID }}
timeout-minutes: 20
Expand Down
30 changes: 0 additions & 30 deletions .github/workflows/notify-on-label.yml

This file was deleted.

3 changes: 2 additions & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ repos:
rev: v0.8.8
hooks:
- id: licenseheaders
args: ["--tmpl=LICENSE_SHORT", "--ext=py", "-f"]
args:
["--tmpl=LICENSE_SHORT", "--ext=py", "-x=**/models/__init__.py", "-f"]
- repo: https://github.com/psf/black
rev: 22.3.0
hooks:
Expand Down
4 changes: 1 addition & 3 deletions airbyte-api/build.gradle
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
import org.openapitools.generator.gradle.plugin.tasks.GenerateTask

plugins {
// 6.1.0-SNAPSHOT contains this fix: https://github.com/OpenAPITools/openapi-generator/issues/13025
// TODO update to v6.1.0 when it officially releases (due end of August 2022) so that we don't need to depend on a SNAPSHOT version anymore
id "org.openapi.generator" version "6.1.0-SNAPSHOT"
id "org.openapi.generator" version "6.2.1"
id "java-library"
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,12 @@
* This class is meant to consolidate all our API endpoints into a fluent-ish client. Currently, all
* open API generators create a separate class per API "root-route". For example, if our API has two
* routes "/v1/First/get" and "/v1/Second/get", OpenAPI generates (essentially) the following files:
*
* <p>
* ApiClient.java, FirstApi.java, SecondApi.java
*
* <p>
* To call the API type-safely, we'd do new FirstApi(new ApiClient()).get() or new SecondApi(new
* ApiClient()).get(), which can get cumbersome if we're interacting with many pieces of the API.
*
* <p>
* This is currently manually maintained. We could look into autogenerating it if needed.
*/
public class AirbyteApiClient {
Expand Down
Loading

0 comments on commit e78514d

Please sign in to comment.