Skip to content

Conversation

@cmanzur
Copy link

@cmanzur cmanzur commented Aug 30, 2021

Add custom extra pod labels.
Example: Useful for k8s logging shipping.

@cla-bot
Copy link

cla-bot bot commented Aug 30, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@hashhar
Copy link
Member

hashhar commented Aug 30, 2021

cc: @aalbu @nineinchnick

@cla-bot
Copy link

cla-bot bot commented Aug 30, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@cla-bot
Copy link

cla-bot bot commented Aug 31, 2021

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

@cmanzur
Copy link
Author

cmanzur commented Aug 31, 2021 via email

additionalJVMConfig: {}

additionalConfigProperties: {}
additionalConfigProperties: []
Copy link

Choose a reason for hiding this comment

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

Why did you change the type for this field?

@mosabua
Copy link
Member

mosabua commented May 29, 2023

Replaced by #75 ?

@mosabua mosabua closed this May 30, 2023
@mosabua
Copy link
Member

mosabua commented May 30, 2023

This is now replaced by the merged PR #75

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants