Skip to content

Rename workflow number variables for clarity#420

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:rename-workflow-step-vars
Mar 3, 2026
Merged

Rename workflow number variables for clarity#420
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:rename-workflow-step-vars

Conversation

@kstrenkova
Copy link
Copy Markdown
Contributor

While working on a different refactor I noticed that the naming of workflow number variables is a bit misleading. The workflowStepNum variable is currently used for parallel execution, but it is not really clear from the code as it blends in with the workflowStep vairable. This change should better clarify the actual difference between these workflow number variables.

While working on a different refactor I noticed that the naming of
workflow number variables is a bit misleading. The workflowStepNum
variable is currently used for parallel execution, but it is not
really clear from the code as it blends in with the workflowStep
vairable. This change should better clarify the actual difference
between these workflow number variables.
@openshift-ci openshift-ci Bot requested review from evallesp and sdatko February 25, 2026 11:45
@softwarefactory-project-zuul
Copy link
Copy Markdown

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/16332c8fb424479fbaf63689c4a4db27

✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 07s
podified-multinode-edpm-deployment-crc-test-operator RETRY_LIMIT in 11m 43s

@danpawlik
Copy link
Copy Markdown

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

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

openstack-k8s-operators-content-provider FAILURE in 14m 58s
⚠️ podified-multinode-edpm-deployment-crc-test-operator SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

@kstrenkova
Copy link
Copy Markdown
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

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/cecb670a931f4d468ec7a6d57be6390a

✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 27s
podified-multinode-edpm-deployment-crc-test-operator RETRY_LIMIT in 11m 33s

@kstrenkova
Copy link
Copy Markdown
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

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/e1c4c49033864e048e1930b896d7b52f

✔️ openstack-k8s-operators-content-provider SUCCESS in 27m 42s
podified-multinode-edpm-deployment-crc-test-operator RETRY_LIMIT in 11m 40s

@kstrenkova
Copy link
Copy Markdown
Contributor Author

recheck

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Mar 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrianfusco, belolipa, kstrenkova

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot Bot merged commit b97156c into openstack-k8s-operators:main Mar 3, 2026
7 checks passed
@kstrenkova kstrenkova deleted the rename-workflow-step-vars branch April 28, 2026 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants