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

Move inventories + Change File Structure #672

Open
wants to merge 2,595 commits into
base: master
Choose a base branch
from
Open

Move inventories + Change File Structure #672

wants to merge 2,595 commits into from

Conversation

thejsj
Copy link
Member

@thejsj thejsj commented May 19, 2017

Why

  • Ultimate goal: Separate out deployment instructions from state + variables in order to use devops-scripts for on-prem. With these changes we're on our way to moving self-hosted customer to kubernetes more easily
  • All environment specific configuration is separated from state
  • Better security since severely limit what part of repo has any secrets + sensitive information

What this PR does

  • Create an /environments directory
  • Move all *-host directories to the /environments/$ENV/inventory directory
  • Move all k8 files to /inventories/$ENV/k8

Dependencies

  • Update papyrus with new changes
  • Create confluence documentation page explaining changes

Tests

  • Make k8 sure all files are correctly created
  • Test with big-poppa
  • Test with api
  • Test with image-builder
  • Test with khronos

Deployment (post-merge)

Ensure that all environments have the given changes.

  • deployed to gamma
  • deployed to delta

We should treat our k8 files as compiled files, since we don't actually (http://blog.andrewray.me/dealing-with-compiled-files-in-git/). We should probably wait some time until we actually have more confidence in our compilation.

@anandkumarpatel
Copy link

@thejsj close this since it is already done?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants