Page MenuHomePhabricator

Implement pipeline config validation
Open, NormalPublic

Description

In its current implementation, pipelinelib will likely throw obscure errors if .pipeline/config.yaml contains incorrect structure or values. We should implement a basic validation system that can throw human understandable errors congruent with existing PipelineStage documentation and, even better, user-facing documentation once that is written.

Event Timeline

dduvall created this task.Jun 8 2019, 12:14 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 8 2019, 12:14 AM
brennen claimed this task.Jun 18 2019, 4:59 PM
brennen triaged this task as Normal priority.
brennen added subscribers: thcipriani, brennen.

Planning to pair on this with @thcipriani this coming Friday.

Change 518333 had a related patch set uploaded (by Brennen Bearnes; owner: Brennen Bearnes):
[integration/pipelinelib@master] config: Add basic validation steps

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

518333 checks that the configuration contains a top-level pipelines. A next step is turning the stage steps into individual classes with run() and validate() methods.

brennen removed brennen as the assignee of this task.Jul 31 2019, 3:54 PM