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

go panic when converting hygieia docker-compose #173

Closed
raffaelespazzoli opened this issue Sep 27, 2016 · 3 comments
Closed

go panic when converting hygieia docker-compose #173

raffaelespazzoli opened this issue Sep 27, 2016 · 3 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@raffaelespazzoli
Copy link

the file can be found here:
https://raw.githubusercontent.com/capitalone/Hygieia/master/docker-compose.yml

@sebgoa
Copy link
Contributor

sebgoa commented Sep 27, 2016

@raffaelespazzoli can you copy the error you are getting in this issue.
thanks

@ngtuna
Copy link
Contributor

ngtuna commented Sep 27, 2016

@raffaelespazzoli : The panic comes from libcompose parsing function.

$ kompose --file hygieia.yml convert 
WARN[0000] Unsupported key volume_driver - ignoring     
panic: runtime error: index out of range

goroutine 1 [running]:
panic(0x15b1de0, 0xc82000e170)
    /usr/local/go/src/runtime/panic.go:481 +0x3e6
github.com/skippbox/kompose/vendor/github.com/docker/libcompose/yaml.toMap(0xc8203a6720, 0x3, 0x3, 0x1814c78, 0x1, 0x40)
    /home/tuna/workspace/gocode-kompose/src/github.com/skippbox/kompose/vendor/github.com/docker/libcompose/yaml/types_yaml.go:223 +0x187
github.com/skippbox/kompose/vendor/github.com/docker/libcompose/yaml.(*MaporEqualSlice).ToMap(0xc8201b92d0, 0xc8201b9180)
    /home/tuna/workspace/gocode-kompose/src/github.com/skippbox/kompose/vendor/github.com/docker/libcompose/yaml/types_yaml.go:126 +0x4b
github.com/skippbox/kompose/pkg/loader/compose.(*Compose).LoadFile(0x24e87f0, 0xc8204cb150, 0xb, 0x0)

@sebgoa I also tested on HEAD's libcompose, it worked fine. It's another sample motivating us to upgrade libcompose. Like this one: #149

@ngtuna ngtuna added the bug label Sep 27, 2016
@kadel kadel added kind/bug Categorizes issue or PR as related to a bug. and removed bug labels Oct 19, 2016
@ngtuna ngtuna closed this as completed Oct 22, 2016
@ngtuna
Copy link
Contributor

ngtuna commented Oct 22, 2016

@raffaelespazzoli you got it passed on master branch now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

4 participants