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

Fix CI for rollups branch #24105

Merged
merged 63 commits into from
Oct 17, 2018
Merged

Conversation

jen-huang
Copy link
Contributor

No description provided.

pavel06081991 and others added 30 commits October 5, 2018 10:06
remove extra span tags generated by FormattedMessage component

translate missed labels
…der accessible (elastic#23832)

* Calendar nav links keyboard accessible

* Prevent default click behavior

* use event default value

* remove unnecessary default param
* remove less styles

* remove less import
* Initial version of CCR monitoring UI

* Adding missing files

* Use icons

* Use new column header text

* Update tests

* Basic of shard detail page

* Do these in parallel

* Disable time picker on ccr page

* Remove summary for now

* Remove unnecessary code here

* Fix a few things on the shard page

* Only send down what we need

* update snapshot

* Handle no ccr_stats documents

* Ensure we fetch the latest

* Updates

* Format the time

* Add api integration tests

* Adding pagination and sorting

* Updated query logic

* Change this back

* Add specific information about the follower and leader lag ops

* Update tests

* UI updates

* Address PR issues

* Fix tests

* Update shapshots

* Add timestamp

* Update tests

* Add a few snapshot tests

* Use timezone formatter

* Fix tests

* Fix aligment of shard table

* PR feedback

* Update snapshots

* Update snapshot
This PR introduces custom charts for detectors that use a rare function (Event Distribution Chart) as well as detectors that use an over field (Population Distribution Chart).
…ic#23874)

This is a refactor of mlExplorerDashboardService to make it available via import instead of angularjs dependency injection. This way it's also not necessary anymore to pass it on as a prop to ExplorerSwimlane, the component can now import the service by itself.
* Adds super select to font picker. Removes fauxSelect component

* Removed import for font_picker.scss
* Add one-time binding to angularjs i18n

* Add watcher for values property

* Watch values field only if it is provided

* Fix ci
…elastic#23907)

Add a wait for the reporting flyout menu to animate. This workaround should be removed if we figure out how to disable animations in our test suite.
* fix: router can render function or class components

* fix: correctly define state

and change the first load detection, since this.state is always set now

* chore: DRY up navigation code

* tests: disable listener cleanup test

there's no way to know when the listener is going to get cleaned up anymore :(
* delete objects belonging to removed space

* remove unused parameters
* [yarn] upgrade required version

* [yarn] update lock files to include integrity

* [yarn] coalesce locked readable-stream versions to avoid yarn bug
…stic#24021)

Fixes an issue where resizing the Anomaly Explorer window could trigger an error if the only job being loaded so far didn't have any anomalyChartRecords.
…ic#24099)

* Revert "Proofreading edits, UI text consistency (elastic#24016)"

This reverts commit f59f1cc.

* Revert "Skip these tests (elastic#24085)"

This reverts commit e79533d.

* Revert "Point at kibana directory for plugin loading (elastic#24049)"

This reverts commit 3a7f5f5.

* Revert "Translate pie and vaslib_basic_options (elastic#23761)"

This reverts commit 1f73ea1.

* Revert "doing config migration during config get if necessary (elastic#23377)"

This reverts commit 0b4f82d.
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

Spencer and others added 11 commits October 16, 2018 18:59
In `ui/public/config/config.js` we have subscription logic that delivers values from an observable synchronously and also ensuring that values are received within a digest cycle. I need to do the same in elastic#23217 but want to keep as few checks for `$rootScope.$$phase` as possible, so I broke the subscription logic into a shared util.

In order to make the utility a little more helpful it will also trigger fatal errors if an observable errors without having an error handler, or if `observer.next()`, `observer.error()`, or `observer.complete()` throw, which would normally be swallowed by RxJS.
* [Tools] Fix pug regular expression

* Add one more test
* Add a typescript readme

* some clean up

* Correct capitalization of typescript and javascript

* Incorporate review suggestions
…lastic#24084)

* Add refresh button for job list

* Show load icon while refreshing

* Move view build logic to jobsListView

* Align job action buttons in ui

* Extract RefreshJobsListButton component

* update jobStatsBar to take list as prop
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

walterra and others added 4 commits October 17, 2018 16:52
…ther metrics than count. (elastic#24083)

- Enables support for showing contextual sampled data in population charts which use detector functions other than just count.
- Use arrow functions where applicable in Anomaly Explorer Charts code.
This fixes an issue where job validation would return that a date field was invalid for nested date fields.
@elasticmachine
Copy link
Contributor

💔 Build Failed

@elasticmachine
Copy link
Contributor

💔 Build Failed

@jen-huang
Copy link
Contributor Author

Merging, build failure is the x-pack one which is consistently failing on master too.

@jen-huang jen-huang merged this pull request into elastic:feature/rollups Oct 17, 2018
@jen-huang jen-huang deleted the feature/rollups-ci branch October 17, 2018 20:04
jen-huang added a commit that referenced this pull request Oct 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.