[4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
Hi, Just an FYI that I noticed tonight that the removed column in the volumes table is not updated when a volume is deleted in some cases. I plan to look into this tomorrow. This looks like something we will need to fix before 4.6 goes out, though. Talk to you later, -- *Mike Tutkowski*

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Remi Bergsma
Hi Mike, Thanks for the heads up. Please file a jira issue with details and keep us posted. Thanks, Remi Sent from my iPhone > On 22 Oct 2015, at 08:27, Mike Tutkowski wrote: > > Hi, > > Just an FYI that I noticed tonight that the removed column in the

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
It looks like this is an issue with managed storage. I can actually put a fix in the SolidFire plug-in to correct what's going on. I opened a JIRA ticket and will submit a PR once I test my changes (there are only a few lines changed). On Thu, Oct 22, 2015 at 1:02 AM, Remi Bergsma

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Daan Hoogland
Mik, isn't it more generic then the solidfire plugin? On Thu, Oct 22, 2015 at 8:20 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > It looks like this is an issue with managed storage. > > I can actually put a fix in the SolidFire plug-in to correct what's going > on. > > I opened a

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
Anyone who uses managed storage, but this actually effectively means just SolidFire at this point (and this code has to be implemented in the plug-in). On Thu, Oct 22, 2015 at 1:07 PM, Daan Hoogland wrote: > Mik, isn't it more generic then the solidfire plugin? > > On

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
Anyone want to take a look at this PR? https://github.com/apache/cloudstack/pull/968 It's quite short. I just needed to make sure that the volume that is to be deleted is not counted in the used-space calculation (this was throwing an exception because it wasn't supposed to be counting the