Woodpecker Charm

charm-woodpecker:unit-tests

  1. Git
  2. lp:charm-woodpecker
  3. unit-tests
Last commit made on 2023-11-24
Get this branch:
git clone -b unit-tests https://git.launchpad.net/charm-woodpecker

Branch merges

Branch information

Name:
unit-tests
Repository:
lp:charm-woodpecker

Recent commits

4a62b7d... by Luciano Lo Giudice

Remove python3.9 from tox.ini

6b00155... by Luciano Lo Giudice

Add unit tests for woodpecker charm

This PR adds unit tests so that changes can be more easily checked
for consistency and correctness. In addition, it also makes the
swift-bench optional instead of mandatory, given that it's not
very useful (IMO), and it requires a binary snap file to be provided,
which is not very ergonomic nor user friendly.

When it comes to requirements, this PR aligns itself with the Jammy
enablement PR so that it can be integrated more easily.

cc41c89... by James Page

Merge pull request #2 from lmlg/bug/1940371

Make template rendering take operation type into account

c4348d8... by Luciano Lo Giudice <email address hidden>

Merge pull request #3 from gustavosr98/master

Support for Erasure Coded pools fio benchmarking

5158f64... by Gustavo Sanchez

Enhance error handling

6cdefcc... by Gustavo Sanchez

Support for Erasure Coded pools fio benchmarking

01d4c9a... by Luciano Lo Giudice

Make template rendering take operation type into account

Both reads and writes use the same set of parameters; however, doing
so can skew the results since reads should be using slightly
different configuration (i.e: 'iodepth' should be one, and the
'time_based' flag should be set). This PR aims to do so by introducing
a small logic change in the templates themselves.

1f9d1c4... by James Page

Deal with alternative charm naming

Ensure that the keyring filename and internal name matches
the deployed application name of the woodpecker charm.

Closes-Bug: 1915422

77d9498... by James Page

Fix internal name reference for woodpecker cephx keyring

b084825... by James Page

Add clat detailed reporting

This requires some metric name munging as '.' is not valid.

Percentiles get remapped from {label}_99.90000 -> {label}_99_90000.

Ideally we'd just report in the latency for all IOPS but we can't
actuall get that information easily from fio -> prometheus.