server: skip zone check for PERHOST iso during attachIso#5270
server: skip zone check for PERHOST iso during attachIso#5270yadvr merged 3 commits intoapache:4.15apache/cloudstack:4.15from shapeblue:fix-attach-hypervisor-tools-isoshapeblue/cloudstack:fix-attach-hypervisor-tools-isoCopy head branch name to clipboard
Conversation
Hypervisor tools ISO - vmware-toools.iso, xs-tools.iso are marked as PERHOST in DB. They are active but not downloaded to the secondary storages and hence no template-zone entry. Skips the template-zone check for such templates. Fixes apache#5265 Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
server/src/main/java/com/cloud/template/TemplateManagerImpl.java
Outdated
Show resolved
Hide resolved
|
@blueorangutan package |
|
@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔️ el7 ✔️ el8 ✔️ debian. SL-JID 741 |
|
cmk output added for tests using VMware env attaching vmware-tools ISO to a VM |
|
@blueorangutan test |
|
@shwstppr a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-1465)
|
|
tested ok. |
|
@blueorangutan test matrix |
|
@sureshanaparti a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
Trillian test result (tid-1476)
|
|
Trillian test result (tid-1475)
|
davidjumani
left a comment
There was a problem hiding this comment.
Verified, able to mount vmware-tools.iso and xs-tools.iso
|
Trillian test result (tid-1477)
|
server/src/main/java/com/cloud/template/TemplateManagerImpl.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
|
@blueorangutan package |
|
@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔️ el7 ✔️ el8 ✖️ debian. SL-JID 780 |
|
@blueorangutan package |
|
@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔️ el7 ✔️ el8 ✔️ debian. SL-JID 787 |
|
@blueorangutan test matrix |
|
@nvazquez a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
Trillian Build Failed (tid-1522) |
|
Trillian test result (tid-1523)
|
|
Trillian test result (tid-1524)
|
|
@blueorangutan test centos7 xenserver-71 |
|
@shwstppr a Trillian-Jenkins test job (centos7 mgmt + xenserver-71) has been kicked to run smoke tests |
|
Trillian test result (tid-1536)
|
Description
Hypervisor tools ISO - vmware-toools.iso, xs-tools.iso are marked as PERHOST in DB. They are active but not downloaded to the secondary storages and hence no template-zone entry.
Skips the template-zone check for such templates.
Fixes #5265
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
Error on attach ISO:

How Has This Been Tested?
Using cmk with an env (VMware advanced zone)
Before fix:
After fix: