Skip to content

Bound local snapshot names for long registry refs#38

Merged
CMGS merged 1 commit into
cocoonstack:mainfrom
zili-simular:linux-cocoon
Jul 16, 2026
Merged

Bound local snapshot names for long registry refs#38
CMGS merged 1 commit into
cocoonstack:mainfrom
zili-simular:linux-cocoon

Conversation

@zili-simular

Copy link
Copy Markdown
Contributor

Summary

  • keep Cocoon local snapshot names within its 63-character limit
  • preserve a recognizable prefix and add a deterministic hash suffix
  • cover long Artifact Registry repository/tag combinations and collision separation

Verification

  • ok github.com/cocoonstack/vk-cocoon (cached)
    ok github.com/cocoonstack/vk-cocoon/guest (cached)
    ? github.com/cocoonstack/vk-cocoon/guest/rdp [no test files]
    ok github.com/cocoonstack/vk-cocoon/guest/sac (cached)
    ? github.com/cocoonstack/vk-cocoon/metrics [no test files]
    ok github.com/cocoonstack/vk-cocoon/network (cached)
    ok github.com/cocoonstack/vk-cocoon/probes (cached)
    ok github.com/cocoonstack/vk-cocoon/provider (cached)
    ok github.com/cocoonstack/vk-cocoon/provider/cocoon (cached)
    ? github.com/cocoonstack/vk-cocoon/snapshots [no test files]
    ? github.com/cocoonstack/vk-cocoon/version [no test files]
    ok github.com/cocoonstack/vk-cocoon/vm (cached)

This fixes snapshot imports that currently exit before reading the stream when a valid remote repository plus run-scoped tag exceeds Cocoon's local-name limit.

@CMGS
CMGS merged commit f61e610 into cocoonstack:main Jul 16, 2026
2 checks passed
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.

2 participants