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

SoftLayer: Added flavor option #56036

Open
wants to merge 3 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@MajeedMirza
Copy link

commented May 2, 2019

SUMMARY

Added flavor option to allow for use of SoftLayer flavor templates when creating new virtual machines. This new way of ordering machines allows for suspend billing support where machines that are shut off have their billing suspended till they are turned back on.

The flavor parameter can only be used if the cpus and memory parameters are left blank as those parameters are specified in the flavor template (e.g. B1_1X4X25).

See here for examples of ordering a vm with the flavor parameter:
https://softlayer-python.readthedocs.io/en/latest/api/managers/vs.html

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

SoftLayer flavor support

ADDITIONAL INFORMATION

To see all flavor options use the slcli command:
slcli vs create-options

Added flavor option
Added flavor option to allow for use of SoftLayer flavor templates when creating new virtual machines.

See here for examples of ordering a vm with the flavor parameter:
https://softlayer-python.readthedocs.io/en/latest/api/managers/vs.html

To see flavor options use:
`slcli vs create-options`

@MajeedMirza MajeedMirza changed the title Added flavor option SoftLayer: Added flavor option May 2, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented May 2, 2019

@mcltn

This comment has been minimized.

Copy link
Contributor

commented May 6, 2019

shipit

@ansibot ansibot removed the needs_triage label May 6, 2019

@YuriG7

This comment has been minimized.

Copy link

commented May 14, 2019

shipit

@YuriG7

This comment has been minimized.

Copy link

commented May 14, 2019

It would be great to pull available flavours from SoftLayer and show them in the error message when an incorrect flavour is specified.

@ansibot ansibot added the stale_ci label May 14, 2019

@MajeedMirza

This comment has been minimized.

Copy link
Author

commented May 14, 2019

Stale PR, closing and reopening

@MajeedMirza MajeedMirza reopened this May 14, 2019

@ansibot ansibot removed the stale_ci label May 14, 2019

@mcltn

This comment has been minimized.

Copy link
Contributor

commented May 14, 2019

shipit

@MajeedMirza

This comment has been minimized.

Copy link
Author

commented May 15, 2019

bot_status

@ansibot

This comment has been minimized.

Copy link
Contributor

commented May 15, 2019

Components

lib/ansible/modules/cloud/softlayer/sl_vm.py
support: community
maintainers: mcltn

Metadata

waiting_on: maintainer
changes_requested_by: null
needs_info: False
needs_revision: False
needs_rebase: False
merge_commits: []
too many files or commits: False
mergeable_state: clean
shippable_status: success
maintainer_shipits (module maintainers): 1
community_shipits (namespace maintainers): 0
ansible_shipits (core team members): 0
shipit_actors (maintainers or core team members): mcltn
shipit_actors_other: YuriG7
automerge: automerge shipit test failed

click here for bot help

@MajeedMirza

This comment has been minimized.

Copy link
Author

commented May 15, 2019

bot_broken
Maintainer has approved however has not been automerged and has not given a reason for why it has not been automerged. Requesting bot status shows automerge: automerge shipit test failed with no reason provided and no updated labels.

@ansibot ansibot added the bot_broken label May 15, 2019

@mcltn

mcltn approved these changes May 17, 2019

Copy link
Contributor

left a comment

approved

@mcltn

This comment has been minimized.

Copy link
Contributor

commented May 17, 2019

shipit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.