Skip to content

chore: Use / install python 3.12 on analytics-api.#279

Merged
macdiesel merged 1 commit intomasterfrom
bbeggs/analytics_api_312
Oct 23, 2025
Merged

chore: Use / install python 3.12 on analytics-api.#279
macdiesel merged 1 commit intomasterfrom
bbeggs/analytics_api_312

Conversation

@macdiesel
Copy link
Copy Markdown
Member


Make sure that the following steps are done before merging:

  • Have a Site Reliability Engineer review the PR if you don't own all of the services impacted.
  • If you are adding any new default values that need to be overridden when this change goes live, update internal repos and add an entry to the top of the CHANGELOG.
  • Performed the appropriate testing.

Copilot AI review requested due to automatic review settings October 23, 2025 14:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR upgrades the analytics-api service from Python 3.11 to Python 3.12. The change addresses an image build issue by disabling Python 3.11 and enabling Python 3.12 in the Ansible role configuration.

Key Changes:

  • Disabled Python 3.11 installation/usage in the analytics-api service
  • Enabled Python 3.12 installation/usage in the analytics-api service

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


ANALYTICS_API_GIT_IDENTITY: !!null

# Upgrading to python 3.12, lets leave 3.11 out of the image as it's preventing the image from building.
Copy link

Copilot AI Oct 23, 2025

Choose a reason for hiding this comment

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

Corrected spelling of 'lets' to 'let's' in the comment.

Suggested change
# Upgrading to python 3.12, lets leave 3.11 out of the image as it's preventing the image from building.
# Upgrading to python 3.12, let's leave 3.11 out of the image as it's preventing the image from building.

Copilot uses AI. Check for mistakes.
@macdiesel macdiesel force-pushed the bbeggs/analytics_api_312 branch from 33bf0d5 to df0183e Compare October 23, 2025 14:58
@macdiesel macdiesel merged commit 044bdfc into master Oct 23, 2025
3 checks passed
@macdiesel macdiesel deleted the bbeggs/analytics_api_312 branch October 23, 2025 15:25
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.

3 participants