Skip to content

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

Merged
PhilippeOberti merged 1 commit intoelastic:mainfrom
PhilippeOberti:max-unassociated-note
Feb 28, 2025
Merged

[Security Solution][Notes] change upper limit for max unassociated notes advanced setting to be 10k instead of 1k#212786
PhilippeOberti merged 1 commit intoelastic:mainfrom
PhilippeOberti:max-unassociated-note

Conversation

@PhilippeOberti
Copy link
Copy Markdown
Contributor

@PhilippeOberti PhilippeOberti commented Feb 28, 2025

Summary

This previous PR 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) 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

After

Screenshot 2025-02-28 at 9 18 47 AM

…tes advanced setting to be 10k instead of 1k
@PhilippeOberti PhilippeOberti added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team backport:all-open Backport to all branches that could still receive a release v8.18.0 v9.1.0 v8.19.0 v8.17.3 v8.16.5 labels Feb 28, 2025
@PhilippeOberti PhilippeOberti requested a review from a team as a code owner February 28, 2025 15:22
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@PhilippeOberti PhilippeOberti merged commit eabf95d into elastic:main Feb 28, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 7.17, 8.16, 8.17, 8.18, 8.x, 9.0

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

@PhilippeOberti PhilippeOberti deleted the max-unassociated-note branch February 28, 2025 21:26
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 28, 2025
…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 pushed a commit to kibanamachine/kibana that referenced this pull request Feb 28, 2025
…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 pushed a commit to kibanamachine/kibana that referenced this pull request Feb 28, 2025
…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 pushed a commit to kibanamachine/kibana that referenced this pull request Feb 28, 2025
…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 pushed a commit to kibanamachine/kibana that referenced this pull request Feb 28, 2025
…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
Copy link
Copy Markdown
Contributor

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts
8.16
8.17
8.18
8.x
9.0

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 212786

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 28, 2025
…ated notes advanced setting to be 10k instead of 1k (#212786) (#212823)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(#212786)](#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x","8.17","8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 28, 2025
…ted notes advanced setting to be 10k instead of 1k (#212786) (#212825)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(#212786)](#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x","8.17","8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 28, 2025
…ated notes advanced setting to be 10k instead of 1k (#212786) (#212822)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(#212786)](#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x","8.17","8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 28, 2025
…ted notes advanced setting to be 10k instead of 1k (#212786) (#212824)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(#212786)](#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x","8.17","8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
@PhilippeOberti
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
8.16

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

Questions ?

Please refer to the Backport tool documentation

PhilippeOberti added a commit to PhilippeOberti/kibana that referenced this pull request Mar 3, 2025
…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)
PhilippeOberti added a commit that referenced this pull request Mar 3, 2025
…ated notes advanced setting to be 10k instead of 1k (#212786) (#212930)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(#212786)](#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/212825","number":212825,"state":"MERGED","mergeCommit":{"sha":"aa141aa281bb5c6f6c6503536349cb8bfefa7e7b","message":"[9.0]
[Security Solution][Notes] change upper limit for max unassociated notes
advanced setting to be 10k instead of 1k (#212786) (#212825)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.0`:\n- [[Security Solution][Notes] change upper limit for max
unassociated\nnotes advanced setting to be 10k instead of
1k\n(#212786)](https://github.com/elastic/kibana/pull/212786)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Philippe Oberti
<philippe.oberti@elastic.co>"}},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/212823","number":212823,"state":"MERGED","mergeCommit":{"sha":"b32e708ffc49f3d28cec6a85cc72dfcca3e7052d","message":"[8.18]
[Security Solution][Notes] change upper limit for max unassociated notes
advanced setting to be 10k instead of 1k (#212786) (#212823)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.18`:\n- [[Security Solution][Notes] change upper limit for max
unassociated\nnotes advanced setting to be 10k instead of
1k\n(#212786)](https://github.com/elastic/kibana/pull/212786)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Philippe Oberti
<philippe.oberti@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (#212786)\n\n## Summary\n\nThis previous
[PR](#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/212824","number":212824,"state":"MERGED","mergeCommit":{"sha":"b35c244503bff376b44ca7e2eca2a8ba06c263b2","message":"[8.x]
[Security Solution][Notes] change upper limit for max unassociated notes
advanced setting to be 10k instead of 1k (#212786) (#212824)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.x`:\n- [[Security Solution][Notes] change upper limit for max
unassociated\nnotes advanced setting to be 10k instead of
1k\n(#212786)](https://github.com/elastic/kibana/pull/212786)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Philippe Oberti
<philippe.oberti@elastic.co>"}},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/212822","number":212822,"state":"MERGED","mergeCommit":{"sha":"229aac1adf86b96d2718bc66670496263c1dc6c8","message":"[8.17]
[Security Solution][Notes] change upper limit for max unassociated notes
advanced setting to be 10k instead of 1k (#212786) (#212822)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.17`:\n- [[Security Solution][Notes] change upper limit for max
unassociated\nnotes advanced setting to be 10k instead of
1k\n(#212786)](https://github.com/elastic/kibana/pull/212786)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Philippe Oberti
<philippe.oberti@elastic.co>"}},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
SoniaSanzV pushed a commit to SoniaSanzV/kibana that referenced this pull request Mar 4, 2025
…ted notes advanced setting to be 10k instead of 1k (elastic#212786) (elastic#212824)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][Notes] change upper limit for max unassociated
notes advanced setting to be 10k instead of 1k
(elastic#212786)](elastic#212786)

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

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"philippe.oberti@elastic.co"},"sourceCommit":{"committedDate":"2025-02-28T21:21:37Z","message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (elastic#212786)\n\n## Summary\n\nThis previous
[PR](elastic#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:all-open","v8.18.0","v9.1.0","v8.19.0","v8.17.3","v8.16.5"],"title":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of
1k","number":212786,"url":"https://github.com/elastic/kibana/pull/212786","mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (elastic#212786)\n\n## Summary\n\nThis previous
[PR](elastic#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x","8.17","8.16"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/212786","number":212786,"mergeCommit":{"message":"[Security
Solution][Notes] change upper limit for max unassociated notes advanced
setting to be 10k instead of 1k (elastic#212786)\n\n## Summary\n\nThis previous
[PR](elastic#194947) made\nthe maximum
number of unassociated notes an advanced settings so that\nuser can
change the value within a certain limit. The intent was to\nallow that
value to be between 1 and 10,000 (see acceptance criteria of\nthe
[original ticket](https://github.com/elastic/kibana/issues/193097))\nbut
we missed one 0 and the maximum value allowed got set to 1000.\n\nThis
PR fixes that.\n\n#### Before\n\n![Screenshot 2025-02-28 at 9
18\n14 AM](https://github.com/user-attachments/assets/cf1d473c-5bd5-4759-a834-60888b0c8f78)\n\n####
After\n\n![Screenshot 2025-02-28 at 9
18\n47 AM](https://github.com/user-attachments/assets/129ba898-bbad-420a-b615-b0a456640af4)","sha":"eabf95d6dc157d1c1ac2bb8832c7021986285aef"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.3","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.5","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <philippe.oberti@elastic.co>
@mistic
Copy link
Copy Markdown
Contributor

mistic commented Mar 4, 2025

This PR didn't make it into the latest BC of both v8.16.5 and v8.17.3. Updating the labels accordingly.

CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
…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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:all-open Backport to all branches that could still receive a release release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v8.16.6 v8.17.4 v8.18.0 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants