Skip to content

fix(deps): update all go dependencies main (main) #38865

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 8, 2025
Merged

fix(deps): update all go dependencies main (main) #38865

merged 1 commit into from
May 8, 2025

Conversation

cilium-renovate[bot]
Copy link
Contributor

@cilium-renovate cilium-renovate bot commented Apr 10, 2025 •
edited by aanm
Loading

This PR contains the following updates:

Package Type Update Change
github.com/aliyun/alibaba-cloud-sdk-go require patch v1.63.104 -> v1.63.105
github.com/cilium/charts require digest dd713bc -> f7c721a
github.com/cilium/endpointslice-controller require digest 75cb5d6 -> ffb33e2
github.com/prometheus/client_model require patch v0.6.1 -> v0.6.2
github.com/vishvananda/netlink require digest 5a4ec4b -> d85a66b
google.golang.org/genproto/googleapis/rpc require digest de1ac95 -> 207652e

Release Notes

aliyun/alibaba-cloud-sdk-go (github.com/aliyun/alibaba-cloud-sdk-go)

v1.63.105

Compare Source

  • Generated 2017-05-25, 2018-05-01 for Dysmsapi.
prometheus/client_model (github.com/prometheus/client_model)

v0.6.2

Compare Source

What's Changed

Full Changelog: prometheus/client_model@v0.6.1...v0.6.2

Configuration

📅 Schedule: Branch creation - "on sunday" (UTC), Automerge - At any time (no schedule defined).

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

♻ Rebasing: Never, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

@cilium-renovate cilium-renovate bot requested a review from a team as a code owner April 10, 2025 07:07
@cilium-renovate cilium-renovate bot added kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact. renovate/stop-updating Tell Renovate to stop updating PR labels Apr 10, 2025
@cilium-renovate cilium-renovate bot requested review from marseel and ciliumbot April 10, 2025 07:07
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Apr 10, 2025
@julianwiedmann
Copy link
Member

@joamaki heads-up, looks like the hive upgrade is giving CI a bad time.

@julianwiedmann julianwiedmann mentioned this pull request Apr 11, 2025
Merged
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch from 8acc02e to a3b31ac Compare April 11, 2025 15:08
@cilium-renovate cilium-renovate bot enabled auto-merge April 11, 2025 15:08
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 11, 2025 15:08 Inactive
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 11, 2025 15:11 Inactive
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 11, 2025 15:16 Inactive
@tklauser
Copy link
Member

tklauser commented Apr 15, 2025 •
edited
Loading

@joamaki heads-up, looks like the hive upgrade is giving CI a bad time.

It looks like the change from github.com/mitchellh/mapstructure to github.com/go-viper/mapstructure in cilium/hive#49 caused this. That's the only change in cilium/hive@7f3c86c -> cilium/hive@8eacab6

While not fixing the underlying issue, I think #38951 might help circumvent this issue and simplify the code a bit while at it.

@tklauser tklauser mentioned this pull request Apr 15, 2025
Merged
@cilium-renovate cilium-renovate bot force-pushed the renovate/main-all-go-deps-main branch from a3b31ac to f3934ed Compare April 16, 2025 08:08
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 16, 2025 08:08 Inactive
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 16, 2025 08:08 Inactive
@cilium-renovate cilium-renovate bot temporarily deployed to release-base-images-renovate April 16, 2025 08:10 Inactive
@julianwiedmann
Copy link
Member

@joamaki heads-up, looks like the hive upgrade is giving CI a bad time.

It looks like the change from github.com/mitchellh/mapstructure to github.com/go-viper/mapstructure in cilium/hive#49 caused this. That's the only change in cilium/hive@7f3c86c -> cilium/hive@8eacab6

While not fixing the underlying issue, I think #38951 might help circumvent this issue and simplify the code a bit while at it.

I didn't check closer, but after a rebase it looks like hive is still unhappy.

@tklauser
Copy link
Member

@joamaki heads-up, looks like the hive upgrade is giving CI a bad time.

It looks like the change from github.com/mitchellh/mapstructure to github.com/go-viper/mapstructure in cilium/hive#49 caused this. That's the only change in cilium/hive@7f3c86c -> cilium/hive@8eacab6
While not fixing the underlying issue, I think #38951 might help circumvent this issue and simplify the code a bit while at it.

I didn't check closer, but after a rebase it looks like hive is still unhappy.

Yeah, still seems to be the same issue. I'll take a look whether I can find a proper fix or otherwise we can revert back to mitchellh/mapstructure for now and report an issue upstream.

@julianwiedmann julianwiedmann marked this pull request as draft April 16, 2025 10:37
auto-merge was automatically disabled April 16, 2025 10:37

Pull request was converted to draft

@cilium-renovate @aanm
fix(deps): update all go dependencies main
4d9e9c3 
Signed-off-by: cilium-renovate[bot] <134692979+cilium-renovate[bot]@users.noreply.github.com>
@aanm aanm force-pushed the renovate/main-all-go-deps-main branch from f3934ed to 4d9e9c3 Compare May 8, 2025 09:44
@aanm aanm marked this pull request as ready for review May 8, 2025 09:44
@aanm aanm enabled auto-merge May 8, 2025 09:44
@aanm aanm added this pull request to the merge queue May 8, 2025
Merged via the queue into main with commit 6fffaa9 May 8, 2025
374 of 381 checks passed
@aanm aanm deleted the renovate/main-all-go-deps-main branch May 8, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ciliumbot ciliumbot ciliumbot approved these changes

@marseel marseel Awaiting requested review from marseel marseel is a code owner automatically assigned from cilium/vendor

Assignees
No one assigned
Labels
kind/community-contribution This was a contribution made by a community member. kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact. renovate/stop-updating Tell Renovate to stop updating PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@julianwiedmann @tklauser @ciliumbot @aanm