Skip to content

Fix dependency check missing clones with stale Y-stream fixVersion - #441

Merged
lbarcziova merged 1 commit into
packit:mainfrom
lbarcziova:fix-stale-ystream-dependency-check
May 5, 2026
Merged

lbarcziova merged 1 commit into
packit:mainfrom
lbarcziova:fix-stale-ystream-dependency-check

Conversation

@lbarcziova

Copy link
Copy Markdown
Member

The Z-stream clone dependency check (used for Y-stream and maintenance eligibility) built its relevant_z_streams set using only the .z form from config (e.g. rhel-9.6.z). Clones with a stale Y-stream fixVersion (rhel-9.6, not yet updated after GA) were silently skipped, potentially unblocking Y-stream/maintenance issues prematurely.

Use get_fix_version_variants() to include both forms in the set.

Assisted-by: Claude Opus 4.6

The Z-stream clone dependency check (used for Y-stream and maintenance
eligibility) built its relevant_z_streams set using only the .z form
from config (e.g. rhel-9.6.z). Clones with a stale Y-stream fixVersion
(rhel-9.6, not yet updated after GA) were silently skipped, potentially
unblocking Y-stream/maintenance issues prematurely.

Use get_fix_version_variants() to include both forms in the set.

Assisted-by: Claude Opus 4.6

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the Jira tool to correctly recognize Z-stream clones by including fix version variants. Specifically, it modifies the _check_zstream_clones_shipped function to use get_fix_version_variants, ensuring that versions like rhel-9.6 are matched alongside rhel-9.6.z. A corresponding unit test has been added to verify this behavior. I have no feedback to provide.

@lbarcziova
lbarcziova merged commit 14fd9fb into packit:main May 5, 2026
8 checks passed
@lbarcziova
lbarcziova deleted the fix-stale-ystream-dependency-check branch May 11, 2026 13:51
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