Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Show node events in node details #285

Merged
merged 1 commit into from
Apr 21, 2020

Conversation

nevalla
Copy link
Contributor

@nevalla nevalla commented Apr 20, 2020

image

Fixes #252

Signed-off-by: Lauri Nevala <lauri.nevala@gmail.com>
@@ -25,6 +25,9 @@ export class EventStore extends KubeObjectStore<KubeEvent> {

getEventsByObject(obj: KubeObject): KubeEvent[] {
return this.items.filter(evt => {
if(obj.kind == "Node") {
return obj.getName() == evt.involvedObject.uid && evt.involvedObject.kind == "Node";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Node events have node name as uid.

@nevalla nevalla added the enhancement New feature or request label Apr 20, 2020
@nevalla nevalla requested a review from a team April 20, 2020 19:49
Copy link
Contributor

@aleksfront aleksfront left a comment

Choose a reason for hiding this comment

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

LGTM

@jakolehm jakolehm added this to the 3.4.0 milestone Apr 21, 2020
@jakolehm jakolehm merged commit c1010c1 into master Apr 21, 2020
@jakolehm jakolehm deleted the feature/show-node-events-in-node-details branch April 21, 2020 16:51
@jakolehm jakolehm mentioned this pull request Apr 28, 2020
@jakolehm jakolehm mentioned this pull request May 9, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

show node events in node
3 participants