globus / globus-compute

Globus Compute: High Performance Function Serving for Science
https://www.globus.org/compute
Apache License 2.0
146 stars 47 forks source link

Change version mismatch message, add logic to ignore Python micro # if desired #1601

Closed LeiGlobus closed 2 months ago

LeiGlobus commented 3 months ago

Our typical warning message:

Warning - dependency versions are mismatched between local SDK and remote worker on endpoint f93919c6-ceb7-4833-1dfe-ef3edbbea706: local SDK uses Python 3.11.9/Dill 0.3.6 but worker used 3.11.8/0.3.6 (worker SDK version: 2.22.0; worker OS: Linux-5.10.217-205.860.amzn2.x86_64-x86_64-with-glibc2.36)

has come up a few times when users were unsure about its actual impact. It is also displayed with minor PY version mismatches when that may not impact serialization.

We should make the message more clear, and possibly avoid minor version mismatches triggering the warning.

For now, until more team members give input, this PR leaves the micro-version checking logic alone, and only updates the warning message itself to be more helpful.