Skip to content

Conversation

@m4dcoder
Copy link
Contributor

@m4dcoder m4dcoder commented Apr 8, 2021

The coordinator service is required in most cases for running workflows. Install redis-server and configure redis as the default coordinator backend used by tooz. This will avoid a number of workflow issues, especially for new StackStorm users.

The coordinator service is required in most cases for running workflows.
Install redis-server and configure redis as the default coordinator
backend used by tooz. This will avoid a number of workflow issues,
especially for new StackStorm users.
@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label Apr 8, 2021
@m4dcoder m4dcoder added this to the 3.5.0 milestone Apr 8, 2021
@m4dcoder m4dcoder requested review from Kami and arm4b April 8, 2021 19:14
@m4dcoder
Copy link
Contributor Author

m4dcoder commented Apr 8, 2021

This PR pairs with StackStorm/st2-packages#698.

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.

👍

Would be nice to identify if stackstorm docs would need an update around this.
I believe we'll also need to highlight the change in the v3.5.0 Roadmap.

@m4dcoder
Copy link
Contributor Author

m4dcoder commented Apr 9, 2021

@armab Sure, will update docs to make mention

@m4dcoder m4dcoder merged commit 4519fa8 into master Apr 9, 2021
@m4dcoder m4dcoder deleted the redis-coordinator-default branch April 9, 2021 20:29
Copy link
Member

@Kami Kami left a comment

Choose a reason for hiding this comment

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

LGTM. This means we can now also simplify github st2.conf.dev step a bit.

@Kami
Copy link
Member

Kami commented Apr 9, 2021

I assume there will also be a corresponding change for st2vagrantdev and other places (e.g. docker files) in the future?

@Kami
Copy link
Member

Kami commented Apr 10, 2021

StackStorm/st2vagrantdev#9 - will merge it as soon as the tests / CI there pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS PR that changes 0-9 lines. Quick fix/merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants