Skip to content

Conversation

@warrenvw
Copy link
Contributor

Use latest versions of the rabbitmq-ha and mongodb-replicaset charts.

@warrenvw warrenvw self-assigned this Feb 23, 2019
@warrenvw warrenvw requested a review from arm4b February 23, 2019 01:56
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

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

Looks good.

Need to bump chart version as well then, due to updated deps.

Additionally, apart of these 2 dependencies, are there any other upstream updates we can do?

@warrenvw
Copy link
Contributor Author

I've bumped all other dependencies, updated chart version and added an entry in the changelog.

@arm4b
Copy link
Member

arm4b commented Feb 28, 2019

Per previous discussion, if new rabbitmq-ha Helm chart includes pod naming changes, then it needs more work TBD to verify if ST2 HA Helm templating works with those upstream changes well.

@warrenvw warrenvw merged commit f7498be into master Mar 6, 2019
@warrenvw warrenvw deleted the bump/dependencies branch March 6, 2019 00:01
appVersion: 3.0dev
name: stackstorm-ha
version: 0.9.0
version: 0.10.0
Copy link
Member

Choose a reason for hiding this comment

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

Ideally version needs to be bumped just before the release, as by the fact this is not yet 0.10.0.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops, this slipped in there.. agreed.

@warrenvw
Copy link
Contributor Author

warrenvw commented Mar 6, 2019

Note that rabbitmq-ha 1.21.0 has a problem with the security settings / permissions, so I held its version at 1.20.1.

@arm4b
Copy link
Member

arm4b commented Mar 8, 2019

Useful info, It was a good opportunity to include that in code comments.

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.

3 participants