-
Notifications
You must be signed in to change notification settings - Fork 937
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
Conformance results from v1.8/cfcr #63
Conversation
bsnchan
commented
Nov 7, 2017
[#152479548] Signed-off-by: Brenda Chan <brchan@pivotal.io>
version.txt is no longer needed as it's plugged directly into the e2e.log file: cncf#55
There is one test that is failing but should be resolved by kubernetes/kubernetes#54581 Current failing test:
|
[#152479548] Signed-off-by: Iain Sproat <isproat@pivotal.io>
Thanks for fixing that issue, looks like it was just merged (tracking in #27). If we assume the test will pass with that fix for now, can you commit to updating your conformance results once the fix is available in the e2e test image? |
Thanks @WilliamDenniss - Would it be possible to still get the k8s certification even with this one failure? We will definitely update our test results once the e2e test image has be updated. Let us know if there is anything we can do to bump the e2e test image. |
tests [#152479548] Signed-off-by: Brenda Chan <brchan@pivotal.io>
+1 from my perspective, since you have proactively fixed the issue and I assume tested that you're now passing. Be aware that if the test still fails after the fix is merged, you could be decertified but I trust that you will re-test and verify that. |
Looks like a test bug, so I'm fine with excluding it until it is fixed. |
@kbarnard10 CFF has signed the participation form now |
@bsnchan Looks like your version.txt file is missing. Can you add? |
Hi @kbarnard10 - Sorry about that. We assumed that the because of issue mentioned here that the version can be extracted from the e2e.log. I've updated the PR with the test a new test run and a version.txt. Please let me know if you need anything else. |
+1 |