Skip to content
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.

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

upgrade go modules #651

Merged
merged 2 commits into from
Feb 20, 2025
Merged

upgrade go modules #651

merged 2 commits into from
Feb 20, 2025

Conversation

frezes
Copy link
Collaborator

@frezes frezes commented Feb 19, 2025

No description provided.

// Pin kuberesolver/v5 to support new grpc version. Need to upgrade kuberesolver version on weaveworks/common.
github.com/sercand/kuberesolver/v4 => github.com/sercand/kuberesolver/v5 v5.1.1

github.com/thanos-io/thanos => github.com/thanos-io/thanos v0.0.0-20240723122114-639bf8f21649
github.com/thanos-io/thanos => github.com/frezes/thanos v0.0.0-20250219031509-186a9979b843
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

临时先用我当前repo,等待 thanos-io/thanos#8112 合入

Copy link
Member

Choose a reason for hiding this comment

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

升级这个的目的是为了解决什么问题?thanos-io/thanos#8112
我看这个 PR 的 ci 失败了

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

thanos 里 prometheus 依赖库升级不彻底,导致 whizard 依赖更新也出问题,等稍后有时间会修复 CI 问题

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

上游 CI 报错应该不是我这个 PR 引起的,而且已经 approve 了

@benjaminhuo benjaminhuo requested a review from junotx February 19, 2025 09:24
@junotx junotx merged commit 55fa459 into WizTelemetry:main Feb 20, 2025
1 check 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