-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
CKS: add ConfigDrive to cloud-init datasource_list in systemvm template #7650
base: main
Are you sure you want to change the base?
CKS: add ConfigDrive to cloud-init datasource_list in systemvm template #7650
Conversation
This requires a new systemvm template |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7650 +/- ##
============================================
- Coverage 15.81% 15.81% -0.01%
+ Complexity 12554 12553 -1
============================================
Files 5629 5629
Lines 492031 492031
Branches 62921 63552 +631
============================================
- Hits 77823 77813 -10
- Misses 405888 405895 +7
- Partials 8320 8323 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@blueorangutan package |
@weizhouapache a [SF] 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 6269 |
@blueorangutan test |
@weizhouapache a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
[SF] Trillian test result (tid-6772)
|
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 didn't test the change, but have tested the URL fix - for the packer/systemvmtemplate build job (using my local jenkins instance)
[SF] Trillian test result (tid-6837)
|
[SF] Trillian test result (tid-6839)
|
[SF] Trillian test result (tid-6838)
|
[SF] Trillian test result (tid-6840)
|
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.
code looks good
thanks @rohityadavcloud @DaanHoogland for review I have kicked some trillian test using the new templates. The kvm/vmware test results (see previous comments) look good. |
[SF] Trillian test result (tid-6841)
|
[SF] Trillian Build Failed (tid-6859) |
[SF] Trillian test result (tid-6861)
|
[SF] Trillian test result (tid-6858)
|
[SF] Trillian test result (tid-6860)
|
ok, is main the right target for this? |
yes @DaanHoogland |
@weizhouapache @DaanHoogland @rohityadavcloud are we missing something or can we merge this PR? |
@JoaoJandre |
Oh, I didn't see it 😆 |
@blueorangutan package |
@kiranchavala 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]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 11131 |
@blueorangutan package |
@weizhouapache 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]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 11133 |
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, did not test it
@weizhouapache when we merge this we will need a new systemvm template, or is the current good enough right now? cc @JoaoJandre, @kiranchavala . |
@DaanHoogland we need a new template. @kiranchavala is now testing the new systemvm template with new |
@DaanHoogland |
[SF] Trillian Build Failed (tid-11501) |
[SF] Trillian test result (tid-11502)
|
[SF] Trillian test result (tid-11500)
|
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
Test manually on a kvm hypervisor
Able to launch a cks cluster successfully on a isolated network
Steps
- Registered the new systemvm template
- Create a new network offering with userdata as config drive
- Created a isolated network with the network offering mentioned step 1
- Launch a cks cluster
Performed lifecycle operations on the cks cluster like scaling, aquiring loadbalancer IP and destroying cks cluster
thanks a lot @kiranchavala for the testing ! let's wait for smoke tests to be complete. |
@weizhouapache , I see three smoke test feedback replies above. did you start any others? |
[SF] Trillian test result (tid-11504)
|
@DaanHoogland the tests with xcpng82 and vmware-80u are unstable |
[SF] Trillian test result (tid-11506)
|
[SF] Trillian Build Failed (tid-11517) |
[SF] Trillian test result (tid-11514)
|
Description
This PR fixes #7647
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?