Refactor envoy access_log_path to access loggers
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	RLazarus
	Mar 7 2022, 11:02 PM

Description

From the 1.18.0 release notes:

admin: access_log_path is deprecated in favor for access loggers.

I think we're mostly only using access loggers ever since we moved to the v3 config API, but access_log_path still has some hits, which we should convert.

Details

	Subject	Repo	Branch	Lines +/-
	envoyproxy: Migrate from access_log_path field to FileAccessLog API	operations/puppet	production	+16 -5
	Refactor envoy access_log_path to access loggers	operations/deployment-charts	master	+6 -2

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		JMeybohm	T300324 Upgrade Envoy to supported version
		Resolved		JMeybohm	T303231 Refactor envoy access_log_path to access loggers

Event Timeline

RLazarus created this task.Mar 7 2022, 11:02 PM

Restricted Application removed a project: Patch-For-Review. · View Herald TranscriptMar 7 2022, 11:02 PM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

RLazarus triaged this task as Medium priority.Mar 7 2022, 11:04 PM

Change 769807 had a related patch set uploaded (by RLazarus; author: RLazarus):

[operations/puppet@production] envoyproxy: Migrate from access_log_path field to FileAccessLog API

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

gerritbot added a project: Patch-For-Review.Mar 11 2022, 1:15 AM

RLazarus updated the task description. (Show Details)Mar 11 2022, 6:01 PM

jijiki moved this task from Incoming 🐫 to 🙈🙉🙊Backlog on the serviceops board.Sep 28 2022, 2:19 PM

BCornwall removed a project: SRE.May 1 2023, 9:31 PM

BCornwall moved this task from Backlog to Radar/Not for service by Traffic on the Traffic board.

JMeybohm mentioned this in T300324: Upgrade Envoy to supported version.May 3 2023, 1:19 PM

JMeybohm claimed this task.May 3 2023, 1:45 PM

JMeybohm moved this task from 🙈🙉🙊Backlog to Doing 😎 on the serviceops board.

Change 915458 had a related patch set uploaded (by JMeybohm; author: JMeybohm):

[operations/deployment-charts@master] Refactor envoy access_log_path to access loggers

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

Change 915458 merged by jenkins-bot:

[operations/deployment-charts@master] Refactor envoy access_log_path to access loggers

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

Change 769807 merged by JMeybohm: