-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix warning when setup cloudstack-common #4523
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
Fix warning when setup cloudstack-common #4523
Conversation
Unpacking cloudstack-common (4.15.0.0-1607306448~bionic) over (4.15.0.0-SNAPSHOT~bionic) ...
Setting up cloudstack-common (4.15.0.0-1607306448~bionic) ...
/usr/lib/python3/dist-packages/cloud_utils.py:434: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("augtool has not been properly installed on this system")
/usr/lib/python3/dist-packages/cloud_utils.py:649: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("cgconfig has not been properly installed on this system")
/usr/lib/python3/dist-packages/cloud_utils.py:675: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("cgrulesd has not been properly installed on this system")
/usr/lib/python3/dist-packages/cloud_utils.py:696: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("qemu has not been properly installed on this system")
/usr/lib/python3/dist-packages/cloud_utils.py:715: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("libvirt has not been properly installed on this system")
/usr/lib/python3/dist-packages/cloud_utils.py:745: SyntaxWarning: "is" with a literal. Did you mean "=="?
if e.errno is 2: raise TaskFailed("libvirt has not been properly installed on this system")
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2469 |
|
@blueorangutan test |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
Trillian test result (tid-3322)
|
* master: server: add conditions for custom offerings (apache#4540) vr: Ensuring dnsmasq.leases file is populated (apache#4529) template: Ensuring template is cross zone if type changed to system (apache#4522) storage: Fix hypervisor type cast to string (apache#4516) db upgrade: fix sql exception: Access denied; you need (at least one of) the SUPER privilege(s) for this operation (apache#4533) CLOUDSTACK-10423:Potential sensitive information disclosure (apache#4536) jobs: The patch remove the password from resultObject and make it be humanreadable (apache#4538) listphysicalnetworks: Honouring keyword parameter (apache#4511) Fix NPE when Volume exists on secondary store but doesn't have a download URL (apache#4530) apidoc issue (apache#4532) db: Fix description of volume.stats.interval which is in milliseconds not seconds (apache#4526) kvm: set cpu topology only if cpucore per socket is positive value (apache#4527) xenserver: check and eject patch vbd for systemvms (apache#4525) Fix warning when setup cloudstack-common (apache#4523) kvm: FIX cpucorespersocket is not working on KVM (apache#4497) change debug to warn for unknown exceptions (apache#4521) Fix failure in validating IP address in case of multiple Management Servers (apache#4507) Update log output for FirstFitPlanner (apache#4515) ui: deprecate old UI and move to legacy to be served at /client/legacy (apache#4518)
Description
This PR fixes the warnings below when install/setup cloudstack-common
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?