puppet-nova

compute_manager is removed, we should stop using it

Bug #1674375 reported by Emilien Macchi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Nikita Karpin
Fuel for OpenStack 11.0
Packstack
Fix Released
High
Emilien Macchi
puppet-nova
Fix Released
High
Emilien Macchi
tripleo
Fix Released
High
Unassigned
tripleo pike-rc1

Bug Description

https://github.com/openstack/nova/commit/d9a7c16c305cd7016195cada883867411d3a5eb9

compute_manager is removed, we should stop using it everywhere in puppet-nova, fuel, tripleo, packstack, etc.

See original description
Emilien Macchi (emilienm)
Changed in puppet-nova:
assignee: nobody → Emilien Macchi (emilienm)
importance: Undecided → High
status: New → Triaged
summary: - compute_manager is deprecated, we should stop using it
+ compute_manager is removed, we should stop using it
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to instack-undercloud (master)

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

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

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

Emilien Macchi (emilienm)
Changed in tripleo:
status: New → In Progress
milestone: none → pike-1
assignee: nobody → Emilien Macchi (emilienm)
importance: Undecided → High
Changed in puppet-nova:
status: Triaged → In Progress
Changed in packstack:
status: New → In Progress
assignee: nobody → Emilien Macchi (emilienm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to packstack (master)

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

Nikita Karpin (mkarpin)
Changed in fuel:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Nikita Karpin (mkarpin)
milestone: none → 11.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

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

Reviewed: https://review.openstack.org/447561
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=c25dbe92487225e1209734ecaba2ff93aec42320
Submitter: Jenkins
Branch: master

commit c25dbe92487225e1209734ecaba2ff93aec42320
Author: Mykyta Karpin <email address hidden>
Date: Mon Mar 20 17:51:00 2017 +0200

    Fix nova deprecated parameters

    Upstream patch https://review.openstack.org/447533 removes
    deprecated parameters from puppet-nova, so we need update
    them in fuel-library

    Change-Id: I1c5c2d86e5ac100c2635cae8ee3506d8719d56f9
    Closes-Bug: #1674375

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

Reviewed: https://review.openstack.org/447535
Committed: https://git.openstack.org/cgit/openstack/packstack/commit/?id=88200b5f7592336fbb3d9d6213a7175a92643f0d
Submitter: Jenkins
Branch: master

commit 88200b5f7592336fbb3d9d6213a7175a92643f0d
Author: Emilien Macchi <email address hidden>
Date: Mon Mar 20 11:03:18 2017 -0400

    Remove nova compute manager option

    The option was removed in Nova:
    https://github.com/openstack/nova/commit/d9a7c16c305cd7016195cada883867411d3a5eb9

    The option had no effect and we don't need it anymore, therefore we're
    removing it everywhere.

    Change-Id: I079e239aa7ea8a1e849cd95f4297878e1b70f87b
    Related-Bug: #1674375

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

Reviewed: https://review.openstack.org/447533
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=1063de82f6ba7dfaadc25abd4c0feebcd8b4a0ff
Submitter: Jenkins
Branch: master

commit 1063de82f6ba7dfaadc25abd4c0feebcd8b4a0ff
Author: Emilien Macchi <email address hidden>
Date: Mon Mar 20 10:57:08 2017 -0400

    compute: remove deprecated options

    Remove deprecated compute options:

    - default_availability_zone
    - default_schedule_zone
    - internal_service_availability_zone
    - compute_manager

    Change-Id: Ie5dd8499eaa6770cfc11a86c7b8380c8217d3b68
    Related-Bug: #1674375

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on instack-undercloud (master)

Change abandoned by Emilien Macchi (<email address hidden>) on branch: master
Review: https://review.openstack.org/447531

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

Reviewed: https://review.openstack.org/447531
Committed: https://git.openstack.org/cgit/openstack/instack-undercloud/commit/?id=b8789f1db3942437aef13e35d00b42ff85c7eaee
Submitter: Jenkins
Branch: master

commit b8789f1db3942437aef13e35d00b42ff85c7eaee
Author: Emilien Macchi <email address hidden>
Date: Mon Mar 20 10:50:31 2017 -0400

    Remove compute_manager, deprecated in Nova

    compute_manager option has been deprecated in Nova:
    https://github.com/openstack/nova/commit/d9a7c16c305cd7016195cada883867411d3a5eb9

    And is not needed with:
    https://github.com/openstack/nova/commit/6047d790a32ef5a65d4d6b029f673ce53c3d4141

    Let's clean nova.conf and remove the useless option.

    Change-Id: Ia3ec20998db71b799874d3aaf9daa3df3ff78e3b
    Related-Bug: #1674375

Emilien Macchi (emilienm)
Changed in tripleo:
milestone: pike-1 → pike-2
Emilien Macchi (emilienm)
Changed in tripleo:
milestone: pike-2 → pike-3
Revision history for this message
Emilien Macchi (emilienm) wrote :

There are no currently open reviews on this bug, changing the status back to the previous state and unassigning. If there are active reviews related to this bug, please include links in comments.

Changed in tripleo:
status: In Progress → New
assignee: Emilien Macchi (emilienm) → nobody
Emilien Macchi (emilienm)
Changed in tripleo:
status: New → Triaged
Emilien Macchi (emilienm)
Changed in tripleo:
milestone: pike-3 → pike-rc1
Revision history for this message
Ben Nemec (bnemec) wrote :

It looks like this was fixed in TripleO. Feel free to reopen if I'm mistaken.

Changed in tripleo:
status: Triaged → Fix Released
Revision history for this message
Javier Peña (jpena-c) wrote :

This was fixed for Packstack in https://github.com/openstack/packstack/commit/88200b5f7592336fbb3d9d6213a7175a92643f0d

Changed in packstack:
status: In Progress → Fix Released
importance: Undecided → High
Tobias Urdin (tobias-urdin)
Changed in puppet-nova:
status: In Progress → Fix Released
Tobias Urdin (tobias-urdin)
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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