-
Notifications
You must be signed in to change notification settings - Fork 39
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
How to not allow PR's merged in Staging to get merged in Prod! #2889
Comments
I think this depends on the kinds of changes. What kinds of changes do you have in mind? |
@yuvipanda the idea would be to permit me to make changes to things to grafana or perhaps the hubs that would require testing over multiple days without holding up other deploys to prod. Essentially is there a way to cherry pick commits to staging for deployments to prod? |
@felder the way I do this is to deploy from my local machine to particular staging instances. It'll get wiped everytime anyone merges a PR though, but then I just deploy again and go as it just takes a minute or two. What do you think of that? |
@yuvipanda that will probably work. I may need a short tutorial on hubploy to make sure I know how to do this. |
@felder makes sense! I think it would be:
Can you try that out? |
@yuvipanda That worked for me back in the git-crypt days, but not with sops. hubploy wants things to be decrypted. CI decrypts in place at various stages with Since hubploy ultimately runs helm with the path to deployments/{deployment}/secrets/staging.yaml, I suppose you can manually replace that with process substitution, e.g. |
@ryanlovett are you on the latest version of hubploy? I added sops support to hubploy so you wouldn't have to do that. |
@yuvipanda No, sorry, I should have checked! |
Per @yuvipanda 's advice in slack, one should install hubploy from github, https://github.com/yuvipanda/hubploy. |
I tried to use hubploy to deploy biology-staging yesterday, I was not successful but also have not yet tried other hubs or dug into why it failed. |
@felder cool, let me know what errors you run into and I'll see how I can help! You'll need to install it from git, not from pypi |
@yuvipanda yeah did the git installation and before roping you in I definitely wanted to confirm myself whether or not it was chair to computer interface problem. |
If we want to play around with staging environment without making changes to prod/ensuring that the changes get merged in prod, what would be an effective way forward?
The text was updated successfully, but these errors were encountered: