Skip to content

install: Quieten noisy build output#43960

Merged
joestringer merged 1 commit intomainfrom
pr/joe/fix-noisy-helm-updates
Jan 25, 2026
Merged

install: Quieten noisy build output#43960
joestringer merged 1 commit intomainfrom
pr/joe/fix-noisy-helm-updates

Conversation

@joestringer
Copy link
Copy Markdown
Member

@joestringer joestringer commented Jan 23, 2026

The grep output being printed here added noise to the build process
without any actionable outcomes. Hide it by using "grep -q". Note
for the problem that the linter is trying to pick up, the Makefile
explicitly prints the affected filename already.

Fixes: 0b0cfc7 ("chore: Add a linter for non-idempotent secrets")

cc @fgiloux

The grep output being printed here added noise to the build process
without any actionable outcomes. Hide it by using "grep -q".

Fixes: 0b0cfc7 ("chore: Add a linter for non-idempotent secrets")
Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer requested a review from a team as a code owner January 23, 2026 20:21
@joestringer joestringer added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Jan 23, 2026
@joestringer joestringer requested a review from squeed January 23, 2026 20:21
@joestringer
Copy link
Copy Markdown
Member Author

/test

@joestringer joestringer enabled auto-merge January 23, 2026 21:28
@joestringer joestringer added this pull request to the merge queue Jan 25, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jan 25, 2026
Merged via the queue into main with commit 9927bda Jan 25, 2026
517 of 522 checks passed
@joestringer joestringer deleted the pr/joe/fix-noisy-helm-updates branch January 25, 2026 12:53
@mhofstetter mhofstetter mentioned this pull request Jan 26, 2026
9 tasks
@mhofstetter mhofstetter added backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. and removed needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Jan 26, 2026
@github-actions github-actions bot added backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. and removed backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. labels Jan 26, 2026
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

4 participants