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

Bootscript parameter seems broken #387

Closed
kudos opened this issue Jul 27, 2016 · 8 comments · Fixed by #388
Closed

Bootscript parameter seems broken #387

kudos opened this issue Jul 27, 2016 · 8 comments · Fixed by #388
Assignees
Milestone

Comments

@kudos
Copy link

kudos commented Jul 27, 2016

Not sure how to get better error messaging from the cli either.

❯ ~ scw create --bootscript=docker xenial
FATA[0000] cannot execute 'create':   StatusCode:                    %!s(int=400)  Type:                          invalid_request_error  Message:                         APIMessage:                    Validation Error
❯ ~ scw create xenial
f14c1201-b064-4878-80c3-30944d95e892
@QuentinPerez
Copy link
Contributor

QuentinPerez commented Jul 27, 2016

Hi @kudos,

Can you give me your version with scw version

@kudos
Copy link
Author

kudos commented Jul 27, 2016

❯ ~ scw version
Client version: v1.9.0+dev
Go version (client): go1.6.2
Git commit (client): homebrew
OS/Arch (client): darwin/amd64

@QuentinPerez
Copy link
Contributor

Can you retry with the version on master branch ? We will release a new version soon

@kudos
Copy link
Author

kudos commented Jul 27, 2016

I presume --HEAD gets me head on master?

❯ ~ brew uninstall scw
Uninstalling /usr/local/Cellar/scw/HEAD... (4 files, 11.9M)
❯ ~ brew install scaleway/scaleway/scw --HEAD
==> Installing scw from scaleway/scaleway
==> Cloning https://github.com/scaleway/scaleway-cli.git
Updating /Users/jonathan/Library/Caches/Homebrew/scw--git
==> Checking out branch master
==> go build -o /usr/local/Cellar/scw/HEAD-7492b3d/bin/scw -v -ldflags -X  github.com/scaleway/scaleway-cli/pkg/scwversion.GITCOMMIT=homebrew github.com/scaleway/scaleway-cli/cm
==> Caveats
Use `scw login` to set up the correct environment to use scaleway-cli.

Bash completion has been installed to:
  /usr/local/etc/bash_completion.d

zsh completion has been installed to:
  /usr/local/share/zsh/site-functions
==> Summary
🍺  /usr/local/Cellar/scw/HEAD-7492b3d: 4 files, 11.9M, built in 9 seconds
❯ ~ scw version
Client version: v1.9.0+dev
Go version (client): go1.6.3
Git commit (client): homebrew
OS/Arch (client): darwin/amd64
❯ ~ scw create --bootscript=docker xenial
FATA[0001] cannot execute 'create': Type: invalid_request_error Message:  APIMessage: Validation Error StatusCode: 400

@QuentinPerez
Copy link
Contributor

I reproduce the bug, I will try to fix the bootscript ASAP.

Thank you for your feedback, really appreciate 😊 👍🏻

@QuentinPerez QuentinPerez self-assigned this Jul 27, 2016
@QuentinPerez QuentinPerez added this to the 1.10.0 milestone Jul 27, 2016
@kudos
Copy link
Author

kudos commented Jul 27, 2016

Apart from fixing the issue, it would be great if the cli surfaced the error messages better. I went as far as running API calls directly but couldn't reproduce.

@QuentinPerez
Copy link
Contributor

QuentinPerez commented Jul 27, 2016

I added the details in the error message LGTY ?

./scw run --bootscript=docker xenial
FATA[0000] cannot execute 'run': failed to create server: StatusCode: 400, Type: invalid_request_error, APIMessage: Validation Error, Details: map[bootscript:[not a valid value] volumes:[required key not provided]]

@kudos
Copy link
Author

kudos commented Jul 27, 2016

LGTM, thanks!

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 a pull request may close this issue.

2 participants