-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
chore(operator): Update build and runtime deps #14416
Conversation
2401693
to
ae5db7f
Compare
388c113
to
318fcb5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to work fine per my test, only have two small comments/questions...
@@ -1,7 +1,7 @@ | |||
module _ // Auto generated by https://github.com/bwplotka/bingo. DO NOT EDIT | |||
|
|||
go 1.21 | |||
go 1.22.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did bingo
produce this? Usually this line is just major.minor
and not the complete version ...
I noticed later that there's go 1.22.0
on the operator/go.mod
and now I'm even more confused.
@@ -178,7 +178,7 @@ func main() { | |||
os.Exit(1) | |||
} | |||
} else { | |||
fmt.Fprintf(os.Stdout, "---\n%s", b) | |||
_, _ = fmt.Fprintf(os.Stdout, "---\n%s", b) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we put a "ignore this line in lint" comment on this instead of the _, _ =
?
[release-6.0] Backport PR grafana#14416
[release-5.6] Partial backport PR grafana#14416
[release-5.9] Partial backport PR grafana#14416
[release-5.8] Partial backport PR grafana#14416
What this PR does / why we need it:
Regular upgrade cycle before releasing the next Community Loki Operator version:
v1.21
v1.22.8
1.28.7
v1.30.5
v0.16.5
v0.18.5
0.71.2
v0.76.1
v1.19.0
v1.20.4
v1.33.0
v1.33.0
v1.56.2
v1.61.0
v0.6.0
v0.7.0
v0.22.0
v0.22.0
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Checklist
CONTRIBUTING.md
guide (required)feat
PRs are unlikely to be accepted unless a case can be made for the feature actually being a bug fix to existing behavior.docs/sources/setup/upgrade/_index.md
deprecated-config.yaml
anddeleted-config.yaml
files respectively in thetools/deprecated-config-checker
directory. Example PR