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

Failing test: License plugin API Integration Tests.x-pack/test/licensing_plugin/public/updates·ts - Licensing plugin public client changes in license types provides changes in license types #53575

Closed
kibanamachine opened this issue Dec 19, 2019 · 22 comments · Fixed by #53882 or #163199
Assignees
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@kibanamachine
Copy link
Contributor

kibanamachine commented Dec 19, 2019

A test failed on a tracked branch

{ JavascriptError: javascript error: Service Unavailable
JavaScript stack:
Error: Service Unavailable
    at _callee3$ (http://localhost:6131/bundles/commons.bundle.js:12770:23)
    at tryCatch (http://localhost:6131/built_assets/dlls/vendors.bundle.dll.js:504:570349)
    at Generator.invoke [as _invoke] (http://localhost:6131/built_assets/dlls/vendors.bundle.dll.js:504:574198)
    at Generator.forEach.prototype.<computed> [as next] (http://localhost:6131/built_assets/dlls/vendors.bundle.dll.js:504:571473)
    at asyncGeneratorStep (http://localhost:6131/bundles/commons.bundle.js:12468:24)
    at _next (http://localhost:6131/bundles/commons.bundle.js:12490:9)
  (Session info: headless chrome=79.0.3945.88)
  (Driver info: chromedriver=79.0.3945.36 (3582db32b33893869b8c1339e8f4d9ed1816f143-refs/branch-heads/3945@{#614}),platform=Linux 4.9.0-11-amd64 x86_64)
    at Object.checkLegacyResponse (/dev/shm/workspace/kibana/node_modules/selenium-webdriver/lib/error.js:585:15)
    at parseHttpResponse (/dev/shm/workspace/kibana/node_modules/selenium-webdriver/lib/http.js:554:13)
    at Executor.execute (/dev/shm/workspace/kibana/node_modules/selenium-webdriver/lib/http.js:489:26)
    at process._tickCallback (internal/process/next_tick.js:68:7) name: 'JavascriptError', remoteStacktrace: '' }

First failure: Jenkins Build

@kibanamachine kibanamachine added the failed-test A test failure on a tracked branch, potentially flaky-test label Dec 19, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-test-triage (failed-test)

@mshustov mshustov self-assigned this Dec 19, 2019
@mshustov
Copy link
Contributor

#53002 reverted

@mshustov mshustov removed their assignment Dec 19, 2019
@kibanamachine kibanamachine reopened this Dec 19, 2019
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@tsullivan
Copy link
Member

I am seeing flakiness with this test, and it looks like a different error message result:

https://kibana-ci.elastic.co/job/elastic+kibana+pipeline-pull-request/120699/testReport/License%20plugin%20API%20Integration%20Tests/x-pack_test_licensing_plugin_public_updates%C2%B7ts/Kibana_Pipeline___general___Licensing_plugin_public_client_changes_in_license_types_provides_changes_in_license_types/

[00:00:31]               │ info Saving page source to: /dev/shm/workspace/parallel/3/kibana/x-pack/test/licensing_plugin/failure_debug/html/Licensing plugin public client changes in license types provides changes in license types.html
[00:00:31]               └- ✖ fail: Licensing plugin public client changes in license types provides changes in license types
[00:00:31]               │      Error: expected 'basic' to equal 'trial'
[00:00:31]               │       at Assertion.assert (/dev/shm/workspace/parallel/3/kibana/node_modules/@kbn/expect/expect.js:100:11)
[00:00:31]               │       at Assertion.be.Assertion.equal (/dev/shm/workspace/parallel/3/kibana/node_modules/@kbn/expect/expect.js:227:8)
[00:00:31]               │       at Assertion.be (/dev/shm/workspace/parallel/3/kibana/node_modules/@kbn/expect/expect.js:69:22)
[00:00:31]               │       at Context.<anonymous> (test/licensing_plugin/public/updates.ts:74:12)
[00:00:31]               │       at runMicrotasks (<anonymous>)
[00:00:31]               │       at processTicksAndRejections (internal/process/task_queues.js:93:5)
[00:00:31]               │       at Object.apply (/dev/shm/workspace/parallel/3/kibana/node_modules/@kbn/test/src/functional_test_runner/lib/mocha/wrap_function.js:73:16)
[00:00:31]               │ 
[00:00:31]               │ 

@kibanamachine kibanamachine reopened this Apr 21, 2021
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

1 similar comment
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@botelastic botelastic bot added the needs-team Issues missing a team label label Apr 21, 2021
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

1 similar comment
@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@mistic mistic added the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Apr 21, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Apr 21, 2021
mistic added a commit that referenced this issue Apr 21, 2021
@mistic
Copy link
Member

mistic commented Apr 21, 2021

The test keeps failing and looks flaky. Skipping for now.

master: ffc3db5

@kibanamachine
Copy link
Contributor Author

New failure: Jenkins Build

@tsullivan
Copy link
Member

It's definitely happening because of #94966 which has not yet been backported to 7.x.

I tried to add a change that would stop this test from being flaky, but it looks like it wasn't enough (sorry about that @mistic): 2e732f2

In the backport branch, I am going to add the skip for this test as well, so it will be skipped in 7.x

tsullivan pushed a commit to tsullivan/kibana that referenced this issue Apr 22, 2021
tsullivan added a commit that referenced this issue Apr 22, 2021
#97777)

* [Reporting] Kibana Application Privileges for Reporting (#94966)

* Implement Reporting features as subfeatures of applications

* add setting to the docker list

* update doc images

* finish docs

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* typo fix

* "PDF / PNG Reports" => "Reporting"

* Update x-pack/plugins/reporting/server/config/index.ts

Co-authored-by: Larry Gregory <lgregorydev@gmail.com>

* Update x-pack/test/functional/apps/security/secure_roles_perm.js

Co-authored-by: Larry Gregory <lgregorydev@gmail.com>

* update ids of report privileges

* combine dashboard privileges into 1 group

* update jest snapshot

* fix tests

* fix tests

* updates from feedback

* add note

* update screenshot

* fix grammer

* fix bad link breaks in doc

* update doc heading

* Apply suggestions documentation feedback

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

* simplify

* use const assertions

* Apply text change suggestion from code review

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

* more test for oss_features and reporting subFeatures

* reduce loc diff

* fix snapshot

* fix flakiness in licensing plugin public functional tests

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>
Co-authored-by: Larry Gregory <lgregorydev@gmail.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
# Conflicts:
#	x-pack/plugins/reporting/server/core.ts
#	x-pack/plugins/reporting/server/lib/enqueue_job.test.ts
#	x-pack/plugins/reporting/server/lib/store/store.test.ts
#	x-pack/plugins/reporting/server/lib/tasks/execute_report.test.ts
#	x-pack/plugins/reporting/server/lib/tasks/monitor_report.test.ts
#	x-pack/plugins/reporting/server/plugin.ts
#	x-pack/plugins/reporting/server/test_helpers/create_mock_reportingplugin.ts

* fix ci

* fix eslint

* skip flaky suite (#53575)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Tiago Costa <tiagoffcc@hotmail.com>
tsullivan added a commit to tsullivan/kibana that referenced this issue Apr 29, 2021
…c#94966) (elastic#97777)

* [Reporting] Kibana Application Privileges for Reporting (elastic#94966)

* Implement Reporting features as subfeatures of applications

* add setting to the docker list

* update doc images

* finish docs

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* Apply suggestions from code review

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>

* typo fix

* "PDF / PNG Reports" => "Reporting"

* Update x-pack/plugins/reporting/server/config/index.ts

Co-authored-by: Larry Gregory <lgregorydev@gmail.com>

* Update x-pack/test/functional/apps/security/secure_roles_perm.js

Co-authored-by: Larry Gregory <lgregorydev@gmail.com>

* update ids of report privileges

* combine dashboard privileges into 1 group

* update jest snapshot

* fix tests

* fix tests

* updates from feedback

* add note

* update screenshot

* fix grammer

* fix bad link breaks in doc

* update doc heading

* Apply suggestions documentation feedback

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

* simplify

* use const assertions

* Apply text change suggestion from code review

Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>

* more test for oss_features and reporting subFeatures

* reduce loc diff

* fix snapshot

* fix flakiness in licensing plugin public functional tests

Co-authored-by: Kaarina Tungseth <kaarina.tungseth@elastic.co>
Co-authored-by: Larry Gregory <lgregorydev@gmail.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: gchaps <33642766+gchaps@users.noreply.github.com>
# Conflicts:
#	x-pack/plugins/reporting/server/core.ts
#	x-pack/plugins/reporting/server/lib/enqueue_job.test.ts
#	x-pack/plugins/reporting/server/lib/store/store.test.ts
#	x-pack/plugins/reporting/server/lib/tasks/execute_report.test.ts
#	x-pack/plugins/reporting/server/lib/tasks/monitor_report.test.ts
#	x-pack/plugins/reporting/server/plugin.ts
#	x-pack/plugins/reporting/server/test_helpers/create_mock_reportingplugin.ts

* fix ci

* fix eslint

* skip flaky suite (elastic#53575)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Tiago Costa <tiagoffcc@hotmail.com>
madirey pushed a commit to madirey/kibana that referenced this issue May 11, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

@rudolf rudolf removed the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Sep 15, 2021
@tsullivan tsullivan added needs-team Issues missing a team label and removed Team:Reporting Services labels Dec 27, 2021
@lukeelmers lukeelmers added the Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc label Dec 29, 2021
@botelastic botelastic bot removed the needs-team Issues missing a team label label Dec 29, 2021
@rudolf
Copy link
Contributor

rudolf commented Aug 3, 2023

It's definitely happening because of #94966

@tsullivan Could you please prioritise fixing this?

@rudolf rudolf added the Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) label Aug 3, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@tsullivan
Copy link
Member

tsullivan commented Aug 8, 2023

It's definitely happening because of #94966

@tsullivan Could you please prioritise fixing this?

@rudolf I'm getting stuck on this:
https://github.com/elastic/kibana/blob/3730dd0/x-pack/test/licensing_plugin/scenario.ts#L62-L69

From the code:

// elasticsearch allows to request trial only once. other attempts will throw 403.

In local development, I can get the test to pass the first time, but have to restart the server to get it to pass again. Without a restart, the test fails because the request that was expected to return a 200 returns a 403.

Question: How is this expected to work in a flaky test runner?

Also, I don't recall the original reason why I thought these test failures were related to my PR, but I vaguely remember not being able to get this test passing when working on that PR.

@tsullivan
Copy link
Member

In local development, I can get the test to pass the first time, but have to restart the server to get it to pass again. Without a restart, the test fails because the request that was expected to return a 200 returns a 403.

Question: How is this expected to work in a flaky test runner?

Scratch that, the only failures I ended up seeing were due to an expired license coded into a test.

I have a fix ready for unskipping this here: #163199

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants