Page MenuHomePhabricator

Graphoid not working under restbase/api
Closed, ResolvedPublic

Description

Marko, thanks for deploying graphoid restbase, but its not working and produces strange URLs

There is a page with this graph on betacluster:
http://graphoid-beta.wmflabs.org/en.wikipedia.beta.wmflabs.org/v1/png/Wikipedia%3AEdit_filter%2FFalse_positives/220336/503f9a644803897675cf01969ce49644e3eae03a.png

The help page works ok: http://en.wikipedia.beta.wmflabs.org/api/rest_v1/?doc#!/Page_content/page_graph_png__title___revision___graph_id__get
Entering graph's params into the form produces this non-working URL:

http://en.wikipedia.beta.wmflabs.org:80/page/graph/png/Wikipedia%3AEdit_filter%2FFalse_positives/0/503f9a644803897675cf01969ce49644e3eae03a.png   (gives 404)

For comparison, the production URL has this format:
http://www.mediawiki.org/api/rest_v1/page/graph/png/Extension%3AGraph/0/be66c7016b9de3188ef6a585950f10dc83239837.png

If I use production format for betacluster, it times out (shouldn't restbase throw an error?): http://en.wikipedia.beta.wmflabs.org/api/rest_v1/page/graph/png/Wikipedia%3AEdit_filter%2FFalse_positives/0/503f9a644803897675cf01969ce49644e3eae03a.png

Event Timeline

Yurik assigned this task to mobrovac.
Yurik raised the priority of this task from to High.
Yurik updated the task description. (Show Details)
Yurik added projects: Graphoid, Services.
Yurik added subscribers: Yurik, GWicke.

Is this the right host/port (from hieradata/labs/restbase/common.yaml)?

graphoid_host_port: http://10.68.16.145:19000

Change 212743 had a related patch set uploaded (by GWicke):
Fix labs graphoid IP

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

Change 212743 merged by Alexandros Kosiaris:
Fix labs graphoid IP

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

The fixed IP patch has been deployed. However, the deployment-restbase0x hosts keep getting prod's graphoid address, not the beta-cluster one. Need to investigate more.

For the time being, I have manually fixed the IPs and disabled (again) Puppet until further investigations.

Change 213500 had a related patch set uploaded (by Mobrovac):
RESTBase: Set Graphoid address for deployment-prep

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

Change 213500 merged by Alexandros Kosiaris:
RESTBase: Set Graphoid address for deployment-prep

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