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

DNSPolicy: Add routing strategy type conflict test #606

Closed
mikenairn opened this issue May 2, 2024 · 0 comments · Fixed by #699
Closed

DNSPolicy: Add routing strategy type conflict test #606

mikenairn opened this issue May 2, 2024 · 0 comments · Fixed by #699
Assignees
Labels
area/tests Changes to tests only size/small
Milestone

Comments

@mikenairn
Copy link
Member

What

Add a test that checks the status of a DNSPolicy that has a conflicting strategy to an existing DNSPolicy with the same dnsName.

Steps:

  1. Create Gateway 1 with listener foo -> foo.example.com
  2. Create Gateway 2 with listener foo -> foo.example.com
  3. Update status of gateway 1 to show attached routes for listener foo and address (127.0.0.1)
  4. Update status of gateway 2 to show attached routes for listener foo and address (127.0.0.1)
  5. Create DNSPolicy 1 targeting gateway 1 with a simple routing strategy
  6. Verify DNSPolicy 1 becomes ready and the expected DNSRecord is created and ready
  7. Create DNSPolicy 2 targeting gateway 2 with a loadbalanced routing strategy
  8. Verify DNSPolicy 2 does not become ready and the expected DNSRecord is created but not ready and that both have the expected error:
record type conflict, cannot update endpoint 'foo.example.com' with record type 'CNAME' when endpoint already exists with record type 'A'

Note: Since this test will run in the kuadrant operator using a dns operator with default configurations for default and validation re-queue times it might not be possible to check much else without it taking too long.

Depends on #601

@mikenairn mikenairn added the area/tests Changes to tests only label May 2, 2024
@philbrookes philbrookes moved this to Todo in Kuadrant May 9, 2024
@maksymvavilov maksymvavilov self-assigned this May 28, 2024
@maksymvavilov maksymvavilov moved this from Todo to In Progress in Kuadrant May 28, 2024
@maleck13 maleck13 added this to the Kuadrant v1 milestone May 31, 2024
@maksymvavilov maksymvavilov linked a pull request Jun 11, 2024 that will close this issue
@maksymvavilov maksymvavilov moved this from In Progress to Ready For Review in Kuadrant Jun 11, 2024
@github-project-automation github-project-automation bot moved this from Ready For Review to Done in Kuadrant Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tests Changes to tests only size/small
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants