-
Notifications
You must be signed in to change notification settings - Fork 166
get prod dockerfile working properly #10790
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
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
352a10c
make sure keys are readable
timothy-spencer 9ac6f5d
remove build-essential in a way that will not bloat the image
timothy-spencer 68859a5
yarn install needs build-essential too it seems
timothy-spencer 560d6aa
add make for asset stuff
timothy-spencer 8964ab5
add some network debugging
timothy-spencer 0924cb7
tracepath is not apparently a thing
timothy-spencer 1fc2d49
add more debug
timothy-spencer 92d27a4
use proper netcat
timothy-spencer ce32d07
remove network debugging tools
timothy-spencer 70b4177
changelog: Internal, Platform Automation, Update production idp image…
timothy-spencer fee5936
fix pwned_passwords.txt
timothy-spencer ee4a8a4
add puma header
92d37a1
trigger another build
timothy-spencer 4dcb3ff
move gem build and large files out into a multi-stage build
timothy-spencer 7c0159e
Update dockerfiles/idp_prod.Dockerfile
timothy-spencer 1e2d210
try removing some sqlite stuff
timothy-spencer 5a18996
move everything up and just copy rails root over
timothy-spencer f124d00
change where bundle install gets run and make sure it's configured pr…
timothy-spencer b5c8097
we need curl for healthchecks
timothy-spencer 36f9268
remove suid/sgid from binaries
timothy-spencer 32283fd
add service_providers.yml and related files
timothy-spencer aecc5b5
forgot to wire the env vars up to args
timothy-spencer 8a24005
trigger build
timothy-spencer 9041fe3
forgot to copy keys we generated over
timothy-spencer 0aaf75f
try using readonly deploy key
timothy-spencer 13ec4b8
get the host fingerprint, map the key in as an arg
timothy-spencer 2b71b59
change perms on key so it works
timothy-spencer 3f85789
just noticed that the service_provider stuff wasn't in the build dir
timothy-spencer 8fd25a6
move config copy to where workdir works
timothy-spencer c87f906
some debugging
timothy-spencer 18d83fa
try full path
timothy-spencer 91dd66b
I was totally missing that COPY was coming from the local filesystem,…
timothy-spencer 21192b1
don't overwrite the identity-idp-config files with localdev ones
timothy-spencer 09bc2d4
exclude node_modules
timothy-spencer 29c35f4
be more specific about node_modules
timothy-spencer 52b110a
testing
21ded89
revert
14acf63
remove package restrictions in builder
timothy-spencer 80f8cb1
try some debugging of the seeder error
timothy-spencer b6fd69e
forgot to assign error to a variable
timothy-spencer 84656cf
logger needs more log-ness to actually log I think
timothy-spencer 72588ab
get all exceptions
timothy-spencer d56ed58
did Agency.destroy_all and it doesn't have this issue
timothy-spencer 363535f
fix --no-install-recommends
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| /app/node_modules | ||
|
|
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.
Uh oh!
There was an error while loading. Please reload this page.