source-test-doc-generate should flag "x new warnings" line as error on taskcluster/treeherder
Categories
(Developer Infrastructure :: Source Documentation, defect)
Tracking
(firefox121 fixed)
| Tracking | Status | |
|---|---|---|
| firefox121 | --- | fixed |
People
(Reporter: Gijs, Assigned: standard8)
References
Details
Attachments
(2 files)
I recently introduced a new warning in a patch and got backed out. This is the failing job:
https://treeherder.mozilla.org/logviewer?job_id=434078251&repo=autoland&lineNumber=5426
The highlighted test output in taskcluster/treeherder is:
FileNotFoundError: [Errno 2] No such file or directory: '/builds/worker/checkouts/gecko/taskcluster/ci/docker-image/kind.yml'
3804 subprocess.CalledProcessError: Command '['/builds/worker/.mozbuild/srcdirs/gecko-8a5b87fe5d69/_virtualenvs/docs/bin/python', 'doc', '--outdir', 'docs-out', '--no-open', '--no-serve', '--archive', '--fatal-warnings', '--check-num-warnings', '--dump-trees=/builds/worker/artifacts/trees.json']' returned non-zero exit status 2.
3820 NameError: name 'self' is not defined
3840 mach.base.MachError: Mach command 'cmd_foo' implemented incorrectly. Conditions argument must take a list of functions. Found <class 'str'> instead.
3866 KeyError: "'OS_TARGET'"
3887 KeyError: "'BIN_SUFFIX'"
I would argue this does not really provide any context for why the patch was backed out. :-)
It would be useful if at least the "1 new warnings" line was highlighted, and possibly if some of the above things were fixed/hidden.
|
Assignee | |
Comment 1•1 month ago
|
||
I think I may have at least partial fixes for this.
|
|
Assignee | |
Comment 2•1 month ago
|
||
I've not been able to figure out all the lines that are raised, but I figured out the first one which also cleaned up a bunch of warnings.
Example try push of new failure mode: https://treeherder.mozilla.org/jobs?repo=try&revision=7179a13d9c6cba7d36b6f629a4e2a6c7e08f4f61
Try push for success: https://treeherder.mozilla.org/jobs?repo=try&revision=1116a44a2c7266ea70104472f5a552081187a12f
|
|
Assignee | |
Comment 3•1 month ago
|
||
|
|
Assignee | |
Comment 4•1 month ago
|
||
Depends on D193242
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2cbc910428ee Add yaml files to sphinx-docs sparse checkout to avoid file not found issues in source generation. r=sylvestre https://hg.mozilla.org/integration/autoland/rev/0726d7c488d7 Improve mach doc errors so that they show up on treeherder summaries. r=sylvestre
|
||
Comment 6•29 days ago
|
||
| bugherder | ||
Description
•