-
Notifications
You must be signed in to change notification settings - Fork 6
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
seed data from a private repo #518
Conversation
@WillDaSilva @magreenbaum two questions:
For 2 I dont think this will be an uncommon pattern to install private packages so its good to come up with a pattern now. cc @tayloramurphy @aaronsteers |
|
@magreenbaum awesome thanks for the info! I just tested and I can use a fine grain access token for my dbt package git url so this will work for me. It seemed from stack overflow comments that deploy keys were preferable to access token because they have limited scope, but I guess with fine grain access tokens now you get the same feature, so thats fine with me. For this approach I'll need someone to create me an access token from an account that has ownership of the internal-data repos, maybe meltybot? When I create fine grain tokens I can only select from my own pnadolny13 repos. Is that something you can help me with? |
@pnadolny13 let me take a look at what's involved in the ssh implementation and get back to you on this. I think we might want to go that route to start since it will probably solve more than one use case. |
Related: Thanks @magreenbaum! |
One more update, but after this PR we should be good. We will be able to use the When you're copying the private key to the
And that'll encrypt it into I'll do an instructional write-up in https://github.com/meltano/cloud-docs. Edit: We should be good now. Update PR was merged. |
@magreenbaum thanks for that! I updated this PR with the secrets changes, can you verify that its normal to have all of them change even though I'm really just adding the git private key and cloudwatch keys? |
@pnadolny13 yep, all the encrypted values get updated when using kms-ext to encrypt your .env file. That's normal. 👍 |
Closes https://github.com/meltano/internal-data/issues/56
Imports the private dbt package for seeds. I tested that it worked locally but I still need to give CI access to the repo for testing and deployment.
I decided to use the
webfactory/ssh-agent
github action to add ssh keys easier but theres one issue that I want to confirm we wont have problems with webfactory/ssh-agent#129 and webfactory/ssh-agent#106, which relates to having self hosted runners. I dont think we use self hosted runners but wanted to confirm before merging.