Skip to content

[ci] Fix Travis jobs timing out#3017

Merged
olivielpeau merged 4 commits into
masterfrom
olivielpeau/fix-postgres-ci
Nov 11, 2016
Merged

[ci] Fix Travis jobs timing out#3017
olivielpeau merged 4 commits into
masterfrom
olivielpeau/fix-postgres-ci

Conversation

@olivielpeau

@olivielpeau olivielpeau commented Nov 10, 2016

Copy link
Copy Markdown
Member

Fixes:

  • Travis jobs that were timing out by re-enabling the cleanup phase (for some reason some jobs on Travis wouldn't exit unless the processes that are spawned are all killed)
  • Postgres setup phase (psql's default user works fine for the first connection, and for some reason the USER env var wasn't set correctly on the Travis env)

@olivielpeau
olivielpeau force-pushed the olivielpeau/fix-postgres-ci branch from 0b6d475 to 372e52c Compare November 10, 2016 20:45
- Run FluentD as a daemon
- Increase rabbitmq timeout
@olivielpeau olivielpeau changed the title [postgres][ci] Fix user that connects to CI postgres db [ci] Fix Travis jobs timing out Nov 11, 2016
If we remove the conf during the `before_cache` stage we can't stop
apache afterwards. Instead move the conf file out of the cached dir
and move it back in apache's dir before stopping.
@olivielpeau
olivielpeau force-pushed the olivielpeau/fix-postgres-ci branch from c7190ac to 768d5b7 Compare November 11, 2016 01:28

@degemer degemer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@olivielpeau
olivielpeau merged commit 3e575c0 into master Nov 11, 2016
@olivielpeau
olivielpeau deleted the olivielpeau/fix-postgres-ci branch November 11, 2016 16:04
degemer added a commit that referenced this pull request Nov 15, 2016
* master: (254 commits)
  Reduce the maximum amount of argument to pylint.
  [forwarder] stop flushing after 10s
  [etcd] Report errors connecting to etcd endpoint (#3007)
  Postfix check should pass raise_on_empty_output=False
  [status] Silence requests exception
  [psutil] Only set `psutil.PROCFS_PATH` once in the collector (#3013)
  [ci] fix check name detection of test files (#3021)
  [ci][rabbitmq] Increase wait timeout (#3022)
  [core] SpooledTemporaryFile for subprocess output (#3002)
  [collector] isolate system checks (#3001)
  [ci] Fix Travis jobs timing out (#3017)
  [mongo] Use db.current_op instead of manually querying (#3016)
  [ci] fix bad citizens detection (#3020)
  [ci] add debug logs (#3019)
  [mongo] use `currentOp` for mongodb 3.2+
  Use proxy for API key check in info page (#3012)
  [mongo] Add MongoDB 3.2 support
  [mongo] Add MongoDB 3.2 to travis-ci config
  [packaging] 5.11.0 nightlies (#3009)
  [core] hard-deprecate start/stop/restart/status (#3004)
  ...
truthbk pushed a commit that referenced this pull request Nov 16, 2016
* [postgres][ci] Fix user that connects to CI postgres db

* [ci] Do run the cleanup step on Travis

For some reason some jobs on Travis wouldn't exit
unless the processes that are spawned are all killed

* [ci] Improve reliability on Travis

- Run FluentD as a daemon
- Increase rabbitmq timeout

* [ci][apache] Fix cleanup

If we remove the conf during the `before_cache` stage we can't stop
apache afterwards. Instead move the conf file out of the cached dir
and move it back in apache's dir before stopping.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants