-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[Bug fix] VMware: Fix for SSVM recreation on deployasis systemVM templates #4437
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
|
@blueorangutan package |
|
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2310 |
|
@blueorangutan test centos7 vmware67-u3 |
|
@nvazquez unsupported parameters provided. Supported mgmt server os are: |
|
@blueorangutan test centos7 vmware-67u3 |
|
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests |
|
@blueorangutan test centos7 vmware-67u3 |
|
@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests |
Pearl1594
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.
Verified fix. LGTM
...chestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
Show resolved
Hide resolved
|
Trillian test result (tid-3109)
|
rohityadavcloud
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.
Approach LGTM, once the issue wrt other systemvm types is resolved we can test and merge. (I did n't manually test it)
|
@blueorangutan package |
1 similar comment
|
@blueorangutan package |
|
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2332 |
|
@blueorangutan test centos7 vmware-67u3 |
|
@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests |
|
@blueorangutan test centos7 vmware-67u3 |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests |
|
Let's merge once this passes, pl check if the last test env failed due to a regression @nvazquez thnx |
|
@rhtyd tests running now, previous env failed to start the marvin VM |
|
Trillian test result (tid-3132)
|
Description
Fix for VMware SSVM recreation using deploy-as-is system VM templates
Fixes:
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?
VMware environment