[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-09 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1815 Thanks @rhtyd ! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-09 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1815 I've not verified this but I'll proceed with merging this based on review/test/feedback from @pdion891 @resmo and @sebgoa . --- If your project is set up for it, you can reply to this email and h

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-06 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1815 Thanks @rhtyd, let merge it so we have the simulator back online on docker hub and let's improve our Docker images... @jburwell, @rhtyd , do I merge it or you do it? I'm just wondering

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1815 Thanks, @pdion891 are you satisfied with the changes or you want to improve them. I'm LGTM on this, can merge on your conclusion. Thanks. --- If your project is set up for it, you can reply to th

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread resmo
Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1815 IMHO this is not meant for production, but quick fire up a docker having the current state of development. The image will be huge but for non-devs (like me) a easy way to get a running devel cloud

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1815 @sebgoa , I agree with you, it's just the easiest way to have a cloudstack with simulator preconfigured ready to work. We should not use package for the image management also. I've mad

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread sebgoa
Github user sebgoa commented on the issue: https://github.com/apache/cloudstack/pull/1815 LGTM but really the database should be outside the mgt server container and we should not be using supervisord. --- If your project is set up for it, you can reply to this email and hav

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1815 @pdion891 @resmo @swill @sebgoa can you help review, test this? thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your proj

[GitHub] cloudstack issue #1815: Fix docker image build for cloudstack-management

2016-12-05 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1815 @pdion891 thanks, how may I test this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature en