diff options
| author | Peter Dillinger <peterd@fb.com> | 2020-12-22 00:19:44 -0800 |
|---|---|---|
| committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | 2020-12-22 00:20:57 -0800 |
| commit | 4d897e51dfe1abfd9b5e9f11db1efd26265b4ba3 (patch) | |
| tree | 497b00a69a02cf95d7dd2fc44302355521a93d52 /.travis.yml | |
| parent | 861b0d1a496fdb7e77ed8027e34d9a2e0a9a4789 (diff) | |
Migrate away from Travis+Linux+amd64 (#7791)
Summary:
This disables Linux/amd64 builds in Travis for PRs, and adds a
gcc-10+c++20 build in CircleCI, which should fill out sufficient coverage
vs. what we had in Travis
Fixed a use of std::is_pod, which is deprecated in c++20
Fixed ++ on a volatile in db_repl_stress.cc, with bigger refactoring.
Although ++ on this volatile was probably ok with one thread writer and
one thread reader, the code was still overly complex. There was a
deadcode check for error
`if (replThread.no_read < dataPump.no_records)` which can be proven
never to happen based on the structure of the code. It infinite loops
instead for the case intended to be checked. I just simplified the code
for what should be the same checking power.
Also most configurations seem to be using make parallelism = 2 * vcores,
so fixing / using that.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/7791
Test Plan:
CI
and `while ./db_repl_stress; do echo again; done` for a while
Reviewed By: siying
Differential Revision: D25669834
Pulled By: pdillinger
fbshipit-source-id: b2c688053d0b1d52c989903449d3cd27a04130d6
Diffstat (limited to '.travis.yml')
| -rw-r--r-- | .travis.yml | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/.travis.yml b/.travis.yml index 51604a86e..2acff576e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -86,19 +86,11 @@ matrix: env: JOB_NAME=make-gcc4.8 - os: linux compiler: clang - # Exclude all but most unique cmake variants for pull requests, but build all in branches + # With migration to CircleCI, exclude Linux/amd64 for pull requests + # (but build in branches for now) - if: type = pull_request AND commit_message !~ /FULL_CI/ os : linux arch: amd64 - env: JOB_NAME=cmake - - if: type = pull_request AND commit_message !~ /FULL_CI/ - os : linux - arch: amd64 - env: JOB_NAME=cmake-gcc8 - - if: type = pull_request AND commit_message !~ /FULL_CI/ - os : linux - arch: amd64 - env: JOB_NAME=cmake-gcc9 # Exclude most osx, arm64 and ppc64le tests for pull requests, but build in branches # Temporarily disable ppc64le cmake test while snapd is broken - if: type = pull_request AND commit_message !~ /FULL_CI/ |