Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove docker-maven-plugin #40

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

urhot
Copy link

@urhot urhot commented Sep 2, 2024

The current version of the docker-maven-plugin is no longer supported, and causes issues with docker hosts where the daemon is not reachable via TCP.

Since integrating docker with the maven build is not currently needed in development, this feature is now completely removed.

The current version of the docker-maven-plugin is no longer supported,
and causes issues with docker hosts where the daemon is not reachable
via TCP.

Since integrating docker with the maven build is not currently needed
in development, this feature is now completely removed.
@urhot urhot requested a review from kkyttala September 2, 2024 12:31
@urhot
Copy link
Author

urhot commented Sep 2, 2024

Integration testing still needs to be sorted out

Previous commit removed spotify's docker-maven-plugin for building,
but fabric8's docker-maven-plugin still remains for integration tests.

This commit bumps up the plugin version to 0.38.1, which should work
with docker using a socket file.

Elasticsearch version is also updated to the same (but still old)
version that is used by the docker-compose setup in the project.
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