Open abhinavsingh opened 1 year ago
you could request the full nodeid instead of a abbreviation in a custom command
ts not lear to me whats the issue
i suspect there may be inonsistenc in the way setuptools_scm abbreviates (i do recall we have a fewl ....[:7] there
We use the following to automatically write a version file upon
pip install -e .
which then gets shipped within the container.During Python repo's build process, we collect static and push to a cloud storage. Path for upload is generated using
setuptools_scm.get_version()
by the build process. During runtime, server is able to appropriately point to storage based upon version written in_scm_version.py
. This has worked for 99% of the scenarios for us till now, but we are starting to experience inconsistencies between the two results. Below is an example:_scm_version.py
setuptools_scm.get_version()
Notice 7 vs 8 character hash, which leads to service unable to load necessary static. Per https://git-scm.com/docs/git-describe#_examples I understand that "The length of the abbreviation scales as the repository grows, using the approximate number of objects in the repository and a bit of math around the birthday paradox, and defaults to a minimum of 7", but why this inconsistency within
setuptools_scm
.As a workaround:
+g....
from our static path annotations. It is just good to have a commit hash directly in the url for debugging purposes.