Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Change CI base to GovCloud #1106

Merged
merged 4 commits into from
Jun 1, 2017
Merged

Change CI base to GovCloud #1106

merged 4 commits into from
Jun 1, 2017

Conversation

fureigh
Copy link
Contributor

@fureigh fureigh commented May 31, 2017

This PR does the following:

  • Change Circle CI's base from East/West-or-GovCloud to GovCloud-only
  • Remove East/West manifests
  • Remove East/West conditionals from circle_deploy.sh
  • Make a few small copy edits to the relevant section of the README

Ready for review. @msecret, I'd especially like your take on whether there's any reason not to remove lines 5 and 6 of circle_deploy.sh, which I suspect are no longer needed.

https://favro.com/card/1e11108a2da81e3bd7153a7a/18F-4826

fureigh added 3 commits May 31, 2017 11:54
As we're shutting down East/West, there's no need to keep vestigial
elements for working in that environment.

https://favro.com/card/1e11108a2da81e3bd7153a7a/18F-4826
Removing more East-West-specific configuration.
 * Delete `eastwest` manifests
 * Move `govcloud` manifests to the main `manifests` directory
 * Remove env-specific manifest directory switcher from circle_deploy.sh

https://favro.com/card/1e11108a2da81e3bd7153a7a/18F-4826
@fureigh fureigh requested review from msecret and jcscottiii May 31, 2017 19:31
@jcscottiii jcscottiii merged commit a1d8b50 into master Jun 1, 2017
@jcscottiii jcscottiii deleted the change-ci-base-to-govcloud branch June 15, 2017 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants