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

Keeper PVC is not taking input from VolumeClaimTemplates #1362

Open
zheyu001 opened this issue Mar 5, 2024 · 8 comments
Open

Keeper PVC is not taking input from VolumeClaimTemplates #1362

zheyu001 opened this issue Mar 5, 2024 · 8 comments
Assignees

Comments

@zheyu001
Copy link

zheyu001 commented Mar 5, 2024

We are trying to migrate our own keeper config to CHK, but it seems the PVC template is not taking into account. By quickly scan the code, it seems when creating CHK, it only cares about how many keeper volumes are declared instead of the content of the pvc? At least in our case, k8s complains about "both-paths" not found.

https://github.com/Altinity/clickhouse-operator/blob/0.23.3/pkg/model/chk/creator.go#L97-L118

@dxygit1
Copy link

dxygit1 commented Mar 14, 2024

I also encountered the same problem

Hope to specify PVC binding to the corresponding directory

@alex-zaitsev
Copy link
Member

Will be fixed in 0.23.4. There is a workaround in the current version, but better not to use it, it will be deprecated.

@dxygit1
Copy link

dxygit1 commented Mar 14, 2024

Please let me know after you've fixed it. Thank you very much

@dxygit1
Copy link

dxygit1 commented Mar 18, 2024

Approximately when can this be fix

@dustinmoris
Copy link

I run into the same issue. What do we need to do in the meantime to get things working? I just need to get CHK working with my CHI and I don't want to wait a few days or longer until a fix gets released. Ideally I get this working today with whatever workarounds are needed for now.

@genestack-okunitsyn
Copy link

+1 to the issue

@dustinmoris
Copy link

I got CHK working with this manifest:

apiVersion: "clickhouse-keeper.altinity.com/v1"
kind: "ClickHouseKeeperInstallation"
metadata:
  name: clickhouse-keeper
  namespace: plausible
spec:
  configuration:
    clusters:
      - name: prod
        layout:
          replicasCount: 3
    settings:
      logger/level: "trace"
      logger/console: "true"
      listen_host: "0.0.0.0"
      keeper_server/storage_path: /var/lib/clickhouse-keeper
      keeper_server/tcp_port: "2181"
      keeper_server/four_letter_word_white_list: "*"
      keeper_server/coordination_settings/raft_logs_level: "information"
      keeper_server/raft_configuration/server/port: "9444"
      prometheus/endpoint: "/metrics"
      prometheus/port: "7000"
      prometheus/metrics: "true"
      prometheus/events: "true"
      prometheus/asynchronous_metrics: "true"
      prometheus/status_info: "false"
  templates:
    podTemplates:
      - name: default
        spec:
          containers:
            - name: clickhouse-keeper
              image: "clickhouse/clickhouse-keeper:24-alpine"
              imagePullPolicy: IfNotPresent
              resources:
                requests:
                  memory: "256M"
                  cpu: "0.5"
                limits:
                  memory: "2Gi"
                  cpu: "2"
    volumeClaimTemplates:
      - name: both-paths
        spec:
          storageClassName: standard-rwo
          accessModes:
            - ReadWriteOnce
          resources:
            requests:
              storage: 10Gi

@g-marius
Copy link

@alex-zaitsev was this issue partly fixed in 0.23.4? we are seeing a similar error to #1374 for 0.23.5, will need to dig a bit next week on this.

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

No branches or pull requests

7 participants