-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Release v1.8.0-rc.1 #7286
Release v1.8.0-rc.1 #7286
Conversation
It looks like we still have a race where our tests will directly @DnPlas we could copy-paste the logic for this from my original PR in #7252 (the part that we wait for the deployment, not the one that we patch it) Could you send a PR for this? Asking you since we are burning with the release and I can help merge this fast also cc @thesuperzapper since this is related to our discussion on the CI |
Signed-off-by: Daniela Plascencia <daniela.plascencia@canonical.com>
543dfe2
to
2ee2fef
Compare
@TobiasGoerke could you help us debug the failing unit tests for the PVCViewer Controller? I see the following relevant logs, that come from the following line in the code
|
Not sure why this test starts failing now and without changes. I've only been able to reproduce the error about twice by running a few hundred times and have yet to see it in production. Judging by this issue and this one, we're not alone with this issue and I'd assume its related to a race condition in our testing environment. |
Seems like it happened again see latest run. I will let @kimwnasptd consider the next steps. |
Well, now we're running into a new problem. The tests download the latest istio CRD which is required for testing. However, that path just changed to I'll created PRs against both master and your
|
Changes filepaths on master caused the download of CRDs to fail. Instead, we now add the CRDs statically, to prevent issues like these. See for more info: kubeflow#7286 (comment)
Changes filepaths on master caused the download of CRDs to fail. Instead, we now add the CRDs statically, to prevent issues like these. See for more info: kubeflow#7286 (comment) (cherry picked from commit 16b3743)
Changes filepaths on master caused the download of CRDs to fail. Instead, we now add the CRDs statically, to prevent issues like these. See for more info: #7286 (comment)
Changes filepaths on master caused the download of CRDs to fail. Instead, we now add the CRDs statically, to prevent issues like these. See for more info: kubeflow#7286 (comment) (cherry picked from commit 16b3743)
Thanks @TobiasGoerke ! |
So nice to see everything green! Thank you both @DnPlas @TobiasGoerke! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kimwnasptd The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Nice! Great to be part of this release. Thanks for the support @DnPlas @kimwnasptd |
Bump release version v1.8.0-rc.0 -> v1.8.0-rc.1
cc: @kimwnasptd