Closed tdooner closed 7 years ago
Merging #356 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #356 +/- ##
======================================
Coverage 63.2% 63.2%
======================================
Files 109 109
Lines 4180 4180
Branches 457 457
======================================
Hits 2642 2642
Misses 1538 1538
Impacted Files | Coverage Δ | |
---|---|---|
deploy/chef/cookbook/files/default/credentials.js | 80% <ø> (ø) |
:white_check_mark: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 5e2a640...81001a2. Read the comment docs.
ssl/tls
This adds some configuration for manually provisioning SSL with the Amazon Certificate Manager. It's not as cool as let's encrypt, but it's a hell of a lot easier to administer.
worker process
This creates a second system service,
clientcomm-worker
, that needs to be manually started/enabled on one web machine. This is kind of weird, and ideally would be automated, but I decided to punt on figuring out how to figure out which node is the "first web node" for now.networking gotchas
See the commit message for "Fix AWS VPS subnet[...]" to see the new network layout.