Skip to content

Commit

Permalink
Revert skips added while triaging ES OOMs (#97876) (#98066)
Browse files Browse the repository at this point in the history
* Revert "skip flaky suite (#97382)"

This reverts commit e321f57.

* Revert "skip flaky suite (#97387)"

This reverts commit a89b756.

* Revert "Skip test to try and stabilize master" (#97378)

This reverts commit 194355f.

* upload heap dumps when they are created

Co-authored-by: spalger <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Spencer <[email protected]>
Co-authored-by: spalger <[email protected]>
  • Loading branch information
3 people authored Apr 22, 2021
1 parent 1293ea8 commit 84046ed
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 6 deletions.
1 change: 1 addition & 0 deletions vars/kibanaPipeline.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,7 @@ def withGcsArtifactUpload(workerName, closure) {
'x-pack/test/**/screenshots/session/*.png',
'x-pack/test/functional/apps/reporting/reports/session/*.pdf',
'x-pack/test/functional/failure_debug/html/*.html',
'.es/**/*.hprof'
]

withEnv([
Expand Down
3 changes: 1 addition & 2 deletions x-pack/test/api_integration/apis/lens/existing_fields.ts
Original file line number Diff line number Diff line change
Expand Up @@ -162,8 +162,7 @@ export default ({ getService }: FtrProviderContext) => {
const esArchiver = getService('esArchiver');
const supertest = getService('supertest');

// FLAKY: https://github.com/elastic/kibana/issues/97387
describe.skip('existing_fields apis', () => {
describe('existing_fields apis', () => {
before(async () => {
await esArchiver.loadIfNeeded('logstash_functional');
await esArchiver.loadIfNeeded('visualize/default');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,7 @@ export default function ({ getService }: FtrProviderContext) {
const supertest = getService('supertest');
const retry = getService('retry');

// FIX: https://github.com/elastic/kibana/issues/97378
describe.skip('Matrix DNS Histogram', () => {
describe('Matrix DNS Histogram', () => {
describe('Large data set', () => {
before(() => esArchiver.load('security_solution/matrix_dns_histogram/large_dns_query'));
after(() => esArchiver.unload('security_solution/matrix_dns_histogram/large_dns_query'));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ export default function featureControlsTests({ getService }: FtrProviderContext)
const supertest = getService('supertestWithoutAuth');
const security = getService('security');

// FLAKY: https://github.com/elastic/kibana/issues/97382
describe.skip('feature controls', () => {
describe('feature controls', () => {
const kibanaUsername = 'kibana_admin';
const kibanaUserRoleName = 'kibana_admin';

Expand Down

0 comments on commit 84046ed

Please sign in to comment.