Paths

Table of Contentst

phabricator.lister: Use credentials setup from config file
ClosedPublic
Actions

Authored by ardumont on Jun 14 2019, 6:23 PM.

Details

Reviewers

anlambert
olasd
douardda

Group Reviewers

Reviewers

Commits

rDLS6d117059082f: phabricator.lister: Use credentials setup from configuration file
rDLSecdce4b0cc85: gitlab.lister: Remove request_params method override

Summary

Prior to this commit, this expected the api.token to be provided at task
initialization. That behavior has been kept for cli purposes. It's no good for
production purposes though (as this leaks the credentials in the scheduler db).

So now, the credentials is fetched from the lister's configuration file as the
other listers do.

Another change is the authentication mechanism which is slighly different. It's
not using a basic auth mechanism. It's expecting an api.token query
parameter so the request_params is overriden to provide that.

Related T1809

Annex:
gitlab.lister: Remove request_params method override (as the
default implementation is generic enough for most listers since
b816212). This should have been removed in that same commit.

Related b816212
Related T1772

Test Plan

tox

docker-dev ~> listed our forge with swhbot's credential

Diff Detail

Repository

rDLS Listers

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Jun 14 2019, 6:23 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJun 14 2019, 6:23 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tox/190/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tox/190/console

Harbormaster failed remote builds in B6229: Diff 5253!Jun 14 2019, 6:24 PM

ardumont planned changes to this revision.Jun 14 2019, 6:24 PM

ardumont retitled this revision from gitlab.lister: Remove request_params method override to wip: phabricator.lister: Fallback credentials setup from config file.

ardumont edited the summary of this revision. (Show Details)

ardumont edited the test plan for this revision. (Show Details)

Fix pep8 violations

ardumont planned changes to this revision.Jun 14 2019, 6:29 PM

ardumont added a project: Lister.

ardumont mentioned this in T1809: Deploy phabricator lister on swh forge.

Build is green
See https://jenkins.softwareheritage.org/job/DLS/job/tox/191/ for more details.

Harbormaster completed remote builds in B6230: Diff 5254.Jun 14 2019, 6:33 PM

And no, too optimistic:

docker-dev:

(swh) tony@corellia $ swh scheduler --url $SCHEDULER_API_URL task add list-phabricator-full --policy recurring
Created 1 tasks

Task 1
  Next run: just now (2019-06-17 12:53:50+00:00)
  Interval: 90 days, 0:00:00
  Type: list-phabricator-full
  Policy: recurring
  Args:
  Keyword args:

doco logs -d:

(swh) tony@corellia $ ...
...
swh-lister_1                  | TypeError: request() got an unexpected keyword argument 'api.token'
...

phabricator.lister: Use credentials setup from configuration file

ardumont retitled this revision from wip: phabricator.lister: Fallback credentials setup from config file to wip: phabricator.lister: Use credentials setup from config file.Jun 17 2019, 4:25 PM

ardumont edited the summary of this revision. (Show Details)

ardumont edited the test plan for this revision. (Show Details)

nahimilega added a subscriber: nahimilega.Jun 17 2019, 4:26 PM

Build is green
See https://jenkins.softwareheritage.org/job/DLS/job/tox/196/ for more details.

Harbormaster completed remote builds in B6258: Diff 5280.Jun 17 2019, 4:26 PM

ardumont retitled this revision from wip: phabricator.lister: Use credentials setup from config file to phabricator.lister: Use credentials setup from config file.Jun 17 2019, 5:15 PM

ardumont edited the summary of this revision. (Show Details)Jun 17 2019, 5:20 PM

ardumont added reviewers: anlambert, olasd, douardda.Jun 17 2019, 5:40 PM

ardumont added inline comments.Jun 17 2019, 5:43 PM