[GitHub] [cloudstack] blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-545231925 Trillian test result (tid-425) Environment: vmware-65u2 (x2), Advanced Networking with Mgm

[GitHub] [cloudstack] blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-545208697 Trillian test result (tid-426) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 26556 se

[GitHub] [cloudstack] blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-545132843 Trillian test result (tid-422) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 36375 seconds Ma

[GitHub] [cloudstack] ustcweizhou commented on issue #3597: kvm: Logrotate should not touch agent.log

2019-10-22 Thread GitBox
ustcweizhou commented on issue #3597: kvm: Logrotate should not touch agent.log URL: https://github.com/apache/cloudstack/pull/3597#issuecomment-545126565 @rhtyd can you cherry-pick this commit to 4.13 branch ? This is an auto

[cloudstack-documentation] branch remove-mysql-python-connector-requirements updated (2bb8f7f -> 4f7901d)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch remove-mysql-python-connector-requirements in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from 2bb8f7f remove mysql-python-connector requirements

[cloudstack-documentation] 01/01: remove mysql-python-connector requirements

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a commit to branch remove-mysql-python-connector-requirements in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git commit 2bb8f7f186998d1d9c41627060eeb4a897f3e8d3 Author: Andrija Pan

[cloudstack-documentation] branch remove-mysql-python-connector-requirements created (now 2bb8f7f)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch remove-mysql-python-connector-requirements in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. at 2bb8f7f remove mysql-python-connector requirements Thi

[GitHub] [cloudstack] rhtyd commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
rhtyd commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-545044778 @blueorangutan test This is an automated message from the Apache Git Service

[GitHub] [cloudstack] blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-545044892 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[cloudstack-documentation] 01/01: test

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a commit to branch test in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git commit 83aabff6a0d12307c4df78f6a957aef86e97c359 Author: Andrija Panic <45762285+andrijapani...@users.norep

[cloudstack-documentation] branch test created (now 83aabff)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch test in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. at 83aabff test This branch includes the following new commits: new 83aabff test The 1 r

[cloudstack-documentation] annotated tag 4.13.0.0 updated (701ee28 -> 14486f6)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to annotated tag 4.13.0.0 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. *** WARNING: tag 4.13.0.0 was modified! *** from 701ee28 (commit) to 14486f6 (ta

[cloudstack-documentation] branch volumes-from-snaps-doc-update updated (6c6b088 -> e9cd752)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch volumes-from-snaps-doc-update in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from 6c6b088 Update source/adminguide/storage.rst add e9cd752 Update

[cloudstack-documentation] branch volumes-from-snaps-doc-update updated (4c5fdd6 -> 6c6b088)

2019-10-22 Thread andrijapanic
This is an automated email from the ASF dual-hosted git repository. andrijapanic pushed a change to branch volumes-from-snaps-doc-update in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git. from 4c5fdd6 spell check add 6c6b088 Update source/adminguide/storag

[GitHub] [cloudstack] blueorangutan commented on issue #3632: add class cleanup method

2019-10-22 Thread GitBox
blueorangutan commented on issue #3632: add class cleanup method URL: https://github.com/apache/cloudstack/pull/3632#issuecomment-544926141 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-330 This is an automated message from

[GitHub] [cloudstack] blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-544922342 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-329 This is an aut

[GitHub] [cloudstack] blueorangutan commented on issue #3632: add class cleanup method

2019-10-22 Thread GitBox
blueorangutan commented on issue #3632: add class cleanup method URL: https://github.com/apache/cloudstack/pull/3632#issuecomment-544919603 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] [cloudstack] borisstoyanov commented on issue #3632: add class cleanup method

2019-10-22 Thread GitBox
borisstoyanov commented on issue #3632: add class cleanup method URL: https://github.com/apache/cloudstack/pull/3632#issuecomment-544919321 @blueorangutan package This is an automated message from the Apache Git Service. To re

[GitHub] [cloudstack] rhtyd commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
rhtyd commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-544914393 I don't have ceph setup to test this, maybe @andrijapanicsb can review? @blueorangutan package ---

[GitHub] [cloudstack] blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3605: fix issue #3590 'Revert Ceph/RBD Snapshot' URL: https://github.com/apache/cloudstack/pull/3605#issuecomment-544914494 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --

[GitHub] [cloudstack] rhtyd merged pull request #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
rhtyd merged pull request #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641 This is an automated message from the Apache Git Service. To respond to the messa

[cloudstack] branch master updated (9258c7c -> 848860d)

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 9258c7c Revert "9ee129ae6a08935645734ad520bec72bf5f22217" add 848860d security_group.py: fix NameError: nam

[GitHub] [cloudstack] rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641#issuecomment-544913932 Tested locally, the issue is pythonic iteration. LGTM. This

[GitHub] [cloudstack] rhtyd merged pull request #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
rhtyd merged pull request #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648 This is an automated message from the Apache Git Service. To respond to the messag

[cloudstack] branch 4.13 updated (cc27095 -> 3114907)

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch 4.13 in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from cc27095 server: Do NOT cleanup dhcp and dns when stop a vm (#3627) add 3114907 security_group: Add `head -1`

[GitHub] [cloudstack] rhtyd commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
rhtyd commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544913006 Thanks for confirming the change and tests. I'll merge this.

[GitHub] [cloudstack] blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544907552 @borisstoyanov a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + vmware65,

[GitHub] [cloudstack] borisstoyanov commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
borisstoyanov commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544907345 @blueorangutan test matrix

[GitHub] [cloudstack] blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544903312 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-328 ---

[GitHub] [cloudstack] borisstoyanov commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
borisstoyanov commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544896557 @blueorangutan package This

[GitHub] [cloudstack] blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
blueorangutan commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544896719 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I ma

[GitHub] [cloudstack] ustcweizhou commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR

2019-10-22 Thread GitBox
ustcweizhou commented on issue #3477: RvR: Set up metadata/password/dhcp server on gateway IP instead of guest IP in RVR URL: https://github.com/apache/cloudstack/pull/3477#issuecomment-544894194 @rhtyd can you kick off a test ? I have rebased with 4.13, and run some integration tests, i

[GitHub] [cloudstack] blueorangutan commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
blueorangutan commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641#issuecomment-544892922 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-326

[GitHub] [cloudstack] blueorangutan commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused

2019-10-22 Thread GitBox
blueorangutan commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused URL: https://github.com/apache/cloudstack/pull/3617#issuecomment-544892924 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-327

[GitHub] [cloudstack] ustcweizhou commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
ustcweizhou commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544892275 > state forwarding priority 32 cost 2 @RPDiep thanks . this change looks good to me.

[GitHub] [cloudstack] blueorangutan commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
blueorangutan commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544891731 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-325

[GitHub] [cloudstack] blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544880521 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests ---

[GitHub] [cloudstack] rhtyd commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
rhtyd commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544880302 @blueorangutan test This is an automated message from the Apache Git Service. To respo

[GitHub] [cloudstack] rhtyd commented on issue #3612: systemvm: for ip route show command don't use the throw command

2019-10-22 Thread GitBox
rhtyd commented on issue #3612: systemvm: for ip route show command don't use the throw command URL: https://github.com/apache/cloudstack/pull/3612#issuecomment-544879891 Can you review this @ustcweizhou ? Thanks. This is an

[GitHub] [cloudstack] rhtyd commented on issue #3525: NioServer: retain links by address string to minimize resource leak

2019-10-22 Thread GitBox
rhtyd commented on issue #3525: NioServer: retain links by address string to minimize resource leak URL: https://github.com/apache/cloudstack/pull/3525#issuecomment-544879554 Sorry, reverting this PR because I remembered why it was put On-Hold, as the change would cause a single Link to be

[cloudstack] branch master updated (9ee129a -> 9258c7c)

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 9ee129a NioServer: retain links by address string to minimize resource leak (#3525) add 9258c7c Revert "9e

[cloudstack] branch master updated (294813e -> 9ee129a)

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 294813e Merge remote-tracking branch 'origin/4.13' add 9ee129a NioServer: retain links by address string to

[GitHub] [cloudstack] blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544876701 Packaging result: ✖centos6 ✔centos7 ✔debian. JID-324 This is an automated mess

[GitHub] [cloudstack] rhtyd merged pull request #3525: NioServer: retain links by address string to minimize resource leak

2019-10-22 Thread GitBox
rhtyd merged pull request #3525: NioServer: retain links by address string to minimize resource leak URL: https://github.com/apache/cloudstack/pull/3525 This is an automated message from the Apache Git Service. To respond to

[GitHub] [cloudstack] rhtyd commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused

2019-10-22 Thread GitBox
rhtyd commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused URL: https://github.com/apache/cloudstack/pull/3617#issuecomment-544876227 @blueorangutan package This is an automated

[GitHub] [cloudstack] blueorangutan commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused

2019-10-22 Thread GitBox
blueorangutan commented on issue #3617: [KVM] Agent LB Fix: Connections from disabled KVM host agents are refused URL: https://github.com/apache/cloudstack/pull/3617#issuecomment-544876588 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] blueorangutan commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
blueorangutan commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641#issuecomment-544875447 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. ---

[GitHub] [cloudstack] rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641#issuecomment-544875155 @blueorangutan package This is an automated message from the

[GitHub] [cloudstack] rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined

2019-10-22 Thread GitBox
rhtyd commented on issue #3641: security_group.py: fix NameError: name 'd' is not defined URL: https://github.com/apache/cloudstack/pull/3641#issuecomment-544875056 The changes are in SG related file that trillian doesn't test by default, can you share any confirmation of testing @ustcweiz

[GitHub] [cloudstack] blueorangutan commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
blueorangutan commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544874707 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] [cloudstack] rhtyd commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
rhtyd commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544874560 @blueorangutan package This is an automated message from the

[cloudstack-primate] 01/02: CONTRIBUTING: fix new repo url

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-primate.git commit ae2c190d8827cd4516d5a20bdf17e0e53a7a6643 Author: Rohit Yadav AuthorDate: Tue Oct 22 14:12:39 2019 +0530

[cloudstack-primate] branch master updated (df58a20 -> 7047634)

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-primate.git. from df58a20 README: add voting/result history new ae2c190 CONTRIBUTING: fix new repo url new 70476

[cloudstack-primate] 02/02: api: fix sorting, checking of outstanding APIs to support

2019-10-22 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack-primate.git commit 70476340331ae88b2483edb3a20ba45ebcfdd5b8 Author: Rohit Yadav AuthorDate: Tue Oct 22 14:32:26 2019 +0530

[GitHub] [cloudstack] blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
blueorangutan commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544869412 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. -

[GitHub] [cloudstack] rhtyd commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
rhtyd commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544869058 Something went wrong at the backend, I'll rekick test. @blueorangutan package This

[GitHub] [cloudstack] RPDiep commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
RPDiep commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544865583 @ustcweizhou I'm sure this is something now on the hypervisor, but it appears that corner cases exist in which a seco

[GitHub] [cloudstack] GabrielBrascher commented on issue #3623: kvm: Use 'ip' instead of 'brctl'

2019-10-22 Thread GitBox
GabrielBrascher commented on issue #3623: kvm: Use 'ip' instead of 'brctl' URL: https://github.com/apache/cloudstack/pull/3623#issuecomment-544863217 This one looks ready to merge. @rhtyd Is there any issue with the **blueorangutan** test?

[GitHub] [cloudstack] ustcweizhou commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1

2019-10-22 Thread GitBox
ustcweizhou commented on issue #3648: Security Group: limit returns in get_bridge_physdev to 1 URL: https://github.com/apache/cloudstack/pull/3648#issuecomment-544838226 @RPDiep @GabrielBrascher I did not see this issue on all our platforms. we also use bonding on some platforms. it