Skip to content

Version binaries#4276

Merged
mslipper merged 2 commits intodevelopfrom
jg/version_op_binaries
Dec 7, 2022
Merged

Version binaries#4276
mslipper merged 2 commits intodevelopfrom
jg/version_op_binaries

Conversation

@trianglesphere
Copy link
Contributor

Description

This adds two layers of versioning to builds. The first is providing git information to docker builds. We already pulled the information into the build via make & ldflags, but did not have it available because we did not have the .git folder inside docker.

The second is setting a reminder in the tag-bedrock-go-modules.sh script to set the version in the op-node, op-batcher, & op-proposer. This should try to correspond to the tag version. This might be able to be scripted, but scripting it is hard, so I chose to prompt the developer.

Metadata

  • Fixes ENG-3081

This ensure that the git metadata is available to make when it is
building the binaries.
This is easier to do than writing a script to update the individual
versions, but should be enough to remind the developer cutting
the release to do the updates.
@changeset-bot
Copy link

changeset-bot bot commented Dec 6, 2022

⚠️ No Changeset found

Latest commit: 8ec44c7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mergify
Copy link
Contributor

mergify bot commented Dec 7, 2022

This PR has been added to the merge queue, and will be merged soon.

mergify bot added a commit that referenced this pull request Dec 7, 2022
@mergify
Copy link
Contributor

mergify bot commented Dec 7, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify
Copy link
Contributor

mergify bot commented Dec 7, 2022

Hey @trianglesphere, this pull request failed to merge and has been dequeued from the merge train. If you believe your PR failed in the merge train because of a flaky test, requeue it by commenting with @mergifyio requeue.
More details can be found on the Queue: Embarked in merge train check-run.

@mergify mergify bot removed the on-merge-train label Dec 7, 2022
@mslipper mslipper merged commit 5645c35 into develop Dec 7, 2022
@mslipper mslipper deleted the jg/version_op_binaries branch December 7, 2022 17:21
@mslipper mslipper mentioned this pull request Dec 8, 2022
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.

5 participants