search

kubenow / KubeNow

Deploy Kubernetes. Now!
Apache License 2.0
292 stars 40 forks source link

Tune Slack notification #365

Closed mcapuccini closed 6 years ago

mcapuccini commented 6 years ago

Change content and motivation

Tuning Slack notification, so that we get notified only on failure (and if the build status changes).

carmat88 commented 6 years ago

Hey @mcapuccini . The master CI is keep failing even this morning. The main error is unfortunately happening during the setup of the environment. Precisely when shellcheck is being installed:

The command "sudo -E apt-get -yq --no-install-suggests --no-install-recommends --force-yes install shellcheck" failed and exited with 100 during .

I actually reported this to Travis' folks [here] (https://github.com/travis-ci/travis-ci/issues/9361). Eventually as a workaround I decided to remove completely the following code block from .travis.yml:

addons:
  apt:
    sources:
      - debian-sid
    packages:
- shellcheck # code review tool

I was actually surprised at first to see that in the main KubeNow repo it was not failing. Now it seems to be happening, but don't ask me "Why now?" XD Therefore, you may want to change that bit of code and install shellcheck via an apt-get line. Cheers!