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

[Bugfix] Do not create the default data storage volume when deploying be with multiple volumes #613

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yandongxiao
Copy link
Collaborator

@yandongxiao yandongxiao commented Jan 9, 2025

Description

Please provide a detailed description of the changes you have made. Include the motivation for these changes, and any
additional context that may be important.

the PR should include helm chart changes and operator changes.

Related Issue(s)

#611

If this PR is included:

  volumes:
  - name: be1-data
    persistentVolumeClaim:
      claimName: be1-data-kube-starrocks-be-0
  - name: be-log
    persistentVolumeClaim:
      claimName: be-log-kube-starrocks-be-0
  - name: be0-data
    persistentVolumeClaim:
      claimName: be0-data-kube-starrocks-be-0
  - configMap:
      defaultMode: 420
      name: kube-starrocks-be-cm
    name: kube-starrocks-be-cm

Checklist

For operator, please complete the following checklist:

  • run make generate to generate the code.
  • run golangci-lint run to check the code style.
  • run make test to run UT.
  • run make manifests to update the yaml files of CRD.

…deploying BE with multiple volumes

Signed-off-by: yandongxiao <[email protected]>
@yandongxiao yandongxiao force-pushed the bugfix/should-not-create-the-default-data-storage-volume-when-deploying-BE-with-multiple-volumes branch from 1d48466 to d4fa323 Compare January 9, 2025 07:06
@yandongxiao yandongxiao marked this pull request as ready for review January 9, 2025 07:12
@yandongxiao yandongxiao requested a review from kevincai January 9, 2025 07:13
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