Skip to content

Conversation

@suez1224
Copy link
Contributor

@suez1224 suez1224 commented Nov 11, 2024

What is the purpose of the change

This PR add operatorPod.webhook.container.env in the operator helm chart to allow customization of flink-webhook container envs.

Brief change log

  • add operatorPod.webhook.container.env in values.yaml of the helm chart
  • Modify flink-webhook container spec in flink-operator.yaml to populate the envs from operatorPod.webhook.container.env in values.yaml

Verifying this change

This change is verified as follows:

  • manually deploy the helm chart, and verfiy the envs are correctly passed in to flink-webhook container

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): No
  • The public API, i.e., is any changes to the CustomResourceDescriptors: No
  • Core observer or reconciler logic that is regularly executed: No

Documentation

  • Does this pull request introduce a new feature? yes
  • If yes, how is the feature documented? docs

@suez1224 suez1224 changed the title add operatorPod.webhook.container.env to allow customizating of flink-webhook container envs [FLINK-36686][Kubernetes Operator] add operatorPod.webhook.container.env to allow customizating of flink-webhook container envs Nov 11, 2024
@suez1224 suez1224 force-pushed the dev/FLINK-36686 branch 2 times, most recently from 5afff3f to c22b473 Compare November 11, 2024 21:55
…allow customizating of flink-webhook container envs
@gyfora gyfora merged commit 14ded7e into apache:main Nov 21, 2024
98 checks passed
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