-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
ISSUE-1327 start ambassador only if mapping configured #1328
Conversation
Hi @ftardif, Thanks a lot for fixing it! Could you please add a simple test to verify the fix? |
Do you have a strategy to test these kind of behaviours? I cannot do a unit test by mocking the ambassador generic container because it is instantiated within the class. And I am not too sure how I could do an integration test to verify that socat container is not launched. Any recommendation? |
@ftardif AFAIR it will fail if there are no ports exposed, right? Can we test it? |
No the idea is to work, but without the socat container, only the ryuk and actual containers unwind from the compose file. When the containers already have host port exposed, we can use these for waiting strategies, hence the basic socat check becomes un relevant. |
this PR is meant to fix : #1327 |
reported #1331, will merge this one without a test |
So is #1121 fixed for this is it still a typo? |
OK because it was marked as closed due to this PR. |
Releasing this in 1.11.0 🎉 |
fix #1121
edit: that was a typo, this PR fix: #1327