Bug 1269481 - Setting Prefix on a network instead of netmask recognized as a difference and network considered as unsynced
Summary: Setting Prefix on a network instead of netmask recognized as a difference an...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Network
Version: 3.6.0
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ovirt-3.6.2
: 3.6.2
Assignee: Dan Kenigsberg
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-10-07 12:17 UTC by Michael Burman
Modified: 2016-02-18 11:14 UTC (History)
5 users (show)

Fixed In Version: ovirt-engine-3.6.0_alpha1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-02-18 11:14:36 UTC
oVirt Team: Network
Embargoed:
rule-engine: ovirt-3.6.z+
ylavi: Triaged+
ylavi: planning_ack+
danken: devel_ack+
myakove: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 47469 0 master MERGED engine: add integer to IPv4 format converter Never
oVirt gerrit 47470 0 master MERGED engine: Introduce SubnetUtils Never
oVirt gerrit 47471 0 master MERGED engine: fix for netmask prefix out of sync Never
oVirt gerrit 47714 0 master MERGED webadmin: CidrValidation refactor Never
oVirt gerrit 47809 0 master ABANDONED engine: Introduce ArgumentAreEqual Never
oVirt gerrit 49921 0 ovirt-engine-3.6 MERGED engine: add integer to IPv4 format converter Never
oVirt gerrit 49922 0 ovirt-engine-3.6 MERGED webadmin: CidrValidation refactor Never
oVirt gerrit 49923 0 ovirt-engine-3.6 MERGED engine: Introduce SubnetUtils Never
oVirt gerrit 49924 0 ovirt-engine-3.6 MERGED engine: fix for netmask prefix out of sync Never

Description Michael Burman 2015-10-07 12:17:15 UTC
Description of problem:
Setting Prefix on a network instead of netmask recognized as a difference and network considered as unsynced.

As part of the [RFE] Allow filling also network prefix to 'netmask' fields of static IP configuration - BZ  1012881 that is already verified by me and worked fine, it is now allowed to fill Prefix (0-32) in the netmask field. But now, setting Prefix causing the network to be unsynced. 
- If using netmask(255.255.255.0 for example) network is synced.
- If using Prefix (24 or /24 for example) network becomes unsynced once approving operation via Setup Networks.

Version-Release number of selected component (if applicable):
3.6.0-0.18.el6
vdsm-4.17.8-1.el7ev.noarch

How reproducible:
100

Steps to Reproduce:
1. Create new network and attach to host
2. Edit network(pencil) and set static ip 7.7.7.7 with Routing Prefix 24 or /24
3. Approve operation

Actual results:
Network become unsynced. A difference is recognized for some reason. 
The 'out-of-sync' tool tip displays:
Host network's configurations differ from DC
Host - 255.255.255.0
DC /24

Expected results:
Network should be synced.
Used to work fine on 3.6.0-0.0.master.20150412172306.git55ba764.el6
with vdsm-4.17.0-632.git19a83a2.el7.x86_64. 


Additional info:
See also https://bugzilla.redhat.com/show_bug.cgi?id=1012881

Comment 1 Red Hat Bugzilla Rules Engine 2015-11-27 04:39:18 UTC
Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA.

Comment 2 Michael Burman 2015-11-29 14:30:51 UTC
Thanks Dan for setting target release for this report, i was just about to failedQa this bug on 3.6.1-0.2.el6 ))

Comment 3 Red Hat Bugzilla Rules Engine 2015-12-06 09:12:20 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 4 Sandro Bonazzola 2015-12-23 13:41:34 UTC
oVirt 3.6.2 RC1 has been released for testing, moving to ON_QA

Comment 5 Michael Burman 2015-12-27 10:06:39 UTC
Verified on - 3.6.2-0.1.el6 and vdsm-4.17.14-0.el7ev.noarch


Note You need to log in before you can comment on or make changes to this bug.