Skip to content
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

Upgrade markdown-it dependency (10.0.012.3.2). #125526

Merged
merged 9 commits into from
Feb 21, 2022

Conversation

azasypkin
Copy link
Member

@azasypkin azasypkin commented Feb 14, 2022

Summary

Upgrade markdown-it dependency (10.0.012.3.2).

Change log: https://github.com/markdown-it/markdown-it/blob/master/CHANGELOG.md

Note for reviewers: I haven't noticed any significant breaking changes, but would appreciate another pair of eyes on the code you own (kibana_react/public/markdown/markdown.tsx, vis_types/vislib/public/vislib/lib/handler.js, vis_types/vega/public/vega_view/vega_map_view/service_settings/service_settings.ts, and actions/server/builtin_action_types/lib/send_email.ts).

@azasypkin azasypkin added chore release_note:skip Skip the PR/issue when compiling release notes v7.17.1 v8.2.0 labels Feb 14, 2022
@azasypkin azasypkin marked this pull request as ready for review February 14, 2022 17:17
@azasypkin azasypkin requested review from a team February 14, 2022 17:19
@flash1293
Copy link
Contributor

@elasticmachine merge upstream

Copy link
Contributor

@flash1293 flash1293 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested and VisEditors stuff still works 👍

Copy link
Contributor

@jportner jportner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nits below to deduplicate packages!

Edit: I also opened #125712, hopefully we can automate this process in the near future.

yarn.lock Outdated
Comment on lines 10352 to 10364
browserslist@^4.14.5, browserslist@^4.19.1:
version "4.19.1"
resolved "https://registry.yarnpkg.com/browserslist/-/browserslist-4.19.1.tgz#4ac0435b35ab655896c31d53018b6dd5e9e4c9a3"
integrity sha512-u2tbbG5PdKRTUoctO3NBD8FQ5HdPh1ZXPHzp1rwaa5jTc+RV9/+RlWiAIKmjRPQF+xbGM9Kklj5bZQFa2s/38A==
dependencies:
caniuse-lite "^1.0.30001286"
electron-to-chromium "^1.4.17"
escalade "^3.1.1"
node-releases "^2.0.1"
picocolors "^1.0.0"

browserslist@^4.17.5, browserslist@^4.17.6:
version "4.18.1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: you could collapse these two dependencies into one

yarn.lock Outdated
Comment on lines 19464 to 19470
resolved "https://registry.yarnpkg.com/jquery/-/jquery-3.5.0.tgz#9980b97d9e4194611c36530e7dc46a58d7340fc9"
integrity sha512-Xb7SVYMvygPxbFMpTFQiHh1J7HClEaThguL15N/Gg37Lri/qKyhRGZYzHRyLH8Stq3Aow0LsHO2O2ci86fCrNQ==

jquery@^3.6.0:
version "3.6.0"
resolved "https://registry.yarnpkg.com/jquery/-/jquery-3.6.0.tgz#c72a09f15c1bdce142f49dbf1170bdf8adac2470"
integrity sha512-JVzAR/AjBvVt2BmYhxRCSYysDsPcssdmTFnzyLEts9qNwmjmu4JTAMYubEfwVOSwpQ1I1sKKFcxhZCI2buerfw==
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like jquery@^3.6.0 and jquery@^3.5.0 above can be collapsed into a single dependency.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Someone still uses jquery, oh no 🙈

yarn.lock Outdated
Comment on lines 22582 to 22590
dependencies:
mimic-fn "^2.1.0"

onetime@^5.1.2:
version "5.1.2"
resolved "https://registry.yarnpkg.com/onetime/-/onetime-5.1.2.tgz#d0e96ebb56b07476df1dd9c4806e5237985ca45e"
integrity sha512-kbpaSSGJTWdAY5KPVeMOKXSrPtr8C8C7wodJbcsd51jRnmD+GZu8Y0VoU6Dm5Z4vWr0Ig/1NKuWRKf7j5aaYSg==
dependencies:
mimic-fn "^2.1.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like onetime@^5.1.2 and onetime@^5.1.0 above can be collapsed into a single dependency.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, not sure, it's onetime not twotimes! 🙂

yarn.lock Outdated
Comment on lines 26769 to 26779
ajv "^6.12.5"
ajv-keywords "^3.5.2"

schema-utils@^3.1.0, schema-utils@^3.1.1:
version "3.1.1"
resolved "https://registry.yarnpkg.com/schema-utils/-/schema-utils-3.1.1.tgz#bc74c4b6b6995c1d88f76a8b77bea7219e0c8281"
integrity sha512-Y5PQxS4ITlC+EahLuXaY86TXfR7Dc5lw294alXOq86JAHCihAIZfqv8nNCWvaEJvaC51uN9hbLGeV0cFBdH+Fw==
dependencies:
"@types/json-schema" "^7.0.8"
ajv "^6.12.5"
ajv-keywords "^3.5.2"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like schema-utils@^3.1.0, schema-utils@^3.1.1 and schema-utils@^3.0.0 above can be collapsed into a single dependency.

yarn.lock Outdated
Comment on lines 26457 to 26887
semver@^7.2.1, semver@^7.3.2, semver@~7.3.2:
semver@^7.2.1, semver@^7.3.2:
version "7.3.2"
resolved "https://registry.yarnpkg.com/semver/-/semver-7.3.2.tgz#604962b052b81ed0786aae84389ffba70ffd3938"
integrity sha512-OrOb32TeeambH6UrhtShmF7CRDqhL6/5XpPNp2DuRH6+9QLw/orhp72j87v8Qa1ScDkvrrBNpZcDejAirJmfXQ==
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like semver@^7.2.1, semver@^7.3.2 and semver@^7.3.4, semver@^7.3.5, semver@~7.3.0 below can be collapsed into a single dependency.

yarn.lock Outdated
Comment on lines 27169 to 27175
resolved "https://registry.yarnpkg.com/signal-exit/-/signal-exit-3.0.2.tgz#b5fdc08f1287ea1178628e415e25132b73646c6d"
integrity sha1-tf3AjxKH6hF4Yo5BXiUTK3NkbG0=

signal-exit@^3.0.3:
version "3.0.7"
resolved "https://registry.yarnpkg.com/signal-exit/-/signal-exit-3.0.7.tgz#a9a1767f8af84155114eaabd73f99273c8f59ad9"
integrity sha512-wnD2ZE+l+SPC/uoS0vXeE9L1+0wuaMqKlfz9AMUo38JsyLSBWSFcHR1Rri62LZc12vLr1gb3jl7iwQhgwpAbGQ==
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like signal-exit@^3.0.3 and signal-exit@^3.0.0, signal-exit@^3.0.2 above can be collapsed into a single dependency.

yarn.lock Outdated
Comment on lines 27475 to 27484
buffer-from "^1.0.0"
source-map "^0.6.0"

source-map-support@~0.5.20:
version "0.5.21"
resolved "https://registry.yarnpkg.com/source-map-support/-/source-map-support-0.5.21.tgz#04fe7c7f9e1ed2d662233c28cb2b35b9f63f6e4f"
integrity sha512-uBHU3L3czsIyYXKX88fdrGovxdSCoTGDRZ6SYXtSRxLZUzHg5P/66Ht6uoUlHu9EZod+inXhKo3qQgwXUT/y1w==
dependencies:
buffer-from "^1.0.0"
source-map "^0.6.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like source-map-support@~0.5.20, source-map-support@^0.5.20 above, and source-map-support@^0.5.16, source-map-support@^0.5.19, source-map-support@^0.5.6, source-map-support@~0.5.12, source-map-support@~0.5.19 above that can be collapsed into a single dependency.

yarn.lock Outdated
Comment on lines 28867 to 28877
source-map "~0.7.2"
source-map-support "~0.5.19"

terser@^5.7.2:
version "5.10.0"
resolved "https://registry.yarnpkg.com/terser/-/terser-5.10.0.tgz#b86390809c0389105eb0a0b62397563096ddafcc"
integrity sha512-AMmF99DMfEDiRJfxfY5jj5wNH/bYO09cniSqhfoyxc8sFoYIgkJy86G04UoZU5VjlpnplVu0K6Tx6E9b5+DlHA==
dependencies:
commander "^2.20.0"
source-map "~0.7.2"
source-map-support "~0.5.20"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like terser@^5.7.2 and terser@^5.3.4, terser@^5.7.1 above can be collapsed into a single dependency.

yarn.lock Outdated
Comment on lines 31457 to 31463
triple-beam "^1.3.0"
winston-transport "^4.4.0"

winston@^3.5.1:
winston@^3.4.0, winston@^3.5.1:
version "3.5.1"
resolved "https://registry.yarnpkg.com/winston/-/winston-3.5.1.tgz#b25cc899d015836dbf8c583dec8c4c4483a0da2e"
integrity sha512-tbRtVy+vsSSCLcZq/8nXZaOie/S2tPXPFt4be/Q3vI/WtYwm7rrwidxVw2GRa38FIXcJ1kUM6MOZ9Jmnk3F3UA==
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like winston@^3.4.0, winston@^3.5.1 and winston@^3.0.0, winston@^3.3.3 above can be collapsed into a single dependency.

@jportner
Copy link
Contributor

jportner commented Feb 15, 2022

I pulled down these changes and ran yarn kbn bootstrap locally on my Apple Silicon machine, and I got a bunch of bazel warnings:

 info [bazel] [3/5] Fetching packages...
 info [bazel] info esbuild-android-arm64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-android-arm64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-darwin-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info "esbuild-darwin-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-freebsd-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-freebsd-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-freebsd-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-freebsd-arm64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-freebsd-arm64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-32@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-32@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-32@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-arm@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-arm@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-arm@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-arm64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-arm64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-mips64le@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-mips64le@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-mips64le@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-ppc64le@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-ppc64le@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-ppc64le@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-riscv64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-riscv64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-riscv64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-linux-s390x@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-linux-s390x@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-linux-s390x@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-netbsd-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-netbsd-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-netbsd-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-openbsd-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-openbsd-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-openbsd-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-sunos-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-sunos-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-sunos-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-windows-32@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-windows-32@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-windows-32@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-windows-64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-windows-64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.
 info [bazel] info esbuild-windows-64@0.14.21: The CPU architecture "arm64" is incompatible with this module.
 info [bazel] info esbuild-windows-arm64@0.14.21: The platform "darwin" is incompatible with this module.
 info [bazel] info "esbuild-windows-arm64@0.14.21" is an optional dependency and failed compatibility check. Excluding it from installation.

I was wondering if this might break the yarn build:apidocs script but I can't get it working on this branch or on main 🙈 digging into it further...

Edit: I was able to eventually build the API docs using node --max-old-space-size=10000 scripts/build_api_docs.js, both on this branch and main. So I think the incompatible optional dependencies are not problematic 👍

@azasypkin azasypkin requested a review from a team as a code owner February 16, 2022 10:04
@azasypkin azasypkin requested review from jportner and removed request for a team February 16, 2022 13:11
@pmuellr
Copy link
Member

pmuellr commented Feb 16, 2022

I looked at the markdown-it CHANGELOG.md, and didn't see anything that would affect us.

For Kibana alerting, we have some very simple jest and functional tests for our markdown usage. Basically just to make sure the basic rendering works; so since checks have passed, the basic functionality still seems to work:

it('should render html from markdown', async () => {
await supertest
.post(`/api/actions/connector/${createdActionId}/_execute`)
.set('kbn-xsrf', 'foo')
.send({
params: {
to: ['kibana-action-test@elastic.co'],
subject: 'message with markdown',
message: '_italic_ **bold** https://elastic.co link',
},
})
.expect(200)
.then((resp: any) => {
const { text, html } = resp.body.data.message;
expect(text).to.eql(
'_italic_ **bold** https://elastic.co link\n\n--\n\nThis message was sent by Kibana. [Go to Kibana](https://localhost:5601).'
);
expect(html).to.eql(
`<p><em>italic</em> <strong>bold</strong> <a href="https://elastic.co">https://elastic.co</a> link</p>\n<p>--</p>\n<p>This message was sent by Kibana. <a href=\"https://localhost:5601\">Go to Kibana</a>.</p>\n`
);
});
});

expect(sendMailMock.mock.calls[0]).toMatchInlineSnapshot(`
Array [
Object {
"bcc": Array [],
"cc": Array [
"bob@example.com",
"robert@example.com",
],
"from": "fred@example.com",
"html": "<p>a message</p>
",
"subject": "a subject",
"text": "a message",
"to": Array [
"jim@example.com",
],
},
]
`);

Copy link
Member

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, for Kibana alerting

I can't really comment on the change to packages/kbn-pm/dist/index.js though - I assume there was some kind of change requiring rebuild of the webpack bits, and it got rebuilt with a newer version of webpack, or something.

I added a comment to the PR pointing out the locations Kibana alerting is doing some basic markdown tests already.

@spalger
Copy link
Contributor

spalger commented Feb 16, 2022

Do we need to update apidoc-markdown too? It seems it's only used by ML in a build scripts and is the reason esbuild is being installed, which is a pretty massive dependency and responsible for all the errors that @jportner mentioned.

@azasypkin
Copy link
Member Author

Do we need to update apidoc-markdown too? It seems it's only used by ML in a build scripts and is the reason esbuild is being installed, which is a pretty massive dependency and responsible for all the errors that @jportner mentioned.

I assume you meant apidoc since it's the one that needs esbuild - sure, I can skip upgrading apidoc in this PR if it's causing troubles for Apple Silicon machines.

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1130 1131 +1
dashboard 261 265 +4
data 530 531 +1
fleet 569 570 +1
indexManagement 479 480 +1
kibanaUtils 159 160 +1
lens 747 748 +1
mapsEms 70 72 +2
ml 1611 1612 +1
monitoring 496 497 +1
osquery 246 250 +4
securitySolution 2852 2853 +1
snapshotRestore 183 184 +1
telemetry 29 30 +1
upgradeAssistant 185 186 +1
total +22

Async chunks

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

id before after diff
advancedSettings 40.6KB 40.6KB -14.0B
apm 2.7MB 2.7MB +116.0B
canvas 1.0MB 1.0MB -175.0B
cases 294.9KB 295.0KB +37.0B
console 379.6KB 379.5KB -18.0B
controls 379.6KB 379.6KB +28.0B
core 132.9KB 133.0KB +20.0B
crossClusterReplication 163.2KB 163.2KB -35.0B
customIntegrations 3.2KB 3.2KB +4.0B
dashboard 293.2KB 301.7KB +8.6KB
data 116.6KB 116.7KB +75.0B
dataViewEditor 115.5KB 115.5KB +4.0B
dataViewFieldEditor 148.8KB 148.8KB -39.0B
dataViewManagement 119.8KB 119.8KB +12.0B
dataVisualizer 539.9KB 539.8KB -87.0B
discover 337.1KB 337.1KB +12.0B
enterpriseSearch 1.3MB 1.3MB +46.0B
expressionPartitionVis 47.0KB 47.0KB +24.0B
expressions 45.1KB 45.1KB +26.0B
fileUpload 824.2KB 824.0KB -296.0B
fleet 661.2KB 661.2KB -6.0B
graph 448.8KB 448.8KB +14.0B
indexManagement 515.1KB 515.1KB -14.0B
infra 928.0KB 927.8KB -133.0B
ingestPipelines 452.4KB 452.3KB -46.0B
inputControlVis 77.8KB 77.9KB +39.0B
kibanaReact 211.7KB 212.7KB +931.0B
kibanaUtils 52.1KB 52.1KB +4.0B
lens 1.1MB 1.1MB +53.0B
lists 140.8KB 140.7KB -52.0B
maps 2.5MB 2.5MB +335.0B
mapsEms 76.9KB 81.5KB +4.7KB
ml 3.5MB 3.5MB +106.0B
monitoring 471.4KB 471.6KB +161.0B
observability 394.8KB 394.7KB -34.0B
osquery 941.6KB 950.1KB +8.5KB
remoteClusters 83.5KB 83.5KB -5.0B
rollup 129.0KB 129.0KB -5.0B
security 494.5KB 494.5KB +15.0B
securitySolution 4.7MB 4.7MB +35.0B
snapshotRestore 258.3KB 258.4KB +87.0B
timelines 226.0KB 226.0KB -17.0B
transform 375.4KB 375.4KB +24.0B
triggersActionsUi 678.1KB 678.1KB -44.0B
uiActionsEnhanced 127.5KB 126.9KB -604.0B
upgradeAssistant 168.4KB 168.2KB -143.0B
uptime 740.2KB 740.1KB -23.0B
ux 161.7KB 161.6KB -46.0B
visTypeTimelion 121.4KB 121.4KB +34.0B
visTypeTimeseries 462.7KB 462.1KB -626.0B
visTypeVega 2.0MB 2.0MB +990.0B
visTypeVislib 354.9KB 356.1KB +1.2KB
visTypeXy 60.3KB 60.3KB -13.0B
visualizations 157.2KB 157.8KB +630.0B
watcher 272.6KB 272.6KB -34.0B
total +24.2KB

Page load bundle

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

id before after diff
alerting 37.9KB 37.9KB -37.0B
cases 70.4KB 70.3KB -46.0B
core 292.7KB 292.7KB -21.0B
dashboard 65.8KB 66.0KB +114.0B
dashboardEnhanced 41.8KB 41.8KB +20.0B
data 452.6KB 452.7KB +16.0B
discover 50.5KB 50.5KB -37.0B
esUiShared 122.5KB 122.5KB -7.0B
expressions 87.2KB 87.5KB +305.0B
fieldFormats 47.8KB 47.8KB +4.0B
fleet 110.3KB 110.3KB +73.0B
globalSearchProviders 4.6KB 4.6KB +19.0B
indexManagement 27.1KB 27.2KB +114.0B
infra 89.2KB 89.1KB -46.0B
inspector 22.3KB 22.6KB +305.0B
interactiveSetup 58.4KB 58.4KB +4.0B
kbnUiSharedDeps-npmDll 4.8MB 4.8MB -265.0B
kbnUiSharedDeps-srcJs 3.8MB 3.8MB -69.0B
kibanaUtils 67.5KB 67.9KB +419.0B
maps 65.9KB 65.9KB +20.0B
observability 86.9KB 87.0KB +20.0B
remoteClusters 7.8KB 7.8KB -8.0B
reporting 38.4KB 38.4KB -47.0B
runtimeFields 11.0KB 11.0KB -41.0B
security 50.1KB 50.1KB +4.0B
securitySolution 245.3KB 245.3KB -17.0B
snapshotRestore 26.7KB 26.7KB -8.0B
spaces 20.3KB 20.3KB +8.0B
telemetry 25.1KB 25.2KB +83.0B
timelines 136.4KB 136.3KB -41.0B
uiActions 19.3KB 19.6KB +305.0B
upgradeAssistant 18.8KB 18.9KB +114.0B
urlDrilldown 15.0KB 15.0KB +23.0B
visTypeTimeseries 15.9KB 15.9KB +4.0B
visTypeVislib 19.5KB 19.5KB +10.0B
visTypeXy 42.8KB 42.9KB +13.0B
total +1.3KB
Unknown metric groups

