Project

General

Profile

Actions

Bug #17286

closed

Print workbench URL for submitted CR

Added by Peter Amstutz almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
CWL
Target version:
Story points:
-
Release relationship:
Auto

Description

arvados-cwl-runner logs the UUID of the container request but it would be more convenient if it also printed a clickable URL to workbench so that smart terminals like gnome terminal or vscode terminal can take you directly to the workflow page.


Subtasks 1 (0 open1 closed)

Task #17288: Review 17286-print-urlsResolvedLucas Di Pentima01/22/2021Actions
Actions #1

Updated by Peter Amstutz almost 4 years ago

  • Assigned To set to Peter Amstutz
  • Status changed from New to In Progress
  • Description updated (diff)
Actions #3

Updated by Lucas Di Pentima almost 4 years ago

I've just a couple of concerns:

  • When no workbenches are set up this would output a confusing message.
  • I think this isn't taking into account when the user submits a runner to a federated cluster.

Other than that, it LGTM.

Actions #4

Updated by Peter Amstutz almost 4 years ago

Lucas Di Pentima wrote:

I've just a couple of concerns:

  • When no workbenches are set up this would output a confusing message.

The messages all either fall back or don't print if neither workbench2 or workbench1 is defined.

  • I think this isn't taking into account when the user submits a runner to a federated cluster.

I guess it is technically possible that the user could send the workflow runner to a different cluster, but currently nobody does that. If that was a thing, ideally we would have federation features that let you visit the local workbench and it would fetch the remote container request for you.

Other than that, it LGTM.

Actions #6

Updated by Lucas Di Pentima almost 4 years ago

LGTM, thanks!

Actions #7

Updated by Peter Amstutz almost 4 years ago

  • Status changed from In Progress to Resolved
Actions #8

Updated by Peter Amstutz over 3 years ago

  • Release set to 38
Actions

Also available in: Atom PDF