Re: [VOTE] Apache CloudStack 4.6.1

Sorry for the absence… testing the RC now! Cheers, Wilder > On 03 Dec 2015, at 18:52, Glenn Wagner wrote: > > Good day > > +1 > > Centos 6,7 management server > Xenserver 6,2 > NFS storage > > Tests done , > Standard VM tests > Template uploads > ISO uploads > Basic network functions , add a

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/1164 --- 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 featur

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161898597 @remibergsma thanks, opened another PR #1169 please merge that. Closing this one, as once the other PR will be merged on 4.6 and 4.6 on master; we won't need this

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1166#issuecomment-161898618 I just saw this PR, sorry, I created #1168 for 4.6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user bhaisaab closed the pull request at: https://github.com/apache/cloudstack/pull/1149 --- 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: debian: allow rules to pick ACS_BUILD_OPT...

GitHub user bhaisaab opened a pull request: https://github.com/apache/cloudstack/pull/1169 debian: allow rules to pick ACS_BUILD_OPTS from env Only now debian builds can be noredist etc. Old PR: https://github.com/apache/cloudstack/pull/1149 cc @remibergsma You ca

[GitHub] cloudstack pull request: CLOUDSTACK-8845: set isRevertable of snap...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1168 CLOUDSTACK-8845: set isRevertable of snapshot to false if the volume is removed Some users encounter an exception when listSnapshots. We should set the isRevertable of snapshot to false

[GitHub] cloudstack pull request: CLOUDSTACK-9105: Logging enhancement: Han...

Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1167#issuecomment-161897298 Now all logs related to an operation can be traced by running something like "grep logid:62314271 ". Sample logs 2015-12-02 12:40:27,628 DEBUG [o.a.

[GitHub] cloudstack pull request: CLOUDSTACK-9105: Logging enhancement: Han...

GitHub user koushik-das opened a pull request: https://github.com/apache/cloudstack/pull/1167 CLOUDSTACK-9105: Logging enhancement: Handle/reference to track API c… …alls end to end in the MS logs Added logid to logging framework, now all API call logs can be tracked wit

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1164#issuecomment-161893553 @anshul1886 Thanks, you can close this PR as the other one will go to both 4.6 and master. --- If your project is set up for it, you can reply to this email a

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/1164#issuecomment-161891307 Created PR https://github.com/apache/cloudstack/pull/1166 for 4.6. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1166 CLOUDSTACK-8845: Fixed list snapshots command throwing exception https://issues.apache.org/jira/browse/CLOUDSTACK-8845 CLOUDSTACK-8845: Fixed list snapshots command throwing exceptio

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

GitHub user priyankparihar opened a pull request: https://github.com/apache/cloudstack/pull/1165 CLOUDSTACK-9104: VM naming convention in case vmware is used I have reverted all the changes. Now functionality is same as it was in earlier version. You can merge this pull request i

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1164#issuecomment-161889769 @anshul1886 Thanks for the fix! Could you please make this PR against 4.6? Once merged in 4.6, it will be fwd-merged to master. Thanks! --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-8845: Fixed list snapshots com...

GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/1164 CLOUDSTACK-8845: Fixed list snapshots command throwing exception https://issues.apache.org/jira/browse/CLOUDSTACK-8845 Fixed list snapshots command throwing exception if any one of s

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161885184 @bhaisaab Check! This I cannot merge to 4.6 with this PR so if you make one against 4.6 that would be easy. Mention this one so it can go through faster. Thanks

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1142#discussion_r46650221 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1276,6 +1277,53 @@ public String toString() {

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1142#discussion_r46650119 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java --- @@ -1276,6 +1277,53 @@ public String toString() {

[GitHub] cloudstack pull request: CLOUDSTACK-9103 : Missing OS Mappings for...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1159#issuecomment-161878161 @maneesha-p thanks for the PR, please fix as John suggests. We also support vmware 6.0 on 4.5 branch, so do you think we need this in 4.5.2 to 4.5.3 upgrade path a

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161878014 @remibergsma let me know if you want me to open a new PR against 4.6 branch. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161877977 @remibergsma sorry for the source/target branches; this needs to go into both 4.6 and master branches. So merge on 4.6, and fwd-merge 4.6 on master. Thanks. ---

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-161877685 @rafaelweingartner I agree. The use of "_" has no place in modern Java code. Constants should be all caps with _ separators. Everything else should be camelCase

[4.6.1/4.7.0] Database upgrade path issues/forks between 4.6.0/4.6.1 and 4.7.0/master

There seems to be a db view related change between 4.6.0 and 4.6.1 (DB) versions; and there exists separate DB upgrade paths from 4.6.0 to 4.7.0 and 4.6.1 to 4.7.0 on master branch which shares the same changes. The issue is that the upgrade path from 4.6.0 to 4.7.0 does not include the db view

[GitHub] cloudstack pull request: CLOUDSTACK-9099: SecretKey is returned fr...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1152#discussion_r46649508 --- Diff: api/src/com/cloud/user/AccountService.java --- @@ -136,4 +140,6 @@ void checkAccess(Account account, AccessType accessType, boolean sameOwner,

[GitHub] cloudstack pull request: CLOUDSTACK-9099: SecretKey is returned fr...

Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1152#discussion_r46649429 --- Diff: api/src/org/apache/cloudstack/api/command/admin/user/ListKeysCmd.java --- @@ -0,0 +1,72 @@ +// Licensed to the Apache Software Foundation

[GitHub] cloudstack pull request: CLOUDSTACK-9103 : Missing OS Mappings for...

Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1159#issuecomment-161876783 Master is now 4.7.0. Therefore, these database changes should be in ``schema-460to470.sql`` and ``schema-461to470.sql`` files. --- If your project is set up for

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

Github user jharshman commented on the pull request: https://github.com/apache/cloudstack/pull/1013#issuecomment-161840311 @remibergsma just squashed the commits, let me know if there is anything else I can do to assist testing. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Add support for not (re)starting server a...

Github user ProjectMoon commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1162#discussion_r46618985 --- Diff: client/bindir/cloud-setup-management.in --- @@ -24,20 +24,23 @@ from cloudutils.globalEnv import globalEnv from cloudutils.serviceConfi

[GitHub] cloudstack pull request: Send arping to the gateway instead of our...

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1163#issuecomment-161788201 Makes sense @remibergsma Is that test an example of the failure before the fix? --- If your project is set up for it, you can reply to this email and have you

[GitHub] cloudstack pull request: Add support for not (re)starting server a...

Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1162#discussion_r46613321 --- Diff: client/bindir/cloud-setup-management.in --- @@ -24,20 +24,23 @@ from cloudutils.globalEnv import globalEnv from cloudutils.serviceConf

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-161780789 So looking at the rpm spec options using +%config(noreplace) will cause a new version of cloudstack-catalina (cloudstack-catalina.rpmnew) to be created if cloudst

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161775836 @remibergsma awesome :) --- 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

[GitHub] cloudstack pull request: [WIP] CLOUDSTACK-6276: project support in...

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

[GitHub] cloudstack pull request: [4.7] CLOUDSTACK-9004: Add features to Hy...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1013#issuecomment-161751305 @jharshman I will run some tests and report back. Can you please squash the commits? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

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

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161750205 @pdube indeed, I waited because of the comments but that has been addressed now :-) --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161748134 @bhaisaab Should I merge this to master, or should it be merged to 4.6? --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

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

[GitHub] cloudstack pull request: Merge 4.6

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

[GitHub] cloudstack pull request: Merge 4.6

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1155#issuecomment-161745983 LGTM, run some tests and they are all fine: ``` Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_crea

[GitHub] cloudstack pull request: Send arping to the gateway instead of our...

GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1163 Send arping to the gateway instead of our own address We need to send an Unsolicited ARP to the gateway, instead of our own address. We now encounter problems when people deploy/destroy/dep

[GitHub] cloudstack pull request: Add support for not (re)starting server a...

GitHub user ProjectMoon opened a pull request: https://github.com/apache/cloudstack/pull/1162 Add support for not (re)starting server after cloud-setup-management. This adds an option to the cloud-setup-management script to not start the management server after a successful confi

Re: [VOTE] Apache CloudStack 4.6.1

Good day +1 Centos 6,7 management server Xenserver 6,2 NFS storage Tests done , Standard VM tests Template uploads ISO uploads Basic network functions , add and remove IPs VM Snapshots Didn't get to vpc testing Regards Glenn Wagner Shape blue. Pty ltd On Thu, Dec 3, 2015 at 8:48 AM -0800, "

Re: [VOTE] Apache CloudStack 4.6.1

Hello, My vote +1 (binding) My tests : 1/ Fresh install of 4.6.1 (adv net) on Ubuntu 14.04 + KVM + NFS : OK Some standard tests (create vm, migration, HA, create network, create user, create ssh key, destroy vm, register template, create snapshot, restore snapshot, create template, ip associat

[GitHub] cloudstack pull request: [4.6/4.6.1] IMPORTANT for 4.6.1 release -...

Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1149#issuecomment-161709051 Seems good, 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 fea

[GitHub] cloudstack pull request: CLOUDSTACK-8892:If VR enters out-of band ...

Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/1158#issuecomment-161684501 big -1 We tested out of band migrations on VR on VMware and they work without reboots. A VM/VR does not "loose" the IP just because it gets migrated. Lo

RE: [VOTE] Apache CloudStack 4.6.1

Sorry should say 4.6.1 on Centos6.7 mgmt server with ESXi 5.5 hosts/vCenter. Paul Angus VP Technology , ShapeBlue d: +44 2036170528 | t: @cloudyangus | m: +44 7711418784 e: paul.an...@shapeblue.com| w

RE: [VOTE] Apache CloudStack 4.6.1

I've lost track - how long has the vote got left? [ShapeBlue] Paul Angus VP Technology , ShapeBlue d: +44 2036170528 | t: @cloudyangus | m: +44 7711418784 e: paul.an...@shapeblue.com

RE: [VOTE] Apache CloudStack 4.6.1

Thanks Daan, I've only had time for some very basic testing, but nothing failed. +0 [ShapeBlue] Paul Angus VP Technology , ShapeBlue d: +44 2036170528 | t: @cloudyangus | m: +44 7711418784 e: paul.an...@shapeblue.com

Re: [VOTE] Apache CloudStack 4.6.1

should be closed today On Thu, Dec 3, 2015 at 4:40 PM, Paul Angus wrote: > I've lost track - how long has the vote got left? > > > [image: ShapeBlue] Paul Angus VP Technology , > ShapeBlue d: *+44 2036170528* <+44%202036170528> | t: *@cloudyangus* > <@cloudyangus>

Re: [VOTE] Apache CloudStack 4.6.1

To be on the record: +0 I have an issue but cannot lay the blame with cloudstack. I will keep looking and as I saw lots of reasons to go through with this version reported by others. On Thu, Dec 3, 2015 at 3:31 PM, Remi Bergsma wrote: > Hi all, > > FYI run some integration tests against 4.6.1 an

Re: CloudMonkey Import error in cloudmonkey.requester : cannot import name connection

Thanks Rohit. I think the pip upgrade fixed it. Ubuntu 14 python comes with python requests by default at version 2.2.1 and looks like cloudmonkey need requests at 2.8.1 Regards, Kenny On Thu, Dec 3, 2015 at 1:25 AM, Rohit Yadav wrote: > Hi Kenny, > > On 03-Dec-2015, at 5:25 AM, Kenny Ho wro

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-161657343 Hey guys, there are enough LGTMs, can we get this merged? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack-www pull request: developer page: fixed some repo links...

GitHub user swill opened a pull request: https://github.com/apache/cloudstack-www/pull/21 developer page: fixed some repo links and new tab for external links Fixed repo links as they were broken before. I also changed the external links on this page to be opened in new tabs

Re: CS slack channel

https://drewdevault.com/2015/11/01/Please-stop-using-slack.html -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Nguyen Anh Tu" > To: dev@cloudstack.apache.org > Sent: Thursday, 3 December, 2015 14:35:24 > Subject: Re: CS slack channel

Re: CS slack channel

very welcome guys (and potentially dolls) On Thu, Dec 3, 2015 at 3:35 PM, Nguyen Anh Tu wrote: > Great to hear CS is now happening in Slack. Could you send me an invitation > as well, Daan ? > > > On Thu, Dec 3, 2015 at 9:08 PM, Simon Weller wrote: > > > Daan, > > > > > > Could you send me an i

Re: CS slack channel

Great to hear CS is now happening in Slack. Could you send me an invitation as well, Daan ? On Thu, Dec 3, 2015 at 9:08 PM, Simon Weller wrote: > Daan, > > > Could you send me an invite to the CloudStack Slack Team please? > > > Thanks, > > > - Si >

Re: [VOTE] Apache CloudStack 4.6.1

Hi all, FYI run some integration tests against 4.6.1 and they all pass. Talked to Daan and I cannot reproduce his UI issue. Test results: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \ component/test_vpc_redundant.py \ component/test_route

[GitHub] cloudstack pull request: Quota

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161653555 @bhaisaab The SQL issue is indeed resolved. I still see the icon when the plugin is disabled (and it of course gives errors then). Tried clearing the cache and b

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user ustcweizhou commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1134#discussion_r46554416 --- Diff: server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java --- @@ -229,59 +205,99 @@ public AffinityGroupVO doInTransaction(T

[GitHub] cloudstack pull request: CLOUDSTACK-9087:adding projectid paramete...

Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1153#issuecomment-161650382 Isn't the project inferred from the instance? The [CreateVmSnapshot](http://cloudstack.apache.org/docs/api/apidocs-4.6/root_admin/createVMSnapshot.html) API doesn't e

CS slack channel

Daan, Could you send me an invite to the CloudStack Slack Team please? Thanks, - Si

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1134#discussion_r46553780 --- Diff: server/src/org/apache/cloudstack/affinity/AffinityGroupServiceImpl.java --- @@ -229,59 +205,99 @@ public AffinityGroupVO doInTransaction(Transac

[GitHub] cloudstack pull request: CLOUDSTACK-9101: fix some issues in resiz...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1161 CLOUDSTACK-9101: fix some issues in resize volume (1) fix issue: volume size is not updated even if the operation succeed (2) Add ui support for root volume resize (3) resize on qcow

[GitHub] cloudstack pull request: CLOUDSTACK-9101: fix some issues in resiz...

Github user ustcweizhou closed the pull request at: https://github.com/apache/cloudstack/pull/1160 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-9101: fix some issues in resiz...

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1160#issuecomment-161647136 wrong branch --- 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 pull request: CLOUDSTACK-9101: fix some issues in resiz...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1160 CLOUDSTACK-9101: fix some issues in resize volume (1) fix issue: volume size is not updated even if the operation succeed (2) Add ui support for root volume resize (3) resize on qcow

Job opening

All, We're currently hiring for a Product Manager position dedicated to our Cloud Product (based on CloudStack). Where would it be appropriate to post the ad? Thanks, - Si

[GitHub] cloudstack pull request: Quota

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161645283 @abhinandanprateek I will test it soon, thanks. Personally, I think a black & white icon is better in line with the other icons. --- If your project is set up f

[GitHub] cloudstack pull request: CLOUDSTACK-8892:If VR enters out-of band ...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1158#issuecomment-161641914 -1 I don't think it is a good idea to reboot routers when out-of-band operation is detected. We discussed this before, let's not do it again. Please se

[GitHub] cloudstack pull request: CLOUDSTACK-9103 : Missing OS Mappings for...

GitHub user maneesha-p opened a pull request: https://github.com/apache/cloudstack/pull/1159 CLOUDSTACK-9103 : Missing OS Mappings for VMware 6.0 Added suitable db entries in tables hypervisor_capabilities and guest_os_hypervisor to support VMware 6.0 by copying from 5.5 and excludi

[GitHub] cloudstack pull request: Quota

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161622406 @remibergsma completed my internal tests around UI and backend implementation against a small cloud usage db (with a live environment), along with db upgrades (shou

[GitHub] cloudstack pull request: CLOUDSTACK-8892:If VR enters out-of band ...

GitHub user maneesha-p opened a pull request: https://github.com/apache/cloudstack/pull/1158 CLOUDSTACK-8892:If VR enters out-of band context, routers lose their private/link-local IP If router is powered-on out-of-band , we try to reboot it n times with some retry interval where n

[GitHub] cloudstack pull request: CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREA...

GitHub user SudharmaJain opened a pull request: https://github.com/apache/cloudstack/pull/1157 CLOUDSTACK-9100: ISO.CREATE/TEMPLATE.CREATE event missing for usage_e… …vent by template sync thread If there is a Management server restart while template is in downloading or

[GitHub] cloudstack pull request: Quota

Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161604884 @remibergsma the new UI changes improve the UI and also take care of the issue you have noted above. If quota.enable.service is set to false then the quota

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1156#issuecomment-161599503 Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS === ok -

[GitHub] cloudstack pull request: Fixed issues with test_vpc_vpn script

GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1156 Fixed issues with test_vpc_vpn script There are few issues with this test file. Please refer to JIRA Ticket https://issues.apache.org/jira/browse/CLOUDSTACK-9102 for details on the issues th

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161595328 @remibergsma time to merge! It also got a LGTM from @DaanHoogland :) Cheers, Wilder --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: Quota

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161570202 @bhaisaab That sounds like a good solution. I will now test this PR again and let you know. --- If your project is set up for it, you can reply to this email an

[GitHub] cloudstack pull request: Merge 4.6

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1155#issuecomment-161569317 Will run some tests to double check. --- 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 pro

[GitHub] cloudstack pull request: Merge 4.6

Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1155#issuecomment-161568869 LGTM, code are almost same to #1029 , fine --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cloudstack pull request: Merge 4.6

GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1155 Merge 4.6 there is one (simple) conflict so for formailty's sake I make this a pull request. @utscweizhou en @remibergsma please have a look/test-run You can merge this pull request i

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

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

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

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

[GitHub] cloudstack pull request: Create test cases to getPatchFilePath met...

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

[GitHub] cloudstack pull request: Create test cases to getPatchFilePath met...

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

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

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

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161557066 The test passes fine here, also on a 2 node cluster: test_01_vpc_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... ===

[GitHub] cloudstack pull request: CLOUDSTACK-8868: use PasswordGenerator.ge...

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

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

Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161555776 LGTM, ran the tests as well and reviewed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack pull request: CLOUDSTACK-8868: use PasswordGenerator.ge...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9080: Resource limits for Prim...

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

[GitHub] cloudstack pull request: CLOUDSTACK-9080: Resource limits for Prim...

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

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

Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-161552200 @DaanHoogland @rafaelweingartner The failed expunge test was a false alarm due to our testing environment. It passes again now. Please ping me when there's anoth

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

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

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9075 - As a Developer I ...

Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1151#issuecomment-161549425 @remibergsma @borisroman Please, be aware that there is a change in the ```tools/marvin/marvin/lib/common.py``` file. It means that you have to bu

[GitHub] cloudstack pull request: Quota

Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-161547150 @remibergsma I hope I've fixed the db issues and code, the db upgrade path implementation from 4.6.0/4.6.1 to 4.7.0 (on master) duplicates code. May be we can de-du

[GitHub] cloudstack pull request: [UI] fix bug: Cannot delete SSH keypairs ...

GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1154 [UI] fix bug: Cannot delete SSH keypairs in projects Before change: error message: ProjectId and account/domainId can't be specified together After change: SSH keypairs can