Skip to content

OCM-00000 | ci: Update helpers to v3#3307

Closed
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/major-helpers
Closed

OCM-00000 | ci: Update helpers to v3#3307
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/major-helpers

Conversation

@red-hat-konflux

@red-hat-konflux red-hat-konflux Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
gopkg.in/yaml.v2 v2.4.0v3.0.1 age confidence

Warning

Some dependencies could not be looked up. Check the Dependency Dashboard for more information.


Release Notes

go-yaml/yaml (gopkg.in/yaml.v2)

v3.0.1

Compare Source

v3.0.0

Compare Source


Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

To execute skipped test pipelines write comment /ok-to-test.


Documentation

Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.

Summary by CodeRabbit

  • Chores
    • Updated a transitive YAML library to a newer major version, which may improve compatibility and reliability for YAML-related operations.

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@red-hat-konflux red-hat-konflux Bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Jul 1, 2026
@red-hat-konflux

Copy link
Copy Markdown
Contributor Author

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: go.sum
Command failed: mod upgrade --mod-name=gopkg.in/yaml.v2 -t=3
could not load package: err: exit status 1: stderr: go: inconsistent vendoring in /tmp/renovate/repos/github/openshift/rosa:
	gopkg.in/yaml.v2@v2.2.8: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt
	gopkg.in/yaml.v2@v2.4.0: is marked as explicit in vendor/modules.txt, but not explicitly required in go.mod

	To ignore the vendor directory, use -mod=readonly or -mod=mod.
	To sync the vendor directory, run:
		go mod vendor


@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: e8e9b2bf-2417-449c-945d-9b3e22787393

📥 Commits

Reviewing files that changed from the base of the PR and between e6c81eb and 7804a8a.

📒 Files selected for processing (1)
  • go.mod

📝 Walkthrough

Walkthrough

This pull request updates an indirect dependency in go.mod, replacing gopkg.in/yaml.v2 (v2.4.0) with gopkg.in/yaml.v3 (v3.0.1). No other dependency versions or replace directives were modified, and no exported or public entity declarations were altered.

Changes

Files Change Summary
go.mod Replaced indirect dependency gopkg.in/yaml.v2 v2.4.0 with gopkg.in/yaml.v3 v3.0.1

Sequence Diagram(s)

Not applicable for this change.

Estimated code review effort: 1 (Trivial)

Related issues: None specified

Related PRs: None specified

Suggested labels: dependencies, go.mod

Suggested reviewers: None specified

Poem
A rabbit hopped through modules deep,
Traded v2 for v3's leap,
One line changed, quiet and small,
Yaml's new version answers the call,
Hop, commit, and off to sleep. 🐇

🚥 Pre-merge checks | ✅ 14 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the version bump and release notes, but misses the required template sections and validation details. Add the PR Summary, issue context, related links, type of change, before/after behavior, test steps, proof, and verification checklist items.
✅ Passed checks (14 passed)
Check name Status Explanation
Title check ✅ Passed The title matches the dependency upgrade to v3, though it is broader than the exact package name.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed Only go.mod was updated for a yaml dependency bump; no test files or Ginkgo titles were changed.
Test Structure And Quality ✅ Passed PR only bumps yaml in go.mod; no test files or Ginkgo blocks are changed, so the test-quality checklist is not applicable.
Microshift Test Compatibility ✅ Passed Only go.mod changed (yaml v2→v3); no e2e/test files or Ginkgo additions appear in the patch, so MicroShift compatibility is unaffected.
Single Node Openshift (Sno) Test Compatibility ✅ Passed Only go.mod dependency version changed; no new Ginkgo e2e tests or node-assumption code was added, so the SNO check is not applicable.
Topology-Aware Scheduling Compatibility ✅ Passed Only go.mod was updated to gopkg.in/yaml.v3; no deployment, operator, or controller manifests/code were changed.
Ote Binary Stdout Contract ✅ Passed Only go.mod changed from yaml.v2 to yaml.v3; no main/init/suite code or stdout writes were added.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR only updates go.mod from yaml.v2 to yaml.v3; no new Ginkgo e2e tests or network-dependent test code were added.
No-Weak-Crypto ✅ Passed PR only updates gopkg.in/yaml.v2→v3 in go.mod; no weak-crypto, custom crypto, or secret-comparison code was added.
Container-Privileges ✅ Passed PASS: The PR only changes go.mod; no container/K8s manifests were modified, so no privilege settings were introduced.
No-Sensitive-Data-In-Logs ✅ Passed The PR only updates go.mod dependency versions; no code or logging statements were added, so there’s no new sensitive data exposure in logs.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch konflux/mintmaker/master/major-helpers

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.12.2)

Command failed


Comment @coderabbitai help to get the list of available commands.

@openshift-ci openshift-ci Bot requested review from amandahla and jerichokeyne July 1, 2026 15:07
@openshift-ci

openshift-ci Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: red-hat-konflux[bot]
Once this PR has been reviewed and has the lgtm label, please assign olucasfreitas for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci

openshift-ci Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Hi @red-hat-konflux[bot]. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-ci

openshift-ci Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

@red-hat-konflux[bot]: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-presubmits-images 7804a8a link true /test e2e-presubmits-images
ci/prow/build 7804a8a link true /test build
ci/prow/images-images 7804a8a link true /test images-images
ci/prow/test 7804a8a link true /test test
ci/prow/security 7804a8a link false /test security
ci/prow/lint 7804a8a link true /test lint

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@red-hat-konflux

Copy link
Copy Markdown
Contributor Author

Renovate Ignore Notification

Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future 3.x releases. But if you manually upgrade to 3.x then Renovate will re-enable minor and patch updates automatically.

If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.

@red-hat-konflux red-hat-konflux Bot deleted the konflux/mintmaker/master/major-helpers branch July 1, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Indicates a non-member PR verified by an org member that is safe to test.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant