Skip to content

fix(deps): update module go.yaml.in/yaml/v2 to v3 (main) - #7284

Closed
renovate-sh-app[bot] wants to merge 1 commit into
mainfrom
renovate/main-go.yaml.in-yaml-v2-3.x
Closed

renovate-sh-app[bot] wants to merge 1 commit into
mainfrom
renovate/main-go.yaml.in-yaml-v2-3.x

Conversation

@renovate-sh-app

@renovate-sh-app renovate-sh-app Bot commented May 27, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
go.yaml.in/yaml/v2 v2.4.4v3.0.4 age confidence

Release Notes

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

v3.0.4

Compare Source

v3.0.3

Compare Source

v3.0.2

Compare Source

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: Enabled.

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

Need help?

You can ask for more help in the following Slack channel: #proj-renovate-self-hosted. In that channel you can also find ADR and FAQ docs in the Resources section.

@renovate-sh-app renovate-sh-app Bot added dependencies Pull requests that update a dependency file gomod labels May 27, 2026
@renovate-sh-app
renovate-sh-app Bot requested a review from mdisibio as a code owner May 27, 2026 21:37
@renovate-sh-app renovate-sh-app Bot added major renovate Applied to PR's created by renovatebot update-major labels May 27, 2026
@renovate-sh-app
renovate-sh-app Bot enabled auto-merge (squash) May 27, 2026 21:37
@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch 3 times, most recently from b47fb5c to 069eaaa Compare June 4, 2026 17:40
@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch 6 times, most recently from 5c613f6 to 6a59290 Compare June 9, 2026 08:12

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

#7284: yaml v3 API break -> adapted strict decode/unmarshalers, opened #7435 -> main. CI green; CLA pending.

View PR

Open in Web View Automation 

Sent by Cursor Automation: Renovate PR Unblocker

@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch from 6a59290 to 23397f5 Compare June 9, 2026 11:11
@cursor cursor Bot mentioned this pull request Jun 9, 2026
3 tasks

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

#7284: YAML v3 API break -> adapted strict decoding/unmarshalers, opened #7436 -> main. CI passed; CLA pending.

View PR

Open in Web View Automation 

Sent by Cursor Automation: Renovate PR Unblocker

@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch 11 times, most recently from 805e2ce to 4bb19c8 Compare June 12, 2026 17:21
@tempo-ci-app

tempo-ci-app Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🤖 fix-renovate-prs failed_to_fix: yaml.v2→v3 is a breaking API change: UnmarshalYAML signature changes (6 sites in overrides/listtomap/duration) and yaml.UnmarshalStrict (12 call sites) was removed in v3. A naive signature port would silently break modules/overrides/config.go:317 and runtime_config_overrides.go:39 — they detect legacy-format YAML by catching strict-mode errors from the inner unmarshal callback, but in v3 Node.Decode always creates an inner decoder with knownFields=false, so legacy tenant override files would parse as ConfigTypeNew with all limits zeroed. A correct fix needs hand-rolled key validation inside each UnmarshalYAML against the *yaml.Node — a real refactor of the overrides module rather than a Renovate cleanup. Next step: a human should redo the overrides UnmarshalYAML methods using the Node API (or revisit whether the legacy-fallback pattern can be dropped entirely).

@grafana grafana deleted a comment from tempo-ci-app Bot Jun 12, 2026
@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch from 4bb19c8 to 36dd2bc Compare June 12, 2026 20:11
@github-actions

This comment has been minimized.

@tempo-ci-app

tempo-ci-app Bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

🤖 fix-renovate-prs: opened a CI fix → #7484

| datasource | package            | from   | to     |
| ---------- | ------------------ | ------ | ------ |
| go         | go.yaml.in/yaml/v2 | v2.4.4 | v3.0.4 |


Signed-off-by: renovate-sh-app[bot] <219655108+renovate-sh-app[bot]@users.noreply.github.com>
@renovate-sh-app
renovate-sh-app Bot force-pushed the renovate/main-go.yaml.in-yaml-v2-3.x branch from 36dd2bc to 23a2214 Compare June 16, 2026 08:10
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ The workflow jobs listed below don't declare a permissions: block and may break when the organization's default GITHUB_TOKEN permissions are restricted to read-only.

Expand for findings
warning[excessive-permissions]: overly broad permissions
  --> ./.github/workflows/deploy-pr-preview.yml:13:3
   |
13 | /   deploy-pr-preview:
14 | |     if: "!github.event.pull_request.head.repo.fork"
15 | |     uses: grafana/writers-toolkit/.github/workflows/deploy-preview.yml@4dbecae7443c69b3e5fdfeeac7037a339cb82e4c # main
16 | |     with:
...  |
31 | |       title: ${{ github.event.pull_request.title }}
   | |                                                    ^
   | |                                                    |
   | |____________________________________________________this job
   |                                                      default permissions used due to no permissions: block
   |
   = note: audit confidence → Medium
   = help: audit documentation → https://docs.zizmor.sh/audits/#excessive-permissions

@stoewer stoewer closed this Jun 16, 2026
auto-merge was automatically disabled June 16, 2026 09:26

Pull request was closed

@renovate-sh-app

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.

@renovate-sh-app
renovate-sh-app Bot deleted the renovate/main-go.yaml.in-yaml-v2-3.x branch June 16, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file gomod major renovate Applied to PR's created by renovatebot update-major

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant