-
Notifications
You must be signed in to change notification settings - Fork 139
[radvd] Add radvd role for IPv6 RA management #3483
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
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: 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 |
e72772e to
ff5a80a
Compare
| become: true | ||
| ansible.builtin.package: | ||
| name: radvd | ||
| state: present |
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.
so if we handle in this file cleanup and install, so it for sure can handle also present/absent.
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.
But do we want to remove the RPM as part of cleanup? IMO it is enough to ensure the service is stopped and config removed.
I also think this got messy with configure/cleanup calling each other. Let me re-factor.
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 changed it so configure.yml does only conigure, and all cleanup is in cleanup.yml - I think it is better like that, keep it simple ...
I also added a handler for the service restart. I'll rebase #3484 once this merges.
Integration radvd role with libvirt_manager: - Added cifmw_libvirt_manager_radvd_networks parameter - Automatically configures radvd when networks are created - Documented in libvirt_manager README with usage examples Depends-On: openstack-k8s-operators#3483 Assisted-By: Claude Code/claude-4.5-sonnet Signed-off-by: Harald Jensås <[email protected]>
Integration radvd role with libvirt_manager: - Added cifmw_libvirt_manager_radvd_networks parameter - Automatically configures radvd when networks are created - Documented in libvirt_manager README with usage examples Depends-On: openstack-k8s-operators#3483 Assisted-By: Claude Code/claude-4.5-sonnet Signed-off-by: Harald Jensås <[email protected]>
|
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/0d5df123a5704f489a71b90f6f300026 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 44m 52s |
7a69afe to
baf207d
Compare
Creates a new Ansible role to manage radvd (Router Advertisement Daemon) configuration for IPv6 networks. Features: - Manages system radvd.service and /etc/radvd.conf - Configuration fragment assembly from /etc/cifmw-radvd.d/ - Support for SLAAC and DHCPv6 (M-flag, O-flag) - IPv6 prefix, route, and RDNSS advertisement - Dynamic network addition/removal via tasks_from - Molecule test coverage Assisted-By: Claude Code/claude-4.5-sonnet Signed-off-by: Harald Jensås <[email protected]>
Creates a new Ansible role to manage radvd (Router Advertisement Daemon) configuration for IPv6 networks.
Features:
JIRA: OSPRH-20084
Assisted-By: Claude Code/claude-4.5-sonnet