-
Notifications
You must be signed in to change notification settings - Fork 221
Create channels for Slack, PagerDuty, Jira, and Shuffle integrations #7827
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.
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
Open
gonzaarancibia
wants to merge
17
commits into
main
Choose a base branch
from
enhancement/7814-create-channels-for-slack-pagerduty-jira-and-shuffle-integrations
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Create channels for Slack, PagerDuty, Jira, and Shuffle integrations #7827
gonzaarancibia
wants to merge
17
commits into
main
from
enhancement/7814-create-channels-for-slack-pagerduty-jira-and-shuffle-integrations
+761
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ess to add default channels Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
2 tasks
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
plugins/main/server/health-check/notification-default-channels/common/constants.test.ts
Outdated
Show resolved
Hide resolved
plugins/main/server/health-check/notification-default-channels/plugin/notification-plugin.ts
Outdated
Show resolved
Hide resolved
plugins/main/server/health-check/notification-default-channels/plugin/notification-plugin.ts
Outdated
Show resolved
Hide resolved
plugins/main/server/health-check/notification-default-channels/tasks/index.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
plugins/main/server/health-check/notification-default-channels/common/constants.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
plugins/main/server/health-check/notification-default-channels/common/constants.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
plugins/main/server/health-check/notification-default-channels/common/constants.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
plugins/main/server/health-check/notification-default-channels/common/constants.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
plugins/main/server/health-check/notification-default-channels/tasks/index.ts
Outdated
Show resolved
Hide resolved
Desvelao
reviewed
Oct 23, 2025
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
…ss test Signed-off-by: gonzaarancibia <gonzaloarancibia48@gmail.com>
ensure notification health check only runs if both indexer and dashboard plugins are available add detection for plugin availability and log when skipping registration update plugin setup types to reflect optional dashboard notifications contract
2 tasks
This reverts commit 217f272.
add reminder to implement validation ensuring the notifications plugin is installed and enabled before registering the default notification channel helps prevent potential errors from registering notification channels when dependencies are missing
provide a link to a relevant discussion for future context and easier code maintenance
ensure notification health check only runs if both indexer and dashboard plugins are available add detection for plugin availability and log when skipping registration update plugin setup types to reflect optional dashboard notifications contract
|
|
guidomodarelli
approved these changes
Oct 24, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🟢
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.












Description
When a user logs into the Wazuh dashboard, we add preconfigured integration channels for Slack, PagerDuty, Jira, and Shuffle on the Notifications plugin. This makes it easier for the user to configure them.
Issues Resolved
#7814
Evidence
Logs from dashboard server when creating default channels
Default channels created
Testing PagerDuty test sending message
Jira test sending message
Shuffle test sending message
Slack test sending message
Running test
Important
Before starting to run the tests, you should apply this patch to resolve permission issues.
See this comment: #7814 (comment)
Test
🟢 Test Legend
UI
npm run test:jest notification-default-channels/⚫ Verify that default channels are created successfully on dashboard initialization (check dashboard server logs).
* Open the Wazuh Dashboard and restart the server. * Check the dashboard server logs and confirm that default channels are created without errors. * Verify the log entries contain proper information for each created channel. Chrome - ⚫ Firefox - ⚫ Safari - ⚫⚫ Verify that all default channels render correctly in the Notification → Channels section.
* Go to Notification → Channels. * Check that all default channels (e.g., Slack, Jira, PagerDuty, Shuffle, Email, etc.) are displayed correctly. * Verify name, description, and configuration type are rendered as expected. Chrome - ⚫ Firefox - ⚫ Safari - ⚫⚫ Test Jira channel by going to Alerting → Monitors → Custom Monitor → Action and sending an event following the example shown in the channel description.
* Create a custom monitor and add an action for the Jira channel. * Use the payload example from the channel description. * Confirm that the event is successfully received in Jira and check for any errors in logs. Chrome - ⚫ Firefox - ⚫ Safari - ⚫⚫ Test PagerDuty channel by going to Alerting → Monitors → Custom Monitor → Action and sending an event following the example shown in the channel description.
* Create a custom monitor and add an action for the PagerDuty channel. * Use the provided example payload from the channel description. * Confirm that PagerDuty receives the event successfully. Chrome - ⚫ Firefox - ⚫ Safari - ⚫⚫ Test the rest of the channels directly using the “Send Test Message” button in the Channels section.
* Go to Notification → Channels. * Use the “Send Test Message” button for channels like Slack, Email, and Shuffle. * Confirm that each test message is successfully delivered and reflected in the logs. Chrome - ⚫ Firefox - ⚫ Safari - ⚫⚫ Run the tests by running:
yarn test:jest:runner notification-default-channels/(Before apply the patch)⚫
Check List
yarn test:jest