-
Notifications
You must be signed in to change notification settings - Fork 43
Open
Description
Now there are two weaknesses in release CI:
- Checking chart version before building images: https://github.com/oam-dev/cluster-gateway/blob/master/.github/workflows/go-release.yml#L35.
This causes several pre-release failure:
- https://github.com/oam-dev/cluster-gateway/actions/runs/3844678538
- https://github.com/oam-dev/cluster-gateway/actions/runs/3827369890
- https://github.com/oam-dev/cluster-gateway/actions/runs/3637240597
This is a great check. However, its success relies that the maintainers remember to modify the chart version. We can make the chart version a placeholder and replace it in CI.
- The ACR domain isn't correct: https://github.com/oam-dev/cluster-gateway/blob/master/.github/workflows/go-release.yml#L72
This make the recent v1.7.0 release fail and no arm64 image builded and pushed.
Besides, there is also a suggestion: use --platform argument to distinguish the image platforms rather than including platform in tag. GitHub Action like docker/build-push-action@v2 can help with that. This makes developers' lives easier. π
wonderflow and Somefive
Metadata
Metadata
Assignees
Labels
No labels