NetApp&Citrix Storage plugin meeting notes -- Sep 17, 2013

Meeting notes: Date: Sep 17, 2013 Attendees: NetApp: Chris, David Citrix: Alex, Animesh, Edison, Srinivas 1. TakeSnapshot method on storage plugin. Chris: TakeSnapshot method is implemented, but backup snapshot is failed on the hypervisor side, as the default backup snapshot procedure

RE: LocalHostEndPoint seems to get called

> -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] > Sent: Wednesday, September 25, 2013 12:38 PM > To: dev@cloudstack.apache.org > Subject: LocalHostEndPoint seems to get called > > While I'm doing development and restarting things and what not, it seems >

RE: [DISCUSS] UI: New look and feel

That's nice! Do you need help to setup a demo, or coding? Just finished read < Mastering Web Application Development with AngularJS >, so trying to mastering something:) > -Original Message- > From: Shiva Teja [mailto:shivate...@gmail.com] > Sent: Thursday, September 26, 2013 9:53 PM > T

RE: [DISCUSS] UI: New look and feel

Is there anybody like Win8 style UI: especially for the icons: http://aozora.github.io/bootmetro/docs/docs-advanced-components.html > -Original Message- > From: Brian Federle [mailto:brian.fede...@citrix.com] > Sent: Friday, September 27, 2013 10:53 AM > To: dev@cloudstack.apache.org; Son

RE: [DISCUSS] Breaking out Marvin from CloudStack

Seems Marvin won't depend on commands.xml any more, thus won't depend on any artifacts build from cloudstack. So I am +1. > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Wednesday, October 02, 2013 10:14 AM > To: CloudStack Dev > Subject: [DISCUSS] Breakin

RE: [DISCUSS] Breaking out Marvin from CloudStack

any user can just simply "pip install marvin", without download/build cloudstack source code. > -Original Message- > From: Alex Huang [mailto:alex.hu...@citrix.com] > Sent: Wednesday, October 02, 2013 12:39 PM > To: dev@cloudstack.apache.org > Subject: RE: [DISCUSS] Breaking out Marvin f

RE: [PROPOSAL] Modularize Spring

> -Original Message- > From: SuichII, Christopher [mailto:chris.su...@netapp.com] > Sent: Thursday, October 03, 2013 9:02 PM > To: > Subject: Re: [PROPOSAL] Modularize Spring > > Sure - I could see that working. Anyone have thoughts whether an enum > could be used instead of an integer?

Re: Review Request 14477: Refactor Storage Related Resource Code

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14477/#review26693 --- Ship it! Ship It! - edison su On Oct. 4, 2013, 12:57 a.m

Re: Review Request 14477: Refactor Storage Related Resource Code

> On Oct. 4, 2013, 8:23 p.m., Chip Childers wrote: > > api/src/org/apache/cloudstack/api/command/user/snapshot/RevertSnapshotCmd.java, > > line 36 > > > > > > Are there any unit or integration tests for this new API ca

RE: Master doesn't build

Sorry, forget to check in some files. > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Friday, October 04, 2013 2:25 PM > To: dev@cloudstack.apache.org > Subject: Re: Master doesn't build > > Looks like the missing files were just added. > > We sh

[DISCUSS] Pluggable VM snapshot related operations?

In 4.2, we added VM snapshot for Vmware/Xenserver. The current workflow will be like the following: createVMSnapshot api -> VMSnapshotManagerImpl: creatVMSnapshot -> send CreateVMSnapshotCommand to hypervisor to create vm snapshot. If anybody wants to change the workflow, then need to either cha

Re: Review Request 14516: Added storage_provider_name to storage_pool_view

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14516/#review26743 --- Ship it! Ship It! - edison su On Oct. 7, 2013, 12:43 p.m

RE: [DISCUSS] Pluggable VM snapshot related operations?

r Platforms - Cloud Solutions > Citrix, Cisco & Red Hat > > On Oct 5, 2013, at 12:11 AM, Mike Tutkowski > wrote: > > > Well, my first thought on this is that the storage driver should not > > be telling the hypervisor to do anything. It should be responsible for

RE: [DISCUSS] Breaking out Marvin from CloudStack

Few questions: 1. About the "more object-oriented" CloudStack API python binding: Is the proposed api good enough? For example, The current hand written create virtual machine looks like: class VirtualMachine(object): @classmethod def create(cls, apiclient, services, templateid=None

RE: [DISCUSS] Breaking out Marvin from CloudStack

> -Original Message- > From: Santhosh Edukulla [mailto:santhosh.eduku...@citrix.com] > Sent: Tuesday, October 08, 2013 1:28 AM > To: dev@cloudstack.apache.org > Subject: RE: [DISCUSS] Breaking out Marvin from CloudStack > > Comments Inline. > > -Original

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

522/#comment52129> The ui change here, is there way to disable it from ui, if the storage provider is not NetApp? Or move the ui change into your plugin? - edison su On Oct. 7, 2013, 8:26 p.m., Chris Suich wrote: > > ---

RE: [DISCUSS] Pluggable VM snapshot related operations?

i-volume snapshot. Again, the difference > being that this snapshot was not truly quiesced like a hypervisor snapshot > would be. > >>>>> > >>>>> -- > >>>>> Chris Suich > >>>>> chris.su...@netapp.com > >>>&

RE: [New Feature FS] SSL Offload Support for Cloudstack

There is command in ACS, UploadCustomCertificateCmd, which can receive ssl cert, key can chain as input. Maybe can share some code? > -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] > Sent: Tuesday, October 08, 2013 1:54 PM > To: dev@cloudstack.apache.org >

RE: [DISCUSS] Pluggable VM snapshot related operations?

orage vendor Y. > > > > In this case, a new VDI appears on each SR after a hypervisor snapshot. > > > > Same idea for VMware. > > > > I don't really know how (or if) this works for KVM. > > > > I'm not clear how this multi

RE: High CPU utilization on KVM hosts while doing RBD snapshot - was Re: snapshot caused host disconnected

> -Original Message- > From: Indra Pramana [mailto:in...@sg.or.id] > Sent: Wednesday, October 09, 2013 11:07 PM > To: dev@cloudstack.apache.org; us...@cloudstack.apache.org > Subject: Re: High CPU utilization on KVM hosts while doing RBD snapshot - > was Re: snapshot caused host disconnec

Re: Review Request 14609: Added Categorized Sorting of SnapshotStrategy and DataMotionStrategy

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14609/#review26990 --- Ship it! Ship It! - edison su On Oct. 11, 2013, 7:32 p.m

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14522/#review26989 --- Ship it! Ship It! - edison su On Oct. 7, 2013, 8:26 p.m., Chris

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14577/#review26991 --- Ship it! Ship It! - edison su On Oct. 14, 2013, 1:04 p.m

RE: CLVM broken on master

For CLVM, the copy template from secondary to primary and create volume from template logic is handled by CloudStackPrimaryDataStoreDriverImpl->copyAsync, not in AncientDataMotionStrategy You can check the code: 4fb459355337c874a10f47c0224af72d6fef1ff2. > -Original Message- > From: Marcu

RE: CLVM broken on master

gt; > > On Thu, Oct 17, 2013 at 4:03 PM, Marcus Sorensen > wrote: > >> All of the above mentioned is in > >> > core/src/com/cloud/storage/resource/StorageSubsystemCommandHandler > Bas > >> e.java > >> , by the way. > &g

Re: Review Request 14715: [CLOUDSTACK-4887] CLVM broken

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14715/#review27164 --- Ship it! Ship It! - edison su On Oct. 17, 2013, 6:59 p.m

[Merge]pluggable_vm_snapshot branch

Hi All, Per discussion on the thread: http://markmail.org/message/ybw2yy5snkvkuc57, we decide to use coarse-graind interface to make taking vm snapshot pluggable. On pluggable_vm_snapshot, a new interface is added: public interface VMSnapshotStrategy { VMSnapshot takeVMSnapshot(VMSnapsho

RE: [Merge]pluggable_vm_snapshot branch

snapshot branch > > Has any testing been run against this? > > --David > > On Mon, Oct 21, 2013 at 6:26 PM, Edison Su wrote: > > Hi All, > > Per discussion on the thread: > http://markmail.org/message/ybw2yy5snkvkuc57, we decide to use coarse- > graind

RE: Question about StoragePoolHostVO

be >one entry in storage_pool_host_ref for each host and its primary storage. As >you pointed out, we should remove the entry when hostDisconnected is called. From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Tuesday, October 22, 2013 12:28 PM To: dev@cloudstack.apach

Re: Review Request 15250: Fix Hyper-V plugin RAT issue

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15250/#review28235 --- Ship it! Ship It! - edison su On Nov. 5, 2013, 11:08 p.m

Re: Review Request 15326: Added option to reload VM during in RevertToVMSnapshotCommand

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15326/#review28456 --- Ship it! Ship It! - edison su On Nov. 7, 2013, 9 p.m., Chris

Re: Review Request 15313: Updated VMSnapshotDetails* to match *Details* pattern (e.g. UserVMDetails*)

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15313/#review28457 --- Ship it! Ship It! - edison su On Nov. 7, 2013, 9 p.m., Chris

Re: Review Request 15309: Fixed bug with deleting VMWare VM Snapshots

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15309/#review28458 --- Ship it! Ship It! - edison su On Nov. 7, 2013, 9:30 p.m., Chris

RE: [VOTE] Release Apache CloudStack 4.2.1

+1(binding) Tested on my local machine with one KVM host, basic marvin vm test passed. > -Original Message- > From: Abhinandan Prateek [mailto:abhinandan.prat...@citrix.com] > Sent: Tuesday, November 12, 2013 7:53 AM > To: CloudStack Dev > Subject: [VOTE] Release Apache CloudStack 4.2.1 >

RE: Windows support for KVM

Certain Windows server hard code on how many sockets it supports: http://www.openwebit.com/c/how-to-run-windows-vm-on-more-than-2-cores-under-kvm/ > -Original Message- > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] > Sent: Monday, November 18, 2013 1:53 PM > To: dev@cloudst

RE: Windows support for KVM

> -Original Message- > From: Arnaud Gaillard [mailto:arnaud.gaill...@xtendsys.net] > Sent: Monday, November 18, 2013 6:56 AM > To: dev@cloudstack.apache.org > Subject: Windows support for KVM > > Hello, > > A few days ago I created a new Jira ticket for the support of topology in > netw

RE: [MERGE]object_store branch into master

, the implementation will be limited to storing > files no > larger than 5GB in size. Oh, this is something we don't know yet. We haven't try to upload a template which is large than 5GB, so haven't met this issue. Could you help to hack it up?:) > > Thanks, > -Jo

Review Request: object storage refactor

/scripts/zoneWizard.js 9b28c32 utils/src/com/cloud/utils/S3Utils.java b7273a1 utils/src/com/cloud/utils/UriUtils.java 3bcee7a utils/src/com/cloud/utils/script/Script.java 3632bf5 Diff: https://reviews.apache.org/r/11277/diff/ Testing --- Thanks, edison su

RE: [MERGE]object_store branch into master

> -Original Message- > From: Edison Su [mailto:edison...@citrix.com] > Sent: Monday, May 20, 2013 2:30 PM > To: dev@cloudstack.apache.org > Subject: RE: [MERGE]object_store branch into master > > > > > -Original Message- > > From: Joh

RE: Using rados-java as a new Maven dependency for KVM

Just curious: Do you plan or already implement librbd api call with progress callback? The storage operation usually takes a long time, without progress bar, it's very difficult to know what's going on at the storage side. > -Original Message- > From: Wido den Hollander [mailto:w...@wid

RE: [MERGE]object_store branch into master

> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Wednesday, May 22, 2013 1:26 PM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE]object_store branch into master > > On Wed, May 22, 2013 at 08:15:41PM +, Animesh Chaturvedi wrote: > > > > > >

RE: [MERGE]object_store branch into master

meant. > >>>>> > >>>>> Thanks > >>>>> -min > >>>>> > >>>>> On 5/22/13 2:49 PM, "John Burwell" wrote: > >>>>> >

RE: [MERGE]object_store branch into master

locked because > > of this. Yes, the master would be unstable but it would be even if we > > merge it post cutting 4.2 branch. I would rather see this coming in > > 4.2 than wait for another 6 months or so for it. Yes, this is an > > architectural change and we are learning a

RE: [VOTE] Release Apache CloudStack 4.1.0 (fifth round)

+1, tested on the same environment and at the same time. > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Tuesday, May 28, 2013 6:08 PM > To: dev@cloudstack.apache.org > Subject: RE: [VOTE] Release Apache CloudStack 4.1.0 (fifth round) > > +1

RE: [MERGE]object_store branch into master

> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Wednesday, May 29, 2013 1:41 PM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE]object_store branch into master > > On Tue, May 28, 2013 at 11:42:49PM +

RE: 4.2 snapshot: setting up ISOs - no go

Can you try: select status from host where type = 'SecondaryStorageVM'; in db? Make sure the agent in ssvm can connect back to mgt serer. > -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Wednesday, May 29, 2013 2:28 PM > To: > Subject: 4.2 snapshot:

RE: [MERGE]object_store branch into master

nks, > -John > > On May 29, 2013, at 5:15 PM, Edison Su wrote: > > > > > > >> -Original Message- > >> From: Chip Childers [mailto:chip.child...@sungard.com] > >> Sent: Wednesday, May 29, 2013 1:41 PM > >> To: dev@cloudstack.apache.o

RE: [VOTE] Pushback 4.2.0 Feature Freeze

+1[binding] on pushing back feature freeze date. > -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, May 31, 2013 8:00 AM > To: dev@cloudstack.apache.org > Subject: [VOTE] Pushback 4.2.0 Feature Freeze > > Following our discussion on the proposal

[DISCUSS] NFS cache storage issue on object_store

Let's start a new thread about NFS cache storage issues on object_store. First, I'll go through how NFS storage works on master branch, then how it works on object_store branch, then let's talk about the "issues". 0. Why we need NFS secondary storage? Nfs secondary storage is used as a pla

[DISCUSS] The plan for object_store branch

We send out the following threads to discuss the issues of object_store branch: http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201306.mbox/%3C77B337AF224FD84CBF8401947098DD87036A76%40SJCPEX01CL01.citrite.net%3E http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201306.mbox/%3CCDD22955

RE: KVM development, libvirt

I think we miss a VOTE from Jenkins, the vote from Jenkins should be taken as highest priority in each release. This kind of regression should be easily identified in Jenkins(If we have a regression test for each environment). > -Original Message- > From: Marcus Sorensen [mailto:shadow.

RE: [DISCUSS] NFS cache storage issue on object_store

dary storage services that allow clients to explicitly > request or create resources using files (i.e. java.io.File) instead of streams > (e.g. createXXX(File) or readXXXAsFile). These interfaces would provide the > storage subsystem with the hint that the client requires file access to th

RE: [DISCUSS] NFS cache storage issue on object_store

r plugins from the storage subsystem, I think we should expose > methods on the secondary storage services that allow clients to explicitly > request or create resources using files (i.e. java.io.File) instead of streams > (e.g. createXXX(File) or readXXXAsFile). These interfaces would provide

RE: [DISCUSS] NFS cache storage issue on object_store

gt; > Thanks, > -John > > On Jun 5, 2013, at 6:55 PM, Edison Su wrote: > > > > > > >> -Original Message- > >> From: John Burwell [mailto:jburw...@basho.com] > >> Sent: Wednesday, June 05, 2013 1:04 PM > >> To:

RE: Object based Secondary storage.

The Etag created by both RIAK CS and Amazon S3 seems a little bit different, in case of multi part upload. Here is the result I tested on both RIAK CS and Amazon S3, with s3cmd. Test environment: S3cmd: version: version 1.5.0-alpha1 Riak cs: Name: riak Arch: x86_64 Version : 1

RE: Object based Secondary storage.

>> > >>> Riak CS and S3 seed their hashes differently -- causing the form to > appear slightly different. In particular, Riak CS uses URI-safe base64 > encoding > which explains why the ETag values contain "-"s instead of "_"s. From a >

RE: [MERGE] disk_io_throttling to MASTER

> -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Monday, June 10, 2013 4:07 PM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE] disk_io_throttling to MASTER > > That's a good point about the "managed" property being storable in the > storage

RE: [MERGE] disk_io_throttling to MASTER

Will you be on today's Go to meeting? We can talk about your stuff. > -Original Message- > From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] > Sent: Tuesday, June 11, 2013 10:20 AM > To: dev@cloudstack.apache.org > Subject: Re: [MERGE] disk_io_throttling to MASTER > > I am OK wit

RE: PCI-Passthrough with CloudStack

> -Original Message- > From: Marcus Sorensen [mailto:shadow...@gmail.com] > Sent: Tuesday, June 11, 2013 12:10 PM > To: dev@cloudstack.apache.org > Cc: Ryousei Takano; Kelven Yang > Subject: Re: PCI-Passthrough with CloudStack > > What we need is some sort of plugin system for the libvir

Re: Review Request: Protect VNC port with password on KVM.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11058/#review21882 --- Ship it! Ship It! - edison su On May 13, 2013, 10:32 p.m., Fang

RE: [MERGE] disk_io_throttling to MASTER (Second Round)

How about set hypervisorType to Any? Haven't take a look at the master change yet. From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] Sent: Thursday, June 13, 2013 1:41 PM To: dev@cloudstack.apache.org Cc: Edison Su Subject: Re: [MERGE] disk_io_throttling to MASTER (Second

[MERGE]object_store branch into master [Second round]

nks. > -Original Message- > From: Edison Su [mailto:edison...@citrix.com] > Sent: Friday, May 17, 2013 1:11 AM > To: dev@cloudstack.apache.org > Subject: [MERGE]object_store branch into master > > Hi all, > Min and I worked on object_store branch during the last one and hal

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

I think it's due to this https://cwiki.apache.org/confluence/display/CLOUDSTACK/Zone-wide+primary+storage+target There are zone-wide storages, may only work with one particular hypervisor. For example, the data store created on VCenter can be shared by all the clusters in a DC, but only for vmwa

RE: enableStorageMaintenance

> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Friday, June 14, 2013 7:54 AM > To: > Subject: enableStorageMaintenance > > ...works great for putting down the storage into maintenance mode (looking > forward seeing this for secondary storage as we

RE: enableStorageMaintenance

> -Original Message- > From: La Motta, David [mailto:david.lamo...@netapp.com] > Sent: Monday, June 17, 2013 8:37 AM > To: > Subject: Re: enableStorageMaintenance > > Along the same lines... is there a REST command coming in 4.2 to quiesce one > or multiple virtual machines? Quiesce me

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

gt; >break; > > > > > >} > > > > > >StoragePool pol = (StoragePool)this.dataStoreMgr > > > .getPrimaryDataStore(storage.getId()); > > > > > >if (filter(avoid, pol, dskCh, plan)) { > > &

RE: Hypervisor Host Type Required at Zone Level for Primary Storage?

t > the Hypervisor orchestration components or any aspect of the Storage layer. > > Thanks, > -John > > On Jun 17, 2013, at 4:27 PM, Edison Su wrote: > > > There are storages which can only work with one hypervisor, e.g. > > Currently, Cep

RE: Hack Day at CloudStack Collaboration Conference

+1:) > -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Monday, June 17, 2013 2:21 PM > To: dev@cloudstack.apache.org > Subject: Re: Hack Day at CloudStack Collaboration Conference > > All, > > I added a Storage Architecture topic to discuss the next steps in e

RE: NFS Cache storage query

Currently, no, I can add an API for it. > -Original Message- > From: Jessica Wang [mailto:jessica.w...@citrix.com] > Sent: Tuesday, June 18, 2013 4:24 PM > To: Min Chen > Cc: dev@cloudstack.apache.org > Subject: RE: NFS Cache storage query > > Min, > > > We may need to provide a way from

RE: NFS Cache storage query

> -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Wednesday, June 19, 2013 10:42 AM > To: dev@cloudstack.apache.org > Cc: Edison Su > Subject: Re: NFS Cache storage query > > Chip, > > Your concern had not occurred to me --

RE: NFS Cache storage query

visor side refactor, to eliminate the dependence on NFS as much as possible, then we may can truly make the statement that, S3 will be the "secondary storage". > -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Wednesday, June 19, 2013 11:00 AM &

RE: NFS Cache storage query

> -Original Message- > From: John Burwell [mailto:jburw...@basho.com] > Sent: Wednesday, June 19, 2013 11:43 AM > To: Edison Su > Cc: dev@cloudstack.apache.org > Subject: Re: NFS Cache storage query > > Edison, > > Based on the stance you have out

