Skip to content

DDSaaS: Genesys: Updated Assets as per metric prefix #20162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 13, 2025
Merged

DDSaaS: Genesys: Updated Assets as per metric prefix #20162

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
423bfd2
Added metric prefix changes
avaid-crest Apr 28, 2025
6a5485b
Merge branch 'master' of https://github.com/DataDog/integrations-core…
avaid-crest Apr 28, 2025
d425e20
Added monitor name changes
avaid-crest Apr 28, 2025
821ffe6
Removed check metric as per review comment
avaid-crest May 12, 2025
0c8321c
Merge branch 'master' of https://github.com/DataDog/integrations-core…
avaid-crest May 12, 2025
2c93fc0
Reverting the change regarding the check metric
avaid-crest May 12, 2025
25262b3
Added changes as per the check metric in manifest
avaid-crest May 13, 2025
76aafc4
Added suffix for metric
avaid-crest May 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
84 changes: 42 additions & 42 deletions genesys/assets/dashboards/genesys_callback.json
View file
Open in desktop

Large diffs are not rendered by default.

98 changes: 49 additions & 49 deletions genesys/assets/dashboards/genesys_chat.json
View file
Open in desktop

Large diffs are not rendered by default.

100 changes: 50 additions & 50 deletions genesys/assets/dashboards/genesys_conversation_overview.json
View file
Open in desktop

Large diffs are not rendered by default.

102 changes: 51 additions & 51 deletions genesys/assets/dashboards/genesys_email.json
View file
Open in desktop

Large diffs are not rendered by default.

94 changes: 47 additions & 47 deletions genesys/assets/dashboards/genesys_message.json
View file
Open in desktop

Large diffs are not rendered by default.

108 changes: 54 additions & 54 deletions genesys/assets/dashboards/genesys_voice.json
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion genesys/assets/monitors/high_number_of_abandoned_outbound_conversations_detected.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"id": 157631120,
"name": "High Number of Abandoned Outbound Voice Conversations Detected",
"type": "query alert",
"query": "sum(last_1d):sum:genesys.voice.abandoned_outbound_conversations_count{*}.as_count() > 100",
"query": "sum(last_1d):sum:genesys_cloud.voice.abandoned_outbound_conversations_count{*}.as_count() > 100",
"message": "{{#is_alert}}\nThe abandoned outbound conversations refer to the number of outbound dialer calls that were abandoned.\n\nA high number of abandoned outbound conversations has been detected. Please review and take necessary action.\n{{/is_alert}}",
"tags": [
"source:genesys"
Expand Down
2 changes: 1 addition & 1 deletion genesys/assets/monitors/high_number_of_errored_conversations_detected.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"id": 157628372,
"name": "High Number of Errored Conversations Detected",
"type": "query alert",
"query": "sum(last_1d):sum:genesys.callback.errored_conversations_count{*}.as_count() + sum:genesys.chat.errored_conversations_count{*}.as_count() + sum:genesys.email.errored_conversations_count{*}.as_count() + sum:genesys.message.errored_conversations_count{*}.as_count() + sum:genesys.voice.errored_conversations_count{*}.as_count() > 100",
"query": "sum(last_1d):sum:genesys_cloud.callback.errored_conversations_count{*}.as_count() + sum:genesys_cloud.chat.errored_conversations_count{*}.as_count() + sum:genesys_cloud.email.errored_conversations_count{*}.as_count() + sum:genesys_cloud.message.errored_conversations_count{*}.as_count() + sum:genesys_cloud.voice.errored_conversations_count{*}.as_count() > 100",
"message": "{{#is_alert}}\nErrored conversations refer to the number of active sessions aborted due to an Edge or adapter error event.\n\nHigh number of errored conversations has been detected. Please review and take necessary action.\n{{/is_alert}}",
"tags": [
"source:genesys"
Expand Down
2 changes: 1 addition & 1 deletion genesys/assets/monitors/high_number_of_over_sla_conversations_detected.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"id": 157630365,
"name": "High Number of over SLA Conversations Detected",
"type": "query alert",
"query": "sum(last_1d):sum:genesys.callback.over_sla_count{*}.as_count() + sum:genesys.chat.over_sla_count{*}.as_count() + sum:genesys.email.over_sla_count{*}.as_count() + sum:genesys.message.over_sla_count{*}.as_count() + sum:genesys.voice.over_sla_count{*}.as_count() > 100",
"query": "sum(last_1d):sum:genesys_cloud.callback.over_sla_count{*}.as_count() + sum:genesys_cloud.chat.over_sla_count{*}.as_count() + sum:genesys_cloud.email.over_sla_count{*}.as_count() + sum:genesys_cloud.message.over_sla_count{*}.as_count() + sum:genesys_cloud.voice.over_sla_count{*}.as_count() > 100",
"message": "{{#is_alert}}\nThe over SLA conversations refer to the number of answered interactions that were over the SLA threshold.\n\nHigh number of over SLA conversations has been detected. Please review and take necessary action.\n{{/is_alert}}",
"tags": [
"source:genesys"
Expand Down
2 changes: 1 addition & 1 deletion genesys/assets/monitors/high_number_of_state_transition_errors_detected.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"id": 157630544,
"name": "High Number of State Transition Errors Detected",
"type": "query alert",
"query": "sum(last_1d):sum:genesys.callback.state_transition_errors_count{*}.as_count() + sum:genesys.chat.state_transition_errors_count{*}.as_count() + sum:genesys.email.state_transition_errors_count{*}.as_count() + sum:genesys.message.state_transition_errors_count{*}.as_count() + sum:genesys.voice.state_transition_errors_count{*}.as_count() > 100",
"query": "sum(last_1d):sum:genesys_cloud.callback.state_transition_errors_count{*}.as_count() + sum:genesys_cloud.chat.state_transition_errors_count{*}.as_count() + sum:genesys_cloud.email.state_transition_errors_count{*}.as_count() + sum:genesys_cloud.message.state_transition_errors_count{*}.as_count() + sum:genesys_cloud.voice.state_transition_errors_count{*}.as_count() > 100",
"message": "{{#is_alert}}\nThe state transition errors refer to the number of errors caused by Edge clock skew.\n\nA high number of state transition errors has been detected. Please review and take necessary action.\n{{/is_alert}}",
"tags": [
"source:genesys"
Expand Down
4 changes: 2 additions & 2 deletions genesys/manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,8 +65,8 @@
"creates_events": false
},
"metrics": {
"prefix": "genesys.",
"check": ["genesys.voice.total_conversations", "genesys.callback.wait_time_avg"],
"prefix": "genesys_cloud.",
"check": ["genesys_cloud.voice.total_conversations", "genesys_cloud.callback.wait_time_avg"],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

"metadata_path": "metadata.csv"
},
"service_checks": {
Expand Down
844 changes: 422 additions & 422 deletions genesys/metadata.csv
View file
Open in desktop

Large diffs are not rendered by default.

Loading