Page MenuHomePhabricator

Replace using certificates with tokens when using its-phabricator
Closed, ResolvedPublic

Description

its-phabricator needs to be converted from using certificates to using tokens, as certificates is deprecated and really confusing.

See https://secure.phabricator.com/T11856

this makes it harder to test the plugin as i keep forgetting how to get a certificates as there is really no docs on how to get it.

Event Timeline

Paladox triaged this task as High priority.Oct 17 2017, 1:17 PM

Setting high priority due to the depreciation and the confusing behaviour.

Please do not change the priority as this is important. This change should be done for gerrit 2.14+ and not for 2.13.

I am not sure about keeping support for certificates.

Change 384901 had a related patch set (by Paladox) published:
[operations/puppet@production] Gerrit: Replace certificates with tokens for its-phabricator

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

Change 384902 merged by Chad:
[labs/private@master] Gerrit: Replace certificates with tokens for its-phabricator

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

Change 384901 merged by Dzahn:
[operations/puppet@production] Gerrit: Replace certificates with tokens for its-phabricator

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

@Dzahn this will be resolved once we are on gerrit 2.14 :)

Dzahn lowered the priority of this task from High to Medium.Nov 3 2017, 2:35 PM

fair enough, setting to Normal to match the priority of the 2.14 upgrade task

@demon this needs to be done at the same time of the upgrade (replace certificates in the private repo or where you had it :))

We just need to remove the cert param and then we can close this as resolved :).