-
-
Notifications
You must be signed in to change notification settings - Fork 746
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
Minor updates for Rocky and upgrade docker engine for CI #5552
Conversation
contrib/linux/actions/service.py
Outdated
@@ -70,6 +70,7 @@ def get_linux_distribution(): | |||
re.search(distro, "Redhat") | |||
or re.search(distro, "Fedora") | |||
or re.search(distro, "CentOS Linux") |
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 exactly related to this PR, but can we modify this to say CentOS
instead of CentOS Linux
? The stream releases won't match the current version, since platform.linux_distribution()
for stream returns
platform.linux_distribution()
>> ('CentOS Stream', '8', '')
and the lsb_release
way returns
→ lsb_release -i -s
CentOSStream
On CentOS 7, I guess this will still fail as well with Python >= 3.8
→ lsb_release -i -s
CentOS
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 would be happy with that. Could you file a new PR?
Pull Request is not mergeable
Co-authored-by: Eugen Cusmaunsa <[email protected]>
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 @amanda11 ! 💯
Looks we're good now!
just added one more minor commit to sync up the CircleCI config with the upstream st2-packages version.
Part of StackStorm/community#88