camel-k icon indicating copy to clipboard operation
camel-k copied to clipboard

Provide default value in a specific field for trait properties in CRD

Open apupier opened this issue 2 years ago • 5 comments

Requirement

As a user I would like to have default value pre-filled

Problem

the default value is provided inside the description text which is not convenient to read from for tooling that want to read/display the display value more prohemitently

Proposal

No response

Open questions

No response

apupier avatar Sep 07 '23 08:09 apupier

@apupier Just to clarify, you are expecting the default values only for trait fields declared in the documentation to be in the CRD ? For now I plan to add only those present in the documentation.

@squakez I will take care of it.

gansheer avatar Oct 31 '23 09:10 gansheer

you are expecting the default values only for trait fields declared in the documentation to be in the CRD ?

From Camel Language Server point of view, I identified only these ones missing. But if there are other places that are missing default values, it would be better to add them too.

apupier avatar Nov 02 '23 07:11 apupier

I added all the default values I could find/extract. Most of the others are either given by the Kubernetes Platform implementation either conditional. @squakez this issue is fixed AFAIK.

gansheer avatar Nov 06 '23 08:11 gansheer

We had to revert the change that fixed this issue. Still open.

squakez avatar Jan 17 '24 13:01 squakez

This issue has been automatically marked as stale due to 90 days of inactivity. It will be closed if no further activity occurs within 15 days. If you think that’s incorrect or the issue should never stale, please simply write any comment. Thanks for your contributions!

github-actions[bot] avatar Apr 17 '24 00:04 github-actions[bot]