[Backport 2.x] fix alert constructor with noop trigger to use executi…

[sbcd90]

…on id and workflow id

Issue #, if available:

Description of changes:
backport of pr #981

CheckList:

• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

Merging #994 (06fb662) into 2.x (c2250d5) will decrease coverage by 0.10%.
The diff coverage is 76.27%.

@@             Coverage Diff              @@
##                2.x     #994      +/-   ##
============================================
- Coverage     75.43%   75.33%   -0.10%     
  Complexity      116      116              
============================================
  Files           141      144       +3     
  Lines          8418     8519     +101     
  Branches       1203     1222      +19     
============================================
+ Hits           6350     6418      +68     
- Misses         1461     1486      +25     
- Partials        607      615       +8     

Impacted Files	Coverage Δ
...in/kotlin/org/opensearch/alerting/MonitorRunner.kt	66.66% <ø> (ø)
...lin/org/opensearch/alerting/alerts/AlertIndices.kt	75.29% <0.00%> (-0.40%)	⬇️
...org/opensearch/alerting/model/destination/Chime.kt	80.00% <ø> (ø)
...search/alerting/model/destination/CustomWebhook.kt	96.10% <ø> (ø)
...org/opensearch/alerting/model/destination/Slack.kt	80.00% <ø> (ø)
...rch/alerting/model/destination/email/EmailGroup.kt	90.12% <ø> (ø)
...erting/transport/TransportGetDestinationsAction.kt	86.30% <ø> (ø)
...h/alerting/transport/TransportGetFindingsAction.kt	89.18% <ø> (ø)
.../alerting/transport/TransportIndexMonitorAction.kt	56.34% <0.00%> (-3.43%)	⬇️
.../org/opensearch/alerting/util/AlertingException.kt	71.42% <ø> (ø)
... and 15 more

... and 4 files with indirect coverage changes