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.

Sign up for GitHub

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

Jump to bottom

Update base Docker image to Python 3.11.3 #102

Merged
merged 4 commits into from
Apr 17, 2023
Merged

Update base Docker image to Python 3.11.3 #102

merged 4 commits into from
Apr 17, 2023

Conversation

jsf9k
Copy link
Member

@jsf9k jsf9k commented Apr 12, 2023 •
edited
Loading

🗣 Description

This pull request upgrades the Docker base image to Python 3.11.3.

💭 Motivation and context

The latest version of Python should contain security and bug fixes and may also be more performant.

🧪 Testing

All automated tests pass. I also performed a full BOD 18-01 run with these changes without incident.

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Finalize version.

✅ Post-merge checklist

  • Create a release.

@jsf9k jsf9k added the improvement This issue or pull request will add or improve functionality, maintainability, or ease of use label Apr 12, 2023
@jsf9k jsf9k self-assigned this Apr 12, 2023
@jsf9k
Copy link
Member Author

jsf9k commented Apr 17, 2023

I used these changes for this weekend's BOD 18-01 run and they functioned as expected.

@jsf9k jsf9k requested review from jasonodoom and a team April 17, 2023 12:49
@jsf9k jsf9k marked this pull request as ready for review April 17, 2023 12:49
@jsf9k jsf9k requested review from dav3r and mcdonnnj as code owners April 17, 2023 12:49
jsf9k added a commit to cisagov/orchestrator that referenced this pull request Apr 17, 2023
@jsf9k
Use the latest gatherer, pshtt_reporter, and trustymail_reporter Dock…
2c11586 
…er images

See cisagov/gatherer#102 cisagov/pshtt_reporter#81, and
cisagov/trustymail_reporter#106 for more details.
@jsf9k jsf9k merged commit 53d2a47 into develop Apr 17, 2023
@jsf9k jsf9k deleted the improvement/update-base-docker-image branch April 17, 2023 15:38
@jsf9k jsf9k mentioned this pull request Apr 17, 2023
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmorrowomni jmorrowomni jmorrowomni approved these changes

@dav3r dav3r dav3r approved these changes

@jasonodoom jasonodoom jasonodoom approved these changes

@mcdonnnj mcdonnnj Awaiting requested review from mcdonnnj mcdonnnj is a code owner

Assignees

@jsf9k jsf9k

Labels
improvement This issue or pull request will add or improve functionality, maintainability, or ease of use
Projects
BOD 18-01
Status: Done
CyHy System
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jsf9k @jasonodoom @dav3r @jmorrowomni