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

Misc Helm updates #1814

Merged
merged 9 commits into from
Nov 30, 2023
Merged

Conversation

lambdanis
Copy link
Contributor

@lambdanis lambdanis commented Nov 29, 2023

See commits for details.

So that they can use PodInfo goodness.

Signed-off-by: Anna Kapuscinska <[email protected]>
It's templated already in clusterrolebinding.

Signed-off-by: Anna Kapuscinska <[email protected]>
Adding podLabels Helm value, which allows the user to add custom labels to pods
without overriding the default labels, like podLabelsOverride does. Overriding
all labels required the user to specify correct selector labels too, so having
an option to keep them is more convenient.

Signed-off-by: Anna Kapuscinska <[email protected]>
See https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.24/#container-v1-core

> [...] FallbackToLogsOnError will use the last chunk of container log
> output if the termination message file is empty and the container
> exited with an error. The log output is limited to 2048 bytes or 80
> lines, whichever is smaller. [...]

Signed-off-by: Anna Kapuscinska <[email protected]>
It seems it was templated in an incorrect place anyway (before the tag).

Signed-off-by: Anna Kapuscinska <[email protected]>
Signed-off-by: Anna Kapuscinska <[email protected]>
Signed-off-by: Anna Kapuscinska <[email protected]>
@lambdanis lambdanis added release-note/minor This PR introduces a minor user-visible change area/helm Related to the Helm chart labels Nov 29, 2023
@lambdanis lambdanis requested a review from a team as a code owner November 29, 2023 22:25
@lambdanis lambdanis requested a review from olsajiri November 29, 2023 22:25
Copy link

netlify bot commented Nov 29, 2023

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit fba888d
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/6567ba5919ca3c00078ff6ba
😎 Deploy Preview https://deploy-preview-1814--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@michi-covalent michi-covalent left a comment

Choose a reason for hiding this comment

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

seems legit

Comment on lines -165 to -166
# tetragon-operator image-digest
suffix: ""
Copy link
Contributor

Choose a reason for hiding this comment

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

🥰

@lambdanis lambdanis merged commit 3faebde into cilium:main Nov 30, 2023
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm Related to the Helm chart release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants