-
Notifications
You must be signed in to change notification settings - Fork 752
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
panic: runtime error: invalid memory address or nil pointer dereference #59
Comments
thanks for reporting the issue. What OS and arch are you on ? |
@sdouche also share the docker-compose file so that we can recreate it. |
This look related to to env variables. |
@sdouche That is ok if you can't share it. environment:
FOO : $BAR |
Good hint @kadel I've a bunch of
|
Opps.. just notice that the issue log showing your info @Runseb |
yeah @ngtuna I noticed that, which is why I am thinking it is a build issue. dynamic linking of sorts ? |
@sdouche I tested your case with HEAD master including @surajssd 's patch a63d641, it worked. $ export DOCKER_REGISTRY="http://foo.bar/"
$ cat sdouche.yml
nginx:
image: ${DOCKER_REGISTRY}redis
$ kompose convert --file sdouche.yml
WARN[0000] [nginx] Service cannot be created because of missing port.
file "nginx-deployment.json" created
$ cat redis-deployment.json
...
"spec": {
"containers": [
{
"name": "redis",
"image": "http://foo.bar/redis",
"resources": {}
}
],
... So I assume you should be good with that. The next release is coming soon this week so you can either check the master now or stay a little bit for the release. |
@sdouche Check out the latest release: https://github.com/skippbox/kompose/releases/tag/v0.0.1-beta.2 I'm going to close this issue. Feel free to reopen if it can't solve your case as we can't test your private compose file :-). |
Tried with v0.0.1-beta.1:
The text was updated successfully, but these errors were encountered: