Fuel for OpenStack

Missing extension management capabilities for nailgun

  1. Series mitaka
  2. Bug #1614526
Bug #1614526 reported by Alexander Gordeev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Alexander Gordeev
Fuel for OpenStack 10.0
Mitaka
Fix Committed
High
Alexander Gordeev
Fuel for OpenStack 9.1

Bug Description

Nailgun allows the extensions to be executed on per cluster basis. However, this capability isn't exposed to API yet. Meanwhile, at least it gets already used for data pipelines and inside of orcherstrator/_serializers.py

For some reasons, the code for enabling the extensions isn't landed yet:

Not finished patches:
https://review.openstack.org/#/c/264714/
https://review.openstack.org/#/c/264699/

https://blueprints.launchpad.net/fuel/+spec/extensions-management

Both, 9 and 10 version affected.

OpenStack Infra (hudson-openstack)
Changed in fuel:
status: New → In Progress
OpenStack Infra (hudson-openstack)
Changed in fuel:
assignee: Alexander Gordeev (a-gordeev) → Vladimir Kuklin (vkuklin)
OpenStack Infra (hudson-openstack)
Changed in fuel:
assignee: Vladimir Kuklin (vkuklin) → Alexander Gordeev (a-gordeev)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/359857

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

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

commit 5a6bd3f4905f3b8f39aec78fc98c139c3db224f4
Author: Sylwester Brzeczkowski <email address hidden>
Date: Thu Jan 7 11:10:38 2016 +0100

    Add extensions API handler for cluster

    The patch includes new handlers for extensions management for
    clusters, unit and integration tests.

    Co-Authored-By: Sylwester Brzeczkowski <email address hidden>

    Change-Id: I4b81ec5b4ae6986068832bd28c08831881feeee6
    Blueprint: extensions-management
    Closes-Bug: #1614526

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/360016

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/360021

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

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

commit 06686519a0945e955b6c95a3305255dc3f033445
Author: Alexander Gordeev <email address hidden>
Date: Wed Aug 24 20:11:04 2016 +0300

    Cluster.extenstions shouldn't have duplications

    PUT converts input data to set(), so
    ClusterExtensionsHandler will never produce duplications
    in a list of enabled extensions.

    Change-Id: I3631f9ba8bebc24102c13544d8463afdddd0bef4
    Blueprint: extensions-management
    Related-Bug: #1614526

Alexander Gordeev (a-gordeev)
Changed in fuel:
status: Fix Committed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/359857
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=0a61251dbb5600d1c081ce770d677825d3513a4e
Submitter: Jenkins
Branch: stable/mitaka

commit 0a61251dbb5600d1c081ce770d677825d3513a4e
Author: Sylwester Brzeczkowski <email address hidden>
Date: Thu Jan 7 11:10:38 2016 +0100

    Add extensions API handler for cluster

    The patch includes new handlers for extensions management for
    clusters, unit and integration tests.

    Co-Authored-By: Sylwester Brzeczkowski <email address hidden>

    Backported from 5a6bd3f4905f3b8f39aec78fc98c139c3db224f4

    Change-Id: I4b81ec5b4ae6986068832bd28c08831881feeee6
    Blueprint: extensions-management
    Closes-Bug: #1614526

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/360021
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=2d8575ae4b9fd399945d118a1a41a906ad874c6b
Submitter: Jenkins
Branch: stable/mitaka

commit 2d8575ae4b9fd399945d118a1a41a906ad874c6b
Author: Alexander Gordeev <email address hidden>
Date: Wed Aug 24 20:11:04 2016 +0300

    Cluster.extenstions shouldn't have duplications

    PUT converts input data to set(), so
    ClusterExtensionsHandler will never produce duplications
    in a list of enabled extensions.

    Change-Id: I3631f9ba8bebc24102c13544d8463afdddd0bef4
    Blueprint: extensions-management
    Related-Bug: #1614526
    (cherry picked from commit 06686519a0945e955b6c95a3305255dc3f033445)

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/362842

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

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

commit 28dd1ff3a1a80174b199ebc43c7596b4a37eeb51
Author: Sylwester Brzeczkowski <email address hidden>
Date: Fri Jan 22 15:37:24 2016 +0100

    Extensions handler for Nailgun

    Added new API handler /extensions/ which returns a list of available
    extensions.

    Co-Authored-By: Sylwester Brzeczkowski <email address hidden>

    Change-Id: I072f3e4d9cd8f71498a20312af7c2016e951456d
    Implements: blueprint extensions-management
    Partial-Bug: #1614526

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/364853

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/364853
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=e1dbcecb766b5d436a5147e964f1199bff98304b
Submitter: Jenkins
Branch: stable/mitaka

commit e1dbcecb766b5d436a5147e964f1199bff98304b
Author: Sylwester Brzeczkowski <email address hidden>
Date: Fri Jan 22 15:37:24 2016 +0100

    Extensions handler for Nailgun

    Added new API handler /extensions/ which returns a list of available
    extensions.

    Co-Authored-By: Sylwester Brzeczkowski <email address hidden>

    Change-Id: I072f3e4d9cd8f71498a20312af7c2016e951456d
    Implements: blueprint extensions-management
    Partial-Bug: #1614526
    (back ported from commit 28dd1ff3a1a80174b199ebc43c7596b4a37eeb51)

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/368018

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

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

commit d9f01a647272dfab1426653704f1ad1d2df6ec50
Author: Alexander Gordeev <email address hidden>
Date: Fri Sep 9 16:48:47 2016 +0300

    Improve cluster extensions handlers

    There handlers available under api/v1/clusters/{id}/extensions

    * Re-work GET handler. Since now it's possible to enable extensions
      in more granular way which doesn't affect already enabled
      extensions.
    * Add DELETE handler. Accepts comma separated list of extension
      names to disable via query string under 'extension_names' param.

    Change-Id: Idd5680b489b4003bb2f58b5279a7257a128cc4e1
    Related-Bug: #1614526

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/369192

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/369223

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

Reviewed: https://review.openstack.org/369192
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=21f2a940fe505ecc416b4d91784fe777dde537ee
Submitter: Jenkins
Branch: stable/mitaka

commit 21f2a940fe505ecc416b4d91784fe777dde537ee
Author: Alexander Gordeev <email address hidden>
Date: Fri Sep 9 16:48:47 2016 +0300

    Improve cluster extensions handlers

    There handlers available under api/v1/clusters/{id}/extensions

    * Re-work GET handler. Since now it's possible to enable extensions
      in more granular way which doesn't affect already enabled
      extensions.
    * Add DELETE handler. Accepts comma separated list of extension
      names to disable via query string under 'extension_names' param.

    Change-Id: Idd5680b489b4003bb2f58b5279a7257a128cc4e1
    Related-Bug: #1614526
    (back ported from commit d9f01a647272dfab1426653704f1ad1d2df6ec50)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-web (stable/mitaka)

Related fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/370053

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/370291

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

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

commit ea08c45295b788d9f51fd4d969a4235f221ba3aa
Author: Alexander Gordeev <email address hidden>
Date: Wed Aug 31 20:59:21 2016 +0300

    Add fuel2 commands to operate on extensions

    Releases new fuel2 commands to operate on extensions:
        fuel2 env extension enable
        fuel2 env extension disable
        fuel2 env extension show
        fuel2 extension list

    DocImpact

    Also, this change introduces the option for BaseListCommand for
    specifying the default fields to sort by.

    Change-Id: Idee06633689efece18838766de5c4afcd24190d7
    Implements: blueprint extensions-management
    Partial-Bug: #1614526

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (stable/mitaka)

Reviewed: https://review.openstack.org/370291
Committed: https://git.openstack.org/cgit/openstack/python-fuelclient/commit/?id=6d69ee31e72f36d8e4266e887f8ec332df8127e6
Submitter: Jenkins
Branch: stable/mitaka

commit 6d69ee31e72f36d8e4266e887f8ec332df8127e6
Author: Alexander Gordeev <email address hidden>
Date: Wed Aug 31 20:59:21 2016 +0300

    Add fuel2 commands to operate on extensions

    Releases new fuel2 commands to operate on extensions:
        fuel2 env extension enable
        fuel2 env extension disable
        fuel2 env extension show
        fuel2 extension list

    DocImpact

    Also, this change introduces the option for BaseListCommand for
    specifying the default fields to sort by.

    Change-Id: Idee06633689efece18838766de5c4afcd24190d7
    Implements: blueprint extensions-management
    Partial-Bug: #1614526
    (cherry picked from commit ea08c45295b788d9f51fd4d969a4235f221ba3aa)

Revision history for this message
Alexander Gordeev (a-gordeev) wrote :

All required code pieces were merged both to stable/mitaka and master branches.

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

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

commit f72dd0967d24d00f9c82e341ce427ca309ce8f59
Author: Alexander Gordeev <email address hidden>
Date: Tue Sep 13 11:15:23 2016 +0300

    Remove list conversion for cluster extension handler

    Gets rid of unnecessary conversion to list for accepted
    data as a set for DELETE handler and then passing it straight
    to validator without extra ordinary conversion back to set.

    Change-Id: Ie24d20f68b96feba4443214de47603c5802be613
    Related-Bug: #1614526

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0rc1

This issue was fixed in the openstack/fuel-web 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0

This issue was fixed in the openstack/fuel-web 10.0.0 release.

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

Reviewed: https://review.openstack.org/370053
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=eb33869a2ebb7e306b9a1a4a5c3df5b80321df1f
Submitter: Jenkins
Branch: stable/mitaka

commit eb33869a2ebb7e306b9a1a4a5c3df5b80321df1f
Author: Alexander Gordeev <email address hidden>
Date: Tue Sep 13 11:15:23 2016 +0300

    Remove list conversion for cluster extension handler

    Gets rid of unnecessary conversion to list for accepted
    data as a set for DELETE handler and then passing it straight
    to validator without extra ordinary conversion back to set.

    Change-Id: Ie24d20f68b96feba4443214de47603c5802be613
    Related-Bug: #1614526
    (cherry picked from commit f72dd0967d24d00f9c82e341ce427ca309ce8f59)

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

Other bug subscribers

Related blueprints

Remote bug watches

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