Skip to content

Issue401/expose security capabilities #404

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

Merged

Conversation

mingmxu
Copy link
Contributor

@mingmxu mingmxu commented Jan 17, 2024

This PR adds the function required by Issue-401.

It expose Capabilities in StarRocksComponentSpec which is required for Datadog profiling integration. Some unit tests are added to verify it.

go test -mod=mod ./pkg/k8sutils/templates/pod/...

cc @yandongxiao

@mingmxu mingmxu force-pushed the issue401/expose_security_capabilities branch from 61ebc25 to adf6341 Compare January 17, 2024 20:54
@yandongxiao
Copy link
Collaborator

should alose expose security capabilities in helm chart

@mingmxu
Copy link
Contributor Author

mingmxu commented Jan 18, 2024

Right, I’ll submit another PR after this one is cleared.

@mingmxu mingmxu force-pushed the issue401/expose_security_capabilities branch 2 times, most recently from 6690c5e to ea5f71c Compare January 18, 2024 05:01
Signed-off-by: Mingmin Xu <[email protected]>

use a short name for Capabilities

Signed-off-by: Mingmin Xu <[email protected]>

add more unit tests

Signed-off-by: Mingmin Xu <[email protected]>

fix lint error and json:"Capabilities,omitempty"

Signed-off-by: Mingmin Xu <[email protected]>

run make manifests and make generate

Signed-off-by: Mingmin Xu <[email protected]>

fix format error

Signed-off-by: Mingmin Xu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants