Skip to content

Make hostapp-deploy depend on a successful s3-deploy #649

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

Merged
merged 1 commit into from
May 28, 2025

Conversation

klutchell
Copy link
Contributor

If something failed on S3 upload, we do not want to deploy the hostapp as it will likely need to be invalidated.

This order will change when we move to web resources, and uploads depend on the hostapp creation.

Change-type: minor

@klutchell klutchell temporarily deployed to balena-staging.com May 27, 2025 18:46 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-staging.com May 27, 2025 18:46 — with GitHub Actions Inactive
@flowzone-app flowzone-app bot enabled auto-merge May 27, 2025 18:53
@klutchell klutchell temporarily deployed to balena-cloud.com May 27, 2025 19:17 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-cloud.com May 27, 2025 19:17 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-cloud.com May 27, 2025 19:17 — with GitHub Actions Inactive
@klutchell klutchell requested a review from a team May 27, 2025 19:53
If something failed on S3 upload, we do not want to deploy the
hostapp as it will likely need to be invalidated.

This order will change when we move to web resources, and uploads
depend on the hostapp creation.

Change-type: minor
Signed-off-by: Kyle Harding <[email protected]>
@klutchell klutchell force-pushed the kyle/hostapp-needs branch from 599b2c9 to 7e5c501 Compare May 28, 2025 10:55
@klutchell klutchell temporarily deployed to balena-staging.com May 28, 2025 10:55 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-staging.com May 28, 2025 10:55 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-cloud.com May 28, 2025 11:17 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-cloud.com May 28, 2025 11:17 — with GitHub Actions Inactive
@klutchell klutchell temporarily deployed to balena-cloud.com May 28, 2025 13:10 — with GitHub Actions Inactive
@flowzone-app flowzone-app bot merged commit e72c09b into master May 28, 2025
83 of 87 checks passed
@flowzone-app flowzone-app bot deleted the kyle/hostapp-needs branch May 28, 2025 14:01
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