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

Flesh out live namespace #5

merged 10 commits into from Nov 15, 2019

Flesh out live namespace #5

merged 10 commits into from Nov 15, 2019


Copy link

barrucadu commented Nov 15, 2019

This PR makes some quality-of-life improvements to the live namespace. The highlights are:

  • All the live apps work
  • Ruby is only built once (done by adding a new CI base name)
  • App definitions have environment-specific config split out, meaning they can be shared across environments
  • Apps run in production mode
  • Documentation exists

Closes #3

barrucadu added 3 commits Nov 12, 2019
This is possibly an abuse of secrets, because it's not secret at all,
but secrets are a convenient way to set per-environment environment
I don't know why the slaves and concourse need so much space, the
registry is far below this and stores all the images.
@barrucadu barrucadu force-pushed the live-apps branch 3 times, most recently from 8835c21 to 6680479 Nov 15, 2019
barrucadu added 7 commits Nov 12, 2019
- Copy over only the required apps
- Get the jumpbox IP from terraform automatically
- GOVUK_ASSET_ROOT: which should be "" to serve from the current host
- SECRET_KEY_BASE: which is generated anew for every app
- RAILS_SERVE_STATIC_FILES: needed to serve assets in production

SECRET_KEY_BASE generation is done by adding "SECRET_KEY_BASE-appname"
entries to secrets with the value "GENERATED_SECRET_KEY_BASE". then replaces one GENERATED_SECRET_KEY_BASE at a time with a
UUID.  I'm not sure what the recommendations for a Rails secret key
base are, but UUIDs are probably good enough for my purposes.
@barrucadu barrucadu force-pushed the live-apps branch from 6680479 to 9567d00 Nov 15, 2019
@barrucadu barrucadu merged commit ccc8ad8 into master Nov 15, 2019
@barrucadu barrucadu deleted the live-apps branch Nov 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
1 participant
You can’t perform that action at this time.