Skip to content

semver of container with latest tag can now sometimes extracted#653

Open
jlssmt wants to merge 1 commit intogetwud:mainfrom
jlssmt:feature/latest-semver
Open

semver of container with latest tag can now sometimes extracted#653
jlssmt wants to merge 1 commit intogetwud:mainfrom
jlssmt:feature/latest-semver

Conversation

@jlssmt
Copy link
Contributor

@jlssmt jlssmt commented Mar 30, 2025

i started the work to get the semver of running containers when tag latest is used

use f.e.
wud.inspect.tag.path=Config/Labels/org.opencontainers.image.version
for most containers

#647

@jlssmt jlssmt force-pushed the feature/latest-semver branch 14 times, most recently from c7cc119 to f88dbf4 Compare March 31, 2025 15:51
@jlssmt jlssmt marked this pull request as ready for review March 31, 2025 15:52
@jlssmt
Copy link
Contributor Author

jlssmt commented Mar 31, 2025

functionality should work now.
documentation will be added after first review

open question: watcher env to try to get local semver from org.opencontainers.image.version by default?
-> leads to problems in some containers.
f.e. ollama misuses the label and writes the ubuntu version inside (20.04)

docker run -d --rm --name ollama ollama/ollama
docker inspect ollama | grep org.opencontainers.image.version

@jlssmt jlssmt force-pushed the feature/latest-semver branch from f88dbf4 to 6ce3eae Compare March 31, 2025 16:06
@PrivatePuffin
Copy link
Contributor

org.opencontainers.image.version

Important to note:
This is an open standard and does not necirarily have any reflection to "tags" or versioning employed in tags.

Its important to note, because it can lead to false positives.

@PrivatePuffin
Copy link
Contributor

I also see a number of changes that are not fundemental to this PR, might be worth it to cut it up in logical smaller PRs

@jlssmt
Copy link
Contributor Author

jlssmt commented Nov 30, 2025

org.opencontainers.image.version

Important to note: This is an open standard and does not necirarily have any reflection to "tags" or versioning employed in tags.

Its important to note, because it can lead to false positives.

For me this was the only way to receive the tag version from latest.

@jlssmt
Copy link
Contributor Author

jlssmt commented Nov 30, 2025

I also see a number of changes that are not fundemental to this PR, might be worth it to cut it up in logical smaller PRs

I left WUD some time ago to renovate.
Please feel free to make adjustments or close the PR.

I never received any feedback here, and JavaScript without types is really a mess for contributors.

@glebaccon
Copy link

Sounds like a great idea to me. I hope the merge will happen.

@s-b-e-n-s-o-n
Copy link

This has been integrated into WUD-CE as of 9.0.0-ce. Thanks for the contribution!

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.

4 participants