Add CI to homer-deploy repo
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• ayounsi
	Mar 15 2021, 9:16 AM

Description

At least to check wmf-netbox.py against our best practices rules.

Details

	Subject	Repo	Branch	Lines +/-
	Zuul: [operations/software/homer/deploy] Add CI	integration/config	master	+4 -0
	Add tox and fix reported issues	operations/software/homer/deploy	master	+35 -8

Customize query in gerrit

Related Objects

Mentioned In: rOSHP16629933edaa: Add tox and fix reported issues

Event Timeline

• ayounsi triaged this task as Medium priority.Mar 15 2021, 9:16 AM

• ayounsi created this task.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 15 2021, 9:16 AM

Aklapper added a project: Infrastructure-Foundations.Jun 21 2021, 8:59 PM

• ayounsi merged a task: T323274: Add basic CI to operations/software/homer/deploy repo.Dec 2 2022, 11:04 AM

• ayounsi added a subscriber: cmooney.

Change 888168 had a related patch set uploaded (by Ayounsi; author: Ayounsi):

[operations/software/homer/deploy@master] Add tox and fix reported issues

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

gerritbot added a project: Patch-For-Review.Feb 10 2023, 8:56 AM

With the patch above, running tox in the root of this repo runs basic checks. So we can at least manually run it for now. Not sure how to have it automated by CI.

In T277440#8603799, @ayounsi wrote:

With the patch above, running tox in the root of this repo runs basic checks. So we can at least manually run it for now. Not sure how to have it automated by CI.

I've replied on the CR for this too, basically send a patch like this one (to be merged by RelEng).

Change 888171 had a related patch set uploaded (by Ayounsi; author: Ayounsi):

[integration/config@master] Add CI to homer/deploy

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

Change 888168 merged by Ayounsi:

[operations/software/homer/deploy@master] Add tox and fix reported issues

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

• ayounsi mentioned this in rOSHP16629933edaa: Add tox and fix reported issues.Feb 10 2023, 9:30 AM

Change 888171 merged by jenkins-bot:

[integration/config@master] Zuul: [operations/software/homer/deploy] Add CI

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

Mentioned in SAL (#wikimedia-releng) [2023-02-10T21:52:25Z] <James_F> Zuul: [operations/software/homer/deploy] Add CI for T277440