8000 fix test timeout handling by dothebart · Pull Request #11118 · arangodb/arangodb · GitHub
[go: up one dir, main page]
Skip to content

fix test timeout handling #11118

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Feb 19, 2020
Merged

fix test timeout handling #11118

merged 4 commits into from
Feb 19, 2020

Conversation

dothebart
Copy link
Contributor

This PR changes the test timeout from effectively 4.5 minutes to 10 minutes.

http://jenkins.arangodb.biz:8080/job/arangodb-matrix-pr-linux/10354/EDITION=community,STORAGE_ENGINE=mmfiles,TEST_SUITE=single,limit=(linux&&test)%7C%7Cgce/ would show errnous timeout behaviour; rspec would continue running.

running with a 15 seconds timeout will now do:

./scripts/unittest http_replication --test tests/rb/HttpReplication/api-replication-global-spec.rb --oneTestTimeout 3
...
      fetches the empty follow log
      tails the WAL with a tick far in the future
Killing rspec - ["--color","-I","tests/arangodbRspecLib","--format","d","--format","j","--out","/tmp/arangosh_Cso8Bt/http_replication/testresult.json","--require","/tmp/arangosh_Cso8Bt/http_replication/testconfig.rb","tests/rb/HttpReplication/api-replication-global-spec.rb"]
oops! Skipping remaining tests, server is gone.
Tue Feb 18 2020 14:21:04 GMT+0100 (Central European Standard Time) Shutting down...
...
\n  status: ABORTED\n  signal: 6\nmarking build as crashy."}}
done.
not cleaning up as some tests weren't successful:
["/tmp/arangosh_Cso8Bt/http_replication"] false - false - true


================================================================================'
TEST RESULTS
================================================================================

* Test "http_replication"
    [FAILED]  SKIPPED

      "test" failed: 

    [FAILED]  tests/rb/HttpReplication/api-replication-global-spec.rb

      "test" failed: server crashed

 * Overall state: Fail [http_replication] :     
    during: shutdown timeout; instance forcefully KILLED because of fatal timeout in testrun : Core dump written; Process facts :
    role: single
    port: 4630
    endpoint: 'tcp://127.0.0.1:4630'
    rootDir: /tmp/arangosh_Cso8Bt/http_replication
    url: 'http://127.0.0.1:4630'
    args:
      configuration: /home/willi/src/devel3/etc/testing/arangod-single.conf
      define: TOP_DIR=/home/willi/src/devel3
      wal.flush-timeout: 30000
      javascript.app-path: /tmp/arangosh_Cso8Bt/http_replication/apps
      javascript.copy-installation: false
      http.trusted-origin: all
      cluster.create-waits-for-sync-replication: false
      temp.path: /tmp/arangosh_Cso8Bt/http_replication/tmp
      server.storage-engine: rocksdb
      server.endpoint: 'tcp://127.0.0.1:4630'
      database.directory: /tmp/arangosh_Cso8Bt/http_replication/data
      log.file: /tmp/arangosh_Cso8Bt/http_replication/log
    pid: 3385988
    name: single - 4630
    stats:
      minorPageFaults: 8655
      majorPageFaults: 0
      userTime: 0.76
      systemTime: 0.07
      numberOfThreads: 57
      residentSize: 348999680
      residentSizePercent: 0.005172577284515001
      virtualSize: 1088315392
      rchar: 13044061
      wchar: 329269
      syscr: 2073
      syscw: 145
      read_bytes: 8192
      write_bytes: 466944
      cancelled_write_bytes: 4096
    exitStatus:
      pid: 3385988
      status: ABORTED
      signal: 6
    marking build as crashy.
Marking crashy!
   Suites failed: 2 Tests Failed: 0

@dothebart dothebart requested review from jsteemann and KVS85 February 18, 2020 13:25
@dothebart
Copy link
Contributor Author

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/8620/

@dothebart
Copy link
Contributor Author
dothebart commented Feb 18, 2020 •
edited
Loading

tests green except for enterprise merge time sequence. Launching full matrix as discussed:
http://jenkins01.arangodb.biz:8080/view/Windows/job/arangodb-ANY-windows-matrix/983/
http://jenkins01.arangodb.biz:8080/view/Mac/job/arangodb-ANY-mac-matrix/787/

@dothebart
Copy link
Contributor Author

tests green enough.

@jsteemann jsteemann merged commit c9abf9d into devel Feb 19, 2020
@jsteemann jsteemann deleted the bug-fix/fix-external-test-timeout branch February 19, 2020 09:55
dothebart added a commit that referenced this pull request Feb 19, 2020
@dothebart
fix / implement linux test timeout handling (#11118)
1b4240c
@dothebart dothebart mentioned this pull request Feb 19, 2020
Merged
KVS85 pushed a commit that referenced this pull request Feb 20, 2020
@dothebart
Bug fix 3.6/fix external test timeout (#11127)
f9ba2eb 
* fix jemaloc integration

* allow post body on delete, as we did previously

* fix / implement linux test timeout handling (#11118)
ObiWahn added a commit that referenced this pull request Feb 21, 2020
@ObiWahn
Merge remote-tracking branch 'origin/devel' into feature/validation-p…
11e90b2 
…hase-2

* origin/devel:
  fix python invocation (<
8BB5
a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="568953711" data-permission-text="Title is private" data-url="https://github.com/arangodb/arangodb/issues/11142" data-hovercard-type="pull_request" data-hovercard-url="/arangodb/arangodb/pull/11142/hovercard" href="https://github.com/arangodb/arangodb/pull/11142">#11142)
  increase timeout to 15 minutes (#11141)
  Update CHANGELOG
  Docs: Remove misleading statement ...about the non-existing API /_api/database/properties
  Update CHANGELOG
  Bug fix/fixes 19022020 (#11136)
  Move subqueries down (#11135)
  Update OskarTestSuitesBlackList (#11132)
  fix shrink method in aqlitemblock (#11122)
  Documentation/rotate jwt docublocks (#11120)
  remove non-deterministic test assumptions
  fix test timeout handling (#11118)
KVS85 pushed a commit that referenced this pull request Feb 26, 2020
@dothebart @KVS85
fix test timeout handling (#11118)
a8272c1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsteemann jsteemann Awaiting requested review from jsteemann

@KVS85 KVS85 Awaiting requested review from KVS85

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dothebart @jsteemann
0