Add a new hook for storing Tempest timing data #3330

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

kstrenkova wants to merge 1 commit into openstack-k8s-operators:main from kstrenkova:add-new-timing-data-url

+88 −0

Contributor

kstrenkova commented Sep 24, 2025 •

edited

Loading

This new hook is enabling timing data to be stored into a repository. Later the timing data will be used to reduce test execution time.

openshift-ci bot added the do-not-merge/work-in-progress label

Contributor

openshift-ci bot commented Sep 24, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Contributor

openshift-ci bot commented Sep 24, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

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

Needs approval from an approver in each of these files:

OWNERS

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

softwarefactory-project-zuul bot commented Sep 24, 2025

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/1f9b86d5f04a42658a303da281d6d535

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 49m 12s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 39s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 10s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 00s
❌ cifmw-pod-pre-commit FAILURE in 8m 13s
✔️ build-push-container-cifmw-client SUCCESS in 20m 53s

kstrenkova force-pushed the add-new-timing-data-url branch 3 times, most recently from 398bd18 to 74c8900 Compare

September 25, 2025 11:20

softwarefactory-project-zuul bot commented Sep 25, 2025

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/9287b80c72e74d6e92b6d72d5faa469e

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 00m 58s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 46s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 43m 38s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 04s
❌ cifmw-pod-pre-commit FAILURE in 8m 25s
✔️ build-push-container-cifmw-client SUCCESS in 21m 02s

kstrenkova force-pushed the add-new-timing-data-url branch from 74c8900 to 0b0dd18 Compare

September 25, 2025 14:26

softwarefactory-project-zuul bot commented Sep 25, 2025

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/522658dda8824dce9759668ac0ea4eb6

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 26m 32s
❌ podified-multinode-edpm-deployment-crc FAILURE in 33m 15s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 39m 51s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 13s
❌ cifmw-pod-pre-commit FAILURE in 8m 41s
✔️ build-push-container-cifmw-client SUCCESS in 21m 11s

kstrenkova force-pushed the add-new-timing-data-url branch from 0b0dd18 to 535a08d Compare

September 26, 2025 11:13

softwarefactory-project-zuul bot commented Sep 26, 2025

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/68aee9b2e4eb4c9baf2aa57633c8b658

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 53m 49s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 35m 10s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 18m 56s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 25s
❌ cifmw-pod-pre-commit FAILURE in 7m 32s
✔️ build-push-container-cifmw-client SUCCESS in 20m 53s

kstrenkova force-pushed the add-new-timing-data-url branch from 535a08d to 50523c7 Compare

October 1, 2025 10:16

softwarefactory-project-zuul bot commented Oct 1, 2025

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/9ebac03417e8453ab0416539526cba1e

✔️ openstack-k8s-operators-content-provider SUCCESS in 52m 51s
❌ podified-multinode-edpm-deployment-crc FAILURE in 33m 34s
❌ cifmw-crc-podified-edpm-baremetal FAILURE in 38m 07s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 56s
❌ cifmw-pod-pre-commit FAILURE in 8m 06s
✔️ build-push-container-cifmw-client SUCCESS in 20m 30s

github-actions bot commented Oct 17, 2025

This PR is stale because it has been for over 15 days with no activity.
Remove stale label or comment or this will be closed in 7 days.

github-actions bot added the Stale label

kstrenkova marked this pull request as ready for review

October 21, 2025 09:09

kstrenkova requested a review from a team as a code owner

October 21, 2025 09:09

openshift-ci bot removed the do-not-merge/work-in-progress label

kstrenkova added do-not-merge/hold and removed Stale labels

kstrenkova force-pushed the add-new-timing-data-url branch from 50523c7 to 3e559b3 Compare

October 22, 2025 15:50

github-actions bot added the Ready For Review label

kstrenkova force-pushed the add-new-timing-data-url branch from 3e559b3 to c3e37fe Compare

October 29, 2025 14:11

github-actions bot removed the Ready For Review label

kstrenkova requested review from danpawlik and sdatko

October 29, 2025 14:13

Contributor Author

kstrenkova commented Oct 29, 2025

Hi @danpawlik, I would love to hear your opinion about this PR. It is working as intended, but maybe the code is not as clean as it could be :D.

github-actions bot added the Ready For Review label

sdatko reviewed

View reviewed changes

hooks/playbooks/tempest_store_timing_data.yaml Outdated Show resolved Hide resolved

sdatko reviewed

View reviewed changes

hooks/playbooks/tempest_store_timing_data.yaml Outdated Show resolved Hide resolved


          Add a new hook for storing Tempest timing data

c1c28ed

This new hook is enabling timing data to be stored into an
internal repository. Later the timing data will be used to reduce
test execution time.

kstrenkova force-pushed the add-new-timing-data-url branch from c3e37fe to c1c28ed Compare

October 30, 2025 15:00

github-actions bot added Ready For Review and removed Ready For Review labels

danpawlik reviewed

View reviewed changes

hooks/playbooks/tempest_store_timing_data.yaml

+                        register: tempest_dirs
+                      - name: Store timing data for every tempest-test directory
+                        ansible.builtin.include_tasks: ../tasks/tempest_store_timing_data_each.yaml

Contributor

danpawlik Nov 6, 2025 •

edited

Loading

normally I like it, nice work, but...
I don't see "tasks" in hooks. Maybe would be better to... create role and add it there then call the role? WDYT
NIT: you need to rebase to get roles symlink available in hooks/playbooks.

Contributor Author

kstrenkova Nov 11, 2025

Hi @danpawlik, the only reason I added it under task folder is that playbook folder expects playbooks :D.
About your suggestion, I did the rebase and have the symlink, but I find it hard to imagine creating a role for one task 🤔

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

Labels

do-not-merge/hold Ready For Review