Skip to content

Feat: Move link implementation to the interactive session module#318

Draft
l-mansouri wants to merge 7 commits into
mainfrom
mv-link
Draft

Feat: Move link implementation to the interactive session module#318
l-mansouri wants to merge 7 commits into
mainfrom
mv-link

Conversation

@l-mansouri
Copy link
Copy Markdown
Collaborator

Overview

This PR moves the link command implementation to the interactive session module and changes the --mountoption of cloudos interactive-session to --link --copy

JIRA

Please add here as many related tasks this PR covers with its brief description, if more than one ticket

https://lifebit.atlassian.net/browse/LP-110779

Changes

  • Moves cloudos link into the interactive-session module as cloudos interactive-session link
  • File Explorer paths now infer the project name from the first path segment (e.g. my-project/Data/folder); standard top-level folder names (Data, AnalysesResults, Analyses_Results, Analyses-Results, Cohorts) are treated as relative to the profile project
  • Removes --mount from cloudos interactive-session create
  • Introduces --copy as an optional flag of --link in cloudos interactive-session create to copy data into the session

Acceptance Criteria

Please add here as many scenarios as in the Story

At the time of opening the PR, this is only testable in DEV as the feature is not released to prod yet and it is not supported in Azure.

Scenario 1: cloudos interactive-session link replaces cloudos link with identical functionality
cloudos interactive-session link --session-id 6a180a8a558610e459defb70 Data/ai_data_test/person.csv,leila-test/Data/MultiQC,Daniel_Test_Files/Data/20131219.populations.tsv,s3://lifebit-featured-datasets/pipelines/VCF-s3table-ingestion/vcf_list.txt --profile internal-DEV
Screenshot 2026-05-28 at 17 24 16 Screenshot 2026-05-28 at 17 24 41 Screenshot 2026-05-28 at 17 45 17 Screenshot 2026-05-28 at 17 45 25
Scenario 2: create --link links a folder or file to the interactive session
cloudos interactive-session create --session-type vscode --name test-link --link Data/ai_data_test/person.csv,leila-test/Data/MultiQC,Daniel_Test_Files/Data/20131219.populations.tsv,s3://lifebit-featured-datasets/pipelines/VCF-s3table-ingestion/vcf_list.txt --profile internal-DEV
Screenshot 2026-05-28 at 17 46 49 Screenshot 2026-05-28 at 17 47 23
Scenario 3: create --link --copy copies data into the interactive session
cloudos interactive-session create --session-type vscode --name test-mount --link Data/ai_data_test/person.csv,leila-test/Data/MultiQC,Daniel_Test_Files/Data/20131219.populations.tsv,s3://lifebit-featured-datasets/pipelines/VCF-s3table-ingestion/vcf_list.txt  --copy --profile internal-DEV
Screenshot 2026-05-28 at 17 46 23 Screenshot 2026-05-28 at 17 47 11
Scenario 4: Files are linkable via cloudos interactive-session link if supported by the DEV endpoint Screenshot 2026-05-28 at 17 45 17 Screenshot 2026-05-28 at 17 45 25
Scenario 5: Detailed API error messages are surfaced to the user on failure
cloudos interactive-session link --session-id 6a180a8a558610e459defb70 Data/ai_data_test/person.csv,leila-test/Data/MultiQC,Daniel_Test_Files/Data/20131219.populations.tsv,s3://lifebit-featured-datasets/pipelines/VCF-s3table-ingestion/vcf_list.txt --profile internal-DEV --api-key not-working-api-key
Screenshot 2026-05-28 at 18 00 10

@l-mansouri l-mansouri marked this pull request as draft May 28, 2026 16:45
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 this pull request may close these issues.

1 participant