OpenStack Compute (nova)

Instance tag not taken care of when bury in cell0

Bug #1806515 reported by Zhenyu Zheng
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Zhenyu Zheng
Pike
Fix Committed
Medium
Matt Riedemann
Queens
Fix Committed
Medium
Matt Riedemann
Rocky
Fix Committed
Medium
Matt Riedemann

Bug Description

We allowed add instance tags when booting in
Iac54b9627cb4398e45f1f15a2f4e7d6f86242093, conductor
will create tags for us in the selected cell, but
we forgot to consider the case that no valid host
was found.

So we will be unable to filter those instances in
cell0 with tags.

Tags: api cells
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
assignee: nobody → Zhenyu Zheng (zhengzhenyu)
status: New → In Progress
Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Medium
tags: added: api cells
OpenStack Infra (hudson-openstack)
Changed in nova:
assignee: Zhenyu Zheng (zhengzhenyu) → Matt Riedemann (mriedem)
Matt Riedemann (mriedem)
Changed in nova:
assignee: Matt Riedemann (mriedem) → Zhenyu Zheng (zhengzhenyu)
OpenStack Infra (hudson-openstack)
Changed in nova:
assignee: Zhenyu Zheng (zhengzhenyu) → Matt Riedemann (mriedem)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/621856
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=3cd439ad8a99f122566f0b5d0264ff5c8d34f998
Submitter: Zuul
Branch: master

commit 3cd439ad8a99f122566f0b5d0264ff5c8d34f998
Author: Kevin_Zheng <email address hidden>
Date: Tue Dec 4 10:25:52 2018 +0800

    Handle tags in _bury_in_cell0

    We allowed add instance tags when booting in
    Iac54b9627cb4398e45f1f15a2f4e7d6f86242093, conductor
    will create tags for us in the selected cell, but
    we forgot to consider the case that no valid host
    was found.

    So we will be unable to filter those instances in
    cell0 with tags.

    Change-Id: Idfe37d356b349024dfe963c934728111e1a1d314
    closes-bug: #1806515

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/624813

Matt Riedemann (mriedem)
Changed in nova:
assignee: Matt Riedemann (mriedem) → Zhenyu Zheng (zhengzhenyu)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/624824

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.openstack.org/624825

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/rocky)

Reviewed: https://review.openstack.org/624813
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=cbb9eb5b056b96ea507d65fea0e32c27bbf6cdfa
Submitter: Zuul
Branch: stable/rocky

commit cbb9eb5b056b96ea507d65fea0e32c27bbf6cdfa
Author: Kevin_Zheng <email address hidden>
Date: Tue Dec 4 10:25:52 2018 +0800

    Handle tags in _bury_in_cell0

    We allowed add instance tags when booting in
    Iac54b9627cb4398e45f1f15a2f4e7d6f86242093, conductor
    will create tags for us in the selected cell, but
    we forgot to consider the case that no valid host
    was found.

    So we will be unable to filter those instances in
    cell0 with tags.

    NOTE(mriedem): The backport needed to remove the mock
    of notify_about_compute_task_error since that method
    was added in Stein: Ibfb0a6db5920d921c4fc7cabf3f4d2838ea7f421

    Change-Id: Idfe37d356b349024dfe963c934728111e1a1d314
    closes-bug: #1806515
    (cherry picked from commit 3cd439ad8a99f122566f0b5d0264ff5c8d34f998)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 18.1.0

This issue was fixed in the openstack/nova 18.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/queens)

Reviewed: https://review.openstack.org/624824
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=fa178ed4e5ab925d39f7176d851d3c735e328189
Submitter: Zuul
Branch: stable/queens

commit fa178ed4e5ab925d39f7176d851d3c735e328189
Author: Kevin_Zheng <email address hidden>
Date: Tue Dec 4 10:25:52 2018 +0800

    Handle tags in _bury_in_cell0

    We allowed add instance tags when booting in
    Iac54b9627cb4398e45f1f15a2f4e7d6f86242093, conductor
    will create tags for us in the selected cell, but
    we forgot to consider the case that no valid host
    was found.

    So we will be unable to filter those instances in
    cell0 with tags.

    Change-Id: Idfe37d356b349024dfe963c934728111e1a1d314
    closes-bug: #1806515
    (cherry picked from commit 3cd439ad8a99f122566f0b5d0264ff5c8d34f998)
    (cherry picked from commit cbb9eb5b056b96ea507d65fea0e32c27bbf6cdfa)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 19.0.0.0rc1

This issue was fixed in the openstack/nova 19.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 17.0.10

This issue was fixed in the openstack/nova 17.0.10 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/pike)

Reviewed: https://review.openstack.org/624825
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=02c6e1364f3cd4fb1cd6d6e7f03a55681a1e364c
Submitter: Zuul
Branch: stable/pike

commit 02c6e1364f3cd4fb1cd6d6e7f03a55681a1e364c
Author: Kevin_Zheng <email address hidden>
Date: Tue Dec 4 10:25:52 2018 +0800

    Handle tags in _bury_in_cell0

    We allowed add instance tags when booting in
    Iac54b9627cb4398e45f1f15a2f4e7d6f86242093, conductor
    will create tags for us in the selected cell, but
    we forgot to consider the case that no valid host
    was found.

    So we will be unable to filter those instances in
    cell0 with tags.

    Conflicts:
          nova/conductor/manager.py

    NOTE(mriedem): The conflict is due to not having change
    I70b11dd489d222be3d70733355bfe7966df556aa in Pike.

    Change-Id: Idfe37d356b349024dfe963c934728111e1a1d314
    closes-bug: #1806515
    (cherry picked from commit 3cd439ad8a99f122566f0b5d0264ff5c8d34f998)
    (cherry picked from commit cbb9eb5b056b96ea507d65fea0e32c27bbf6cdfa)
    (cherry picked from commit fa178ed4e5ab925d39f7176d851d3c735e328189)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 16.1.8

This issue was fixed in the openstack/nova 16.1.8 release.

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.