Skip to content
This repository has been archived by the owner on Dec 8, 2020. It is now read-only.

Restructure vagrant vcloud-status to vagrant vcloud command for future additions #53

Closed
tsugliani opened this issue May 11, 2014 · 2 comments
Assignees
Milestone

Comments

@tsugliani
Copy link
Collaborator

The way the actual vagrant vcloud-status command is a bit limited.

In the future, we are very likely going to implement new commands such as :

  • Redeploy Edge Gateway

A little refactoring is needed to make it simple to add new commands.

@tsugliani tsugliani added this to the v0.3.2 milestone May 11, 2014
@tsugliani tsugliani self-assigned this May 11, 2014
@tsugliani
Copy link
Collaborator Author

Changed to:

Usage: vagrant vcloud [options]
    -n, --network                    Display the vCloud Director network mapping information
    -s, --status                     Display the vCloud Director objects IDs
    -h, --help                       Print this help

@StefanScherer
Copy link
Contributor

I would suggest to remove the "--" from the keywords. I use some other plugins (vagrant-vbox-snapshot) which does have commands like vagrant snapshot take back (where back is the name of the snapshot), and I often use vagrant snapshot go back to fast reset to the test point.

So, the commands would look like

vagrant vcloud status [-all]
vagrant vcloud network
vagrant vcloud deploy-edge-gateway
...

What do you think?

StefanScherer added a commit to StefanScherer/vagrant-vcloud that referenced this issue May 12, 2014
* upstream/develop: (32 commits)
  Fixes frapposelli#54 - Added vShield Edge Gateway name
  Fixes frapposelli#54 - Added vagrant vcloud --redeploy-edge-gw
  Fixes frapposelli#45 - vCloud Auth token issue
  Fixes frapposelli#53 - Restructure vagrant vcloud-status to vagrant vcloud <namespace>
  Fixes frapposelli#52 - Missed autoload
  Fixes frapposelli#52 - Block Ability to ssh into halted/suspended VMs
  Fixes frapposelli#47 - Shell provisioner error
  Finally fixed API Version checker to work as expected
  version bump.
  Keep waiting on task if it is in 'queued' or 'preRunning' status
  Update README.md
  Version bump
  Modularized the destroy part to fix issue frapposelli#43 - Split poweroff.rb into poweroff_vapp.rb and poweroff_vm.rb. - Split destroy.rb into destroy_vapp.rb and destroy_vm.rb. - Added is_last_vm.rb to check if vApp has a single VM remaining.
  also show vapp to vm rules
  fix cygdrive path for rsync
  cache return value of get_vapp_edge_public_ip
  added optional vapp_prefix to name the vapp
  removed set_edge_gateway_rules, no longer needed
  add_edge_gateway_rules for DNAT rule per port
  removed edge_gateway_rules code from power_on
  ...
StefanScherer pushed a commit to StefanScherer/vagrant-vcloud that referenced this issue May 13, 2014
* upstream/develop:
  Fixes frapposelli#54 - Added vShield Edge Gateway name
  Fixes frapposelli#54 - Added vagrant vcloud --redeploy-edge-gw
  Fixes frapposelli#45 - vCloud Auth token issue
  Fixes frapposelli#53 - Restructure vagrant vcloud-status to vagrant vcloud <namespace>
  Fixes frapposelli#52 - Missed autoload
  Fixes frapposelli#52 - Block Ability to ssh into halted/suspended VMs
  Fixes frapposelli#47 - Shell provisioner error
StefanScherer pushed a commit to StefanScherer/vagrant-vcloud that referenced this issue May 13, 2014
* upstream/develop:
  Fixes frapposelli#54 - Added vShield Edge Gateway name
  Fixes frapposelli#54 - Added vagrant vcloud --redeploy-edge-gw
  Fixes frapposelli#45 - vCloud Auth token issue
  Fixes frapposelli#53 - Restructure vagrant vcloud-status to vagrant vcloud <namespace>
  Fixes frapposelli#52 - Missed autoload
  Fixes frapposelli#52 - Block Ability to ssh into halted/suspended VMs
  Fixes frapposelli#47 - Shell provisioner error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants