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

CI: drop 3.6 & ubuntu switch #870

Merged

Conversation

tylerjereddy
Copy link
Collaborator

  1. 22.04 is being rolled out as the new default anyway
  2. CI: Python 3.10 on ubuntu-latest issue #851 with the Python garbage collection issues in 3.10 was resolved by using contexts, so special treatment not justified anymore

* the Github Actions infrastructure is progressively
phasing in a newer base image of Ubuntu:
https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/

* that means that we are somewhat randomly going to
see images that lack Python `3.6` from the build cache
sometimes per:
actions/setup-python#355 (comment)

* instead of pinning to an old version of Ubuntu in GHA for
3.6 support, let's just drop 3.6 from the testing matrix
per:
darshan-hpc#510 (comment)
(it has been EOL for 1 year)

* there's also no reason to retain the special Python `3.10`
handling where we use Ubuntu `22.04` for that case, for two reasons:
1) `22.04` is being rolled out as the new default anyway
2) darshan-hpcgh-851 with the Python garbage collection issues in `3.10` was
resolved by using contexts, so special treatment not justified
anymore
@tylerjereddy tylerjereddy added CI continuous integration maintenance labels Dec 7, 2022
@shanedsnyder
Copy link
Contributor

To fully remove on from 3.6, there are still some other tasks, correct? Off the top of my head, I guess pyproject.toml needs to be updated so cibuildwheel doesn't build wheels for it, but probably other things, too, I'm guessing.

Are those out of scope here? Or is it worth ripping the bandaid off?

@tylerjereddy
Copy link
Collaborator Author

I was just going to remove the CI support to start, I don't really care if we continue here though, but you'll end up with randomly failing CI for awhile if you wait

@shanedsnyder
Copy link
Contributor

Maybe I'll just merge this in, and can start up a more general PR shortly, just to avoid any distractions on any other PRs in the mean time.

@shanedsnyder shanedsnyder merged commit f6dcf15 into darshan-hpc:main Dec 9, 2022
tylerjereddy added a commit to tylerjereddy/darshan-logs that referenced this pull request Dec 16, 2022
* for the same reasons as in the main repo:
darshan-hpc/darshan#870

* `3.6` causing issues in PRs here now like in:
darshan-hpcgh-46
@tylerjereddy tylerjereddy deleted the treddy_ci_py36_and_ubuntu branch December 16, 2022 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants