OpenStack Octavia Dashboard Charm

Built charm contains configuration options that are unexpected and/or invalid for the charm

Bug #1879088 reported by Frode Nordahl
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceph RBD Mirror Charm
Fix Released
High
Frode Nordahl
Ceph RBD Mirror Charm 20.10
OpenStack Barbican SoftHSM Charm
Fix Released
High
Frode Nordahl
OpenStack Barbican SoftHSM Charm 20.08
OpenStack Barbican-Vault Charm
Fix Committed
High
Frode Nordahl
OpenStack Cinder Backup Swift Proxy Charm
Fix Released
High
Frode Nordahl
OpenStack Cinder Backup Swift Proxy Charm 20.08
OpenStack Cinder Pure Storage Charm
Fix Committed
High
Frode Nordahl
OpenStack Designate-Bind Charm
Triaged
Medium
Unassigned
OpenStack Keystone LDAP integration
Fix Released
High
Frode Nordahl
OpenStack Keystone LDAP integration 20.08
OpenStack Manila Generic Backend Charm
Fix Released
High
Unassigned
OpenStack Manila Generic Backend Charm 20.08
OpenStack Octavia Dashboard Charm
Fix Released
High
Frode Nordahl
OpenStack Octavia Dashboard Charm 20.08
charm-octavia-diskimage-retrofit
Fix Released
High
Unassigned
charm-ovn-central
Fix Released
Critical
Frode Nordahl
charm-ovn-central 20.05
charm-ovn-chassis
Invalid
Undecided
Unassigned
charm-ovn-dedicated-chassis
Fix Released
Critical
Frode Nordahl
charm-ovn-dedicated-chassis 20.05

Bug Description

The reactive charms consume layers for re-use of common reactive helpers and configuration options.

However some of the configuration options added to the layers are not valid for all charms.

Some layers have ``deletes`` lists under the ``config`` section to save the charms from having to do this, but when the upper charm layer also adds a ``deletes`` list it appears these are not merged but the charm one replaces the above layers deletes lists. This may provide unexpected results for the built charm.

We need to revisit each individual charm and make sure that the configuration options that show up in the built charm actually make sense and have supporting code in the charm.

Any configuration options that should not be there must be removed by adding them to the ``deletes`` list under the ``config`` section in ``layer.yaml`` of the upper charm layer.

See original description
Frode Nordahl (fnordahl)
description: updated
summary: - Built charm contains configuration options that are unexpacted and/or
+ Built charm contains configuration options that are unexpected and/or
invalid for the charm
Frode Nordahl (fnordahl)
Changed in charm-ovn-central:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
milestone: none → 20.05
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-ovn-central (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728651

Frode Nordahl (fnordahl)
Changed in charm-ovn-chassis:
status: New → Invalid
Changed in charm-ovn-dedicated-chassis:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-ovn-dedicated-chassis (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728652

Frode Nordahl (fnordahl)
Changed in charm-ceph-rbd-mirror:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
milestone: none → 20.05
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-ceph-rbd-mirror (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728653

Frode Nordahl (fnordahl)
Changed in charm-octavia-dashboard:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
milestone: none → 20.05
Changed in charm-octavia-diskimage-retrofit:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
milestone: none → 20.05
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-octavia-dashboard (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728654

Frode Nordahl (fnordahl)
Changed in charm-barbican-softhsm:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
milestone: none → 20.05
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-barbican-softhsm (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728656

Frode Nordahl (fnordahl)
Changed in charm-barbican-vault:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-barbican-vault (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728657

Frode Nordahl (fnordahl)
Changed in charm-cinder-backup-swift-proxy:
status: New → In Progress
importance: Undecided → Critical
assignee: nobody → Frode Nordahl (fnordahl)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-cinder-backup-swift-proxy (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728658

Frode Nordahl (fnordahl)
Changed in charm-cinder-purestorage:
assignee: nobody → Frode Nordahl (fnordahl)
importance: Undecided → Critical
status: New → In Progress
Changed in charm-keystone-ldap:
assignee: nobody → Frode Nordahl (fnordahl)
importance: Undecided → Critical
milestone: none → 20.05
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-keystone-ldap (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728660

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-octavia-dashboard (master)

Fix proposed to branch: master
Review: https://review.opendev.org/728770

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-ovn-central (master)

Reviewed: https://review.opendev.org/728651
Committed: https://git.openstack.org/cgit/x/charm-ovn-central/commit/?id=8b733fed727048379caa34c1ca094d989be6d174
Submitter: Zuul
Branch: master

commit 8b733fed727048379caa34c1ca094d989be6d174
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 22:11:36 2020 +0200

    Remove inherited configuration options invalid for charm

    The shared OVN code that previously was distributed using the
    OVN layer is now in c-h. The ovn-central charm does not have
    common code with the chassis charms so it is safe to switch to
    consuming the openstack layer directly.

    Change-Id: I87706f0ba005d18688c4b87bc5fc1b7115fcf7d5
    Closes-Bug: #1879088

Changed in charm-ovn-central:
status: In Progress → Fix Committed
Frode Nordahl (fnordahl)
Changed in charm-barbican-softhsm:
importance: Critical → High
Changed in charm-barbican-vault:
importance: Critical → High
Changed in charm-cinder-purestorage:
importance: Critical → High
Changed in charm-keystone-ldap:
importance: Critical → High
Changed in charm-manila-generic:
importance: Undecided → High
Changed in charm-ceph-rbd-mirror:
importance: Critical → High
Changed in charm-cinder-backup-swift-proxy:
importance: Critical → High
Changed in charm-octavia-dashboard:
importance: Critical → High
Changed in charm-octavia-diskimage-retrofit:
importance: Critical → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-ovn-dedicated-chassis (master)

Reviewed: https://review.opendev.org/728652
Committed: https://git.openstack.org/cgit/x/charm-ovn-dedicated-chassis/commit/?id=15074aa4f043907f7c27376928fd072520bfa7bf
Submitter: Zuul
Branch: master

commit 15074aa4f043907f7c27376928fd072520bfa7bf
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 22:32:51 2020 +0200

    Remove inherited configuration options invalid for charm

    The charm removes some configuration options only valid for the
    subordinate chassis charm. A side effect of this is that the
    deletes section in the OVN layer is replaced by ours, and we then
    need to delete configuration options inherited from upstream
    layers ourself.

    Add missing icon.

    Change-Id: I3fd02dee2968ee773ef2b31c18f7c674b12adf43
    Closes-Bug: #1879088

Changed in charm-ovn-dedicated-chassis:
status: In Progress → Fix Committed
David Ames (thedac)
Changed in charm-ovn-dedicated-chassis:
milestone: none → 20.05
Changed in charm-ovn-central:
status: Fix Committed → Fix Released
Changed in charm-ovn-dedicated-chassis:
status: Fix Committed → Fix Released
David Ames (thedac)
Changed in charm-ceph-rbd-mirror:
milestone: 20.05 → 20.08
Changed in charm-octavia-diskimage-retrofit:
milestone: 20.05 → 20.08
Changed in charm-octavia-dashboard:
milestone: 20.05 → 20.08
Changed in charm-barbican-softhsm:
milestone: 20.05 → 20.08
Changed in charm-keystone-ldap:
milestone: 20.05 → 20.08
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-manila-generic (master)

Reviewed: https://review.opendev.org/728661
Committed: https://git.openstack.org/cgit/openstack/charm-manila-generic/commit/?id=9bf02c7ff897a2f96192c56843e6356846fbddaf
Submitter: Zuul
Branch: master

commit 9bf02c7ff897a2f96192c56843e6356846fbddaf
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:34:29 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: Ia3b01d732d1813bb94a27086e10539c0b5499a49
    Closes-Bug: #1879088

Changed in charm-manila-generic:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-octavia-dashboard (master)

Reviewed: https://review.opendev.org/728770
Committed: https://git.openstack.org/cgit/openstack/charm-octavia-dashboard/commit/?id=0a41348eabd990753da0de1b905f41f0727209e6
Submitter: Zuul
Branch: master

commit 0a41348eabd990753da0de1b905f41f0727209e6
Author: Frode Nordahl <email address hidden>
Date: Mon May 18 09:54:39 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: Ifd096de5ad0c58d3798567608ea99f258d98e1bf
    Closes-Bug: #1879088

Changed in charm-octavia-dashboard:
status: In Progress → Fix Committed
Changed in charm-cinder-purestorage:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-cinder-purestorage (master)

Reviewed: https://review.opendev.org/728659
Committed: https://git.openstack.org/cgit/openstack/charm-cinder-purestorage/commit/?id=761a970780d03f203259a6964fad7f0a6fe0e1bb
Submitter: Zuul
Branch: master

commit 761a970780d03f203259a6964fad7f0a6fe0e1bb
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:24:02 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: I7e0023acc1277a0edbdda09b6069a7c8e62c2b0a
    Closes-Bug: #1879088

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-barbican-softhsm (master)

Reviewed: https://review.opendev.org/728656
Committed: https://git.openstack.org/cgit/openstack/charm-barbican-softhsm/commit/?id=161aaef2f27eff288bdb3de8afc2bfe693d804cc
Submitter: Zuul
Branch: master

commit 161aaef2f27eff288bdb3de8afc2bfe693d804cc
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:08:59 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: I33a9602793c2a19d8d21e4524999c5c2dd51e85f
    Closes-Bug: #1879088

Changed in charm-barbican-softhsm:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-octavia-dashboard (master)

Reviewed: https://review.opendev.org/728654
Committed: https://git.openstack.org/cgit/openstack/charm-octavia-dashboard/commit/?id=df4cc4403327375f4f5cbc05ba18b3b21cad5150
Submitter: Zuul
Branch: master

commit df4cc4403327375f4f5cbc05ba18b3b21cad5150
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 22:58:42 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: Icd93c7289e2da3440b991b9d2cd389f4892c87f4
    Closes-Bug: #1879088

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-cinder-backup-swift-proxy (master)

Reviewed: https://review.opendev.org/728658
Committed: https://git.openstack.org/cgit/openstack/charm-cinder-backup-swift-proxy/commit/?id=bdbaa53af8303abbc7ad2726533464b2eb0ab371
Submitter: Zuul
Branch: master

commit bdbaa53af8303abbc7ad2726533464b2eb0ab371
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:18:44 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: I7fbc6b07d1e6fe93b67cafcba0ad0055d161dcd2
    Closes-Bug: #1879088

Changed in charm-cinder-backup-swift-proxy:
status: In Progress → Fix Committed
Changed in charm-keystone-ldap:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-keystone-ldap (master)

Reviewed: https://review.opendev.org/728660
Committed: https://git.openstack.org/cgit/openstack/charm-keystone-ldap/commit/?id=3f72380e220d44b3886b29f14273bb64d7679ee2
Submitter: Zuul
Branch: master

commit 3f72380e220d44b3886b29f14273bb64d7679ee2
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:29:14 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: I01cd3a3f53b09edb46398c7652ba19a6b1df4623
    Closes-Bug: #1879088

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-barbican-vault (master)

Reviewed: https://review.opendev.org/728657
Committed: https://git.openstack.org/cgit/openstack/charm-barbican-vault/commit/?id=c83001ca8aba7ad425dd349def85e5e03e371164
Submitter: Zuul
Branch: master

commit c83001ca8aba7ad425dd349def85e5e03e371164
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 23:15:10 2020 +0200

    Remove inherited configuration options invalid for charm

    Change-Id: I6089eae63aca83fccb7c2ecb39b21455464835b3
    Closes-Bug: #1879088

Changed in charm-barbican-vault:
status: In Progress → Fix Committed
Revision history for this message
Nobuto Murata (nobuto) wrote :

Designate-bind doesn't offer TLS related service either.

James Page (james-page)
Changed in charm-ceph-rbd-mirror:
milestone: 20.08 → none
Changed in charm-octavia-diskimage-retrofit:
milestone: 20.08 → none
Changed in charm-cinder-backup-swift-proxy:
milestone: none → 20.08
Changed in charm-manila-generic:
milestone: none → 20.08
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-ceph-rbd-mirror (master)

Reviewed: https://review.opendev.org/728653
Committed: https://git.openstack.org/cgit/openstack/charm-ceph-rbd-mirror/commit/?id=3f11c3850acdd624af37046ec2a2556dc51829db
Submitter: Zuul
Branch: master

commit 3f11c3850acdd624af37046ec2a2556dc51829db
Author: Frode Nordahl <email address hidden>
Date: Sat May 16 22:40:48 2020 +0200

    Remove inherited configuration options invalid for charm

    A side effect of adding a config keyword in layer.yaml is that
    any deletes done by upstream layers are overridden, so we
    add the complete set of configuration options inherited that
    should not be included in the built charm.

    Change-Id: I91e4e52600e7d632bbcf62402e57ebf69b6a3ea5
    Closes-Bug: #1879088

Changed in charm-ceph-rbd-mirror:
status: In Progress → Fix Committed
Alex Kavanagh (ajkavanagh)
Changed in charm-octavia-dashboard:
status: Fix Committed → Fix Released
Changed in charm-barbican-softhsm:
status: Fix Committed → Fix Released
Changed in charm-cinder-backup-swift-proxy:
status: Fix Committed → Fix Released
Changed in charm-keystone-ldap:
status: Fix Committed → Fix Released
Changed in charm-manila-generic:
status: Fix Committed → Fix Released
Frode Nordahl (fnordahl)
Changed in charm-octavia-diskimage-retrofit:
assignee: Frode Nordahl (fnordahl) → nobody
status: In Progress → Fix Released
Alex Kavanagh (ajkavanagh)
Changed in charm-ceph-rbd-mirror:
milestone: none → 20.10
Alex Kavanagh (ajkavanagh)
Changed in charm-ceph-rbd-mirror:
status: Fix Committed → Fix Released
Alex Kavanagh (ajkavanagh)
Changed in charm-designate-bind:
importance: Undecided → Medium
status: New → Triaged
tags: added: good-first-bug
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.