RE: fixPath (was: committer wanted for review)

The double slash can happen in every where, there is bug fix long time ago(http://bugs.cloud.com/show_bug.cgi?id=14066), which tries to change all the path concatenation, unfortunately, it's not been checked into cloudstack. I am +1 with your patch, which at least fixes this particular issue, wit

Re: Review Request: FileUtil simplified

/LibvirtComputingResourceTest.java <https://reviews.apache.org/r/11980/#comment45535> unit test will fail when compiling the code on Mac/Windows. How about add something like: http://www.thekua.com/atwork/2008/09/running-tests-on-a-specific-os-under-junit4/ - edison su On June 19, 2013

RE: TemplateAdapterBase broken

I am fixing bugs related to object_store merge, didn't aware there are so many changes on the master related to storage code just in this Monday. Sorry guys. > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Thursday, June 20, 2013 7:15 AM > To: dev@cloudst

[DISCUSS] Do we need code review process for code changes related to storage subsystem?

For interface/API changes, we'd better have a code review, as more storage vendors and more developers outside Citrix are contributing code to CloudStack storage subsystem. The code change should have less surprise for everybody who cares about storage subsystem.

RE: fixPath (was: committer wanted for review)

, 2013 12:13 AM > To: dev > Subject: Re: fixPath (was: committer wanted for review) > > I am not authorized to access the link you are sending Edison, but interested > in the contents. Could you send it please? > > > On Wed, Jun 19, 2013 at 10:26 PM, Edison Su wrote: >

RE: fixPath (was: committer wanted for review)

format, isn't it? > Yah, I agree the patch changes too much(join path, and remove double quote, all over the places). > > On Thu, Jun 20, 2013 at 8:53 PM, Edison Su wrote: > > > I uploaded the patch to dropbox: > > https://www.dropbox.com/s/d9fn17xmho19fdc/clou

Re: Review Request: CLOUDSTACK-2571 ZWPS issues with Enabling/Clearing the Maintenance State of the Storage

efault implementation of how to do storage maintenance. - edison su On June 20, 2013, 11:53 a.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request: Fix primary datastore NPE/incorrect db entry/exception propagation for KVM on cloudstack

eption stack trace also? server/src/com/cloud/storage/StorageManagerImpl.java <https://reviews.apache.org/r/11984/#comment45630> Call lifecycle->deletedatastore - edison su On June 20, 2013, 2:42 a.m., Venkata Siva Vijayendra Bhamidipati wrote: > > ---

Re: Review Request: Bugfix CLOUDSTACK-1594: Secondary storage host always remains Alert status

, finally, it's been removed. - edison su On June 19, 2013, 6:50 a.m., roxanne chang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

RE: [DISCUSS] Do we need code review process for code changes related to storage subsystem?

> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Thursday, June 20, 2013 5:42 PM > To: dev@cloudstack.apache.org > Cc: Edison Su > Subject: Re: [DISCUSS] Do we need code review process for code changes > related to storage subsys

RE: [DISCUSS] Do we need code review process for code changes related to storage subsystem?

t; Do you think this approach will properly balance to our needs to > coordinate/review work with maintaining a smooth flow? > > Thanks, > -John > > > On Jun 21, 2013, at 2:15 PM, Edison Su wrote: > > > > > > >> -Original Message- >

RE: [DISCUSS] Do we need code review process for code changes related to storage subsystem?

think that we should be pushing all patches that rise to merge requests into > Review Board to allow potential reviewers a place to cleanly communicate > and discuss issues. > > Thanks, > -John > > On Jun 21, 2013, at 3:53 PM, Edison Su wrote: > > > > > > >>

RE: [DISCUSS] Do we need code review process for code changes related to storage subsystem?

> -Original Message- > From: Chip Childers [mailto:chip.child...@sungard.com] > Sent: Friday, June 21, 2013 1:22 PM > To: > Subject: Re: [DISCUSS] Do we need code review process for code changes > related to storage subsystem? > > On Jun 21, 2013, at 4

Re: Review Request: CLOUDSTACK-2304 [ZWPS]NPE while migrating volume from one zone wide primary to another

/#comment45767> Again, if pool.getPodId() == null, then newVol.setPodId(pool.getPodId()) will work. - edison su On June 21, 2013, 12:32 p.m., Rajesh Battala wrote: > > --- > This is an automatically generated e-mail

RE: Resource Management/Allocation for CS

Found this paper sounds interesting: http://www.sigmetrics.org/sigmetrics2013/pdfs/p67.pdf "The physical infrastructure is divided into a large pool of compute and storage servers. The former are organized into clusters consisting of tens of servers (typically 32 or so). A public cloud may cont

RE: How does the plugin model work for storage providers?

Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Tuesday, June 25, 2013 7:29 AM > To: dev@cloudstack.apache.org > Cc: Edison Su; John Burwell > Subject: Re: How does the plugin model work for storage providers? > > Understood, I've done that.

RE: Where should a system VM image be uncompressed?

> -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: Wednesday, June 26, 2013 10:01 AM > To: dev@cloudstack.apache.org > Subject: Where should a system VM image be uncompressed? > > I noticed that the system VM template is stored in S3 as a .bz2. E.g. a

Re: Review Request 11992: CLOUDSTACK-2571 ZWPS issues with Enabling/Clearing the Maintenance State of the Storage

quot; + sp.getId() + " is already in PrepareForMaintenance mode "); } } - edison su On June 27, 2013, 12:39 p.m., Rajesh Battala wrote: > > --- > This is an automaticall

Re: Review Request 11980: FileUtil simplified

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11980/#review22474 --- Ship it! Ship It! - edison su On June 27, 2013, 6:30 a.m

Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

fo template = (TemplateInfo)srcData; if (template.getHypervisorType() == HypervisorType.HperV) { return false; } } } - edison su On June 27, 2013, 10:27 a.m., Donal Lafferty wrote: > > --- > This

Re: Review Request 12106: Removed Dead Code from Management Server Hyper-V 2012 Support

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12106/#review22480 --- Ship it! Ship It! - edison su On June 27, 2013, 8:56 a.m

Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

> On June 27, 2013, 6:36 p.m., edison su wrote: > > engine/storage/datamotion/src/org/apache/cloudstack/storage/motion/AncientDataMotionStrategy.java, > > line 170 > > <https://reviews.apache.org/r/12131/diff/1/?file=312666#file312666line170> > > > >

RE: [MERGE] Simulator Storage Fixes to master (was Re: How does the plugin model work for storage providers?)

er is a2ec1daf8422d0a1c789e331a3261b05a4501060, please double check. > -Original Message- > From: Prasanna Santhanam [mailto:t...@apache.org] > Sent: Thursday, June 27, 2013 8:07 AM > To: CloudStack Dev > Cc: Edison Su; Min Chen > Subject: [MERGE] Simulator Storage Fixes to maste

Re: Review Request 12144: Create template from snapshot, failed due to the vmdk file failure.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12144/#review22492 --- Ship it! Ship It! - edison su On June 27, 2013, 11:10 p.m

RE: Review Request 12144: Create template from snapshot, failed due to the vmdk file failure.

he.org; Fang Wang; Edison Su > Cc: cloudstack > Subject: RE: Review Request 12144: Create template from snapshot, failed > due to the vmdk file failure. > > Is this applicable to 4.1? > > > -Original Message- > > From: Fang Wang [mailto:nore...@reviews.apach

RE: Uncompressed SystemVM URL

I created a uncompressed a vhd in http://jenkins.cloudstack.org/view/master/job/build-systemvm-master/lastSuccessfulBuild/artifact/tools/appliance/dist/systemvmtemplate-2013-06-27-master-hyperv.vhd You can find the latest system vm template from http://jenkins.cloudstack.org/view/master/job/build

RE: Master broken on systemVM start

Devcloud has a trick to clean up systemvm.iso: There is a bug in xapi on Ubuntu/debian(not sure the bug still there in the latest debian or not), which can't attach an ISO to a VM. I worked around the bug by creating a VDI on primary storage, copy systemvm iso into that VDI, then attach the vdi

Re: Review Request 12131: CLOUDSTACK-3215 Cannot Deploy VM when using S3 object store without NFS Cache

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12131/#review22548 --- Ship it! Ship It! - edison su On June 28, 2013, 7:46 p.m

  1   2   3   4   >