-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Fix e2e tests, drop Ubuntu18 #332
Conversation
…3==1.26.13 which breaks docker-py and molecule cannont talk to the docker api on the host. version 2.29.0 of the requests python library is currently the known fix -- docker/docker-py#3113
…d mongo are not planing to fix it
Thanks a lot @setswei for your research and the fix! We want to remove However I think we have to release a security patch I guess they could use previous |
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 we're dropping Ubuntu18 in this PR, let's also remove mentions of Ubuntu18 in other places of this repo too.
See: https://github.com/search?q=repo%3AStackStorm%2Fansible-st2%20Ubuntu&type=code
More changes needed in the README, Ansible meta files, Vagrantfile, etc.
Hey @armab no problem I will remove all references to ubuntu18. I didn’t do this initially incase it was still going to be supported for a bit. I will have the updates committed later today |
Makes sense! TBH, I'm still completely not sure if we want to remove it this way.
for a bit more until adding a proper U22 support, but that seems to be more hacky in terms of security and official support. |
…rce pipelines to run
I have made the changes that you asked for. Users who have already deployed the apt repos would still see errors when talking to the repository due to the expired keys. bypassing the certificates can be a classified as a security risk. We can put some additional notes in the readme about ubuntu 18.04 as this is a issue is not really related to ansible or stackstorm. |
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.
Thanks for the changes!
There are more meta files to update for every Ansible role that we have:
ansible-st2/roles/StackStorm.st2/meta/main.yml
Lines 9 to 11 in 222ee03
- name: Ubuntu | |
versions: | |
- bionic |
My apologies, ide search fail :P. |
…sed these in the last pass
the roles meta files have been updated. Builds are running |
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.
Something that was expected to be a fix became a deprecation of Ubuntu Bionic ;)
Thanks a lot for the contribution!
No problem at all. I will see if I can assist with some of the other builds too. |
Check |
Hey @armab
I have found the issues with the ansible-st2 build process
There was an incompatibility with docker-py using the latest versions of the requests and urllib3 libraries. This caused issues with molecule talking to the docker API on the host machine. Installing the requests<=2.29.0 resolved this issue. More info located here -> urllib3 v2 incompatibility docker/docker-py#3113
After getting the build processes working again I found that Ubuntu 18.04 tests would fail. this is due to MongoDB allowing the apt key for MongoDB 4.0 to expire. From the looks of some of the forum posts on the MongoDB community, they are not going to fix this as Ubuntu 18.04 was EOL on May 31 2023. For now, I have removed Ubuntu 18.04 from the build process, to resolve the MongoDB installation issues the role would have to be modified to download the deb packages directly and then install them (which can easily be done) but the main question I would put forward is should ubuntu 18.04 continued to be supported when there will be no mainstream OS level patches and vendors are dropping support as well.
As you can see below the builds are now working (except for Ubuntu 18.04)
If you check my repository you will see subsequent builds that have been completed with ubuntu 18.04 excluded