Skip to content

Add espejote-managed additional facts - #117

Merged
DebakelOrakel merged 2 commits into
masterfrom
managed-additional-facts
Jul 8, 2026
Merged

Add espejote-managed additional facts#117
DebakelOrakel merged 2 commits into
masterfrom
managed-additional-facts

Conversation

@HappyTetrahedron

@HappyTetrahedron HappyTetrahedron commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • The PR has a meaningful title. It will be used to auto-generate the
    changelog.
    The PR has a meaningful description that sums up the change. It will be
    linked in the changelog.
  • PR contains a single logical change (to build a better changelog).
  • Update the documentation.
  • Categorize the PR by adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog.
  • Link this PR to related issues or PRs.

@DebakelOrakel
DebakelOrakel marked this pull request as ready for review July 8, 2026 07:34
@DebakelOrakel DebakelOrakel changed the title [WIP] Add espejote-managed additional facts Add espejote-managed additional facts Jul 8, 2026
@DebakelOrakel
DebakelOrakel force-pushed the managed-additional-facts branch from a3c131b to ef55b38 Compare July 8, 2026 08:05
@DebakelOrakel DebakelOrakel added enhancement New feature or request bump:minor labels Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🚀 This PR has been released as v4.4.0

Triggering workflows Release

🛠️ Auto tagging enabled with label bump:minor

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🚀 Merging this PR will release v4.4.0

Merging will trigger workflows Release

🛠️ Auto tagging enabled with label bump:minor

@DebakelOrakel
DebakelOrakel requested a review from a team July 8, 2026 08:19

@HappyTetrahedron HappyTetrahedron left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

LGTM. I opened the PR so I can't approve it ^^

<1> Label the `ConfigMap` so it's picked up by the `ManagedResource`, see xref:references/parameters.adoc#_additional_facts_config_label[Parameters] for the values.
<2> The `ConfigMaps` will get sorted by their name before merging.
<3> Apply the `ConfigMap` to the `syn` namespace.
<4> Use the key `spec` for your configuration

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

spec ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

woops

@DebakelOrakel
DebakelOrakel force-pushed the managed-additional-facts branch from ef55b38 to 1144e55 Compare July 8, 2026 08:39
@DebakelOrakel
DebakelOrakel merged commit e6816e9 into master Jul 8, 2026
9 checks passed
@DebakelOrakel
DebakelOrakel deleted the managed-additional-facts branch July 8, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bump:minor enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants