-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add the ability to customize k8s client config #31398
Conversation
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm missing some doc updates about this change and maybe a simple test to cover this change.
Good point. I'll update the PR tomorrow
…On Mon, Feb 27, 2023, 08:13 Jose Carvajal ***@***.***> wrote:
***@***.**** requested changes on this pull request.
I'm missing some doc updates about this change and maybe a simple test to
cover this change.
—
Reply to this email directly, view it on GitHub
<#31398 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABBMDPZXWJW4YGRDGEAVSTTWZRAW7ANCNFSM6AAAAAAVG4MGRI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
PR updated |
🙈 The PR is closed and the preview is expired. |
This comment has been minimized.
This comment has been minimized.
This gives users the ability to alter the configuration provided by Quarkus in arbitrary ways The pattern implemented is exactly the same as the one used in other extensions Relates to: quarkusio#31395
✔️ The latest workflow run for the pull request has completed successfully. It should be safe to merge provided you have a look at the other checks in the summary. |
This gives users the ability to alter the
configuration provided by Quarkus in arbitrary ways
The pattern implemented is exactly the same as the one used in other extensions
Relates to: #31395