-
Notifications
You must be signed in to change notification settings - Fork 24
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
NETOBSERV-793 Change default loki CA configmap name #254
Conversation
This will likely be needed in 1.1 |
Changing to "loki-gateway-ca-bundle" so that it matches loki operator 5.6 name
/ok-to-test |
/label ok-to-test |
/ok-to-test |
New image: ["quay.io/netobserv/network-observability-operator:b120adc"]. It will expire after two weeks. |
/label qe-approved |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jotak The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Changing to "loki-gateway-ca-bundle" so that it matches loki operator 5.6 name
* Confgen: init from options (avoid many globals) * Use main model in confgen, use pipeline builder Fixes netobserv#254 - Removed duplicated confgen model - Use PipelineBuilder to simplify pipeline generation - Make confgen easier to consume as a lib (e.g. for NOO) - Do not make it necessary to call "Run": parsing definition files should be sufficient - Do not make it necessary to work with files written on disk: work with []byte instead - SkipWithTags should not result in an error when a file is skipped - Add confgen tests * Avoid using globals A side-effect of removing globals in write_loki is that it changes how the config is read, and set with defaults. Instead of unmarshaling a second time to automatically get defaults, we now call an explicit function that sets the default. Also, now removing loki URL default, it now has to be set explicitely * Update ConnTrack builder * More defer cleanup in tests, and use ioutils / temp dir/files Also fixed jsonnet dir actually used as filename prefix rather than directory * User ConfigFileStruct in confgen * Update pkg/config/config.go Co-authored-by: Ronen Schaffer <ronen.schaffer@ibm.com> * Use config.ConfigFileStruct in tests Co-authored-by: Ronen Schaffer <ronen.schaffer@ibm.com>
Changing to "loki-gateway-ca-bundle" so that it matches loki operator 5.6 name
JIRA: https://issues.redhat.com/browse/NETOBSERV-793