-
Notifications
You must be signed in to change notification settings - Fork 287
We have made this change to a fork of this repo in some of the sqsevents (like pkg/monitor/sqsevent/spot-itn-event.go). We found InstanceType useful to have in the Slack notifications along with InstanceID. It didn't take much as were able to update nodeInfo in getNodeInfo() with InstanceType: *instance.InstanceType. Would this be something that could make it through a pull request?
Anyway, here is the PR.
Thanks,
Hari
All reactions
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment