Patching feature is obsolete and should be removed from nailgun

Bug #1511499 reported by Oleg S. Gelbukh
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Oleg S. Gelbukh

Bug Description

Patching feature was initially intended to apply patches cluster-wide using deployment mechanisms of Fuel. Since then, it was obsoleted by package-based approach to delivering and applying patches.

Patching graph and calls to it should be removed from the codebase. There is a plan to reuse 'pending_release_id' parameter for upgrading of cluster, so it should be left in the model.

Changed in fuel:
milestone: none → 8.0
assignee: nobody → Oleg S. Gelbukh (gelbuhos)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/240310

Changed in fuel:
status: New → In Progress
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
importance: Undecided → Medium
tags: added: tech-debt
tags: added: area-python
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/240310
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=81fb99fc722907833768991bcb694368f53dc7f2
Submitter: Jenkins
Branch: master

commit 81fb99fc722907833768991bcb694368f53dc7f2
Author: Oleg Gelbukh <email address hidden>
Date: Wed Oct 28 21:01:59 2015 +0000

    Remove deployment graph config for patching

    Patching feature was obsoleted and is not supported anymore.

    Delete deployment graph configuration for patching. Delete condition that
    yields this configuration when cluster asked for deployment graph.

    Closes-bug: 1511499
    Change-Id: I7d73aba5b090a5d24729553fdab76bbbaea34ebd

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/280708

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to python-fuelclient (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/281338

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to python-fuelclient (master)

Reviewed: https://review.openstack.org/281338
Committed: https://git.openstack.org/cgit/openstack/python-fuelclient/commit/?id=926a4bf487ba20599338c5d4702981cdeafa7ac6
Submitter: Jenkins
Branch: master

commit 926a4bf487ba20599338c5d4702981cdeafa7ac6
Author: Igor Kalnitsky <email address hidden>
Date: Wed Feb 17 14:30:43 2016 +0200

    Remove legacy patching code

    In Fuel 5.1 we had an experimental feature - 'patching openstack env'.
    The idea was to update and to rollback OpenStack environments between
    minor releases. However, we have encounter a lot of problems with
    restoring OpenStack databases andresolving dependency hell in packages,
    so we buried it and never release it.

    This patch removes legacy code from the source tree. We can do it
    without fear, since it wasn't released in public and, moreover,
    was broken since Fuel 6.0 (no chance someone use it).

    Change-Id: I38625a354ea1f09efa24f3e4d18cd4337b7dcc5b
    Related-Bug: #1511499

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/280708
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=e91363ba9c58f2b90810ea671a234ea41e231bbf
Submitter: Jenkins
Branch: master

commit e91363ba9c58f2b90810ea671a234ea41e231bbf
Author: Igor Kalnitsky <email address hidden>
Date: Fri Feb 5 18:53:07 2016 +0200

    Remove legacy patching code

    In Fuel 5.1 we had an experimental feature - 'patching openstack env'.
    The idea was to update and to rollback OpenStack environments between
    minor releases. However, we have encounter a lot of problems with
    restoring OpenStack databases andresolving dependency hell in packages,
    so we buried it and never release it.

    This patch removes legacy code from the source tree. We can do it
    without fear, since it wasn't released in public.

    Related-Bug: #1511499
    Change-Id: I58b3fedd239eb7fe4226e51c2d6386efab14395d

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.