ensure all kibana.json files have owners and they are consistent

[spalger]

kibana.json files now have owner fields, but they're not using consistent casing for the githubTeam field currently and it was missing from the dashboard_enhanced plugin.

[spalger]

@stacey-gammon Wasn't there supposed to be something that validated these files are correct?

[stacey-gammon]

Not merged yet - #108231 however... that PR only requires owner not githubTeam so this wouldn't have been caught.

[sorenlouv]

Off topic: I love that we have these plugin manifests. Would it make sense to have a test section, thereby making it much easier to get an overview and run all tests for a specific plugin:

{
  "id": "apm",
  "test": {
    "unit": "yarn jest",
    "cypress": "node x-pack/plugin/apm/scripts/ftr_e2e/cypress_run.js",
    "ftr:api:trial": "x-pack/test/apm_api_integration/trial/config.ts",
    "ftr:api:basic": "x-pack/test/apm_api_integration/basic/config.ts",
    "ftr:functional": "x-pack/test/functional/config.js",
  }
}

To run all APM's tests we could have a command like: yarn test apm. To run a single of APM's tests: yarn test apm ftr:api:basic

[spalger]

It's an interesting idea for sure. We're getting closer to moving the tests out of the root x-pack/test dir and into the plugins, which might make this a little less helpful but it's an interesting project to think through.

[elasticmachine]

Pinging @elastic/apm-ui (Team:apm)

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 125d44b
• Storybooks Preview
• Documentation Changes

Metrics [docs]

✅ unchanged

History

• 💚 Build #147909 succeeded e7279ac

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

[kibanamachine]

💚 Backport successful

Status	Branch	Result
✅	7.15
✅	7.x

The backport PRs will be merged automatically after passing CI.