Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add action for deploying oss connector catalog to GCS #18633

Merged
merged 2 commits into from
Nov 3, 2022

Conversation

pedroslopez
Copy link
Contributor

@pedroslopez pedroslopez commented Oct 28, 2022

What

To be able to use the remote OSS catalog as a base, we need to upload it to GCS.

to merge after #18562

close https://github.com/airbytehq/airbyte-cloud/issues/3164

How

Create an action that will be auto triggered when a catalog change is merged to master. This action will generate the new oss_catalog.json and upload it to the GCS bucket.

@pedroslopez pedroslopez temporarily deployed to more-secrets October 28, 2022 21:18 Inactive
@pedroslopez pedroslopez marked this pull request as ready for review November 2, 2022 23:26
@pedroslopez pedroslopez temporarily deployed to more-secrets November 2, 2022 23:27 Inactive
- name: Setup Cloud SDK
uses: google-github-actions/setup-gcloud@v0
with:
service_account_key: ${{ secrets.PROD_SPEC_CACHE_SA_KEY }}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added this new secret after chatting with @git-phu, who verified the prod-spec-cache-bucket-SA-key from LastPass has access to the CMS bucket

branches:
- master
paths:
- airbyte-config/init/src/main/resources/seed
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice - only runs when the right directory has been changed, only on the master branch.

- name: Upload catalog to GCS
shell: bash
run: |
gcs_bucket_name="prod-airbyte-cloud-connector-metadata-service"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a secret, right?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be an OSS bucket that's distinct from the Cloud bucket?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These files are meant to be public so I think it's ok for the bucket name to be in plaintext. The spec cache bucket name is also in the publish scripts.

As for having a separate bucket, infra said it was ok for them to be in the same bucket, and I think it might make things easier down the line if we want to actually put these behind a domain if they're in the same bucket.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 then!

I guess ideally the bucket name wouldn't have had ☁️ in the name... but 🤷

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree 😅

@pedroslopez pedroslopez merged commit b1ca1c2 into master Nov 3, 2022
@pedroslopez pedroslopez deleted the pedroslopez/deploy-catalog branch November 3, 2022 00:17
letiescanciano added a commit that referenced this pull request Nov 3, 2022
* master: (38 commits)
  New Source: Gridly (#18342)
  🎉 New Source: Alpha Vantage (#18320)
  ci_integration_test.sh: cut GITHUB_STEP_SUMMARY (#18895)
  🎉 New Source: Datadog [python cdk] (#18150)
  Hide Reject all button in consent dialog (#18596)
  feat: add doc url to track event (#18690)
  fix: install java in oss catalog deploy action (#18887)
  [CI] Speed up check_images_exist (#18873)
  Extract open API (#18879)
  Remove unused interfaces (#18880)
  add action for deploying oss connector catalog to GCS (#18633)
  feat: generate full connector catalog json (#18562)
  Add unsupported_protocol_version column to Connection (#18876)
  Extract OAuth API (#18818)
  update images to have non-transparent background (#18874)
  DiscoverSchema endpoints calculates diff and breaking change (#18571)
  Validate protocol version on connector update (#18639)
  Bmoric/extract notification api (#18812)
  Show version and changelog status for affected connectors (#18845)
  Bmoric/extract logs api (#18621)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants