-
Notifications
You must be signed in to change notification settings - Fork 86
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
fix: prefer events v1 over v1beta1 #416
Conversation
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com>
Also fixes #414 |
t.Logger.Log("Trying with events eventsv1 API...") | ||
err = t.collectEventsV1(cl, namespace) | ||
t.Logger.Log("Trying with events eventsv1beta1 API...") | ||
err = t.collectEventsBeta1(cl, namespace) | ||
if err != nil { | ||
t.Logger.Log("Trying with events corev1 API...") |
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.
Should the log message be changed?
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.
I changed the log message according to the logic change.
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.
/lgtm
Signed-off-by: Charles-Edouard Brétéché <charles.edouard@nirmata.com> Signed-off-by: Israel Blancas <iblancasa@gmail.com>
What this PR does / why we need it:
This PR gives preference to events v1 over v1beta1 and falls back to v1beta1 if v1 didn't work.
Fixes #373
Fixes #414