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

Initial documentation #51

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Initial documentation #51

wants to merge 16 commits into from

Conversation

arturo-seijas
Copy link
Collaborator

@arturo-seijas arturo-seijas commented Dec 3, 2024

Applicable spec:

Overview

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

@arturo-seijas arturo-seijas added the documentation Improvements or additions to documentation label Dec 3, 2024
@arturo-seijas arturo-seijas requested a review from a team as a code owner December 3, 2024 09:30
README.md Outdated Show resolved Hide resolved
docs/how-to/contribute.md Outdated Show resolved Hide resolved
docs/how-to/contribute.md Outdated Show resolved Hide resolved
docs/how-to/contribute.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link

@erinecon erinecon left a comment

Choose a reason for hiding this comment

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

Thank you for adding all these docs!

docs/explanation/charm-architecture.md Outdated Show resolved Hide resolved
docs/explanation/charm-architecture.md Outdated Show resolved Hide resolved
docs/explanation/charm-architecture.md Outdated Show resolved Hide resolved
docs/explanation/charm-architecture.md Outdated Show resolved Hide resolved
docs/explanation/charm-architecture.md Show resolved Hide resolved
docs/tutorial/getting-started.md Outdated Show resolved Hide resolved
docs/tutorial/getting-started.md Outdated Show resolved Hide resolved
docs/tutorial/getting-started.md Show resolved Hide resolved
docs/tutorial/getting-started.md Show resolved Hide resolved
```

The deployment is complete when the status is `Active`.

Copy link

Choose a reason for hiding this comment

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

Is there a way to test that everything is working? Visit a website or something like that?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There's no website. There is an API exposed on port 55000, and the services for registering agents on 1515 and 1514, nothing very user friendly

Copy link

Choose a reason for hiding this comment

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

Thanks for explaining, I don't think we need to add more complexity to the tutorial. Let's leave it the way it is

Copy link

github-actions bot commented Dec 5, 2024

Test coverage for 29f2663

Name                            Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------------
src/certificates_observer.py       59      0      0      0   100%
src/charm.py                       92     19     24      6    73%   62-69, 107, 140-145, 155, 157, 159, 208-216
src/opensearch_observer.py         12      0      0      0   100%
src/state.py                      128     17     14      2    87%   89-90, 135-136, 163-164, 184-185, 188, 207, 209-211, 336-341
src/traefik_route_observer.py      39      0      6      0   100%
src/wazuh.py                      151     43     22      2    72%   100, 196-197, 210-216, 261->exit, 283-306, 317-329, 380-417
---------------------------------------------------------------------------
TOTAL                             481     79     66     10    82%

Static code analysis report

Run started:2024-12-05 08:52:16.846631

Test results:
  No issues identified.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Libraries: Out of sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants