Skip to content

Feat: Adds interactive session status command#301

Merged
l-mansouri merged 45 commits intomainfrom
ia-status
Mar 24, 2026
Merged

Feat: Adds interactive session status command#301
l-mansouri merged 45 commits intomainfrom
ia-status

Conversation

@l-mansouri
Copy link
Copy Markdown
Collaborator

@l-mansouri l-mansouri commented Mar 19, 2026

Overview

This PR implements the command to check the status of the interactive session .

JIRA

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

LP-105732

Changes

  • Implements the status command
  • Refactors the documentation

Acceptance Criteria

Please add here as many scenarios as in the Story

TESTED IN DEV!

Scenario 1: Check the status of a running IA session

cloudos interactive-session status --session-id <ID>

Screenshot 2026-03-19 at 15 12 52
Scenario 2: Check the status of a paused IA session

cloudos interactive-session status --session-id <ID>

Screenshot 2026-03-19 at 15 13 35
Scenario 3: Check status with an invalid or non-existent session ID

cloudos interactive-session status --session-id 123456789009876543211234

Screenshot 2026-03-19 at 16 08 25
Scenario 4: Watch mode for pre-running session tracks status changes

cloudos interactive-session status --session-id <ID> --watch

Screenshot 2026-03-19 at 15 10 13 Screenshot 2026-03-19 at 15 10 26
Scenario 5: Watch mode shows warning for already running or stopped session

cloudos interactive-session status --session-id <ID> --watch

Screenshot 2026-03-19 at 15 10 55
Scenario 6: Save session status to JSON file

cloudos interactive-session status --session-id <ID> --format json

Screenshot 2026-03-19 at 15 52 31
Scenario 7: Save session status to CSV file with custom base name

cloudos interactive-session status --session-id <ID> --format csv

Screenshot 2026-03-19 at 15 52 56

CI for interactive session status in PROD will fail as the endpoint is currently only open in DEV

@l-mansouri l-mansouri marked this pull request as draft March 19, 2026 15:35
@l-mansouri l-mansouri marked this pull request as ready for review March 19, 2026 15:35
@l-mansouri l-mansouri marked this pull request as draft March 19, 2026 15:48
@l-mansouri l-mansouri marked this pull request as ready for review March 19, 2026 15:48
@l-mansouri l-mansouri marked this pull request as draft March 20, 2026 12:09
@l-mansouri l-mansouri marked this pull request as ready for review March 20, 2026 12:09
@l-mansouri l-mansouri marked this pull request as draft March 20, 2026 16:02
@l-mansouri l-mansouri marked this pull request as ready for review March 20, 2026 16:03
Copy link
Copy Markdown
Collaborator

@danielboloc danielboloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@l-mansouri I tried to tested in my local setup, and I can run list but not status:

I cannot check the status of my interactive session, I can see the ID in the list, but the status I get error:

Image

I cannot test the other scenarios, do you know what could be happening?

Copy link
Copy Markdown
Collaborator

@danielboloc danielboloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@l-mansouri some comments:

  1. is the link correct? it points me to a general page, not a specific IA session
Image Image
  1. Could we change --format to be consistent with the other options like --output-format?

@l-mansouri
Copy link
Copy Markdown
Collaborator Author

Thanks for spotting all these @danielboloc! Updated.

@l-mansouri l-mansouri marked this pull request as draft March 23, 2026 16:57
@l-mansouri l-mansouri marked this pull request as ready for review March 23, 2026 16:57
Copy link
Copy Markdown
Collaborator

@danielboloc danielboloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested again all the scenarios and LGTM

@l-mansouri l-mansouri merged commit f49adfe into main Mar 24, 2026
335 of 366 checks passed
@l-mansouri l-mansouri deleted the ia-status branch March 24, 2026 11:07
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.

2 participants