Closed Bug 1609987 Opened 4 years ago Closed 4 years ago

Port bug 1568006 - Allow pulling taskgraph code from Thunderbird tree

Categories

(Firefox Build System :: Task Configuration, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla75

People

(Reporter: rjl, Assigned: rjl)

References

Details

Attachments

(5 files)

Extend taskgraph.util.verify.verify_docs functionality to check multiple documentation base paths.

For example, when checking that a Kind name is documented, look for it in taskcluster/docs/kinds.rst as well as comm/taskcluster/docs/kinds.rst.

The end goal is to facilitate creating new task kinds that are specific to Thunderbird that are implemented in Thunderbird's repo rather than Firefox's.

There's no way for Thunderbird to define a new task kind at the moment because
the verify_docs function only checks one base path.

This sets up a DocsVerifier class which is initiated early in the task graph
generation process with a list of documentation base paths that are set in
config.yml.

verify_docs then checks all of the provided paths. As it iterates through the
list of identifiers to look for in a file, it records each one it finds, then
compares the matched list to the original.

Assignee: nobody → rob
Status: NEW → ASSIGNED

Some basic tests for new verify docs functionality.

Attachment #9121592 - Attachment is obsolete: true
Summary: Extend verify_docs functionality to check multiple base paths → Port bug 1568006 - Allow pulling taskgraph code from Thunderbird tree

For mozilla-central, all the code related to taskgraph lives in
taskcluster/taskgraph. Thunderbird's build requirements are evolving, and
we want to be able to have repository-specific code. The natural place for it
to live is an a package beside taskcluster/ci. Add that to python path,
and provide some hooks for adding to the various registries in taskgraph.

Attachment #9121591 - Attachment description: Bug 1609987 - Allow verify_docs to check multiple base paths. r?tomprince → Bug 1609987 - Allow checking additional paths in verify_docs. r?tomprince
Attachment #9121592 - Attachment is obsolete: false
This is the start of being able to extend taskcluster functionality
within c-c easier.
D60540 is landing in M-C soonish and will cause bustage without this.
Attachment #9122792 - Flags: review?(geoff)
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/integration/autoland/rev/756e3187d18e
Port bug 1568006: Add project's taskgraph directory to the python path. r=tomprince
Attachment #9122792 - Flags: review?(geoff) → review+
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/bfcc947899e5
Initial support for extending taskgraph with hooks. r=darktrojan
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---
Target Milestone: mozilla74 → ---
Blocks: 1518166

More ports from external taskgraph project. Short term, this will be used to get partial updates
working for Thunderbird.

Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/integration/autoland/rev/3a5bd821afe2
Port bug 1568006: Allow projects to modify decision parameters. r=tomprince
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/integration/autoland/rev/533f15e7cc13
Allow checking additional paths in verify_docs. r=tomprince
https://hg.mozilla.org/integration/autoland/rev/30f4c91398ed
unittests for taskgraph.util.verify. r=tomprince
Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Keywords: leave-open
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: