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

Update calitp-data-infra dependency version #3674

Merged
merged 5 commits into from
Feb 4, 2025

Conversation

mjumbewu
Copy link
Contributor

@mjumbewu mjumbewu commented Feb 4, 2025

Description

In order to upgrade the Cloud Composer image, we needed to update calitp-data-infra to be compatible with Python 3.11. This PR updates the airflow dependency. The dependency versions in other packages are being held back, as they are not direct blockers for this work. They should be updated in future PRs.

Relates to #3551

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

How has this been tested?

The dependency is functionally the same, just a new version for Python 3.11 compatibility.

Post-merge follow-ups

  • No action required
  • Actions required (specified below)
    • Update to the latest Cloud Composer image

- gtfs-rt-archiver-v3 depends on calitp-data-infra, but given that it has its own image, I am choosing to not upgrade its dependency right now. Same with gtfs-aggregator-scraper.
- The gtfs-schedule-validator also depends on it; the dependency version was bumped, but I did not bump the version of the package itself, as it should have no impact, and should only come into play if the image needs to be rebuilt. Same with gtfs-rt-parser-v2.
This reverts commit e683630.

Images builds are failing, and are technically not required for doing the Cloud Composer update, so we can address the dependency update in a issue.
Copy link
Contributor

@vevetron vevetron left a comment

Choose a reason for hiding this comment

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

Short and sweet.

@mjumbewu mjumbewu merged commit 58fb581 into main Feb 4, 2025
1 check passed
@mjumbewu mjumbewu deleted the calitp-data-infra-dependency-20250130 branch February 4, 2025 19:55
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.

2 participants