search

draios / instruqt-assets

3 stars 3 forks source link

[init.sh] fix fix #84

Closed pabloopez closed 1 year ago

pabloopez commented 1 year ago

The last changes are producing more false negatives than ever: agent deployed all right, but we inform about a failed install.

pabloopez commented 1 year ago

Just to have some context, the last changes we worked on last week in draios/instruqt-assets#83 are the cause breaking the tests here.

2023-01-26T22:08:59Z bzowpgescmms INFO: setup-server: Testing if Sysdig Agent is running correctly...
2023-01-26T22:09:57Z bzowpgescmms INFO: setup-server:   FAIL
2023-01-26T22:09:57Z bzowpgescmms INFO: setup-server:   Agent failed to connect to back-end. Check your Agent Key.
mikescholl-sysdig commented 1 year ago

@pabloopez check https://github.com/draios/instruqt-assets/pull/85

We need to add a proper check for a docker install. e.g. when it installs on docker check for the existence of the file mentioned in PR via something like docker exec <containerid> test -e filename

For the host level check we need something like a systemctl status check against the service since this indicates the agent is healthy.