[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-18 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu thanks, can you change PR's base branch to 4.9. --- 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 h

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 Trillian test result (tid-667) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 34721 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-12 Thread murali-reddy
Github user murali-reddy commented on the issue: https://github.com/apache/cloudstack/pull/1783 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 feature enabled and wishes so, or if

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-12 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 @borisstoyanov a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-12 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @blueorangutan test matrix --- 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 enable

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-390 --- 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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-11 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-11 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 an

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-11 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1783 @rhtyd It is already rebased on 4.9 and also added changes for @ustcweizhou comments --- 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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu can you change PR's base branch to 4.9 as well? How blocking/critical is this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1783 @murali-reddy can you help review 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 project does not have this

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-363 --- 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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 an

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1783 LGTM we have the same change in our productions. --- 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

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu can you please look into the failures if you can fix them and push again. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 Packaging result: ✖centos6 ✔centos7 ✖debian. JID-354 --- 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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @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 an

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-08 Thread jayapalu
Github user jayapalu commented on the issue: https://github.com/apache/cloudstack/pull/1783 rebased it on 4.9 --- 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 s

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-12-07 Thread borisstoyanov
Github user borisstoyanov commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu could you please rebase this branch against 4.9 so we could trigger smoke tests? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-11-27 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu can you rebase against 4.9 and change PR's base branch to 4.9? Thanks. @murali-reddy can you help review this? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1783: CLOUDSTACK-9615: Fixd applying ingress rules without...

2016-11-25 Thread ustcweizhou
Github user ustcweizhou commented on the issue: https://github.com/apache/cloudstack/pull/1783 @jayapalu line 179 should be changed as well, same to line 143 --- 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