and stop creating some useless directories

To remove GIT_EXEC_PATH from environment as this can break running
buildout with slapos.recipe.build:gitclone sections using an
incompatible git version

because python-language-server-requirements.txt contains a git URL

includes vsx registry and our resolution fixes

log accesses (the logrotate conf already exists for them), and
remove the pattern "<IfDefine Port>" which was used when this
apache conf was extended by the software's apache extending
the monitor stack

Profiles for erp5!948

Testnode expose the log folder for test suite, so that they can expose more logs and snapshots.

Because these can be big, we expose a new "keep-log-days" option so that we can se shorter delay for test nodes running software release tests.

Another important visible change is that testnode now automatically restart when configuration is changed.

See merge request !731

Allow to configure the number of days to keep logs

Use NameWidth=* so that long filenames are never truncated

Use SuppressColumnSorting to disable the ablity to sort so that we can
wget -r all the logs for local analysis

Since 2de4c80f generic_mysql does not
sleep so long between retries

This recipe now uses strings for ip

See merge request !741

See merge request !720

See merge request !739

TODO: There are still direct references to specific version of Python in:
  - slapos component
  - bazel templates
  - nextcloud & theia SR

if last_backup was an int, the script failed with this error:

result undefined, global datetime = <module 'datetime' from '/srv/slapgrid/
slappart6...6d33cee74/lib/python2.7/lib-dynload/datetime.so'>,
datetime.datetime = <type 'datetime.datetime'>,
datetime.datetime.fromtimestamp = <built-in method fromtimestamp of type
object>, last_backup = '1587724103'
<type 'exceptions.TypeError'>: a float is required
      args = ('a float is required',)
      message = 'a float is required'

This was broken in 8725ab1a.

fix the way of getting parameters in the new recipe.

 * fix section pbs in instance-pull-backup.cfg.in after change of
argument names.
 * fix the order of the 2 callbacks as the order is very important