Page MenuHomePhabricator

Reconcile "Scap: scap_source correct gid" Beta Cluster hot patch
Closed, ResolvedPublic

Description

Patch https://gerrit.wikimedia.org/r/c/operations/puppet/+/361796 has been hot patched in the Beta Cluster in 2017, and routinely rebased ever since.

Six years later, it seems we still don't have a solution for Scap's UID in Beta.

Is the patch still needed?

Is this easier to solve in 2023, in a way that works for both prod and beta alike?

Or could we update the patch to recognise the realm difference and commit it as normal?

Event Timeline

Change 361796 had a related patch set uploaded (by Ahmon Dancy; author: Thcipriani):

[operations/puppet@production] Scap: scap_source Use the "group" consistently

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

@Krinkle: I think we should merge https://gerrit.wikimedia.org/r/361796 as-is. I update the commit message today to hopefully make it clear why it is a good change.

Change 361796 merged by RLazarus:

[operations/puppet@production] Scap: scap_source Use the "group" consistently

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

Krinkle assigned this task to dancy.

Thanks!