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.

Sign up for GitHub

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

Jump to bottom

Update DNSRecord ownerID to take into account gw name and namespace #546

Closed
mikenairn opened this issue Apr 16, 2024 · 1 comment
Closed

Update DNSRecord ownerID to take into account gw name and namespace #546

mikenairn opened this issue Apr 16, 2024 · 1 comment
Assignees
@philbrookes
Labels
size/small

Comments

@mikenairn
Copy link
Member

What

Update the ownerID added to the spec of generated DNSRecords to include the gateway name and namespace in the calculation.

Why

The ownerID currently is cluster wide meaning that every DNSRecord created on that cluster will have the same ownerID. This prevents us from being able to distribute DNS across gateways in different namespaces on the same cluster. This will also be an issue on multi tenant clusters.
@mikenairn mikenairn changed the title Update DNSRecord ownerID to take into account gw name and namesapce Update DNSRecord ownerID to take into account gw name and namespace Apr 16, 2024
@mikenairn mikenairn mentioned this issue Apr 16, 2024
Closed
11 tasks
@mikenairn mikenairn moved this to Todo in Kuadrant Apr 16, 2024
@philbrookes philbrookes self-assigned this Apr 16, 2024
@maleck13
Copy link
Collaborator

@philbrookes is this done now?

@maleck13 maleck13 moved this from Todo to In Progress in Kuadrant Apr 17, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Kuadrant Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philbrookes philbrookes

Labels
size/small
Projects
Kuadrant
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@maleck13 @mikenairn @philbrookes