[ti_google_threat_intelligence] Resize metric panels on IOC-related dashboards

[niraj-crest]

Proposed commit message

[ti_google_threat_intelligence] Resize metric panels on IOC-related dashboards (#14763) These panels previously used decimal values for width and height, but that no longer works. This change fixes the layout by using integer values. 

Earlier proposed commit message:

Remove decimal values in metric visualisation.

Checklist

• I have reviewed tips for building integrations and this pull request is aligned with them.
• I have verified that all data streams collect metrics or logs.
• I have added an entry to my package's changelog.yml file.
• I have verified that Kibana version constraints are current according to guidelines.
• I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related Issues

• Closes ti_google_threat_intelligence: Dashboards incorrectly formatted #14794

Dashboard

Before (before the layout was broken)

After (after the layout was fixed)

[elasticmachine]

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

[chrisberkhout]

Looks good. Just a bit shorter vertically.

What motivated this? Just to use the space a bit more effectively or something else?

[efd6]

/test

[elastic-vault-github-plugin-prod]

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

[niraj-crest]

Thanks! Yes, the idea was to keep all the matrices consistent by adding decimal values for width and height in the dashboard.
However, it looks like this formatting isn't supported in the newer version, so we have removed decimal values.

[niraj-crest]

@chrisberkhout If every thing looks good can we please merge this PR.

[kcreddy]

Thanks! Yes, the idea was to keep all the matrices consistent by adding decimal values for width and height in the dashboard. However, it looks like this formatting isn't supported in the newer version, so we have removed decimal values.

@niraj-crest Can you update PR Proposed commit message to reflect this?

[niraj-crest]

@kcreddy Updated proposed commit Message.

[chrisberkhout]

I see the point of the change now.

I suggested a different changelog message and put a clearer proposed commit message in the description.

This was originally confusing because the before and after screenshots in the PR description looked about the same, with slight size changes (which were mostly due to different zoom). Now I get the point that the metric panels layout was broken, and this PR fixes it. I adjusted the description to show that the before screenshots are before the layout was broken.

I think this should be a bugfix rather than an enhancement.

There's one question left about what looks like extra changes.

---

Out of the scope of this PR: the dashboard names and descriptions are very confusing. They seem to cover a lot of the same things and in some cases use almost identical descriptions.

[niraj-crest]

@chrisberkhout We have updated PR.

[chrisberkhout]

👍 I set it to auto-merge when the tests pass.

[niraj-crest]

@chrisberkhout as all comments have been resolved so can you please Merge this PR.

[kcreddy]

/test

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: e038a8b

History

• 💚 Build #29476 succeeded 940c35d

[elastic-sonarqube]

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
96.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

[elastic-vault-github-plugin-prod]

Package ti_google_threat_intelligence - 0.6.1 containing this change is available at https://epr.elastic.co/package/ti_google_threat_intelligence/0.6.1/