Use followOutput for lra-coordinator container

[marcosgopen]

#39390

Adding lra-coordinator container log in microprofile-lra tck tests logs.

[marcosgopen]

@mmusgrov

[mmusgrov]

I approve of the PR but I suspect that there needs to be an associated issue.

[mmusgrov]

@gsmet does @marcosgopen need a related issue in order to trigger a review?

[mmusgrov]

Good PR, we need these logs to be able to effectively debug issues with the LRA extension.

[xstefank]

Is slf4j appropriate for this?

I don't know if we should use the logs here if it's not custom with anything else.

[mmusgrov]

Is slf4j appropriate for this?

testcontainers relies on SLF4J for logging

I don't know if we should use the logs here if it's not custom with anything else.

What do you mean by "use the logs" and what does "if it's not custom ..." mean?

Can you suggest alternate functionality that will give us what we need?

[xstefank]

@mmusgrov @marcosgopen together with Marco we created an integration job in lra-coordinator-quarkus that'll also test the image which is the correct fix for this problem - https://github.com/jbosstm/lra-coordinator-quarkus/blob/main/.github/workflows/build.yml#L62-L99. This PR can be closed.

[mmusgrov]

@xstefank Thanks.

So can we go ahead and ask someone from the quarkus team to merge this PR?

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 41e2a2b.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.