DDSaaS: Genesys: Updated Assets as per metric prefix

[avaid-crest]

What does this PR do?

• In this pull request, the assets have been updated to reflect the change in the metric prefix from genesys to genesys_cloud.

Review checklist (to be filled by reviewers)

• Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
• Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
• If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

[aliciascott]

good for docs

[cit-pr-commenter]

PR Security Update

All commits in this PR up to and including 76aafc4 have been reviewed and marked safe by SDLC security. For any questions, please reach out to #ci-for-external-contributors-collab on Slack.

[nathanmadams]

Thanks for this... just the one comment about the check metrics.

[nathanmadams]

I think we should remove these "check" metrics... reason being that these metrics are optional and can be disabled by the user (so indicating that there's a problem when they're missing is likely to be confusing.)

[avaid-crest]

Hi @nathanmadams, we tried removing the 'check' metrics, but the pipeline fails with the error.
The assets > integration > metrics > check field is required if metrics metadata is present.

[nathanmadams]

Apologies for the confusion... for now can you just add a metric from each "stream" to the list? That way, no matter which streams they enable, we'll recognize that they're getting data.

[avaid-crest]

Hi @nathanmadams, we've added a check metric for each stream.

[nathanmadams]

Making the check metric removal request an explicit change request so it's visible

Review from aliciascott is dismissed. Related teams and files:

• documentation
  • genesys/manifest.json

Review from apiazza-dd is dismissed. Related teams and files:

• saas-integrations
  • genesys/manifest.json