Skip to content

Skip unnecessary CI and Renovate updates for non-code changes 1.1 - #323

Merged
JasonPowr merged 2 commits into
release-1.1from
reduce-mintmaker-noise-1.1
Nov 4, 2026
Merged

JasonPowr merged 2 commits into
release-1.1from
reduce-mintmaker-noise-1.1

Conversation

@JasonPowr

Copy link
Copy Markdown
Member

Summary

  • Add paths-ignore filters to all 20 GitHub Actions workflows so Dockerfile-only MintMaker PRs and other non-code changes (docs, markdown, Tekton pipelines, Renovate config, LICENSE) skip unnecessary CI — including expensive KinD e2e tests that previously ran ~80+ jobs per Docker digest bump
  • Align Renovate config with the securesign/fulcio pattern: extend org-inherited config, limit enabled managers, add ignorePaths for files we don't build/ship (Dockerfile.coverage, .ko.yaml, .goreleaser.yaml, test/testdata/, hack/, release/, config/, .github/workflows/), and disable upstream-only Docker images (golang, cgr.dev/chainguard/static)
  • Add missing release-* branch restrictions to workflows that only targeted main
  • Preserve correct CI triggers: e2e-coverage.yaml still runs on Dockerfile.coverage changes (its build input), verify-docs.yaml still runs on docs/** changes (its verification target)

What still triggers full CI

  • Go dependency updates (go.mod, go.sum)
  • Go source code changes
  • Test and hack script changes
  • Workflow file changes (via workflow_dispatch)

What no longer triggers CI

  • Dockerfile.rh digest bumps (validated by Konflux)
  • Dockerfile.coverage changes (except e2e-coverage.yaml)
  • Renovate config, Tekton pipelines, docs, markdown, LICENSE

Required status checks

No required status checks are configured in branch rulesets — skipped workflows will not block PR merges.

Validation

  • Renovate dry-run confirms only Dockerfile.rh and go.mod generate PRs
  • All paths-ignore entries verified as not referenced by CI workflow steps
  • e2e-coverage.yaml correctly excludes Dockerfile.coverage from its ignore list
  • verify-docs.yaml correctly excludes docs/** from its ignore list

JasonPowr and others added 2 commits November 4, 2026 12:10
MintMaker Docker digest PRs (Dockerfile.rh) were triggering ~80+ CI
jobs including expensive KinD e2e tests despite changing zero Go code.
These image updates are already validated by Konflux pipelines.

Add paths-ignore filters to all 20 PR/push-triggered workflows for
files that don't affect Go builds or tests: Dockerfile.rh, renovate.json,
.tekton/**, docs/**, LICENSE, and *.md.

Also add missing release-* branch restrictions to workflows that only
targeted main (lint, tests, release-snapshot, kind-e2e-cosigned,
kind-e2e-trustroot-crd, e2e-coverage, verify-docs, depsreview,
codeql-analysis, scorecard_action).

No required status checks are configured in branch rulesets, so skipped
workflows will not block PR merges.
Align Renovate config with the securesign/fulcio pattern: extend the
org-inherited config, explicitly limit enabled managers to dockerfile,
gomod, tekton and custom.regex, and add ignorePaths for files we don't
build or ship (Dockerfile.coverage, .ko.yaml, .goreleaser.yaml, hack/,
release/, config/, .github/workflows/, test/testdata/).

Add package rules to disable updates for upstream-only Docker images
(golang, cgr.dev/chainguard/static) and GitHub Actions.

Move existing Go dependency pins from gomod-scoped packageRules to
top-level packageRules to ensure they apply across all managers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter

codecov-commenter commented Nov 4, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.24%. Comparing base (be30cc8) to head (d33ceb6).
⚠️ Report is 2 commits behind head on release-1.1.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-1.1     #323       +/-   ##
================================================
+ Coverage        30.70%   52.24%   +21.53%     
================================================
  Files              122      122               
  Lines             7444     7444               
================================================
+ Hits              2286     3889     +1603     
+ Misses            4917     3482     -1435     
+ Partials           241       73      -168     
Flag Coverage Δ
e2e 63.62% <ø> (?)
unit 30.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JasonPowr
JasonPowr merged commit 46c9569 into release-1.1 Nov 4, 2026
95 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.

3 participants