Skip to content

adding credentials CLI commands for nas,vs,server#453

Merged
underscorephil merged 6 commits into
softlayer:masterfrom
Neetuj:issue431
Dec 8, 2014
Merged

adding credentials CLI commands for nas,vs,server#453
underscorephil merged 6 commits into
softlayer:masterfrom
Neetuj:issue431

Conversation

@Neetuj
Copy link
Copy Markdown
Member

@Neetuj Neetuj commented Nov 17, 2014

  1. ADDED 
    :...........:............:
:  username :  password  :
:...........:............:
: SL1319*\* : **\* :
:...........:............:



2. remove passwords and username info from sl nas list 
3. Added 
``` sl vs  credentials d09e0fcc-e03c-4c48-9447-****
:..........:..........:
: username : password :
:..........:..........:
:   root   : ***** :
:..........:..........:
  1. Added
sl server credentials 7f47544b-ffff-4ee7-a9b8-******
:..........:..........:
: username : password :
:..........:..........:
:   root   : ***** :
:..........:..........:
  1. leaving credentials in details CLI (va, server)
  2. the server details --password display was nto consistent. made it consistent with VS
:              users : :..........:..........:              :
:                    : : username : password :              :
:                    : :..........:..........:              :
:                    : :   root   : ***** :              :
:                    : :..........:..........:
  1. changed the test to match the change

@Neetuj Neetuj changed the title removing credenstials from "nas list" and adding then to "nas credentia... addin credentials CLI commands for nas,vs,server Nov 18, 2014
@Neetuj Neetuj changed the title addin credentials CLI commands for nas,vs,server adding credentials CLI commands for nas,vs,server Nov 18, 2014
sudorandom
sudorandom reviewed Nov 18, 2014
View reviewed changes
Comment thread SoftLayer/CLI/routes.py Outdated
Copy link
Copy Markdown
Contributor

@sudorandom sudorandom Nov 18, 2014

Choose a reason for hiding this comment

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

Why add a new line?

@Neetuj
Copy link
Copy Markdown
Member Author

Neetuj commented Nov 19, 2014

unnecessory extra new lines removed

underscorephil added a commit that referenced this pull request Dec 8, 2014
@underscorephil
Merge pull request #453 from Neetuj/issue431
0cbcf3f 
adding credentials CLI commands for nas,vs,server
@underscorephil underscorephil merged commit 0cbcf3f into softlayer:master Dec 8, 2014
@underscorephil underscorephil mentioned this pull request Dec 9, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

CLI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Neetuj @sudorandom @underscorephil