Page MenuHomePhabricator

Update scap deploy-local args to have more information

Authored by thcipriani on Oct 2 2016, 4:56 PM.
Referenced Files
Unknown Object (File)
Tue, Nov 1, 1:19 PM
Unknown Object (File)
Jul 30 2022, 12:28 PM
Unknown Object (File)
Jun 30 2022, 8:20 PM
Unknown Object (File)
Jun 20 2022, 5:19 AM
Unknown Object (File)
Apr 11 2017, 8:48 AM
Unknown Object (File)
Apr 5 2017, 6:29 AM
Unknown Object (File)
Mar 30 2017, 10:10 AM
Unknown Object (File)
Mar 26 2017, 7:59 AM


Maniphest Tasks
T146602: scap deploy-local should make fewer assumptions about server/directories
Group Reviewers
Patch without arc
git checkout -b D399 && curl -L | git apply

Rather than relying so heavily on the config for scap to understand the
state of the world.

This change will modify the way that deploy-local is invoked for scap.

scap deploy-local [stage] [remote-git-repo] [local-final-path]

For example to run the config_fetch phase of parsoid deployment:

scap deploy-local config_deploy http://deployment.eqiad.wmnet/parsoid/deploy /srv/deployment/parsoid/deploy

This will require puppet changes to be merged at the same time.

Fixes T146602

Diff Detail

Event Timeline

thcipriani retitled this revision from to Update scap deploy-local args to have more information.
thcipriani updated this object.
thcipriani edited the test plan for this revision. (Show Details)
thcipriani added reviewers: mmodell, dduvall, demon.
thcipriani added subscribers: mobrovac, Joe.

Build is unstable

W: scap source: changelog-should-mention-nmu
N: When you NMU a package, that fact should be mentioned on the first line
N: in the changelog entry. Use the words "NMU" or "Non-maintainer upload"
N: (case insensitive).
N: Maybe you didn't intend this upload to be a NMU, in that case, please
N: double-check that the most recent entry in the changelog is
N: byte-for-byte identical to the maintainer or one of the uploaders. If
N: this is a local package (not intended for Debian), you can suppress this
N: warning by putting "local" in the version number or "local package" on
N: the first line of the changelog entry.
N: Refer to Debian Developer's Reference section 5.11.3 (Using the DELAYED/
N: queue) for details.
N: Severity: normal, Certainty: certain
N: Check: nmu, Type: source
W: scap source: source-nmu-has-incorrect-version-number 3.3.0-1+0~20161002165613.133~1.gbp0d76e5
N: A source NMU should have a Debian

Link to build:
See console output for more information:

thcipriani edited edge metadata.
thcipriani updated the revert plan for this revision. (Show Details)
thcipriani removed a subscriber: jenkins.
mmodell edited edge metadata.

One inline nitpick but it's not critical.


Shouldn't these all be required=True

This revision is now accepted and ready to land.Oct 3 2016, 11:13 PM

This still good to land?

In D399#13204, @demon wrote:

This still good to land?

Probably not. I still think this is a good thing to do

scap deploy-local [server] [destination]

makes more sense than the current state of things:

scap deploy-local --repo foo/bar

But I made this patch a while ago and couldn't figure out how to roll it out at the time. The easiest thing may be to lock deployments until scap is updated on all targets. In any event, this patch is old and dusty and will probably break stuff at this point.

Will abandon and revisit later.