-
Notifications
You must be signed in to change notification settings - Fork 87
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
DNM - Decouple uni07eta network stage from control plane stage #490
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: dsariel The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/97e777d9b09249fa946ae391513df630 ✔️ noop SUCCESS in 0s |
no need in this change according to the support matrix |
7e24cb7
to
b97d854
Compare
b97d854
to
74aca4c
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/aa3a71b99cc74b4da617451f42be2cc8 ✔️ noop SUCCESS in 0s |
b0ce80d
to
5523609
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/3785f8440d30470d87f34695274330f5 ✔️ noop SUCCESS in 0s |
d6fd1ed
to
f7f65d3
Compare
recheck |
a2e52dd
to
5d9b2d0
Compare
8c575e2
to
4bf74b4
Compare
4bf74b4
to
25d3057
Compare
@abays hi, where from network.tools.multus and network.network_name are coming from with relate to This line is a suspect that we have no
in logs/controller-0/ci-framework-data/artifacts/kustomize_deploy/network.yaml |
@dsariel I think the base source is this [1], but that is ingested by CIFMW and then goes through further refinement/processing before it is fed to the template you mentioned [2]. I am not familiar with the internal details of that CIFMW process though. [1] architecture/examples/dt/uni07eta/control-plane/nncp/values.yaml Lines 160 to 183 in 32d7703
[2] https://github.com/openstack-k8s-operators/ci-framework/blob/main/roles/ci_gen_kustomize_values/templates/uni07eta/network-values/values.yaml.j2#L87 |
No description provided.