Skip to content

ensure fake OS phase 2 images contain the version#48

Merged
iliana merged 1 commit into
mainfrom
iliana/fake-os-image-oops
Jul 8, 2026
Merged

ensure fake OS phase 2 images contain the version#48
iliana merged 1 commit into
mainfrom
iliana/fake-os-image-oops

Conversation

@iliana

@iliana iliana commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

One of the late changes to generating fake artifacts in the rewrite PR broke the rule that images of different versions must have different contents. Tests on my Nexus integration branch caught this mistake. This adds a test to ensure the rule holds between two fake repos of version 1.0.0 and 2.0.0, and fixes fake OS phase 2 image generation.

@iliana
iliana requested a review from sunshowers July 8, 2026 03:28
@iliana
iliana force-pushed the iliana/fake-os-image-oops branch from 9318024 to d88862b Compare July 8, 2026 04:05
@iliana
iliana force-pushed the iliana/fake-os-image-oops branch from d88862b to bdb71b7 Compare July 8, 2026 18:08

@sunshowers sunshowers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice catch!

@iliana
iliana merged commit de59b95 into main Jul 8, 2026
5 checks passed
@iliana
iliana deleted the iliana/fake-os-image-oops branch July 8, 2026 20:30
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