Skip to content

Support to set the resource configuration for Fluid webhook pod - #4428

Merged
fluid-e2e-bot[bot] merged 1 commit into
fluid-cloudnative:masterfrom
Syspretor:fix/add-resource-configuration-in-webhook-chart-templates
Dec 9, 2024
Merged

Support to set the resource configuration for Fluid webhook pod#4428
fluid-e2e-bot[bot] merged 1 commit into
fluid-cloudnative:masterfrom
Syspretor:fix/add-resource-configuration-in-webhook-chart-templates

Conversation

@Syspretor

Copy link
Copy Markdown
Collaborator

The following configuration in Fluid Chart Values.yaml can be apply to Fluid Webhook Pod.Spec.Container[0].Resource

webhook:
  replicas: 3
  resources:
    requests:
      cpu: 1000m
      memory: 2048Mi
    limits:
      cpu: 2000m
      memory: 4086Mi

Signed-off-by: jiuyu <guotongyu.gty@alibaba-inc.com>
@sonarqubecloud

sonarqubecloud Bot commented Dec 6, 2024

Copy link
Copy Markdown

@Syspretor Syspretor changed the title Support set resource in webhook charts Support set resource for Fluid webhook pod Dec 6, 2024
@Syspretor Syspretor changed the title Support set resource for Fluid webhook pod Support to set the resource configuration for Fluid webhook pod Dec 6, 2024

@cheyang cheyang left a comment

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.

/lgtm
/approve

@fluid-e2e-bot

fluid-e2e-bot Bot commented Dec 9, 2024

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cheyang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants