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

Ensure newest stackstorm/stackstorm image is deployed #113

Merged
merged 12 commits into from
Apr 23, 2018

Conversation

warrenvw
Copy link
Contributor

@warrenvw warrenvw commented Mar 13, 2018

As noted in #112:

The problem is that now the latest container does not contain these fixes

While the stackstorm/stackstorm:latest image has been updated recently, it appears the bits are old. This PR attempts to fix this issue, although I suspect it can only truly be tested in production.

I took this opportunity to refactor some code to simplify.

@warrenvw warrenvw added the WIP label Mar 13, 2018
@warrenvw warrenvw self-assigned this Mar 13, 2018
@warrenvw warrenvw requested review from arm4b and bigmstone March 13, 2018 18:53
@warrenvw
Copy link
Contributor Author

I'd like to run a few more tests before calling this ready for review... but wouldn't be against any feedback at this time.

@warrenvw warrenvw added RFR and removed WIP labels Mar 14, 2018
@warrenvw warrenvw merged commit 12cd255 into master Apr 23, 2018
@warrenvw warrenvw deleted the wvw/no-tag-latest branch April 23, 2018 16:07
transhapHigsn pushed a commit to DiligenceVault/st2-docker that referenced this pull request Jun 8, 2021
Ensure newest stackstorm/stackstorm image is deployed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants