Skip to content

Update NEWS and release process #1687

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

Merged
merged 3 commits into from
Aug 8, 2024
Merged

Conversation

kevinAlbs
Copy link
Collaborator

@kevinAlbs kevinAlbs commented Aug 6, 2024

Summary

  • Update the NEWS following the 1.27.5 release.
  • Specify amd64 platform for snyk-monitor-snapshot target to ensure downloaded snyk binary matches container architecture.
  • Remove Homebrew steps from release.

Background & Motivation

Specifying the amd64 platform on the snyk-monitor-snapshot target is intended to address an error observed during the release:

================================== ❌ FAILURE ===================================

+snyk-monitor-snapshot *failed* | Repeating the failure error...
+snyk-monitor-snapshot *failed* | branch=r1.27 name=release-1.27.5
+snyk-monitor-snapshot *failed* | --> RUN --no-cache snyk monitor --org=$SNYK_ORGANIZATION --target-reference=$name --unmanaged --print-deps --project-name=mongo-c-driver --remote-repo-url=https://github.com/mongodb/mongo-c-driver
+snyk-monitor-snapshot *failed* | qemu-x86_64: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
+snyk-monitor-snapshot *failed* | ERROR Earthfile line 394:4
+snyk-monitor-snapshot *failed* |       The command
+snyk-monitor-snapshot *failed* |           RUN --no-cache snyk monitor --org=$SNYK_ORGANIZATION --target-reference=$name --unmanaged --print-deps --project-name=mongo-c-driver --remote-repo-url=https://github.com/mongodb/mongo-c-driver
+snyk-monitor-snapshot *failed* |       did not complete successfully. Exit code 255

I expect this was due to me running on macOS M1 (arm64). The downloaded Snyk binary is for amd64.

Homebrew steps are removed due to no longer appearing to be necessary:

% brew bump-formula-pr mongo-c-driver --url https://github.com/mongodb/mongo-c-driver/archive/refs/tags/1.27.5.tar.gz
Error: Whoops, the mongo-c-driver formula has its version update
pull requests automatically opened by BrewTestBot every ~3 hours!
We'd still love your contributions, though, so try another one
that's not in the autobump list:
  https://github.com/Homebrew/homebrew-core/blob/master/.github/autobump.txt

The target downloads an amd64 binary. Intended to fix an error if the target is run on arm64.
@kevinAlbs kevinAlbs changed the title Update NEWS for 1.27.5 and fix Earthly target on non-amd64 Update NEWS and release process Aug 6, 2024
@kevinAlbs kevinAlbs requested a review from rcsanchez97 August 6, 2024 17:01
@kevinAlbs kevinAlbs merged commit 8e48bd1 into mongodb:master Aug 8, 2024
46 checks 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