Changing email addresses

2014-03-27 Thread SuichII, Christopher
Hey all, I am switching to my personal email for all CloudStack communication . I will not be reachable at my work address any longer. From now on, you can reach me at csui...@gmail.com. -Chris -- Chris Suich chris.su...@netapp.com NetApp

Re: Commit Revert Request

2014-03-20 Thread SuichII, Christopher
I have reverted the commits on 4.4 and master. Sorry about that, everyone. I thought I recompiled after applying the patch but I must have missed a step… -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Mar

Re: 4.4 and master failing due to missing license header

2014-03-20 Thread SuichII, Christopher
p; Red Hat > > On Mar 20, 2014, at 12:54 PM, Daan Hoogland wrote: > >> So the offending file is in utils. Doesn't it give info above the BUIL >> FAILURE line? >> >> On Thu, Mar 20, 2014 at 5:52 PM, SuichII, Christopher >> wrote: >>> Hey all,

Re: 4.4 and master failing due to missing license header

2014-03-20 Thread SuichII, Christopher
ne? > > On Thu, Mar 20, 2014 at 5:52 PM, SuichII, Christopher > wrote: >> Hey all, >> >> I'm unable to build on 4.4 and master with no changes made in either branch. >> It appears that there are some files which don't meet the license header >> requi

4.4 and master failing due to missing license header

2014-03-20 Thread SuichII, Christopher
Hey all, I’m unable to build on 4.4 and master with no changes made in either branch. It appears that there are some files which don’t meet the license header requirement. I don’t see one, but is there a profile for ignoring the check license plugin? Alternatively, does anyone know what files a

Re: [ANNOUNCE] Change of Apache CloudStack PMC Chair

2014-03-20 Thread SuichII, Christopher
Thanks for everything, Chip and thanks to Hugo for stepping up! -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Mar 20, 2014, at 6:55 AM, Paul Angus wrote: > Congratulations Hugo! > And thanks to Chip for

Re: CloudStack implementations

2014-03-18 Thread SuichII, Christopher
I was thinking something along the same lines. The current API & UI plugin frameworks would allow for an optional plugin which can be added to any deployment post-install. It could allow for configurable reporting so that users can decide what information is acceptable to report. If we can get t

Re: orchestrateTakeVolumeSnapshot Call

2014-03-18 Thread SuichII, Christopher
Ah, yes. Now seeing this snippet I can explain the issue. So, during "orchestrateTakeVolumeSnapshot(work.getVolumeId(), work.getPolicyId(), work.getSnapshotId(), account, work.isQuiesceVm());” if an exception is thrown, it is caught and ignored by VolumeServiceImpl.takeSnapshot(VolumeInfo): @O

Re: 4.4 Feature Freeze

2014-03-04 Thread SuichII, Christopher
I’d be a fan of using 4.4 as a learning process for the community and an attempt to meet our proposed schedule. While I can appreciate the desire to get features into 4.4, there appears to be a significant backlog on JIRA and we’ve now had two consecutive versions (I’m not sure about 4.1) not me

Re: Unable to inject keystore on Mac OS

2014-02-28 Thread SuichII, Christopher
sses/cloud.keystore > -storepass vmops.com -keypass vmops.com -keyalg RSA -validity 3650 -dname > cn="Cloudstack > > Alternatively, modify your sudoers file so your user isn't prompted for a > password when attempting to use sudo. > > Hope this helps! > Ian > >

Unable to inject keystore on Mac OS

2014-02-27 Thread SuichII, Christopher
I tried searching for this issue and see that it has come up several times, but I cannot seem to find a resolution. When deploying devcloud on my Mac, I run in to this exception: INFO [c.c.s.ConfigurationServerImpl] (main:null) Processing updateSSLKeyStore INFO [c.c.s.ConfigurationServerImpl]

Re: devcloud - new version (0.4) ready for testing

2014-02-27 Thread SuichII, Christopher
; > Many thanks, > > Chris > > On Thu, Feb 27, 2014 at 3:31 PM, SuichII, Christopher > wrote: >> Chris, >> >> I tried pulling down the box this morning and playing around...a couple of >> notes: >> >> 1) For anyone trying this, make s

Re: devcloud - new version (0.4) ready for testing

2014-02-27 Thread SuichII, Christopher
Chris, I tried pulling down the box this morning and playing around…a couple of notes: 1) For anyone trying this, make sure to ‘git checkout v0.4’. The changes on master are breaking the Vagrantfile. 2) When I run ‘vagrant up xen’, I can see in VirtualBox that the machine is booted and appears

Re: Confluence edit permissions

2014-02-25 Thread SuichII, Christopher
i, I don't >> know how to make myself an administrator any more (or my privilege has been >> revoked?). Anyone know how to do it? >> >> --Alex >> >>> -Original Message- >>> From: SuichII, Christopher [mailto:chris.su...@netapp.com] &g

Confluence edit permissions

2014-02-25 Thread SuichII, Christopher
Can someone please grant me permissions to edit pages on Confluence? My username is: csuich2 Thanks! -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat

Re: [ANNOUNCE] New Committer: Chris Suich

2014-02-10 Thread SuichII, Christopher
Thanks everyone! I’m excited to continue working with this community. -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Feb 6, 2014, at 12:13 PM, Mike Tutkowski wrote: > Congratulations, Chris! > > > On T

Re: Creating a CloudStack "SDK"

2014-02-03 Thread SuichII, Christopher
I think this is a good idea. > > On 1/31/14 7:27 AM, "SuichII, Christopher" wrote: > >> I¹m curious if there is any interest in creating a ³SDK² for developing >> plugins in CloudStack. Right now, to develop my plugin, I need a >> development environment

Creating a CloudStack "SDK"

2014-01-31 Thread SuichII, Christopher
I’m curious if there is any interest in creating a “SDK” for developing plugins in CloudStack. Right now, to develop my plugin, I need a development environment set up in order to have the maven artifacts required for compiling against. This can end up being quite a bit of work to maintain if yo

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
ue as a value to using something like storage_tag as a value? >> >> If so, do you feel it is an acceptable amount of risk for 4.3 now that we >> have already begun spinning up RC builds? >> >> Thanks >> >> >> On Mon, Jan 27, 2014 at 8:40 AM, SuichII,

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
arcus have storage plug-ins. > > We could document this issue and the easy workaround of not using true and > false as a value and try to address it in 4.4. > > > On Mon, Jan 27, 2014 at 7:53 AM, SuichII, Christopher < > chris.su...@netapp.com> wrote: > >> An

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
n 24, 2014, at 3:34 PM, SuichII, Christopher wrote: > This is the approach we’ll have to take if we can’t fix this in 4.3. It > certainly works, but it isn’t ideal. > > Does anyone else have any thoughts on the impact this might have to 4.3? > > -Chris > -- > Chris Su

Re: Tags on storagePool

2014-01-24 Thread SuichII, Christopher
2014 at 8:05 AM, Mike Tutkowski < >> mike.tutkow...@solidfire.com> wrote: >> >>> I think your idea would be acceptable risk for 4.3. The upgrade logic >>> would have to perform this true-to-storage_tag conversion, too, though. >>> >>> >>

Re: Tags on storagePool

2014-01-24 Thread SuichII, Christopher
l each time you > pull it out of the DB. > > That might be too risky of a change, though, for 4.3 at this point. > > > On Fri, Jan 24, 2014 at 5:24 AM, SuichII, Christopher < > chris.su...@netapp.com> wrote: > >> I have found an additional issue related to t

Re: Tags on storagePool

2014-01-24 Thread SuichII, Christopher
I have found an additional issue related to this. The allocators do properly ignore any storage pool details whose value is true that is not actually a storage pool. However, the list storage pools API does NOT. When creating the StoragePoolResponse, it is still assumed that any storage pool det

Re: The commit ba96c8cadfa4b1dc7ee74fdb26ba5cea87e29b91 breaks a clean installation

2014-01-15 Thread SuichII, Christopher
Can you apply this to the 4.3 branch as well, please? -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Jan 15, 2014, at 9:22 AM, Hugo Trippaers wrote: > Should be fixed: > > commit 9aaea28d0dedc370888d346d

Commit 032888c broke deploydb

2014-01-15 Thread SuichII, Christopher
Sateesh, It appears that your recent commit broke the deploydb mvn profile. When running the profile this morning, I encountered the error below. I believe it comes from this line of Upgrade421to430.java: updatePstmt = conn.prepareStatement("UPDATE `cloud`.`service_offering` SET ram_size=? WHE

RE: Unable to start Virtual Router

2014-01-14 Thread SuichII, Christopher
h it...the agents' states of my VMs don't ever seem to get into the Up state. Maybe you're having similar trouble and it's impacting your virtual router? On Tue, Jan 14, 2014 at 3:03 PM, SuichII, Christopher < chris.su...@netapp.com> wrote: > I created a fresh, cle

Unable to start Virtual Router

2014-01-14 Thread SuichII, Christopher
I created a fresh, clean install earlier today and everything worked up until trying to deploy a VM. When the virtual router attempts to start, it quickly shuts down after spewing out: ‘[…] Stopping web server: apache2apache2: Could not reliably determine the server’s fully qualified domain nam

Re: Revert to volume snapshot

2014-01-13 Thread SuichII, Christopher
I can try to provide a direction to look in, but take this with a grain of salt… I believe that the XenServer best practice for ‘restoring’ a VM volume is creating a NEW volume from the snapshot and then attaching that volume to the VM INSTEAD of reverting the volume back to it’s previous state.

Re: netapp plugin

2014-01-09 Thread SuichII, Christopher
Mike - thanks for pointing Victor to our video. Victor - I’m not sure that everyone on the dev list wants to see this whole conversation, so feel free to contact David (cc’d) and I off list and we can talk more in depth. -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Dat

Re: createSnapshot API always returns success

2014-01-07 Thread SuichII, Christopher
For what it is worth, a very similar issue is occurring for deleting snapshots. However, that issue is only with the UI. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Jan 7, 2014, at 10:08 AM, Sui

createSnapshot API always returns success

2014-01-07 Thread SuichII, Christopher
It appears that, no matter what, the createSnapshot API will ALWAYS return a success. If you drill down from: VolumeApiServiceImpl.orchestrateTakeVolumeSnapshot() to VolumeServiceImpl.takeSnapshot(), you’ll notice that ANY exception that is thrown inside of SnapshotManager.takeSnapshot() is simp

Old jars on nightly System VM templates

2014-01-06 Thread SuichII, Christopher
I updated to the latest System VM templates from http://jenkins.buildacloud.org/job/build-systemvm-master/ and the CloudStack jars in /usr/local/cloud/systemvm/ appear to be from Nov. 1. Should the System VM build be pulling newer jars than that? -Chris -- Chris Suich chris.su...@netapp.com

Re: [Doc] [4.3] Pluggable VM Snapshot for Review

2013-12-23 Thread SuichII, Christopher
Hm. I may just be missing it, but I don’t see a section on implementing VMSnapshotStrategy… -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Dec 22, 2013, at 5:33 AM, Radhika Puthiyetath wrote: > Hi, > >

Re: Blocker bug: can't create the volumes on Xen 6.2.0 on the latest 4.3

2013-12-09 Thread SuichII, Christopher
Any chance this is a vhd-util issue? This is the generic error I generally see when vhd-util isn’t in place or can’t be executed. -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Dec 9, 2013, at 6:58 PM, Ale

Re: Review Request 15362: Fixed issue with detail view fields marked as 'isPassword' not being obfuscated

2013-11-14 Thread SuichII, Christopher
Bumping this. Can someone take a look at it, please? -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Nov 8, 2013, at 3:33 PM, Chris Suich wrote: > > ---

Re: Question regarding ServerApiException

2013-11-14 Thread SuichII, Christopher
> get caught elsewhere and a more generic (somewhat useless) error message is > displayed instead of the text I provided. > > Talk to you later > > > On Wed, Nov 13, 2013 at 7:28 AM, SuichII, Christopher > mailto:chris.su...@netapp.com>> wrote: > Alena & Ed

Question regarding ServerApiException

2013-11-13 Thread SuichII, Christopher
Alena & Edison (or others), Is there a reason that ServerApiException has a getDescription() method rather than using getMessage() like all other exceptions? This causes problems in places where you’d like to treat all exceptions generically and be able to simply call getMessage(), but ServerAp

Re: Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-07 Thread SuichII, Christopher
change behind its back will cause problem and it does not provide official > API to manipulate at volume level for a VM snapshot. > > This is a bug > > Kelven > > On 11/6/13, 5:58 AM, "SuichII, Christopher" wrote: > >> Bumping this. I believe we nee

Re: Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-06 Thread SuichII, Christopher
can or can't get a VMware person to chime in > on this issue, we may have to disallow disks from being detached from > VMware VMs with snapshots in 4.3. > > > On Tue, Nov 5, 2013 at 11:44 AM, SuichII, Christopher < > chris.su...@netapp.com> wrote: > >> Corr

Re: Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-05 Thread SuichII, Christopher
Xen OK with 6? > > > On Tue, Nov 5, 2013 at 11:26 AM, SuichII, Christopher < > chris.su...@netapp.com> wrote: > >> FWIW, after looking into this more with Xen, when the VM is restored in >> step 4, it simply no longer has the volume attached, so this appears to &

Re: Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-05 Thread SuichII, Christopher
Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Nov 5, 2013, at 1:05 PM, SuichII, Christopher wrote: > We currently don’t allow volumes to be attached to VMs with snapshots and > allowing volumes to be detached causes quite a bug: > > 1) Attach a data

Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-05 Thread SuichII, Christopher
We currently don’t allow volumes to be attached to VMs with snapshots and allowing volumes to be detached causes quite a bug: 1) Attach a data disk to a VM 2) Snapshot the VM 3) Detach the data disk 4) Attempt to restore the VM from the snapshot — FAILS since the data disk is no longer there, al

Re: [DISCUSS] Add 'Other' category to gen_toc.py for apidoc

2013-11-04 Thread SuichII, Christopher
Citrix, Cisco & Red Hat On Nov 4, 2013, at 11:29 AM, Chip Childers wrote: > On Mon, Nov 04, 2013 at 04:19:12PM +, SuichII, Christopher wrote: >> I would like to be able to leverage the existing apidoc tools to generate >> apidoc for my plugin’s exposed APIs. However, gen

[DISCUSS] Add 'Other' category to gen_toc.py for apidoc

2013-11-04 Thread SuichII, Christopher
I would like to be able to leverage the existing apidoc tools to generate apidoc for my plugin’s exposed APIs. However, gen_toc.py expects all APIs to fall into one of the hard coded ‘known_categories’. Since my APIs don’t fall into one of the existing categories, gen_toc.py fails with the error

Re: CitrixResourceBase execute(StartCommand) only works with ISCSI managed storage

2013-11-01 Thread SuichII, Christopher
u tell me, are you using NFS? What is your managed storage type? Thanks! On Fri, Nov 1, 2013 at 8:32 AM, SuichII, Christopher mailto:chris.su...@netapp.com>> wrote: It looks like some changes made in 858ce766659101eb731c83c806892dd5d9baa976 prohibit any managed storage pool other than IS

CitrixResourceBase execute(StartCommand) only works with ISCSI managed storage

2013-11-01 Thread SuichII, Christopher
It looks like some changes made in 858ce766659101eb731c83c806892dd5d9baa976 prohibit any managed storage pool other than ISCSI from starting a VM. It looks like you’ assuming that if the storage pool is managed then we should use getIscsiSR() to get the xen SR, which isn’t the case. I also beli

[DISCUSS] Scalable Backup and Recovery

2013-10-25 Thread SuichII, Christopher
tapp.com<mailto:chris.su...@netapp.com> NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Sep 30, 2013, at 8:37 AM, SuichII, Christopher mailto:chris.su...@netapp.com>> wrote: See responses inline. -- Chris Suich chris.su...@netapp.com<ma

Re: [PROPOSAL] Enable "non-ACS distributed" API extension to be added CloudStack w/o modifying commands.properties

2013-10-24 Thread SuichII, Christopher
On first glance it looks great! I’ll look at it first thing in the morning since it is getting late here in EST. Thanks! Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 24, 2013, at 6:08 PM, Darren Sheph

Re: Modularized Spring Modules

2013-10-24 Thread SuichII, Christopher
ypes to be registered, then you > should use ComponentLifecycle.start(). All other initialization code > should be in @PostConstruct. ComponentLifecycle.configure() is > largely useless in my mind and should be avoided. > > I hope somewhere in all that is something that answered yo

Re: [DISCUSS] make commands.properties the exception, not the rule

2013-10-24 Thread SuichII, Christopher
, Cisco & Red Hat On Oct 9, 2013, at 3:49 PM, SuichII, Christopher wrote: > I just wanted to add a little clarification from a plugin perspective. > > Having commands.properties as a whitelist just adds another place that > plugins have to register with CloudStack. For plugins th

Re: Modularized Spring Modules

2013-10-24 Thread SuichII, Christopher
& Red Hat On Oct 24, 2013, at 4:04 PM, SuichII, Christopher wrote: > Darren, > > I’m switching my plugin over to use the new modularized Spring stuff you just > merged and there is something I’m still battling with. I have other beans > that were previously ins

Modularized Spring Modules

2013-10-24 Thread SuichII, Christopher
Darren, I’m switching my plugin over to use the new modularized Spring stuff you just merged and there is something I’m still battling with. I have other beans that were previously instantiated before my DataStoreProvider which get injected into the provider, lifecycle, etc. So, those beans nee

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
e really optimizing a couple of CPU > cycles. > > Darren > > On Wed, Oct 23, 2013 at 3:30 PM, SuichII, Christopher > wrote: >> Take a look at revision 3 of my changes here: >> https://reviews.apache.org/r/14522/diff/3/#index_header >> >> The changes I ma

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
to do? As it stands right now, what is on master, I'm not aware of any issues. Darren On Wed, Oct 23, 2013 at 3:22 PM, SuichII, Christopher mailto:chris.su...@netapp.com>> wrote: Darren, Would you be able to look into copy the logic back into your refactoring today or tomorrow? If not,

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
2013, at 5:56 PM, SuichII, Christopher wrote: > My understanding is that it is still a work in progress to get those test > back running. Is this correct, Edison? > > -- > Chris Suich > chris.su...@netapp.com > NetApp Software Engineer > Data Center Platforms – Cloud Sol

Re: Deploy systemvms broken on master

2013-10-23 Thread SuichII, Christopher
Bumping this as system vms still can’t be deployed on master. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 12:43 PM, SuichII, Christopher wrote: > Adding the commit author. > >

Re: Unable to start SSVM in latest code of master branch: "Unable to start s-15-VM due to java.lang.NullPointerException at com.cloud.hypervisor.xen.resource.XenServer56FP1Resource.createVmFromTemplat

2013-10-23 Thread SuichII, Christopher
Yes - I sent out an email about it earlier. I’ll bump the thread. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 6:19 PM, Jessica Wang wrote: > Hi, > > My management server is unable to start S

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
rote: > I fixed all the compilation errors in engine/storage/integration-test. > I don't know how to run those test though, so I can't validate the > changes. > > Darren > > On Wed, Oct 23, 2013 at 1:53 PM, SuichII, Christopher > wrote: >> Yep. I’m running

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
a clean version > of master? > > Darren > >> On Oct 23, 2013, at 1:17 PM, "SuichII, Christopher" >> wrote: >> >> Er, sorry. That was poorly worded on my part. Some classes, like >> SnapshotTest.java and all the storage providers, did not get updat

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
gt; > On Wed, Oct 23, 2013 at 1:05 PM, SuichII, Christopher > wrote: >> And it looks like some of your changes may have not merged correctly. I’m >> getting compile errors like: >> >> The method close() is undefined for the type Transaction >> >>

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
And it looks like some of your changes may have not merged correctly. I’m getting compile errors like: The method close() is undefined for the type Transaction This shouldn’t have come from our merge. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
It looks like the changes from us didn’t make it through your merge at all. I should have been more clear in my review - there are logic changes that needed to be carried from my changes into yours, not simply a rebase. I will work on those changes and try to get a patch up shortly. -- Chris S

