Skip to content

Conversation

@vyzigold
Copy link
Contributor

@vyzigold vyzigold commented Sep 15, 2025

This PR creates a LokiStack, which will be used as a storage for CloudKitty. It creates the necessary client certificate for mTLS authentication between CloudKitty and Loki.

Based-On: #741
Depends-On-Upstream-Change: https://review.opendev.org/c/openstack/cloudkitty/+/961209

@softwarefactory-project-zuul
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/telemetry-operator for 751,2ada72edbb1411cb9caeaa1f96604ddfc74c2f34

@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 15, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vyzigold

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@softwarefactory-project-zuul
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/telemetry-operator for 751,2d8132ef3fa79bceec06b2b858d6be5a8989b943

The s3StorageConfig.secret is passed to Lokistack, which requires
some value be set.
The default is needed, or else there will be an error when it is not
configured.

The default is set to a reasonable value for the secret name
(cloudkitty-loki-s3) and the type (s3), which match the values we're
planning on using for the CI environment.

Lines added: 1
Lines generated: 16
@softwarefactory-project-zuul
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/telemetry-operator for 751,88199a1cd9a6e4386f4d00ef741d0921aafb609c

@softwarefactory-project-zuul
Copy link

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/telemetry-operator for 751,93afe864da689f7166f2718cb0765800266d5ac2

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 19, 2025

@vyzigold: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/precommit-check 93afe86 link true /test precommit-check
ci/prow/telemetry-operator-build-deploy 93afe86 link false /test telemetry-operator-build-deploy
ci/prow/telemetry-operator-build-deploy-kuttl 93afe86 link true /test telemetry-operator-build-deploy-kuttl
ci/prow/images 93afe86 link true /test images

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@vyzigold vyzigold closed this Sep 22, 2025
@vyzigold
Copy link
Contributor Author

This ended up being merged into Juan's fork to allow for further development of cloudkitty related parts of the telemetry-operator even when we aren't able to merge it into main here yet. The PR: jlarriba#5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants