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

PERFORMANCE_level_manager test failing on Bionic github action #826

Closed
scpeters opened this issue May 19, 2021 · 2 comments
Closed

PERFORMANCE_level_manager test failing on Bionic github action #826

scpeters opened this issue May 19, 2021 · 2 comments
Labels
bug Something isn't working performance Runtime performance tests Broken or missing tests / testing infra

Comments

@scpeters
Copy link
Member

Environment

  • OS Version: Ubuntu 18.04
  • Source or binary build? source build of pull request 5 ➡️ 6 #811 targeting main

Description

  • Expected behavior: PERFORMANCE_level_manager passes
  • Actual behavior: PERFORMANCE_level_manager fails

Steps to reproduce

  1. Run a github action with Bionic
  2. Check for PERFORMANCE_level_manager test status

Output

Both Bionic GitHub workflow runs for 80e7b85 of #811 failed:

  [Dbg] [SimulationRunner.cc:511] Exiting postupdate worker thread (0)
  [Dbg] [level_manager.cc:73] 
  Using levels = 7633 ms
  Without levels = 7479 ms
  /github/workspace/test/performance/level_manager.cc:78: Failure
  Expected: (levelsDuration.count()) <= (nolevelsDuration.count()), actual: 7633023951 vs 7479205571
  [  FAILED  ] LevelManagerPerfrormance.LevelVsNoLevel (16266 ms)
  [Dbg] [SimulationRunner.cc:511] Exiting postupdate worker thread (0)
  [Dbg] [level_manager.cc:73] 
  Using levels = 6947 ms
  Without levels = 6796 ms
  /github/workspace/test/performance/level_manager.cc:78: Failure
  Expected: (levelsDuration.count()) <= (nolevelsDuration.count()), actual: 6947880294 vs 6796243004
  [  FAILED  ] LevelManagerPerfrormance.LevelVsNoLevel (14824 ms)
@scpeters scpeters added bug Something isn't working tests Broken or missing tests / testing infra performance Runtime performance labels May 19, 2021
@scpeters scpeters mentioned this issue May 19, 2021
@darksylinc
Copy link
Contributor

I'm also on Bionic and saw this failure. I wasn't sure if it was my fault (I was in my own custom branch when it failed)

@nkoenig
Copy link
Contributor

nkoenig commented Oct 20, 2021

Fixed in #1129

@nkoenig nkoenig closed this as completed Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working performance Runtime performance tests Broken or missing tests / testing infra
Projects
None yet
Development

No branches or pull requests

3 participants