Skip to content

Conversation

tonyhogsten
Copy link
Contributor

@tonyhogsten tonyhogsten commented Aug 8, 2025

Based on this feature request: #966, created by me, I have added support in the helm chart to be able to set annotations on the various services.

I have run the linter;

helm lint charts/spegel
==> Linting charts/spegel
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

Fixes #966

@phillebaba phillebaba force-pushed the feature/add-support-service-annotations branch from 1c7111b to 9fcf0b2 Compare August 11, 2025 19:27
Copy link

codecov bot commented Aug 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@phillebaba
Copy link
Member

@tonyhogsten could you run make helm-docs to fix the docs issues? Other than that things looks good.

@tonyhogsten
Copy link
Contributor Author

@phillebaba I ran make helm-docs again now. Seems I changed an URL manually due to my OCD of seeing the markdown warning which was now reverted. Hopefully it will work better now?

@phillebaba phillebaba force-pushed the feature/add-support-service-annotations branch from d91a862 to ca15107 Compare August 12, 2025 15:07
@phillebaba phillebaba force-pushed the feature/add-support-service-annotations branch from ca15107 to eea57ef Compare August 13, 2025 10:50
@phillebaba phillebaba self-requested a review August 13, 2025 11:45
@phillebaba phillebaba merged commit 80c8110 into spegel-org:main Aug 13, 2025
8 of 13 checks passed
@tonyhogsten tonyhogsten deleted the feature/add-support-service-annotations branch August 13, 2025 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability to set annotations on service
2 participants