Page MenuHomeDevCentral
Differential D2227

Provide PaaS Docker nginx default vhosts
ClosedPublic
Actions

Authored by dereckson on Feb 18 2020, 21:06.
Tags
None
Referenced Files
F2848229: D2227.id5610.diff
Wed, Apr 24, 18:13
Unknown Object (File)
Wed, Apr 24, 07:32
Unknown Object (File)
Tue, Apr 23, 07:10
Unknown Object (File)
Tue, Apr 23, 03:35
Unknown Object (File)
Tue, Apr 23, 00:22
Unknown Object (File)
Mon, Apr 22, 23:59
Unknown Object (File)
Mon, Apr 22, 23:59
Unknown Object (File)
Sun, Apr 21, 20:02
View All Files
Subscribers
None

Details

Reviewers
dereckson
Maniphest Tasks
T1595: Reprovision Dwellers through Salt
Commits
rOPSa727e397d214: Provide PaaS Docker nginx default vhosts
Summary

Server cover page and domain not found

Ref T1595.

Test Plan

Tested on Dwellers.

Unit tests provided for new node.get_ipv6_list function.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Errors
SeverityLocationCodeMessage
Error_tests/modules/test_node.py:5E501flake8 E501
Error_tests/modules/test_node.py:125E501flake8 E501
Advice_modules/node.py:221F821flake8 F821
Unit
No Test Coverage
Branch
arcpatch-D2227
Build Status
Buildable 3476
Build 3725: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Feb 18 2020, 21:06
dereckson created this revision.
Harbormaster completed remote builds in B3451: Diff 5601.Feb 18 2020, 21:06
dereckson added a parent revision: D2228: Provide PaaS Docker nginx base content.Feb 19 2020, 00:39
dereckson updated this revision to Diff 5610.Feb 19 2020, 02:34

Include vhosts in nginx

Harbormaster completed remote builds in B3457: Diff 5610.Feb 19 2020, 02:34
dereckson planned changes to this revision.Feb 19 2020, 05:20
dereckson added inline comments.
roles/paas-docker/nginx/config.sls
63

[]

dereckson updated this revision to Diff 5639.Feb 20 2020, 03:25

Drop TLS for fallback (we'll add it back later with wildcard domains)

Harbormaster completed remote builds in B3475: Diff 5639.Feb 20 2020, 03:25
dereckson mentioned this in T1599: Install TLS wildcard certificates for nginx fallback vhost.Feb 20 2020, 03:27
dereckson updated this revision to Diff 5640.Feb 20 2020, 05:18

+node.get_ipv6_list to get []

Harbormaster completed remote builds in B3476: Diff 5640.Feb 20 2020, 05:18
dereckson accepted this revision.Feb 20 2020, 05:18
$ ssh dwellers sudo nginx -t
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful
This revision is now accepted and ready to land.Feb 20 2020, 05:18
dereckson edited the test plan for this revision. (Show Details)Feb 20 2020, 05:20
This revision was landed with ongoing or failed builds.Feb 20 2020, 05:21
Closed by commit rOPSa727e397d214: Provide PaaS Docker nginx default vhosts (authored by dereckson). · Explain Why
This revision was automatically updated to reflect the committed changes.
dereckson added a commit: rOPSa727e397d214: Provide PaaS Docker nginx default vhosts.

Revision Contents
Changeset List

PathSize
_modules/
15 lines
_tests/
modules/
12 lines
roles/
paas-docker/
nginx/
22 lines
files/
4 lines
vhosts/
base/
27 lines
58 lines

Diff 5640

View Options

_modules/node.py

Loading...
View Options

_tests/modules/test_node.py

Loading...
View Options

roles/paas-docker/nginx/config.sls

Loading...
View Options

roles/paas-docker/nginx/files/nginx.conf

Loading...
View Options

roles/paas-docker/nginx/files/vhosts/base/fallback.conf

Loading...
View Options

roles/paas-docker/nginx/files/vhosts/base/server.conf

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator