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 busy loop to wait for instance to be ssh'able #285

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

jfrancoa
Copy link
Contributor

@jfrancoa jfrancoa commented Dec 10, 2024

We are getting recently errors that the SSH key didn't have
time enought to propage. This PR adds a busy loop to wait for SSH to be ready.

Also, it was missed to add cleanup for AWS in the quantization script variant.

Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@jfrancoa
Copy link
Contributor Author

We are getting recently errors that the SSH key didn't have
time enought to propage. This PR adds a busy loop to wait for
SSH to be ready.

Also, it was missed to add cleanup for AWS in the quantization
script variant.
@jfrancoa jfrancoa force-pushed the jose/clean-gcp-keys branch from 2c5ae5a to 51c9a2f Compare December 10, 2024 13:08
@jfrancoa
Copy link
Contributor Author

This PR messed up with all the SSH keys stored in the project, the good news is that the limit isn't reached anymore but all of them got cleaned up.
There doesn't seem to be an easy way to clean the ssh key created for the instance, so I'm adding a busy loop instead to wait for the VM to be ready.

@jfrancoa
Copy link
Contributor Author

@jfrancoa jfrancoa changed the title Clean ssh keys metadata after instance deletion. Add busy loop to wait for instance to be ssh'able Dec 10, 2024
@jfrancoa jfrancoa merged commit 28763e6 into main Dec 10, 2024
114 checks passed
@jfrancoa jfrancoa deleted the jose/clean-gcp-keys branch December 10, 2024 15:46
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