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

When specifying/updating cloudStorage in the CRD, the cluster config is not updated #5876

Closed
patrickangeles opened this issue Aug 6, 2022 · 1 comment
Assignees
Labels
area/cloud-storage Shadow indexing subsystem area/k8s kind/bug Something isn't working

Comments

@patrickangeles
Copy link

Version & Environment

Redpanda version: 22.1.5

What went wrong?

In Kubernetes, using the operator, when you specify cloudStorage parameters to enable shadow indexing, the values do not make it into centralized config. (So they don't take effect.)

@patrickangeles patrickangeles added kind/bug Something isn't working area/k8s area/cloud-storage Shadow indexing subsystem labels Aug 6, 2022
@nicolaferraro
Copy link
Member

nicolaferraro commented Aug 8, 2022

Just checking it on a test cluster. I see values are correctly injected into the central config.

Using the following snippet:

# ...
  cloudStorage:
    enabled: true
    accessKey: "anaccesskey"
    secretKeyRef:
      name: asecretkey
      namespace: default
    region: eu-west-1
    bucket: abucket
# ...
redpanda-controller-manager-7f65c4db78-nbrhq manager 2022-08-08T09:31:35.221Z	INFO	controllers.redpanda.Cluster	Applying patch to the cluster configuration	{"redpandacluster": "default/example", "patch": "+cloud_storage_access_key +cloud_storage_bucket +cloud_storage_enabled +cloud_storage_region +cloud_storage_secret_key"}

I can also find them using rpk:

$ kubectl exec example-0 -c redpanda -- rpk cluster config get cloud_storage_enabled
true

What's the effect that you see. Are you checking the values with rpk or you see that shadow indexing is not enabled?

In case rpk gives the right values and shadow index does not work, maybe the cluster needs a restart (there's an issue for which need for restart is not always catched, see #5835).

Another thing I noticed is that the cluster is undergoing a double restart, one for changing the pod spec to include a volume for indexing, another for enabling shadow indexing (the one that may fail to be done).

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cloud-storage Shadow indexing subsystem area/k8s kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants