Catch openstack auth issues

yhaliaw commented 1 month ago

Applicable spec:

Overview

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

[ ] The charm style guide was applied
[ ] The contributing guide was applied
[ ] The changes are compliant with ISD054 - Managing Charm Complexity
[ ] The documentation is generated using src-docs
[ ] The documentation for charmhub is updated.
[ ] The PR is tagged with appropriate label (urgent, trivial, complex)

github-actions[bot] commented 1 month ago

Lint checks failed for 720cf367e3858b75433cb415e14130acb08d2e23

```

/home/runner/work/github-runner-operator/github-runner-operator/src/openstack_cloud/openstack_cloud.py:92:5: DCO054 "OpenStackError" exception should not be described in the docstring, more information: https://github.com/jdkandersson/flake8-docstrings-complete#fix-dco054 /home/runner/work/github-runner-operator/github-runner-operator/src/openstack_cloud/openstack_cloud.py:123:15: DCO053 "OpenStackUnauthorizedError" exception should be described in the docstring, more information: https://github.com/jdkandersson/flake8-docstrings-complete#fix-dco053

github-actions[bot] commented 1 month ago

Test coverage for 720cf367e3858b75433cb415e14130acb08d2e23

Name                                   Stmts   Miss Branch BrPart  Cover   Missing
----------------------------------------------------------------------------------
src/charm.py                             590    148    151     26    72%   234-236, 302-321, 339-341, 342->346, 372-376, 449, 456-458, 485-490, 507-513, 534, 546-552, 566-567, 580, 585, 615-616, 618->627, 622->627, 632-638, 672, 676-681, 732-737, 746->749, 772-784, 788-789, 817-844, 857-862, 881-891, 905-907, 954-955, 957-958, 960-961, 1040->1042, 1107-1108, 1146-1148, 1156-1162, 1240-1293, 1297
src/charm_state.py                       468     17    106      5    95%   335-347, 572-576, 698-699, 754-755, 1190->1193, 1197-1198, 1245
src/errors.py                             44      0      0      0   100%
src/event_timer.py                        54      7      2      1    86%   105-106, 131, 148-149, 165-166
src/firewall.py                           51     18     20      0    61%   42-43, 66-69, 111-185
src/github_client.py                      96     16     40      5    79%   65-72, 119->exit, 124-125, 195, 218, 231-238, 260->300, 294
src/github_type.py                        50      0      0      0   100%
src/logrotate.py                          43      0      2      0   100%
src/lxd_type.py                           35      0      2      0   100%
src/manager/cloud_runner_manager.py       53      0      6      0   100%
src/manager/github_runner_manager.py      18      0      6      0   100%
src/manager/runner_manager.py            135     34     48      5    70%   173, 185, 189, 205-210, 228-229, 266-282, 296-303, 323-327, 336
src/manager/runner_scaler.py              95     10     32      3    88%   123-124, 135-136, 140, 147, 194-195, 210-211
src/metrics/__init__.py                    0      0      0      0   100%
src/metrics/events.py                     55      2      8      1    95%   56->59, 166-167
src/metrics/github.py                     16      0      0      0   100%
src/metrics/runner.py                    144     10     32      3    91%   164, 190-203, 239, 276, 458-459
src/metrics/runner_logs.py                24      5      4      1    79%   30-33, 47->46, 51-52
src/metrics/storage.py                    70      8     12      0    90%   89-90, 117-118, 182-183, 189-190
src/metrics/type.py                        5      0      0      0   100%
src/openstack_cloud/__init__.py           26      0      2      0   100%
src/reactive/__init__.py                   0      0      0      0   100%
src/reactive/consumer.py                  40      2      6      0    96%   98-101
src/reactive/runner_manager.py            53      0     14      1    99%   102->exit
src/runner_manager_type.py                51      0      8      0   100%
src/runner_type.py                        37      0     10      0   100%
src/shared_fs.py                          93     17     10      1    83%   51-52, 119-120, 145-146, 154-155, 161-162, 180, 183-184, 196-197, 240-241
src/utilities.py                          68      7     20      7    82%   86->88, 90->96, 103, 133, 147, 186-189, 244
----------------------------------------------------------------------------------
TOTAL                                   2414    301    541     59    85%

Static code analysis report

Run started:2024-09-06 09:24:50.142170

Test results:
    No issues identified.

Code scanned:
    Total lines of code: 8369
    Total lines skipped (#nosec): 4
    Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 7

Run metrics:
    Total issues (by severity):
        Undefined: 0
        Low: 0
        Medium: 0
        High: 0
    Total issues (by confidence):
        Undefined: 0
        Low: 0
        Medium: 0
        High: 0
Files skipped (0):

canonical / github-runner-operator