Skip to content
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

GH workflow to publish images for PRs labeled 'publish' #1540

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

dandelany
Copy link
Collaborator

@dandelany dandelany commented Aug 27, 2024

Supporting this issue: #1538

This change will cause the GH "Publish" workflow to run whenever an aerie PR is given the "publish" label, which will cause Docker images to be published from the PR branch, tagged with the PR number eg. pr-1540

The published aerie PR can then be referenced in an aerie-ui PR with a flag in the PR body that looks like eg. ___REQUIRES_AERIE_PR___="1540", which will cause the aerie-ui e2e tests to use the published pr-1540 aerie images during testing rather than the default develop branch.

@dandelany dandelany added ci Changes to our CI configuration files and scripts publish-images and removed publish-images labels Aug 27, 2024
@dandelany dandelany added publish Tells GH to publish docker images for this PR and removed publish-images labels Aug 28, 2024
@dandelany dandelany changed the title WIP : GH workflow to publish images for PRs labeled 'publish-images' GH workflow to publish images for PRs labeled 'publish' Aug 28, 2024
@dandelany dandelany marked this pull request as ready for review August 28, 2024 01:45
@dandelany dandelany requested a review from a team as a code owner August 28, 2024 01:45
@dandelany dandelany requested review from mattdailis, JoelCourtney and skovati and removed request for mattdailis August 28, 2024 01:45
@Mythicaeda
Copy link
Contributor

In order to test the synchronization event, could you flatten the working commits and force push?

Copy link
Contributor

@skovati skovati left a comment

Choose a reason for hiding this comment

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

Will removing the publish tag on this PR remove the failed check and allow the PR to be merged?

@Mythicaeda
Copy link
Contributor

The PR isn't actually blocked on the failure, it's blocked on a lack of an approval. The reason it isn't blocked is because the Publish workflow isn't a required workflow, unlike our test workflow.

@dandelany dandelany force-pushed the feature/publish-pr-branch-workflow branch from 46b989b to 1811bed Compare August 28, 2024 20:13
@dandelany
Copy link
Collaborator Author

In order to test the synchronization event, could you flatten the working commits and force push?

Done! will do the same for the other 2 PRs.

@dandelany dandelany merged commit 2186764 into develop Aug 28, 2024
21 of 22 checks passed
@dandelany dandelany deleted the feature/publish-pr-branch-workflow branch August 28, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to our CI configuration files and scripts publish Tells GH to publish docker images for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants