Skip to content

Commit

Permalink
Merge pull request #1641 from dcos/running-vms-only
Browse files Browse the repository at this point in the history
Do not look at Virtualbox details which are not for VMs
  • Loading branch information
adamtheturtle authored Apr 8, 2019
2 parents ff768e8 + 8cb5ff2 commit ad897df
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions src/dcos_e2e_cli/dcos_vagrant/commands/_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,8 @@ def existing_cluster_ids() -> Set[str]:
"""
Return the IDs of existing clusters.
"""
ls_output = vertigo_py.ls() # type: ignore
vm_ls_output = ls_output['vms']
lines = vm_ls_output.decode().strip().split('\n')
ls_output = vertigo_py.ls(option='vms') # type: ignore
lines = ls_output.decode().strip().split('\n')
lines = [line for line in lines if line]
cluster_ids = set()
for line in lines:
Expand Down Expand Up @@ -166,9 +165,8 @@ def _vm_names(self) -> Set[str]:
"""
Return VirtualBox and Vagrant names of VMs in this cluster.
"""
ls_output = vertigo_py.ls() # type: ignore
vm_ls_output = ls_output['vms']
lines = vm_ls_output.decode().strip().split('\n')
ls_output = vertigo_py.ls(option='vms') # type: ignore
lines = ls_output.decode().strip().split('\n')
lines = [line for line in lines if line]
vm_names = set()
for line in lines:
Expand Down

0 comments on commit ad897df

Please sign in to comment.