-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[7.17](backport #32574) [automation] Update go release version 1.18.5 #32693
Conversation
* [Automation] Update go release version to 1.18.5 * Merge Go 1.18 fixes (#32653) * Update Kafka base image to Debian buster. Done to allow updating to OpenJDK 11 to ensure compatibility with the output from keytool on my local machine that used a much more recent version of OpenJDK. * Regenerate Kafka certs to use SHA-256. The certificates were previously signed with SHA-1 which is deprecated in Go 1.18 due to being insecure. * Rename test cases so they can be run individually * Upgrade golangci-lint in actions. * Update notice file. * Fix misplaced changelog entry. * Add release note for Go 1.18 upgrade. * Upgrade to elastic-agent-libs v0.2.11 Updates the linter targets to auto upgrade when the linter version changes. * Work around errorlint bug in wineventlog pkg. errorlint does not correctly parse indexed fmt verbs when they have a verb modifier. Co-authored-by: apmmachine <infra-root-apmmachine@elastic.co> Co-authored-by: Craig MacKenzie <craig.mackenzie@elastic.co> (cherry picked from commit 56ca6f5) # Conflicts: # .github/workflows/golangci-lint.yml # .go-version # .golangci.yml # NOTICE.txt # auditbeat/Dockerfile # go.mod # go.sum # heartbeat/Dockerfile # libbeat/docs/version.asciidoc # metricbeat/Dockerfile # packetbeat/Dockerfile # winlogbeat/sys/wineventlog/publisher_metadata.go # winlogbeat/sys/wineventlog/syscall_windows.go # x-pack/functionbeat/Dockerfile
💔 Build Failed
Expand to view the summary
Build stats
Test stats 🧪
Steps errorsExpand to view the steps failures
|
The current plan is to merge this after the release of 7.17.6. |
This pull request has not been merged yet. Could you please review and merge it @cmacknz, @apmmachine? 🙏 |
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request has not been merged yet. Could you please review and merge it @cmacknz, @apmmachine? 🙏 |
Reminder: we need to bump the version to go 1.18 in go.mod as well. #32940 |
Bringing this PR back to life now that 7.17.6 is released. |
Note https://groups.google.com/g/golang-dev/c/cXXh_u-iRmg
|
Running
7.17 doesn't actually import |
This pull request has not been merged yet. Could you please review and merge it @cmacknz, @apmmachine? 🙏 |
Packaging is failing with this error which I think is unrelated:
|
/package |
3 similar comments
/package |
/package |
/package |
/package x-pack/heartbeat |
/package |
Managed to get packaging to succeed, merging: https://beats-ci.elastic.co/job/Beats/job/packaging/view/change-requests/job/PR-32693/ |
This is an automatic backport of pull request #32574 done by Mergify.
Cherry-pick of 56ca6f5 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally
Mergify commands and options
More conditions and actions can be found in the documentation.
You can also trigger Mergify actions by commenting on this pull request:
@Mergifyio refresh
will re-evaluate the rules@Mergifyio rebase
will rebase this PR on its base branch@Mergifyio update
will merge the base branch into this PR@Mergifyio backport <destination>
will backport this PR on<destination>
branchAdditionally, on Mergify dashboard you can:
Finally, you can contact us on https://mergify.com