Skip to content

[Alerts page] Fix Failed to check if maintenance windows are active#261048

Merged
maryam-saeidi merged 9 commits intoelastic:mainfrom
maryam-saeidi:246011-failed-maintenance-window
Apr 7, 2026
Merged

[Alerts page] Fix Failed to check if maintenance windows are active#261048
maryam-saeidi merged 9 commits intoelastic:mainfrom
maryam-saeidi:246011-failed-maintenance-window

Conversation

@maryam-saeidi
Copy link
Copy Markdown
Member

@maryam-saeidi maryam-saeidi commented Apr 2, 2026

Fixes #246011

Summary

Fixes "Failed to check if maintenance windows are active" error by skipping the maintenance window API call when user does not have sufficient licence.

@maryam-saeidi maryam-saeidi self-assigned this Apr 2, 2026
@maryam-saeidi maryam-saeidi added release_note:fix backport:skip This PR does not require backporting labels Apr 2, 2026
@maryam-saeidi maryam-saeidi marked this pull request as ready for review April 7, 2026 11:56
@maryam-saeidi maryam-saeidi requested review from a team as code owners April 7, 2026 11:56
Copy link
Copy Markdown
Contributor

@ana-davydova ana-davydova left a comment

Choose a reason for hiding this comment

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

Actionable-obs changes look good to me!

Copy link
Copy Markdown
Contributor

@adcoelho adcoelho left a comment

Choose a reason for hiding this comment

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

Thank you for doing this. I remember the team discussing it, but we never got around to it 😅

@maryam-saeidi maryam-saeidi enabled auto-merge (squash) April 7, 2026 13:45
@elasticmachine
Copy link
Copy Markdown
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #4 / buildEsqlFetchSubscribe changing a ES|QL query with no transformational commands should not change state when loading and finished if index pattern and columns are the same
  • [job] [logs] Jest Tests #8 / hooks useTabs should return only the graph view tab when no other tabs are available
  • [job] [logs] Jest Tests #8 / hooks useTabs should return only the graph view tab when no other tabs are available
  • [job] [logs] FTR Configs #168 / serverless search UI Rule details Edit rule with deleted connector should show and update deleted connectors when there are existing connectors of the same type

History

cc @maryam-saeidi

@maryam-saeidi maryam-saeidi merged commit 37ce2e2 into elastic:main Apr 7, 2026
19 checks passed
@maryam-saeidi maryam-saeidi deleted the 246011-failed-maintenance-window branch April 7, 2026 17:18
@maryam-saeidi maryam-saeidi added v9.3.4 backport:version Backport to applied version labels and removed backport:skip This PR does not require backporting labels Apr 9, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.3

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

@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.3

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

@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
9.3 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 261048

Questions ?

Please refer to the Backport tool documentation

1 similar comment
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
9.3 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 261048

Questions ?

Please refer to the Backport tool documentation

@maryam-saeidi
Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
9.3

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

Questions ?

Please refer to the Backport tool documentation

maryam-saeidi added a commit to maryam-saeidi/kibana that referenced this pull request Apr 10, 2026
…lastic#261048)

Fixes elastic#246011

 ## Summary

Fixes "Failed to check if maintenance windows are active" error by
skipping the maintenance window API call when user does not have
sufficient licence.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 37ce2e2)

# Conflicts:
#	src/platform/packages/shared/kbn-alerts-ui-shared/moon.yml
#	src/platform/packages/shared/kbn-alerts-ui-shared/tsconfig.json
#	x-pack/platform/plugins/shared/triggers_actions_ui/public/application/sections/rules_list/components/rules_list.test.tsx
#	x-pack/solutions/observability/plugins/observability/public/pages/alerts/alerts.test.tsx
maryam-saeidi added a commit that referenced this pull request Apr 10, 2026
…tive (#261048) (#262461)

# Backport

This will backport the following commits from `main` to `9.3`:
- [[Alerts page] Fix Failed to check if maintenance windows are active
(#261048)](#261048)

<!--- Backport version: 11.0.1 -->

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

<!--BACKPORT [{"author":{"name":"Maryam
Saeidi","email":"maryam.saeidi@elastic.co"},"sourceCommit":{"committedDate":"2026-04-07T16:09:28Z","message":"[Alerts
page] Fix Failed to check if maintenance windows are active
(#261048)\n\nFixes https://github.com/elastic/kibana/issues/246011\n \n
## Summary\n \nFixes \"Failed to check if maintenance windows are
active\" error by\nskipping the maintenance window API call when user
does not have\nsufficient licence.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"37ce2e26d34f61bef9c81bc10be64d6fcbd68466","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","backport:version","v9.4.0","v9.3.4"],"title":"[Alerts
page] Fix Failed to check if maintenance windows are
active","number":261048,"url":"https://github.com/elastic/kibana/pull/261048","mergeCommit":{"message":"[Alerts
page] Fix Failed to check if maintenance windows are active
(#261048)\n\nFixes https://github.com/elastic/kibana/issues/246011\n \n
## Summary\n \nFixes \"Failed to check if maintenance windows are
active\" error by\nskipping the maintenance window API call when user
does not have\nsufficient licence.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"37ce2e26d34f61bef9c81bc10be64d6fcbd68466"}},"sourceBranch":"main","suggestedTargetBranches":["9.3"],"targetPullRequestStates":[{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/261048","number":261048,"mergeCommit":{"message":"[Alerts
page] Fix Failed to check if maintenance windows are active
(#261048)\n\nFixes https://github.com/elastic/kibana/issues/246011\n \n
## Summary\n \nFixes \"Failed to check if maintenance windows are
active\" error by\nskipping the maintenance window API call when user
does not have\nsufficient licence.\n\n---------\n\nCo-authored-by:
kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"37ce2e26d34f61bef9c81bc10be64d6fcbd68466"}},{"branch":"9.3","label":"v9.3.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment