Bug 1277495

Summary: remove the pre-3.6 setupnetworks api
Product: [oVirt] ovirt-engine Reporter: Martin Mucha <mmucha>
Component: RestAPIAssignee: Martin Mucha <mmucha>
Status: CLOSED CURRENTRELEASE QA Contact: Michael Burman <mburman>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.0.0CC: bugs, danken, juan.hernandez, mburman, myakove, ylavi
Target Milestone: ovirt-4.0.0-alphaKeywords: Improvement
Target Release: 4.0.0Flags: rule-engine: ovirt-4.0.0+
pstehlik: testing_plan_complete-
rule-engine: planning_ack+
danken: devel_ack+
myakove: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ovirt 4.0.0 alpha1 Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-05 08:03:42 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Martin Mucha 2015-11-03 13:19:11 UTC
Description of problem:
Currently we support three ways to do certain tasks, for example detaching network, breaking bond etc. The oldest one is quite buggy and it's not worthy to have it in code base if it's not even maintained any more.

Comment 1 Martin Mucha 2015-11-03 13:36:55 UTC
*** Bug 1277496 has been marked as a duplicate of this bug. ***

Comment 2 Dan Kenigsberg 2015-11-05 14:26:36 UTC
Juan, would you guide us how to mark an existing API as deprecated?

Comment 3 Red Hat Bugzilla Rules Engine 2015-11-05 14:26:40 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 Juan Hernández 2015-11-05 14:40:09 UTC
The old "setupnetworks" action is already marked as deprecated in 3.6:

  https://github.com/oVirt/ovirt-engine/commit/d269e5f0552f99c9b70b95456a9f481be8c795e9#diff-3205c1e6ccdb0cbdefc9e4392d9feb9cR2094

The next logical step is to remove it completely for the master branch. To do so remove it and update this file:

  https://github.com/oVirt/ovirt-engine/blob/master/backend/manager/modules/restapi/README.md

If possible include there instructions on how to to migrate from the old action to the new one. The more detailed the better.

Comment 5 Sandro Bonazzola 2016-07-05 08:03:42 UTC
oVirt 4.0.0 has been released, closing current release.