[GitHub] borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-342402783 @blueorangutan test This is an automated

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-342402712 Packaging result: ?centos6 ?centos7 ?debian. JID-1214

[GitHub] blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

blueorangutan commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-342398269 @borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-342398140 @blueorangutan package This is an automated

[GitHub] borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

borisstoyanov commented on issue #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309#issuecomment-342398287 @nvazquez thanks for this PR, I'll run marvin tests

[GitHub] blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342383079 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342382960 Okay @jayapalu rekicking tests. @blueorangutan test

[GitHub] jayapalu commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

jayapalu commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342377225 @rhtyd @mrunalinikankariya Test failures are not related these changes. If we look at the changes there are

[GitHub] rhtyd commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API

rhtyd commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API URL: https://github.com/apache/cloudstack/pull/2242#issuecomment-342371587 Tests LGTM, no new regressions seen. This is an

[cloudstack] branch master updated: CLOUDSTACK-9958: Include tags of resources in listUsageRecords API (#2242)

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 32807cb CLOUDSTACK-9958: Include tags of

[GitHub] rhtyd closed pull request #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API

rhtyd closed pull request #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API URL: https://github.com/apache/cloudstack/pull/2242 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 #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342371413 @jayapalu @mrunalinikankariya looks like there are several regression failures, this cannot be accepted as

[GitHub] rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342371413 @jayapalu @mrunalinikankariya looks like there are several regression failures, this cannot be accepted as

[GitHub] jayapalu commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

jayapalu commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342370780 @rhtyd We can add list of allowed/supported protocols in the port forwarding capabilities rest will be

[cloudstack] branch master updated: CLOUDSTACK-8599 [VMware] Successful migration was reported as failure when vCenter session timed out

This is an automated email from the ASF dual-hosted git repository. sateesh 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 c5692ce CLOUDSTACK-8599 [VMware]

[GitHub] sateesh-chodapuneedi closed pull request #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out

sateesh-chodapuneedi closed pull request #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out URL: https://github.com/apache/cloudstack/pull/2092 This is a PR merged from a forked repository. As GitHub hides the original diff on

[GitHub] sateesh-chodapuneedi commented on issue #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out

sateesh-chodapuneedi commented on issue #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out URL: https://github.com/apache/cloudstack/pull/2092#issuecomment-342369828 Merging this based on 2 LGTMs and test results confirmation from

[GitHub] nvazquez opened a new pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents

nvazquez opened a new pull request #2309: CLOUDSTACK-10132: Multiple Management Servers Support for agents URL: https://github.com/apache/cloudstack/pull/2309 JIRA Ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-10132

[GitHub] resmo commented on issue #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out

resmo commented on issue #2092: CLOUDSTACK-8599: [VMware] Successful migration was reported as failure when vCenter session timed out URL: https://github.com/apache/cloudstack/pull/2092#issuecomment-342303180 code LGTM, IMHO hard to have a test case for it.

[GitHub] resmo closed pull request #2068: systemvm: fix pep8 errors

resmo closed pull request #2068: systemvm: fix pep8 errors URL: https://github.com/apache/cloudstack/pull/2068 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 pull request

[GitHub] blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342269810 Trillian test result (tid-1622) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] blueorangutan commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API

blueorangutan commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API URL: https://github.com/apache/cloudstack/pull/2242#issuecomment-342234376 Trillian test result (tid-1621) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] yvsubhash opened a new pull request #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor?

yvsubhash opened a new pull request #1740: CLOUDSTACK-9572 Snapshot on primary storage not cleaned up after Stor? URL: https://github.com/apache/cloudstack/pull/1740 Snapshot on primary storage not cleaned up after Storage migration. This happens in the following scenario ## Steps To

[GitHub] yvsubhash opened a new pull request #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped

yvsubhash opened a new pull request #2308: CLOUDSTACK-8908 After copying the template charging for that template is stopped URL: https://github.com/apache/cloudstack/pull/2308 This is an automated message from the Apache

[GitHub] rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342143367 Thanks @mrunalinikankariya for the explanation, however, I'm wondering if this might break use-cases where you

[GitHub] mrunalinikankariya commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

mrunalinikankariya commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342141179 @rhtyd Here we are hardcoding the Port Forwarding capabilities which is used to validate the

[cloudstack] branch master updated: packging: Raise compat mode to 9 (#2307)

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 7bd460c packging: Raise compat mode to 9

[GitHub] rhtyd closed pull request #2307: packging: Raise compat mode to 9

rhtyd closed pull request #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307 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 pull

[GitHub] rhtyd commented on issue #2307: packging: Raise compat mode to 9

rhtyd commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342138182 Packaging LGTM, thanks @wido This is an automated message from the Apache Git

[GitHub] DaanHoogland commented on a change in pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

DaanHoogland commented on a change in pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#discussion_r149066641 ## File path: server/src/com/cloud/configuration/ConfigurationManagerImpl.java ## @@

[GitHub] DaanHoogland commented on a change in pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range.

DaanHoogland commented on a change in pull request #2048: CLOUDSTACK-9880: Expansion of Management IP Range. URL: https://github.com/apache/cloudstack/pull/2048#discussion_r149066501 ## File path:

[GitHub] wido commented on issue #2307: packging: Raise compat mode to 9

wido commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342133572 @rhtyd I checked the building of the packages and it works for me. Installed and ran just fine. As far as I know the *compat* file is only for

[GitHub] blueorangutan commented on issue #2307: packging: Raise compat mode to 9

blueorangutan commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342127135 Packaging result: ?centos6 ?centos7 ?debian. JID-1213 This is an automated

[GitHub] rhtyd commented on issue #2307: packging: Raise compat mode to 9

rhtyd commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342122460 @blueorangutan package This is an automated message from the Apache Git Service.

[GitHub] blueorangutan commented on issue #2307: packging: Raise compat mode to 9

blueorangutan commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342117482 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] rhtyd commented on issue #2307: packging: Raise compat mode to 9

rhtyd commented on issue #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307#issuecomment-342117291 LGTM, @wido can you comment if this may break packaging on old ubuntu/debian distros? (i.e. I would expect this works with 14.04+). I'll kick some

[GitHub] wido opened a new pull request #2307: packging: Raise compat mode to 9

wido opened a new pull request #2307: packging: Raise compat mode to 9 URL: https://github.com/apache/cloudstack/pull/2307 We are already compatible with that mode and should solve these warnings: dh: Compatibility levels before 9 are deprecated (level 7 in use)

[GitHub] blueorangutan commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API

blueorangutan commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API URL: https://github.com/apache/cloudstack/pull/2242#issuecomment-342088598 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

blueorangutan commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342088604 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] rhtyd commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API

rhtyd commented on issue #2242: CLOUDSTACK-9958:Include tags of resources in listUsageRecords API URL: https://github.com/apache/cloudstack/pull/2242#issuecomment-342088404 @blueorangutan test This is an automated message

[GitHub] rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ?

rhtyd commented on issue #2273: CLOUDSTACK-10090:createPortForwardingRule api call accepts 'halt' as ? URL: https://github.com/apache/cloudstack/pull/2273#issuecomment-342088344 @blueorangutan test This is an automated