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

Upgrade admin tests and container for bullseye / Python 3.9 #6645

Merged
merged 3 commits into from
Oct 18, 2022

Conversation

legoktm
Copy link
Member

@legoktm legoktm commented Oct 17, 2022

Status

Ready for review

Description of Changes

Tails 5 is Debian Bullseye / Python 3.9, this updates the CI/dev environment for things that run on Tails to match.

  • journalist_gui: Run tests on bullseye / Python 3.9
  • admin: Update container and tests to use bullseye / Python 3.9
  • admin: Upgrade flake8 and pylint. I kept this as a separate commit instead of squashing it with the main admin commit just to make it easier to distinguish why each dependency is changing.

Fixes #6635.

See commit messages for further details.

Testing

How should the reviewer test this PR?

  • Run make -C admin test locally
  • CI passes

Deployment

Any special considerations for deployment? No, these are all CI/dev-only changes.

Checklist

  • Linting and tests (make -C admin test) pass in the admin development container
  • These changes do not require documentation

We've moved to Tails 5, which is based on Debian Bullseye and uses
Python 3.9.

Additionally:
* Use a plain Debian image instead of a CircleCI one
* Use `apt update` instead of `apt-get update`, because it automatically
  accepts releaseinfo changes
* Drop the `$2>/dev/null`, which doesn't appear to do anything. I assume
  it's a typo for `2>/dev/null` to hide stderr, except all the output is
  on stderr, so we probably don't want that.

Refs #6635.
We've moved to Tails 5, which is based on Debian Bullseye and uses
Python 3.9. Some dependencies are dropped because they were only needed
for older Python versions.

Additionally:
* Stop installing lsb-release, we dropped it in #6530.
* Use a plain Debian image instead of a CircleCI one

Fixes #6635.
We primarily needed to upgrade pyflakes for Python 3.8 compatibility
(PyCQA/pyflakes#367), but upgrading flake8
also meant pulling in a newer pylint since they have a common
dependency, so here we are.
@legoktm legoktm requested a review from a team as a code owner October 17, 2022 20:35
@eaon eaon self-assigned this Oct 18, 2022
Copy link
Contributor

@eaon eaon left a comment

Choose a reason for hiding this comment

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

🎉 Thank you!

(Adding a note that I technically tested with a locally patched admin/bin/dev-shell to make it use podman instead of docker, but given that CI works as well, I'm sure this is good as is)

@eaon eaon merged commit 3288be8 into develop Oct 18, 2022
@eaon eaon deleted the admin-bullseye branch October 18, 2022 21:48
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.

admin Dockerfile and tests need to be updated for bullseye
2 participants