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
🎉 CI secrets: migration to GSM #7528
Conversation
…zendesk_support/streams.py Co-authored-by: George Claireaux <george@claireaux.co.uk>
/test connector=connectors/source-s3
|
/test connector=connectors/source-s3
|
/test connector=connectors/source-s3
|
/test connector=connectors/source-s3
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you let me know once you've written up .md instructions for how to add devs to the GCP project? I'll add everyone to that project
Otherwise LGTM
Co-authored-by: Sherif A. Nada <snadalive@gmail.com>
Co-authored-by: Sherif A. Nada <snadalive@gmail.com>
|
/test connector=connectors/source-zendesk-support
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very exciting!
…rbyte into antixar/3732-gsm-ci-secrets
/test connector=connectors/source-zendesk-support
|
/test connector=connectors/source-s3
|
/test connector=connectors/source-zendesk-support
|
/test connector=connectors/source-commercetools
|
* fix 404 responses for the ticket_comments stream * add unit test * add unit test * add oauth2 access token * Update airbyte-integrations/connectors/source-zendesk-support/source_zendesk_support/streams.py Co-authored-by: George Claireaux <george@claireaux.co.uk> * switching among auth methods * update spec file * update CI secrets logic * update CI secrets logic * remove debug data * add a debug message * fix json convertation * fix json convertation * support one secret by several connectors * Update tools/bin/ci_credentials.sh Co-authored-by: Sherif A. Nada <snadalive@gmail.com> * Update tools/bin/ci_credentials.sh Co-authored-by: LiRen Tu <tuliren@gmail.com> * update function names * update docs * reset failed changes * update json set value * update secrets json key * Update tools/bin/ci_credentials.sh * add JSON validation * Update docs/connector-development/README.md Co-authored-by: Sherif A. Nada <snadalive@gmail.com> * update doc * Update README.md * rename the function write_standard_creds Co-authored-by: Maksym Pavlenok <maksym.pavlenok@globallogic.com> Co-authored-by: George Claireaux <george@claireaux.co.uk> Co-authored-by: Sherif A. Nada <snadalive@gmail.com> Co-authored-by: LiRen Tu <tuliren@gmail.com>
What
Details here
How
This decision provides backward compatibility with all secrets workflows. All old settings should work after this changes.
How to add permissions for working with Google Secret Manager(GSM):
Each developer should have the role
Development_CI_Secrets
of the projectdataline-integration-testing
Now these permissions are provided for GL Python/Java teams only.
How to add new secrets:
SECRET_
./secrets/config.json
test
command for your connector/For examples:
Labels:
Result: value =>
airbyte-integrations/connectors/source-s3/secrets/config.json
Labels:
Result: value =>
airbyte-integrations/connectors/source-zendesk-support/secrets/config_auth.json
Labels:
Result:
airbyte-integrations/connectors/source-google-analytics/secrets/config.json
airbyte-integrations/connectors/source-google-drive/secrets/config.json
How to migrate to the new secrets' logic:
tools/bin/ci_credentials.sh
Recommended reading order
.github/workflows/{publish|test}-command.yml
- removed all secrets' values because the new script version will select necessary ones only.tools/lib/gcp-token.sh
- functions for generation of Google tokens. It can be used not only for the current task.tools/bin/ci_credentials.sh
- changes of all CI secrets' logicPre-merge Checklist
Connector Generator
-scaffold
in their name) have been updated with the latest scaffold by running./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates
then checking in your changes