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

chore: fix refresh job #1758

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

katebygrace
Copy link
Contributor

@katebygrace katebygrace commented Apr 16, 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


SNOWFLAKE_REFRESH_SNOWPIPE_JOB_EXTRA_VARS.yaml

KEY_PATH: "snowflake/rsa_key_snowpipe_user.p8"
PASSPHRASE_PATH: 'snowflake/rsa_key_passphrase_snowpipe_user'
USER: 'SNOWPIPE'
ACCOUNT: 'edx.us-east-1'
JOB_FREQUENCY: "H 22 * * *"

@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch 30 times, most recently from a6df4ff to ae1c8d2 Compare April 17, 2024 19:16
@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch 7 times, most recently from f09c703 to d288022 Compare April 18, 2024 20:29
@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch 14 times, most recently from 01b3bff to 773103e Compare May 3, 2024 15:51
@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch 2 times, most recently from aeb2178 to 6f59269 Compare May 6, 2024 18:00
@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch 2 times, most recently from 9721d85 to 4ae96ce Compare May 8, 2024 19:39
@katebygrace katebygrace force-pushed the katebygrace/fix-refresh-job branch from 4ae96ce to 8c2d2fe Compare May 8, 2024 19:40
@@ -35,4 +35,4 @@ else
exit 1
fi
fetch_specific_key "$1" "$2"
fi
Copy link
Contributor

Choose a reason for hiding this comment

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

We'd eventually remove this right ?

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.

None yet

2 participants