lint(scripts): apply shellcheck suggestions on dhall checking script

ci: add shellcheck to lint shell scripts in automatic tests

deploy: revert back to git push + polling

In #19 we removed the polling script to rely on the clever cli's
ability to wait on `clever deploy` command.

After more than 6 months of experimentation it seems the command is
still not reliable (sometimes the command never returns, sometimes it
returns even when the deployment has finished..).

Thus we are reverting back to our manual process of polling `clever
activity` every 5 seconds.

The timeout for this polling is set at the Ansible level (not in the
bash script) and set for 40 minutes. This should give enough time for
any of our applications to deploy.

Merge pull request #48 from clementd-fretlink/remove-ssh-key

Remove `sshKey` from the vault type

Remove `sshKey` from the vault type

It's not actually used by the role, but by code that uses this role.

Merge pull request #47 from clementd-fretlink/debug

Fix `CC_DISABLE_METRICS` computation

Fix `CC_DISABLE_METRICS` computation

Merge pull request #46 from clementd-fretlink/cleanup

update metrics and entrypoint default definitions

Clever Cloud metrics are now enabled by default

Merge pull request #45 from clementd-fretlink/new-clever-vault

Clean up `dhall` type definitions

dhall: remove optional arguments from mkConfig

It was just constructing a record without adding any information. Now
it constructs a record from its mandatory parts. The caller can then
override fields as they please.

dhall: expose optional fields as optional values

Instead of `nonifying` them

Remove default environment from Dhall config

remove org_id from vault type, add sshKey in vault

Merge pull request #44 from gaetanfl/fix_shell_pipefail

use bash with -o pipefail

use bash with -o pipefail

Merge pull request #42 from paulrbr-fl/deploy-errors-on-application-is-up-to-date

fix(deploy): fail the task correctly when last deploymt failed

fix(deploy): fail the task correctly when last deploymt failed

Merge pull request #41 from paulrbr-fl/dhall-version-upgrade

dhall(upgrade): upgrade code to be compatible with dhall 1.21.0

dhall(upgrade): upgrade code to be compatible with dhall 1.21.0

Merge pull request #40 from gaetanfl/fix_ansible_lint_4.1

fix(lint): ansible lint 4.1.0 compatibiliy

fix(lint): ansible 4.1.0 compatibiliy

Merge pull request #39 from paulrbr-fl/fix-login

Fix login by removing any existing linked app

login: make sure no existing apps are linked to the clever tools

Revert "login: force the `.clever.json` file to be created"

This reverts commit 85d1a7838d032ec7dfe858fce5dca3f5535199e4.

Merge pull request #38 from paulrbr-fl/fix-login

login: force the `.clever.json` file to be created

login: force the `.clever.json` file to be created

In case a project already has a `.clever.json` file (for debugging
purpose to communicate with dev environment for instance) this step
was never done because of the `creates:`.

Removing it forces the role to always create a new clever.json file.

Merge pull request #37 from gaetanfl/fix_path

Missing /

Missing /

Merge pull request #36 from gaetanfl/fix_typo

Fix duplicate version number in url

Fix duplicate version number in url

Merge pull request #35 from gaetanfl/use_path_tool

use full path for clever helper invocation

liniting

use full path for clever helper invocation

Merge pull request #34 from paulrbr-fl/domain-case

domain: DNS names are case insensitive

domain: DNS names are case insensitive

Merge pull request #33 from paulrbr-fl/successful-deploy-up-to-date

deploy: check "up-to-date" error to succeeded if last commit is OK

deploy: check "up-to-date" error to succeeded if last commit is OK

Merge pull request #22 from paulrbr-fl/fix-v1.10

Upgrade clever tools to latest version

fix: Fail module takes one argument

deploy_task: Update clever CLI tools and use it instead of git push

Updating to latest version of 1.1.1 helps to have a stable `clever
deploy` command.

This commit relies on the output of the command instead of manually
pooling the results with `clever activity` command.

Merge pull request #32 from clementd-fretlink/quote-env

fix(deploy): properly quote variables in environment files

fix(deploy): properly quote variables in environment files

adding exports in the env file also quotes the variable values

tags: add clever-env tag to post-deploy tasks

Merge pull request #31 from paulrbr-fl/fix-tags

fix: statically include sub tasks for 'tags' to work

fix: statically include sub tasks for 'tags' to work

In ansible the `include_tasks` loads the tasks dynamically. Thus it
does not apply the tag specified at the include level to all sub
tasks.

In order to make sure all included tasks inherit the specified `tags:`
we need to include the tasks files "statically". It is very similar to
cases where we have a `when:` condition that we want to apply to all
included subtasks. cf https://docs.ansible.com/ansible/latest/user_guide/playbooks_conditionals.html#applying-when-to-roles-imports-and-includes

Merge pull request #30 from paulrbr-fl/rename-entry-point

app: rename `clever_entry_point`. It's only used by Haskell apps

app: rename `clever_entry_point`. It's only used by Haskell apps

Merge pull request #29 from paulrbr-fl/missing-chdir

fix: Yet again a missing chdir for "clever" command

fix: Yet again a missing chdir for "clever" command

This is a followup of #26

Merge pull request #28 from paulrbr-fl/fix-post-deploy

Fix: post depoy tasks were not launched. This fixes it.

Fix: post depoy tasks were not launched. This fixes it.

Merge pull request #27 from paulrbr-fl/chdir-all-the-things

fix(deploy): Yet again a missing chdir to execute from app root

fix(deploy): Yet again a missing chdir to execute from app root

Merge pull request #26 from paulrbr-fl/chdir-all-the-things

fix(deploy): Make sure to execute clever commands in app_root dir

fix(deploy): Make sure to execute clever commands in app_root dir

Merge pull request #25 from paulrbr-fl/feat-get-env

feat(post_deploy): Add a post deploy task to fetch deployed env

feat(post_deploy): Add a post deploy task to fetch deployed env

In some cases you might need to retrieve the ENV which is currently in
use by your clever application recently deployed.

If you specify a `clever_env_output_file` that is exactly what you
will get!

Merge pull request #24 from paulrbr-fl/revert-clever-cli-update

Reverting upgrade of clever-tools CLI

Revert "Merge pull request #19 from paulrbr-fl/update-clever-tools"

This reverts commit 695ceaa28821660e17a1cf819cffebe1b5d8c7ca, reversing
changes made to 5485dbc4db3e141673457475bcda5fc88057739f.

Revert "Merge pull request #20 from paulrbr-fl/update-clever-tools"

This reverts commit b17a839d622bdb36ddbe2e0667559c33e9b7fd8a, reversing
changes made to 695ceaa28821660e17a1cf819cffebe1b5d8c7ca.

Merge pull request #21 from haitlahcen/fix-scripts

avoid hardcoded bash path in scripts

avoid hardcoded bash path in scripts

Merge pull request #20 from paulrbr-fl/update-clever-tools

Fix: this is a 🤦‍♂️  PR fixing a bug introduced in v1.10

fix: Fail module takes one argument

Merge pull request #19 from paulrbr-fl/update-clever-tools

deploy_task: Update clever CLI tools and use it instead of git push

deploy_task: Update clever CLI tools and use it instead of git push

Updating to latest version of 1.1.1 helps to have a stable `clever
deploy` command.

This commit relies on the output of the command instead of manually
pooling the results with `clever activity` command.

Merge pull request #18 from haitlahcen/dhall-interface

add dhall interface

update default value (readme aswell) for `clever_metrics`

add travis ci step for typechecking dhall

add dhall interface

Merge pull request #17 from gaetanfl/change_clever_app_root_default

change default value for clever_app_root

change default value for clever_app_root

Merge pull request #16 from paulrbr-fl/fix-clever-app-root-for-good

fix: 😬 really fix the usage of "clever_app_root" when waiting deploy

fix: 😬 really fix the usage of "clever_app_root" when waiting deploy

Followup after the fix of #15 it was still missing another case of
"git" usage.

This should be the last fix about this variable 🤞.

Merge pull request #15 from paulrbr-fl/fix-git-push

fix: clever_app_root is provided & exec is outside of "parent dir"

fix: clever_app_root is provided & exec is outside of "parent dir"

Imagine such a usecase:
- project is in `/project`
- executing Ansible in `/ansible` with `clever_app_root: /project`
will fail deploying with the git push with the following error:
```
Not a git repository (or any of the parent directories): .git"
```

This PR fixes this.

Merge pull request #14 from paulrbr-fl/rename-syslog-server-var

vars: add a clever_syslog_server variable replacing syslog_server

vars: add a clever_syslog_server variable replacing syslog_server

All variables "namespaced" with the role name is helpful to read
"client" configurations

Merge pull request #13 from paulrbr-fl/prefix-domain-variable

domain: allow the domain to be specified via clever_domain variable

domain: allow the domain to be specified via clever_domain variable

Every other variables are prefixed with `clever_` which makes the
definition of variables clearer from a "app" configuration point of
view.

WDYT?

Merge pull request #12 from paulrbr-fl/first-time-deploy

deploy: Make sure repo is unshallow copy before pushing to clever

deploy: Make sure repo is unshallow copy before pushing to clever

If your build tool only uses a shallow clone of your project, Clever
cloud will fail on first time git push. You need to git push a full
clone of your git repo for the first time deployment to clever.

This PR fixes that.

Merge pull request #11 from gaetanfl/lint

add ansible-lint test and fix warning

Install ansible-lint

add ansible-lint test and fix warning

Merge pull request #10 from paulrbr-fl/python3-compat

python3: Change iteritems() to items() to be compatible with py3

python3: Change iteritems() to items() to be compatible with py3

warnings: Remove warning given by Ansible 2.5

```
[DEPRECATION WARNING]: Using tests as filters is deprecated. Instead of using `result|failed` instead
use `result is failed`. This feature will be removed in version 2.9. Deprecation warnings can be
disabled by setting deprecation_warnings=False in ansible.cfg.
```

Merge pull request #9 from paulrbr-fl/ci-status-img

readme: Add the CI status as a image badge

readme: Add the CI status as a image badge

Merge pull request #8 from fmenou/nix

Use Nix for build

Use Nix for build

Update Travis-Slack integration

Merge pull request #7 from fmenou/better-wait-deploy

Better wait deploy - take #2

Watch and wait for Clever

Fix typo

Merge pull request #5 from fmenou/metrics-support

Fix bool dump

Fix bool dump

Merge pull request #4 from fmenou/metrics-support

Metrics support