-
Notifications
You must be signed in to change notification settings - Fork 141
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
fixup e2e tests: wrong check during codemodules getInstaller inside provisioner #4347
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #4347 +/- ##
=======================================
Coverage 64.18% 64.19%
=======================================
Files 401 401
Lines 26327 26320 -7
=======================================
- Hits 16897 16895 -2
+ Misses 8125 8121 -4
+ Partials 1305 1304 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
ran the tests several times locally and they worked for me.
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.
Tests work but still trying to understand the logic (I'm comparing rel-1.4
, main
before refactoring , main
after CSI refactoring and the fix)
One more time:
=== RUN TestIstio/cloudnative-csi-resilience/install_sample-apps
app.go:151:
Error Trace: /home/a/dynatrace-operator/test/helpers/sample/app.go:151
...
Error: Received unexpected error:
client rate limiter Wait returned an error: rate: Wait(n=1) would exceed context deadline
Test: TestIstio/cloudnative-csi-resilience/install_sample-apps
=== RUN TestIstio/cloudnative-csi-resilience/check_for_dummy_volume
network_problems.go:103:
Error Trace: /home/a/dynatrace-operator/test/features/cloudnative/network_problems/network_problems.go:103
...
Error: Received unexpected error:
container "app" in pod "sample-dynakube-854dc5bbf5-47qcl" is waiting to start: PodInitializing
Test: TestIstio/cloudnative-csi-resilience/check_for_dummy_volume
=== RUN TestIstio/cloudnative-csi-resilience/install_sample-apps
app.go:151:
Error Trace: /home/a/dynatrace-operator/test/helpers/sample/app.go:151
...
Error: Received unexpected error:
client rate limiter Wait returned an error: rate: Wait(n=1) would exceed context deadline
Test: TestIstio/cloudnative-csi-resilience/install_sample-apps
=== RUN TestIstio/cloudnative-csi-resilience/check_for_dummy_volume
network_problems.go:103:
Error Trace: /home/a/dynatrace-operator/test/features/cloudnative/network_problems/network_problems.go:103
...
Error: Received unexpected error:
container "app" in pod "sample-dynakube-854dc5bbf5-47qcl" is waiting to start: PodInitializing
Test: TestIstio/cloudnative-csi-resilience/check_for_dummy_volume
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.
Tests PASS (not sure if it is stable result)
We only used to check if the CodeModulesImage was set, nothing more. It failed most recently because there was no case for the actual default, which is just a Version in the Status.
Description
This PR solves this problem:
How can this be tested?
run e2e tests: