Skip to content

Conversation

@mcserep
Copy link
Collaborator

@mcserep mcserep commented Oct 15, 2020

Switch to the minimal image in Travis CI from generic.
See differences here: https://docs.travis-ci.com/user/languages/minimal-and-generic/

The generic image has many software preinstalled, e.g.:

  • various database services and tools (Apache Cassandra, CouchDB, ElasticSearch, MongoDB, MySQL, Neo4j, PostgreSQL, RabbitMQ, Redis, Riak, SQLite);
  • go (multiple versions);
  • jvm (multiple versions);
  • nodejs (multiple versions);
  • php (multiple versions);
  • ruby (multiple versions).

We do not need them and they take up a lot of disk space. Therefore we ran out of disk space when performing a self-parse test on CodeCompass. On the minimal image only essential tools (e.g. build tools, version control tools) are included and we have approx. 24GB free disk space which shall be enough.

Additional modifications with this PR related to CI:

  • Only install required Boost components (as in the Docker image, also saving disk space and making the CI jobs a little bit faster).
  • Manual installation of PostgreSQL server as it is no longer bundled in the image.
  • Build and install CodeCompass in $HOME instead of $TRAVIS_BUILD_DIR, so the build and install directories won't be parsed on the self-parse test.

@mcserep mcserep added the Target: Developer environment Developer environment issues consist of CodeCompass or 3rd-party build tooling, configuration or CI. label Oct 15, 2020
@mcserep mcserep added this to the Future milestone Oct 15, 2020
@bruntib
Copy link
Collaborator

bruntib commented Oct 15, 2020

If you would rebase this patch to master (since #429 has been merged) then we can see if the segfaul has really been fixed. I'm excited to see the result :)

Only install required Boost components.
Extend Xenial and Bionic PostgreSQL build with postgresql server install.
Build and install CodeCompass in $HOME instead of $TRAVIS_BUILD_DIR, so the build and install directories won't be parsed on the self-parse test.
Run CodeCompass self-parse test on number of available CPU cores.
@mcserep
Copy link
Collaborator Author

mcserep commented Oct 15, 2020

If you would rebase this patch to master (since #429 has been merged) then we can see if the segfaul has really been fixed. I'm excited to see the result :)

I have applied these modifications to #477, the CI works there. I have rebased this PR now to the new master including USR, let's what happens :)

@mcserep
Copy link
Collaborator Author

mcserep commented Oct 15, 2020

It works! 🏆

I don't understand why it works now and not before. Maybe it was an out of disk space issue after all, but the error message was misleading? (Segmentation fault instead of out of disk space.)

I will get this merged.

@mcserep mcserep merged commit 47210c2 into master Oct 15, 2020
@mcserep mcserep deleted the ci branch October 15, 2020 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Target: Developer environment Developer environment issues consist of CodeCompass or 3rd-party build tooling, configuration or CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants