Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
tag:mergeready
---
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
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1862
yes, @sateesh-chodapuneedi those are addressed in another PR.
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Thanks @borisstoyanov
The 3 test failures listed in results doesn't seem related to the code
changes in PR.
```
2017-03-02 17:45:11,699 - CRITICAL -
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Trillian test result (tid-924)
Environment: vmware-60u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40322 seconds
Marvin logs:
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + vmware-60u2) has
been kicked to run smoke tests
---
If your project is set up for it, you can reply to this email and have your
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@blueorangutan test centos7 vmware-60u2
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@borisstoyanov unsupported parameters provided. Supported mgmt server os
are: `centos6, centos7, ubuntu`. Supported hypervisors are: `kvm-centos6,
kvm-centos7, kvm-ubuntu,
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@blueorangutan test centos7 vmware65
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Packaging result: âcentos6 âcentos7 âdebian. JID-557
---
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
Github user blueorangutan commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep
you posted as I make progress.
---
If your project is set up for it, you can reply to this email and have your
Github user borisstoyanov commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Sure @sateesh-chodapuneedi
@blueorangutan package
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@borisstoyanov @borisroman Can you please run trillian tests for this PR?
This already has enough code LGTMs, and needs tests for merge.
---
If your project is set up for it,
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@borisstoyanov Can you please run trillian tests for this PR?
This already has enough code LGTMs, and needs tests for merge.
---
If your project is set up for it, you can reply
Github user anshul1886 commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@sateesh-chodapuneedi code changes LGTM
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
ping @anshulgangwar @karuturi
---
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
Github user sureshanaparti commented on the issue:
https://github.com/apache/cloudstack/pull/1862
@sateesh-chodapuneedi Code changes LGTM
---
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
Github user sateesh-chodapuneedi commented on the issue:
https://github.com/apache/cloudstack/pull/1862
Thanks @sureshanaparti for review.
I have addressed that now. Pushed the code changes, please check.
---
If your project is set up for it, you can reply to this email and have
17 matches
Mail list logo