Fix pages deploy
Needs ReviewPublic
Actions

Authored by tusooa on Wed, Oct 29, 8:28 PM.

Details

Reviewers

None

Group Reviewers

Summary

Pages deploy relies on masterdest which is no longer available
in the job_to_steps function after the refactor. This fixes the problem
by moving the pages step into the get_upload_artifacts_jobs function.

Type: fix

Test Plan

Verify you can run a pipeline with pages.

Diff Detail

Repository

rB lilybuild

Branch

servant

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 475
Build 933: Invoke Buildbot	Buildbot build #407
Build 932: arc lint + arc unit

Event Timeline

tusooa created this revision.Wed, Oct 29, 8:28 PM

Owners added a reviewer: O2: lilybuild.Wed, Oct 29, 8:28 PM

Harbormaster completed remote builds in B474: Diff 739.Wed, Oct 29, 8:30 PM

tusooa requested review of this revision.Wed, Oct 29, 8:30 PM

Fix job.is_pages()

Harbormaster completed remote builds in B475: Diff 740.Wed, Oct 29, 9:16 PM

Revision Contents
Changeset List

Path

Size

Packages

lilybuild/

ci_steps.py

40 lines

O2: lilybuild

Diff 740

View Options

lilybuild/lilybuild/ci_steps.py

Fix pages deployNeeds ReviewPublicActions