Skip to content

Fix task name truncation in ci_script plugin: use rstrip instead of rsplit#3641

Open
ciecierski wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
ciecierski:ci_script_improvment
Open

Fix task name truncation in ci_script plugin: use rstrip instead of rsplit#3641
ciecierski wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
ciecierski:ci_script_improvment

Conversation

@ciecierski
Copy link
Contributor

@ciecierski ciecierski commented Jan 29, 2026

Using rsplit("_", 1)[0] dropped the last segment when there was no trailing underscore (e.g. for short ansible task names like "Deploy ceph" became "deploy").

Use rstrip("_") so we only remove a trailing underscore at the end of string.

Signed-off-by: Mikołaj Ciecierski mikolaj.ciecierski@redhat.com

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign danpawlik 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

@ciecierski ciecierski force-pushed the ci_script_improvment branch from 678a4cc to 6a47f8e Compare January 29, 2026 19:46
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/6199d890a3d146a8bfc89d15b79823cd

openstack-k8s-operators-content-provider FAILURE in 6m 23s
⚠️ podified-multinode-edpm-deployment-crc SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ cifmw-crc-podified-edpm-baremetal-minor-update SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ noop SUCCESS in 0s
cifmw-pod-ansible-test FAILURE in 9m 05s
cifmw-pod-pre-commit FAILURE in 8m 18s

@ciecierski
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3a7eff0065194ae6aa9f147ed9d3482b

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 02m 53s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 18m 04s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 44m 30s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 48m 57s
✔️ noop SUCCESS in 0s
cifmw-pod-ansible-test FAILURE in 8m 16s
cifmw-pod-pre-commit FAILURE in 8m 16s

…split

Using rsplit("_", 1)[0] dropped the last segment when there was no
trailing underscore (e.g. for short ansible task names like "Deploy ceph"
became "deploy").
Use rstrip("_") so we only remove a trailing underscore at the end of string.

Signed-off-by: Mikołaj Ciecierski <mikolaj.ciecierski@redhat.com>
@ciecierski ciecierski force-pushed the ci_script_improvment branch from 6a47f8e to c9dfefe Compare February 3, 2026 09:50
@softwarefactory-project-zuul
Copy link

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/b9d3c04e5a5642c7948ced3d85aeedcc

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 39m 09s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 21m 25s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 32m 04s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 2h 01m 07s
✔️ noop SUCCESS in 0s
cifmw-pod-ansible-test FAILURE in 8m 47s
cifmw-pod-pre-commit FAILURE in 7m 59s

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.

1 participant