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.

Sign up for GitHub

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

Jump to bottom

check boot of vm with ssh connection #222

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

check boot of vm with ssh connection #222

wants to merge 4 commits into from

Conversation

lukas016
Copy link
Contributor

@lukas016 lukas016 commented Mar 6, 2024

Proposed Changes

Fixes #217

@lukas016 lukas016 added the integration-tests to run integration tests label Mar 6, 2024
@lukas016 lukas016 linked an issue Mar 6, 2024 that may be closed by this pull request
Use Eventually and try to wait until the SSH port of the machine is open #217
Open
@lukas016 lukas016 self-assigned this Mar 6, 2024
@lukas016 lukas016 force-pushed the 217-use-eventually-and-try-to-wait-until-the-ssh-port-of-the-machine-is-open branch from 68fc357 to 0bc3690 Compare March 15, 2024 15:18
@github-actions github-actions bot added size/L and removed size/M labels Mar 15, 2024
@lukas016 lukas016 force-pushed the 217-use-eventually-and-try-to-wait-until-the-ssh-port-of-the-machine-is-open branch 15 times, most recently from 8dc4a00 to af989e4 Compare March 16, 2024 18:04
@lukas016 lukas016 force-pushed the 217-use-eventually-and-try-to-wait-until-the-ssh-port-of-the-machine-is-open branch from e3a4396 to 8bd9470 Compare March 19, 2024 16:03
@lukas016
Copy link
Contributor Author

i did over 300 runs and every run was successful. i couldn't simulate problem with getting private IP. It looks as env problem.

@lukas016 lukas016 marked this pull request as ready for review March 19, 2024 16:13
@lukas016 lukas016 requested a review from a team as a code owner March 19, 2024 16:13
@lukas016 lukas016 added the do-not-merge Do not merge !! label Mar 20, 2024
@lukas016
Copy link
Contributor Author

PR can be merge after PR #245. Some small changes for improve stability from this PR will be implemented there.

@lukas016 lukas016 removed the do-not-merge Do not merge !! label Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@lukas016 lukas016

Labels
integration-tests to run integration tests server size/L
Projects
Compute
Status: Todo
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use Eventually and try to wait until the SSH port of the machine is open
1 participant
@lukas016