1537990 - Use raw HTTPS connection for simple conduit API, instead of `arc call-conduit`

Status: RESOLVED FIXED

(Conduit :: moz-phab, enhancement, P2)

Description

[Tooru Fujisawa [:arai]]

arc_call_conduit [1] should be easily replaced with simple HTTPS request.
I'd like to do that part as a first step removing arc dependency.

[1] https://github.com/arai-a/review/blob/d255b1cdb46e2472f35fb7b20cf19b6990b05e03/moz-phab#L2147-L2179

Comment 1

[Tooru Fujisawa [:arai]]

WIP patch here
https://github.com/arai-a/review/commit/b58669ba23fe07037e91cf74b8aac6691c03708a

Comment 2

[Tooru Fujisawa [:arai]]

updated to keep using arc_ping for now, in parallel with conduit_ping for raw connection
https://github.com/arai-a/review/commits/raw-conduit

Comment 3

[Piotr Zalewa [:zalun]]

Hi arai, do you want to continue on the development of this patch?

Comment 4

[Tooru Fujisawa [:arai]]

Yes, I forgot to mention that I was testing this patch to post my other patches for a while.
I'll submit the patch shortly.

Comment 5

[Tooru Fujisawa [:arai]]

Attachment: Bug 1537990 - Use raw HTTPS connection for simple Conduit API. r?zalun

Comment 6

[Piotr Zalewa [:zalun]]

Attachment: Bug 1537990 - Fix tests after rebasing r?glob!

After rebasing few of the tests were not working.
We do read more from get_revisions which is now using repo.call_conduit.

Depends on D32125

Comment 7

[Piotr Zalewa [:zalun]]

Attachment: Bug 1537990 - Refactor Conduit API out of the Repository class r=glob!