-
Notifications
You must be signed in to change notification settings - Fork 588
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
Fix containers name in docker compose files. #6105
Conversation
Hi @afsg77 Thanks for the contribution! I have a few questions/comments to clarify why this is needed etc.. 🔹 What behavior exactly are you seeing when you say
A stacktrace may be helpful. Also what is "it" in the sentence above? 🔹 Can you clarify where the host name is currently being set when you say "Rename the containers to match the host name in the properties"? 🔹 If this change is required then it is required in all of the DC files in this directory as all of the container names are prefixed with 🔹 Can you please adjust your commit according to the DCO - there is something it is complaining about currently. |
Rename the containers to match the host name in the properties. Otherwise it fails to connect to the containers from the applications. Remove the 'version: 3' to match the main docker compose file. Signed-off-by: Alejandro Fabian Silva Grife <[email protected]>
f6be7d9
to
fe54d25
Compare
Hi Sure, gladly will clarify it! I was trying guide: https://dataflow.spring.io/docs/installation/local/docker/ With Kafka and Postgres files:
SCDF starts fine with:
But when creating simple pipeline
It should be able to resolve from the service name. But it was not. These are the properties that specified the kafka host:
I tried to update the properties to use the container name, but got similar stacktrace exception:
Renaming the kafka container the same as the service works fine and the log-sink application was able to connect and run the simple pipeline. This might indicate some issue with Docker’s built-in service discovery assigning the hostname to each service. For postgres and other services it seems to be working fine. So, not sure it will require this change in all DC files. The DCO was complaining since I signed with my full name and personal email and my git author had only my single name / last name with my work email. I have amended this in the commit. Please let me know if I'm missing something or if you want me close this PR, create a ticket and then create a PR for the ticket with a different resolution. Or what are the next steps. Thank you for your time! |
Hi @afsg77 , I am unable to reproduce - this is working for me. Please run these commands and report the result:
Also, here are the exact steps I took: Get the compose files...
Start the stack...
Create and deploy stream in the SCDF UI...
At the very end of the log output you should see repeated entries like:
If you don't mind running the request version commands above and then trying the exact steps I posted above that would be helpful. Thank you |
Thats great news @afsg77 ! You are more than welcome. Docker can be flakey wrt to networking etc.. so who knows what it was? Glad its working now though.
Agreed. The best for that would be just to open a separate PR that just removes the version from all of the DC files. We would be most thankful if you could do that. If you don't get to it though - no worries. |
Hi @onobc , Sorry! I deleted the previous message before seeing your reply. I just added some more information. Thanks heaps! I tried the same steps you mentioned and wasn't able to reproduce it. I had some additional configurations:
But even with those changes it's not showing in the simple stream. I do see it sometimes when I create a simple cdc-debezium stream:
But I'm not able to reliably reproduce it. Once it happens it also shows in the simple stream. Anyway thanks! I still think the I can create a different PR if you agree and want me to do that. Really appreciate your time and response. |
No worries. You are more than welcome.
Does it go away if you set
Agreed. The best for that would be just to open a separate PR that just removes the version from all of the DC files. We would be most thankful if you could do that. If you don't get to it though - no worries.
Likewise. It was fun. Take care. |
Rename the containers to match the host name in the properties. Otherwise it fails to connect to the containers from the applications. Remove the 'version: 3' to match the main docker compose file.
Signed-off-by: Alejandro Fabian Silva Grife [email protected]