Skip to content

Add a chi example for sync users#1304

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

Add a chi example for sync users#1304
Slach merged 2 commits intoAltinity:0.23.0from
ccsxs:addChiExample

Conversation

@ccsxs
Copy link
Copy Markdown

@ccsxs ccsxs commented Jan 10, 2024

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
	<[email protected]>
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
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
    <[email protected]>
@ccsxs
Copy link
Copy Markdown
Author

ccsxs commented Jan 12, 2024

@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
Copy link
Copy Markdown
Collaborator

Slach commented Jan 12, 2024

@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