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

Python high-level tests for zone-sync feature - no TLS #7285

Open
jjngx opened this issue Feb 5, 2025 · 1 comment · May be fixed by #7347
Open

Python high-level tests for zone-sync feature - no TLS #7285

jjngx opened this issue Feb 5, 2025 · 1 comment · May be fixed by #7347
Assignees
Labels
documentation Pull requests/issues for documentation python Pull requests that update Python code tests Pull requests that update tests

Comments

@jjngx
Copy link
Contributor

jjngx commented Feb 5, 2025

Add Python tests to cover the following scenarios:

  • start NIC with the zone-sync configured in the nginx-ingress config map -> verify headless service created

  • start NIC with the zone-sync NOT configured in the nginx-ingress config map -> verify no headless-svc present

  • add zone-sync config params to the nginx-ingress config map -> verify the headless svc is configured automatically

  • reconfigure port in zone-sync -> verify desired state (add more info)

  • disable zone-sync in the nginx-ingress config map -> verify headless svc is removed from K8s

@jjngx jjngx added documentation Pull requests/issues for documentation proposal An issue that proposes a feature request labels Feb 5, 2025
Copy link

github-actions bot commented Feb 5, 2025

Hi @jjngx thanks for reporting!

Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂

Cheers!

@jjngx jjngx moved this from Todo ☑ to In Progress 🛠 in NGINX Ingress Controller Feb 11, 2025
@jjngx jjngx added python Pull requests that update Python code tests Pull requests that update tests and removed proposal An issue that proposes a feature request labels Feb 11, 2025
@AlexFenlon AlexFenlon linked a pull request Feb 20, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation python Pull requests that update Python code tests Pull requests that update tests
Projects
Status: In Progress 🛠
Development

Successfully merging a pull request may close this issue.

2 participants