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

[alerts] adds defaultActionMessage to index threshold alert UI type definition #82753

Conversation

pmuellr
Copy link
Member

@pmuellr pmuellr commented Nov 5, 2020

Fixes #78148

note: this PR is a "copy" of PR #80936, moved from the community contributor's repo into mine, and placed in a separate branch from master

Screenshot from 2020-10-17 18-36-07

The above screenshot displays the defaultActionMessage in the text box when an action is created for index threshold alert.

@kibanamachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps


Test Failures

X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/rollup/rollup·js.apis management rollup jobs Index patterns should return the date, numeric and keyword fields when an index pattern matches indices

Link to Jenkins

Standard Out

Failed Tests Reporter:
  - Test has not failed recently on tracked branches

[00:00:00]       │
[00:00:00]         │ info [o.e.x.i.IndexLifecycleTransition] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] moving index [ilm-history-4-000001] from [null] to [{"phase":"new","action":"complete","name":"complete"}] in policy [ilm-history-ilm-policy]
[00:00:00]         │ info [o.e.c.r.a.AllocationService] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] current.health="GREEN" message="Cluster health status changed from [YELLOW] to [GREEN] (reason: [shards started [[ilm-history-4-000001][0]]])." previous.health="YELLOW" reason="shards started [[ilm-history-4-000001][0]]"
[00:00:00]         │ info [o.e.x.i.IndexLifecycleTransition] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] moving index [ilm-history-4-000001] from [{"phase":"new","action":"complete","name":"complete"}] to [{"phase":"hot","action":"unfollow","name":"wait-for-indexing-complete"}] in policy [ilm-history-ilm-policy]
[00:00:00]         │ info [o.e.x.i.IndexLifecycleTransition] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] moving index [ilm-history-4-000001] from [{"phase":"hot","action":"unfollow","name":"wait-for-indexing-complete"}] to [{"phase":"hot","action":"unfollow","name":"wait-for-follow-shard-tasks"}] in policy [ilm-history-ilm-policy]
[00:00:00]         └-: apis
[00:00:00]           └-> "before all" hook
[00:04:21]           └-: management
[00:04:21]             └-> "before all" hook
[00:04:36]             └-: rollup
[00:04:36]               └-> "before all" hook
[00:04:36]               └-: jobs
[00:04:36]                 └-> "before all" hook
[00:04:37]                 └-: Index patterns
[00:04:37]                   └-> "before all" hook
[00:04:37]                   └-> should return the date, numeric and keyword fields when an index pattern matches indices
[00:04:37]                     └-> "before each" hook: global before each
[00:04:37]                     │ info [o.e.c.m.MetadataCreateIndexService] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] [lkvpsuaksigyi-1604602512925] creating index, cause [api], templates [], shards [1]/[1]
[00:04:37]                     └- ✖ fail: apis management rollup jobs Index patterns should return the date, numeric and keyword fields when an index pattern matches indices
[00:04:37]                     │       Error: expected { doesMatchIndices: true,
[00:04:37]                     │   doesMatchRollupIndices: false,
[00:04:37]                     │   dateFields: [ 'testCreatedField' ],
[00:04:37]                     │   numericFields: [ '_doc_count', 'testTotalField' ],
[00:04:37]                     │   keywordFields: [ 'testTagField' ] } to sort of equal { dateFields: [ 'testCreatedField' ],
[00:04:37]                     │   keywordFields: [ 'testTagField' ],
[00:04:37]                     │   numericFields: [ 'testTotalField' ],
[00:04:37]                     │   doesMatchIndices: true,
[00:04:37]                     │   doesMatchRollupIndices: false }
[00:04:37]                     │       + expected - actual
[00:04:37]                     │ 
[00:04:37]                     │          "keywordFields": [
[00:04:37]                     │            "testTagField"
[00:04:37]                     │          ]
[00:04:37]                     │          "numericFields": [
[00:04:37]                     │       -    "_doc_count"
[00:04:37]                     │            "testTotalField"
[00:04:37]                     │          ]
[00:04:37]                     │        }
[00:04:37]                     │       
[00:04:37]                     │       at Assertion.assert (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:100:11)
[00:04:37]                     │       at Assertion.eql (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:244:8)
[00:04:37]                     │       at Context.it (test/api_integration/apis/management/rollup/rollup.js:48:25)
[00:04:37]                     │ 
[00:04:37]                     │ 

Stack Trace

{ Error: expected { doesMatchIndices: true,
  doesMatchRollupIndices: false,
  dateFields: [ 'testCreatedField' ],
  numericFields: [ '_doc_count', 'testTotalField' ],
  keywordFields: [ 'testTagField' ] } to sort of equal { dateFields: [ 'testCreatedField' ],
  keywordFields: [ 'testTagField' ],
  numericFields: [ 'testTotalField' ],
  doesMatchIndices: true,
  doesMatchRollupIndices: false }
    at Assertion.assert (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:100:11)
    at Assertion.eql (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:244:8)
    at Context.it (test/api_integration/apis/management/rollup/rollup.js:48:25)
  actual:
   '{\n  "dateFields": [\n    "testCreatedField"\n  ]\n  "doesMatchIndices": true\n  "doesMatchRollupIndices": false\n  "keywordFields": [\n    "testTagField"\n  ]\n  "numericFields": [\n    "_doc_count"\n    "testTotalField"\n  ]\n}',
  expected:
   '{\n  "dateFields": [\n    "testCreatedField"\n  ]\n  "doesMatchIndices": true\n  "doesMatchRollupIndices": false\n  "keywordFields": [\n    "testTagField"\n  ]\n  "numericFields": [\n    "testTotalField"\n  ]\n}',
  showDiff: true }

X-Pack API Integration Tests.x-pack/test/api_integration/apis/management/rollup/rollup·js.apis management rollup jobs Index patterns should return the date, numeric and keyword fields when an index pattern matches indices

Link to Jenkins

Standard Out

Failed Tests Reporter:
  - Test has failed 1 times on tracked branches: https://dryrun

[00:00:00]       │
[00:00:00]         └-: apis
[00:00:00]           └-> "before all" hook
[00:04:18]           └-: management
[00:04:18]             └-> "before all" hook
[00:04:33]             └-: rollup
[00:04:33]               └-> "before all" hook
[00:04:33]               └-: jobs
[00:04:33]                 └-> "before all" hook
[00:04:33]                 └-: Index patterns
[00:04:33]                   └-> "before all" hook
[00:04:33]                   └-> should return the date, numeric and keyword fields when an index pattern matches indices
[00:04:33]                     └-> "before each" hook: global before each
[00:04:33]                     │ info [o.e.c.m.MetadataCreateIndexService] [kibana-ci-immutable-centos-tests-xxl-1604598222355747611] [qzgsyamk-1604601262812] creating index, cause [api], templates [], shards [1]/[1]
[00:04:33]                     └- ✖ fail: apis management rollup jobs Index patterns should return the date, numeric and keyword fields when an index pattern matches indices
[00:04:33]                     │       Error: expected { doesMatchIndices: true,
[00:04:33]                     │   doesMatchRollupIndices: false,
[00:04:33]                     │   dateFields: [ 'testCreatedField' ],
[00:04:33]                     │   numericFields: [ '_doc_count', 'testTotalField' ],
[00:04:33]                     │   keywordFields: [ 'testTagField' ] } to sort of equal { dateFields: [ 'testCreatedField' ],
[00:04:33]                     │   keywordFields: [ 'testTagField' ],
[00:04:33]                     │   numericFields: [ 'testTotalField' ],
[00:04:33]                     │   doesMatchIndices: true,
[00:04:33]                     │   doesMatchRollupIndices: false }
[00:04:33]                     │       + expected - actual
[00:04:33]                     │ 
[00:04:33]                     │          "keywordFields": [
[00:04:33]                     │            "testTagField"
[00:04:33]                     │          ]
[00:04:33]                     │          "numericFields": [
[00:04:33]                     │       -    "_doc_count"
[00:04:33]                     │            "testTotalField"
[00:04:33]                     │          ]
[00:04:33]                     │        }
[00:04:33]                     │       
[00:04:33]                     │       at Assertion.assert (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:100:11)
[00:04:33]                     │       at Assertion.eql (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:244:8)
[00:04:33]                     │       at Context.it (test/api_integration/apis/management/rollup/rollup.js:48:25)
[00:04:33]                     │ 
[00:04:33]                     │ 

Stack Trace

{ Error: expected { doesMatchIndices: true,
  doesMatchRollupIndices: false,
  dateFields: [ 'testCreatedField' ],
  numericFields: [ '_doc_count', 'testTotalField' ],
  keywordFields: [ 'testTagField' ] } to sort of equal { dateFields: [ 'testCreatedField' ],
  keywordFields: [ 'testTagField' ],
  numericFields: [ 'testTotalField' ],
  doesMatchIndices: true,
  doesMatchRollupIndices: false }
    at Assertion.assert (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:100:11)
    at Assertion.eql (/dev/shm/workspace/parallel/18/kibana/packages/kbn-expect/expect.js:244:8)
    at Context.it (test/api_integration/apis/management/rollup/rollup.js:48:25)
  actual:
   '{\n  "dateFields": [\n    "testCreatedField"\n  ]\n  "doesMatchIndices": true\n  "doesMatchRollupIndices": false\n  "keywordFields": [\n    "testTagField"\n  ]\n  "numericFields": [\n    "_doc_count"\n    "testTotalField"\n  ]\n}',
  expected:
   '{\n  "dateFields": [\n    "testCreatedField"\n  ]\n  "doesMatchIndices": true\n  "doesMatchRollupIndices": false\n  "keywordFields": [\n    "testTagField"\n  ]\n  "numericFields": [\n    "testTotalField"\n  ]\n}',
  showDiff: true }

Metrics [docs]

page load bundle size

id before after diff
triggersActionsUi 131.9KB 132.3KB +413.0B

Saved Objects .kibana field count

id before after diff
_doc_count - 1 +1

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

@pmuellr
Copy link
Member Author

pmuellr commented Nov 9, 2020

closing - I had this branch as a backup

@pmuellr pmuellr closed this Nov 9, 2020
@pmuellr pmuellr added the backport:skip This commit does not require backporting label Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add defaultActionMessage to index threshold alert UI type definition
3 participants