[virt] setvcpus/setmem: fix return value parsing issue when calling vm_state(vm_) #38675

Merged
merged 1 commit into from Jan 15, 2017

Projects

None yet

2 participants

@chenttuuvv

What does this PR do?

[virt] setvcpus/setmem: always return false because we are unable to get vm state

What issues does this PR fix or reference?

#38446

Previous Behavior

vm_state(vm_) != 'shutdown' is evaluated as {'centos7.0': 'shutdown'} != 'shutdown'

New Behavior

vm_state(vm_) is modified to vm_state(vm_)[vm_]

Tests written?

No

root [virt] setvcpus/setmem: fix return value parsing issue when calling v…
…m_state(vm_)
3595494
@cachedout
Contributor

Go Go Jenkins!

@cachedout cachedout merged commit a1d8023 into saltstack:develop Jan 15, 2017

2 of 6 checks passed

default Pull Requests » Salt PR - Main Build #60
Details
jenkins/PR/salt-pr-linode-ubuntu14-n Pull Requests » Salt Linode Ubuntu14.04 #7640 — FAILURE
Details
jenkins/PR/salt-pr-lint-n Pull Requests » Salt PR - Code Lint #7791 — FAILURE
Details
jenkins/PR/salt-pr-rs-cent7-n Pull Requests » Salt PR - RS CentOS 7 #7938 — FAILURE
Details
jenkins/PR/salt-pr-clone Pull Requests » Salt PR - Clone #7974 — SUCCESS
Details
jenkins/PR/salt-pr-docs-n Pull Requests » salt-pr-docs-n #3265 — SUCCESS
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment