8000 Add a chi example for sync users by ccsxs · Pull Request #1304 · Altinity/clickhouse-operator · GitHub
[go: up one dir, main page]

Skip to content

Add a chi example for sync users - #1304

Merged
Slach merged 2 commits into
Altinity:0.23.0from
ccsxs:addChiExample
Jan 12, 2024
Merged

Slach merged 2 commits into
Altinity:0.23.0from
ccsxs:addChiExample

Conversation

@ccsxs
@ccsxs ccsxs commented Jan 10, 2024
Copy link
Copy Markdown

I submit a yuml template for migrating users when expanding capacity, so that people in need can use it.

Date:   Wed Jan 10 09:59:56 2024 +0200

    Add user synchronization chi example
	<songxiangduan@cmss.chinamobile.com>
defaults:
templates:
podTemplate: clickhouse
dataVolumeClaimTemplate: localpvc-storage
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove unnecessary details,
let's rename to data-pvc-template

Comment thread docs/chi-examples/02-templates-07-syncUser.yaml < 8000 span title="Label: Outdated" data-view-component="true" class="Label Label--warning"> Outdated
volumeClaimTemplates:
- name: local-pvc-my
spec:
storageClassName: my-localpv
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove storageClassName, this is unnecessary

- mountPath: /var/lib/clickhouse
name: local-pvc-my
volumeClaimTemplates:
- name: local-pvc-my
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rename to data-pvc-template

Date:   Fri Jan 12 16:41:56 2024 +0200

    Add user synchronization chi example
    <songxiangduan@cmss.chinamobile.com>
@ccsxs
ccsxs commented Jan 12, 2024
Copy link
Copy Markdown
Author

@Slach Hello, I have revised and submitted it according to your comments,thanks.

@Slach
Slach merged commit 2a66d25 into Altinity:0.23.0 Jan 12, 2024
@Slach
Slach commented Jan 12, 2024
Copy link
Copy Markdown
Collaborator

@ccsxs thanks for contribution

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

Successfully merging this pull request may close these issues.

2 participants

0