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

Adding some doc examples - Issues258 #527

Merged
merged 11 commits into from Aug 11, 2015

Conversation

allmightyspiff
Copy link
Member

some basic examples on the vs and sever managers mostly.

@@ -288,6 +335,65 @@ def place_order(self, **kwargs):
:param boolean no_public: True if this server should only have private
interfaces
:param list extras: List of extra feature names

.. warning::
Copy link
Contributor

Choose a reason for hiding this comment

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

This example/description is no longer true. place_order uses the simplified hardware ordering and is able to take more-or-less human readable values. See the new slcli server create-options and slcli server create commands to see how it works now.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.64% when pulling 5cd4b10 on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@allmightyspiff
Copy link
Member Author

Removed the examples for place order, when I figure out the new hotness I'll add a more relevant example.
Removed the print statements, they were superfluous anyway

I removed the cancel_metal example

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 7958a6e on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 7958a6e on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.64% when pulling 7958a6e on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.64% when pulling 7958a6e on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 3525b36 on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.64% when pulling 3525b36 on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@@ -26,6 +26,15 @@ class HardwareManager(utils.IdentifierMixin, object):
manager to handle ordering.
If none is provided, one will be
auto initialized.
Example::
# Initialize the Manager.
Copy link
Contributor

Choose a reason for hiding this comment

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

A blank line is required after Example::

Example::

# Cancels hardware id 1234
result = mrg.cancel_hardware(hardware_id=1234)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be mgr?

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 9d6fb2c on allmightyspiff:issues258 into 43b0b4e on softlayer:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 3856c40 on allmightyspiff:issues258 into fa58330 on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.09% when pulling 3856c40 on allmightyspiff:issues258 into fa58330 on softlayer:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 57d5e4f on allmightyspiff:issues258 into 6ece889 on softlayer:master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling c9e3445 on allmightyspiff:issues258 into 6ece889 on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.09% when pulling 57d5e4f on allmightyspiff:issues258 into 6ece889 on softlayer:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.09% when pulling c9e3445 on allmightyspiff:issues258 into 6ece889 on softlayer:master.

underscorephil added a commit that referenced this pull request Aug 11, 2015
Adding some doc examples - Issues258
@underscorephil underscorephil merged commit 5330151 into softlayer:master Aug 11, 2015
@allmightyspiff allmightyspiff deleted the issues258 branch August 31, 2020 22:26
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.

None yet

5 participants