Page MenuHomePhabricator

Make instrument/experiment details publicly available via Test Kitchen UI
Closed, ResolvedPublic1 Estimated Story Points

Description

Description

At this time the Test Kitchen UI webapp requires user authentication for everything except the Test Kitchen API endpoints which are publicly available.

@phuedx has suggested why not providing unauthenticated access to the read view for instruments and experiments configurations so that unauthenticated users could take a look at instruments or experiments configuration when following links like https://test-kitchen.wikimedia.org/experiment/sds2-4-11-synth-aa-test. This would bring some benefits:

  • Users wouldn't need to log in when they only want to take a look at any instrument/experiment configuration (just a readonly view)
  • At this time, when following a link like the above one when your session has expired, the login mechanism interrupts the browsing and, once you have logged-in again successfully, you have to click/paste again the link to see the configuration. Making this links publicly available would remove this problem
  • As @mpopov mentioned in that discussion

Worth noting we have links to experiment details in the dashboard and the wikitech page, so curious reader can be a P&T senior leader or a volunteer.

Acceptance criteria

  • Read view for instruments, /instrument/{machine-readable-name} is publicly available only to read the instrument configuration
  • Read view for experiments, /experiment/{machine-readable-name} is publicly available only to read the experiment configuration

Event Timeline

Change #1233696 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.7 release to staging

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

Change #1233696 merged by jenkins-bot:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.7 release to staging

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

This new feature has been already deployed to staging and it's being tested there before deploying to production

Change #1235033 had a related patch set uploaded (by Kareid; author: Kareid):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.7 release to production

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

Change #1235033 merged by jenkins-bot:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.7 release to production

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

Last patch makes registered-experiments also public so public Read view can show when the experiment is registered and also the quick link to its superset dashboard

Change #1236276 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.8 release to staging

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

Change #1236277 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.8 release to production

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

Change #1236276 merged by jenkins-bot:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.8 release to staging

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

The last patch we deployed is not still showing the quick link to the superset dashboard when a logged-out user is viewing an experiment via the public Read View. We need to investigate further how to do it

Change #1236277 abandoned by Santiago Faci:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v.1.1.8 release to production

Reason:

v1.2.0 was already deployed

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

Change #1239307 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v1.2.1 release to staging

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

Change #1239308 had a related patch set uploaded (by Santiago Faci; author: Santiago Faci):

[operations/deployment-charts@master] Test Kitchen UI: Deploy v1.2.1 release to production

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

Change #1239307 merged by jenkins-bot:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v1.2.1 release to staging

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

Change #1239308 merged by jenkins-bot:

[operations/deployment-charts@master] Test Kitchen UI: Deploy v1.2.1 release to production

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