Skip to content

[8.16] [Security Solution][Notes] change upper limit for max unassociated notes advanced setting to be 10k instead of 1k (#212786)#212821

Closed
kibanamachine wants to merge 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-212786
Closed

[8.16] [Security Solution][Notes] change upper limit for max unassociated notes advanced setting to be 10k instead of 1k (#212786)#212821
kibanamachine wants to merge 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-212786

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

…tes advanced setting to be 10k instead of 1k (elastic#212786)

## Summary

This previous [PR](elastic#194947) made
the maximum number of unassociated notes an advanced settings so that
user can change the value within a certain limit. The intent was to
allow that value to be between 1 and 10,000 (see acceptance criteria of
the [original ticket](elastic#193097))
but we missed one 0 and the maximum value allowed got set to 1000.

This PR fixes that.

#### Before

![Screenshot 2025-02-28 at 9 18
14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)

#### After

![Screenshot 2025-02-28 at 9 18
47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)

(cherry picked from commit eabf95d)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Feb 28, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) February 28, 2025 21:28
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #71 / ObservabilityApp o11y sidenav sidenav & breadcrumbs renders the correct nav and navigate to links

Metrics [docs]

✅ unchanged

cc @PhilippeOberti

@logeekal
Copy link
Copy Markdown
Contributor

logeekal commented Mar 3, 2025

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

There are no new commits on the base branch.

@logeekal
Copy link
Copy Markdown
Contributor

logeekal commented Mar 3, 2025

@PhilippeOberti , looks very strange. The check is failing because PR is outdated but base branch has no new commits 🤷

@PhilippeOberti
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

There are no new commits on the base branch.

auto-merge was automatically disabled March 3, 2025 15:02

Pull request was closed

@PhilippeOberti
Copy link
Copy Markdown
Contributor

@PhilippeOberti , looks very strange. The check is failing because PR is outdated but base branch has no new commits 🤷

Weird indeed... I closed this PR and opened a new one, let's see!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants