Skip to content

Add molecule tests for cifwm_helpers#3405

Merged
evallesp merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-molecule-helper
Oct 16, 2025
Merged

Add molecule tests for cifwm_helpers#3405
evallesp merged 1 commit intoopenstack-k8s-operators:mainfrom
danpawlik:add-molecule-helper

Conversation

@danpawlik
Copy link
Copy Markdown
Contributor

@danpawlik danpawlik commented Oct 13, 2025

The cifmw_helpers role does not have molecule job, would be good to verify if it always returns correct result.

Depends-On: #3404

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 13, 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

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Oct 13, 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.

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

@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/d2b8d32f7dcd4ddca85158f853d873df

✔️ cifmw-pod-zuul-files SUCCESS in 4m 33s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 01s
cifmw-pod-pre-commit FAILURE in 7m 59s
✔️ build-push-container-cifmw-client SUCCESS in 19m 44s
cifmw-molecule-cifmw_helpers FAILURE in 3m 27s

@danpawlik danpawlik force-pushed the add-molecule-helper branch 2 times, most recently from 07ddd83 to 92a461e Compare October 14, 2025 15:43
@softwarefactory-project-zuul
Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change #3404 is needed.

@danpawlik danpawlik force-pushed the add-molecule-helper branch 2 times, most recently from d13447d to 0d63ab7 Compare October 14, 2025 16:03
@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/7422c9bcedb845ad9164e4b61ac907f8

✔️ cifmw-pod-zuul-files SUCCESS in 4m 53s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 28s
cifmw-pod-pre-commit FAILURE in 7m 51s
✔️ build-push-container-cifmw-client SUCCESS in 20m 31s
✔️ cifmw-molecule-cifmw_helpers SUCCESS in 4m 54s

@danpawlik danpawlik force-pushed the add-molecule-helper branch 3 times, most recently from 25f431e to 037b565 Compare October 14, 2025 16:58
@danpawlik danpawlik marked this pull request as ready for review October 14, 2025 16:58
@danpawlik danpawlik requested a review from a team as a code owner October 14, 2025 16:58
The cifmw_helpers role does not have molecule job, would be good
to verify if it always returns correct result.

Depends-On: openstack-k8s-operators#3404

Signed-off-by: Daniel Pawlik <dpawlik@redhat.com>
@danpawlik danpawlik force-pushed the add-molecule-helper branch from 037b565 to d2deb81 Compare October 14, 2025 17:10
@danpawlik danpawlik changed the title Add molecule tests Add molecule tests for cifwm_helper Oct 14, 2025
@danpawlik danpawlik changed the title Add molecule tests for cifwm_helper Add molecule tests for cifwm_helpers Oct 14, 2025
Copy link
Copy Markdown
Contributor

@evallesp evallesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@evallesp evallesp merged commit 3e31ef9 into openstack-k8s-operators:main Oct 16, 2025
6 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.

3 participants