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

Output option with convert -c fails. #886

Closed
jmatsushita opened this issue Dec 4, 2017 · 5 comments · Fixed by #938
Closed

Output option with convert -c fails. #886

jmatsushita opened this issue Dec 4, 2017 · 5 comments · Fixed by #938

Comments

@jmatsushita
Copy link

Hi there,

When using kompose convert -c with the -o option, the charts are created in the desired folder but the last step fails:

> kompose convert -c -o chart
kompose convert -c -o chart
INFO Kubernetes file "chart/service-service.yaml" created
INFO Kubernetes file "chart/service-deployment.yaml" created
WARN open docker-compose/templates/chart/service-service.yaml: no such file or directory
WARN open docker-compose/templates/chart/service-deployment.yaml: no such file or directory
INFO chart created in "./docker-compose/"

The resulting docker-compose folder lacks the template files:

> tree docker-compose
docker-compose
├── Chart.yaml
├── README.md
└── templates

It would be nice to be able to choose the name of the outputting directory rather than it being hardcoded to docker-compose.

Cheers,

Jun

@cdrage
Copy link
Member

cdrage commented Dec 5, 2017

Honestly, charts are bit broken at the moment. Thanks for filing the issue! Hopefully it's just a simple CLI option fixed.

@jmatsushita
Copy link
Author

Thanks for update! Can you tell me more about how they are broken? I'm looking into using kompose for this, so it'd be good to have a heads up! 😄

@cdrage
Copy link
Member

cdrage commented Dec 5, 2017

@jmatsushita Ah, let me correct myself. There's not broken per-say, just a bit disorganized. Helm has changed quite a bit since the implementation was added (such as extra variables). Kompose will work fine converting to Helm no-problem, it's more.. properly naming the files, setting up comments and proper versioning within the Helm chart.

If you don't mind me asking, what's your main use-case?

I'm assuming:

Docker Compose (dev environment)
Docker Compose file -> Helm Chart (provisioning)
Then Kubernetes for deployment?

@jmatsushita
Copy link
Author

Yes that's exactly the main use case. Glad to hear that the chart problem is mostly cosmetic :)

@hangyan
Copy link
Contributor

hangyan commented Feb 8, 2018

I will take a look

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.

3 participants