Skip to content

Conversation

allmightyspiff
Copy link
Member

Adds the following CLI commands, and a manager for vs_placementgroups

slcli vs placementgroup --help
Usage: slcli vs placementgroup [OPTIONS] COMMAND [ARGS]...

  Base command for all capacity related concerns

Options:
  -h, --help  Show this message and exit.

Commands:
  create          Create a placement group
  create-options  List options for creating Reserved Capacity
  delete          Delete a placement group.
  detail          View details of a placement group.
  list            List Reserved Capacity groups.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 90.266% when pulling ed7b636 on allmightyspiff:issues1069 into d047447 on softlayer:master.

Copy link
Contributor

@acamacho82 acamacho82 left a comment

Choose a reason for hiding this comment

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

I couldn't find issues and the code looks good! :)

Copy link
Contributor

@FernandoOjeda FernandoOjeda left a comment

Choose a reason for hiding this comment

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

The same for me, the code looks good.

@allmightyspiff allmightyspiff merged commit 2bbbab1 into softlayer:master Feb 4, 2019
@allmightyspiff allmightyspiff deleted the issues1069 branch February 4, 2019 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants