Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-20 Thread prasanna
On 20 March 2013 20:26, Wido den Hollander wrote: > > > On 03/20/2013 12:19 PM, Prasanna Santhanam wrote: >> >> On Wed, Mar 13, 2013 at 02:19:57PM +0530, Prasanna Santhanam wrote: >>> >>> On Wed, Mar 13, 2013 at 01:45:46PM +0530, prasanna wrote: On 13 March 2013 12:56, Pradeep Soundarara

Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-20 Thread Wido den Hollander
On 03/20/2013 12:19 PM, Prasanna Santhanam wrote: On Wed, Mar 13, 2013 at 02:19:57PM +0530, Prasanna Santhanam wrote: On Wed, Mar 13, 2013 at 01:45:46PM +0530, prasanna wrote: On 13 March 2013 12:56, Pradeep Soundararajan wrote: David, I did not come across this issue "cpio: Bad magic" eve

Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-20 Thread Prasanna Santhanam
On Wed, Mar 13, 2013 at 02:19:57PM +0530, Prasanna Santhanam wrote: > On Wed, Mar 13, 2013 at 01:45:46PM +0530, prasanna wrote: > > On 13 March 2013 12:56, Pradeep Soundararajan > > wrote: > > > David, > > > > > > I did not come across this issue "cpio: Bad magic" even before or after > > > your

Re: [ACS41][Patch Request] CLOUDSTACK-1706

2013-03-18 Thread Chip Childers
On Mon, Mar 18, 2013 at 12:14:17PM -0700, Prachi Damle wrote: > Already committed to master. > > > > Commit hash:d9b85e397d10f2291058ff972094decd8fc4ad7d > > > > -- > > CLOUDSTACK-1706 Failed to deploy VM with error "cannot find > DeployPlannerSelector" > > > > Changes: >

[ACS41][Patch Request] CLOUDSTACK-1706

2013-03-18 Thread Prachi Damle
Already committed to master. Commit hash:d9b85e397d10f2291058ff972094decd8fc4ad7d -- CLOUDSTACK-1706 Failed to deploy VM with error "cannot find DeployPlannerSelector" Changes: - Regular plugin/adapter components should usually be loaded at run level RUNLEVEL_COMPONEN

Re: [ACS41][Patch Request]

2013-03-15 Thread Chip Childers
On Fri, Mar 15, 2013 at 10:16:24AM +0100, Wido den Hollander wrote: > Sorry for the late response! > > Ack from my side, this seems good to have in 4.1 and will resolve > some weird issues. > > Wido > All three commit applied to 4.1 now. Thanks. Please resolve the related bugs! > On 03/15/20

Re: [ACS41][Patch Request]

2013-03-15 Thread Wido den Hollander
Sorry for the late response! Ack from my side, this seems good to have in 4.1 and will resolve some weird issues. Wido On 03/15/2013 04:31 AM, Marcus Sorensen wrote: Just wanted to ping on this again. I'm confident enough in it if widow doesn't get a chance to respond. On Mar 13, 2013 1:27 P

Re: [ACS41][Patch Request]

2013-03-14 Thread Marcus Sorensen
Just wanted to ping on this again. I'm confident enough in it if widow doesn't get a chance to respond. On Mar 13, 2013 1:27 PM, "Marcus Sorensen" wrote: > Ok, the fix for CLOUDSTACK-600 is the first patch mentioned plus two > fixes to it. The first additional patch handles upgrades, and the > se

Re: [ACS41][Patch Request]CLOUDSTACK-1218: Use the first IP in the first vlan range for VR in shared network

2013-03-14 Thread Chip Childers
On Thu, Mar 14, 2013 at 01:29:48PM -0700, Sheng Yang wrote: > Already checked in MASTER. > > commit 98291d043de47ef6c07aca3d1d31464f6a87 > Author: Sheng Yang > Date: Wed Mar 13 17:02:19 2013 -0700 > > CLOUDSTACK-1218: Use the first IP in the first vlan range for VR > in shared network

[ACS41][Patch Request]CLOUDSTACK-1218: Use the first IP in the first vlan range for VR in shared network

2013-03-14 Thread Sheng Yang
Already checked in MASTER. commit 98291d043de47ef6c07aca3d1d31464f6a87 Author: Sheng Yang Date: Wed Mar 13 17:02:19 2013 -0700 CLOUDSTACK-1218: Use the first IP in the first vlan range for VR in shared network Thanks! --Sheng

Re: [ACS41][Patch Request] Several patches to security_group.py

2013-03-14 Thread Chip Childers
On Thu, Mar 14, 2013 at 12:28:55PM -0700, John Kinsella wrote: > I've fixed several bugs in security_group.py in the last few days. Would be > nice if we could get this into 4.1. Ack and pushed. Can you please resolve all relevant bugs? > > commit 381f737e64ed9192e6eea4aeffe1920637f7d835 > Aut

Re: [ACS41][Patch Request] CLOUDSTACK-1604

2013-03-14 Thread Chip Childers
On Wed, Mar 13, 2013 at 06:18:30PM -0700, Prachi Damle wrote: > Patch: https://reviews.apache.org/r/9917/ > > > Fix for CLOUDSTACK-1604: deploy VM failed when global setting > "vm.allocation.algorithm" is set to "userdispersing" > > > Ack - this was pushed.

Re: [ACS41][Patch Request] - CLOUDSTACK-1648

2013-03-14 Thread Chip Childers
On Wed, Mar 13, 2013 at 04:46:44PM -0600, Marcus Sorensen wrote: > Tested the commit against 4.0 branch, 4.1 branch, and master dev > environments. Successfully deployed zones with local and NFS primary > storage, to the point of instances, then rebooted KVM host, and > everything came back up. Ask

[ACS41][Patch Request] Several patches to security_group.py

2013-03-14 Thread John Kinsella
I've fixed several bugs in security_group.py in the last few days. Would be nice if we could get this into 4.1. commit 381f737e64ed9192e6eea4aeffe1920637f7d835 Author: John Kinsella Date: Wed Mar 13 16:52:49 2013 -0700 Summary: Fix exception handling in security_group.py Detail:

[ACS41][Patch Request] CLOUDSTACK-1604

2013-03-13 Thread Prachi Damle
Patch: https://reviews.apache.org/r/9917/ Fix for CLOUDSTACK-1604: deploy VM failed when global setting "vm.allocation.algorithm" is set to "userdispersing"

[ACS41][Patch Request] - CLOUDSTACK-1648

2013-03-13 Thread Marcus Sorensen
Tested the commit against 4.0 branch, 4.1 branch, and master dev environments. Successfully deployed zones with local and NFS primary storage, to the point of instances, then rebooted KVM host, and everything came back up. Asked reporter to test patch, but reporter found a workaround and moved on.

Re: [ACS41][Patch Request] CLOUDSTACK-1557

2013-03-13 Thread David Nalley
ACK Applies and builds cleanly Committed in 4.1 Thanks, --David On Wed, Mar 13, 2013 at 5:28 PM, Prachi Damle wrote: > Patch: https://reviews.apache.org/r/9823/ > > > Reviewed and checked in master. > > Commit hash:7bb8ed4d05ac9ea1242201e0e4fafc36916d5c51 > > > > ---

[ACS41][Patch Request] CLOUDSTACK-1557

2013-03-13 Thread Prachi Damle
Patch: https://reviews.apache.org/r/9823/ Reviewed and checked in master. Commit hash:7bb8ed4d05ac9ea1242201e0e4fafc36916d5c51 Author:Pradeep Soundararajan (pradeep.soundarara...@citrix.com) Author date: 11 minutes ago

Re: [ACS41][Patch Request]

2013-03-13 Thread Marcus Sorensen
Ok, the fix for CLOUDSTACK-600 is the first patch mentioned plus two fixes to it. The first additional patch handles upgrades, and the second one is a minor fix for when we shut down a non-persistent VM that will avoid printing useless scary messages (we throw an error in the timeout loop that wait

Re: [ACS41][Patch Request] CLOUDSTACK-1427: Failed to delete Guestnetwork which has LB with Netscaler

2013-03-13 Thread David Nalley
On Wed, Mar 13, 2013 at 8:14 AM, Murali Reddy wrote: > CLOUDSTACK-1427: Failed to delete Guestnetwork which has LB with > Netscaler > > There is no need for getIpDeployer to depend on the fact a NetScaler > device is allocated (network is implemented state) or not-allocated > (network is in shutdo

[ACS41][Patch Request] CLOUDSTACK-1427: Failed to delete Guestnetwork which has LB with Netscaler

2013-03-13 Thread Murali Reddy
CLOUDSTACK-1427: Failed to delete Guestnetwork which has LB with Netscaler There is no need for getIpDeployer to depend on the fact a NetScaler device is allocated (network is implemented state) or not-allocated (network is in shutdown state) Branch: refs/heads/master Commit: 2feda05ecd15d7fe63b

Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-13 Thread Prasanna Santhanam
On Wed, Mar 13, 2013 at 01:45:46PM +0530, prasanna wrote: > On 13 March 2013 12:56, Pradeep Soundararajan > wrote: > > David, > > > > I did not come across this issue "cpio: Bad magic" even before or after > > your doc changes. I am doing the build and packaging daily on both 4.1 and > > master.

Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-13 Thread prasanna
On 13 March 2013 12:56, Pradeep Soundararajan wrote: > David, > > I did not come across this issue "cpio: Bad magic" even before or after your > doc changes. I am doing the build and packaging daily on both 4.1 and master. > > I suspect it could be the setup issue. > Can you tell us the followin

RE: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-13 Thread Pradeep Soundararajan
chip.child...@sungard.com] Sent: Tuesday, March 12, 2013 6:03 AM To: cloudstack-dev@incubator.apache.org Subject: Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging On Mon, Mar 11, 2013 at 08:24:27PM -0400, David Nalley wrote: > Chip, et al: > > CLOUDSTACK-1631 is for the RPM build

Re: [ACS41][Patch Request]

2013-03-12 Thread Marcus Sorensen
Looks like this patch might need to be adjusted slightly. We can use the domain.isPersistent() method to determine if we need to call domain.undefine() in addition to dm.shutdown and dm.destroy. That will allow upgrades to clean up existing definitions as VMs are shutdown. If you don't have a chan

Re: [ACS41][Patch Request]

2013-03-12 Thread Marcus Sorensen
To clarify, this is a fairly serious bug, and it should probably be addressed, I'm just wanting clarification that the patch is good to go as a fix. On my way home I was thinking about how I'm not sure if it addresses upgrades, i.e. does it still allow existing definitions to be wiped, or will thos

Re: [ACS41][Patch Request]

2013-03-12 Thread Marcus Sorensen
We need to verify that upgrades are addressed too On Mar 12, 2013 6:43 PM, "Chip Childers" wrote: > On Tue, Mar 12, 2013 at 06:39:09PM -0600, Marcus Sorensen wrote: > > This addresses CLOUDSTACK-600. Please get buyoff from Wido if > > possible, since it's his patch. I've reviewed and tested it,

Re: [ACS41][Patch Request]

2013-03-12 Thread Chip Childers
On Tue, Mar 12, 2013 at 06:39:09PM -0600, Marcus Sorensen wrote: > This addresses CLOUDSTACK-600. Please get buyoff from Wido if > possible, since it's his patch. I've reviewed and tested it, but I > want to make sure I'm not wrong in assuming that this should be in 4.1 > > commit 5dfcd309f10e5bd

[ACS41][Patch Request]

2013-03-12 Thread Marcus Sorensen
This addresses CLOUDSTACK-600. Please get buyoff from Wido if possible, since it's his patch. I've reviewed and tested it, but I want to make sure I'm not wrong in assuming that this should be in 4.1 commit 5dfcd309f10e5bd6a918f7fdff3f44a3dff2374a Author: Wido den Hollander Date: Thu Feb 7 22:

Re: [ACS41][Patch Request]

2013-03-12 Thread Chip Childers
On Tue, Mar 12, 2013 at 12:21:58PM -0600, Marcus Sorensen wrote: > commit 0ca85659a66bbf722bd02d35ee899e8d074e6fbe > Author: Marcus Sorensen > Date: Tue Mar 12 12:20:22 2013 -0600 > > CLOUDSTACK-1651 - point agent scripts to correct logging location, > due to move > of /var/log/cloud to

[ACS41][Patch Request]

2013-03-12 Thread Marcus Sorensen
commit 0ca85659a66bbf722bd02d35ee899e8d074e6fbe Author: Marcus Sorensen Date: Tue Mar 12 12:20:22 2013 -0600 CLOUDSTACK-1651 - point agent scripts to correct logging location, due to move of /var/log/cloud to /var/log/cloudstack Signed-off-by: Marcus Sorensen 1363112422 -0600

Re: [ACS41][Patch Request]

2013-03-11 Thread Chip Childers
On Mon, Mar 11, 2013 at 11:28:41PM -0400, David Nalley wrote: > On Mon, Mar 11, 2013 at 8:41 PM, Chip Childers > wrote: > > On Mon, Mar 11, 2013 at 08:11:22PM -0400, David Nalley wrote: > >> Chip, et al: > >> > >> RPM Packaging is currently broken in 4.1 - this fixes CLOUDSTACK-1630 > >> - which d

Re: [ACS41][Patch Request]

2013-03-11 Thread David Nalley
On Mon, Mar 11, 2013 at 8:41 PM, Chip Childers wrote: > On Mon, Mar 11, 2013 at 08:11:22PM -0400, David Nalley wrote: >> Chip, et al: >> >> RPM Packaging is currently broken in 4.1 - this fixes CLOUDSTACK-1630 >> - which doesn't solve the breakage. But once packaging works again, >> this needs to

Re: [ACS41][Patch Request]

2013-03-11 Thread Chip Childers
On Mon, Mar 11, 2013 at 08:11:22PM -0400, David Nalley wrote: > Chip, et al: > > RPM Packaging is currently broken in 4.1 - this fixes CLOUDSTACK-1630 > - which doesn't solve the breakage. But once packaging works again, > this needs to be applied to fix upgrades. > > 797eb009e239d904c52de8c5be4c

Re: [ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-11 Thread Chip Childers
On Mon, Mar 11, 2013 at 08:24:27PM -0400, David Nalley wrote: > Chip, et al: > > CLOUDSTACK-1631 is for the RPM build failures (that have been > occurring for some days now.) > > The error is: > error: create archive failed on file > /home/jenkins/build/workspace/package-rhel63-4.1/dist/rpmbuild/

[ACS41][Patch Request] CLOUDSTACK-1631 - fix RPM packaging

2013-03-11 Thread David Nalley
Chip, et al: CLOUDSTACK-1631 is for the RPM build failures (that have been occurring for some days now.) The error is: error: create archive failed on file /home/jenkins/build/workspace/package-rhel63-4.1/dist/rpmbuild/SOURCES/cloudstack-4.1.0-SNAPSHOT.tgz: cpio: Bad magic Bad Magic is _generall

[ACS41][Patch Request]

2013-03-11 Thread David Nalley
Chip, et al: RPM Packaging is currently broken in 4.1 - this fixes CLOUDSTACK-1630 - which doesn't solve the breakage. But once packaging works again, this needs to be applied to fix upgrades. 797eb009e239d904c52de8c5be4cb0c82a63fea4 <-- applied on master

Re: [ACS41][Patch Request] CLOUDSTACK-1461: Don't set dns server for non-default ipv6 network

2013-03-10 Thread Chip Childers
On Fri, Mar 08, 2013 at 05:58:27PM -0800, Sheng Yang wrote: > Already checked in master. > > commit e35ce6587a2519fdbc56e8859ad6aa7dac00dd47 > Author: Sheng Yang > Date: Fri Mar 8 17:42:06 2013 -0800 > > CLOUDSTACK-1461: Don't set dns server for non-default ipv6 network > > The non-de

Re: [ACS41][Patch Request}

2013-03-09 Thread Marcus Sorensen
Authored: Fri Mar 8 23:14:08 2013 -0700 >> Committer: Pranav Saxena >> Committed: Sat Mar 9 12:00:47 2013 +0530 >> >> Regards, >> Pranav >> >> -----Original Message- >> From: Marcus Sorensen [mailto:shadow...@gmail.com] >> Sent: Saturday, March 09, 2013 11:4

Re: [ACS41][Patch Request}

2013-03-09 Thread Wido den Hollander
Mar 8 23:14:08 2013 -0700 Committer: Pranav Saxena Committed: Sat Mar 9 12:00:47 2013 +0530 Regards, Pranav -Original Message- From: Marcus Sorensen [mailto:shadow...@gmail.com] Sent: Saturday, March 09, 2013 11:46 AM To: cloudstack-dev@incubator.apache.org Subject: [ACS41][Pa

RE: [ACS41][Patch Request}

2013-03-08 Thread Pranav Saxena
Sent: Saturday, March 09, 2013 11:46 AM To: cloudstack-dev@incubator.apache.org Subject: [ACS41][Patch Request} commit ebafcf826b2506e779c8de8887d867ca4e0306bf Author: Marcus Sorensen Date: Fri Mar 8 23:14:08 2013 -0700 CLOUDSTACK-1201 - Fix update to cloud user's home dir on mgmt

[ACS41][Patch Request}

2013-03-08 Thread Marcus Sorensen
commit ebafcf826b2506e779c8de8887d867ca4e0306bf Author: Marcus Sorensen Date: Fri Mar 8 23:14:08 2013 -0700 CLOUDSTACK-1201 - Fix update to cloud user's home dir on mgmt server. It was printing a meaningless failure if the cloud user's home dir didn't need to be updated.

[ACS41][Patch Request] CLOUDSTACK-1461: Don't set dns server for non-default ipv6 network

2013-03-08 Thread Sheng Yang
Already checked in master. commit e35ce6587a2519fdbc56e8859ad6aa7dac00dd47 Author: Sheng Yang Date: Fri Mar 8 17:42:06 2013 -0800 CLOUDSTACK-1461: Don't set dns server for non-default ipv6 network The non-default parameter can be used by ipv4 as well in the future. Thanks! --Sheng

[ACS41][Patch Request] - Reverting UI for IPv6 in 4.1

2013-03-08 Thread Jessica Wang
Reverting UI for IPv6 in 4.1 https://reviews.apache.org/r/9833/ thank you.

[ACS41][Patch Request] CLOUDSTACK-1509

2013-03-08 Thread Likitha Shetty
Patch - https://reviews.apache.org/r/9817/ Thank you, Likitha

[ACS41][Patch Request] CLOUDSTACK-1574 : updateResourceCount API is failed saying to specify valida resource type even after parsing the valid resource type.

2013-03-07 Thread Sanjay Tripathi
> -Original Message- > From: Sanjay Tripathi [mailto:nore...@reviews.apache.org] On Behalf Of > Sanjay Tripathi > Sent: Friday, March 08, 2013 11:05 AM > To: Chip Childers; Abhinandan Prateek > Cc: cloudstack; Sanjay Tripathi > Subject: Review Request: CLOUDSTACK-1574 : updateResourceCoun

[ACS41][Patch Request] CLOUDSTACK-1541 NPE while deleting snapshot

2013-03-07 Thread Deepti Dohare
> -Original Message- > From: deepti dohare [mailto:nore...@reviews.apache.org] On Behalf Of > deepti dohare > Sent: Friday, March 08, 2013 12:40 AM > To: Chip Childers; Sateesh Chodapuneedi; Abhinandan Prateek; Devdeep > Singh > Cc: cloudstack; Deepti Dohare > Subject: Re: Review Request:

Re: [ACS41][Patch Request] CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5

2013-03-07 Thread Chip Childers
On Wed, Mar 06, 2013 at 04:54:40PM -0800, Sheng Yang wrote: > commit 27ba0208f068e60f588d25a5d0ad9c4d6503ad14 > Author: Sheng Yang > Date: Wed Mar 6 16:49:44 2013 -0800 > > CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5 > > Still return a valid object when rever

[ACS41][Patch Request] CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5

2013-03-06 Thread Sheng Yang
commit 27ba0208f068e60f588d25a5d0ad9c4d6503ad14 Author: Sheng Yang Date: Wed Mar 6 16:49:44 2013 -0800 CLOUDSTACK-1315: Fix NPE when try to upgrade network from VR to SRX-F5 Still return a valid object when reverting non-existed rules. Thanks. --Sheng

Re: [ACS41][Patch Request]

2013-03-05 Thread Marcus Sorensen
done. Thanks On Tue, Mar 5, 2013 at 5:42 PM, Chip Childers wrote: > Please go ahead and apply it to 4.1. I'm not going to be at a terminal > tonight. Looks good to me. > > On Mar 5, 2013, at 5:16 PM, Marcus Sorensen wrote: > >> commit 7dbea68b8728e4b3a5b727a4fd69712cd8a1c468 >> Author: Marcus So

Re: [ACS41][Patch Request]

2013-03-05 Thread Chip Childers
Please go ahead and apply it to 4.1. I'm not going to be at a terminal tonight. Looks good to me. On Mar 5, 2013, at 5:16 PM, Marcus Sorensen wrote: > commit 7dbea68b8728e4b3a5b727a4fd69712cd8a1c468 > Author: Marcus Sorensen > Date: Tue Mar 5 15:10:53 2013 -0700 > >Summary: Wait for VPC r

[ACS41][Patch Request]

2013-03-05 Thread Marcus Sorensen
commit 7dbea68b8728e4b3a5b727a4fd69712cd8a1c468 Author: Marcus Sorensen Date: Tue Mar 5 15:10:53 2013 -0700 Summary: Wait for VPC router nics to hotplug before assigning IPs Detail: CLOUDSTACK-1452, CLOUDSTACK-1523 - When rebooting VPC routers, ip addr assignment was inconsistent,

Re: [ACS41][Patch Request] agent.properties from RPM is blocking proper agent setup

2013-03-02 Thread Marcus Sorensen
I went back and looked at the wiki, and I think I misunderstood one of the sections. It shows examples of things to put into the commit, Reviewed-by:,Signed-off-by:,Reported-by:, and then below that, CLOUDSTACK-bug-id:, Maven:, Doc:, etc. It says the latter group should be prefixes, which I appare

Re: [ACS41][Patch Request] agent.properties from RPM is blocking proper agent setup

2013-03-02 Thread Chip Childers
On Sat, Mar 02, 2013 at 12:28:55PM -0700, Marcus Sorensen wrote: > The bug was created a few hours before the commit. I didn't even know > I'd end up fixing it. I thought we had standardized on a BUG-ID: > header in commits, way back in October or so when Rohit was asking for > better commit messag

Re: [ACS41][Patch Request] agent.properties from RPM is blocking proper agent setup

2013-03-02 Thread Marcus Sorensen
The bug was created a few hours before the commit. I didn't even know I'd end up fixing it. I thought we had standardized on a BUG-ID: header in commits, way back in October or so when Rohit was asking for better commit messages. We even created a pre-commit hook people could use to prepopulate the

Re: [ACS41][Patch Request] agent.properties from RPM is blocking proper agent setup

2013-03-02 Thread Chip Childers
On Fri, Mar 01, 2013 at 07:37:24PM -0700, Marcus Sorensen wrote: > Please cherry pick to 4.1, it allows agent setup to complete properly > when host is added. This might actually have something to do with all > of the 'no private.network.device in agent.properties' issues popping > up from the test

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-02 Thread Chip Childers
On Fri, Mar 01, 2013 at 05:58:34PM -0700, Marcus Sorensen wrote: > It can go under CLOUDSTACK-1201, since it's the same issue, just in > the case of upgrade rather than fresh install. Done. Thanks Marcus. > > On Fri, Mar 1, 2013 at 5:54 PM, Marcus Sorensen wrote: > > I can create one. I just d

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-02 Thread Chip Childers
Is the hash right? On 4.1 branch: # git cherry-pick 6317f0bd33605c4943dcf472415e9d5c7a443d93 fatal: bad object 6317f0bd33605c4943dcf472415e9d5c7a443d93 On Fri, Mar 01, 2013 at 05:58:34PM -0700, Marcus Sorensen wrote: > It can go under CLOUDSTACK-1201, since it's the same issue, just in > the ca

[ACS41][Patch Request] agent.properties from RPM is blocking proper agent setup

2013-03-01 Thread Marcus Sorensen
Please cherry pick to 4.1, it allows agent setup to complete properly when host is added. This might actually have something to do with all of the 'no private.network.device in agent.properties' issues popping up from the testers. commit e08281838a428a26f1993519de96fc45a26c0920 Author: Marcus Sor

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-01 Thread Chip Childers
On Fri, Mar 01, 2013 at 05:58:34PM -0700, Marcus Sorensen wrote: > It can go under CLOUDSTACK-1201, since it's the same issue, just in > the case of upgrade rather than fresh install. I asked what the bug ID was, because we want to limit to blocker fixes or critical fixes (which have low risk). I

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-01 Thread Marcus Sorensen
It can go under CLOUDSTACK-1201, since it's the same issue, just in the case of upgrade rather than fresh install. On Fri, Mar 1, 2013 at 5:54 PM, Marcus Sorensen wrote: > I can create one. I just didn't realize we needed a bug ID for every > time we touch the code. I'll look and see if there's a

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-01 Thread Marcus Sorensen
I can create one. I just didn't realize we needed a bug ID for every time we touch the code. I'll look and see if there's already a general one about making RPM builds work. On Fri, Mar 1, 2013 at 5:51 PM, Chip Childers wrote: > Bug ID? > > On Mar 1, 2013, at 7:51 PM, Marcus Sorensen wrote: > >>

Re: [ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-01 Thread Chip Childers
Bug ID? On Mar 1, 2013, at 7:51 PM, Marcus Sorensen wrote: > This adds to cloud.spec post installation of cloudstack-management. It > updates the cloud user's home dir to the new location if it finds that > the cloud user's home is set to the old location. > > master 6317f0bd33605c4943dcf472415e

[ACS41][Patch Request] Update cloud user's home dir on cloudstack-management RPM install

2013-03-01 Thread Marcus Sorensen
This adds to cloud.spec post installation of cloudstack-management. It updates the cloud user's home dir to the new location if it finds that the cloud user's home is set to the old location. master 6317f0bd33605c4943dcf472415e9d5c7a443d93 applies cleanly to 4.1 as of now.

Re: [ACS41][Patch Request] CLOUDSTACK-1450

2013-03-01 Thread Chip Childers
On Fri, Mar 01, 2013 at 05:54:24PM +0530, Likitha Shetty wrote: > Patch - https://reviews.apache.org/r/9694/ > > Thank you, > Likitha > Applied to 4.1. Please pull the latest from that branch and confirm the fix now (I'll leave it to you to resolve the bug if it works). -chip

Re: [ACS41][Patch Request] CLOUDSTACK-1450

2013-03-01 Thread Chip Childers
On Mar 1, 2013, at 7:25 AM, Likitha Shetty wrote: > Patch - https://reviews.apache.org/r/9694/ > > Thank you, > Likitha Acknowledged! I'll apply as soon as I get into the office.

[ACS41][Patch Request] CLOUDSTACK-1450

2013-03-01 Thread Likitha Shetty
Patch - https://reviews.apache.org/r/9694/ Thank you, Likitha