Skip to content

ci: add auto-generated PR build containers#85

Merged
synackd merged 6 commits into
mainfrom
ci/pr-artifacts
May 20, 2026
Merged

ci: add auto-generated PR build containers#85
synackd merged 6 commits into
mainfrom
ci/pr-artifacts

Conversation

@synackd
Copy link
Copy Markdown
Collaborator

@synackd synackd commented May 19, 2026

Pull Request Template

Thank you for your contribution! Please ensure the following before submitting:

Checklist

  • My code follows the style guidelines of this project
  • I have added/updated comments where needed
  • I have added tests that prove my fix is effective or my feature works
  • I have run make test (or equivalent) locally and all tests pass
  • DCO Sign-off: All commits are signed off (git commit -s) with my real name and email
  • REUSE Compliance:
    • Each new/modified source file has SPDX copyright and license headers
    • Any non-commentable files include a <filename>.license sidecar
    • All referenced licenses are present in the LICENSES/ directory

Description

Auto-generate OCI container with a pr-<pr_number> tag on pull requests to be able to be pulled during testing in order to avoid having to manually build it.

Also add a cleanup job that deletes PR container tags after it is closed.

Also fix some previously-uncaught Goreleaser issues:

  • linux/amd64 on arm64 container builds
  • add --platform=$TARGETPLATFORM to multi-stage Dockerfile.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

For more info, see Contributing Guidelines.

Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd synackd force-pushed the ci/pr-artifacts branch 2 times, most recently from c707c5d to dc24018 Compare May 19, 2026 19:52
synackd added 2 commits May 19, 2026 15:18
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd synackd force-pushed the ci/pr-artifacts branch from dc24018 to 1113fd6 Compare May 19, 2026 22:07
@OpenCHAMI OpenCHAMI deleted a comment from github-actions Bot May 19, 2026
@synackd synackd force-pushed the ci/pr-artifacts branch 3 times, most recently from f3d2fa6 to d8b2b3a Compare May 19, 2026 22:20
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd synackd force-pushed the ci/pr-artifacts branch 8 times, most recently from dc55f8b to 235117b Compare May 20, 2026 03:45
synackd added 2 commits May 19, 2026 21:54
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
Signed-off-by: Devon Bautista <17506592+synackd@users.noreply.github.com>
@synackd synackd force-pushed the ci/pr-artifacts branch from 235117b to a260e2e Compare May 20, 2026 03:56
@synackd synackd changed the title ci: add auto-generated PR build artifacts ci: add auto-generated PR build containers May 20, 2026
@synackd synackd marked this pull request as ready for review May 20, 2026 04:21
Copy link
Copy Markdown
Contributor

@davidallendj davidallendj left a comment

Choose a reason for hiding this comment

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

LGTM and nice to finally have this for ochami

@synackd synackd merged commit 68d91f2 into main May 20, 2026
12 checks passed
@synackd synackd deleted the ci/pr-artifacts branch May 20, 2026 15:35
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.

2 participants