Skip to content

[SR] render alert icon in policy table if last snapshot failed#46960

Merged
alisonelizabeth merged 3 commits intoelastic:masterfrom
alisonelizabeth:sr/policy_snapshot_failure
Oct 2, 2019
Merged

[SR] render alert icon in policy table if last snapshot failed#46960
alisonelizabeth merged 3 commits intoelastic:masterfrom
alisonelizabeth:sr/policy_snapshot_failure

Conversation

@alisonelizabeth
Copy link
Contributor

This PR adds an alert icon next to the snapshot name of a policy, if the last snapshot failed. This is determined by comparing the timestamps of the last failed snapshot and last successful snapshot.

Fixes #46795

Screen Shot 2019-09-30 at 2 21 36 PM

@alisonelizabeth alisonelizabeth added v8.0.0 Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// Feature:Snapshot and Restore Elasticsearch snapshots and repositories UI v7.5.0 labels Sep 30, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/es-ui

@alisonelizabeth alisonelizabeth added the release_note:skip Skip the PR/issue when compiling release notes label Sep 30, 2019
@alisonelizabeth
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@sebelga sebelga left a comment

Choose a reason for hiding this comment

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

LGTM! It seems that there is an empty css entry.

*/
.snapshotRestorePolicyTableSnapshotFailureContainer {
max-width: 200px;
> .euiFlexItem:first-child {
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this empty selector?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, forgot to remove that. thanks for the catch!

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Snapshot and Restore Elasticsearch snapshots and repositories UI release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SR] Highlight policy if the last snapshot failed

3 participants