-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add e2e tests for listing resources #8281
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8281 +/- ##
============================================
+ Coverage 28.95% 30.83% +1.88%
- Complexity 31460 34020 +2560
============================================
Files 5329 5341 +12
Lines 373523 374759 +1236
Branches 54315 54510 +195
============================================
+ Hits 108163 115568 +7405
+ Misses 250569 243972 -6597
- Partials 14791 15219 +428
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
d82e05f to
2294a23
Compare
|
@blueorangutan package |
|
@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 7873 |
2294a23 to
3723613
Compare
3723613 to
cff8e1a
Compare
5c32b76 to
1fc6920
Compare
|
@blueorangutan package |
|
@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
DaanHoogland
left a comment
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 look good, just the UI build failed, which is unrelated (annoying though)
|
@blueorangutan package |
|
@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
@blueorangutan package |
|
@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 7912 |
|
@blueorangutan test |
|
[SF] Trillian test result (tid-8615)
|
|
@vishesh92 similar failures in VMware test results. Anything related? |
No. It doesn't have anything related to my changes. |
|
@shwstppr I checked a few failures and most of them seem related to the environment itself or some error while cleaning up resources. |
|
@blueorangutan package |
|
@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Considering the current smoke tests are mostly very stable (based on the results of health check PRs), can we revisit this after 4.19 release ? |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8076 |
|
@blueorangutan test matrix |
|
@vishesh92 a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
@weizhouapache Just rebuilt the packages again. Last time there was some issues while packaging. Let's wait for the results of the latest test run. |
|
[SF] Trillian test result (tid-8619)
|
|
[SF] Trillian test result (tid-8621)
|
|
[SF] Trillian test result (tid-8620)
|
|
smoke tests look ok. the failures might be caused by the busy trillian envs. |
|
@blueorangutan LLtest matrix |
|
@weizhouapache a [LL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
[LL]Trillian test result (tid-6901)
|
|
[LL]Trillian test result (tid-6900)
|
|
Based on a discussion with contributors involved in #8392, I'm reverting this for now. We will revisit e2e tests later |
This PR adds e2e tests for listing resources
This reverts commit 1411da1.
Description
This PR adds e2e tests for listing resources
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?