-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Shared Filesystem support on Config Drive Networks #11726
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 Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11726 +/- ##
============================================
+ Coverage 17.39% 17.42% +0.02%
- Complexity 15285 15333 +48
============================================
Files 5889 5892 +3
Lines 526183 526521 +338
Branches 64242 64293 +51
============================================
+ Hits 91542 91757 +215
- Misses 424297 424410 +113
- Partials 10344 10354 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 lgtm
@abh1sar |
Yes, @weizhouapache |
good @abh1sar @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 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15171 |
@blueorangutan test |
@weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
[SF] Trillian test result (tid-14441)
|
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. tested with an isolated network which uses configdrive for userdata. I could see the mount point in the system VM. Works as expected after the change.
@vishesh92 thanks for the testing |
Yes. I used the template @abh1sar shared with me for this change. I verified the cloud init configuration changes as well in the sharedfs vm. |
great, do we need to (1) change the systemvm template version to 4.22.0.0 in pom.xml (2) upload systemvm template to https://download.cloudstack.org/systemvm/ (new 4.22 directory) ? |
Yes. We will need to do that. |
maybe after merge of #11654 ? |
Description
This PR fixes #10315
This will require a new systemvmtemplate.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
Before fix on config drive :



After fix on config drive :
How Has This Been Tested?
How did you try to break this feature and the system with this change?