Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Security Solution][Notes] - fix an issue that breaks the notes management page, an enum value was missing from the api #196912

Conversation

PhilippeOberti
Copy link
Contributor

@PhilippeOberti PhilippeOberti commented Oct 18, 2024

Summary

Notes: please ignore the spaces introduced in the first commit, Webstorm is trying to be smart... CI should take care of fixing that in a second commit shortly

This PR fixes a small issue that prevents the notes management page to load. The issue comes from the fact that one of the enum values passed to filter notes was missing from the open api spec. This issue was introduced in this recent PR.

Current notes management borken page
Screenshot 2024-10-18 at 10 30 13 AM

This is the error in the network tab
Screenshot 2024-10-18 at 10 30 29 AM

This PR fixes the issue
Screenshot 2024-10-18 at 10 27 01 AM

…ement page, an enum value was missing from the api
@PhilippeOberti PhilippeOberti added backport release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Threat Hunting:Investigations Security Solution Investigations Team v8.16.0 labels Oct 18, 2024
@PhilippeOberti PhilippeOberti requested a review from a team as a code owner October 18, 2024 15:33
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 20.7MB 20.7MB +6.0B

@PhilippeOberti PhilippeOberti merged commit bf7c5aa into elastic:main Oct 18, 2024
44 of 45 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 18, 2024
…ement page, an enum value was missing from the api (elastic#196912)

## Summary

_**Notes: please ignore the spaces introduced in the first commit,
Webstorm is trying to be smart... CI should take care of fixing that in
a second commit shortly**_

This PR fixes a small issue that prevents the notes management page to
load. The issue comes from the fact that one of the enum values passed
to filter notes was missing from the open api spec. This issue was
introduced in this recent
[PR](elastic#195501).

Current notes management borken page
![Screenshot 2024-10-18 at 10 30
13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)

This is the error in the network tab
![Screenshot 2024-10-18 at 10 30
29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)

This PR fixes the issue
![Screenshot 2024-10-18 at 10 27
01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit bf7c5aa)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 18, 2024
…ement page, an enum value was missing from the api (elastic#196912)

## Summary

_**Notes: please ignore the spaces introduced in the first commit,
Webstorm is trying to be smart... CI should take care of fixing that in
a second commit shortly**_

This PR fixes a small issue that prevents the notes management page to
load. The issue comes from the fact that one of the enum values passed
to filter notes was missing from the open api spec. This issue was
introduced in this recent
[PR](elastic#195501).

Current notes management borken page
![Screenshot 2024-10-18 at 10 30
13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)

This is the error in the network tab
![Screenshot 2024-10-18 at 10 30
29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)

This PR fixes the issue
![Screenshot 2024-10-18 at 10 27
01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit bf7c5aa)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
8.x

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

Questions ?

Please refer to the Backport tool documentation

@PhilippeOberti PhilippeOberti deleted the fix-notes-management-page-not-loading branch October 18, 2024 18:17
kibanamachine added a commit that referenced this pull request Oct 18, 2024
… management page, an enum value was missing from the api (#196912) (#196936)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution][Notes] - fix an issue that breaks the notes
management page, an enum value was missing from the api
(#196912)](#196912)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-18T18:09:02Z","message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","v8.16.0"],"title":"[Security Solution][Notes] -
fix an issue that breaks the notes management page, an enum value was
missing from the
api","number":196912,"url":"https://github.com/elastic/kibana/pull/196912","mergeCommit":{"message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196912","number":196912,"mergeCommit":{"message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <[email protected]>
kibanamachine added a commit that referenced this pull request Oct 18, 2024
…s management page, an enum value was missing from the api (#196912) (#196935)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Security Solution][Notes] - fix an issue that breaks the notes
management page, an enum value was missing from the api
(#196912)](#196912)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Philippe
Oberti","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-18T18:09:02Z","message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","v8.16.0"],"title":"[Security Solution][Notes] -
fix an issue that breaks the notes management page, an enum value was
missing from the
api","number":196912,"url":"https://github.com/elastic/kibana/pull/196912","mergeCommit":{"message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196912","number":196912,"mergeCommit":{"message":"[Security
Solution][Notes] - fix an issue that breaks the notes management page,
an enum value was missing from the api (#196912)\n\n##
Summary\r\n\r\n_**Notes: please ignore the spaces introduced in the
first commit,\r\nWebstorm is trying to be smart... CI should take care
of fixing that in\r\na second commit shortly**_\r\n\r\nThis PR fixes a
small issue that prevents the notes management page to\r\nload. The
issue comes from the fact that one of the enum values passed\r\nto
filter notes was missing from the open api spec. This issue
was\r\nintroduced in this
recent\r\n[PR](https://github.com/elastic/kibana/pull/195501).\r\n\r\nCurrent
notes management borken page\r\n![Screenshot 2024-10-18 at 10
30\r\n13 AM](https://github.com/user-attachments/assets/4926a62f-1ebf-4698-8a13-bf761d77f4ba)\r\n\r\nThis
is the error in the network tab\r\n![Screenshot 2024-10-18 at 10
30\r\n29 AM](https://github.com/user-attachments/assets/90b56246-c116-4050-bcfa-2c6668274e74)\r\n\r\nThis
PR fixes the issue\r\n![Screenshot 2024-10-18 at 10
27\r\n01 AM](https://github.com/user-attachments/assets/7d3338ce-ad73-4be5-b94c-15bcf0234680)\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"bf7c5aaf29e475979023a607a4f318b5f005d0d8"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Philippe Oberti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.16.0 v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants