Skip to content

Fix pod annotation/label templates#212

Merged
lindhe merged 1 commit intoinveniosoftware:masterfrom
jobo-prod:patch-1
Jul 31, 2025
Merged

Fix pod annotation/label templates#212
lindhe merged 1 commit intoinveniosoftware:masterfrom
jobo-prod:patch-1

Conversation

@jobo-prod
Copy link
Copy Markdown
Contributor

@jobo-prod jobo-prod commented Jul 30, 2025

Description

When using pod annotations or labels, rendering of the chart fails with

Helm Error: template: (...) executing "invenio/charts/invenio/templates/(...)" at <tpl>: wrong number of args for tpl: want 2 got 1

due to a missing context parameter.

Btw, below links from the PR template are outdated ;)

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

  1. You license your contribution under the same terms as the current repository’s license.
  2. You agree that you have the right to license your contribution under the current repository’s license.

Co-authored-by: Johannes Bohn <johannes.bauer@uk-erlangen.de>
@jobo-prod
Copy link
Copy Markdown
Contributor Author

May I ask for another release of the chart after merge of this fix, please? :)

Copy link
Copy Markdown
Contributor

@lindhe lindhe left a comment

Choose a reason for hiding this comment

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

Great contribution, thank you!

@lindhe lindhe merged commit e84f40e into inveniosoftware:master Jul 31, 2025
2 checks passed
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.

2 participants