Skip to content
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 Python 3.6 from CI (Github Actions) config #38

Merged
merged 1 commit into from
Dec 1, 2022
Merged

Conversation

AdamWill
Copy link
Contributor

@AdamWill AdamWill commented Dec 1, 2022

We can't test on Python 3.6 using the setup-python action with ubuntu-latest any more, as ubuntu-latest changed from 20.04 to 22.04 and 22.04 doesn't have Python 3.6. Let's just drop 3.6 testing and note in the README that we can no longer guarantee compatibility with 3.6.

Signed-off-by: Adam Williamson awilliam@redhat.com

@AdamWill
Copy link
Contributor Author

AdamWill commented Dec 1, 2022

We do have other choices here. We could indeed just stuck with Ubuntu 20.04. We could run two workflows, one on ubuntu-latest to test 3.7 and later, one on ubuntu 20.04 to test 3.6. But on the whole I think I'm fine with this. What does everyone else think?

We can't test on Python 3.6 using the setup-python action with
ubuntu-latest any more, as ubuntu-latest changed from 20.04 to
22.04 and 22.04 doesn't have Python 3.6. Let's just drop 3.6
testing and note in the README that we can no longer guarantee
compatibility with 3.6.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
@AdamWill AdamWill changed the title Drop Python 3.6 from CI (Github Actions) config (#35) Drop Python 3.6 from CI (Github Actions) config Dec 1, 2022
Copy link
Member

@kalikiana kalikiana left a comment

Choose a reason for hiding this comment

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

Seems fine by me. I'm not concerned with using 3.6 at this point.

@AdamWill AdamWill merged commit 8c7040e into main Dec 1, 2022
@AdamWill AdamWill deleted the ci-drop-36 branch December 1, 2022 18:36
@okurz
Copy link
Member

okurz commented Dec 1, 2022

Uhm, python 3.6 is the version of SLE15 and Leap 15 so I would prefer if we keep the support for a bit longer

@kalikiana
Copy link
Member

Uhm, python 3.6 is the version of SLE15 and Leap 15 so I would prefer if we keep the support for a bit longer

I just realized that os-autoinst/openqa_review#217 is also affected - so I'm trying out the alternative option of using a different image. If that works we could also do that here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants