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

NetworkId parameter in list-ip-adresses #85

Closed
inge4pres opened this issue Nov 11, 2016 · 7 comments
Closed

NetworkId parameter in list-ip-adresses #85

inge4pres opened this issue Nov 11, 2016 · 7 comments
Assignees
Labels
Milestone

Comments

@inge4pres
Copy link

Hi,

after successful login

$ ~/bin/clc.exe login --user user.name --password sup3rs3cretPWD
Logged in as user.name.

I try to get the IPs in a network and get such error

$ ~/bin/clc.exe network list-ip-addresses --data-center VA1 --network-id xxxxxxxzzzzzzzzyyyyy1234
Unknown option or argument: `NetworkId`.
@inge4pres
Copy link
Author

Same goes with NetworkName

$ ~/bin/clc.exe network list-ip-addresses --data-center VA1 --network-name "NET_NAME"
Unknown option or argument: `NetworkName`.

@chrislittle
Copy link
Contributor

This looks like a bug

@richardcase I can't get this to work either. can you check the code?

if I use --network (instead of network-name or network-id) it actually goes forward and wants a --type (which is supposed to be optional). even using that then I get a 404.

Error occured while sending request to API. Status code: 404. Server says: The requested resource does not exist.

@inge4pres you have the logic right in my view.

@richardcase
Copy link
Contributor

Sure, I'll check the code and let you both know.

@richardcase richardcase self-assigned this Nov 11, 2016
@richardcase
Copy link
Contributor

@inge4pres & @chrislittle : i've had a look at the code and i can see there is a bug. I am in the process of migrating to a new laptop but i will get it fixed asap and will let you both know.

@inge4pres
Copy link
Author

I'm on v1.1.0-rc.1

@chrislittle
Copy link
Contributor

thanks @inge4pres as @richardcase noted he's going to dig into the code and fix this up. if you find anything else let us know as we appreciate the feedback

richardcase added a commit to richardcase/clc-go-cli that referenced this issue Nov 12, 2016
Contains a fix for the list-ip-adresses command where the command
was ignoring the --network-id and --network-name parameters. The request
model was changed to include the Network model which allows infering off
network-id if network-name was supplied. Also fixed an issue with the
type parameter as it should be optional.

Resolves: CenturyLinkCloud#85

Signed-off-by: Richard Case <richard.case@outlook.com>
@richardcase richardcase added this to the v1.1.0 milestone Nov 12, 2016
@richardcase
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants