Skip to content

Add op-rbuilder release ci action#432

Merged
ferranbt merged 2 commits intoflashbots:developfrom
NethermindEth:msozin/op-rbuilder/only-op-docker
Feb 26, 2025
Merged

Add op-rbuilder release ci action#432
ferranbt merged 2 commits intoflashbots:developfrom
NethermindEth:msozin/op-rbuilder/only-op-docker

Conversation

@SozinM
Copy link
Copy Markdown
Contributor

@SozinM SozinM commented Feb 17, 2025

📝 Summary

💡 Motivation and Context


✅ I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

@avalonche
Copy link
Copy Markdown
Contributor

are you able to push a test tag to see if a docker image is published?

@SozinM
Copy link
Copy Markdown
Contributor Author

SozinM commented Feb 19, 2025

Hi @avalonche !
I think you could only test it with tag after you merge the branch.
I tested it locally with act, and it tried to push to gchr.io/flashbots/op-rbuilder:latest (as expected)

@SozinM SozinM force-pushed the msozin/op-rbuilder/only-op-docker branch from e54e49b to 5132034 Compare February 20, 2025 07:19
@SozinM SozinM changed the title Add only op-rbuilder to the github actions Add op-rbuilder docker relese action Feb 21, 2025
@SozinM SozinM changed the title Add op-rbuilder docker relese action Add op-rbuilder relese ci action Feb 21, 2025
@SozinM SozinM changed the title Add op-rbuilder relese ci action Add op-rbuilder release ci action Feb 21, 2025
id: extract_version
run: |
if [[ "${GITHUB_REF_TYPE}" == "tag" ]]; then
VERSION="${GITHUB_REF#refs/tags/op-}"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

op-v1.2.2 -> v.1.2.2

tags: ${{ steps.meta.outputs.tags }}
build-args: |
FEATURES=${{ matrix.features }}
RBUILDER_BIN=op-rbuilder
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

specify binary

# - target: aarch64-apple-darwin
# runner: warp-macos-14-arm64-6x
features:
- ${{ github.event.inputs.features }}${{ github.event.inputs.features && ',' }}optimism
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Add optimism to feature flags

uses: docker/metadata-action@v5
id: meta
with:
images: ghcr.io/${{ github.repository_owner }}/op-rbuilder
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

repo name would be ghcr.io/flashbots/op-rbuilder

@ferranbt ferranbt merged commit 6d6f703 into flashbots:develop Feb 26, 2025
4 checks passed
ferranbt added a commit that referenced this pull request Feb 26, 2025
ferranbt added a commit that referenced this pull request Feb 26, 2025
This reverts commit 6d6f703.

## 📝 Summary

<!--- A general summary of your changes -->

## 💡 Motivation and Context

<!--- (Optional) Why is this change required? What problem does it
solve? Remove this section if not applicable. -->

---

## ✅ I have completed the following steps:

* [ ] Run `make lint`
* [ ] Run `make test`
* [ ] Added tests (if applicable)
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