Skip to content

Fix Tobiko workflow ConfigMaps to use step-specific names#422

Merged
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
kstrenkova:refactor-tobiko-config-maps
Mar 4, 2026
Merged

Fix Tobiko workflow ConfigMaps to use step-specific names#422
openshift-merge-bot[bot] merged 1 commit intoopenstack-k8s-operators:mainfrom
kstrenkova:refactor-tobiko-config-maps

Conversation

@kstrenkova
Copy link
Copy Markdown
Contributor

Tobiko was reusing the same ConfigMap names across all workflow steps, causing later steps to overwrite earlier configs. Now it uses workflow-specific names to ensure each step preserves its own configuration.

@openshift-ci openshift-ci bot requested review from dprince and sdatko March 3, 2026 15:17
@openshift-ci openshift-ci bot added the approved label Mar 3, 2026
@kstrenkova
Copy link
Copy Markdown
Contributor Author

@eduolivares Hi! Could you please take a look and check if it makes sense to you? If it is not needed I can change it to simple refactor :D

@kstrenkova kstrenkova requested a review from eduolivares March 3, 2026 15:50
Tobiko was reusing the same ConfigMap names across all workflow
steps, causing later steps to overwrite earlier configs. Now it
uses workflow-specific names to ensure each step preserves its
own configuration.
@kstrenkova kstrenkova force-pushed the refactor-tobiko-config-maps branch from 8136d75 to 0512a9c Compare March 4, 2026 10:14
@openshift-ci
Copy link
Copy Markdown

openshift-ci bot commented Mar 4, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: eduolivares, 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 9117f35 into openstack-k8s-operators:main Mar 4, 2026
7 checks passed
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.

2 participants