We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Nearly all the properties are required, whereas I'm pretty sure they should be optional (typically the proxy stuff).
Please use Optional where it makes sense.
Optional
@geoand @gastaldi could you have a look? It's not high priority but I just included the generated doc and it ends up with required everywhere.
required
I'm also wondering if most of the config should be runtime instead of build time?
The text was updated successfully, but these errors were encountered:
I'll check it out
Sorry, something went wrong.
Make KubernetesClientConfig values Optional
fd85f74
Fixes: quarkusio#4707
e5c718d
Merge pull request #4735 from geoand/#4707
a911cb5
3b87e15
geoand
Successfully merging a pull request may close this issue.
Nearly all the properties are required, whereas I'm pretty sure they should be optional (typically the proxy stuff).
Please use
Optional
where it makes sense.@geoand @gastaldi could you have a look? It's not high priority but I just included the generated doc and it ends up with
required
everywhere.I'm also wondering if most of the config should be runtime instead of build time?
The text was updated successfully, but these errors were encountered: