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

contrib: test: switch node-e2e to e2e #631

Merged
merged 1 commit into from
Sep 9, 2017
Merged

contrib: test: switch node-e2e to e2e #631

merged 1 commit into from
Sep 9, 2017

Conversation

runcom
Copy link
Member

@runcom runcom commented Jun 26, 2017 •
edited
Loading

ref: #533

Signed-off-by: Antonio Murdaca runcom@redhat.com

@runcom runcom force-pushed the e2e-conformance branch 3 times, most recently from a0f0ed6 to 816857b Compare June 26, 2017 09:40
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 27, 2017
@runcom
Copy link
Member Author

runcom commented Aug 27, 2017

[retest]

@rh-atomic-bot
Copy link

133/133 passed on Fedora - Passed.
0/0 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1265/fullresults.txt

@runcom
Copy link
Member Author

runcom commented Aug 27, 2017 •
edited
Loading

RHEL/Fedora CI having OOM issues actually (when building kube + tests):

 _____________
< TASK [Make] >
 -------------
        \   ^__^
         \  (oo)\_______
            (__)\       )\/\
                ||----w |
                ||     ||

fatal: [redacted]: FAILED! => {"ansible_job_id": "878718883690.4857", "changed": false, "cmd": "/usr/bin/make", "failed": true, "finished": 1, "msg": "# k8s.io/kubernetes/cmd/hyperkube\n/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory\n\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: /root/src/k8s.io/kubernetes/hack/lib/golang.sh:710 kube::golang::build_binaries_for_platform(...)\n!!! [0827 17:56:25]  2: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\nmake: *** [all] Error 1", "rc": 2, "stderr": "# k8s.io/kubernetes/cmd/hyperkube\n/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory\n\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: /root/src/k8s.io/kubernetes/hack/lib/golang.sh:710 kube::golang::build_binaries_for_platform(...)\n!!! [0827 17:56:25]  2: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\n!!! [0827 17:56:25] Call tree:\n!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)\nmake: *** [all] Error 1\n", "stderr_lines": ["# k8s.io/kubernetes/cmd/hyperkube", "/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory", "", "!!! [0827 17:56:25] Call tree:", "!!! [0827 17:56:25]  1: /root/src/k8s.io/kubernetes/hack/lib/golang.sh:710 kube::golang::build_binaries_for_platform(...)", "!!! [0827 17:56:25]  2: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)", "!!! [0827 17:56:25] Call tree:", "!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)", "!!! [0827 17:56:25] Call tree:", "!!! [0827 17:56:25]  1: hack/make-rules/build.sh:27 kube::golang::build_binaries(...)", "make: *** [all] Error 1"], "stdout": "+++ [0827 17:53:46] Building the toolchain targets:\n    k8s.io/kubernetes/hack/cmd/teststale\n    k8s.io/kubernetes/vendor/github.com/jteeuwen/go-bindata/go-bindata\n+++ [0827 17:53:46] Generating bindata:\n    test/e2e/generated/gobindata_util.go\n~/src/k8s.io/kubernetes ~/src/k8s.io/kubernetes/test/e2e/generated\n~/src/k8s.io/kubernetes/test/e2e/generated\n+++ [0827 17:53:46] Building go targets for linux/amd64:\n    cmd/kube-proxy\n    cmd/kube-apiserver\n    cmd/kube-controller-manager\n    cmd/cloud-controller-manager\n    cmd/kubelet\n    cmd/kubeadm\n    cmd/hyperkube\n    vendor/k8s.io/kube-aggregator\n    vendor/k8s.io/apiextensions-apiserver\n    plugin/cmd/kube-scheduler\n    cmd/kubectl\n    federation/cmd/kubefed\n    cmd/gendocs\n    cmd/genkubedocs\n    cmd/genman\n    cmd/genyaml\n    cmd/mungedocs\n    cmd/genswaggertypedocs\n    cmd/linkcheck\n    federation/cmd/genfeddocs\n    vendor/github.com/onsi/ginkgo/ginkgo\n    test/e2e/e2e.test\n    cmd/kubemark\n    vendor/github.com/onsi/ginkgo/ginkgo\n    test/e2e_node/e2e_node.test\n    cmd/gke-certificates-controller\n", "stdout_lines": ["+++ [0827 17:53:46] Building the toolchain targets:", "    k8s.io/kubernetes/hack/cmd/teststale", "    k8s.io/kubernetes/vendor/github.com/jteeuwen/go-bindata/go-bindata", "+++ [0827 17:53:46] Generating bindata:", "    test/e2e/generated/gobindata_util.go", "~/src/k8s.io/kubernetes ~/src/k8s.io/kubernetes/test/e2e/generated", "~/src/k8s.io/kubernetes/test/e2e/generated", "+++ [0827 17:53:46] Building go targets for linux/amd64:", "    cmd/kube-proxy", "    cmd/kube-apiserver", "    cmd/kube-controller-manager", "    cmd/cloud-controller-manager", "    cmd/kubelet", "    cmd/kubeadm", "    cmd/hyperkube", "    vendor/k8s.io/kube-aggregator", "    vendor/k8s.io/apiextensions-apiserver", "    plugin/cmd/kube-scheduler", "    cmd/kubectl", "    federation/cmd/kubefed", "    cmd/gendocs", "    cmd/genkubedocs", "    cmd/genman", "    cmd/genyaml", "    cmd/mungedocs", "    cmd/genswaggertypedocs", "    cmd/linkcheck", "    federation/cmd/genfeddocs", "    vendor/github.com/onsi/ginkgo/ginkgo", "    test/e2e/e2e.test", "    cmd/kubemark", "    vendor/github.com/onsi/ginkgo/ginkgo", "    test/e2e_node/e2e_node.test", "    cmd/gke-certificates-controller"]}
        to retry, use: --limit @/home/amurdaca/go/src/github.com/kubernetes-incubator/cri-o/contrib/test/crio-integration-playbook.retry

@rh-atomic-bot
Copy link

133/133 passed on RHEL - Passed.
0/0 k8s node-e2e passed on RHEL - Failed.

133/133 passed on Fedora - Passed.
0/0 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1266/fullresults.txt

@rh-atomic-bot
Copy link

133/133 passed on RHEL - Passed.
0/0 k8s node-e2e passed on RHEL - Failed.

133/133 passed on Fedora - Passed.
0/0 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1268/fullresults.txt

@runcom
Copy link
Member Author

runcom commented Aug 28, 2017

nada, can't make kubernetes at all in the CI 😕

@runcom runcom mentioned this pull request Aug 29, 2017
Merged
@runcom
Copy link
Member Author

runcom commented Aug 30, 2017

Pushed @cevich's commits here (squashed them), let's see how it goes and if we can compile kube (after I'll take care of actually make tests run)

@rh-atomic-bot
Copy link

30/31 passed on RHEL - Failed.
0/0 k8s node-e2e passed on RHEL - Failed.

133/133 passed on Fedora - Passed.
0/0 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1327/fullresults.txt

@runcom
Copy link
Member Author

runcom commented Aug 31, 2017

[retest]

This was referenced Aug 31, 2017
Closed
Merged
@runcom
Copy link
Member Author

runcom commented Sep 6, 2017

[retest]

@runcom
Copy link
Member Author

runcom commented Sep 7, 2017

we might need something like this containerd/cri@e6332a7 - leaving here for future reference as I'm re-working this PR

@runcom runcom force-pushed the e2e-conformance branch 2 times, most recently from 840e841 to 20d13cb Compare September 8, 2017 10:46
@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

(no worry, it's not the CI not reporting, it's me cancelling the build in the CI once I see they do not work 😄 )

@cevich
Copy link
Contributor

cevich commented Sep 8, 2017

(no worry, it's not the CI not reporting

Good on you for doing it that way, it reduces resource-consumption. Just pay attention the 'teardown' of VMs happens, otherwise they get "orphaned". Let me know if you don't know how to spot that in the logs.

@runcom runcom force-pushed the e2e-conformance branch 3 times, most recently from 9605c45 to 3c28a67 Compare September 8, 2017 15:39
@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

1 similar comment
@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@runcom runcom changed the title [DO NOT MERGE] contrib: test: switch node-e2e to e2e contrib: test: switch node-e2e to e2e Sep 8, 2017
@runcom runcom mentioned this pull request Sep 8, 2017
Closed
@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@runcom
contrib: test: switch node-e2e to e2e
dd1c617 
Signed-off-by: Antonio Murdaca <runcom@redhat.com>
@runcom
Copy link
Member Author

runcom commented Sep 8, 2017

[retest]

@rh-atomic-bot
Copy link

138/138 passed on RHEL - Passed.
149/152 k8s node-e2e passed on RHEL - Failed.

138/138 passed on Fedora - Passed.
149/152 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1520/fullresults.txt

@rh-atomic-bot
Copy link

138/138 passed on RHEL - Passed.
149/152 k8s node-e2e passed on RHEL - Failed.

138/138 passed on Fedora - Passed.
149/152 k8s node-e2e passed on Fedora - Failed.

Log - https://aos-ci.s3.amazonaws.com/kubernetes-incubator/cri-o/crio-integration-tests-prs/1521/fullresults.txt

@runcom
Copy link
Member Author

runcom commented Sep 9, 2017

🎉 🎉 🎉

So, here is the thing with 149/152: the tests ran and passed are 149, 0 failed, but the CI has a grep which is failing to find the failed number and is adding +3 because it finds 3 "Failure" string. I'm fixing that as we speak and I'll merge this meanwhile :)

@runcom runcom merged commit f35108d into cri-o:master Sep 9, 2017
@runcom runcom deleted the e2e-conformance branch September 9, 2017 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@runcom @rh-atomic-bot @cevich @k8s-ci-robot