Skip to content

ezmomi status' can show more VM info & new way to print the output (a 'grep' firendly one) #68

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

Merged
merged 6 commits into from
Jul 1, 2016

Conversation

ciorceri
Copy link
Contributor

In this pull request contains 2 changes

  1. 'ezmoni status' has one extra parameter named '--extra' which will print multiple VM details.
    2 added a new '--parserFriendly' parameter for 'ezmomi status'.

Some comments related with those changes:

  • I've tried to use the same code style in my changes, if I've failed to do that, please give me some feedback and I will try to fix that. you will see that I've used 'sys.stdout.write' instead of 'print'. The reason is this: http://stackoverflow.com/questions/255147/how-do-i-keep-python-print-from-adding-newlines-or-spaces . For python2 I don't see a simpler way to do that without to confuse others.
  • I've changed the 'tabulate' name to 'print_as_table' since I've added another way to deliver the output 'print_as_lines'.

All those changes were done based on my actual needs. I hope that they help the community.

I hope this one will pass PEP8 checks.

Sorin Ciorceri added 4 commits June 20, 2016 14:48
added a new way to print status which is 'grep' friendly
@snobear
Copy link
Owner

snobear commented Jun 21, 2016

Thanks @ciorceri for the great contributions. I'll take it for a spin today or tomorrow and get back to you.

Sorin Ciorceri added 2 commits June 28, 2016 14:45
In case we have a snapshot tree, only the first snapshot was printed
@snobear snobear merged commit 849489d into snobear:master Jul 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants