canonical / charm-microceph

Charm to deploy/manage microceph

Apache License 2.0

2 stars 9 forks source link

Patches to make CI healthy #67

Closed UtkarshBhatthere closed 2 months ago

UtkarshBhatthere commented 2 months ago

Description

PR to make the CI healthy, minor patches and tuning of CI system.

Fixes:

Upgrade tests when "non-exist" channels confused charm bootstrap params.
Increased Timeouts for operations which were frequently timing out.
Upgrade tests when an older snap revision could not support the "mount-observe" connection.
Cleans up detached storage before rettaching another.
Better waiting for microceph app to prevent ops before cluster bootstrap.

Type of change

Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] CleanCode (Code refactor, test updates, does not introduce functional changes)
[ ] Documentation update (Doc only change)

How Has This Been Tested?

NOTE: All functional changes should accompany corresponding tests (unit tests, functional tests etc).

Please describe the addition/modification of tests done to verify this change. Please also list any relevant details for your test configuration.

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.
[ ] updated the user documentation with corresponding changes.
[ ] added tests to verify effectiveness of this change.