add comment about poor error handling when headscale isnt becoming available
This commit is contained in:
parent
2090a13dcd
commit
772541afab
1 changed files with 5 additions and 1 deletions
|
@ -258,7 +258,11 @@ func (s *IntegrationTestSuite) SetupSuite() {
|
||||||
}
|
}
|
||||||
return nil
|
return nil
|
||||||
}); err != nil {
|
}); err != nil {
|
||||||
log.Fatalf("Could not connect to docker: %s", err)
|
// TODO(kradalby): If we cannot access headscale, or any other fatal error during
|
||||||
|
// test setup, we need to abort and tear down. However, testify does not seem to
|
||||||
|
// support that at the moment:
|
||||||
|
// https://github.com/stretchr/testify/issues/849
|
||||||
|
return // fmt.Errorf("Could not connect to headscale: %s", err)
|
||||||
}
|
}
|
||||||
fmt.Println("headscale container is ready")
|
fmt.Println("headscale container is ready")
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue