Charm needs policyd functional test (tracking)

Bug #1845639 reported by Alex Kavanagh
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard Charm
Fix Released
Medium
Alex Kavanagh
OpenStack Designate Charm
Triaged
Medium
Unassigned
OpenStack Neutron Gateway Charm
Invalid
Undecided
Unassigned
OpenStack Nova Cloud Controller Charm
Triaged
Medium
Unassigned
OpenStack Nova Compute Charm
Invalid
Undecided
Unassigned

Bug Description

This is a tracking bug for charms that have policyd override implemented, but don't have a corresponding functional test as they still use the amulet framework for testing. This bug should be revisited when the charm is converted to zaza tests so that the policyd override can be explicitly tested as part of the charm gate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-neutron-gateway (master)

Change abandoned by Alex Kavanagh (tinwood) (<email address hidden>) on branch: master
Review: https://review.opendev.org/685331
Reason: neutron-gateway isn't actually an API charm and so doesn't need this change!

Changed in charm-neutron-gateway:
status: New → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to charm-nova-compute (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/685720

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-nova-compute (master)

Change abandoned by Alex Kavanagh (tinwood) (<email address hidden>) on branch: master
Review: https://review.opendev.org/685720
Reason: not needed, as not an api charm. Good job it was quick to do!

Changed in charm-nova-compute:
status: New → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to charm-nova-cloud-controller (master)

Reviewed: https://review.opendev.org/685726
Committed: https://git.openstack.org/cgit/openstack/charm-nova-cloud-controller/commit/?id=f7f6fa295c82ea88953ec08695c68c04694fc1a5
Submitter: Zuul
Branch: master

commit f7f6fa295c82ea88953ec08695c68c04694fc1a5
Author: Alex Kavanagh <email address hidden>
Date: Mon Sep 30 15:47:53 2019 +0100

    Policyd override implementation

    This patchset implements policy overrides for nova-cloud-controler.

    This change includes a charm-helpers sync to bring in the policyd helper
    code.

    Note there are no functional tests for this feature as the charm still
    uses the old style non-zaza amulet framework. The Related-Bug below is
    tracking this issue.

    Change-Id: Ia5f3f8189d4a7b7b46a827707d964ebe40740aeb
    Closes-Bug: #1741723
    Related-Bug: #1845639

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

Change abandoned by Alex Kavanagh (tinwood) (<email address hidden>) on branch: master
Review: https://review.opendev.org/686006
Reason: OpenStack dashboard requires a much more complex set of policy overrides than the system in charm-helpers can support. It'll require a completely different approach, but can be based on the existing code.

Changed in charm-openstack-dashboard:
status: New → In Progress
assignee: nobody → Alex Kavanagh (ajkavanagh)
importance: Undecided → Medium
Changed in charm-nova-cloud-controller:
status: New → Triaged
importance: Undecided → Medium
Changed in charm-designate:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Alex Kavanagh (ajkavanagh) wrote :

Note that (for designate) this bug (https://bugs.launchpad.net/charm-designate/+bug/1808590) can probably be resolved using the policy.d override feature.

Changed in charm-openstack-dashboard:
status: In Progress → Fix Committed
James Page (james-page)
Changed in charm-openstack-dashboard:
milestone: none → 20.02
Liam Young (gnuoy)
Changed in charm-openstack-dashboard:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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