Inconsistent use of auxiliary files path

Bug #1843826 reported by Ryan Beisner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Glance-Simplestreams-Sync Charm
Fix Released
Medium
Ryan Beisner
OpenStack HA Cluster Charm
Fix Released
Medium
Ryan Beisner
OpenStack RabbitMQ Server Charm
Fix Released
Medium
Ryan Beisner

Bug Description

Some charms need to cargo and ship a script or other file in the charm tree. These are typically not part of actions, hooks, or other libraries. These are also typically not appropriate for Juju resource usage. Across OpenStack Charms, there are varying locations for this type of auxiliary file.

In the name of consistency, and in line with a batch normalization of tox files across OpenStack Charms, I'm proposing to settle on one consistent parent path:

rabbitmq-server has a ./scripts/ dir, let's move that to ./files/.

hacluster has an ./ocf/ dir, let's move that into a new ./files/ dir.

glance-simplestreams-sync has a ./scripts/ dir, let's move that to ./files/.

Tags: uosci
Revision history for this message
Ryan Beisner (1chb1n) wrote :
Changed in charm-glance-simplestreams-sync:
assignee: nobody → Ryan Beisner (1chb1n)
Changed in charm-hacluster:
assignee: nobody → Ryan Beisner (1chb1n)
Changed in charm-rabbitmq-server:
assignee: nobody → Ryan Beisner (1chb1n)
Changed in charm-glance-simplestreams-sync:
importance: Undecided → Medium
Changed in charm-hacluster:
importance: Undecided → Medium
Changed in charm-rabbitmq-server:
importance: Undecided → Medium
Changed in charm-glance-simplestreams-sync:
milestone: none → 19.10
Changed in charm-hacluster:
milestone: none → 19.10
Changed in charm-rabbitmq-server:
milestone: none → 19.10
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-hacluster (master)

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

Changed in charm-hacluster:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-glance-simplestreams-sync (master)

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

Changed in charm-glance-simplestreams-sync:
status: New → In Progress
Changed in charm-rabbitmq-server:
status: New → In Progress
Revision history for this message
Ryan Beisner (1chb1n) wrote :

For added context, the ceph-* charms already use the ./files/ dir location.

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

Reviewed: https://review.opendev.org/681913
Committed: https://git.openstack.org/cgit/openstack/charm-rabbitmq-server/commit/?id=8cc2df51aac4b7abda636ecdd04c4bbe3932db5e
Submitter: Zuul
Branch: master

commit 8cc2df51aac4b7abda636ecdd04c4bbe3932db5e
Author: Ryan Beisner <email address hidden>
Date: Thu Sep 12 15:12:45 2019 -0500

    Standardize auxiliary file location across os-charms

    Change-Id: I644db5de7a5dc1f41d13f2aa13cc1fa5c00c8b83
    Closes-Bug: #1843826

Changed in charm-rabbitmq-server:
status: In Progress → Fix Committed
Changed in charm-hacluster:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-hacluster (master)

Reviewed: https://review.opendev.org/681916
Committed: https://git.openstack.org/cgit/openstack/charm-hacluster/commit/?id=6ed2bb09433fb2da8d0a22185be23e04d8360814
Submitter: Zuul
Branch: master

commit 6ed2bb09433fb2da8d0a22185be23e04d8360814
Author: Ryan Beisner <email address hidden>
Date: Thu Sep 12 15:20:51 2019 -0500

    Standardize auxiliary file location across os-charms

    Change-Id: Ifaa5453bc0703c77184184e05c53d21649f6b92e
    Closes-Bug: #1843826

Changed in charm-glance-simplestreams-sync:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-glance-simplestreams-sync (master)

Reviewed: https://review.opendev.org/681920
Committed: https://git.openstack.org/cgit/openstack/charm-glance-simplestreams-sync/commit/?id=e69a562494579d3846054390636a0775421b4436
Submitter: Zuul
Branch: master

commit e69a562494579d3846054390636a0775421b4436
Author: Ryan Beisner <email address hidden>
Date: Thu Sep 12 15:27:36 2019 -0500

    Standardize auxiliary file location across os-charms

    Change-Id: Ic06c4fcdd87d350a0e41cb9de62bd7887aae7168
    Closes-Bug: #1843826

David Ames (thedac)
Changed in charm-rabbitmq-server:
status: Fix Committed → Fix Released
Changed in charm-hacluster:
status: Fix Committed → Fix Released
Changed in charm-glance-simplestreams-sync:
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.