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 AWS AMI Exists check to External Checks Registry #643

Merged
merged 1 commit into from
Sep 15, 2020

Conversation

mtougeron
Copy link
Contributor

No description provided.

@joshulyne
Copy link
Collaborator

@mtougeron Thanks so much for contributing this! Tested this locally in our k8s cluster and it worked well!! @jonnydawg looking for your review since you contributed the original ami check but this looks GREAT to me!! 🎉

@joshulyne joshulyne self-requested a review September 15, 2020 19:49
@mtougeron
Copy link
Contributor Author

Tested this locally in our k8s cluster and it worked well!!

That's great! Hopefully this will be useful for folks who are not using kops

@integrii
Copy link
Collaborator

This rocks!

@integrii integrii merged commit b0d0e4f into kuberhealthy:master Sep 15, 2020
@mtougeron mtougeron deleted the ami-exists-check-listing branch September 16, 2020 14:18
@mtougeron
Copy link
Contributor Author

thanks!

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.

None yet

3 participants