-
Notifications
You must be signed in to change notification settings - Fork 486
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 base agent-operator tanka config #1900
Conversation
local this = self, | ||
|
||
_images:: { | ||
agent_operator: 'grafana/agent-operator:v0.26.0-rc.0', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a little worried this image is in a bad place, and will not be updated very easily. The agent libs import lib/version.libsonnet
. Maybe make another operator version file, or I think preferably change version.libsonnet into a file like images.libsonnet and have both images in one place.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah i thought lib/version.libsonnet
was deprecated since those manifests aren't generated anymore - was just matching the approach used in the agent tanka lib - happy to create a new version file though and use that in the agent, operator, etc. base libs
LGTM, besides Craig's feedback. |
PR Description
This PR adds a library to generate the base agent-operator manifest found here. Subsequent PRs will cover the CRs (metrics, logs, etc.) in the custom resource quickstart.
Will investigate using the k8s library, which seems to generate the jsonnet libraries from the CRDs directly...
This work is to support the addition of generated agent-operator configs in Grafana Cloud K8s monitoring.
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist