1443540 - Add autoland support for phabricator-qa-dev and phabricator-qa-stage repositories

Status: RESOLVED FIXED

(Conduit Graveyard :: Transplant, enhancement)

Description

[Israel Madueme [:imadueme]]

Our initial release of lando will only allow landing to these two repos. Transplant support will be needed.

Lando will be passing in the following "tree" name for these repos:

"phabricator-qa-dev": https://hg.mozilla.org/automation/phabricator-qa-dev
"phabricator-qa-stage": https://hg.mozilla.org/automation/phabricator-qa-stage

Comment 1

[:glob ✱]

neither of those trees exist on treestatus.

i'm going to assume you mean repo names (ie. the 'destination' field in the request).

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/hgcustom/version-control-tools/rev/23168c0291f8
autoland: add phabricator-qa-dev and phabricator-qa-stage repos

Comment 3

[:glob ✱]

repos cloned on the autoland server, and ansible updated for the config changes.

i have some other work to do on autoland that also requires a production deployment, this change will ride along with those.  will close this bug when that happens.

Comment 4

[Israel Madueme [:imadueme]]

We do pass this in as the "tree" and the destination is currently set to always be "upstream" (https://github.com/mozilla-conduit/lando-api/blob/master/landoapi/transplant_client.py#L55).

glob: do you think we will run into any problems doing this?

Comment 5

[:glob ✱]

config changes deployed.

(In reply to Israel Madueme [:imadueme] from comment #4)
> We do pass this in as the "tree" and the destination is currently set to
> always be "upstream"
> (https://github.com/mozilla-conduit/lando-api/blob/master/landoapi/
> transplant_client.py#L55).

it took a while to unravel things because the names of these fields are deceptive, and it pans i was wrong in my earlier assessments; sorry about that.

i wrote up my findings at https://docs.google.com/document/d/15mFRONQySDHgOfSf8ieHqbx_b1tLJYANO-mwXrhwGk4


`tree` needs to be the name, and `destination` the _push_ url.
eg.
  "tree": "phabricator-qa-dev",
  "destination": "ssh://hg.mozilla.org/automation/phabricator-qa-dev",

here's a random example pulled from prod:
> {
>   "commit_descriptions": {
>     "699a98d0564c": "Bug 1443419 - Extract gtk30.mo files from all the Ubuntu langpacks and bundle them into the snap. p=kenvandine r=sfraser\n\nMozReview-Commit-ID: JB2PojGgike"
>   },
>   "destination": "ssh://hg.mozilla.org/integration/autoland",
>   "ldap_username": "[e-mail]",
>   "pingback_url": "https://reviewboard.mozilla.org/api/extensions/mozreview.extension.MozReviewExtension/autoland-request-updates/",
>   "push_bookmark": "",
>   "rev": "699a98d0564c8f2a73a0e3687eebcc2885632572",
>   "tree": "gecko"
> }

Comment 6

[:glob ✱]

filed bug 1444026