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

[pull] master from apache:master #50

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Conversation

pull[bot]
Copy link

@pull pull bot commented Dec 5, 2024

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

### Why are the changes needed?

Adding  an example to `servicemonitor` in values.yaml to improve the usability .

### How was this patch tested?

execute command `helm install kyuubi-test .`
check the servicemonitor after deployment

### Was this patch authored or co-authored using generative AI tooling?

No

Closes #6839 from zhifanggao/add_default_endpoints.

Closes #6839

59aac41 [Cheng Pan] Update charts/kyuubi/values.yaml
869f45e [zhifanggao] include the key itself

Lead-authored-by: Cheng Pan <[email protected]>
Co-authored-by: zhifanggao <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
@pull pull bot added the ⤵️ pull label Dec 5, 2024
@pull pull bot merged commit dc3ac89 into huangxiaopingRD:master Dec 5, 2024
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.

1 participant