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
Check if instance has Ipv4 or IPv6 enabled #128
Conversation
When using '--force-new --smt-ip ...' options, validate that the instance can use the --smt-ip value in order to save debugging or unnecessary errors
- Leverage has_ipv6_access - Refactor has_ipv6_access to accomodate IPv4 check
once approve, I will squash the commits into one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This litters various pieces of knowledge in many places making it difficult to follow the code and the intention. Combined with generic names such as __connection_check
that appears to be implemented for IPv4 use only.
- Less knowledge littering - Check request exception for - established connection - timeout - Handle general request exception
- The message is checked if connection failed, use that and remove the if to set the message
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The state purpose, which I agree with does not match the implementation. If we want an early exit when the user gives us an IP address, then construct an smt object in the cli code and call is_responsive
then get out, it will amount to about a 3 line change.
We want to check if the IP provided (IPv4 or IPv6) is enabled on the client, so they can actually use that IP |
- Rename method to reflect the correct intend - Check if a request could be made, handle behaviour accordingly
- Check socket connection - Restore has_ipv6_access as it was - Remove __connection_check
- When checking if instance can access IPv6 infra server, make sure the instance can handle IPv6 addresses
When using '--force-new --smt-ip ...' options,
validate that the instance can use the --smt-ip value in order to save debugging or unnecessary errors