-
Notifications
You must be signed in to change notification settings - Fork 7
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
fix: split up build and deploy steps #2838
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:04
fe0abd3
to
9d8be3b
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:16
9d8be3b
to
4951d75
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:18
4951d75
to
84cd284
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:19
84cd284
to
ac31642
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:25
ac31642
to
9562573
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:34
9562573
to
f076385
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 4, 2024 18:39
f076385
to
13a0bb3
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 5, 2024 00:47
13a0bb3
to
fc2b7f5
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 5, 2024 01:35
fc2b7f5
to
929252a
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 5, 2024 01:40
929252a
to
730dbaf
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 17:05
166de24
to
bc0c160
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 17:08
bc0c160
to
8bf218c
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 17:26
8bf218c
to
14c604e
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 17:36
14c604e
to
28f6ef8
Compare
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 17:41
28f6ef8
to
e0f233b
Compare
hannahpurcell
approved these changes
Oct 7, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, and tests well in your dev deployments. One question I don't understand, but otherwise, this seems reasonable.
firestack
force-pushed
the
kf/fix/split-steps
branch
from
October 7, 2024 23:02
e0f233b
to
70087e4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When deploying #2831 I was having difficulties with getting the deployment to succeed due to our database issues. It was getting pretty frustrating to have to build the docker container just to attempt a redeploy. I tried a few ways of parallelizing this, but so far this seems like the best option.
I was having trouble getting a deploy to work for testing this, so this also depends on #2841
Depends on:
tzdata
#2841