Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

publish cross platform artifacts for PRs #2041

Merged
merged 2 commits into from
Aug 30, 2021

Conversation

ndeloof
Copy link
Collaborator

@ndeloof ndeloof commented Aug 25, 2021

What I did
Build cross platform binaries and publish artifacts as part of PR builds to make it easier testing changes

@github-actions github-actions bot added the ci Continuous integration label Aug 25, 2021
@ndeloof ndeloof force-pushed the pr-binaries branch 5 times, most recently from b2cc2ff to 817c9ee Compare August 25, 2021 10:04
@ndeloof ndeloof force-pushed the pr-binaries branch 3 times, most recently from 100b6ec to b912b9d Compare August 25, 2021 13:45
Copy link
Contributor

@ulyssessouza ulyssessouza left a comment

Choose a reason for hiding this comment

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

I think we should have any mechanism to trigger that and not just generate the artifacts for all the PRs.

Maybe using something like the "/test-aci" tag to enable tests on ACI, for example.
A /generate-artifacts would be nice!

@ndeloof ndeloof force-pushed the pr-binaries branch 5 times, most recently from 79551d3 to 46d3150 Compare August 25, 2021 14:43
@ndeloof
Copy link
Collaborator Author

ndeloof commented Aug 25, 2021

@ulyssessouza makes sense.
unfortunately, this makes it harder to retrieve the matching GA workflow run to get binaries

@ndeloof ndeloof force-pushed the pr-binaries branch 2 times, most recently from a0b8154 to 93d0bc0 Compare August 25, 2021 15:12
@ndeloof
Copy link
Collaborator Author

ndeloof commented Aug 25, 2021

Added an extra step to update the PR comment with a link to workflow run page, so that user can access built binaries
see sample run on ndeloof#4

Copy link
Contributor

@mat007 mat007 left a comment

Choose a reason for hiding this comment

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

Nice!

.github/workflows/pending_answered.yml Outdated Show resolved Hide resolved
.github/workflows/artifacts.yml Outdated Show resolved Hide resolved
ndeloof and others added 2 commits August 30, 2021 14:43
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Co-authored-by: Mathieu Champlon <mathieu.champlon@docker.com>
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof merged commit 55dd768 into docker-archive:main Aug 30, 2021
@ndeloof ndeloof deleted the pr-binaries branch August 30, 2021 12:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ci Continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants