-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ML][Fleet] Link to ML assets from Integration > Assets tab #189767
Changes from all commits
796a79d
454e274
9c7e66a
70338f4
dd8e5b7
791863a
c024c1d
ca89c98
3e81b35
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -33,11 +33,9 @@ const getKibanaLinkForESAsset = (type: ElasticsearchAssetType, id: string): stri | |
case 'data_stream_ilm_policy': | ||
return `/app/management/data/index_lifecycle_management/policies/edit/${id}`; | ||
case 'transform': | ||
// TODO: Confirm link for transforms | ||
return ''; | ||
return `/app/management/data/transform?_a=(transform:(queryText:${id}))`; | ||
case 'ml_model': | ||
// TODO: Confirm link for ml models | ||
return ''; | ||
return `/app/ml/trained_models?_a=(trained_models:(queryText:'model_id:(${id})'))`; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same for models - would be nice to add the description, if it exists. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ❓ is it possible to utilize our ML locator to retrieve these URLs? It'd make it easier to maintain There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I was directed to keep it consistent in that file, for now. If maintainability becomes an issue then we can bring it up. |
||
default: | ||
return ''; | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might not be possible, but adding the description of the transform, if set, would be nice here, as happens for dashboard assets:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently for ES assets that isn't possible but I can create a follow up issue to allow it for ES assets.