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.commailto:csui...@gmail.com. -Chris -- Chris Suich chris.su...@netapp.commailto:chris.su...@netapp.com NetApp

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 paul.an...@shapeblue.com wrote: Congratulations Hugo!

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

Re: 4.4 and master failing due to missing license header

2014-03-20 Thread SuichII, Christopher
line? On Thu, Mar 20, 2014 at 5:52 PM, SuichII, Christopher chris.su...@netapp.com 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 requirement. I don't see one

Re: 4.4 and master failing due to missing license header

2014-03-20 Thread SuichII, Christopher
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 chris.su...@netapp.com 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

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: 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):

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

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

Re: Unable to inject keystore on Mac OS

2014-02-28 Thread SuichII, Christopher
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 On 27 February 2014 19:49, SuichII, Christopher chris.su...@netapp.comwrote: I tried searching

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: 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 chris.su...@netapp.com wrote: 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

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: Confluence edit permissions

2014-02-25 Thread SuichII, Christopher
, 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] Sent: Tuesday, February 25, 2014 10:18 AM To: dev Subject: Confluence

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 mike.tutkow...@solidfire.com wrote:

Re: Creating a CloudStack SDK

2014-02-03 Thread SuichII, Christopher
...@citrix.com wrote: +1. I think this is a good idea. On 1/31/14 7:27 AM, SuichII, Christopher chris.su...@netapp.com 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 set up

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

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
, SolidFire, and Marcus 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: Any final thoughts on this? Letting

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
, 2014, at 3:34 PM, SuichII, Christopher chris.su...@netapp.com 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 Suich chris.su

Re: Tags on storagePool

2014-01-27 Thread SuichII, Christopher
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, Christopher chris.su...@netapp.com wrote: I think my only real concern with working around it in this manner

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

Re: Tags on storagePool

2014-01-24 Thread SuichII, Christopher
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 this. The allocators do properly ignore any storage

Re: Tags on storagePool

2014-01-24 Thread SuichII, Christopher
: 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. On Fri, Jan 24, 2014 at 8:00 AM, SuichII, Christopher chris.su...@netapp.com wrote: I think the quickest, easiest change would be to keep using

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=?

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 h...@trippaers.nl wrote: Should be fixed: commit

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

RE: Unable to start Virtual Router

2014-01-14 Thread SuichII, Christopher
with 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, clean install earlier today and everything worked

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

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

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

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, SuichII

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

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

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,

Re: Question regarding ServerApiException

2013-11-14 Thread SuichII, Christopher
message is displayed instead of the text I provided. Talk to you later On Wed, Nov 13, 2013 at 7:28 AM, SuichII, Christopher chris.su...@netapp.commailto:chris.su...@netapp.com wrote: Alena Edison (or others), Is there a reason that ServerApiException has a getDescription() method

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 chris.su...@netapp.com wrote:

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

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 chris.su...@netapp.com wrote: Bumping this. I believe we need the imput of a VMWare expert

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

2013-11-06 Thread SuichII, Christopher
, depending on if we 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: Correct. #6 FAILS with VMWare

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,

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 chris.su...@netapp.com 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

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

2013-11-05 Thread SuichII, Christopher
, as well? Is 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 really be a VMWare issue. Any

[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

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 chipchild...@apache.org 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

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

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

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

[DISCUSS] Scalable Backup and Recovery

2013-10-25 Thread SuichII, Christopher
...@netapp.commailto: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 chris.su...@netapp.commailto:chris.su...@netapp.com wrote: See responses inline. -- Chris Suich chris.su...@netapp.commailto:chris.su

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 chris.su...@netapp.com 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

Re: Modularized Spring Modules

2013-10-24 Thread SuichII, Christopher
that is something that answered your question. If not you can email me directly with your spring config and I can help, or we can setup a GTM. Darren On Thu, Oct 24, 2013 at 1:04 PM, SuichII, Christopher chris.su...@netapp.com wrote: Darren, I’m switching my plugin over to use the new

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

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

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 chris.su...@netapp.com wrote: It appears that the deployment of systemvms was broken

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

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
, 2013 at 1:05 PM, SuichII, Christopher chris.su...@netapp.com 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 This shouldn’t have come from our merge. -- Chris Suich

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
looking at a clean version of master? Darren On Oct 23, 2013, at 1:17 PM, SuichII, Christopher chris.su...@netapp.com wrote: Er, sorry. That was poorly worded on my part. Some classes, like SnapshotTest.java and all the storage providers, did not get updated references to your

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
darren.s.sheph...@gmail.com wrote: 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 chris.su...@netapp.com wrote: Yep. I’m running

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 jessica.w...@citrix.com wrote: Hi, My management server

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 chris.su...@netapp.com wrote: Adding the commit

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
, at 5:56 PM, SuichII, Christopher chris.su...@netapp.com 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 Solutions Citrix

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 chris.su...@netapp.commailto:chris.su...@netapp.com wrote: Darren, Would you be able to look into copy the logic back into your refactoring today or tomorrow

Re: StrategyPriority changes w/ Spring Changes

2013-10-23 Thread SuichII, Christopher
. Darren On Wed, Oct 23, 2013 at 3:30 PM, SuichII, Christopher chris.su...@netapp.com wrote: Take a look at revision 3 of my changes here: https://reviews.apache.org/r/14522/diff/3/#index_header The changes I made were due to discussion in the reviews. It should be simpler, cleaner and more

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.commailto:chris.su...@netapp.com NetApp Software Engineer Data Center Platforms – Cloud Solutions Citrix,

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 aemne...@gmail.com wrote: Congrats Animesh. Your contributions to 4.2 were tremendous... Looking forward

Re: Snapshot Strategy Review

2013-10-17 Thread SuichII, Christopher
:28 PM, SuichII, Christopher chris.su...@netapp.commailto: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 couple comments

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,

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
, 2013 at 1:16 PM, Marcus Sorensen shadow...@gmail.commailto:shadow...@gmail.com wrote: Ok, let me test it. On Thu, Oct 17, 2013 at 12:56 PM, SuichII, Christopher chris.su...@netapp.commailto:chris.su...@netapp.com wrote: Oh, I noticed this and created a fix, which I thought I already had submitted

Re: CLVM broken on master

2013-10-17 Thread SuichII, Christopher
, 2013 at 1:45 PM, SuichII, Christopher chris.su...@netapp.com wrote: Hm, interesting. Since nothing else in the if/else if series there depends on the src being a template, I'd imagine it would be safe to just have the check be: } else if (srcData.getObjectType

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

2013-10-14 Thread SuichII, Christopher
with it, though it looks ok on first glance. regards, Daan On Fri, Oct 11, 2013 at 9:33 PM, SuichII, Christopher chris.su...@netapp.com wrote: Just bumping this. It should be a fairly simple review. -- Chris Suich chris.su...@netapp.commailto:chris.su...@netapp.com NetApp Software Engineer

Re: [PROPOSAL] Revert to VM disk Snapshot

2013-10-11 Thread SuichII, Christopher
: 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:35 + +1. On 9/30/13 2:31 PM, SuichII, Christopher chris.su...@netapp.com wrote: The storage subsystem API

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.commailto: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

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-10 Thread SuichII, Christopher
need to enforce the whole process for everybody. Darren On Tue, Oct 8, 2013 at 2:05 PM, SuichII, Christopher chris.su...@netapp.com wrote: Whether the hypervisor snapshot happens depends on whether the 'quiesce' option is specified with the snapshot request. If a user doesn't care

Re: [PROPOSAL] Remove Setters from *JoinVO

2013-10-10 Thread SuichII, Christopher
– Cloud Solutions Citrix, Cisco Red Hat On Oct 10, 2013, at 1:40 AM, Koushik Das koushik@citrix.com wrote: Views are meant to be read only. So +1 for removing setters. On 04-Oct-2013, at 10:59 PM, SuichII, Christopher chris.su...@netapp.com wrote: *JoinVOs are used to store entries

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-10 Thread SuichII, Christopher
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 that we are still working on. I believe there are a couple options

Re: VM Backup plug-in framework

2013-10-10 Thread SuichII, Christopher
, 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/13 10:02 AM, SuichII, Christopher chris.su...@netapp.com wrote: http

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

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

2013-10-09 Thread SuichII, Christopher
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 chris.su...@netapp.com wrote: Maybe we could consider switching from a whitelist

Re: [PROPOSAL] Modularize Spring

2013-10-09 Thread SuichII, Christopher
three, I don't have much vested interest in this. Darren On Fri, Oct 4, 2013 at 3:00 PM, SuichII, Christopher chris.su...@netapp.com wrote: Well, it seems OK, but I think we should keep on discussing our options. One concern I have with the global config approach is that it adds manual

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

2013-10-09 Thread SuichII, Christopher
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. On Wed, Oct 9, 2013 at 1:32 PM, SuichII, Christopher chris.su

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

Re: [PROPOSAL] Modularize Spring

2013-10-09 Thread SuichII, Christopher
at 1:13 PM, SuichII, Christopher chris.su...@netapp.com wrote: I think I'll look into a version of (2). The difference being that I think using an int is too large of a range and provides unnecessary granularity. If two strategies or providers both have snapshot strategies, they are both

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-08 Thread SuichII, Christopher
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 chris.su...@netapp.com wrote: I can comment on the second half. Through storage operations, storage providers can create

Re: [DISCUSS] Pluggable VM snapshot related operations?

2013-10-08 Thread SuichII, Christopher
, at 4:57 PM, Darren Shepherd darren.s.sheph...@gmail.com 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 chris.su...@netapp.com wrote: -- 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

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: [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,

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

Re: Removal of cloud-plugin-netapp

2013-10-04 Thread SuichII, Christopher
/api_3.0.6/user/associateLun.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 chipchild...@apache.org wrote: On Mon, Sep 23, 2013 at 04:44:51PM +, SuichII, Christopher wrote

[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

Re: [PROPOSAL] Modularize Spring

2013-10-04 Thread SuichII, Christopher
: SuichII, Christopher [mailto:chris.su...@netapp.com] Sent: Thursday, October 03, 2013 9:02 PM To: dev@cloudstack.apache.org Subject: Re: [PROPOSAL] Modularize Spring Sure - I could see that working. Anyone have thoughts whether an enum could be used instead of an integer? That way we can

Re: [PROPOSAL] Modularize Spring

2013-10-04 Thread SuichII, Christopher
PM, Darren Shepherd darren.s.sheph...@gmail.com 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 strategies or providers

Re: [PROPOSAL] Modularize Spring

2013-10-03 Thread SuichII, Christopher
:42 PM, SuichII, Christopher chris.su...@netapp.com wrote: Thanks for the quick reply. After talking with Edison, I think what we could do is allow the strategies to either specify a priority or 'type' and then order them by that when they are loaded. For example, we could have an enum

Re: Scalable Backup and Recovery

2013-09-30 Thread SuichII, Christopher
interface and let storage providers implement it if they can. Darren On Sat, Sep 28, 2013 at 5:41 PM, SuichII, Christopher chris.su...@netapp.com wrote: Well, yes, in part. By scalable I mean that if CloudStack is expected to be able to manage such a large number of vms, it should be able

[PROPOSAL] Revert to VM disk Snapshot

2013-09-30 Thread SuichII, Christopher
The storage subsystem API currently has an interface for takeSnapshot() and an associated externally facing API for takeSnapshot. There is also a method on the primary data store interface for revertSnapshot(). However, this method is unused. We would like this storage subsystem interface

[PROPOSAL] SSVM Generic Copy

2013-09-30 Thread SuichII, Christopher
The storage subsystem currently uses a number of hypervisor APIs for transferring files between data stores (both primary and secondary). However, while implementing the storage subsystem API as a storage provider, we have discovered that there is a need for a generic copy method that can copy

[PROPOSAL] Provide Option to Quiesce VMs While Taking Snapshots

2013-09-30 Thread SuichII, Christopher
CloudStack currently snapshots vm disks by taking hypervisor snapshots. However, when implementing the storage subsystem API interface takeSnapshot(), the VM associated with the requested volume is not quiesced since a hypervisor snapshot is not necessarily taken. When creating a storage level

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

2013-09-30 Thread SuichII, Christopher
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 wrote: Is VM Quiescing sufficient to ensure consistency of the snapshot? On 9/30/13 2:43 PM, SuichII, Christopher chris.su

Re: Scalable Backup and Recovery

2013-09-28 Thread SuichII, Christopher
. The actual scalability of the snapshot process itself is limited by available disk and network bandwidth. Also certain hypervisors have various quirks which stand in the way of an efficient solution. On 9/27/13 10:27 AM, SuichII, Christopher chris.su...@netapp.com wrote: I'd like to start

Re: [DISCUSS] UI: New look and feel

2013-09-27 Thread SuichII, Christopher
Brian - The new style looks great, but I'd like to repeat someone else's question: Are we getting rid of the icons on the nav bar? As a plugin dev, it would be really nice to keep our company logo by our UI plugin. Shiva Sebastien - What impact would this angular.js project have on UI

  1   2   >