canonical / charm-microceph

Charm to deploy/manage microceph

Apache License 2.0

2 stars 9 forks source link

Fix set-pool-size action with multiple pools #87

Closed hemanthnakkina closed 3 weeks ago

hemanthnakkina commented 3 weeks ago

Description

Currently for set-pool-size action, the input param pools is passed directly to microceph pool set-rf command. So in case of multiple pools, the pools is passed as single string to microceph command. Instead the pools should be passed as list or space delimitted strings.

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

CI job Juju Upgrade Test is updated to verify this scenario.

Contributor's Checklist

Please check that you have:

[x] self-reviewed the code in this PR.
[x] added code comments, particularly in hard-to-understand areas.
[x] added tests to verify effectiveness of this change.