/reviewed-on nexedi/slapos.core!183

…t makes no sense to test it

/reviewed-on nexedi/slapos.core!182

Our condition was only checking executables, which missed some non
executable libraries.

Following up on nexedi/slapos.core!172 (comment 96033)

/reviewed-on nexedi/slapos.core!180

@luke , As we discussed. Merging dierctly.

Tests passing in
https://nexedi.erp5.net/test_result_module/20200117-9F5C0733

/reviewed-on nexedi/slapos.core!181

Slave will display state of the Software Instance which it is attached to.

The reported state is informed by the partition giving user more accurate view on the state, so it can know if the partition was processed or not.

  This allow us define if the instance is failing to change state (start/stop) or failing after started or stopped

  Invoke activities are not expected on this test, as there are
  others partitions that can allocate before we call the API we
  would like to test.

/reviewed-on nexedi/slapos.core!179

During teardown, we sometimes have errors that processes are still
running after being stopped, something like:
    
    Fault: <Fault 91: 'STILL_RUNNING: TestTelegraf-0'>

This is because we only retried `slapos node report` two times, but
`slapos node report` tells supervisor to stop processes without
waiting and when processes takes time to stop, this is sometimes not
enough. Increase to retry up to 10 times by default.

/reviewed-on !178

This means the user gave up, so we cancel so the new payment is created or Subscription Request is cancelled.

On some test machines, we saw test failures because openssl version mismatch:

```
Traceback (most recent call last):
  File "/srv/slapgrid/slappart14/srv/testnode/cqg/soft/18d28ecd49e5e0f732e2ce0c00c57519/parts/slapos.core-repository/slapos/testing/testcase.py", line 227, in installSoftwareUrlList
    checkSoftware(cls.slap, software_url)
  File "/srv/slapgrid/slappart14/srv/testnode/cqg/soft/18d28ecd49e5e0f732e2ce0c00c57519/parts/slapos.core-repository/slapos/testing/testcase.py", line 211, in checkSoftware
    raise RuntimeError('\n'.join(error_list))
RuntimeError: ./parts/proftpd/libexec/mod_auth_web.so:
./parts/proftpd/libexec/mod_auth_web.so: /usr/lib/x86_64-linux-gnu/libssl.so.1.1: version `OPENSSL_1_1_1' not found (required by /srv/slapgrid/slappart14/srv/testnode/cqg/inst/test0-0/tmp/shared/curl/724c785f86fff6993ff24ef745d4dbb9/lib/libcurl.so.4)
```

What we see in this traceback is that the version of openssl used was the one from system ( in `/usr/lib/` ), which should not happen because we are using setting rpath to slapos libraries. We had a test checking for missing rpath that could not resolved, but our test did not catch missing rpath that can be resolved on a system library, which is also wrong because this system library might be a different version like here or even be missing.

/reviewed-on !172

Except a short whitelist of system libraries, we should have always set
rpath so that the slapos version of the library is used.

We had issues with components which where compiled with slapos version
of openssl but linked without rpath. At runtime they were using system
openssl.

Rewrite our complex grep shell pipeline in python, with a function
parsing ldd output so that we can check not only for not found dynamic
libraries, but also for libraries that are found outside of slapos
library dirs.

This also fixes problems that this grep pipeline did not support paths
with spaces in their names.

Each time slave partition changes (it is added or updated) the master
partition timestamp has to be recalculated.

/reviewed-on !176

Fix long lines, incorrect type annotations and adjust a bit to make mypy
happy.

Sometimes instanciation fail because some service do not start and the
"check port listening" promises fail. In this case, it's very important
to also have the partition content to see what's in the log files.

/reviewed-on nexedi/slapos.core!175

the pagination doesn't work because
SoftwareInstance_getConnectionParameterList returns always all
parameters.

/reviewed-on nexedi/slapos.core!174

/reviewed-on !173

  'since' should be unmutable.

Also define a short cache, to prevent creation explosion