Page MenuHomePhabricator

scap leaving /srv/patches in a mess
Closed, ResolvedPublic

Description

It seems a mix of scap prep and scap clean aren't necessarily dealing with /srv/patches correctly

Since 1.33.0-wmf.21 to 1.34.0-wmf.3 were not added, nor committed...

1.33.0-wmf.9 to `1.33.0-wmf.16 files and dirs were deleted, but not committed

Example (simplified and redacted):

reedy@deploy1001:/srv/patches$ git status
On branch master
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	deleted:    1.33.0-wmf.12/core/01-REDACTED.patch
	deleted:    1.33.0-wmf.13/core/01-REDACTED.patch
	deleted:    1.33.0-wmf.14/core/01-REDACTED.patch
	deleted:    1.33.0-wmf.16/core/01-REDACTED.patch
	deleted:    1.33.0-wmf.9/core/01-REDACTED.patch

Untracked files:
  (use "git add <file>..." to include in what will be committed)

	1.33.0-wmf.21/
	1.33.0-wmf.22/
	1.33.0-wmf.23/
	1.33.0-wmf.24/
	1.33.0-wmf.25/
	1.34.0-wmf.1/
	1.34.0-wmf.3/

no changes added to commit (use "git add" and/or "git commit -a")

Event Timeline

greg triaged this task as Medium priority.Jul 23 2019, 8:49 PM
greg moved this task from Needs triage to Debt on the Scap board.

afaik scap patch doesn't automatically manage the git repo in /srv/patches, though it probably should.

Change 526509 had a related patch set uploaded (by Thcipriani; owner: Thcipriani):
[operations/mediawiki-config@master] scap: prep and clean git ops for /srv/patches

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

Change 526509 merged by jenkins-bot:
[operations/mediawiki-config@master] scap: prep and clean git ops for /srv/patches

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

Mentioned in SAL (#wikimedia-operations) [2019-12-03T19:07:02Z] <thcipriani@deploy1001> Synchronized scap/plugins: [[gerrit:526509|scap: prep and clean git ops for /srv/patches]] T222240 (no-op sync) (duration: 01m 01s)

thcipriani claimed this task.
thcipriani subscribed.

I manually cleaned-up the repo just now. I also added some git commands to scap clean and scap prep so that when we remove or add a wikiversion in /srv/patches the git repo will be cleaned appropriately.