Andrew Bogott has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/331638 )
Change subject: Use LE for wikitech ...................................................................... Use LE for wikitech Bug: T154913 Change-Id: I325ab9c1f2a56e42f1460435958d27445f5a1a6b --- D files/ssl/labtestwikitech.wikimedia.org.crt D files/ssl/wikitech.wikimedia.org.crt M hieradata/regex.yaml M modules/openstack/manifests/openstack_manager.pp M modules/openstack/templates/common/wikitech.wikimedia.org.erb M modules/role/manifests/labs/openstack/nova/manager.pp 6 files changed, 9 insertions(+), 92 deletions(-) Approvals: Andrew Bogott: Looks good to me, approved jenkins-bot: Verified Dzahn: Looks good to me, but someone else must approve diff --git a/files/ssl/labtestwikitech.wikimedia.org.crt b/files/ssl/labtestwikitech.wikimedia.org.crt deleted file mode 100644 index f6adbbd..0000000 --- a/files/ssl/labtestwikitech.wikimedia.org.crt +++ /dev/null @@ -1,34 +0,0 @@ ------BEGIN CERTIFICATE----- -MIIF1jCCA74CCQDKK3TuPMyhMzANBgkqhkiG9w0BAQsFADCBrDELMAkGA1UEBhMC -VVMxEzARBgNVBAgMCkNhbGlmb3JuaWExFjAUBgNVBAcMDVNhbiBGcmFuY2lzY28x -HTAbBgNVBAoMFFdpa2ltZWRpYSBGb3VuZGF0aW9uMRMwEQYDVQQLDApPcGVyYXRp -b25zMRYwFAYDVQQDDA1BbmRyZXcgQm9nb3R0MSQwIgYJKoZIhvcNAQkBFhVhYm9n -b3R0QHdpa2ltZWRpYS5vcmcwHhcNMTYwMTE2MTQwNTIzWhcNMTcwMTE1MTQwNTIz -WjCBrDELMAkGA1UEBhMCVVMxEzARBgNVBAgMCkNhbGlmb3JuaWExFjAUBgNVBAcM -DVNhbiBGcmFuY2lzY28xHTAbBgNVBAoMFFdpa2ltZWRpYSBGb3VuZGF0aW9uMRMw -EQYDVQQLDApPcGVyYXRpb25zMRYwFAYDVQQDDA1BbmRyZXcgQm9nb3R0MSQwIgYJ -KoZIhvcNAQkBFhVhYm9nb3R0QHdpa2ltZWRpYS5vcmcwggIiMA0GCSqGSIb3DQEB -AQUAA4ICDwAwggIKAoICAQDMOc5OH9PRGSM5AKVL6F7hyNCRYJduvFva367/yAyc -ImCPzyU8yMu/O82LQgLFR8fQS/AEnTSZT0ihIj85PJ+FHRf+gdmqmjOHr7b2bsHm -kb77pkWNvLxkbEM4lJWq+8t5juEGf5d3hf7lIkSjISLz3aI9rQYYWYR2oEi1L9Tg -cRfYQGJ3Zt8Npoi9Rq2JYIyKem+4+akIkvNxDIgxCi3w43U67rQuin39T9WAyhuk -9xBZ74ACtvBBLguavpRIm2zAwm2G/uAacoYpmTB5yDmGTEnemLnc1q8Ko76T9+V2 -LrZ763LfuhsNcJZ3xXKPznqIV0LVtRco3UkuGnecs2MtMzPu4faf3qAyVGes5oae -EiHrP7FQgLiT9YDsPL+7lXUyyKrke3QKkVj2hZJfDg4E1PUrhccwAPLih2Sqjl2y -DuNRq8ldJdXZ3GWmnru+H7UyXd+L07utajAz857No7sBEBSDxHJMqyGmzy9x10PK -itPpAE39u/4hJUHQhTz2S7Ujmo+X+iEeYNp48FapARMjIcoM9h8ayojxu7cEH5GA -EL/NITu1LZW6269KFLTYagXP/pZuQCUrLoIKjPoo0S+EgER9g6HaFb/srecZWaZe -/uEbBAffFjFwFQfxOC//WlyZA8Gz2k+DZ3isDWAJj9rczCN0hAeGALb8nrmiaGpD -dQIDAQABMA0GCSqGSIb3DQEBCwUAA4ICAQC51UH5CP76zYABj2PyZGCzLLBXDZQ1 -AW1b06McCixV8Igr3VZKQYL70Wysc9j3/iKtTVAm1grcgnPGXfB8t0hl5OZd5UVg -/fJCN2DOBfwzLWa/7ApAILDra6ydPVbhM0+f+piqFi9wWShyC2Hi+Ew5CyhHj0P9 -V3BCqIqYQvlBex6jtHgE+FvWQC1p3TEYZ4VeM7FCBZWzVG5NWMfgZMwEnFfTUAMC -F0ZB90FD8/EG+8SGMFqwZ3Ulk+2gt/aH4tYCbaaMJ52hemefeFx5UMRBYoXAkVEQ -IRzKIjksLdzgoQMgQ8xfF9vOhmG5zK2RIbs/ZtqpDVr0OATmuONvR5+v/746/DMX -dyaKPcLziM25TtOtZHCB9Uvt/yJkNzRrSDpQ8RI+Ec5p/H6QcO+58bZWg4/V/hqA -7YX53SEQ/vMPEy7REcOrMpQPiQ/qALvolIHL+Uf6jqiBj3C7tM5HO2S6OXDDWgaN -NPwqj6IkSzeh6THuUlsawb8XuYYzdzie0QzXUJTsUqxgSmmF0k2VVLeW6iAx9ehv -7KMy6oauTbYoP/vx6eSYgZ/Mk73ip5JGu3T6A63+uucO6uWIEgD0+rVegIsL12TB -mdFtZIGoCevVLLkKX4Ps4AvW3HKXRsZBKndrS2vMDimTjgVxVl5v2AV5lGHmMUJI -Sy236b+rdcO+Gg== ------END CERTIFICATE----- diff --git a/files/ssl/wikitech.wikimedia.org.crt b/files/ssl/wikitech.wikimedia.org.crt deleted file mode 100644 index 10abaa1..0000000 --- a/files/ssl/wikitech.wikimedia.org.crt +++ /dev/null @@ -1,31 +0,0 @@ ------BEGIN CERTIFICATE----- -MIIFTzCCBDegAwIBAgISESHOsj6ApadNbvj3xLyEHmFzMA0GCSqGSIb3DQEBCwUA -MGYxCzAJBgNVBAYTAkJFMRkwFwYDVQQKExBHbG9iYWxTaWduIG52LXNhMTwwOgYD -VQQDEzNHbG9iYWxTaWduIE9yZ2FuaXphdGlvbiBWYWxpZGF0aW9uIENBIC0gU0hB -MjU2IC0gRzIwHhcNMTUxMjExMjAyMTExWhcNMTcwMjI0MDQ0NDEzWjCBgDELMAkG -A1UEBhMCVVMxEzARBgNVBAgTCkNhbGlmb3JuaWExFjAUBgNVBAcTDVNhbiBGcmFu -Y2lzY28xIzAhBgNVBAoTGldpa2ltZWRpYSBGb3VuZGF0aW9uLCBJbmMuMR8wHQYD -VQQDExZ3aWtpdGVjaC53aWtpbWVkaWEub3JnMIIBIjANBgkqhkiG9w0BAQEFAAOC -AQ8AMIIBCgKCAQEAtdROlVzflJGw1k+RxVu7HHG8QSpiPLqbardSkB63aVxlHKN7 -26K5sSS2SLYuju4a3nEIErkZBXtAMGfBlBJVFZ4/UZgD4vjaAP5HiPpUKMx4xHz/ -2OOrqF68PXc2+yTTOkEMeniH4XPRmmY/GzIIpmDRzVzlRsVBtEdqvfoyCyeC8zTb -n+bHd6lFGK0R3sRDBO7soJXd58svcBijI1mXpDo3WT4q9O3QDfvoGZO1xK/7Jz8m -Yk2vYt5zG+dFvqjN62fQxD8JFNdargrgfkoZxiBmskii03e6+ymxoNdQUc0T8CkZ -l8D3IeRXb8bsXcG8O+QFiZne+yNc0kL/xwcb0wIDAQABo4IB2jCCAdYwDgYDVR0P -AQH/BAQDAgWgMEkGA1UdIARCMEAwPgYGZ4EMAQICMDQwMgYIKwYBBQUHAgEWJmh0 -dHBzOi8vd3d3Lmdsb2JhbHNpZ24uY29tL3JlcG9zaXRvcnkvMCEGA1UdEQQaMBiC -Fndpa2l0ZWNoLndpa2ltZWRpYS5vcmcwCQYDVR0TBAIwADAdBgNVHSUEFjAUBggr -BgEFBQcDAQYIKwYBBQUHAwIwSQYDVR0fBEIwQDA+oDygOoY4aHR0cDovL2NybC5n -bG9iYWxzaWduLmNvbS9ncy9nc29yZ2FuaXphdGlvbnZhbHNoYTJnMi5jcmwwgaAG -CCsGAQUFBwEBBIGTMIGQME0GCCsGAQUFBzAChkFodHRwOi8vc2VjdXJlLmdsb2Jh -bHNpZ24uY29tL2NhY2VydC9nc29yZ2FuaXphdGlvbnZhbHNoYTJnMnIxLmNydDA/ -BggrBgEFBQcwAYYzaHR0cDovL29jc3AyLmdsb2JhbHNpZ24uY29tL2dzb3JnYW5p -emF0aW9udmFsc2hhMmcyMB0GA1UdDgQWBBT0xfeMunJJ2IWrUAelaPocSGrN6jAf -BgNVHSMEGDAWgBSW3mHxvRwWKVMcwMx9O4MAQOYafDANBgkqhkiG9w0BAQsFAAOC -AQEAMF6mlBiBOfOirbVejNcaOcwxLTBEc/WTFTvk3KDs1369sWWBrvkWebkJNRic -DJ6PAloCwyC7qZUfbYC4g1UWVo0UrFdGOpwK7HcLWzZB6CLpAX+q3lRs8BowAExI -ShT3WdlkG24jdGIKtCYlRAYIxCnayba5Ad/FSaiOXXzIjC998UAMaSST20ONfsEl -mAIbgK33bI69rem7Br977Bj4nFZ4JSsqMJS7GSvmedtF7XtGCFNE0pTh+DEs6lym -iWdS14/ywS15gDEeTsKAfHYzLpc6lawYnm05a7b+a6uuqEBtPZU4XIp3oShTUjtd -eTtgOQNq1NXcwSKTzYAc6qFQxg== ------END CERTIFICATE----- diff --git a/hieradata/regex.yaml b/hieradata/regex.yaml index 531accb..a72204a 100644 --- a/hieradata/regex.yaml +++ b/hieradata/regex.yaml @@ -467,7 +467,6 @@ labs_glance_controller: &labsglancecontroller "labtestcontrol2001.wikimedia.org" labs_puppet_master: &labspuppetmaster "labtestcontrol2001.wikimedia.org" labs_keystone_host: &labskeystonehost "labtestcontrol2001.wikimedia.org" - wikitech_use_letsencrypt: true wikitech_db_name: 'labtestwiki' labsldapconfig: hostname: labtestservices2001.wikimedia.org diff --git a/modules/openstack/manifests/openstack_manager.pp b/modules/openstack/manifests/openstack_manager.pp index e1c32ce..aca4a05 100644 --- a/modules/openstack/manifests/openstack_manager.pp +++ b/modules/openstack/manifests/openstack_manager.pp @@ -59,10 +59,6 @@ 'wikitech.wikimedia.org' => '208.80.154.136', 'labtestwikitech.wikimedia.org' => '208.80.153.14' } - $labtest = $webserver_hostname ? { - 'wikitech.wikimedia.org' => false, - 'labtestwikitech.wikimedia.org' => true - } apache::site { $webserver_hostname: content => template('openstack/common/wikitech.wikimedia.org.erb'), } diff --git a/modules/openstack/templates/common/wikitech.wikimedia.org.erb b/modules/openstack/templates/common/wikitech.wikimedia.org.erb index 29dfec1..56a01a9 100644 --- a/modules/openstack/templates/common/wikitech.wikimedia.org.erb +++ b/modules/openstack/templates/common/wikitech.wikimedia.org.erb @@ -31,10 +31,8 @@ RewriteEngine on RewriteCond %{SERVER_PORT} !^443$ -<% if @labtest -%> Include /etc/acme/challenge-apache.conf RewriteCond %{REQUEST_URI} !^/\.well-known/acme-challenge/ -<% end -%> RewriteRule ^/(.*)$ https://<%= @webserver_hostname %>/$1 [L,R=301] ErrorLog /var/log/apache2/error.log @@ -56,15 +54,9 @@ RewriteEngine on SSLEngine on -<% if @labtest -%> SSLCertificateFile /etc/acme/cert/<%= @certificate %>.crt SSLCertificateChainFile /etc/acme/cert/<%= @certificate %>.chain.crt SSLCertificateKeyFile /etc/acme/key/<%= @certificate %>.key -<% else -%> - SSLCertificateFile /etc/ssl/localcerts/<%= @certificate %>.crt - SSLCertificateChainFile /etc/ssl/localcerts/<%= @certificate %>.chain.crt - SSLCertificateKeyFile /etc/ssl/private/<%= @certificate %>.key -<% end -%> <%= @ssl_settings.join("\n") %> RedirectMatch ^/$ https://<%= @webserver_hostname %>/wiki/ diff --git a/modules/role/manifests/labs/openstack/nova/manager.pp b/modules/role/manifests/labs/openstack/nova/manager.pp index ed0647e..22fccf2 100644 --- a/modules/role/manifests/labs/openstack/nova/manager.pp +++ b/modules/role/manifests/labs/openstack/nova/manager.pp @@ -13,24 +13,19 @@ $novaconfig = $role::labs::openstack::nova::common::novaconfig $sitename = hiera('labs_osm_host') - if hiera('wikitech_use_letsencrypt', false) { - $sitename_split = split($sitename, '\.') - $certificate = $sitename_split[0] - letsencrypt::cert::integrated { $certificate: - subjects => $sitename, - puppet_svc => 'apache2', - system_svc => 'apache2', - } - $cert_type = '_letsencrypt' - } else { - $certificate = $sitename - sslcert::certificate { $sitename: } - $cert_type = '' + $sitename_split = split($sitename, '\.') + $certificate = $sitename_split[0] + letsencrypt::cert::integrated { $certificate: + subjects => $sitename, + puppet_svc => 'apache2', + system_svc => 'apache2', } + + sslcert::certificate { $sitename: ensure => absent } monitoring::service { 'https': description => 'HTTPS', - check_command => "check_ssl_http${cert_type}!${sitename}", + check_command => "check_ssl_http_letsencrypt!${sitename}", } $ssl_settings = ssl_ciphersuite('apache', 'compat', true) -- To view, visit https://gerrit.wikimedia.org/r/331638 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I325ab9c1f2a56e42f1460435958d27445f5a1a6b Gerrit-PatchSet: 4 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Alex Monk <kren...@gmail.com> Gerrit-Reviewer: Andrew Bogott <abog...@wikimedia.org> Gerrit-Reviewer: BBlack <bbl...@wikimedia.org> Gerrit-Reviewer: Dzahn <dz...@wikimedia.org> Gerrit-Reviewer: Ema <e...@wikimedia.org> Gerrit-Reviewer: RobH <r...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits