-
-
Notifications
You must be signed in to change notification settings - Fork 74
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
Drop U16 and add U20 support #298
Conversation
Looks like we'll also need to add |
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.
I have few minor changes and questions.
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.
As well as the comment on mongo 4.2, I think we also need updates on:
- .kitchen.yml - has ubuntu 16 still
- .travis.yml
Will update the ci files (travis, kitchen, etc.) till tomorrow evening. |
ansible-st2/roles/StackStorm.mongodb/defaults/main.yml Lines 2 to 3 in 94e3c4f
But you could do this: # MongoDB default version
-mongodb_version: "4.0"
+mongodb_version: "{% if ansible_facts.os_family == 'Debian' and ansible_facts.distribution_major_version == '20' %}4.4{% else %}4.0{% endif %}" |
I did it like that now. Guess that's the smartest solution to the issue and does not break with any common pattern or best practices. |
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.
LGTM - but wait for those others that gave review comments to give their approval.
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.
LGTM
Use focal instead of xenial Co-authored-by: Eugen Cusmaunsa <[email protected]>
- DISTRO=ubuntu-18 | ||
- DISTRO=ubuntu-20 |
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.
Need to remove ubuntu-20 from here. We can't run it as a stable run yet until 3.5 is released.
So we need to do the changes for ansible-st2 in 2 stages.
- add it to unstable before release - under this PR
- add it to stable after release - separate PR
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.
I think we'll just have to wait for a release to happen before merging this PR.
Otherwise, someone using the fresh Ansible installer will have the same issues with U16/U20.
@@ -1,3 +1,3 @@ | |||
--- | |||
# MongoDB default version | |||
mongodb_version: "4.0" | |||
mongodb_version: "{% if ansible_facts.os_family == 'Debian' and ansible_facts.distribution_major_version == '20' %}4.4{% else %}4.0{% endif %}" |
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.
To decide between 4.0 and 4.4 mongo version, wouldn't it be a better idea to use st2 version > 3.4 rather than only on Ubuntu 20?
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.
@rush-skills Stackstorm 3.5 still uses mongo 4.0 for all OSes EXCEPT for Ubuntu 20
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.
👍
Following the #Ansible community messages, we want to merge it ASAP, considering the 3.5.0 release is ready.
Just migrated repo to the new TravisCI.com to get the build finished.
All green, great work everyone! |
This PR drops the support for Ubuntu 16 (xenial) and adds support for Ubuntu 20 (focal).
In addition this PR also adds meta/main.yml files for the redis and smoketests role which weren't there yet. This is rather a matter of consistency than of functionality but it seems a good chance to get this done.
It also removes support for MongoDB-Versions that are already EoL as there are 3.2 and 3.4.
Tests with st2repo_name = unstable were successful.