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

Trigger cloud catalog regeneration on definitions change #19084

Merged
merged 2 commits into from
Nov 10, 2022

Conversation

pedroslopez
Copy link
Contributor

@pedroslopez pedroslopez commented Nov 7, 2022

What

Trigger the action introduced in https://github.com/airbytehq/airbyte-cloud/pull/3371 whenever the definitions change and the OSS catalog has been successfully uploaded.

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

This should not be merged until both https://github.com/airbytehq/airbyte-cloud/pull/3371 and https://github.com/airbytehq/airbyte-cloud/pull/3300 are merged on Cloud.

How

@pedroslopez pedroslopez temporarily deployed to more-secrets November 7, 2022 19:02 Inactive
@pedroslopez pedroslopez marked this pull request as ready for review November 7, 2022 19:16
@pedroslopez pedroslopez requested review from evantahler, alafanechere and a team November 7, 2022 19:16
Comment on lines +46 to +48
token: ${{ env.PAT }}
repository: airbytehq/airbyte-cloud
event-type: generate-cloud-catalog
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh! this API is much cleaner than it used to be when we had to curl some random api.github.com URL!

Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't know respository-dispatch even existed, guess I've never had to trigger workflows across repos. Cool 😄

Comment on lines +46 to +48
token: ${{ env.PAT }}
repository: airbytehq/airbyte-cloud
event-type: generate-cloud-catalog
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't know respository-dispatch even existed, guess I've never had to trigger workflows across repos. Cool 😄

@pedroslopez pedroslopez temporarily deployed to more-secrets November 8, 2022 06:57 Inactive
@alafanechere alafanechere merged commit b1c9041 into master Nov 10, 2022
@alafanechere alafanechere deleted the pedroslopez/trigger-cloud-gen branch November 10, 2022 12:53
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.

4 participants