Skip to content
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.

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

[uni01alpha] Add support for testing DT #1740

Merged

Conversation

psathyan
Copy link
Contributor

@psathyan psathyan commented May 21, 2024

This change adds networker nodes to solve the below error observed during gate checking.

error: accumulating components: accumulateDirectory: "recursed accumulation of path '/home/zuul/src/github.com/openstack-k8s-operators/architecture/dt/uni01alpha/networker': accumulating components: accumulateDirectory: "recursed accumulation of path '/home/zuul/src/github.com/openstack-k8s-operators/architecture/lib/dataplane': accumulating components: accumulateDirectory: \"recursed accumulation of path '/home/zuul/src/github.com/openstack-k8s-operators/architecture/lib/dataplane/nodeset': fieldPath data.nodeset.nodes is missing for replacement source ConfigMap.[noVer].[noGrp]/edpm-nodeset-values.[noNs]\"""

As a pull request owner and reviewers, I checked that:

  • Appropriate testing is done and actually running

Logs
https://review.rdoproject.org/zuul/build/6746ef28e48f4bc5a27f366a1df40204/console

This change adds networker nodes to solve the below error
observed during gate checking.

error: accumulating components: accumulateDirectory: "recursed accumulation of path
'/home/zuul/src/github.com/openstack-k8s-operators/architecture/dt/uni01alpha/networker':
accumulating components: accumulateDirectory: \"recursed accumulation of path
'/home/zuul/src/github.com/openstack-k8s-operators/architecture/lib/dataplane':
accumulating components: accumulateDirectory: \\\"recursed accumulation of path
'/home/zuul/src/github.com/openstack-k8s-operators/architecture/lib/dataplane/nodeset':
fieldPath `data.nodeset.nodes` is missing for replacement source
ConfigMap.[noVer].[noGrp]/edpm-nodeset-values.[noNs]\\\"\""

Signed-off-by: Pragadeeswaran Sathyanarayanan <psathyan@redhat.com>
Copy link
Contributor

openshift-ci bot commented May 21, 2024

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

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://review.rdoproject.org/zuul/buildset/5f502b3747944b88b955df8e4dcc4859

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 09m 30s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 20m 12s
cifmw-crc-podified-edpm-baremetal FAILURE in 36m 14s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 8m 27s
✔️ cifmw-pod-pre-commit SUCCESS in 7m 46s

@psathyan psathyan marked this pull request as ready for review May 22, 2024 03:40
@openshift-ci openshift-ci bot requested review from bshewale and frenzyfriday May 22, 2024 03:40
@psathyan
Copy link
Contributor Author

recheck

@cjeanner
Copy link
Contributor

/approve
/lgtm

Could get a full green run against https://review.rdoproject.org/zuul/buildset/0fb11338ef734a22bcda5a07f87aa090 (via openstack-k8s-operators/architecture#232 PR)

Since this file isn't affecting anything but those jobs: let's merge it. Thank you @psathyan !

Copy link
Contributor

openshift-ci bot commented May 22, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cjeanner

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

The pull request process is described here

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 7981c98 into openstack-k8s-operators:main May 22, 2024
7 of 8 checks passed
@psathyan psathyan deleted the uni01AlphaCI branch May 22, 2024 08:32
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