Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

actions: Add Tiobe TICS cron job #249

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

claudiubelu
Copy link
Contributor

@claudiubelu claudiubelu commented Jan 15, 2025

Overview

This commit adds:

  • the necessary tox env targets for generating Cobertura coverage report XML format
  • a GitHub Workflow which runs the unit tests with coverage enabled, and calls the TICSServer to upload the analysis results

Rationale

Workflow will allow for daily automatic code quality report uploads to Tiobe

Juju Events Changes

N/A

Module Changes

N/A

Library Changes

N/A

Checklist

This commit adds:

- the necessary tox env targets for generating Cobertura
  coverage report XML format
- a GitHub Workflow which runs the unit tests with coverage enabled,
  and calls the `TICSServer` to upload the analysis results

Signed-off-by: Claudiu Belu <claudiu.belu@canonical.com>
Co-Authored-By: Nashwan Azhari <nashwan.azhari@canonical.com>
@claudiubelu claudiubelu requested a review from a team as a code owner January 15, 2025 08:46
@claudiubelu
Copy link
Contributor Author

Inspired by PR: #172

Copy link
Contributor

Test coverage for 874329d

coverage-report: install_deps /home/runner/work/k8s-operator/k8s-operator/charms/worker/k8s> python -I -m pip install 'coverage[toml]'
coverage-report: commands[0] /home/runner/work/k8s-operator/k8s-operator/charms/worker/k8s> coverage report
Name                                    Stmts   Miss  Cover
-----------------------------------------------------------
lib/charms/k8s/v0/k8sd_api_manager.py     286     29    90%
src/charm.py                              501    260    48%
src/cloud_integration.py                   80      3    96%
src/config/extra_args.py                   31      2    94%
src/containerd.py                         140     22    84%
src/cos_integration.py                     33     12    64%
src/events/update_status.py                68     24    65%
src/inspector.py                           41      4    90%
src/kube_control.py                        43     32    26%
src/literals.py                            25      0   100%
src/protocols.py                           28      5    82%
src/reschedule.py                          77      4    95%
src/snap.py                               193     29    85%
src/token_distributor.py                  181    109    40%
src/upgrade.py                            108     48    56%
-----------------------------------------------------------
TOTAL                                    1835    583    68%
coverage-report: OK (1.49=setup[1.23]+cmd[0.27] seconds)
congratulations :) (1.67 seconds)

Static code analysis report

Run started:2025-01-15 08:48:53.897003

Test results:
  No issues identified.

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

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):

Copy link
Contributor

@bschimke95 bschimke95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bschimke95 bschimke95 merged commit 3a8051d into canonical:main Jan 15, 2025
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants