github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #53381 from dims/remove-conformance-for-internet-connection-e2e-test

Browse Source 
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Remove conformance tag for internet connectivity

**What this PR does / why we need it**:

ICMP ping is not available in many environments, so we should avoid
using a e2e test based the premise as a conformance test.


**Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #

**Special notes for your reviewer**:
Part of Fix for #52098, Please see #51287 for more discussion

**Release note**:

```release-note
NONE
```
This commit is contained in:
Kubernetes Submit Queue 2017-10-03 09:57:17 -07:00 committed by GitHub
commit 3bb0f42ae3
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/e2e/network/networking.go
View File

@ -45,7 +45,7 @@ var _ = SIGDescribe("Networking", func() {
}
})
It("should provide Internet connection for containers [Conformance]", func() {
It("should provide Internet connection for containers", func() {
By("Running container which tries to ping 8.8.8.8")
framework.ExpectNoError(
framework.CheckConnectivityToHost(f, "", "ping-test", "8.8.8.8", 30))