WIP: dns: clean DNS records on clusteringress deletion#114
WIP: dns: clean DNS records on clusteringress deletion#114ironcladlou wants to merge 1 commit intoopenshift:masterfrom
Conversation
When a clusteringress is deleted, ensure that any existing DNS records created for a load balancer service are also deleted.
|
Have done manual testing but need to think about how to do an e2e... |
|
Problem here is that we assume that if the ALIAS name and target match then we own the record, but I'm not entirely sure how we can establish a stronger correlation without reinventing external-dns... |
|
/retest |
1 similar comment
|
/retest |
|
@ironcladlou: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
openshift-ci-robot
added
the
needs-rebase
|
@ironcladlou: PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
Depends on openshift/api#202 |
ironcladlou
changed the title
openshift-ci-robot
added
do-not-merge/work-in-progress
|
Replaced by #121 /close |
|
@ironcladlou: Closed this PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
3 participants
When a clusteringress is deleted, ensure that any existing DNS records created
for a load balancer service are also deleted.