Skip to content

Added docu for vcap additional options #2035

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

Merged
merged 12 commits into from
Aug 22, 2025
Merged

Conversation

mariayord
Copy link
Contributor

No description provided.

@mariayord mariayord requested a review from smahati as a code owner August 19, 2025 09:27
@johannes-vogel
Copy link
Contributor

@mariayord can you please attach a screenshot to see what it looks like eventually? The table layout is hard to imagine.

</tbody>
</table>

To override the inherited configuration from `VCAP_SERVICE`, explicitly set the service property `vcap.label` to `false` in the configuration file, for example:
Copy link
Contributor

Choose a reason for hiding this comment

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

CAP services often come with a default vcap configuration. In rare cases, the default configuration has to be deactivated which can be achieved by explicitly setting the service property vcap.<property> to false:

Copy link
Contributor Author

@mariayord mariayord Aug 19, 2025

Choose a reason for hiding this comment

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

@johannes-vogel I thougth we support only vcap.label : false or this behaviour is valid for all properties ?

Copy link
Contributor

Choose a reason for hiding this comment

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

The behavior is generic for all properties that are set to false.

Copy link
Contributor

Choose a reason for hiding this comment

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

On the other hand, all our defaults are using the label so far.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

{66D8F87B-4CFA-486D-B6B6-0811856F8F49}

@renejeglinsky renejeglinsky merged commit bc9bb36 into main Aug 22, 2025
4 checks passed
@renejeglinsky renejeglinsky deleted the vcap-additional-docu branch August 22, 2025 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants