Skip to content

chore: migrate terraform modules to CC008 - #30

Merged
weiiwang01 merged 4 commits into
mainfrom
chore/terraform-cc008-migration-charmkeeper
Oct 16, 2026
Merged

weiiwang01 merged 4 commits into
mainfrom
chore/terraform-cc008-migration-charmkeeper

Conversation

@weiiwang01

Copy link
Copy Markdown
Collaborator

Migrate Terraform modules to the CC008 standard

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

license-eye has checked 136 files.

Valid Invalid Ignored Fixed
46 1 89 0
Click to see the invalid file list
  • docs/release-notes/artifacts/pr0030.yaml
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

Comment thread docs/release-notes/artifacts/pr0030.yaml
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@weiiwang01
weiiwang01 marked this pull request as ready for review October 15, 2026 04:47
@weiiwang01
weiiwang01 requested a review from a team as a code owner October 15, 2026 04:47
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Test results for commit 2b72c98

Test coverage for 2b72c98

Name           Stmts   Miss Branch BrPart  Cover   Missing
----------------------------------------------------------
src/charm.py      69      0     20      0   100%
----------------------------------------------------------
TOTAL             69      0     20      0   100%

Static code analysis report

Run started:2026-10-15 05:36:19.643899+00:00

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 509
  Total lines skipped (#nosec): 0
  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):

@github-actions

Copy link
Copy Markdown
Contributor

Test results for commit 2b72c98

Test coverage for 2b72c98

Name           Stmts   Miss Branch BrPart  Cover   Missing
----------------------------------------------------------
src/charm.py     159     62     52      5    57%   88-89, 112->114, 117-156, 164, 176-185, 212, 221-226, 234-247, 256-266, 283-293, 303-306, 325
----------------------------------------------------------
TOTAL            159     62     52      5    57%

Static code analysis report

Run started:2026-10-15 05:36:17.136046+00:00

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 800
  Total lines skipped (#nosec): 7
  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):

@weiiwang01
weiiwang01 merged commit e35d1c0 into main Oct 16, 2026
45 checks passed
@weiiwang01
weiiwang01 deleted the chore/terraform-cc008-migration-charmkeeper branch October 16, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants