salt-run virt.* salt/runners/virt.py virtual:physical restriction

[damon-atkins]

Description of Issue/Question

Just wondering why salt-run virt.* has virtual:physical as the target restriction.
These days you can have nested VM. i.e. Hypervisor running on VM's, which is also acting as a Hypervisor running VM's

$egrep -n virtual:physical  /usr/lib/python2.7/site-packages/salt/runners/virt.py
78:        for info in client.cmd_iter('virtual:physical',
123:    for info in client.cmd_iter('virtual:physical',

Setup

(Please provide relevant configs and/or SLS files (Be sure to remove sensitive info).)

Steps to Reproduce Issue

N/A

Versions Report

salt 2016.3.2 (Boron) & develop

[Ch3LL]

@damon-atkins ahh it does seem its possible to nest kvm (which is what salt-virt currently uses) to run on a vm from a quick google search.

Seems we could improve this module to allow VMs to also work with this module. Will approve as a feature request to improve this module thanks!

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

[stale[bot]]

Thank you for updating this issue. It is no longer marked as stale.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

[stale[bot]]

Thank you for updating this issue. It is no longer marked as stale.