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
Fixes #22887 - update persistence initialization code #319
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not much to review here, works as expected, waiting for dependencies.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We now know the Dynflow release containing support for queues will be 0.9.0
, could you please bump the dependency?
9942f97
to
6fee539
Compare
Updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Waiting on dependencies, there is a merge conflict but I can resolve that on merge
6fee539
to
17b5867
Compare
There were the following issues with the commit message:
If you don't have a ticket number, please create an issue in Redmine. More guidelines are available in Coding Standards or on the Foreman wiki. This message was auto-generated by Foreman's prprocessor |
Could you please bump the Dynflow dependency again? |
Updated |
17b5867
to
0831389
Compare
0831389
to
d91c497
Compare
Thanks for the rebase, APJ |
Looks like we will need to open a PR to foreman core and bump Dynflow version there |
Here we go theforeman/foreman#5320 |
this should go in now 👍 |
💚 |
Thanks @iNecas ! |
could we have a release with this? |
Require Dynflow/dynflow#277