[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1273#issuecomment-169927533 Closing this PR as made a new PR #1319 (Against 4.7 which will be merged in master later). --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/1273 --- 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 the

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1319#issuecomment-169927365 Reference PR #1273 (Against master) with 3 LGTMs. This PR (against 4.7) contains the same code change. --- If your project is set up for it, you can reply

Re: Minor releases!

​I totally agree Marcus, with one small addition. In our present scheme we can mark any 4.x as LTS if we maintain the discipline ​of fixing any bug on the oldest version known to contain the bug and merge the fix forward. some LIMITATION OF WARRANTY (tm) apply of course; if a fix requires some

Re: Minor releases!

one other remark: no matter what our release scheme, we need for every one to do twice as much reviews as contributions in order for our present way of working to be successful. And in this doing a review means sticking with a PR until it is good enough to go in, not just make initial remarks. We

Re: CloudStack 4.6.2 RPMs

Hi Rohit, Thanks for the information. Sorry for late reply, I was in leave. As per Apache CloudStack documentation (http://docs.cloudstack.apache.org/projects/cloudstack-release-notes/en/4.6.0/upgrade/upgrade-4.3.html#update-system-vm-templates) we need to register two systemvm templates (4.5

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1150#issuecomment-169931078 Closing this PR as made a new PR #1320 (Against 4.7 which will be merged in master later). --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/1150 --- 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 the

[GitHub] cloudstack pull request: CLOUDSTACK-8847: ListServiceOfferings is ...

Github user nitin-maharana closed the pull request at: https://github.com/apache/cloudstack/pull/823 --- 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 the

[GitHub] cloudstack pull request: CLOUDSTACK-8847: ListServiceOfferings is ...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/823#issuecomment-169935147 Closing this PR as made a new PR #1321 (Against 4.7 which will be merged in master later). --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1319#issuecomment-169935900 Hi @remibergsma, I created all my pending PRs against 4.7. Can you please review once. Thanks. --- If your project is set up for it, you can reply to this

Build failed in Jenkins: build-master-slowbuild #2934

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

Re: CloudStack 4.6.2 RPMs

Hi, > On 08-Jan-2016, at 11:52 AM, mvs babu wrote: > > Hi Rohit, > > Thanks for the information. Sorry for late reply, I was in leave. > > As per Apache CloudStack documentation >

[GitHub] cloudstack pull request: CLOUDSTACK-9132: API createVolume takes e...

GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1319 CLOUDSTACK-9132: API createVolume takes empty string for name parameter Steps to Reproduce: Create a volume using createVolume API where parameter name is

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169928856 Hi @DaanHoogland and @koushik-das Now I see your point. Thanks for the screenshot, @DaanHoogland. I checked the Github diff but did not notice

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1320 CLOUDSTACK-8968: UI icon over VM snapshot to deploy user instance Added a new Icon in Instance page to launch the VM from the snapshot. A new icon over VM snapshot object, which

gitlab.apache.org

We, cloudstack-dev-list-users have been talking about moving more functionality to github. This is being rejected by the foundation so far for (in my view obvious) reasons of governance. The functionality is however missed dearly so speaking with an ex-member he suggested we install a machine with

[GitHub] cloudstack pull request: CLOUDSTACK-8968: UI icon over VM snapshot...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1320#issuecomment-169931271 Reference PR #1150 (Against 4.6). This PR (against 4.7) contains the same code change with one conflict which I resolved. --- If your project is set up for

[GitHub] cloudstack pull request: CLOUDSTACK-8847: ListServiceOfferings is ...

GitHub user nitin-maharana opened a pull request: https://github.com/apache/cloudstack/pull/1321 CLOUDSTACK-8847: ListServiceOfferings is returning incompatible tagged offerings when called with VM id When calling listServiceOfferings with VM id as parameter. It is returning

[GitHub] cloudstack pull request: CLOUDSTACK-8847: ListServiceOfferings is ...

Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/1321#issuecomment-169934934 Reference PR #823 (Against master). This PR (against 4.7) contains the same code change. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169939093 @DaanHoogland and @koushik-das I split the commits and pushed the changes. You now should be able to review the actual changes I have applied to

[GitHub] cloudstack pull request: CLOUDSTACK-9104: VM naming convention in ...

Github user priyankparihar commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1302#discussion_r49171092 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2032,11 +2032,20 @@ int

[GitHub] cloudstack pull request: Fix mariadb related listCapacity bug (CLO...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1314#issuecomment-170095829 LGTM, works fine: ![screen shot 2016-01-08 at 15 43

[GitHub] cloudstack pull request: CLOUDSTACK-9216: Fix DHCP lease removal l...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1318#issuecomment-170097321 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

Build failed in Jenkins: build-master-slowbuild #2937

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[GitHub] cloudstack pull request: CLOUDSTACK-9202 Bump ssh timeout for VR c...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1291#issuecomment-170099354 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-170073454 Many thanks for the help everyone! --- 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

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/969#issuecomment-170073320 Hello everyone. Sorry to bother you but, how are the tests going? Is there anything else that I can do? --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-9181 Prevent syntax error in c...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1296#issuecomment-170099656 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: [4.7] ADD Force UDP encapsulation option ...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1317#issuecomment-170097102 LGTM https://cloud.githubusercontent.com/assets/1630096/12206985/64ba76d8-b644-11e5-89dd-91626b5bf5e4.png;> Tests: ``` nosetests

[GitHub] cloudstack pull request: CLOUDSTACK-9204 Do not error when staticr...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1298#issuecomment-170098587 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: CLOUDSTACK-6485 prevent ip asignment of p...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1299#issuecomment-170097718 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170098946 LGTM based on these tests: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \

Build failed in Jenkins: build-master-slowbuild #2938

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

What happened to the Jenkins CI test on PRs?

Hi, What happened to the Jenkins test on PRs? It seems PR 1287 was the last one to have both Travis CI and Jenkins. Since then, it’s only Travis CI. Why was it removed? (did I miss the mail?). Regards, Remi

[GitHub] cloudstack pull request: Admin cannot see VMs on port forwarding p...

GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1325 Admin cannot see VMs on port forwarding page On commit a902443708ee10acb9f68fff74af346a6a9fb370 the 'listAll=true' is removed. On some places the domainid and accountid are added but not

[GitHub] cloudstack pull request: CLOUDSTACK-9182: Some running VMs turned ...

Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/1252#issuecomment-170130323 Did someone tested it? This can be very handy. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on

Build failed in Jenkins: build-master-slowbuild #2939

See -- [...truncated 28733 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[GitHub] cloudstack pull request: Fix mariadb related listCapacity bug (CLO...

Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1314 --- 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 the feature

[GitHub] cloudstack pull request: [CLOUDSTACK-9218]Test to verify restart n...

GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1323 [CLOUDSTACK-9218]Test to verify restart network after master VR destroyed Please refer CLOUDSTACK-9218 for more details Test Results: === Test restarting RvR

Re: [GitHub] cloudstack pull request: [CLOUDSTACK-9215]Test to verify vm deploy...

Will add some validations On Fri, Jan 8, 2016 at 7:51 PM, DaanHoogland wrote: > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/1316#issuecomment-170016433 > > test succeeds as is : > ``` >

Build failed in Jenkins: build-master-slowbuild #2941

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170202865 On latest master (without any fixes) I see that the config values (network.gc.interval and network.gc.wait) are read correctly. Did the following: - Started

Re: [DISCUSS] Move to Github

+1 to moving to github. On Wed, Jan 6, 2016 at 6:17 PM, ilya wrote: > +1 to moving to github. > > On 1/3/16 3:25 AM, Sebastien Goasguen wrote: > > Bringing this one discuss thread to the top of the ML to get stronger > consensus. > > > > We need it if we want to

Build failed in Jenkins: build-master-slowbuild #2940

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[Issue] XenServer local storage

Hi, Has anyone noticed the following issue? I create a zone where I use local storage to house my system VMs. After the system VMs are all started, I sometimes (but rarely) notice that the local SR that was used to retrieve the system template remains (as opposed to going away once the system

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217

GitHub user pavanb018 opened a pull request: https://github.com/apache/cloudstack/pull/1322 Automation for CLOUDSTACK-9217 This script automates the below functionality: CloudStack should block migration to a Storage pool which is in maintenance mode. You can merge this pull

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9217

Github user pavanb018 commented on the pull request: https://github.com/apache/cloudstack/pull/1322#issuecomment-169954673 Test Results: == Trying to migrate a volume to a pool in maintenance mode should fail ... === TestName:

[GitHub] cloudstack pull request: Automation for CLOUDSTACK-9214 , detect w...

Github user shwetaag commented on the pull request: https://github.com/apache/cloudstack/pull/1315#issuecomment-169947496 @DaanHoogland I did the code review by code walk through and the code looks good to me --- If your project is set up for it, you can reply to this email and

Re: [4.7] account table does NOT have AUTO_INCREMENT set for id

Hi Wido, FYI: I run two 4.7 clouds and both have the AUTO_INCREMENT set properly. Regards, Remi On 08/01/16 14:31, "Wido den Hollander" wrote: >Hi, > >I've been investigating a issue with a freshly installed 4.7 CloudStack >setup and I was wondering if somebody else has

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170017549 Try something, @koushik-das: ``` Do not use this code for the steps below, but Master instead. ``` 1. Deploy a DC 2. Create a

Re: gitlab.apache.org

On 08-01-16 09:25, Daan Hoogland wrote: > We, cloudstack-dev-list-users have been talking about moving more > functionality to github. This is being rejected by the foundation so far > for (in my view obvious) reasons of governance. The functionality is > however missed dearly so speaking with

[GitHub] cloudstack pull request: [CLOUDSTACK-9215]Test to verify vm deploy...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1316#issuecomment-170016433 test succeeds as is : ``` [root@daans-dev-machinina ~]# ssh cs1 nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-kvm2.cfg

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-170016899 @borisroman @wido @resmo @llambiel @pyr @bhaisaab @wilderrodrigues I can run the tests again but I am the contributor. Can someone else, please? --- If your

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169979032 As a general cleanup, all configs defined in Config.java should be gradually moved to the appropriate java classes from where they are used. We should do away

[GitHub] cloudstack pull request: CLOUDSTACK-9104: VM naming convention in ...

Github user rafaelweingartner commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1302#discussion_r49183061 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -2032,11 +2032,20 @@ int

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-169996476 Do you agree that it is a completely different issue you are trying to address? ACS needs refactor in each corner of its code, but we cannot work

[4.7] account table does NOT have AUTO_INCREMENT set for id

Hi, I've been investigating a issue with a freshly installed 4.7 CloudStack setup and I was wondering if somebody else has seen this. The table account was looking like this: account | CREATE TABLE `account` ( `id` bigint(20) unsigned NOT NULL, `account_name` varchar(100) DEFAULT NULL

Re: gitlab.apache.org

This has been discussed rather intensively on the infra ML previously. Here's an answer from David: http://markmail.org/message/msxi7bhkfijydckq It's about half years old, so "short term" might have changed. -- Erik On Fri, Jan 8, 2016 at 2:36 PM, Daan Hoogland

Build failed in Jenkins: build-master-slowbuild #2935

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

Re: gitlab.apache.org

Wido, you are touching on the point of governance there. The maintenance is the price for keeping the goods under your control. Apache wants to make sure force pushes are not garbage collected but tagged for audit purposes, I understand. Btw, let's include infra in the discussion. On Fri, Jan 8,

[GitHub] cloudstack pull request: [4.7] Critical VPCVR issues fixed: CLOUDS...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1277#issuecomment-170011469 I have clearly mentioned that the refactoring needs to be done gradually. But in the changes you have reverted to using the config defined in Config.java when

[GitHub] cloudstack pull request: [CLOUDSTACK-9215]Test to verify vm deploy...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1316#discussion_r49191905 --- Diff: test/integration/component/maint/test_vpc.py --- @@ -0,0 +1,360 @@ +# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cloudstack pull request: Factor in inaccurancy of System.nanoTime ...

GitHub user bheuvel opened a pull request: https://github.com/apache/cloudstack/pull/1324 Factor in inaccurancy of System.nanoTime in class Profiler The time diff measurement by the class Profiler, is done by System.nanoTime(). System.nanoTime() is not 100% accurate (lots of

Re: [4.7] account table does NOT have AUTO_INCREMENT set for id

On Fri, Jan 8, 2016 at 2:31 PM, Wido den Hollander wrote: > Hi, > > I've been investigating a issue with a freshly installed 4.7 CloudStack > setup and I was wondering if somebody else has seen this. > > The table account was looking like this: > > account | CREATE TABLE

[GitHub] cloudstack pull request: Fix mariadb related listCapacity bug (CLO...

Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/1314#issuecomment-170050753 All integration tests look fine! ``` Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status :

Re: [4.7] account table does NOT have AUTO_INCREMENT set for id

On 08-01-16 15:53, Remi Bergsma wrote: > Hi Wido, > > FYI: I run two 4.7 clouds and both have the AUTO_INCREMENT set properly. > Hmm, weird. Were they fresh 4.7 installs? I manually fixed it here and looking at the SQL files I don't understand how this might have gone wrong. Wido > Regards,

Build failed in Jenkins: build-master-slowbuild #2936

See -- [...truncated 28723 lines...] [INFO] [INFO] --- findbugs-maven-plugin:3.0.1:findbugs (findbugs) @ cloud-quickcloud --- [INFO] [INFO] <<< findbugs-maven-plugin:3.0.1:check

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170056433 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

[GitHub] cloudstack-cloudmonkey pull request: Update requester.py

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack-cloudmonkey/pull/11#issuecomment-170059404 @phillipkent can you share examples of API/cloudmonkey commands where we would like to use * in the args? --- If your project is set up for it, you