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

test: Extend "Checks service on same node" test case #10687

Merged
merged 1 commit into from Mar 24, 2020
Merged

test: Extend "Checks service on same node" test case #10687

merged 1 commit into from Mar 24, 2020

Conversation

brb
Copy link
Member

@brb brb commented Mar 24, 2020

Extend the test case by adding a check whether sending a request from a pod to a service endpoint on the same node via ClusterIP works.

Fix: #10568.

@brb
test: Extend "Checks service on same node" test case
5773367 
Extend the test case by adding a check whether sending a request from a
pod to a service endpoint on the same node via ClusterIP works.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added pending-review area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Mar 24, 2020
@brb brb requested a review from a team as a code owner March 24, 2020 16:00
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Mar 24, 2020
@brb
Copy link
Member Author

brb commented Mar 24, 2020

test-me-please

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 45.532% when pulling 5773367 on pr/brb/test-same-node-client into ae4fbbe on master.

@borkmann borkmann merged commit 0dbc05b into master Mar 24, 2020
1.8.0 automation moved this from In progress to Merged Mar 24, 2020
@borkmann borkmann deleted the pr/brb/test-same-node-client branch March 24, 2020 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI.
Projects
No open projects
1.8.0
  
Merged
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Send requests to a service from a client pod in "Checks service on same node" suite
3 participants
@brb @coveralls @borkmann