-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feature: Support for TC_INITSCRIPT
parameter
#26
Comments
Please feel free for pull request |
My use case turned out to be covered by the existing DB user credentials options. |
Could you please clarify what 'copy file' means?
MySQL and MariaDB:
If I understand the proposal correctly, the plugin should accept list of files and copy them into the init folder of the container. Config could look like <copy>
<entries>
<entry>
<sourceFile>filesystem:${project.basedir}/src/test/resources/db/init.sql</sourceFile>
<containerPath>/docker-entrypoint-initdb.d/</containerPath>
</entry>
<entry>
<sourceFile>classpath:db/postgres/init.sh</sourceFile>
<containerPath>/docker-entrypoint-initdb.d/123.sh</containerPath>
</entry>
</entries>
</copy> Current approach in the PR just mimics |
@eddumelendez was referring to using the Copy API. I think your config proposal makes sense @zzzLobster 👍 |
Hey, thanks for this plugin, it's a neat quality of life & code improvement!
While trying to bring this goodie to an internal project I noticed that the JDBC connection parameter TC_INITSCRIPT (and
TC_INITFUNCTION
) are not mentioned / covered by the configuration options.I'm specifically referring to the case when one does not wish to connect to a pre-existing container and thus does not provide a
<jooq><jdbc>...</jdbc></jooq>
block.Did I simply miss this in the documentation?
If not, could this be supported with relative ease?
Possibly naively speaking, I would add this as a nested configuration option e.g.
Happy to look into creating a PR
The text was updated successfully, but these errors were encountered: