-
Notifications
You must be signed in to change notification settings - Fork 17
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
[DPE-3257] Fix network cut tests #346
Conversation
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #346 +/- ##
==========================================
- Coverage 80.00% 79.99% -0.02%
==========================================
Files 10 10
Lines 2176 2179 +3
Branches 344 345 +1
==========================================
+ Hits 1741 1743 +2
- Misses 368 369 +1
Partials 67 67 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
…cut-test Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent! Thank you!
@@ -1633,7 +1633,6 @@ files = [ | |||
{file = "PyYAML-6.0.1-cp311-cp311-win_amd64.whl", hash = "sha256:bf07ee2fef7014951eeb99f56f39c9bb4af143d8aa3c21b1677805985307da34"}, | |||
{file = "PyYAML-6.0.1-cp312-cp312-macosx_10_9_x86_64.whl", hash = "sha256:855fb52b0dc35af121542a76b9a84f8d1cd886ea97c84703eaa6d88e37a2ad28"}, | |||
{file = "PyYAML-6.0.1-cp312-cp312-macosx_11_0_arm64.whl", hash = "sha256:40df9b996c2b73138957fe23a16a4f0ba614f4c0efce1e9406a184b6d07fa3a9"}, | |||
{file = "PyYAML-6.0.1-cp312-cp312-manylinux_2_17_aarch64.manylinux2014_aarch64.whl", hash = "sha256:a08c6f0fe150303c1c6b71ebcd7213c2858041a7e01975da3a99aed1e7a378ef"}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is it constantly flicking? CC: @carlcsaposs-canonical
Each PR removes and re-adds this dependency here.
P.S. ARM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe different versions of poetry on different developer's machines?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will also do some tests using the Multipass Ubuntu VM here to understand it.
* Fix network cut test Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Fix network cut test without IP change Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Update unit test Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Fix retrieval of units IPs Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Improve checks for readiness Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Fix IP retrieval Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Fix IP retrieval Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> * Fix IP retrieval Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com> --------- Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
Issue
The two network cut tests are being skipped from CI due to instabilities on them.
Solution
Enable the tests again on CI.
Fix the causes of the issues in the tests:
juju exec
because the IP reported byjuju status
might be outdated sometimes.