FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Gentoo > Gentoo Documentation

 
 
LinkBack Thread Tools
 
Old 01-26-2011, 11:07 PM
David Cantrell
 
Default Disable partition resize support for DASD labels (#605912)

On Wed, 26 Jan 2011, David Lehman wrote:


On Wed, 2011-01-26 at 12:28 -1000, David Cantrell wrote:

This just doesn't work and I have no idea when it will work in the
future, so we should just prevent users from trying this. Such a narrow
use case.


It would be nicer IMO to implement PartitionDevice.resizable such that
it does this new check and also checks super(StorageDevice,
self).resizable. This would remove the need to check hasattr(self,
"disk").

Dave


---
storage/devices.py | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/storage/devices.py b/storage/devices.py
index 61082be..9695470 100644
--- a/storage/devices.py
+++ b/storage/devices.py
@@ -590,6 +590,9 @@ class StorageDevice(Device):
@property
def resizable(self):
""" Can this type of device be resized? """
+ if hasattr(self, "disk") and self.disk and self.disk.type == 'dasd':
+ return False
+
return self._resizable and self.exists and
((self.format and self.format.resizable) or not self.format)





How about this revision:


commit a703eb5ae5eb4a8585d69d82084748136d551c7e
Author: David Cantrell <dcantrell@redhat.com>
Date: Wed Jan 26 11:32:12 2011 -1000

Disable partition resize support for DASD labels (#605912)

This just doesn't work and I have no idea when it will work in the
future, so we should just prevent users from trying this. Such a narrow
use case.

diff --git a/storage/devices.py b/storage/devices.py
index 61082be..82c6952 100644
--- a/storage/devices.py
+++ b/storage/devices.py
@@ -1503,6 +1503,11 @@ class PartitionDevice(StorageDevice):
else:
return 0

+ @property
+ def resizable(self):
+ """ Can this type of device be resized? """
+ return super(PartitionDevice, self).resizable and
+ (self.disk and self.disk.type != 'dasd')

class DMDevice(StorageDevice):
""" A device-mapper device """

--
David Cantrell <dcantrell@redhat.com>
Supervisor, Installer Engineering Team
Red Hat, Inc. | Honolulu, HI | UTC-10

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 

Thread Tools




All times are GMT. The time now is 09:03 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org