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

Use wnameless/oracle-xe-11g-r2 for Oracle testing #1522

Closed
wants to merge 1 commit into from

Conversation

candrews
Copy link
Contributor

@candrews candrews commented Oct 3, 2019

According to https://github.com/wnameless/docker-oracle-xe-11g the DMCA issues have been resolved.

This change allows Oracle integration tests to be run.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 3, 2019
@vpavic vpavic self-assigned this Oct 3, 2019
@vpavic
Copy link
Contributor

vpavic commented Oct 3, 2019

I see the use of this image as highly problematic and risky, as there's no license acceptance involved, and you do require to accept one when downloading from Oracle's website. See:

This is in contrast to what we have with e.g. DB2 and SQL Server:

ibmcom/db2:11.5.0.0a
mcr.microsoft.com/mssql/server:2017-CU16

@vpavic vpavic added status: on-hold and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 3, 2019
@candrews
Copy link
Contributor Author

candrews commented Oct 3, 2019

I've asked if they've resolved the issues as their README seems to indicate they have: wnameless/docker-oracle-xe-11g#118 (comment)

@vpavic
Copy link
Contributor

vpavic commented Oct 6, 2019

The point is that I still don't see how are we accepting the license terms by using this image? This is problematic, because as indicated in my previous comment, those are required when you download from Oracle's website.

@marcusdacoregio marcusdacoregio added status: declined A suggestion or change that we don't feel we should currently apply and removed status: on-hold labels Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants