Skip to content

[loki] Fix ServiceAccount resource name#319

Merged
jkroepke merged 8 commits intografana-community:mainfrom
vquie:fix_serviceaccount_resource_name
Apr 10, 2026
Merged

[loki] Fix ServiceAccount resource name#319
jkroepke merged 8 commits intografana-community:mainfrom
vquie:fix_serviceaccount_resource_name

Conversation

@vquie
Copy link
Copy Markdown

@vquie vquie commented Apr 10, 2026

What this PR does / why we need it

serviceAccount.name didn't override the serviceaccount resource's name
Now it does.

Special notes for your reviewer

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [grafana])

@vquie vquie force-pushed the fix_serviceaccount_resource_name branch from 5eed70a to dea3f5a Compare April 10, 2026 12:17
@vquie vquie requested a review from a team as a code owner April 10, 2026 13:13
vquie added 2 commits April 10, 2026 19:29
Use loki.serviceAccountName helper with component and target
Add tests for backend serviceAccount.name override
Bump chart version to 11.4.7

Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
@vquie vquie force-pushed the fix_serviceaccount_resource_name branch from 7b84b47 to 0087d9d Compare April 10, 2026 17:51
vquie added 2 commits April 10, 2026 19:55
Deleted outdated guidance for chart contribution
Information is now covered in existing repository docs

Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
Signed-off-by: Vitali Quiering <21147643+vquie@users.noreply.github.com>
@vquie vquie force-pushed the fix_serviceaccount_resource_name branch from bfcd12c to 1d98ef9 Compare April 10, 2026 17:58
@vquie vquie requested a review from jkroepke April 10, 2026 20:12
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
@jkroepke jkroepke enabled auto-merge (squash) April 10, 2026 22:25
jkroepke
jkroepke previously approved these changes Apr 10, 2026
Copy link
Copy Markdown
Member

@jkroepke jkroepke left a comment

Choose a reason for hiding this comment

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

LGTM

@jkroepke jkroepke disabled auto-merge April 10, 2026 22:26
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
Signed-off-by: Jan-Otto Kröpke <mail@jkroepke.de>
@jkroepke jkroepke enabled auto-merge (squash) April 10, 2026 22:27
@jkroepke
Copy link
Copy Markdown
Member

Thanks!

@jkroepke jkroepke merged commit 6c2e29d into grafana-community:main Apr 10, 2026
8 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