[Security Solution]While adding rule exception , the auto added comment not have "Alert Id" text in it.

[sukhwindersingh-qasource]

Describe the bug:

• While adding rule exception , the auto added comment not have "Alert Id" text in it.

Build Details:

VERSION: 8.9.0 BC1
BUILD: 64385
COMMIT: 313dac73d8d3bc5930447f732e3ae163fb1b7f70

Preconditions

• Kibana should be running.
• Alerts should be present.

Steps to Reproduce

• Navigate to Alerts page.
• Click on more option.
• Click on add rule exception .
• Navigate to comment section.
• The auto added comment should have "Alert Id" text in it because we get confuse the digit written belongs to which field.

Actual result

• While adding rule exception , the auto added comment not have "Alert Id" text in it.

Expected Result

• While adding rule exception , the auto added comment Should have "Alert Id" text in it.

Screen-Cast

[elasticmachine]

Pinging @elastic/security-solution (Team: SecuritySolution)

[vgomez-el]

@sukhwindersingh-qasource this PR, which contains some fixes to Auto populating the Exception flyout, will be included in 8.9 BC2, can you check if this issue still happens on that BC2 once it is ready, please? Thank you

[yctercero]

++ the text change was included in that PR that @vgomez-el linked to - just not sure if it made the BC2 cut.

[sukhwindersingh-qasource]

Hi @vgomez-el , @yctercero
We have validated this on the 8.9.0 BC2 and it is not Fixed ❌

Please find the below Testing Details:

Build info

VERSION: 8.9.0 BC2
BUILD: 64459
COMMIT: 6950a2b8207d8388ee8c842d6c0e2b1e1031fd36

Screen-Shot

We will validate this on BC3 once it will be available on productions.

Thanks!!

[yctercero]

@sukhwindersingh-qasource it should be in BC3, code is merged. Thanks!

[elasticmachine]

Pinging @elastic/security-detections-response (Team:Detections and Resp)

[ghost]

Hi @yctercero

Thanks for the update and Yes Issue is Fixed on BC3 ✔️ and _id text is now showing in comment section of Rule Exception.

Build Details:

Version: 8.9.0
Commit: fc463b96275c55dc44524f79f617b0026b7f8667
Build: 64584

Screen-Shot

Hence we are closing this issue and adding "QA:Validated" tag to it.

thanks !!