Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
6316d70
feat: provide Python run-time version support
vchudnov-g Jul 29, 2025
a136ad7
feat: apply Python version suport warnings to api_core
vchudnov-g Jul 29, 2025
45cd647
feat: add deprecation check for the protobuf package
vchudnov-g Jul 30, 2025
25225fa
format files
vchudnov-g Jul 30, 2025
1d567c0
fix lint warning
vchudnov-g Jul 30, 2025
f1dbbb4
add docstring to `warn_deprecation_for_versions_less_than`
vchudnov-g Jul 30, 2025
e3fd56f
Add/fix docstrings
vchudnov-g Jul 30, 2025
8b1dfb1
fix typo
vchudnov-g Jul 31, 2025
4b9208e
add test for _python_package_support.py
vchudnov-g Jul 31, 2025
5cf8652
add constants for various buffer periods
vchudnov-g Jul 31, 2025
7d8b1c7
Update warning code to only require import package names
vchudnov-g Jul 31, 2025
cda8e27
Fix messaegs and test
vchudnov-g Jul 31, 2025
db92fac
Add TODO: provide the functionality in previous versions of api_core
vchudnov-g Jul 31, 2025
118a7c8
Fix mypy failures
vchudnov-g Aug 12, 2025
f1c46aa
Try to remove a round-off error causing a test mock failure
vchudnov-g Aug 12, 2025
474ec0d
Remove more potential test failures/warnings
vchudnov-g Aug 12, 2025
2083b49
Format
vchudnov-g Aug 12, 2025
4ea124e
Try making the specified_timeout a float
vchudnov-g Aug 12, 2025
c5949c4
fix: tweak message parameter names
vchudnov-g Sep 3, 2025
6fc1473
fix: add PYTHON_VERSION_STATUS_UNSPECIFIED enum value
vchudnov-g Sep 3, 2025
8829e20
docs: tweak TODOs
vchudnov-g Sep 3, 2025
bdb6260
fix test to match code changes
vchudnov-g Sep 3, 2025
7896664
fix: restore asyncio designator for async tests
vchudnov-g Sep 4, 2025
54a5611
Remove some workarounds trying to fix presubmit errors (since fixed)
vchudnov-g Sep 5, 2025
2e2990b
Additional test tweaks to prevent non-significant failures
vchudnov-g Sep 5, 2025
35a2074
chore: fix test that was waiting before initiating operation
vchudnov-g Sep 5, 2025
cd64832
fix: skip coverage checks for code specific to Pyton 3.7
vchudnov-g Sep 5, 2025
e36414a
chore: try to address coverage failures
vchudnov-g Sep 8, 2025
b6245ca
chore: try to address more coverage failures
vchudnov-g Sep 8, 2025
3a897ba
chore: fix comments and exported function
vchudnov-g Sep 9, 2025
814ea63
fix lint
vchudnov-g Sep 9, 2025
8dee04b
chore: fix unit test
vchudnov-g Sep 9, 2025
95f4777
chore: lint
vchudnov-g Sep 9, 2025
a34ec15
Use warnings module instead of logging module
vchudnov-g Sep 10, 2025
a04e2e6
fix: print the current package versions correctly
vchudnov-g Sep 15, 2025
8b3f337
wip: before gemini doc fixes
vchudnov-g Sep 15, 2025
29896cf
fix: return tuple of version tuple and version string, fix tests
vchudnov-g Sep 16, 2025
d421f98
lint
vchudnov-g Sep 22, 2025
ac76f7a
feat: add grace period for 3.9; tweak warning
vchudnov-g Oct 1, 2025
e093a96
lint
vchudnov-g Oct 1, 2025
187f848
allow providing a recommended version
vchudnov-g Oct 1, 2025
b575a16
refactor: used namedtuple
vchudnov-g Oct 13, 2025
ecb7211
refactor: s/"--"/UNKNOWN_VERSION_STRING/ in code, not test
vchudnov-g Oct 13, 2025
3c587cf
refactor s/next_supported_version/minimum_fully_supported_version/
vchudnov-g Oct 13, 2025
fbe0f0b
refactor: s/dependent_/consumer_/
vchudnov-g Oct 13, 2025
28b0b32
refactor: add _PACKAGE_DEPENDENCY_WARNINGS
vchudnov-g Oct 13, 2025
21c9aec
refactor: reorder and comment on _init__ statements for clarity
vchudnov-g Oct 13, 2025
e24c13d
fix: exapnd VersionInfo with version field; clarify fake dates
vchudnov-g Oct 13, 2025
dbc3a26
doc: document PythonVersionSupportStatus
vchudnov-g Oct 13, 2025
d472bae
refactor: make fake past and future versions into constants
vchudnov-g Oct 14, 2025
0550f83
refactor: reword default string for min_python
vchudnov-g Oct 14, 2025
491b695
fix: fix populating PYTHON_VERSION_INFO
vchudnov-g Oct 14, 2025
6338389
fix: type hints
vchudnov-g Oct 14, 2025
05bcf6f
feat: allow check_dependency_versions() to take any number of Depende…
vchudnov-g Oct 17, 2025
fc224b7
add unit test to satisfy coverage requirement
vchudnov-g Oct 17, 2025
a6c40ce
fix lint
vchudnov-g Oct 17, 2025
d0414b2
fix lint
vchudnov-g Oct 17, 2025
835df53
test: add test for _flatten_message
vchudnov-g Oct 23, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
allow providing a recommended version
Sometimes the version we recommend is newer than the next
non-deprecated version, so we want to inform users eagerly to reduce
their toil
  • Loading branch information
vchudnov-g committed Oct 23, 2025
commit 187f848b7c02deaa4afb7a1c5b54f7dbd8f877de
21 changes: 15 additions & 6 deletions google/api_core/_python_package_support.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ def warn_deprecation_for_versions_less_than(
dependent_import_package: str,
dependency_import_package: str,
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: it might be nice if these variables were more distinct. Maybe target_package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

next_supported_version: str,
recommended_version: Optional[str] = None,
message_template: Optional[str] = None,
):
"""Issue any needed deprecation warnings for `dependency_import_package`.
Expand All @@ -84,6 +85,8 @@ def warn_deprecation_for_versions_less_than(
dependency_import_package: The import name of the dependency to check.
next_supported_version: The dependency_import_package version number
below which a deprecation warning will be logged.
Copy link
Contributor

Choose a reason for hiding this comment

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

could this be minimum_supported_version? Or am I misunderstanding it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

recommended_version: If provided, the recommended next version, which
could be higher than `next_supported_version`.
message_template: A custom default message template to replace
the default. This `message_template` is treated as an
f-string, where the following variables are defined:
Expand Down Expand Up @@ -118,17 +121,22 @@ def warn_deprecation_for_versions_less_than(
dependent_package,
dependent_distribution_package,
) = _get_distribution_and_import_packages(dependent_import_package)
Copy link
Contributor

@daniel-sanche daniel-sanche Oct 10, 2025

Choose a reason for hiding this comment

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

Can these variables be made less similar? (dependent_x vs dependency_x)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They're the correct terms, but they are very similar. I changed the less common term (IMO), dependent, to consumer.


recommendation = (
" (we recommend {recommended_version})" if recommended_version else ""
)
message_template = message_template or _flatten_message(
"""
DEPRECATION: Package {dependent_package} depends on
{dependency_package}, currently installed at version
{version_used_string}. Future updates to
{dependent_package} will require {dependency_package} at
version {next_supported_version} or higher. Please ensure
that either (a) your Python environment doesn't pin the
version of {dependency_package}, so that updates to
{dependent_package} can require the higher version, or
(b) you manually update your Python environment to use at
version {next_supported_version} or
higher{recommendation}. Please ensure that either (a) your
Python environment doesn't pin the version of
{dependency_package}, so that updates to
{dependent_package} can require the higher version, or (b)
you manually update your Python environment to use at
least version {next_supported_version} of
{dependency_package}.
"""
Expand All @@ -142,6 +150,7 @@ def warn_deprecation_for_versions_less_than(
dependency_package=dependency_package,
dependent_package=dependent_package,
next_supported_version=next_supported_version,
recommendation=recommendation,
version_used=version_used,
version_used_string=version_used_string,
),
Expand All @@ -162,5 +171,5 @@ def check_dependency_versions(dependent_import_package: str):

"""
warn_deprecation_for_versions_less_than(
dependent_import_package, "google.protobuf", "4.25.8"
dependent_import_package, "google.protobuf", "4.25.8", recommended_version="6.x"
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the idea that we will go back and manually add dependencies here in the future?

If so, this feels a little hidden. Can we read this from a config file, or declare it as a constant at the top of the file or something?

something like:

_DependencyConstraint = namedtuple("_DependencyConstraint", ["package_name", "min_version", "recommended_version"])

DEPENDENCIES = [
    _DependencyConstraint("google.protobuf", min_version="4.25.8", recommended_version="6.x")
]

def check_dependency_versions(dependent_import_package: str):
    for package_info in DEPENDENCIES:
        warn_deprecation_for_versions_less_than(
            dependent_import_package, 
            package_info.package_name, 
            package_info.min_version,
            recommended_version=package_info.recommended_version,
        )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think what we have is simple and clear, though it does feel hidden just by the fact it's at the bottom of the file.

OK, I implemented the table at the top of the file.

Copy link
Contributor

Choose a reason for hiding this comment

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

It kind of feels like dependencies should be given as an argument too? Can was assume all sdks will want to check against the same ones?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, as variadic parameters, with the local list as a default.