-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add eco vcenter upstream ci capabilities #538
Add eco vcenter upstream ci capabilities #538
Conversation
903be37
to
b1c45a7
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #538 +/- ##
=======================================
Coverage 37.04% 37.04%
=======================================
Files 145 145
Lines 11388 11388
Branches 2258 2258
=======================================
Hits 4219 4219
Misses 7169 7169
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Build succeeded. ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 15m 16s |
the mergeit label will try to merge the pr AFAIK. it should be added once someone reviews |
78ca83c
to
33dc953
Compare
Build succeeded. ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 13m 09s |
4cd209a
to
5404344
Compare
Build succeeded. ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 13m 48s |
5404344
to
d58e913
Compare
Build succeeded. ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 14m 20s |
Build failed.
|
a01e31f
to
56bcb15
Compare
Build failed.
|
56bcb15
to
4d4eada
Compare
Build failed.
|
4d4eada
to
5e4f2f2
Compare
Build failed.
|
5e4f2f2
to
b16567d
Compare
Build failed.
|
b16567d
to
286628a
Compare
Build failed.
|
286628a
to
b398726
Compare
Build failed.
|
recheck |
08da4e1
to
8479dfa
Compare
8479dfa
to
52e3779
Compare
@anna-savina @mikemorency hey! please review :) this PR insert the eco ci job capabilities without editing a specific integration test - once a PR to adjust a test will be open, the job will catch it and run it by the the prefix of the test - |
Build succeeded. ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 15m 04s |
I was thinking that the test env will need to install the |
@mikemorency yeah I looked at it aswell and assumed we don't need to install any requirements - worst case scenario is to add it later as a step in the Makefile if we will need it for the tests |
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.
LGTM
Build succeeded (gate pipeline). ✔️ ansible-test-cloud-integration-vmware-rest SUCCESS in 12m 52s |
d1b5e83
into
ansible-collections:main
SUMMARY
This PR aims to insert self hosted runner to enable integration tests to run on the eco vcenter environment.
This includes several adjustments to the code and additional action to be triggered for each PR, and run all the
integration tests on that env with the PR's new code.