[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1769 ok, np closing for now --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 Hi @resmo, I've made some changes on the Docker file so it might not be possible to merge this PR in the current state. 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1769 @resmo now that PR by @pdion891 is merge, can this be closed? 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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 ok, I'll update the PR and merge it I think the PR go all required LGTM. I'll confirmed with John, in case he is in a code freeze period. --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1769 @pdion891 I would say merge #1435 first --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 @resmo could you do a PR on top of #1435 ? or this have to be merged first? so we can get your chances and some of mine. --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 So, should we go forward and merge #1435 + #1769 to master? I might just need to rebase and update version number of the current commit. With those 2 PR, we would have master back online on dockerhub so we can get an updater docker image for the simulator! --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user resmo commented on the issue: https://github.com/apache/cloudstack/pull/1769 I didn't test #1435 yet, but code looks good. The only difference I see is: ~~~ pip install cryptography --force-reinstall ~~~ And I used the mysql connector from MySQL.com but would prefer to take it from distribution (as in #1435) --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1769 Changes look good to me, but waiting for @pdion891 to tell us more :) --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1769 Could you look if it conflict with #1435 ? Because I made a pull request to automatically increment version in docker file when changing release of master via ```tools/build/setnextversion.sh```. --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1769: tools: fix Dockerfile
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1769 LGTM. @pdion891 would you like to review? --- 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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---