[Response Ops][Reporting] Adding DISCOVER_APP_LOCATOR to REPORTING_REDIRECT_ALLOWED_LOCATOR_TYPES

[ymao1]

Summary

It was noticed during verification that generating a ES|QL CSV report (either single or scheduled) and then trying to access the Open Dashboard link from the Reporting UI would result in a redirect error:

The locator params for this look like, so it looks like DISCOVER_APP_LOCATOR needs to be added to the REPORTING_REDIRECT_ALLOWED_LOCATOR_TYPES, similar to this previous PR for the CANVAS_APP_LOCATOR

{
    "id": "DISCOVER_APP_LOCATOR",
    "params": {
        "columns": [],
        "dataViewSpec": {
            "allowHidden": false,
            "allowNoIndex": false,
            "fieldFormats": {},
            "id": "bc48ed34f1056457dd1c77b6326d7226c302ea35aecfae268cd142a126bb3143",
            "name": "logs-library_endgame-default",
            "runtimeFieldMap": {},
            "sourceFilters": [],
            "timeFieldName": "@timestamp",
            "title": "logs-library_endgame-default",
            "type": "esql"
        },
        "filters": [],
        "hideChart": false,
        "interval": "auto",
        "query": {
            "esql": "FROM logs-library_endgame-default | LIMIT 10"
        },
        "refreshInterval": {
            "pause": true,
            "value": 60000
        },
        "sort": [
            [
                "@timestamp",
                "desc"
            ]
        ],
        "timeRange": {
            "from": "2024-10-30T14:28:39.718Z",
            "to": "2024-11-04T19:35:40.254Z"
        }
    }
}

To Verify

1. Export a CSV from Discover in ES|QL mode (does not have to be a scheduled export)
2. Navigate to the Reporting UI and click the hamburger menu > Open in Dashboard
3. This should navigate you back to Discover in ES|QL mode instead of showing the above error.

[elasticmachine]

Pinging @elastic/response-ops (Team:ResponseOps)

[tsullivan]

Reviewed the code only, LGTM to unblock

[nreese]

kibana-presentation changes LGTM
code review only

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: 87a67bc

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id	before	after	diff
@kbn/deeplinks-analytics	10	11	+1

Async chunks

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

id	before	after	diff
fleet	2.1MB	2.1MB	+23.0B
ml	5.4MB	5.4MB	-7.0KB
reporting	159.3KB	159.4KB	+113.0B
total			-6.9KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id	before	after	diff
ml	85.7KB	85.7KB	+2.0B
reporting	54.0KB	54.0KB	+1.0B
total			+3.0B

Unknown metric groups

API count

id	before	after	diff
@kbn/deeplinks-analytics	10	11	+1

cc @ymao1

[davismcphee]

Data Discovery changes LGTM, thanks for fixing it 👍

[kibanamachine]

Starting backport for target branches: 8.18, 8.19, 9.0, 9.1

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

[kibanamachine]

💔 Some backports could not be created

Status	Branch	Result
❌	8.18	Backport failed because of merge conflicts
✅	8.19
❌	9.0	Backport failed because of merge conflicts
✅	9.1

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 227297

Questions ?

Please refer to the Backport tool documentation

[ymao1]

💚 All backports created successfully

Status	Branch	Result
✅	9.0
✅	8.18

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

Questions ?

Please refer to the Backport tool documentation