Skip to content
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

Fixes #6657: Allow a user to export his current platform #17

Conversation

peckpeck
Copy link
Member

@peckpeck peckpeck changed the title Fixes #6657: Allow a user to export its current platform Fixes #6657: Allow a user to export his current platform May 27, 2015
rtf
os.system("VBoxManage controlvm " + uuid + " savestate")
else:
print("Currently only running VMs can be exported.")
print("If you wish to export a stopped VM, please open ticket to ask for the feature.")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function continues if the vm is running, is it ok ? (the print statement mkes me think it should not!!)

@peckpeck
Copy link
Member Author

peckpeck commented Jun 1, 2015

PR updated

@peckpeck peckpeck force-pushed the ust_6657/allow_a_user_to_export_its_current_platform branch from 98d7082 to 56403d4 Compare June 1, 2015 09:28
@peckpeck
Copy link
Member Author

peckpeck commented Jun 1, 2015

PR updated

@peckpeck peckpeck force-pushed the ust_6657/allow_a_user_to_export_its_current_platform branch from 56403d4 to 07a5b96 Compare June 1, 2015 09:29
VinceMacBuche added a commit that referenced this pull request Jun 1, 2015
…its_current_platform

Fixes #6657: Allow a user to export his current platform
@VinceMacBuche VinceMacBuche merged commit 7276659 into Normation:master Jun 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants