Skip to content

[8.7] [Infrastructure UI] Track host cloud provider on table entry click (#150685)#150760

Merged
kibanamachine merged 1 commit intoelastic:8.7from
kibanamachine:backport/8.7/pr-150685
Feb 9, 2023
Merged

[8.7] [Infrastructure UI] Track host cloud provider on table entry click (#150685)#150760
kibanamachine merged 1 commit intoelastic:8.7from
kibanamachine:backport/8.7/pr-150685

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.7:

Questions ?

Please refer to the Backport tool documentation

…lastic#150685)

## 📓 Summary

Closes elastic#150681

The tracked event when clicking an entry from the Hosts table was not
applying correctly the default `unknow` string for the `cloud_provider`
parameter.
Now it defaults to it in case the value is not passed or is null.

## 🧪 Testing
- Navigate to the Hosts view
- Open the console dev tools
- Click on a host name from the table and navigate to the host detail.
- Verify the following error does not appear in the log
```
Uncaught Error: Failed to validate payload coming from "Event Type 'Host Entry Clicked'":
[cloud_provider]: {"expected":"string","actual":"object","value":null}
```

Co-authored-by: Marco Antonio Ghiani <marcoantonio.ghiani@elastic.co>
(cherry picked from commit 26eddec)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 9, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) February 9, 2023 18:24
@kibanamachine kibanamachine merged commit d657331 into elastic:8.7 Feb 9, 2023
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
infra 87.1KB 87.1KB +8.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @tonyghiani

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants