Ryan Harper has posted comments on this change.

Change subject: storage: Move nfsSD.getFileUtils() to FileStorageDomain class
......................................................................


Patch Set 1:

I've tested this in the following manner:

Fresh install of vdsm on f16 vm.
Cleaned out /rhev/datacenter of metadata prior to starting vdsm
Then I create localfs storage domain and iso domain, and build a pool with the 
two domains, activate the data as SPM.  This brings the pool online, after 
that, I can drop in an iso file in the isoprefix path and then invoke the 
getIsoList on the pool.


[root@ichigo-dom242 ~]# ./localonly.py 
Connecting to vdsm
Checking path exists (/var/lib/vdsm/images)
Changing storage path ownership (/var/lib/vdsm/images)
Couldnt find a local storage domain, creating one...
Local storage domain configured
 `-> uuid=c132a342-50d0-4777-9e7f-7e3c27ab3f05
 `-> name=localimages
 `-> type=LOCALFS
 `-> remotePath=/var/lib/vdsm/images
Checking path exists (/var/lib/vdsm/isos)
Changing storage path ownership (/var/lib/vdsm/isos)
Couldnt find a local storage domain, creating one...
Local storage domain configured
 `-> uuid=cff0fade-0e2c-4d8a-b0cf-af1a34b97613
 `-> name=localisos
 `-> type=LOCALFS
 `-> remotePath=/var/lib/vdsm/isos
Data domain is't assciated with a pool, creating one
Connecting storage pool uuid=da656dd3-693a-4e0b-9438-93f3a5dc15b7
Acquiring SPM role for pool: uuid=da656dd3-693a-4e0b-9438-93f3a5dc15b7
Activating iso domain (uuid=cff0fade-0e2c-4d8a-b0cf-af1a34b97613) in pool 
(uuid=da656dd3-693a-4e0b-9438-93f3a5dc15b7)
Local storage pool configured
 `-> uuid=da656dd3-693a-4e0b-9438-93f3a5dc15b7
 `-> name=localpool
 `-> type=LOCALFS
 `-> 
isoprefix=/rhev/data-center/da656dd3-693a-4e0b-9438-93f3a5dc15b7/cff0fade-0e2c-4d8a-b0cf-af1a34b97613/images/11111111-1111-1111-1111-111111111111
[root@ichigo-dom242 ~]# touch 
/rhev/data-center/da656dd3-693a-4e0b-9438-93f3a5dc15b7/cff0fade-0e2c-4d8a-b0cf-af1a34b97613/images/11111111-1111-1111-1111-111111111111/winutils.iso
 
[root@ichigo-dom242 ~]# ^C
[root@ichigo-dom242 ~]# ~/work/git/vdsm-helpers/list_isos.py 
da656dd3-693a-4e0b-9438-93f3a5dc15b7
da656dd3-693a-4e0b-9438-93f3a5dc15b7
  winutils.iso

--
To view, visit http://gerrit.ovirt.org/4478
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I31cf133464751ffcf222140e83fa8d0e32940c05
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ryan Harper <[email protected]>
Gerrit-Reviewer: Ryan Harper <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to