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

Add predictable IPs to mdns and bind9 #246

Merged

Conversation

beagles
Copy link
Collaborator

@beagles beagles commented Nov 5, 2024

Adds the predictable IPs to the pod volumes as well as adding some hash tracking logic to the controller.

Note requires availability of openstack-k8s-operators/tcib#218

@openshift-ci openshift-ci bot requested review from johnsom and lewisdenny November 5, 2024 01:13
Copy link
Contributor

openshift-ci bot commented Nov 5, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: beagles

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-ci openshift-ci bot added the approved label Nov 5, 2024
@beagles beagles force-pushed the bind-predictable-ips branch from 93dca98 to af5b0b2 Compare November 7, 2024 15:52
@beagles beagles requested a review from omersch381 November 7, 2024 23:35
@beagles beagles changed the title Add the bind IPs to the bind9 backend mounts Add predictable IPs to mdns and bind9 Nov 7, 2024
config/default/manager_default_images.yaml Outdated Show resolved Hide resolved
pkg/designatemdns/daemonset.go Outdated Show resolved Hide resolved
templates/common/setipalias.py Outdated Show resolved Hide resolved
templates/common/setipalias.py Outdated Show resolved Hide resolved
templates/designate/bin/setipalias.sh Outdated Show resolved Hide resolved
@beagles
Copy link
Collaborator Author

beagles commented Dec 2, 2024

PR still needs updating for the changes how the mdns map has been defined. Standby.

@omersch381
Copy link
Contributor

pre-commit check file has been updated, let's try to retest

@omersch381
Copy link
Contributor

/retest

@beagles beagles force-pushed the bind-predictable-ips branch 5 times, most recently from fc2bbeb to c803892 Compare January 9, 2025 19:21
@omersch381
Copy link
Contributor

/test designate-operator-build-deploy-kuttl

@beagles beagles force-pushed the bind-predictable-ips branch 2 times, most recently from 6934243 to d11464a Compare January 10, 2025 15:53
This PR adds support for mounting generated predictable IPs and
employing them in the relevant pods.
@beagles beagles force-pushed the bind-predictable-ips branch from d11464a to 2babf83 Compare January 10, 2025 17:56
@omersch381
Copy link
Contributor

/lgtm

@openshift-merge-bot openshift-merge-bot bot merged commit c44a879 into openstack-k8s-operators:main Jan 13, 2025
5 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