Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add organisation to tw runs info/dump #375

Closed
adamrtalbot opened this issue Dec 19, 2023 · 2 comments · Fixed by #419
Closed

Add organisation to tw runs info/dump #375

adamrtalbot opened this issue Dec 19, 2023 · 2 comments · Fixed by #419
Assignees
Milestone

Comments

@adamrtalbot
Copy link
Contributor

After adding metadata associated with a run in #364 (issue #361), we're still missing one useful field, organization! Without it workspaces with similar names are not easy to distinguish:

tw -o json runs dump -i 2Jv2m4138rraSF -w 138659136604200 -o 2Jv2m4138rraSF.tar.gz

Contents of workflow-metadata.json in tw runs dump:

{
  "pipelineId" : 173625185910842,
  "workspaceId" : 138659136604200,
  "workspaceName" : "showcase",
  "userId" : 8483,
  "runUrl" : "https://tower.nf/orgs/seqeralabs/workspaces/showcase/watch/2Jv2m4138rraSF"
}

Which showcase is it? Adding an organization to one or both of these would make it easier:

{
  "pipelineId" : 173625185910842,
  "workspaceId" : 138659136604200,
  "workspaceName" : "showcase",
  "organizationName": "seqera",
  "userId" : 8483,
  "runUrl" : "https://tower.nf/orgs/seqeralabs/workspaces/showcase/watch/2Jv2m4138rraSF"
}
@drpatelh
Copy link
Member

Agree! We now need a "organizationId" and "organizationName" field in workflow-metadata.json

@ewels
Copy link
Member

ewels commented Dec 19, 2023

It's all in the URL already 😉

\/orgs\/([^\/]+)\/workspaces\/([^\/]+)\/watch\/([^\/]+)\/?$

But yeah, having it as a field would of course be nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants