Update package descriptions

[bmorelli25]

Summary

This PR updates the package manifest.yml descriptions for every top-level package in the integrations repo. The goal of this PR is to add more descriptive descriptions as these will appear as a sub-heading on each integration's documentation page. In this first attempt, I've taken the following approach:

This Elastic integration collects + {logs/metrics} + from {integration name}

Open to any and all feedback on this as I'm not sure how these changes impact packages elsewhere (like in Kibana). Also, collecting is only one part of an integration. Do we want to add other verbs like parses or visualizes?

Other integrations

As far as I can tell, this PR updates all relevant integrations except APM, Endpoint, and Symantec -- which I presume live in other repos. I know where APM lives, and I think I can figure out where Endpoint and Symantec live.

Context

This is related to conversations in https://github.com/elastic/wordlake/pull/1 (sorry, internal link)

[elasticmachine]

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

• Start Time: 2021-07-29T17:51:57.889+0000

• Duration: 72 min 16 sec

• Commit: 38977e5

Test stats 🧪

Test	Results
Failed	0
Passed	2620
Skipped	3
Total	2623

Trends 🧪

[andrewkroh]

CEF isn't a product that you would run an "instance" of. It's a log format like JSON. So perhaps there is a better phrasing of this description.

[bmorelli25]

Thanks, Andrew! I pulled "CEF instance" from L17 of the manifest file. Should I update that line as well to remove it? https://github.com/elastic/integrations/pull/1364/files#diff-966e13402568cef1412dea521600e7fb5172601510e51030c0bbb41d55351e43L17

[bmorelli25]

Not sure if that link will work. Here's a better one:

integrations/packages/cef/manifest.yml

Line 17 in a5e8dd2

description: Collect logs from CEF instances

[mtojek]

In general we use elastic-package format to format sources of integrations.

[mtojek]

@bmorelli25

We have requirement that CI must report green state to get the PR merged. In this case is red which means there is something wrong with this PR. There are problems with elastic_agent (missing manifest entry) and okta (missing manifest entry).

[bmorelli25]

There are problems with elastic_agent (missing manifest entry) and okta (missing manifest entry).

Thanks! I think everything should pass this time 🤞

[bmorelli25]

Noooooo. Green, but a change was merged to the system integration. I'll fix the conflicts and let the tests run again.

[mtojek]

Remember to bump up package versions accordingly in case of conflicts.

LGTM, feel free to merge this PR if CI is happy.

[bmorelli25]

Remember to bump up package versions accordingly in case of conflicts.

Yup. The merged update in this case happened to be a patch, so my minor update was still the latest -- just needed to merge in the new CL entry.

LGTM, feel free to merge this PR if CI is happy.

Thanks! I'll be clicking the button the second it's green :)

[bmorelli25]

Test just timed out. Restarting.