[GitHub] blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-11-30 Thread GitBox
blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-348419587 Trillian test result (tid-1718) Environment: xenserver-72 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-11-30 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-348383872 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check

2017-11-30 Thread GitBox
rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-348383774 @blueorangutan test This is an automated message from the Apache Git Service. To

[GitHub] blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases

2017-11-30 Thread GitBox
blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases URL: https://github.com/apache/cloudstack/pull/2326#issuecomment-348382404 Trillian test result (tid-1717) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-11-30 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-348341023 Packaging result: ?centos6 ?centos7 ?debian. JID-1331 This is an

[GitHub] blueorangutan commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ? URL: https://github.com/apache/cloudstack/pull/2088#issuecomment-348337754 Packaging result: ?centos6 ?centos7 ?debian. JID-1330

[GitHub] rafaelweingartner commented on issue #2283: CLOUDSTACK-10105: Maven standard

2017-11-30 Thread GitBox
rafaelweingartner commented on issue #2283: CLOUDSTACK-10105: Maven standard URL: https://github.com/apache/cloudstack/pull/2283#issuecomment-348332862 @rhtyd, @marcaurele, and @khos2ow I have been thinking about this PR and #2343, which focus on changing ACS code base to use current

[GitHub] blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check

2017-11-30 Thread GitBox
blueorangutan commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-348329920 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check

2017-11-30 Thread GitBox
rhtyd commented on issue #2289: [4.11/master] Smoketest Health Check URL: https://github.com/apache/cloudstack/pull/2289#issuecomment-348329638 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] blueorangutan commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type

2017-11-30 Thread GitBox
blueorangutan commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type URL: https://github.com/apache/cloudstack/pull/2128#issuecomment-348326385 Packaging result: ?centos6 ?centos7 ?debian. JID-1329

[GitHub] rhtyd commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ?

2017-11-30 Thread GitBox
rhtyd commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ? URL: https://github.com/apache/cloudstack/pull/2088#issuecomment-348325217 @blueorangutan package This is an automated

[GitHub] blueorangutan commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2088: CLOUDSTACK-9892: Primary storage resource check is broken when using ? URL: https://github.com/apache/cloudstack/pull/2088#issuecomment-348325365 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type

2017-11-30 Thread GitBox
blueorangutan commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type URL: https://github.com/apache/cloudstack/pull/2128#issuecomment-348324580 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type

2017-11-30 Thread GitBox
rhtyd commented on issue #2128: CLOUDSTACK-9885: VPCVR: Updated to the private the traffic_type URL: https://github.com/apache/cloudstack/pull/2128#issuecomment-348324514 @blueorangutan package This is an automated message

[GitHub] rhtyd closed pull request #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
rhtyd closed pull request #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[cloudstack] branch master updated: CLOUDSTACK-7793: Create Snaphot with quiesce option set to true fails with InvalidParameterValueException (#2312)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new ed6e3be CLOUDSTACK-7793: Create Snaphot

[GitHub] rhtyd commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
rhtyd commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348322448 Ignoring intermittent failures, lgtm. Merging based on test results and additional manual testing.

[GitHub] rhtyd closed pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rhtyd closed pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[cloudstack] branch master updated: CLOUDSTACK-10164: allow users to create a VPC through the UI. (#2345)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 8f35657 CLOUDSTACK-10164: allow users to

[GitHub] blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348321453 Trillian test result (tid-1713) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rhtyd commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rhtyd commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#issuecomment-348321203 Lgtm, test lgtm as well. I was able to reproduce the issue on latest master. Merging based on nuage plugin tests

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348319155 Trillian test result (tid-1715) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total

[GitHub] khos2ow commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-11-30 Thread GitBox
khos2ow commented on issue #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347#issuecomment-348308088 @rafaelweingartner would you mind review again?

[GitHub] blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-11-30 Thread GitBox
blueorangutan commented on issue #2211: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9 URL: https://github.com/apache/cloudstack/pull/2211#issuecomment-348304871 Packaging result: ?centos6 ?centos7 ?debian. JID-1328

[GitHub] rafaelweingartner opened a new pull request #2348: Remove ejb-api 3.0 dependency

2017-11-30 Thread GitBox
rafaelweingartner opened a new pull request #2348: Remove ejb-api 3.0 dependency URL: https://github.com/apache/cloudstack/pull/2348 This PR is proposing the removal of "ejb-api" 3.0 JAR. We use spring to manage objects creation and injection. Thus, spring beans do not need the annotation

[cloudstack] 10/12: fix thead and tbody in separate table, causes tables to appear weird

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit f5da1d42bd8c4754a56e8aa2e7c91c72eb2e8ab6 Author: Rohit Yadav

[cloudstack] 04/12: CLOUDSTACK-10129: UX improvements and event timeline

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 323d294cf6cda0bf5b919700a754fae811fe0239 Author: Rohit Yadav

[cloudstack] 09/12: fix nio regression

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 021b6c5262cc0f0826be27be6a59d66bf4b4f37c Author: Rohit Yadav

[cloudstack] branch debian9-systemvmtemplate updated (6b061b5 -> b502b79)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a change to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git. discard 6b061b5 fix failures seen in ssvm discard a57c504 use a holder class to pass buffers,

[cloudstack] 02/12: CLOUDSTACK-10013: Make Debian9 systemvmtemplates work with systemd

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 09af7eb571d2f046fc7ad3ea5484820689e1c423 Author: Wido den Hollander AuthorDate:

[cloudstack] 06/12: remove inserv, rearrange pkgs

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 58d184063c0f8f9e7369bc628ecd65147b478125 Author: Rohit Yadav

[cloudstack] 01/12: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit b1b084a5a71cd526776323287c781fddd0d1d7cc Author: Rohit Yadav

[cloudstack] 05/12: wip- make redundant routers work again

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 540a5d96c2c91d0ef1349e11fede53f144be1aad Author: Rohit Yadav

[cloudstack] 12/12: fix failures seen in ssvm

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit b502b7987fee3f9d57379206f05d4d804e67c869 Author: Rohit Yadav

[cloudstack] 07/12: remove old dead code/files

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit eba9c5a8d1fc8b81a64ee2d719324607c5d5ce98 Author: Rohit Yadav

[cloudstack] 11/12: use a holder class to pass buffers, fixes potential leak when objects are changed

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit b460b1d9ad76298b39c43011665d6089b04d4224 Author: Rohit Yadav

[cloudstack] branch debian9-systemvmtemplate updated (8d91971 -> 6b061b5)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a change to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git. from 8d91971 fix thead and tbody in separate table, causes tables to appear weird new

[cloudstack] 02/02: fix failures seen in ssvm

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 6b061b5539810f4f83485742ae750372831273c7 Author: Rohit Yadav

[cloudstack] 01/02: use a holder class to pass buffers, fixes potential leak when objects are changed

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit a57c504d427a0e22ccc08bd63d3113260b7250a4 Author: Rohit Yadav

[GitHub] khos2ow closed pull request #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-11-30 Thread GitBox
khos2ow closed pull request #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] khos2ow opened a new pull request #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user

2017-11-30 Thread GitBox
khos2ow opened a new pull request #2347: CLOUDSTACK-10166: Get accountId and domainId from VPC when tagging a NetworkACL as a user URL: https://github.com/apache/cloudstack/pull/2347 This fixes the issue that a non-root user cannot tag a network ACL item and after the fix a non-root

[GitHub] smeetsr commented on issue #2259: CLOUDSTACK-10024: Network migration support

2017-11-30 Thread GitBox
smeetsr commented on issue #2259: CLOUDSTACK-10024: Network migration support URL: https://github.com/apache/cloudstack/pull/2259#issuecomment-348217391 Verify Migration for an isolated network without VMs ... === TestName: test_01_native_to_nuage_network_migration_novms | Status : SUCCESS

[GitHub] svenvogel commented on issue #2333: Add Java Default Certificat Authorities into the keystore if using a custom cert SSL

2017-11-30 Thread GitBox
svenvogel commented on issue #2333: Add Java Default Certificat Authorities into the keystore if using a custom cert SSL URL: https://github.com/apache/cloudstack/pull/2333#issuecomment-348185270 @rhtyd @rafaelweingartner changed to 4.10

[GitHub] svenvogel commented on issue #2333: Add Java Default Certificat Authorities into the keystore if using a custom cert SSL

2017-11-30 Thread GitBox
svenvogel commented on issue #2333: Add Java Default Certificat Authorities into the keystore if using a custom cert SSL URL: https://github.com/apache/cloudstack/pull/2333#issuecomment-348185270 @rafaelweingartner changed to 4.10

[cloudstack] 01/01: fix thead and tbody in separate table, causes tables to appear weird

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 8d919718c75b22db8d6f6d7c2163baab6ef7c198 Author: Rohit Yadav

[cloudstack] branch debian9-systemvmtemplate updated (267db62 -> 8d91971)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a change to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git. discard 267db62 fix thead and tbody in separate table, causes tables to appear weird new

[cloudstack] branch debian9-systemvmtemplate updated: fix thead and tbody in separate table, causes tables to appear weird

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/debian9-systemvmtemplate by this push: new

[cloudstack] branch debian9-systemvmtemplate updated: fix nio regression

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/debian9-systemvmtemplate by this push: new

[cloudstack] branch master updated: CLOUDSTACK-10070: Fixing some component tests and adding them in travis (#2263)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new ef4adb3 CLOUDSTACK-10070: Fixing some

[GitHub] rhtyd closed pull request #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-11-30 Thread GitBox
rhtyd closed pull request #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-11-30 Thread GitBox
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-348159488 Travis is green, merging now. This is an

[GitHub] rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-11-30 Thread GitBox
rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#issuecomment-348156178 @yvsubhash sure I'll include this in next round of testing. Included in 4.11 milestone for considering.

[GitHub] blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-11-30 Thread GitBox
blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-348151176 Packaging result: ?centos6 ?centos7 ?debian. JID-1327 This

[GitHub] yvsubhash commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-11-30 Thread GitBox
yvsubhash commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#issuecomment-348148416 @rhtyd Previous approach was retrying the transactions in case of deadlocks. Now the root causes of

[GitHub] rafaelweingartner commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rafaelweingartner commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#issuecomment-348147515 Got it. I asked that because I noticed that we have an Ajax inside a success function of another

[GitHub] sgoeminn commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
sgoeminn commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#issuecomment-348146292 @rafaelweingartner Thanks, I agree it would not be necessary if there is no nuage zone, however to find that

[GitHub] blueorangutan commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped

2017-11-30 Thread GitBox
blueorangutan commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped URL: https://github.com/apache/cloudstack/pull/2308#issuecomment-348145120 Packaging result: ?centos6 ?centos7 ?debian. JID-1326

[GitHub] rafaelweingartner commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rafaelweingartner commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#issuecomment-348145031 Thanks for the details! What do you think about calling the `listNuageVspDomainTemplateCmd `

[GitHub] rafaelweingartner commented on a change in pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#discussion_r154034011 ## File path:

[GitHub] rafaelweingartner commented on a change in pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
rafaelweingartner commented on a change in pull request #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#discussion_r154034011 ## File path:

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348138674 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348138648 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases

2017-11-30 Thread GitBox
blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases URL: https://github.com/apache/cloudstack/pull/2326#issuecomment-348138658 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348138409 Ignoring centos6 failure that was intermittent pkging failure @blueorangutan test

[GitHub] blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-11-30 Thread GitBox
blueorangutan commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-348137017 @PaulAngus a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] PaulAngus commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility

2017-11-30 Thread GitBox
PaulAngus commented on issue #2346: WIP - Add XenServer 7.1 and 7.2 interoperablility URL: https://github.com/apache/cloudstack/pull/2346#issuecomment-348136974 @blueorangutan package This is an automated message from

[GitHub] rafaelweingartner commented on issue #2343: [WIP] CLOUDSTACK-10161: Use Maven wrapper

2017-11-30 Thread GitBox
rafaelweingartner commented on issue #2343: [WIP] CLOUDSTACK-10161: Use Maven wrapper URL: https://github.com/apache/cloudstack/pull/2343#issuecomment-348136050 @khos2ow that is very interesting. They are indeed versioning this jar file. I always had Spring projects as an example of

[GitHub] blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348135043 Packaging result: ?centos6 ?centos7 ?debian. JID-1325

[GitHub] blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases

2017-11-30 Thread GitBox
blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases URL: https://github.com/apache/cloudstack/pull/2326#issuecomment-348135042 Packaging result: ?centos6 ?centos7 ?debian. JID-1324

[GitHub] rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis

2017-11-30 Thread GitBox
rhtyd commented on issue #2263: CLOUDSTACK-10070: Fixing some component tests and adding them in travis URL: https://github.com/apache/cloudstack/pull/2263#issuecomment-348133137 Tested and applied a fix manually: Marvin Init Successful === TestName: test_01_zones | Status

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348131558 Packaging result: ?centos6 ?centos7 ?debian. JID-1323

[GitHub] sgoeminn commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI.

2017-11-30 Thread GitBox
sgoeminn commented on issue #2345: CLOUDSTACK-10164: bugfix allow users to create a VPC through the UI. URL: https://github.com/apache/cloudstack/pull/2345#issuecomment-348124549 @rafaelweingartner The problem only occurs in current master. So 4.10 is not affected. To my knowledge all

[GitHub] rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-11-30 Thread GitBox
rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-348123079 @mike-tutkowski thanks for replying, looking forward to your changes. Once you're done, please ping people on this PR and ask

[GitHub] rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage

2017-11-30 Thread GitBox
rhtyd commented on issue #2298: CLOUDSTACK-9620: Enhancements for managed storage URL: https://github.com/apache/cloudstack/pull/2298#issuecomment-348123079 @mike-tutkowski thanks for replying, looking forward to your changes. Once you're done, please ping people on this PR and ask

[GitHub] rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab?

2017-11-30 Thread GitBox
rhtyd commented on issue #1762: CLOUDSTACK-9595 Transactions are not getting retried in case of datab? URL: https://github.com/apache/cloudstack/pull/1762#issuecomment-348122868 @yvsubhash can you comment what was changed since your initial change which got received objections. I see

[cloudstack] 05/08: wip- make redundant routers work again

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 9037cc2cd1e3d7dda7fac71e39cd3c9bc078b0c8 Author: Rohit Yadav

[cloudstack] 04/08: CLOUDSTACK-10129: UX improvements and event timeline

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 893c5344780c227b60b2b6b82508dca432681f51 Author: Rohit Yadav

[cloudstack] 06/08: remove inserv, rearrange pkgs

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit edbf6b3b1ae43f32fa17e28a4445f2d1c01b4317 Author: Rohit Yadav

[cloudstack] 01/08: CLOUDSTACK-10013: Migrate systemvmtemplate to Debian9

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit e61e4ca18c6cdae4b2c43133d33d1d33c6f9c34b Author: Rohit Yadav

[cloudstack] 02/08: CLOUDSTACK-10013: Make Debian9 systemvmtemplates work with systemd

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit dfb9a67da7d8de0b05ac14360e991b79b9e5a59e Author: Wido den Hollander AuthorDate:

[cloudstack] branch debian9-systemvmtemplate updated (d4ad866 -> 37c61ed)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a change to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git. omit d4ad866 refactor files, remove old cruft omit ff94807 remove old dead code/files

[cloudstack] 07/08: remove old dead code/files

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch debian9-systemvmtemplate in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit aff48ad46f549c2065e1760d5cc96a733e1faeaf Author: Rohit Yadav

[GitHub] rhtyd commented on issue #2299: CLOUDSTACK-10118 Domain level settings

2017-11-30 Thread GitBox
rhtyd commented on issue #2299: CLOUDSTACK-10118 Domain level settings URL: https://github.com/apache/cloudstack/pull/2299#issuecomment-348120152 @DaanHoogland can you rebase this and advise if this is ready for review/testing?

[GitHub] blueorangutan commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped

2017-11-30 Thread GitBox
blueorangutan commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped URL: https://github.com/apache/cloudstack/pull/2308#issuecomment-348120143 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[GitHub] rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser

2017-11-30 Thread GitBox
rhtyd commented on issue #2301: CLOUDSTACK-10121 moveUser URL: https://github.com/apache/cloudstack/pull/2301#issuecomment-348120025 @DaanHoogland can you rebase and advise if this is ready for review/test/merging? This is

[GitHub] blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
blueorangutan commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348119917 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped

2017-11-30 Thread GitBox
rhtyd commented on issue #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped URL: https://github.com/apache/cloudstack/pull/2308#issuecomment-348119911 @blueorangutan package This is an

[GitHub] rhtyd commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ?

2017-11-30 Thread GitBox
rhtyd commented on issue #2312: CLOUDSTACK-7793 Create Snaphot with quiesce option set to true fails ? URL: https://github.com/apache/cloudstack/pull/2312#issuecomment-348119766 @yvsubhash we still need to run regression tests and @jahnaviboddu has not shared the test results, how it was

[GitHub] blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases

2017-11-30 Thread GitBox
blueorangutan commented on issue #2326: Re-work method QuotaResponseBuilderImpl.startOfNextDay and its test cases URL: https://github.com/apache/cloudstack/pull/2326#issuecomment-348119470 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd closed pull request #2342: CLOUDSTACK-9586: Adding local storage test

2017-11-30 Thread GitBox
rhtyd closed pull request #2342: CLOUDSTACK-9586: Adding local storage test URL: https://github.com/apache/cloudstack/pull/2342 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[cloudstack] branch master updated: CLOUDSTACK-9586: Adding local storage test (#2342)

2017-11-30 Thread bhaisaab
This is an automated email from the ASF dual-hosted git repository. bhaisaab pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new 919d298 CLOUDSTACK-9586: Adding local

[GitHub] rhtyd commented on issue #2342: CLOUDSTACK-9586: Adding local storage test

2017-11-30 Thread GitBox
rhtyd commented on issue #2342: CLOUDSTACK-9586: Adding local storage test URL: https://github.com/apache/cloudstack/pull/2342#issuecomment-348118936 LGTM, given your manual test output I'll merge this now. This is an

[GitHub] rhtyd commented on issue #2342: CLOUDSTACK-9586: Adding local storage test

2017-11-30 Thread GitBox
rhtyd commented on issue #2342: CLOUDSTACK-9586: Adding local storage test URL: https://github.com/apache/cloudstack/pull/2342#issuecomment-348118580 LGTM. This is an automated message from the Apache Git Service. To respond

[GitHub] rhtyd commented on a change in pull request #2342: CLOUDSTACK-9586: Adding local storage test

2017-11-30 Thread GitBox
rhtyd commented on a change in pull request #2342: CLOUDSTACK-9586: Adding local storage test URL: https://github.com/apache/cloudstack/pull/2342#discussion_r154011322 ## File path: test/integration/component/test_secsr_mount.py ## @@ -0,0 +1,170 @@ +# Licensed to the

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348117874 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-347537786 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
blueorangutan commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-347532787 Packaging result: ?centos6 ?centos7 ?debian. JID-1311

[GitHub] rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-348117789 @blueorangutan package This is an automated message from

[GitHub] rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat

2017-11-30 Thread GitBox
rhtyd commented on issue #2304: CLOUDSTACK-10127: KVM + Ovs: Incorrect devId on static nat URL: https://github.com/apache/cloudstack/pull/2304#issuecomment-347537489 @blueorangutan test This is an automated message from the

[GitHub] borisstoyanov commented on a change in pull request #2342: CLOUDSTACK-9586: Adding local storage test

2017-11-30 Thread GitBox
borisstoyanov commented on a change in pull request #2342: CLOUDSTACK-9586: Adding local storage test URL: https://github.com/apache/cloudstack/pull/2342#discussion_r154009949 ## File path: test/integration/component/test_secsr_mount.py ## @@ -0,0 +1,170 @@ +# Licensed to

  1   2   >