Tolerate errors dumping logs in pipeline tests (#2391)

Errors are ignored in two cases more now: when the status of the stack
cannot be obtained, and when using the compose provider, when the
elasticsearch service doesn't exist.

This should help on executions of pipeline tests against existing
servers using only environment variables on default configuration.

Author: jsoriano

internal/stack/dump.go

@@ -71,7 +71,7 @@ func dumpStackLogs(ctx context.Context, options DumpOptions) ([]DumpResult, erro
 
 	for _, requestedService := range options.Services {
 		if !slices.Contains(services, requestedService) {
-			return nil, fmt.Errorf("local service %s does not exist", requestedService)
+			return nil, fmt.Errorf("%w: local service %s does not exist", ErrUnavailableStack, requestedService)
 		}
 	}
 

internal/testrunner/runners/pipeline/tester.go

@@ -235,14 +235,23 @@ func (r *tester) checkElasticsearchLogs(ctx context.Context, startTesting time.T
 
 	testingTime := startTesting.Truncate(time.Second)
 
+	statusOptions := stack.Options{
+		Profile: r.profile,
+	}
+	_, err := r.provider.Status(ctx, statusOptions)
+	if err != nil {
+		logger.Debugf("Not checking Elasticsearch logs: %s", err)
+		return nil, nil
+	}
+
 	dumpOptions := stack.DumpOptions{
 		Profile:  r.profile,
 		Services: []string{"elasticsearch"},
 		Since:    testingTime,
 	}
 	dump, err := r.provider.Dump(ctx, dumpOptions)
 	var notImplementedErr *stack.ErrNotImplemented
-	if errors.As(err, &notImplementedErr) {
+	if errors.As(err, &notImplementedErr) || errors.Is(err, stack.ErrUnavailableStack) {
 		logger.Debugf("Not checking Elasticsearch logs: %s", err)
 		return nil, nil
 	}