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

Katebygrace/demographics secretsmanager #1761

Merged
merged 1 commit into from
May 21, 2024

Conversation

katebygrace
Copy link
Contributor

@katebygrace katebygrace commented May 8, 2024

Remove calls to other secrets manager scripts in jenkins
Remove calls to secrets manager scripts in shell script
Check and remove the key_path and passphrase_path
Run python script x2
Change calls to python script to include key_file and passphrase_file (note slashes)
rm files
Review vars in analytics secure if there are any others that need to be set



job-configs/SNOWFLAKE_DEMOGRAPHICS_CLEANUP_JOB_EXTRA_VARS.yaml

KEY_PATH: "snowflake/rsa_key_stitch_loader.p8"
PASSPHRASE_PATH: 'snowflake/rsa_key_passphrase_stitch_loader'
USER: 'STITCH_LOADER'
ACCOUNT: 'edx.us-east-1'
USER_ROLES:
edx*edx-aperture:
- "hudson.model.Item.Build"
- "hudson.model.Item.Cancel"
- "hudson.model.Run.Delete"
- "hudson.model.Item.Discover"
- "hudson.model.Item.Read"
- "hudson.model.Run.Update"
- "hudson.model.Item.Workspace"

@katebygrace katebygrace changed the base branch from master to katebygrace/fix-refresh-job May 8, 2024 20:30
@katebygrace katebygrace force-pushed the katebygrace/demographics-secretsmanager branch 2 times, most recently from 772c459 to 4f158fe Compare May 9, 2024 16:58
@katebygrace katebygrace force-pushed the katebygrace/demographics-secretsmanager branch from 4f158fe to f9fdcb2 Compare May 9, 2024 18:54
Base automatically changed from katebygrace/fix-refresh-job to master May 17, 2024 19:51
@katebygrace katebygrace merged commit a78f5c5 into master May 21, 2024
3 checks passed
@katebygrace katebygrace deleted the katebygrace/demographics-secretsmanager branch May 21, 2024 19:24
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