Skip to content

[base-vm] Check VM state in ssh_exec_process - #5237

Open
xmkg wants to merge 2 commits into
mainfrom
bugfix/ssh-exec-process-check-vm-state
Open

[base-vm] Check VM state in ssh_exec_process#5237
xmkg wants to merge 2 commits into
mainfrom
bugfix/ssh-exec-process-check-vm-state

Conversation

@xmkg

@xmkg xmkg commented Sep 11, 2026

Copy link
Copy Markdown
Member

The VM might have been shut down while the cached session is alive, and given that the SSH session's established state does not have a timeout, it might wedge the caller forever.

Check if the VM is running before proceeding with the exec.

Related Issue(s)

#5235

The VM might have been shut down while the cached session is alive and
given that the SSH session's established state does not have a timeout
it might wedge the caller fovever.

Check if VM is running before proceeding with the exec.
@xmkg
xmkg requested review from a team and tobe2098 and removed request for a team September 11, 2026 18:52
make_common_file_rollback(head_path,
head_file,
std::to_string(head_snapshot->get_parents_index()) + "\n");
auto head_file_rollback = make_common_file_rollback(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

stray formatting.

Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
@codecov

codecov Bot commented Sep 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.63%. Comparing base (a58dcf2) to head (67ea6d8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5237      +/-   ##
==========================================
+ Coverage   72.62%   72.63%   +0.02%     
==========================================
  Files         337      337              
  Lines       18190    18195       +5     
==========================================
+ Hits        13208    13215       +7     
+ Misses       4982     4980       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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