[WIP] [Monitoring] APM UI specifics #22508
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…emoved from the selection. (elastic#22242)
…ing messages (elastic#22243) * Update buildHierarchicalData test.
* Rebuild modulePath correctly if on Windows * Change how we find dashboard add panel is loaded * Add and use find.byClassName instead of remote * cleanup a comment
elastic#22124) translate tutorials(osquery_logs, php_fpm_metrics, postgresql_logs, postgresql_metrics, prometheus_metrics, rabbitmq_metrics, redis_logs)
elastic#22122) translate tutorials(kafka_logs, kafka_metrics, kibana_metrics, kubernetes_metrics, logstash_logs, logstash_metrics, memcached_metrics)
elastic#22121) translate tutorials(elasticsearch_metrics, etcd_metrics, golang_metrics, haprozy_metrics, iis_logs)
Upgrades EUI from 3.6.1 => 3.7.0
|
@graphaelli Definitely missed that. Should be updated now |
|
Thanks @chrisronline. Something is a bit off with the instance view - looks like there is no padding between the graphs (instance view on left, overview on right):
|
💔 Build Failed |
|
That's actually consistent with beats, but I can add the padding into the instance page to make it look better
|
💔 Build Failed |
|
Awesome! I think that looks much better. Perhaps beats wants to consider applying the same update. /cc @ruflin |
graphaelli
approved these changes
Sep 11, 2018
LGTM 🎉 🎉 🎉
…stic#22814) This PR addresses parts of elastic#22642: - It gets rid of the use of var that = this;. - dragSelect's action strings are moved to a constants file. - Adds jest tests for the ExplorerSwimlane component. This also fixes the following bugs: - The way we subscribe listeners to the events of the dragSelect library could result in the same event being triggered multiple times. This in turn could cause race conditions when on each event new data gets fetched but in between angular's scope gets updated and could end up in a non-intended way. The result of this were view-by swimlanes not updating correctly or anomaly charts showing non-related charts. This PR fixes it by filtering out consecutive swimlane click events. - When the angular based chart container wrapper directive gets destroyed/re-esetup when using the job pick, it missed unmounting the react component, it didn't trigger componentWillUnmount()and didn't unsubscribe from dragSelectListener.
…ic#22379) Allows Kibana users to configure the max_concurrent_shard_requests param used by Kibana when sending _msearch requests. Exposes the config as an advanced setting. By default we won't send the param at all, relying on the ES default instead.
…lastic#22904) The `deepFreeze()` function used by `core.injectedMetadata` uses a recursive type definition to indicate that all of the child types within the passed argument become readonly, which works fine for objects but represents arrays as objects instead of using the `ReadonlyArray<>` type. This PR fixes the type definition to use a `RecursiveReadonlyArray<>` type that properly represents arrays with their methods like `push()`, and iterates into the array properly to propagate `ReadOnly<>`, as proven by the tests.
* conditionalize Monitoring tests for Cloud testing * make security_page/login wait * revert these files from another PR * revert these files from another PR
…#21505) * Default to chromium and add deprecation warning for phantom. * use int, not float
* migrate reporting top nav to sharing menu * pass share extensions to visualize share menu * start creating the reporting panel component * add buttons * generate report generatation URL * require save if url changes * add print layout UI * putting it all together - generate reports from share context menu * ensure copy button fills entire menu width * inject job params functionallity * refactor print layout out of ReportingContentPanel * CSV report generation * disable report generation when app is dirty * update URL when window is resized * remove all the old stuff * clean up CSV report register provider * fix typescript errors * fix functional retests * remove failure_debug folder * close popover when generate button clicked, more work on functional tests * set size on button and text, do not use EuiFormRow * use ReactElement type * move ShareContextMenuExtensionsRegistryProvider to ui/share and create interfaces for registy * strictor typing, do not use hard coded object type name * move registry files to typescript * remove destructuring in the interface * convert createReportingJob to async function, remove unneeded comment
- The aim of this is to more clearly visualize how the timerange of the cell selected in the swimlane relates to the time span shown in the charts. - The most important change is that the vertical date axis ticks no longer are randomly positioned by d3. Instead they are aligned with the cell interval of the swimlane. This way, the date information shown in the swimlane tooltip will always align with the date tick shown left of the emphasized area in the chart. - The highlighted area now features a gray rounded border to resemble the styling of the selected cell in the swimlane. - The chart also fixes where to long chart headers would wrap the "View" link to a new line. - The x/y axis labels blackness has been reduced to reduce emphasis on the labels.
💔 Build Failed |
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Continuation of #22453
Depends on elastic/elasticsearch#33286
This PR introduces a monitoring UI for monitoring APM.
TODO
Testing
See this comment for how to get the apm environment running with this PR.
Metrics
Here is a list of what we are the visuals in place. They will most likely appear on both the instance and overview page.
Total EventsThe difference between the
minandmaxofbeats_stats.metrics.libbeat.pipeline.events.totalBytes SentThe difference between the
minandmaxofbeats_stats.metrics.libbeat.output.write.bytesServersThe number of unique uuids we have (or, the number of different apm server instances)
Last EventThe timestamp of the last monitoring document where
beats_stats.metrics.libbeat.output.events.acked > 0CPU UtilizationgraphBased off:
beats_stats.metrics.beat.cpu.total.valueSystem LoadgraphBased off:
beats_stats.metrics.system.load.1beats_stats.metrics.system.load.5beats_stats.metrics.system.load.15Memory (MB)graphBased off:
beats_stats.metrics.beat.memstats.gc_nextbeats_stats.metrics.beat.memstats.memory_totalbeats_stats.metrics.beat.memstats.memory_allocbeats_stats.metrics.beat.memstats.rssRequests (/m)graphBased off:
beats_stats.metrics.apm-server.server.request.countIncoming Requests Size (KB)graphBased off:
beats_stats.metrics.apm-server.decoder.deflate.content-lengthbeats_stats.metrics.apm-server.decoder.gzip.content-lengthbeats_stats.metrics.apm-server.decoder.uncompressed.content-length.Transformations (/m)graphBased off:
beats_stats.metrics.apm-server.processor.transaction.transformationsbeats_stats.metrics.apm-server.processor.span.transformationsbeats_stats.metrics.apm-server.processor.error.transformationsbeats_stats.metrics.apm-server.processor.metric.transformationsOutput Events Rate (/m)graphBased off:
beats_stats.metrics.libbeat.output.events.totalbeats_stats.metrics.libbeat.output.events.activebeats_stats.metrics.libbeat.output.events.ackedWe should expect to see the
TotalandAckedas nearly equivalent lines.Output Failed Events Rate (/m)graphBased off:
beats_stats.metrics.libbeat.output.events.failed,beats_stats.metrics.libbeat.output.events.droppedResponse Count (/m)graphBased off:
beats_stats.metrics.apm-server.server.response.valid.*Response Errors (/m)graphBased off:
beats_stats.metrics.apm-server.server.response.errors.*