Re: [Engine-devel] Change in ovirt-engine[master]: core: enable to decrease DC compatibility...
----- Original Message -----
From: "Livnat Peer" <lpeer(a)redhat.com>
To: "Itamar Heim" <iheim(a)redhat.com>, engine-devel(a)ovirt.org
Sent: Wednesday, March 12, 2014 12:42:44 PM
Subject: Re: [Engine-devel] Change in ovirt-engine[master]: core: enable to decrease DC
compatibility...
On 03/12/2014 11:59 AM, Itamar Heim wrote:
> On 03/12/2014 12:26 AM, emesika(a)redhat.com wrote:
>> Eli Mesika has submitted this change and it was merged.
>>
>> Change subject: core: enable to decrease DC compatibility...
>> ......................................................................
>>
>>
>> core: enable to decrease DC compatibility...
>>
>> enable to decrease DC compatibility version if DC has no clusters
>>
>> This patch enables to decrease the DC compatibility version if DC has no
>> clusters.
>
> Eli - just saw this. I'm pretty sure it would be *bad* to downgrade a DC
> version if it has storage domains as well. not sure if this is checked
> already or not.
>
> may also be an issue with some logical network features.
>
Most of the network features are driven from cluster level, we enable
using the features on all DC level (actually >=3.1) but actually enable
/disable the feature when attaching the network to a cluster.
So from network perspective I think it should be fine to downgrade the
DC level even if there are networks in the DC (at least now this could
change in future versions).
Actually we block adding or updating networks if the feature is not supported
on the network's DC level, for example: STP, Jumbo frames and non-vm network.
Therefore if the management network was configured with any of those feature,
there is a need to either block the action or to 'initialize' the network to
the default settings (as new network being added).
In general I believe the use case for this patch is mostly for empty
DCs
so for simplicity we should block it if there are networks or SD in the
DC when downgrading.
Livnat
>>
>> Change-Id: I73284f641b7f80b380b39efbbd7b4566f55119b6
>> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1057029
>> Signed-off-by: Eli Mesika <emesika(a)redhat.com>
>> ---
>> M
>>
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/UpdateStoragePoolCommand.java
>>
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> Approvals:
>> Eli Mesika: Verified
>> Ravi Nori: Looks good to me, but someone else must approve
>> Yair Zaslavsky: Looks good to me, approved
>>
>>
>>
>
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel