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

[E2E] Test service DNS on mutiple clusters returns diffrent IPs (Round Robin) #340

Closed
manosnoam opened this issue Oct 14, 2020 · 1 comment · Fixed by #422
Closed

[E2E] Test service DNS on mutiple clusters returns diffrent IPs (Round Robin) #340

manosnoam opened this issue Oct 14, 2020 · 1 comment · Fixed by #422
Assignees
@aswinsuryan
Labels
automation testing
Milestone
0.8.1

Comments

@manosnoam
Copy link

Add test in E2E:
When exporting same service name (with same DNS) on multiple clusters - It should return a different IP each request, according to the clusterIP (and Round Robin algorithm).

Originally posted by @danibachar in https://github.com/submariner-io/lighthouse/issue_comments/703122297
@manosnoam manosnoam added this to the 0.8.0 milestone Oct 14, 2020
This was referenced Oct 14, 2020
Closed
Closed
@aswinsuryan
Copy link
Contributor

We have UT for this scenario but e2e is missing and we need to add it.

@aswinsuryan aswinsuryan self-assigned this Dec 16, 2020
@nyechiel nyechiel modified the milestones: 0.8.0, 0.8.1 Dec 16, 2020
aswinsuryan added a commit to aswinsuryan/lighthouse that referenced this issue Dec 21, 2020
@aswinsuryan
Add e2e to verify round-robin for Service discovery
5c358bb 
Fixes : submariner-io#340
Signed-off-by: Aswin Surayanarayanan <[email protected]>
@aswinsuryan aswinsuryan mentioned this issue Dec 21, 2020
Merged
aswinsuryan added a commit to aswinsuryan/lighthouse that referenced this issue Jan 5, 2021
@aswinsuryan
Add e2e to verify round-robin for Service discovery
39e3f20 
Fixes : submariner-io#340
Signed-off-by: Aswin Surayanarayanan <[email protected]>
aswinsuryan added a commit to aswinsuryan/lighthouse that referenced this issue Jan 6, 2021
@aswinsuryan
Add e2e to verify round-robin for Service discovery
58d3f92 
Fixes : submariner-io#340
Signed-off-by: Aswin Surayanarayanan <[email protected]>
tpantelis pushed a commit that referenced this issue Jan 7, 2021
@aswinsuryan
Add e2e to verify round-robin for Service discovery (#422)
2863256 
Add e2e to verify round-robin for Service discovery

Fixes : #340

Signed-off-by: Aswin Surayanarayanan <[email protected]>
mangelajo pushed a commit to mangelajo/lighthouse that referenced this issue Feb 3, 2021
@aswinsuryan @mangelajo
Add e2e to verify round-robin for Service discovery (submariner-io#422)
2f95974 
Add e2e to verify round-robin for Service discovery

Fixes : submariner-io#340

Signed-off-by: Aswin Surayanarayanan <[email protected]>
(cherry picked from commit 2863256)
mangelajo pushed a commit to mangelajo/lighthouse that referenced this issue Feb 3, 2021
@aswinsuryan @mangelajo
Add e2e to verify round-robin for Service discovery (submariner-io#422)
9d73529 
Add e2e to verify round-robin for Service discovery

Fixes : submariner-io#340

Signed-off-by: Aswin Suryanarayanan <[email protected]>
(cherry picked from commit 2863256)
mangelajo pushed a commit that referenced this issue Feb 10, 2021
@aswinsuryan @mangelajo
Add e2e to verify round-robin for Service discovery (#422)
2d0cd3d 
Add e2e to verify round-robin for Service discovery

Fixes : #340

Signed-off-by: Aswin Suryanarayanan <[email protected]>
(cherry picked from commit 2863256)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aswinsuryan aswinsuryan

Labels
automation testing
Projects
None yet
Milestone
0.8.1
Development

Successfully merging a pull request may close this issue.

Add e2e to verify round-robin for Service discovery aswinsuryan/lighthouse
3 participants
@nyechiel @manosnoam @aswinsuryan