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

CCI Commands which require an id take 'identifier' #77

Merged
merged 3 commits into from
Mar 22, 2013
Merged

CCI Commands which require an id take 'identifier' #77

merged 3 commits into from
Mar 22, 2013

Conversation

sudorandom
Copy link
Contributor

This should resolve #49

CLI: Now instead of requiring the user to remember or copy/paste numeric ids everywhere in the CCI-related CLI commands, you can now pass in hostname, public ip, private ip or, of course, the numeric id of the CCI.

If no CCIs match, an error is returned.
If more than one CCI matches, an error is returned with the list of relavent ids.

Examples:

sl cci detail production
sl cci dns sync devapp01
sl cci reboot 12.34.56
sl cci shutdown 12345
sl cci detail prod01*

CLI: Now instead of requiring the user to remember or copy/paste numeric ids
     everywhere in the CCI-related CLI commands, you can now pass in hostname,
     public ip, private ip or, of course, the numeric id of the CCI.

     If no CCIs match, an error is returned.
     If more than one CCI matches, an error is returned with the list of relavent ids.

Examples:
    sl cci detail production
    sl cci dns sync devapp01
    sl cci reboot 12.34.56
    sl cci shutdown 12345
API: Moves the layered identifier logic into a mixin to be used in other manager
     classes. Provices the resolve_ids() method which tries to coerce the given
     identifier into an id using methods (in order) defined in the 'resolvers' attribute.
The new mixin lives in SoftLayer.utils
CrackerJackMack added a commit that referenced this pull request Mar 22, 2013
CCI Commands which require an id take 'identifier'
@CrackerJackMack CrackerJackMack merged commit 0c790bc into softlayer:master Mar 22, 2013
@sudorandom sudorandom deleted the identifier branch March 22, 2013 14:21
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.

Make cci detail use name and public-ip address
2 participants