-
Notifications
You must be signed in to change notification settings - Fork 68
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
Add edpm role to deploy Neutron DHCP agent #375
Add edpm role to deploy Neutron DHCP agent #375
Conversation
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
a0e53af
to
03cf708
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/78994494bc30456397948b74c3482006 ✔️ openstack-k8s-operators-content-provider SUCCESS in 42m 41s |
recheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please enable the molecule task in CI for the new role. I think it will reveal a number of issues in the PR.
03cf708
to
47a308e
Compare
This new roles deploys, configures and runs neutron-dhcp-agent on the EDPM node if needed. Jira: OSP-27538
47a308e
to
d1507ca
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/099fc0b830694184baba79bd65f45043 ✔️ openstack-k8s-operators-content-provider SUCCESS in 19m 49s |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/c3f01bb8f1724e59b6bb7368737255b5 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 10m 57s |
recheck |
@@ -0,0 +1,8 @@ | |||
# See the OWNERS docs at https://go.k8s.io/owners | |||
approvers: | |||
- slawqo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually think you could also add yourself to network-approvers list (separate PR). Just saying.
with_items: | ||
- {"src": "neutron.conf.j2", "dest": "01-neutron.conf"} | ||
- {"src": "rootwrap.conf.j2", "dest": "01-rootwrap.conf"} | ||
- {"src": "neutron-dhcp-agent.conf.j2", "dest": "01-neutron-dhcp-agent.conf"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(not exactly the problem with - just - your patch but I wonder if we should be explicit about the order of config files here - making sure that all CRD-derived configs are later in the loading order, by using a prefix of e.g. 02- vs. 01- used for the "EDP" config snippets. - but we can discuss it separately since I think this affects other roles too.)
src: "neutron_dhcp_agent.yaml.j2" | ||
dest: "/var/lib/openstack/config/containers/neutron_dhcp_agent.yaml" | ||
setype: "container_file_t" | ||
mode: 0644 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've heard modes should be put in quotes, "0644". Because apparently otherwise ansible may want to interpret them as numbers. (That's what I've heard. I haven't checked.)
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: karelyatin, slawqo 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 |
fc1fb80
into
openstack-k8s-operators:main
This new roles deploys, configures and runs neutron-dhcp-agent on the EDPM node if needed.
Jira: OSP-27538