-
Notifications
You must be signed in to change notification settings - Fork 70
Re-organize workflows and releasing process #202
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
Conversation
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.
@v9n, could you check out this PR, to see if my understanding of the current flow is correct, and is there any concerns with this optimization?
|
||
# Due to EigenSDK package differences on Ethereum mainnet, | ||
# this workflow ensures the mainnet branch stays in sync with main | ||
# while maintaining its own Docker image tag. |
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.
The original mainnet.yml
is deleted and triggered manually by update-docker-mainnet.yml
on-demand instead.
|
||
- name: Run semantic versioning and release | ||
id: semantic-release | ||
uses: go-semantic-release/action@v1 |
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.
The go-semantic-release
library will replace the existing tagging process offered by the Go native method. It enforces version format, auto-bump versions, and create a release.
The scripts are not tested yet, but if the plan in RELEASE.md makes sense I will test and finalize them.
|
||
on: | ||
push: | ||
branches: | ||
- staging |
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.
The staging Docker image is updated after each PR merge. This branch serves as a caching and testing pool before changes are merged into main, which basically means the code goes live.
uses: docker/build-push-action@v6 | ||
with: | ||
build-args: | | ||
RELEASE_TAG=${{ inputs.tag || github.sha || github.head_ref || github.ref_name }} | ||
RELEASE_TAG=staging-${{ github.sha }} |
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.
i thinkwe should keep:
- branch-name: so we can pin toward
avaprotocol/avs-dev:test-my-new-feature
instead of manuallly update your code everytime to point to new sha - really userful to have the abilitty to trigger build with dispatch and set the tag yourself.
tags: | | ||
avaprotocol/ap-avs:mainnet | ||
build-args: | | ||
RELEASE_TAG=mainnet-${{ github.sha }} |
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.
not github.sha. should be the tag so we can have mainnet
as latest and mainnet-1.5.0
and mainnet-1.6.0
.
we don't usually need mainnet-abcdef
for mainnet.
uses: docker/build-push-action@v6 | ||
with: | ||
build-args: | | ||
RELEASE_TAG=${{ github.sha }} |
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.
we shoukd keep the branch
name: 'Base' | ||
environment: 'Base' | ||
directory: 'base' | ||
- id: 'deploy-ethereum.yml' |
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.
base and ethereum should be trigger after we release the mainnet branc docker image. otherwise they will not be up to date. so you may want to move these to run after update-docker-mainnet
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.
I suggest to hold back this change and wait until you completed your avs-infra
deployment for Minato.
At that time you will have better understanding of what docker images and tags we used in testnet and mainnet.
7422b32
to
63bc1de
Compare
…ile such as deploy-base.yml
47bb934
to
72234c6
Compare
* Re-organize workflows and releasing process * Added Environment details to Release.md * Added commit message naming convension * Change to publish dev docker on PR close * Change to publish dev docker on PR close * Update .github/workflows/deploy-avs.yml so we don’t need individual file such as deploy-base.yml * Added release-on-pr-close.yml * Renamed yaml files * Updated RELEASE.md
* Re-organize workflows and releasing process * Added Environment details to Release.md * Added commit message naming convension * Change to publish dev docker on PR close * Change to publish dev docker on PR close * Update .github/workflows/deploy-avs.yml so we don’t need individual file such as deploy-base.yml * Added release-on-pr-close.yml * Renamed yaml files * Updated RELEASE.md
* Re-organize workflows and releasing process * Added Environment details to Release.md * Added commit message naming convension * Change to publish dev docker on PR close * Change to publish dev docker on PR close * Update .github/workflows/deploy-avs.yml so we don’t need individual file such as deploy-base.yml * Added release-on-pr-close.yml * Renamed yaml files * Updated RELEASE.md
No description provided.