Skip to content

Conversation

@js-jankisalvi
Copy link
Contributor

@js-jankisalvi js-jankisalvi commented Jan 7, 2026

Summary

Internal MW client using internal repository was introduced in #247390.
It created problems while fetching MWs using all spaces in synthetics plugin.

This PR fixes the issue.

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

@js-jankisalvi js-jankisalvi self-assigned this Jan 7, 2026
@js-jankisalvi js-jankisalvi requested a review from a team as a code owner January 7, 2026 16:38
@js-jankisalvi js-jankisalvi added bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// backport:version Backport to applied version labels Feature:Maintenance Window v9.3.0 v9.4.0 labels Jan 7, 2026
@elasticmachine
Copy link
Contributor

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

@js-jankisalvi js-jankisalvi changed the title initial [ResponseOps][MaintenanceWindows] Fix mw internal client by passing hidden SO type Jan 7, 2026
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

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout: [ observability / observability ] plugin / serverless-oblt - Custom Threshold Rule - Ad-hoc Data View - should create a custom threshold rule with an ad-hoc data view
  • [job] [logs] Scout: [ observability / observability ] plugin / should create a custom threshold rule with an ad-hoc data view

Metrics [docs]

✅ unchanged

cc @js-jankisalvi

@js-jankisalvi js-jankisalvi merged commit 345ec13 into elastic:main Jan 8, 2026
13 checks passed
@js-jankisalvi js-jankisalvi deleted the fix-mw-internal-client branch January 8, 2026 09:07
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.3

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 8, 2026
…idden SO type (elastic#248132)

## Summary

Internal MW client using internal repository was introduced in
elastic#247390.
It created problems while fetching MWs using all spaces in synthetics
plugin.

This PR fixes the issue.

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit 345ec13)
@kibanamachine
Copy link
Contributor

💚 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

kibanamachine added a commit that referenced this pull request Jan 8, 2026
…sing hidden SO type (#248132) (#248219)

# Backport

This will backport the following commits from `main` to `9.3`:
- [[ResponseOps][MaintenanceWindows] Fix mw internal client by passing
hidden SO type (#248132)](#248132)

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

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

<!--BACKPORT [{"author":{"name":"Janki
Salvi","email":"[email protected]"},"sourceCommit":{"committedDate":"2026-01-08T09:07:03Z","message":"[ResponseOps][MaintenanceWindows]
Fix mw internal client by passing hidden SO type (#248132)\n\n##
Summary\n\nInternal MW client using internal repository was introduced
in\nhttps://github.com//pull/247390.\nIt created problems
while fetching MWs using all spaces in synthetics\nplugin.\n\nThis PR
fixes the issue.\n\n### Checklist\n\nCheck the PR satisfies following
conditions. \n\nReviewers should verify this PR satisfies this list as
well.\n\n- [x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"345ec13b123cf9b880b5e2a71e8630aa464bd403","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:skip","Team:ResponseOps","backport:version","Feature:Maintenance
Window","v9.3.0","v9.4.0"],"title":"[ResponseOps][MaintenanceWindows]
Fix mw internal client by passing hidden SO
type","number":248132,"url":"https://github.com/elastic/kibana/pull/248132","mergeCommit":{"message":"[ResponseOps][MaintenanceWindows]
Fix mw internal client by passing hidden SO type (#248132)\n\n##
Summary\n\nInternal MW client using internal repository was introduced
in\nhttps://github.com//pull/247390.\nIt created problems
while fetching MWs using all spaces in synthetics\nplugin.\n\nThis PR
fixes the issue.\n\n### Checklist\n\nCheck the PR satisfies following
conditions. \n\nReviewers should verify this PR satisfies this list as
well.\n\n- [x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"345ec13b123cf9b880b5e2a71e8630aa464bd403"}},"sourceBranch":"main","suggestedTargetBranches":["9.3"],"targetPullRequestStates":[{"branch":"9.3","label":"v9.3.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/248132","number":248132,"mergeCommit":{"message":"[ResponseOps][MaintenanceWindows]
Fix mw internal client by passing hidden SO type (#248132)\n\n##
Summary\n\nInternal MW client using internal repository was introduced
in\nhttps://github.com//pull/247390.\nIt created problems
while fetching MWs using all spaces in synthetics\nplugin.\n\nThis PR
fixes the issue.\n\n### Checklist\n\nCheck the PR satisfies following
conditions. \n\nReviewers should verify this PR satisfies this list as
well.\n\n- [x] Review the
[backport\nguidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)\nand
apply applicable `backport:*`
labels.","sha":"345ec13b123cf9b880b5e2a71e8630aa464bd403"}}]}]
BACKPORT-->

Co-authored-by: Janki Salvi <[email protected]>
devamanv pushed a commit to devamanv/kibana that referenced this pull request Jan 12, 2026
…idden SO type (elastic#248132)

## Summary

Internal MW client using internal repository was introduced in
elastic#247390.
It created problems while fetching MWs using all spaces in synthetics
plugin.

This PR fixes the issue.

### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
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 bug Fixes for quality problems that affect the customer experience Feature:Maintenance Window release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v9.3.0 v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants