Skip to content

[Response Ops][Reporting] Updating health API to return whether notification email service is available#219785

Merged
ymao1 merged 3 commits intoelastic:mainfrom
ymao1:reporting-notification
May 8, 2025
Merged

[Response Ops][Reporting] Updating health API to return whether notification email service is available#219785
ymao1 merged 3 commits intoelastic:mainfrom
ymao1:reporting-notification

Conversation

@ymao1
Copy link
Contributor

@ymao1 ymao1 commented Apr 30, 2025

Summary

Updates the internal reporting health API from #216857 to return whether the notification email service is available. We'll be using this notification email service to send the scheduled report notifications so this lets us know if that service is available.

GET kbn:/internal/reporting/_health

Response 
{
  "has_permanent_encryption_key": true,
  "is_sufficiently_secure": true,
  "are_notifications_enabled": true
}

To Verify

  1. Run kibana and ES with a preconfigured email connector in the kibana.yml and the notification configured to use the connector
notifications.connectors.default.email: gmail
xpack.actions.preconfigured:
  gmail:
    name: 'email: my gmail'
    actionTypeId: '.email'
    config:
      <config>
    secrets:
      <secrets>
  1. Access the health API, are_notifications_enabled should be true
  2. Remove the config and restart Kibana
  3. Access the health API, are_notifications_enabled should be false

@ymao1 ymao1 self-assigned this Apr 30, 2025
@ymao1 ymao1 added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// Feature:Reporting:Framework Reporting issues pertaining to the overall framework backport:version Backport to applied version labels v9.1.0 v8.19.0 labels Apr 30, 2025
@ymao1 ymao1 force-pushed the reporting-notification branch from ccc23be to a4d58ed Compare April 30, 2025 18:18
@ymao1 ymao1 marked this pull request as ready for review April 30, 2025 20:14
@ymao1 ymao1 requested review from a team as code owners April 30, 2025 20:14
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@ymao1 ymao1 requested review from js-jankisalvi and pmuellr April 30, 2025 20:14
@ymao1
Copy link
Contributor Author

ymao1 commented May 5, 2025

@elasticmachine merge upstream

Copy link
Contributor

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

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

LGTM

@ymao1
Copy link
Contributor Author

ymao1 commented May 8, 2025

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

cc @ymao1

@ymao1 ymao1 merged commit 25a873c into elastic:main May 8, 2025
9 checks passed
@ymao1 ymao1 deleted the reporting-notification branch May 8, 2025 14:11
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/14908598079

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 219785

Questions ?

Please refer to the Backport tool documentation

@ymao1
Copy link
Contributor Author

ymao1 commented May 8, 2025

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

ymao1 added a commit to ymao1/kibana that referenced this pull request May 8, 2025
…ication email service is available (elastic#219785)

## Summary

Updates the internal reporting health API from
elastic#216857 to return whether the
notification email service is available. We'll be using this
notification email service to send the scheduled report notifications so
this lets us know if that service is available.

```
GET kbn:/internal/reporting/_health

Response
{
  "has_permanent_encryption_key": true,
  "is_sufficiently_secure": true,
  "are_notifications_enabled": true
}
```

## To Verify
1. Run kibana and ES with a preconfigured email connector in the
`kibana.yml` and the notification configured to use the connector

```
notifications.connectors.default.email: gmail
xpack.actions.preconfigured:
  gmail:
    name: 'email: my gmail'
    actionTypeId: '.email'
    config:
      <config>
    secrets:
      <secrets>
```
2. Access the health API, `are_notifications_enabled` should be `true`
3. Remove the config and restart Kibana
4. Access the health API, `are_notifications_enabled` should be `false`

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 25a873c)
ymao1 added a commit that referenced this pull request May 8, 2025
…r notification email service is available (#219785) (#220555)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Response Ops][Reporting] Updating health API to return whether
notification email service is available
(#219785)](#219785)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Ying
Mao","email":"ying.mao@elastic.co"},"sourceCommit":{"committedDate":"2025-05-08T14:11:53Z","message":"[Response
Ops][Reporting] Updating health API to return whether notification email
service is available (#219785)\n\n## Summary\n\nUpdates the internal
reporting health API from\nhttps://github.com//pull/216857
to return whether the\nnotification email service is available. We'll be
using this\nnotification email service to send the scheduled report
notifications so\nthis lets us know if that service is
available.\n\n```\nGET kbn:/internal/reporting/_health\n\nResponse \n{\n
\"has_permanent_encryption_key\": true,\n \"is_sufficiently_secure\":
true,\n \"are_notifications_enabled\": true\n}\n```\n\n## To Verify\n1.
Run kibana and ES with a preconfigured email connector in
the\n`kibana.yml` and the notification configured to use the
connector\n\n```\nnotifications.connectors.default.email:
gmail\nxpack.actions.preconfigured:\n gmail:\n name: 'email: my gmail'\n
actionTypeId: '.email'\n config:\n <config>\n secrets:\n
<secrets>\n```\n2. Access the health API, `are_notifications_enabled`
should be `true`\n3. Remove the config and restart Kibana\n4. Access the
health API, `are_notifications_enabled` should be
`false`\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"25a873c86c2c7fe1f5b5f345d2ee349f721c7c2a","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","Feature:Reporting:Framework","backport:version","v9.1.0","v8.19.0"],"title":"[Response
Ops][Reporting] Updating health API to return whether notification email
service is
available","number":219785,"url":"https://github.com/elastic/kibana/pull/219785","mergeCommit":{"message":"[Response
Ops][Reporting] Updating health API to return whether notification email
service is available (#219785)\n\n## Summary\n\nUpdates the internal
reporting health API from\nhttps://github.com//pull/216857
to return whether the\nnotification email service is available. We'll be
using this\nnotification email service to send the scheduled report
notifications so\nthis lets us know if that service is
available.\n\n```\nGET kbn:/internal/reporting/_health\n\nResponse \n{\n
\"has_permanent_encryption_key\": true,\n \"is_sufficiently_secure\":
true,\n \"are_notifications_enabled\": true\n}\n```\n\n## To Verify\n1.
Run kibana and ES with a preconfigured email connector in
the\n`kibana.yml` and the notification configured to use the
connector\n\n```\nnotifications.connectors.default.email:
gmail\nxpack.actions.preconfigured:\n gmail:\n name: 'email: my gmail'\n
actionTypeId: '.email'\n config:\n <config>\n secrets:\n
<secrets>\n```\n2. Access the health API, `are_notifications_enabled`
should be `true`\n3. Remove the config and restart Kibana\n4. Access the
health API, `are_notifications_enabled` should be
`false`\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"25a873c86c2c7fe1f5b5f345d2ee349f721c7c2a"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219785","number":219785,"mergeCommit":{"message":"[Response
Ops][Reporting] Updating health API to return whether notification email
service is available (#219785)\n\n## Summary\n\nUpdates the internal
reporting health API from\nhttps://github.com//pull/216857
to return whether the\nnotification email service is available. We'll be
using this\nnotification email service to send the scheduled report
notifications so\nthis lets us know if that service is
available.\n\n```\nGET kbn:/internal/reporting/_health\n\nResponse \n{\n
\"has_permanent_encryption_key\": true,\n \"is_sufficiently_secure\":
true,\n \"are_notifications_enabled\": true\n}\n```\n\n## To Verify\n1.
Run kibana and ES with a preconfigured email connector in
the\n`kibana.yml` and the notification configured to use the
connector\n\n```\nnotifications.connectors.default.email:
gmail\nxpack.actions.preconfigured:\n gmail:\n name: 'email: my gmail'\n
actionTypeId: '.email'\n config:\n <config>\n secrets:\n
<secrets>\n```\n2. Access the health API, `are_notifications_enabled`
should be `true`\n3. Remove the config and restart Kibana\n4. Access the
health API, `are_notifications_enabled` should be
`false`\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"25a873c86c2c7fe1f5b5f345d2ee349f721c7c2a"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
…ication email service is available (elastic#219785)

## Summary

Updates the internal reporting health API from
elastic#216857 to return whether the
notification email service is available. We'll be using this
notification email service to send the scheduled report notifications so
this lets us know if that service is available.

```
GET kbn:/internal/reporting/_health

Response 
{
  "has_permanent_encryption_key": true,
  "is_sufficiently_secure": true,
  "are_notifications_enabled": true
}
```

## To Verify
1. Run kibana and ES with a preconfigured email connector in the
`kibana.yml` and the notification configured to use the connector

```
notifications.connectors.default.email: gmail
xpack.actions.preconfigured:
  gmail:
    name: 'email: my gmail'
    actionTypeId: '.email'
    config:
      <config>
    secrets:
      <secrets>
```
2. Access the health API, `are_notifications_enabled` should be `true`
3. Remove the config and restart Kibana
4. Access the health API, `are_notifications_enabled` should be `false`

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
qn895 pushed a commit to qn895/kibana that referenced this pull request Jun 3, 2025
…ication email service is available (elastic#219785)

## Summary

Updates the internal reporting health API from
elastic#216857 to return whether the
notification email service is available. We'll be using this
notification email service to send the scheduled report notifications so
this lets us know if that service is available.

```
GET kbn:/internal/reporting/_health

Response 
{
  "has_permanent_encryption_key": true,
  "is_sufficiently_secure": true,
  "are_notifications_enabled": true
}
```

## To Verify
1. Run kibana and ES with a preconfigured email connector in the
`kibana.yml` and the notification configured to use the connector

```
notifications.connectors.default.email: gmail
xpack.actions.preconfigured:
  gmail:
    name: 'email: my gmail'
    actionTypeId: '.email'
    config:
      <config>
    secrets:
      <secrets>
```
2. Access the health API, `are_notifications_enabled` should be `true`
3. Remove the config and restart Kibana
4. Access the health API, `are_notifications_enabled` should be `false`

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:Reporting:Framework Reporting issues pertaining to the overall framework release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants