Skip to content

[18.0-fr5] Fix AnsibleEE job name collisions for long names#1855

Open
openshift-cherrypick-robot wants to merge 1 commit intoopenstack-k8s-operators:18.0-fr5from
openshift-cherrypick-robot:cherry-pick-1819-to-18.0-fr5
Open

[18.0-fr5] Fix AnsibleEE job name collisions for long names#1855
openshift-cherrypick-robot wants to merge 1 commit intoopenstack-k8s-operators:18.0-fr5from
openshift-cherrypick-robot:cherry-pick-1819-to-18.0-fr5

Conversation

@openshift-cherrypick-robot
Copy link

@openshift-cherrypick-robot openshift-cherrypick-robot commented Mar 16, 2026 •
edited by karelyatin
Loading

This is an automated cherry-pick of #1819

/assign karelyatin
Related-Issue: #OSPRH-26041

@holser
Fix AnsibleEE job name collisions for long names
3a07c08 
Previously, when the combined service + deployment + nodeset name
exceeded 63 characters (DNS1123 max), simple truncation could cause
different jobs to end up with identical names, leading to collisions.

Replace the arbitrary -10 prefix truncation with a hash-based approach:
- Build the full execution name without premature truncation
- If the name exceeds 63 characters, truncate to 54 characters and
  append an 8-character SHA256 hash suffix
- This guarantees unique names even when truncation is required

Closes: OSPRH-26041

Signed-off-by: Sergii Golovatiuk <sgolovat@redhat.com>
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Mar 16, 2026
Merged
@openshift-ci openshift-ci bot requested review from abays and stuggi March 16, 2026 07:28
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openshift-cherrypick-robot
Once this PR has been reviewed and has the lgtm label, please assign dprince for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions
Copy link

github-actions bot commented Mar 16, 2026

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 317791 bytes (310KB)
Base branch size 317791 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@stuggi
Copy link
Contributor

stuggi commented Mar 16, 2026

/retest

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2026

@openshift-cherrypick-robot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/openstack-operator-build-deploy-kuttl-4-18 3a07c08 link true /test openstack-operator-build-deploy-kuttl-4-18

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abays abays Awaiting requested review from abays

@stuggi stuggi Awaiting requested review from stuggi

At least 1 approving review is required to merge this pull request.

Assignees

@karelyatin karelyatin

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@openshift-cherrypick-robot @stuggi @karelyatin @holser