Page MenuHomePhabricator

Status of Jouncebot and dropping the yamllint Jenkins job
Closed, ResolvedPublic

Description

Hello @yuvipanda, I would like to drop the Jenkins job yamllint (T95890) in favor of having developers write the tests they want to make. Jouncebot currently has the jobs tox-flake8 and yamllint, would need to create a test that lint the yaml file and add a tox job to run it.

Though, Jouncebot does not seem to be active at all and there is not even a component in Phabricator. So maybe we can remove it entirely from CI?

Event Timeline

hashar assigned this task to yuvipanda.
hashar raised the priority of this task from to Needs Triage.
hashar updated the task description. (Show Details)
hashar added subscribers: hashar, Aklapper, yuvipanda.

@greg uses it for deployments I think?

greg set Security to None.
greg added subscribers: bd808, Legoktm.

We do use jouncebot for deploy slot announcements in -operations, and it is used/useful.

@Legoktm and @bd808 (I know I know, don't ping bryan about it) have poked at it/restarted it before (as have I).

Sounds easy enough. Jouncebot isn't getting a lot of commits because it's just that awesome. :)

Change 203984 had a related patch set uploaded (by BryanDavis):
Use tox lint yaml and run flake8

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

Change 204028 had a related patch set uploaded (by Hashar):
Add tox-py27 to wikimedia/bots/jouncebot

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

Change 204028 merged by jenkins-bot:
Add tox-py27 to wikimedia/bots/jouncebot

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

I have added the Jenkins job tox-py27 which fails compiling the lxml python module because of libxml/xmlversion.h: No such file or directory.

Change 204031 had a related patch set uploaded (by Hashar):
contint: libxml2-dev libxslt1-dev

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

Change 203984 merged by jenkins-bot:
Use tox lint yaml and run flake8

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

Change 204033 had a related patch set uploaded (by Hashar):
Remove yamllint from wikimedia/bots/jouncebot

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

Change 204033 merged by jenkins-bot:
Remove yamllint from wikimedia/bots/jouncebot

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

Thanks a ton @bd808 , the patch even provided the foundations to add some more tests as needed.

Change 204031 merged by Alexandros Kosiaris:
contint: libxml2-dev libxslt1-dev

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