Re: Deploy systemvms broken on master

2013-10-23 Thread SuichII, Christopher
Adding the commit author. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 23, 2013, at 12:39 PM, SuichII, Christopher wrote: > It appears that the deployment of systemvms was broken by

Deploy systemvms broken on master

2013-10-23 Thread SuichII, Christopher
It appears that the deployment of systemvms was broken by commit e33ae74067d5d7ff165098028c0bd6897f14bbd2, specifically, the change here

Re: [ANNOUNCE] New PMC member: Animesh Chaturvedi

2013-10-21 Thread SuichII, Christopher
Congrats, Animesh! -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 21, 2013, at 3:26 PM, Ahmad Emneina wrote: > Congrats Animesh. Your contributions to 4.2 were tremendous... Looking > forward to more great t

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-21 Thread SuichII, Christopher
John, Will you be able to take a look at this revision soon? It is a small change to address your comments so it should not take very long. Thanks, Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix,

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
ot;,"accountId":1,"c >>>>>> >>>>> hecksum":"6cea42b2633841648040becb588bd8f0","hvm":false,"displayText": >>>>>> "SystemVM Template >>>>>> (KVM)","imageDataStore":{"org.a

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
ate to another 50G primary root disk, since the primary storage is neither thin nor clone-able. On Thu, Oct 17, 2013 at 1:16 PM, Marcus Sorensen mailto:shadow...@gmail.com>> wrote: Ok, let me test it. On Thu, Oct 17, 2013 at 12:56 PM, SuichII, Christopher mailto:chris.su...@netapp.com>&g

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
Oh, I noticed this and created a fix, which I thought I already had submitted since it was a part of the storage refactoring a couple weeks back. I'll post the patch for review now. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Ci

Re: Snapshot Strategy Review

2013-10-17 Thread SuichII, Christopher
17, 2013, at 1:28 PM, "SuichII, Christopher" mailto:chris.su...@netapp.com>> wrote: John, I'm glad to hear you think these changes look better. It sounds like everything else you mentioned is fairly minor, so I have no problems making some changes. However, I have made a cou

Re: Review Request 14577: Remove Setters from *JoinVO Classes

2013-10-14 Thread SuichII, Christopher
I would have to >> find some time to play around with it, though it looks ok on first >> glance. >> >> regards, >> Daan >> >> On Fri, Oct 11, 2013 at 9:33 PM, SuichII, Christopher >> wrote: >>> Just bumping this. It should be a fairly simple re

Re: Review Request 14577: Remove Setters from *JoinVO Classes

2013-10-11 Thread SuichII, Christopher
Just bumping this. It should be a fairly simple review. -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 10, 2013, at 1:04 PM, Chris Suich mailto:chris.su...@netapp.com>> wrote: T

Re: [PROPOSAL] Revert to VM disk Snapshot

2013-10-11 Thread SuichII, Christopher
David Ortiz wrote: >> >>> This sounds like a great idea. >>> >>>> From: chiradeep.vit...@citrix.com >>>> To: dev@cloudstack.apache.org >>>> Subject: Re: [PROPOSAL] Revert to VM disk Snapshot >>>> Date: Mon, 30 Sep 2013 21:53

Re: VM Backup plug-in framework

2013-10-10 Thread SuichII, Christopher
are not the intended recipient, please > notify the sender prior to deleting this email message from your system. > Please note ViFX reserves the right to monitor, from time to time, the > communications sent to and from its email network. > > > > > > > On 9/10/1

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-10 Thread SuichII, Christopher
eds to snapshot the entire volume (which is > analogous to the SR). > > > On Thu, Oct 10, 2013 at 6:29 AM, SuichII, Christopher < > chris.su...@netapp.com> wrote: > >> Multivendor snapshotting: >> The case with two storage providers is a bit trickier and is one tha

Re: [PROPOSAL] Remove Setters from *JoinVO

2013-10-10 Thread SuichII, Christopher
ata Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 10, 2013, at 1:40 AM, Koushik Das wrote: > Views are meant to be read only. So +1 for removing setters. > > On 04-Oct-2013, at 10:59 PM, "SuichII, Christopher" > wrote: > >> *JoinVOs ar

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-10 Thread SuichII, Christopher
; VV >>>>>Current disksnapshot-a >>>>> b. from snapshot-a, find out its parent disk, then take snapshot >>>>> through NetApp >>>>> c. un- quiesce VM, here, go to hypervisor, de

Re: [PROPOSAL] Modularize Spring

2013-10-09 Thread SuichII, Christopher
& Red Hat >> >> On Oct 4, 2013, at 6:10 PM, Darren Shepherd >> wrote: >> >>> Sure, I'm open to suggestions. Basically I think we've discussed >>> >>> 1) Global Setting >>> 2) canHandle() returns an int >>> 3) Strate

Re: [DISCUSS/PROPOSAL] Upgrading Driver Model

2013-10-09 Thread SuichII, Christopher
Interesting. I'm not sure how I missed this thread... I'll try to chime in where I can, then. However, everything going on in here sounds like work for post-4.3, but if we are adding revert volume snapshot to 4.3, we will need a solution to that before then. It seems like the idea I've got for t

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-09 Thread SuichII, Christopher
f ties in with Storage Tagging and how that is problematic, as > well. > > With Storage Tagging, there is no indication of what storage provider > supports the Compute or Disk Offering in question and, as such, we don't > know what fields to show to or hide from users. &g

Re: [PROPOSAL] Modularize Spring

2013-10-09 Thread SuichII, Christopher
> I'm open to all three, I don't have much vested interest in this. > > Darren > > On Fri, Oct 4, 2013 at 3:00 PM, SuichII, Christopher > wrote: >> Well, it seems OK, but I think we should keep on discussing our options. One >> concern I have wit

Re: [DISCUSS] make commands.properties the exception, not the rule

2013-10-09 Thread SuichII, Christopher
s the all mighty master of what is > allowed (a whitelist). But by default, I think the file should be > empty and default to what is defined by the API annotation. > > Darren > > On Tue, Oct 8, 2013 at 5:45 PM, SuichII, Christopher > wrote: >> Maybe we could consider switch

Re: Review Request 14522: [CLOUDSTACK-4771] Support Revert VM Disk from Snapshot

2013-10-09 Thread SuichII, Christopher
Just bumping this since there haven't been any responses. Does anyone have any thoughts on this? I'm ready and prepared to do the work, but I don't want to move on if people have concerns with this approach or can think of a better solution. -Chris -- Chris Suich chris.su...@netapp.com

Re: [DISCUSS] make commands.properties the exception, not the rule

2013-10-08 Thread SuichII, Christopher
Maybe we could consider switching from a whitelist to a blacklist, then. A whitelist is certainly easier in terms of a one-step configuration, but a blacklist would allow for much easier plugin development, installation and removal. Perhaps we could find write a script that generates the complet

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-08 Thread SuichII, Christopher
Hat On Oct 8, 2013, at 4:57 PM, Darren Shepherd wrote: > Who is going to decide whether the hypervisor snapshot should actually > happen or not? Or how? > > Darren > > On Tue, Oct 8, 2013 at 12:38 PM, SuichII, Christopher > wrote: >> >> -- >> Ch

Re: VM Backup plug-in framework

2013-10-08 Thread SuichII, Christopher
Hi Simon, The answer to your question is 'kind of'. The Storage Subsystem API (https://cwiki.apache.org/confluence/display/CLOUDSTACK/Storage+subsystem+2.0) allows storage providers to implement a suite of methods for the purpose of storage management, including taking and reverting snapshots o

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-08 Thread SuichII, Christopher
With filesystems like NFS, things are certainly more complicated, but that is taken care of by our controller's operating system, Data ONTAP, and we simply use APIs to communicate with it. > > Darren > > On Tue, Oct 8, 2013 at 11:10 AM, SuichII, Christopher > wrote: >&g

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-08 Thread SuichII, Christopher
org/confluence/display/CLOUDSTACK/Pluggable+VM+snapshot+related+operations, >> feel free to add items on it. >> And a new branch "pluggable_vm_snapshot" is created. >> >>> -Original Message- >>> From: SuichII, Christopher [mailto:chris.su...

Re: Review Request 14477: Refactor Storage Related Resource Code

2013-10-07 Thread SuichII, Christopher
Thanks for the notes, Edison. Chip - Edison hit all the important points, but I'm not sure what the proper etiquette is here. I'd be more than happy to add some tests, but after looking through the code, I can't find any tests to model after. While I don't mind coming up with a model for testin

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-07 Thread SuichII, Christopher
I'm a fan of option 2 - this gives us the most flexibility (as you stated). The option is given to completely override the way VM snapshots work AND storage providers are given to opportunity to work within the default VM snapshot workflow. I believe this option should satisfy your concern, Mik

Re: RHEV with CloudStack

2013-10-07 Thread SuichII, Christopher
Hi Fabrice, RHEV is not currently supported. At this time, CloudStack supports VMWare, KVM, XenServer and XenCloud. -Chris -- Chris Suich chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix, Cisco & Red Hat On Oct 7, 2013, at 4:11 AM, Fabrice Brazier

Re: [PROPOSAL] Modularize Spring

2013-10-04 Thread SuichII, Christopher
2013, at 12:12 PM, Darren Shepherd wrote: > On 10/04/2013 11:58 AM, SuichII, Christopher wrote: >> Darren, >> >> I think one of the benefits of allowing the priority to be specified in the >> xml is that it can be configured after deployment. If for some reason two

Re: [PROPOSAL] Modularize Spring

2013-10-04 Thread SuichII, Christopher
t returns 100. If >> it >> does not handle at all you return -1. This effectively gives you >> prioritization. >> So the calling code would still loop through all strategies each time >> looking for >> the best match. I don't want the priority to

[PROPOSAL] Remove Setters from *JoinVO

2013-10-04 Thread SuichII, Christopher
*JoinVOs are used to store entries from MySQL views, which are not editable. I think removing setters from the *JoinVOs may help avoid some potential confusion as setters seem to imply that the fields are editable, which they really aren't. I started looking around and it looks like most setter

Re: Removal of cloud-plugin-netapp

2013-10-04 Thread SuichII, Christopher
html> >> API set the LUN mask on the filer for that LUN. Then the ISCSI initiator >> on the bare metal host / vm can login to the LUN. >> >> >> >> On 9/23/13 10:32 AM, "Chip Childers" wrote: >> >>> On Mon, Sep 23, 2013 a

Re: [PROPOSAL] Modularize Spring

2013-10-03 Thread SuichII, Christopher
ogress on the commands.properties as I realized it >>>> will overlap with the rbac work being done. I will start a thread >>>> regarding that so maybe it will be address in the rbac work. >>>> >>>> Regarding log4j. First, why do you have to change

Re: [PROPOSAL] Modularize Spring

2013-10-03 Thread SuichII, Christopher
nfig >> file and configure cloudstack to use that. One of the things I don't >> particularily like is that ACS forces log4j to its own configuration >> file and will not allow the regular log4j config discovery. So what I >> could do is add a JVM property like -Dclou

Re: [PROPOSAL] Modularize Spring

2013-10-03 Thread SuichII, Christopher
opping your jar in a directory and restarting. You pointed out two things I didn't think about though, command.properties and log4j xml. Let me think about those twos as they should be address also. Basically you should never have to edit a file that is packaged as part of ACS. Only

Re: [PROPOSAL] Provide Option to Quiesce VMs While Taking Snapshots

2013-09-30 Thread SuichII, Christopher
uiescing the DB app (like SQL Server via VSS) - does not give >> you a properly quiesced snapshot on the data disk - just a point-in-time >> snapshot. >> >> >> On Mon, Sep 30, 2013 at 3:52 PM, Chiradeep Vittal < >> chiradeep.vit...@citrix.com> wr

  1   2   >