miscellaneous assets size

id before after diff
maps 530.1KB 530.0KB -34.0B
visTypeVega 530.1KB 530.0KB -34.0B
total -68.0B

History

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

Copy link
Contributor

@jportner jportner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay, I think I screwed up my notifications. LGTM!

@azasypkin azasypkin added the auto-backport Deprecated - use backport:version if exact versions are needed label Feb 21, 2022
@azasypkin azasypkin merged commit 847c1eb into elastic:main Feb 21, 2022
@azasypkin azasypkin deleted the issue-xxx-upgrade-markdown-it branch February 21, 2022 14:48
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
7.17 Backport failed because of merge conflicts

How to fix

Re-run the backport manually:

node scripts/backport --pr 125526

Questions ?

Please refer to the Backport tool documentation

academo pushed a commit to academo/kibana that referenced this pull request Feb 22, 2022
lucasfcosta pushed a commit to lucasfcosta/kibana that referenced this pull request Mar 2, 2022
academo added a commit that referenced this pull request Mar 3, 2022
* WIP

* WIP2

* Use new cases context hooks to open and close the flyout

* Update timelines to use new hooks

* CLose flyout on create success

* Add back sucess toast

* Move code to a dedicated component

* Add CasesContext to observability

* Remove dependency

* Small refactor

* Use observabilityAppId instead of observabilityFeatureId for buttons

* Add CasesContext to timetable

* Fix detection engine test cases

* Fix broken tests

* Fix broken tests

* Rename hook

* Add test cases for cases context ui

* Add test for new hook

* Remove state from the provider context

* Remove basevalue

* apply suggested renaming

* Add usecallback

* Add reducer types, fix test type, remove redundant check

* Accept attachments as a prop for the cases select modal

* Expose useCasesAddToExistingCase hook, reducer code and global component

* use the new hook to open the select cases modasl

* Fix tests and types

* Add tests for cases global components

* [Fleet] showing agent policy creation error message on UI (#125931)

* showing agent policy creation error message on UI

* mapping the error instead of showing from the backend

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [ResponseOps] Adds tooltip to time window selector in ES query rule flyout (#125764)

* [Lens] Allow detaching from global time range (#125563)

* allow detaching from global time range

* add test

* fix time field recognition

* fix tests

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Fleet] refactor auto upgrade package policies logic (#125909)

* refactor upgrade package policies

* fixed tests

* code cleanup

* review improvements

* added api test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* skip flaky suite (#126027)

* Remove deprecated api (#125524)

* [Fleet] Remove deprecated kibana APIs - License

* Remove basePath from FleetApp

* Replace AsyncPlugin with Plugin

* Get fieldFormats from fieldFormats plugin rather than data plugin

* Fix ts errors

* Attempt fixing wrong type

* Move licenseService to FleetStartDeps

* Fix types and mocks

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Upgrade `markdown-it` dependency (`10.0.0` → `12.3.2`). (#125526)

* skipping failing tests (#126039)

* remove unused deprecated code and use field format plugin directly for data view field editor (#126029)

* [data views] Improve preview pane (#126013)

* fix preview pane

* fix preview pane

* one less span tag

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Alerting] Provide services to set context for recovered alerts (#124972)

* Rename alert instance to alert and add create fn to alert factory

* Rename alert instance to alert and add create fn to alert factory

* Fixing types

* Fixing types

* Adding flag for rule types to opt into setting recovery context

* Only showing context in action variable menu if flag set to true

* Adding recovery functions to createAlertFactory

* Setting recovery in index threshold and fixing types

* Fixing lint issues and some refactoring

* Cleanup

* Functional tests for index threshold rule recovery context

* Return array of recovered alerts instead of record

* Fixing types

* Fixing types

* Cleanup

* Handling nulls and more tests

* Updating developer docs

* Making getRecoveryAlerts non-optional

* Setting unknown in index threshold recovery value

* PR feedback

* Adding a test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Discover] Re-introduce saved_searches test (#126059)

* [Archive Migration]  index pattern without timefield (#125870)

* kbn_archive date_nanos

* kbn_archive date_nanos in context and discover

* kbn_archiver more date_nanos tests

* split out kbnArchiver for index_pattern_without_timefield

* remove date_nanos files from a different PR

* update another test for usage of the same archives

* set default index pattern for test

* remove duplicate const kibanaServer

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* delete unused es_archive visualize_embedding (#126001)

* delete unused es_archive

* remove other unused es_archive

* more unused es_archives

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Bump packages (#126119)

* url-parse 1.5.3 -> 1.5.9
* follow-redirects 1.y.z -> 1.14.9

* Add tests for all cases selector and attachments

* Add tests for use add to existing case hook

* First version of the cases timeline actions

* export add alert to new case button from cases plugin

* Make Cases ECS compatible with timelines and security_solution

* Delete new case button

* Add helpers

* Use the cases hook directly for add to new case

* Remov unused dependencies

* Rename callbacks, remove timelines calls

* Fixing tests for the dropdown

* Fix broken test

* mocking cases for tests

* Fix detectiosn tests

* Observability now uses the new cases hooks

* Wrap events viewer into cases context

* Open the create case flyout if create case was selected in the modal

* Fix cases mocks for security_solution

* Update tests

* Add tests for use cases toast

* Improve cases mock

* delete security mock

* replace tests mocks for cases

* fix import mock

* Do not require onRowClick

* Show the toast inside the modal

* show the toast inside the flyout

* remove toast logic from the consumer plugin

* fix typescript types

* Rename type

* Fix broken test

* Fix file name and broken test

* Use internal navigation hook

* Update hook dependencies

* Move useCaseToast

* Fix mock paths

* fix eslint

* Add test cases for the toast content

* Add cases context to the overview page

Co-authored-by: Julia Bardi <90178898+juliaElastic@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: István Zoltán Szabó <szabosteve@gmail.com>
Co-authored-by: Joe Reuter <johannes.reuter@elastic.co>
Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
Co-authored-by: Cristina Amico <criamico@users.noreply.github.com>
Co-authored-by: Aleh Zasypkin <aleh.zasypkin@elastic.co>
Co-authored-by: Gloria Hornero <gloria.hornero@elastic.co>
Co-authored-by: Matthew Kime <matt@mattki.me>
Co-authored-by: Ying Mao <ying.mao@elastic.co>
Co-authored-by: Maja Grubic <maja.grubic@elastic.co>
Co-authored-by: Lee Drengenberg <lee.drengenberg@elastic.co>
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
lucasfcosta pushed a commit to lucasfcosta/kibana that referenced this pull request Mar 8, 2022
…#126265)

* WIP

* WIP2

* Use new cases context hooks to open and close the flyout

* Update timelines to use new hooks

* CLose flyout on create success

* Add back sucess toast

* Move code to a dedicated component

* Add CasesContext to observability

* Remove dependency

* Small refactor

* Use observabilityAppId instead of observabilityFeatureId for buttons

* Add CasesContext to timetable

* Fix detection engine test cases

* Fix broken tests

* Fix broken tests

* Rename hook

* Add test cases for cases context ui

* Add test for new hook

* Remove state from the provider context

* Remove basevalue

* apply suggested renaming

* Add usecallback

* Add reducer types, fix test type, remove redundant check

* Accept attachments as a prop for the cases select modal

* Expose useCasesAddToExistingCase hook, reducer code and global component

* use the new hook to open the select cases modasl

* Fix tests and types

* Add tests for cases global components

* [Fleet] showing agent policy creation error message on UI (elastic#125931)

* showing agent policy creation error message on UI

* mapping the error instead of showing from the backend

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [ResponseOps] Adds tooltip to time window selector in ES query rule flyout (elastic#125764)

* [Lens] Allow detaching from global time range (elastic#125563)

* allow detaching from global time range

* add test

* fix time field recognition

* fix tests

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Fleet] refactor auto upgrade package policies logic (elastic#125909)

* refactor upgrade package policies

* fixed tests

* code cleanup

* review improvements

* added api test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* skip flaky suite (elastic#126027)

* Remove deprecated api (elastic#125524)

* [Fleet] Remove deprecated kibana APIs - License

* Remove basePath from FleetApp

* Replace AsyncPlugin with Plugin

* Get fieldFormats from fieldFormats plugin rather than data plugin

* Fix ts errors

* Attempt fixing wrong type

* Move licenseService to FleetStartDeps

* Fix types and mocks

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Upgrade `markdown-it` dependency (`10.0.0` → `12.3.2`). (elastic#125526)

* skipping failing tests (elastic#126039)

* remove unused deprecated code and use field format plugin directly for data view field editor (elastic#126029)

* [data views] Improve preview pane (elastic#126013)

* fix preview pane

* fix preview pane

* one less span tag

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Alerting] Provide services to set context for recovered alerts (elastic#124972)

* Rename alert instance to alert and add create fn to alert factory

* Rename alert instance to alert and add create fn to alert factory

* Fixing types

* Fixing types

* Adding flag for rule types to opt into setting recovery context

* Only showing context in action variable menu if flag set to true

* Adding recovery functions to createAlertFactory

* Setting recovery in index threshold and fixing types

* Fixing lint issues and some refactoring

* Cleanup

* Functional tests for index threshold rule recovery context

* Return array of recovered alerts instead of record

* Fixing types

* Fixing types

* Cleanup

* Handling nulls and more tests

* Updating developer docs

* Making getRecoveryAlerts non-optional

* Setting unknown in index threshold recovery value

* PR feedback

* Adding a test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Discover] Re-introduce saved_searches test (elastic#126059)

* [Archive Migration]  index pattern without timefield (elastic#125870)

* kbn_archive date_nanos

* kbn_archive date_nanos in context and discover

* kbn_archiver more date_nanos tests

* split out kbnArchiver for index_pattern_without_timefield

* remove date_nanos files from a different PR

* update another test for usage of the same archives

* set default index pattern for test

* remove duplicate const kibanaServer

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* delete unused es_archive visualize_embedding (elastic#126001)

* delete unused es_archive

* remove other unused es_archive

* more unused es_archives

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Bump packages (elastic#126119)

* url-parse 1.5.3 -> 1.5.9
* follow-redirects 1.y.z -> 1.14.9

* Add tests for all cases selector and attachments

* Add tests for use add to existing case hook

* First version of the cases timeline actions

* export add alert to new case button from cases plugin

* Make Cases ECS compatible with timelines and security_solution

* Delete new case button

* Add helpers

* Use the cases hook directly for add to new case

* Remov unused dependencies

* Rename callbacks, remove timelines calls

* Fixing tests for the dropdown

* Fix broken test

* mocking cases for tests

* Fix detectiosn tests

* Observability now uses the new cases hooks

* Wrap events viewer into cases context

* Open the create case flyout if create case was selected in the modal

* Fix cases mocks for security_solution

* Update tests

* Add tests for use cases toast

* Improve cases mock

* delete security mock

* replace tests mocks for cases

* fix import mock

* Do not require onRowClick

* Show the toast inside the modal

* show the toast inside the flyout

* remove toast logic from the consumer plugin

* fix typescript types

* Rename type

* Fix broken test

* Fix file name and broken test

* Use internal navigation hook

* Update hook dependencies

* Move useCaseToast

* Fix mock paths

* fix eslint

* Add test cases for the toast content

* Add cases context to the overview page

Co-authored-by: Julia Bardi <90178898+juliaElastic@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: István Zoltán Szabó <szabosteve@gmail.com>
Co-authored-by: Joe Reuter <johannes.reuter@elastic.co>
Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
Co-authored-by: Cristina Amico <criamico@users.noreply.github.com>
Co-authored-by: Aleh Zasypkin <aleh.zasypkin@elastic.co>
Co-authored-by: Gloria Hornero <gloria.hornero@elastic.co>
Co-authored-by: Matthew Kime <matt@mattki.me>
Co-authored-by: Ying Mao <ying.mao@elastic.co>
Co-authored-by: Maja Grubic <maja.grubic@elastic.co>
Co-authored-by: Lee Drengenberg <lee.drengenberg@elastic.co>
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
lucasfcosta pushed a commit to lucasfcosta/kibana that referenced this pull request Mar 8, 2022
…#126265)

* WIP

* WIP2

* Use new cases context hooks to open and close the flyout

* Update timelines to use new hooks

* CLose flyout on create success

* Add back sucess toast

* Move code to a dedicated component

* Add CasesContext to observability

* Remove dependency

* Small refactor

* Use observabilityAppId instead of observabilityFeatureId for buttons

* Add CasesContext to timetable

* Fix detection engine test cases

* Fix broken tests

* Fix broken tests

* Rename hook

* Add test cases for cases context ui

* Add test for new hook

* Remove state from the provider context

* Remove basevalue

* apply suggested renaming

* Add usecallback

* Add reducer types, fix test type, remove redundant check

* Accept attachments as a prop for the cases select modal

* Expose useCasesAddToExistingCase hook, reducer code and global component

* use the new hook to open the select cases modasl

* Fix tests and types

* Add tests for cases global components

* [Fleet] showing agent policy creation error message on UI (elastic#125931)

* showing agent policy creation error message on UI

* mapping the error instead of showing from the backend

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [ResponseOps] Adds tooltip to time window selector in ES query rule flyout (elastic#125764)

* [Lens] Allow detaching from global time range (elastic#125563)

* allow detaching from global time range

* add test

* fix time field recognition

* fix tests

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Fleet] refactor auto upgrade package policies logic (elastic#125909)

* refactor upgrade package policies

* fixed tests

* code cleanup

* review improvements

* added api test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* skip flaky suite (elastic#126027)

* Remove deprecated api (elastic#125524)

* [Fleet] Remove deprecated kibana APIs - License

* Remove basePath from FleetApp

* Replace AsyncPlugin with Plugin

* Get fieldFormats from fieldFormats plugin rather than data plugin

* Fix ts errors

* Attempt fixing wrong type

* Move licenseService to FleetStartDeps

* Fix types and mocks

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Upgrade `markdown-it` dependency (`10.0.0` → `12.3.2`). (elastic#125526)

* skipping failing tests (elastic#126039)

* remove unused deprecated code and use field format plugin directly for data view field editor (elastic#126029)

* [data views] Improve preview pane (elastic#126013)

* fix preview pane

* fix preview pane

* one less span tag

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Alerting] Provide services to set context for recovered alerts (elastic#124972)

* Rename alert instance to alert and add create fn to alert factory

* Rename alert instance to alert and add create fn to alert factory

* Fixing types

* Fixing types

* Adding flag for rule types to opt into setting recovery context

* Only showing context in action variable menu if flag set to true

* Adding recovery functions to createAlertFactory

* Setting recovery in index threshold and fixing types

* Fixing lint issues and some refactoring

* Cleanup

* Functional tests for index threshold rule recovery context

* Return array of recovered alerts instead of record

* Fixing types

* Fixing types

* Cleanup

* Handling nulls and more tests

* Updating developer docs

* Making getRecoveryAlerts non-optional

* Setting unknown in index threshold recovery value

* PR feedback

* Adding a test

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* [Discover] Re-introduce saved_searches test (elastic#126059)

* [Archive Migration]  index pattern without timefield (elastic#125870)

* kbn_archive date_nanos

* kbn_archive date_nanos in context and discover

* kbn_archiver more date_nanos tests

* split out kbnArchiver for index_pattern_without_timefield

* remove date_nanos files from a different PR

* update another test for usage of the same archives

* set default index pattern for test

* remove duplicate const kibanaServer

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* delete unused es_archive visualize_embedding (elastic#126001)

* delete unused es_archive

* remove other unused es_archive

* more unused es_archives

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

* Bump packages (elastic#126119)

* url-parse 1.5.3 -> 1.5.9
* follow-redirects 1.y.z -> 1.14.9

* Add tests for all cases selector and attachments

* Add tests for use add to existing case hook

* First version of the cases timeline actions

* export add alert to new case button from cases plugin

* Make Cases ECS compatible with timelines and security_solution

* Delete new case button

* Add helpers

* Use the cases hook directly for add to new case

* Remov unused dependencies

* Rename callbacks, remove timelines calls

* Fixing tests for the dropdown

* Fix broken test

* mocking cases for tests

* Fix detectiosn tests

* Observability now uses the new cases hooks

* Wrap events viewer into cases context

* Open the create case flyout if create case was selected in the modal

* Fix cases mocks for security_solution

* Update tests

* Add tests for use cases toast

* Improve cases mock

* delete security mock

* replace tests mocks for cases

* fix import mock

* Do not require onRowClick

* Show the toast inside the modal

* show the toast inside the flyout

* remove toast logic from the consumer plugin

* fix typescript types

* Rename type

* Fix broken test

* Fix file name and broken test

* Use internal navigation hook

* Update hook dependencies

* Move useCaseToast

* Fix mock paths

* fix eslint

* Add test cases for the toast content

* Add cases context to the overview page

Co-authored-by: Julia Bardi <90178898+juliaElastic@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: István Zoltán Szabó <szabosteve@gmail.com>
Co-authored-by: Joe Reuter <johannes.reuter@elastic.co>
Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
Co-authored-by: Cristina Amico <criamico@users.noreply.github.com>
Co-authored-by: Aleh Zasypkin <aleh.zasypkin@elastic.co>
Co-authored-by: Gloria Hornero <gloria.hornero@elastic.co>
Co-authored-by: Matthew Kime <matt@mattki.me>
Co-authored-by: Ying Mao <ying.mao@elastic.co>
Co-authored-by: Maja Grubic <maja.grubic@elastic.co>
Co-authored-by: Lee Drengenberg <lee.drengenberg@elastic.co>
Co-authored-by: Joe Portner <5295965+jportner@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed chore release_note:skip Skip the PR/issue when compiling release notes v7.17.1 v8.2.0
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

9 participants