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

Create GA autodeployment file #20

Merged
merged 19 commits into from
Nov 10, 2024
Merged

Create GA autodeployment file #20

merged 19 commits into from
Nov 10, 2024

Conversation

pcraig3
Copy link
Collaborator

@pcraig3 pcraig3 commented Oct 29, 2024

Summary

Adding a GitHub Actions file to set up autodeployments to Cloud Run.

This PR works for most of the PRs we will have come through, but does not work for migrations, which are still performed manually.

This is good enough to merge though, as it will save us lots of time, and then I can reinvest some of that times to figure out this migrations step.

Also bumping the version because this is a big step for the app.

@pcraig3 pcraig3 force-pushed the ga-autodeploy branch 10 times, most recently from 771a7fb to ec6ea5f Compare November 4, 2024 20:18
@pcraig3 pcraig3 force-pushed the ga-autodeploy branch 3 times, most recently from c1fff08 to ba66e29 Compare November 8, 2024 14:58
For now this just logs us in.
@pcraig3 pcraig3 force-pushed the ga-autodeploy branch 7 times, most recently from da739d3 to ecb67b3 Compare November 9, 2024 23:15
It has a large comment block that we don't need for anything.
Also, clean up some of the print statements.
Pull the SHA from the latest commit into the app as an
environment variable.

The idea is that we can clearly see what version of the app is
running if we insert the SHA into the HTML somewhere.

Classic Paul Craig Web App™️
We aren't using this, so we don't need it.
This code is pretty ugly still but I got rid of a few lines.
Specificially, remove mentions of the HTML file, since that's not
really something that we support anymore.

Also make the language around Bloom coaches more general.
According to ChatGPT:

> The COPY . . command in your Dockerfile should typically come right after setting the working directory and before installing dependencies to ensure that any code changes trigger a rebuild of subsequent layers. This guarantees that any changes in your application files force Docker to rerun commands that depend on them, like collectstatic.
There is some warning about how there are duplicate files being
sent to the same destination, which doesn't matter to me at all.

Let's just keep the pipeline that we have.
@pcraig3 pcraig3 changed the title [WIP] First test of GA autodeployment file Create GA autodeployment file Nov 10, 2024
@pcraig3 pcraig3 merged commit a3df4da into main Nov 10, 2024
1 check passed
@pcraig3 pcraig3 deleted the ga-autodeploy branch November 13, 2024 16:22
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.

1 participant