chore: testing constraints target minimum supported versions by daniel-sanche · Pull Request #2590 · googleapis/gapic-generator-python · GitHub
Skip to content
This repository was archived by the owner on Mar 26, 2026. It is now read-only.

chore: testing constraints target minimum supported versions#2590

Merged
gkevinzheng merged 19 commits into
mainfrom
update_min_versions
Mar 25, 2026
Merged

chore: testing constraints target minimum supported versions#2590
gkevinzheng merged 19 commits into
mainfrom
update_min_versions

Conversation

@daniel-sanche

@daniel-sanche daniel-sanche commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

Fixes #2568 (comment), #2453

Currently, our constraints files are above the minimum-supported version. We need to be testing minimum versions, which means either bumping up the minimum, or lowering the constraint

For this fix:

  • added new constraints-3.9-async-rest.txt file, so the async_rest extra can have its own minimum versions
  • changes the minimum api_core version to 2.17.1, to incorporate this fix. Lower versions encountered a fragment test that seemed to be a real bug with Rest clients
  • while touching these constraints, I also added a grpcio constraint, to resolve #2453. We bumped the version to 1.44.0, to match google-cloud-iam

@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Mar 13, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request lowers the pinned versions for google-api-core and google-auth in the constraints file for Python 3.9, as part of an effort to test against minimum supported dependency versions. This is a good exploratory change. However, I noticed that the constraints files for Python 3.10, 3.11, and 3.12 currently use _default_constraints.j2, which results in testing against the latest compatible dependency versions rather than the minimums. To ensure consistent and comprehensive testing of the minimum requirements across all supported Python versions, I recommend extending this change to also pin the lower bounds in the constraints files for Python 3.10-3.12.

@product-auto-label product-auto-label Bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Mar 13, 2026
Comment thread noxfile.py Outdated
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Mar 14, 2026
@daniel-sanche daniel-sanche changed the title [DRAFT] chore: testing constraints target minimum supported versions chore: testing constraints target minimum supported versions Mar 18, 2026
@daniel-sanche daniel-sanche marked this pull request as ready for review March 18, 2026 00:03
@daniel-sanche daniel-sanche requested a review from a team as a code owner March 18, 2026 00:03
Comment thread gapic/templates/testing/constraints-3.9.txt.j2 Outdated
@daniel-sanche daniel-sanche assigned vchudnov-g and unassigned tswast Mar 20, 2026
@gkevinzheng gkevinzheng merged commit d2216aa into main Mar 25, 2026
113 checks passed
@gkevinzheng gkevinzheng deleted the update_min_versions branch March 25, 2026 14:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use google-auth2.14.1 in tests Fragment test fails with protobuf ==3.20.2

4 participants