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.

Sign up for GitHub

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

Jump to bottom

consolidate the multiple terraform deployments at src/core/* into a single deployment at src/terraform #268

Merged
merged 31 commits into from
Jun 29, 2021
Merged

consolidate the multiple terraform deployments at src/core/* into a single deployment at src/terraform #268

merged 31 commits into from
Jun 29, 2021

Conversation

glennmusa
Copy link
Contributor

Description

Moves the multiple src/core/* configurations into a single terraform configuration at src/terraform

Issue reference

The issue this PR will close: #228, close #128

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles or validates correctly
  • BASH scripts have been validated using shellcheck
  • All tests pass (manual and automated)
  • The documentation is updated to cover any new or changed features
  • Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • Relevant issues are linked to this PR

Bree Stryker and others added 30 commits June 8, 2021 09:42
Prelim movements and cleanup.
ed47498
Large conglomerate template in place.
987a1fb
Manually working single deployment.
699b3c2
@glennmusa
hub and spoke working
f50e212
@glennmusa
add remote access
5015de6
@glennmusa
add a vars sample for local deployment
181a8af
@glennmusa
remove core
fde53dd
@glennmusa
support multiple subs and tags
b955c7b
@glennmusa
working apply
d39599a
@glennmusa
working destroy and clean
4017623
@glennmusa
add minimum vars sample
f9622e4
@glennmusa
peering to mlz because dynamic providers
b1f22aa
@glennmusa
update docs
7291d89
@glennmusa
remove frontend references
a2aac25
Fork from new automation for testing, build in outputs, tier3 templat…
0d50cae 
…es started
Untested, needs debugging and additional outputs.
96aa299
Initial post testing.
a32ffcf
Documentation Update. Minor edits.
39eb237
Doc and minor fixes.
9c6d842
@glennmusa
Merge branch 'main' into singledeployment_deconflict
4ad6263
@glennmusa
set keyvault properties
87019d6
Adjust docs, adjust templates, and code
798a11b
@glennmusa
workaround failing az ad sp show queries
7603428
@glennmusa
bump to 2.63
fd29856
@glennmusa
fix merges
1f1b070
@glennmusa
Merge branch 'singledeployment_addtier3' into singledeployment_deconf…
d2e80c3 
…lict
@glennmusa
catch flags and notset params
b734847
@glennmusa
use tfvars.json
483a926
@glennmusa
remove dead code
72c3b50
@glennmusa
Merge branch 'glennmusa/removefrontend' into singledeployment
1dbc9f3
@glennmusa glennmusa mentioned this pull request Jun 29, 2021
Closed
6 tasks
brooke-hamilton
brooke-hamilton approved these changes Jun 29, 2021
View reviewed changes
Copy link
Contributor

@brooke-hamilton brooke-hamilton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deployed to commercial and us gov.

@glennmusa glennmusa merged commit 4228b11 into main Jun 29, 2021
@glennmusa glennmusa deleted the singledeployment branch June 29, 2021 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brooke-hamilton brooke-hamilton brooke-hamilton approved these changes

@Breanna-Stryker Breanna-Stryker Awaiting requested review from Breanna-Stryker

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@glennmusa @brooke-hamilton