Skip to content

Conversation

@grahamalama
Copy link
Contributor

This PR ensures that we copy the new asgi.py file (introduced in #161) into the container, which fixes #178.

It also adds a step in the build / publish workflow to spin up the container and check that it starts successfully.

@grahamalama grahamalama requested a review from a team as a code owner August 2, 2022 14:59
@leplatrem leplatrem added the bug Something isn't working label Aug 2, 2022
@grahamalama grahamalama force-pushed the ensure-container-runs branch from 2ae4dd1 to 138db9d Compare August 2, 2022 15:08
@grahamalama grahamalama force-pushed the ensure-container-runs branch from 138db9d to 3b50867 Compare August 2, 2022 15:10
@grahamalama grahamalama force-pushed the ensure-container-runs branch from 3b8d626 to 969b51b Compare August 2, 2022 15:56
Copy link
Contributor

@bsieber-mozilla bsieber-mozilla left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for adding this step/check!

@grahamalama grahamalama merged commit ac34fac into main Aug 2, 2022
@grahamalama grahamalama deleted the ensure-container-runs branch August 2, 2022 16:30
@leplatrem leplatrem mentioned this pull request Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Container not starting up -- no asgi.py file

4 participants