[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1801 LGTM. Thanks @murali-reddy I'll merge 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 featu

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1801 test_router_dhcp_opts is passing, and we have necessary LGTM's. can we merge this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1801 Trillian test result (tid-510) Environment: vmware-55u3 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33387 seconds Marvin logs: https://github.com/blueoranguta

Re: [GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread Sergey Levitskiy
LGTM

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1801 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1801 @blueorangutan test centos7 vmware-55u3 --- 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 e

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1801 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-288 --- 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 d

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1801 I've kicked packaging and will test matrix this pr, LTGM on code 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

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1801 @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 re

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1801 @murali-reddy can you use a JIRA id in the git commit? --- 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

[GitHub] cloudstack issue #1801: fix marvin test failure test_router_dhcp_opts

2016-11-30 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1801 @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 not have this feature enabled a