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

Move expose ports to constructor to avoid duplicate key exception #13212

Conversation

KyleAure
Copy link
Member

Issue originally resolved in Test Containers repo under PR testcontainers/testcontainers-java#2473

But using a custom SQLServerContainer in Liberty to avoid having to provide an accept license file. Applying fix to this custom SQLServerContainer.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • Please describe in a separate comment how you tested your changes.

  • 1 test infrastructure code files were changed.

  • Test failures/errors in the build could be due to these changes.

@KyleAure
Copy link
Member Author

#build

@LibbyBot
Copy link

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_CxXLQNDkEeqmIcsdsdUKbw

Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

@LibbyBot
Copy link

The build KyleAure-13212-20200728-1620
https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_CxXLQNDkEeqmIcsdsdUKbw
completed and has errors or failures.

For help analyzing your personal build, go to https://cognitive.hursley.ibm.com/buildAnalysis.html?uuid=_CxXLQNDkEeqmIcsdsdUKbw

@KyleAure KyleAure merged commit 6e2d610 into OpenLiberty:integration Jul 30, 2020
@KyleAure KyleAure deleted the bb276269-move-exposeport-to-constructor branch July 30, 2020 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants