Mark Bergsma has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/71607


Change subject: Cleanup
......................................................................

Cleanup

Remove configuration for all old varnish servers

Change-Id: I879b3c1a29a2749734f0aa84418bf4fc55f7122c
---
M manifests/role/cache.pp
M manifests/site.pp
2 files changed, 35 insertions(+), 121 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/07/71607/1

diff --git a/manifests/role/cache.pp b/manifests/role/cache.pp
index e08ba85..ae8ffdd 100644
--- a/manifests/role/cache.pp
+++ b/manifests/role/cache.pp
@@ -123,8 +123,7 @@
                                },
                                "bits" => {
                                        "pmtpa" => ["sq67.wikimedia.org", 
"sq68.wikimedia.org", "sq69.wikimedia.org", "sq70.wikimedia.org"],
-                                       "eqiad" => ["arsenic.wikimedia.org", 
"niobium.wikimedia.org", "strontium.eqiad.wmnet", "palladium.eqiad.wmnet",
-                                               'cp1056.eqiad.wmnet', 
'cp1057.eqiad.wmnet', 'cp1069.eqiad.wmnet', 'cp1070.eqiad.wmnet'],
+                                       "eqiad" => ['cp1056.eqiad.wmnet', 
'cp1057.eqiad.wmnet', 'cp1069.eqiad.wmnet', 'cp1070.eqiad.wmnet'],
                                        "esams" => 
["cp3019.esams.wikimedia.org", "cp3020.esams.wikimedia.org", 
"cp3021.esams.wikimedia.org", "cp3022.esams.wikimedia.org"],
                                },
                                "upload" => {
@@ -156,26 +155,6 @@
                                                'sq85.wikimedia.org',
                                                'sq86.wikimedia.org',
                                        ],
-                                       # FIXME: remove after migration
-                                       "eqiad-old" => [
-                                               'cp1021.eqiad.wmnet',
-                                               'cp1022.eqiad.wmnet',
-                                               'cp1023.eqiad.wmnet',
-                                               'cp1024.eqiad.wmnet',
-                                               'cp1025.eqiad.wmnet',
-                                               'cp1026.eqiad.wmnet',
-                                               'cp1027.eqiad.wmnet',
-                                               'cp1028.eqiad.wmnet',
-                                               'cp1029.eqiad.wmnet',
-                                               'cp1030.eqiad.wmnet',
-                                               'cp1031.eqiad.wmnet',
-                                               'cp1032.eqiad.wmnet',
-                                               'cp1033.eqiad.wmnet',
-                                               'cp1034.eqiad.wmnet',
-                                               'cp1035.eqiad.wmnet',
-                                               'cp1036.eqiad.wmnet',
-                                               'dysprosium.eqiad.wmnet',
-                                       ],
                                        'eqiad' => [
                                                'cp1048.eqiad.wmnet',
                                                'cp1049.eqiad.wmnet',
@@ -199,8 +178,6 @@
                                },
                                "mobile" => {
                                        "pmtpa" => [],
-                                       # FIXME: remove after migration
-                                       "eqiad-old" => ["cp1041.eqiad.wmnet", 
"cp1042.eqiad.wmnet", "cp1043.wikimedia.org", "cp1044.wikimedia.org"],
                                        'eqiad' => ['cp1046.eqiad.wmnet', 
'cp1047.eqiad.wmnet', 'cp1059.eqiad.wmnet', 'cp1060.eqiad.wmnet'],
                                        "esams" => 
['cp3011.esams.wikimedia.org', 'cp3012.esams.wikimedia.org', 
'cp3013.esams.wikimedia.org', 'cp3014.esams.wikimedia.org'],
                                },
@@ -265,7 +242,7 @@
                        },
                        "bits" => {
                                "pmtpa" => [],
-                               "eqiad" => [],
+                               "eqiad" => ['arsenic.wikimedia.org', 
'niobium.wikimedia.org', 'strontium.eqiad.wmnet', 'palladium.eqiad.wmnet'],
                                "esams" => [
                                        "knsq1.esams.wikimedia.org",
                                        "knsq2.esams.wikimedia.org",
@@ -294,7 +271,24 @@
                                        'sq15.wikimedia.org',
                                        'sq47.wikimedia.org',
                                ],
-                               "eqiad" => [],
+                               "eqiad" => [
+                                       'cp1021.eqiad.wmnet',
+                                       'cp1022.eqiad.wmnet',
+                                       'cp1023.eqiad.wmnet',
+                                       'cp1024.eqiad.wmnet',
+                                       'cp1025.eqiad.wmnet',
+                                       'cp1026.eqiad.wmnet',
+                                       'cp1027.eqiad.wmnet',
+                                       'cp1028.eqiad.wmnet',
+                                       'cp1029.eqiad.wmnet',
+                                       'cp1030.eqiad.wmnet',
+                                       'cp1031.eqiad.wmnet',
+                                       'cp1032.eqiad.wmnet',
+                                       'cp1033.eqiad.wmnet',
+                                       'cp1034.eqiad.wmnet',
+                                       'cp1035.eqiad.wmnet',
+                                       'cp1036.eqiad.wmnet',
+                               ],
                                "esams" => [
                                        'knsq8.knams.wikimedia.org',
                                        'knsq9.knams.wikimedia.org',
@@ -308,7 +302,7 @@
                        },
                        "mobile" => {
                                "pmtpa" => [],
-                               "eqiad" => [],
+                               "eqiad" => ['cp1041.eqiad.wmnet', 
'cp1042.eqiad.wmnet', 'cp1043.wikimedia.org', 'cp1044.wikimedia.org'],
                                "esams" => []
                        },
                        "parsoid" => {
@@ -635,11 +629,6 @@
 
                class { "lvs::realserver": realserver_ips => 
$lvs::configuration::lvs_service_ips[$::realm]['upload'][$::site] }
 
-               # FIXME: remove after migration
-               $suffix = $::hostname ? {
-                       /^(dysprosium|cp10[23][0-9])$/ => "-old",
-                       default => "",
-               }
                $varnish_be_directors = {
                        1 => {
                                "backend" => 
$lvs::configuration::lvs_service_ips[$::realm]['swift']['pmtpa'],
@@ -652,12 +641,7 @@
 
                $default_backend = $cluster_tier ? { 1 => 'backend', default => 
'eqiad' }
 
-               # FIXME: remove after migration
-               if $::hostname =~ /^cp10[23][0-9]$/ {
-                       $storage_size_main = 100
-                       $storage_size_bigobj = 10
-               }
-               elsif $::hostname =~ /^cp30[0-9][0-9]$/ {
+               if $::hostname =~ /^cp30[0-9][0-9]$/ {
                        $storage_size_main = 300
                }
                else {
@@ -673,13 +657,8 @@
                include standard,
                        nrpe
 
-               # FIXME: set to default on new servers
                $storage_partitions = $::realm ? {
-                       'production' =>
-                               $::hostname ? {
-                                       'dysprosium' => ['sdc1', 'sdd1'],
-                                       default => ['sda3', 'sdb3'],
-                               },
+                       'production' => ['sda3', 'sdb3'],
                        'labs' => ['vdb']
                }
                varnish::setup_filesystem{ $storage_partitions:
@@ -699,8 +678,7 @@
                        },
                        # FIXME: remove after migration
                        storage => $::hostname ? {
-                               'dysprosium' => "-s 
main1=persistent,/srv/sdc1/varnish.persist,300G -s 
main2=file,/srv/sdd1/varnish.persist,300G -s 
bigobj1=file,/srv/sdc1/large-objects.persist,50G -s 
bigobj2=file,/srv/sdd1/large-objects.persist,50G",
-                               /^cp(10[23][0-9]|30[01][0-9])$/ => "-s 
main1=persistent,/srv/sda3/varnish.persist,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.persist,${storage_size_main}G -s 
bigobj1=file,/srv/sda3/large-objects.persist,${storage_size_bigobj}G -s 
bigobj2=file,/srv/sdb3/large-objects.persist,${storage_size_bigobj}G",
+                               /^cp30[01][0-9]$/ => "-s 
main1=persistent,/srv/sda3/varnish.persist,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.persist,${storage_size_main}G -s 
bigobj1=file,/srv/sda3/large-objects.persist,${storage_size_bigobj}G -s 
bigobj2=file,/srv/sdb3/large-objects.persist,${storage_size_bigobj}G",
                                default => "-s 
main1=persistent,/srv/sda3/varnish.main1,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.main2,${storage_size_main}G -s 
bigobj1=file,/srv/sda3/varnish.bigobj1,${storage_size_bigobj}G -s 
bigobj2=file,/srv/sdb3/varnish.bigobj2,${storage_size_bigobj}G",
                        },
                        directors => $varnish_be_directors[$cluster_tier],
@@ -720,17 +698,6 @@
                                'layer' => 'backend',
                        },
                        backend_options => [
-                               # FIXME: remove after migration
-                               {
-                                       'backend_match' => 
'^dysprosium\.eqiad\.wmnet$',
-                                       'weight' => 80,
-                                       #'port' => 3128,
-                                       #'probe' => "varnish",
-                               },
-                               {
-                                       'backend_match' => 
'^cp10[23][0-9]\.eqiad\.wmnet$',
-                                       'weight' => 20,
-                               },
                                {
                                        'backend_match' => 
'^cp[0-9]+\.eqiad.wmnet$',
                                        'port' => 3128,
@@ -754,7 +721,7 @@
                        port => 80,
                        admin_port => 6082,
                        storage => "-s malloc,${memory_storage_size}G",
-                       directors => { "backend" => 
$role::cache::configuration::active_nodes[$::realm]['upload']["${::site}${suffix}"]
 },
+                       directors => { "backend" => 
$role::cache::configuration::active_nodes[$::realm]['upload'][$::site] },
                        director_type => "chash",
                        vcl_config => {
                                'retry5xx' => 0,
@@ -764,14 +731,6 @@
                                'layer' => 'frontend',
                        },
                        backend_options => [
-                               {
-                                       'backend_match' => 
'^dysprosium\.eqiad\.wmnet$',
-                                       'weight' => 80,
-                               },
-                               {
-                                       'backend_match' => 
'^cp10[23][0-9]\.eqiad\.wmnet$',
-                                       'weight' => 20,
-                               },
                                {
                                        'port' => 3128,
                                        'connect_timeout' => "5s",
@@ -915,10 +874,7 @@
                        }
                }
 
-               $storage_size_main = $::hostname ? {
-                       /^cp104[1-4]$/ => 100,
-                       default => 300,
-               }
+               $storage_size_main = 300,
                
                if $cluster_tier == 1 {
                        $director_retries = 2
@@ -930,26 +886,12 @@
                        before => Varnish::Instance["mobile-backend"]
                }
 
-               # TODO: remove after migration
-               if $::hostname =~ /^cp104[1-4]$/ {
-                       class { "varnish::htcppurger": varnish_instances => [ 
"127.0.0.1:80", "127.0.0.1:81" ] }
-               } else {
-                       class { "varnish::htcppurger": varnish_instances => [ 
"127.0.0.1:80", "127.0.0.1:3128" ] }
-               }
-               # FIXME: remove after migration
-               $suffix = $::hostname ? {
-                       /^cp104[1-4]$/ => "-old",
-                       default => "",
-               }
+               class { "varnish::htcppurger": varnish_instances => [ 
"127.0.0.1:80", "127.0.0.1:3128" ] }
 
                varnish::instance { "mobile-backend":
                        name => "",
                        vcl => "mobile-backend",
-                       # TODO: remove after migration
-                       port => $::hostname ? {
-                               /^cp104[1-4]$/ => 81,
-                               default => 3128,
-                       },
+                       port => 3128,
                        admin_port => 6083,
                        storage => $::realm ? {
                                'production' => "-s 
main1=persistent,/srv/sda3/varnish.main1,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.main2,${storage_size_main}G",
@@ -1005,7 +947,7 @@
                        admin_port => 6082,
                        storage => "-s malloc,${memory_storage_size}G",
                        directors => {
-                               "backend" => 
$::role::cache::configuration::active_nodes[$::realm]['mobile']["${::site}${suffix}"],
+                               "backend" => 
$::role::cache::configuration::active_nodes[$::realm]['mobile'][$::site],
                        },
                        director_options => {
                                'retries' => $backend_weight * 
size($::role::cache::configuration::active_nodes[$::realm]['mobile'][$::site]),
@@ -1018,12 +960,6 @@
                                'layer' => 'frontend',
                        },
                        backend_options => [
-                       # TODO: remove after migration
-                       {
-                               'backend_match' => '^cp104[1-4]\.',
-                               'port' => 81,
-                               'weight' => 10,
-                       },
                        {
                                'port' => 3128,
                                'weight' => $backend_weight,
@@ -1054,12 +990,7 @@
 
                $storage_size_main = $::realm ? { 'labs' => 5, default => 300 }
                $storage_partitions = $::realm ? {
-                       'production' => $::hostname ? {
-                               # FIXME: Use consistent partitions on the new 
servers
-                               "titanium" => ["sdb1", "sdd1"],
-                               "cerium" => ["sda1", "sdb1"],
-                               default => ['sda3', 'sdb3'],
-                       },
+                       'production' => ['sda3', 'sdb3'],
                        'labs' => ["vdb"],
                }
                varnish::setup_filesystem{ $storage_partitions:
@@ -1076,12 +1007,7 @@
                        port => 3128,
                        admin_port => 6083,
                        storage => $::realm ? {
-                               # FIXME: Use consistent partitions on the new 
servers
-                               'production' => $::hostname ? {
-                                       "titanium" => "-s 
main1=persistent,/srv/sdb1/varnish.persist,139G -s 
main2=persistent,/srv/sdd1/varnish.persist,139G",
-                                       "cerium" => "-s 
main1=persistent,/srv/sda1/varnish.persist,139G -s 
main2=persistent,/srv/sdb1/varnish.persist,139G",
-                                       default => "-s 
main1=persistent,/srv/sda3/varnish.persist,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.persist,${storage_size_main}G",
-                               },
+                               'production' => "-s 
main1=persistent,/srv/sda3/varnish.persist,${storage_size_main}G -s 
main2=persistent,/srv/sdb3/varnish.persist,${storage_size_main}G",
                                'labs' => "-s 
main1=persistent,/srv/vdb/varnish.main1,${storage_size_main}G -s 
main2=persistent,/srv/vdb/varnish.main2,${storage_size_main}G",
                        },
                        directors => {
diff --git a/manifests/site.pp b/manifests/site.pp
index 1cfb292..748e7d4 100644
--- a/manifests/site.pp
+++ b/manifests/site.pp
@@ -229,10 +229,6 @@
 }
 
 node /(arsenic|niobium|strontium|palladium)\.(wikimedia\.org|eqiad\.wmnet)/ {
-       if $hostname =~ /^(arsenic|niobium)$/ {
-               $ganglia_aggregator = true
-       }
-
        interface_aggregate { "bond0": orig_interface => "eth0", members => [ 
"eth0", "eth1", "eth2", "eth3" ] }
 
        interface_add_ip6_mapped { "main":
@@ -240,7 +236,7 @@
                interface => "bond0"
        }
 
-       include role::cache::bits
+       include standard
 }
 
 node "bast1001.wikimedia.org" {
@@ -332,17 +328,13 @@
 }
 
 node /^cp10(2[1-9]|3[0-6])\.eqiad\.wmnet$/ {
-       if $hostname =~ /^cp102[12]$/ {
-               $ganglia_aggregator = true
-       }
-
        interface_aggregate { "bond0": orig_interface => "eth0", members => [ 
"eth0", "eth1" ] }
 
        interface_add_ip6_mapped { "main":
                require => Interface_aggregate[bond0],
                interface => "bond0"
        }
-       include role::cache::upload
+       include standard
 }
 
 node /^cp10(3[7-9]|40)\.eqiad\.wmnet$/ {
@@ -357,14 +349,9 @@
 
 # eqiad varnish for m.wikipedia.org
 node /^cp104[1-4]\.(wikimedia\.org|eqiad\.wmnet)$/ {
-
-       if $hostname =~ /^cp104(3|4)$/ {
-               $ganglia_aggregator = true
-       }
-
        interface_add_ip6_mapped { "main": }
 
-       include role::cache::mobile
+       include standard
 }
 
 node 'cp1045.eqiad.wmnet', 'cp1058.eqiad.wmnet' {
@@ -410,6 +397,7 @@
                require => Interface_aggregate[bond0],
                interface => "bond0"
        }
+       include standard
 }
 
 node /^cp30(0[3-9]|10)\.esams\.wikimedia\.org$/ {

-- 
To view, visit https://gerrit.wikimedia.org/r/71607
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I879b3c1a29a2749734f0aa84418bf4fc55f7122c
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mark Bergsma <m...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to