Skip to content

Conversation

@pan3793
Copy link
Contributor

@pan3793 pan3793 commented Feb 14, 2025

The ClickHouseContainer stopped working recently due to the default user is not allowed to access from outside of the container without setting CLICKHOUSE_PASSWORD. See details at testcontainers/testcontainers-java#9942

/entrypoint.sh: neither CLICKHOUSE_USER nor CLICKHOUSE_PASSWORD is set, disabling network access for user 'default'

Unfortunately, the Scala-wrapped ClickHouseContainer does not expose those args so nothing I can do to workaround the test failure in downstream projects.

Changes are verified locally by

$ sbt moduleClickhouse/test
... 
[info] ClickHouseContainer
[info] - should work
[info] Run completed in 5 seconds, 563 milliseconds.
[info] Total number of tests run: 1
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
[success] Total time: 8 s, completed Feb 14, 2025, 11:31:22 AM

package com.dimafeng.testcontainers

import org.testcontainers.containers.{ClickHouseContainer => JavaClickHouseContainer}
import org.testcontainers.clickhouse.{ClickHouseContainer => JavaClickHouseContainer}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

migration that follows the deprecation message

@pan3793
Copy link
Contributor Author

pan3793 commented Feb 14, 2025

cc @dimafeng

@dimafeng dimafeng merged commit 2ff92a8 into testcontainers:master Feb 14, 2025
3 checks passed
@pan3793
Copy link
Contributor Author

pan3793 commented Feb 14, 2025

@dimafeng thanks for merging this, looking forward to a new release, thank you in advance.

@pan3793
Copy link
Contributor Author

pan3793 commented Feb 19, 2025

@dimafeng I see you created the tag 0.41.9 but seems the artifacts are not available on the maven central. Are there any publishing issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants