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

GCE ecdsa ssh keys #6482

Merged
merged 2 commits into from
Feb 25, 2016
Merged

GCE ecdsa ssh keys #6482

merged 2 commits into from
Feb 25, 2016

Conversation

agrare
Copy link
Member

@agrare agrare commented Feb 2, 2016

GCE sometimes adds ecdsa nist keys that are unsupported by the sshkey gem, this skips over them so inventory refresh can continue.

/cc @enoodle can you confirm this fixes your issue?

@agrare
Copy link
Member Author

agrare commented Feb 2, 2016

@miq-bot add-label bug, providers/google

@enoodle
Copy link

enoodle commented Feb 3, 2016

@agrare Thanks! It works well for me. 👍

@agrare
Copy link
Member Author

agrare commented Feb 3, 2016

ping @blomquisg
Will look into if there is anything we can do to support these types of keys, here is the gem issue bensie/sshkey#15

:fingerprint => fingerprint
}
rescue => err
_log.warn { err.backtrace.join('\n') }
Copy link
Member

Choose a reason for hiding this comment

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

Do you want to log a message with it as well?

@chessbyte
Copy link
Member

@blomquisg Please review

@miq-bot
Copy link
Member

miq-bot commented Feb 18, 2016

Checked commits agrare/manageiq@a612c27~...4f8be1e with ruby 2.2.3, rubocop 0.34.2, and haml-lint 0.13.0
2 files checked, 0 offenses detected
Everything looks good. 🍪

chessbyte added a commit that referenced this pull request Feb 25, 2016
@chessbyte chessbyte merged commit 92091a3 into ManageIQ:master Feb 25, 2016
@chessbyte chessbyte added this to the Sprint 37 Ending Mar 7, 2016 milestone Feb 25, 2016
@agrare agrare deleted the gce_ecdsa_ssh_keys branch February 25, 2016 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants