[Fleet] UI to edit custom pipeline|mappings

[nchaulet]

Summary

Related to #133740
In 8.4.0 we plan to allow user to specify custom ingest pipeline for their Fleet managed datastream, that PR allow to show and edit these pipeline in the Fleet UI.

For this we now show two table in the advanced options section of the datastream in the package policy editor:

• one for ingest pipeline
• one for mappings

The link to the mappings are not redirecting bath to the editor it's waiting on that PR to land #134910

In a following PRs I will add:

• a confirm modal to block user to navigate outside the policy editor if they did not save the policy
• reopen the datastream when navigating back to Fleet

Usage by custom package policy editor

We are going to expose DatastreamPipeline and DatastreamMappings name and props still subject to change that could be used by integration with custom policy editor to bring back that functionality.

<EuiFlexGroup>
    <EuiFlexItem>
      <PackagePolicyEditorDatastreamPipelines
        dataStream={packageInputStream.data_stream}
        packageInfo={packageInfo}
      />
    </EuiFlexItem>
    <EuiFlexItem>
      <PackagePolicyEditorDatastreamMappings
        dataStream={packageInputStream.data_stream}
        packageInfo={packageInfo}
      />
    </EuiFlexItem>
</EuiFlexGroup>

UI Changes

[joshdover]

Do we need to do anything else to export this for other plugins?

[nchaulet]

This will need to be in the plugin public/index file, I exposed them in the plugin public index file

[elasticmachine]

Pinging @elastic/fleet (Team:Fleet)

[nchaulet]

not related to that PR but I add to put this in a useMemo otherwise eslint was complaining

[kpollich]

Code all LGTM, but I have one question below.

When I click the "edit" icon for mappings on the first integration policy created (e.g. I haven't installed Nginx yet, but I'm creating my first policy), I get an error page.

Do you think there's a way to disable the button(s) if we haven't installed the package yet? Or maybe a better solution?

[nchaulet]

When I click the "edit" icon for mappings on the first integration policy created (e.g. I haven't installed Nginx yet, but I'm creating my first policy), I get an error page.

@kpollich good catch here, I am wondering if we should maybe not display that panel at all on package policy creation, in a coming PR I will add a confirm modal to block the user to navigate if it does not save the integration, and it seems the experience will be better if only show this on edition what do you think?

[kpollich]

When I click the "edit" icon for mappings on the first integration policy created (e.g. I haven't installed Nginx yet, but I'm creating my first policy), I get an error page.

@kpollich good catch here, I am wondering if we should maybe not display that panel at all on package policy creation, in a coming PR I will add a confirm modal to block the user to navigate if it does not save the integration, and it seems the experience will be better if only show this on edition what do you think?

This makes sense. So we'd only show these panels on edit view, not create view?

[nchaulet]

This makes sense. So we'd only show these panels on edit view, not create view?

Yes I think it will make more sense

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 396f7b3
• Storybooks Preview
• Webpack Bundle Analyzer

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
fleet	723	728	+5

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
fleet	1341	1368	+27

Async chunks

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

id	before	after	diff
fleet	839.9KB	833.5KB	-6.3KB

Page load bundle

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

id	before	after	diff
fleet	95.9KB	110.2KB	+14.3KB

Unknown metric groups

API count

id	before	after	diff
fleet	1468	1499	+31

ESLint disabled line counts

id	before	after	diff
fleet	53	52	-1

Total ESLint disabled count

id	before	after	diff
fleet	61	60	-1

History

• 💚 Build #52907 succeeded fa64b82
• 💔 Build #52890 failed a0cbfd5
• 💔 Build #52832 failed 6d2fa69
• 💚 Build #52542 succeeded 4f7a9cf
• 💔 Build #52539 failed b1eca36506404bac027119c9dcacf5dbe4cfc589
• 💔 Build #52533 failed 67b2e1e5142521f809c0b75bfbe8a1873d637f0d

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @nchaulet