Rename role::cache::(text|upload)_haproxy to role::cache::(text|upload)
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Vgutierrez
	Nov 18 2022, 11:18 AM

Description

As a fallout of T290005, cache roles for text and upload got duplicated. Since we've decided that HAProxy should be used as our main TLS termination we should get rid of this duplicates and let role::cache::text and role::cache::upload be the main roles again.

Details

Subject	Repo	Branch	Lines +/-
role::cache: Remove text\|upload_haproxy roles	operations/puppet	production	+14 -62
site: Move cp nodes to role::cache:text\|upload	operations/puppet	production	+12 -24
role::cache_upload: Include profile:c:v:f:upload class	operations/puppet	production	+1 -0
prometheus::ops: Leverage varnish::frontend::text\|upload classes	operations/puppet	production	+8 -43
cache: Create dummy profile class to tell between text/upload nodes	operations/puppet	production	+21 -40
role::cache: Link/copy (text\|upload)_haproxy to base roles	operations/puppet	production	+61 -46

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		Vgutierrez	T290005 Test haproxy as a WMF's CDN TLS terminator with real traffic
		Resolved		Vgutierrez	T323365 Rename role::cache::(text\|upload)_haproxy to role::cache::(text\|upload)

Event Timeline

Vgutierrez created this task.Nov 18 2022, 11:18 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 18 2022, 11:18 AM

Vgutierrez triaged this task as Medium priority.Nov 18 2022, 11:18 AM

Vgutierrez moved this task from Backlog to Traffic team actively servicing on the Traffic board.

Change 849180 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] role::cache: Link/copy (text|upload)_haproxy to base roles

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

gerritbot added a project: Patch-For-Review.Nov 18 2022, 12:02 PM

Change 849180 merged by Vgutierrez:

[operations/puppet@production] role::cache: Link/copy (text|upload)_haproxy to base roles

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

Maintenance_bot removed a project: Patch-For-Review.Nov 21 2022, 10:30 AM

Change 859015 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] site: Move cp nodes to role::cache:text|upload

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

gerritbot added a project: Patch-For-Review.Nov 21 2022, 11:39 AM

Change 859018 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] cache: Create dummy profile class to tell between text/upload nodes

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

Change 859018 merged by Vgutierrez:

[operations/puppet@production] cache: Create dummy profile class to tell between text/upload nodes

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

Change 859029 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] prometheus::ops: Leverage varnish::frontend::text|upload classes

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

Change 859029 merged by Vgutierrez:

[operations/puppet@production] prometheus::ops: Leverage varnish::frontend::text|upload classes

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

Change 859044 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] role::cache_upload: Include profile:c:v:f:upload class

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

Change 859044 merged by Vgutierrez:

[operations/puppet@production] role::cache_upload: Include profile:c:v:f:upload class

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

Mentioned in SAL (#wikimedia-releng) [2022-11-21T14:41:09Z] <vgutierrez> move deployment-cache-(text|upload)07 from role::cache::(text|upload)_haproxy to role::cache::(text|upload) - T323365