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

ISD-2548 update README #74

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Thanhphan1147
Copy link
Collaborator

Update the charm’s README on GitHub to follow the template outlined in is-charms-template-repo.

Checklist

@Thanhphan1147 Thanhphan1147 requested a review from a team as a code owner November 9, 2024 01:15
@Thanhphan1147 Thanhphan1147 added trivial documentation Improvements or additions to documentation labels Nov 9, 2024
Copy link
Contributor

github-actions bot commented Nov 9, 2024

Test coverage for c24ab8a

Name                                       Stmts   Miss Branch BrPart  Cover   Missing
--------------------------------------------------------------------------------------
src/charm.py                                 171     12     16      3    91%   142-145, 152, 196, 395, 415-420
src/exception.py                               2      2      0      0     0%   6-10
src/resource_manager/gateway.py               77      4      8      0    95%   91, 154, 209-210
src/resource_manager/http_route.py            64      0      0      0   100%
src/resource_manager/permission.py            19      0      2      0   100%
src/resource_manager/resource_manager.py      43      4     10      4    85%   40, 100, 104, 122
src/resource_manager/secret.py                54      2      2      1    95%   65, 160
src/resource_manager/service.py               43      1      0      0    98%   122
src/state/base.py                             11      0      4      0   100%
src/state/config.py                           42      0      4      0   100%
src/state/exception.py                         2      0      0      0   100%
src/state/gateway.py                           8      0      0      0   100%
src/state/http_route.py                       29      0      2      0   100%
src/state/tls.py                              29      0      4      0   100%
src/state/validation.py                       31      2      2      0    94%   75-76
src/tls_relation.py                           78      5     10      4    90%   59, 97-103, 117, 129, 145->exit, 169->exit
--------------------------------------------------------------------------------------
TOTAL                                        703     32     64     12    94%
Coverage failure: total of 94 is less than fail-under=95

Static code analysis report

Run started:2024-11-09 01:17:23.074715

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 2964
  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):

project represents the next generation of Kubernetes Ingress, Load Balancing,
and Service Mesh APIs. From the outset, it has been designed to be generic,
expressive, and role-oriented.
A Juju charm that deploys and manages a [Gateway API integator](https://gateway-api.sigs.k8s.io/) on Kubernetes. Gateway API is an official Kubernetes project focused on L4 and L7 routing in Kubernetes. This project represents the next generation of Kubernetes Ingress, Load Balancing, and Service Mesh APIs. From the outset, it has been designed to be generic, expressive, and role-oriented. This charm simplifies initial deployment and "day N" operations of Gateway API Integrator. It allows for deployment on many different Kubernetes platforms, including [Canonical K8s](https://ubuntu.com/kubernetes).

Choose a reason for hiding this comment

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

I'd prefer to remove jargon like "day N" whenever possible, so possibly rephrase the beginning of that sentence to read "This charm simplifies deployment and operations..."

To begin, refer to the [Getting Started](https://charmhub.io/gateway-api-integrator/docs/tutorial-getting-started) tutorial for step-by-step instructions.

## Learn more
* [Read more](https://charmhub.io/gateway-api-integrator/docs)

Choose a reason for hiding this comment

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

Are there developer docs we can point to here?

I think it would be good to also list the official webpage: https://gateway-api.sigs.k8s.io

## Project and community
## Get started
To begin, refer to the [Getting Started](https://charmhub.io/gateway-api-integrator/docs/tutorial-getting-started) tutorial for step-by-step instructions.

Choose a reason for hiding this comment

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

Are there any basic operations (e.g. actions) or integrations worth mentioning for this charm?


## Project and community
The Gateway API Integrator Operator is a member of the Ubuntu family. It's an
open source project that warmly welcomes community projects, contributions,
suggestions, fixes and constructive feedback.
* [Code of conduct](https://ubuntu.com/community/code-of-conduct)
* [Get support](https://discourse.charmhub.io/)
* [Contribute](https://charmhub.io/gateway-api-integrator#contributing-to-this-documentation)

Choose a reason for hiding this comment

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

I think it would be good to also include a link to Matrix in this list

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 trivial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants