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

Add a Helm Chart and CI Processes #6

Merged
merged 5 commits into from
Aug 9, 2024
Merged

Add a Helm Chart and CI Processes #6

merged 5 commits into from
Aug 9, 2024

Conversation

jonathanio
Copy link
Contributor

Add the initial Helm Chat for the dashboard application, which can configure the deployment and budget, the service and ingress resources, and an initial network policy.

Additionally, configure the CI tooling for Taskfiles, pre-commit, and GitHub Workflows, to check, test, build, and publish the Helm Charts too.

Checklist

Before raising or requesting a review of the pull request, please check and confirm the following items have been performed, where possible:

  • I have performed a self-review of my code and run any tests locally to check
  • I have added tests that prove that any changes are effective and work correctly
  • I have made corresponding changes, as needed, to the repository documentation
  • Each commit in, and this pull request, have meaningful subjects and bodies for context
  • I have added release/..., type/..., and changes/... labels, as needed, to this pull request

@jonathanio jonathanio added priority/normal This is a normal-priority issue or pull request release/feature A new feature is added with this pull request type/enhancement Adding or requesting a new feature update/configuration Update with changes to tool or service configuratons update/documentation Update with improvements to the documentation update/github-workflows Update with changes to GitHub Workflows update/taskfiles Update with changes to the Taskfiles work-in-progress This pull request is currently work-in-progress labels Aug 7, 2024
@jonathanio jonathanio self-assigned this Aug 7, 2024
Copy link

codecov bot commented Aug 7, 2024

Codecov Report

Attention: Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.

Project coverage is 14.79%. Comparing base (d70f651) to head (470d108).

Current head 470d108 differs from pull request most recent head a1d1353

Please upload reports for the commit a1d1353 to get more accurate results.

Files Patch % Lines
internal/serve/alive/main.go 0.00% 7 Missing ⚠️
internal/serve/healthz/main.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   15.10%   14.79%   -0.32%     
==========================================
  Files          13       13              
  Lines         331      338       +7     
==========================================
  Hits           50       50              
- Misses        281      288       +7     
Flag Coverage Δ
unit-tests 14.79% <0.00%> (-0.32%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
internal/serve/healthz/main.go 0.00% <0.00%> (ø)
internal/serve/alive/main.go 0.00% <0.00%> (ø)

@jonathanio jonathanio force-pushed the add-helm-chart branch 9 times, most recently from 9c41812 to 470d108 Compare August 9, 2024 22:24
Add the initial Helm Chat for dashboard which can configure the
deployment and budget, the service and ingress resources, as well as an
initial network policy.
Add support for a dedicated 404 handler in the alive package as its a
simple function which just outputs a JSON response, and therefore can go
into that package to be attached to the service.
Add support for using StatefulSet deployments in the Helm Chart to
prepare for allowing for local storage for each of the nodes, needed for
persistent storage of the EchoVault service built into the application.
Add linting for Helm Charts via helm, and export the Deployment and
StatefulSet type deployments and pass through kubeconform to validate
the Kubernetes resources are valid for the latest version of kubernetes.
Remove the redundant pip checks for Dependabot as we no longer have a
requirements.txt file to check and validate.
@jonathanio jonathanio marked this pull request as ready for review August 9, 2024 22:41
@jonathanio jonathanio merged commit ab67c09 into main Aug 9, 2024
9 checks passed
@jonathanio jonathanio deleted the add-helm-chart branch August 9, 2024 22:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority/normal This is a normal-priority issue or pull request release/feature A new feature is added with this pull request type/enhancement Adding or requesting a new feature update/configuration Update with changes to tool or service configuratons update/documentation Update with improvements to the documentation update/github-workflows Update with changes to GitHub Workflows update/taskfiles Update with changes to the Taskfiles work-in-progress This pull request is currently work-in-progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant