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

Fix the instructions for connecting to XFELs broker offline #20

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

JamesWrigley
Copy link
Member

I'm not sure how this worked originally, but when testing it I found that the KafkaConsumer will connect to the bootstrap server (localhost) correctly with the AMORE_BROKER variable set, but it will then try to connect to exflwebstor01.desy.de and hang. I'm guessing that when it connects to the bootstrap server it gets a list of all brokers and then tries to connect to them; and since it sees localhost and exflwebstor01.desy.de as different brokers it'll try to connect to exflwebstor01.desy.de too.

(cherry-picked from one of my giant branches)

@JamesWrigley JamesWrigley self-assigned this Jul 5, 2023
@takluyver takluyver added the documentation Improvements or additions to documentation label Jul 6, 2023
@takluyver
Copy link
Member

this is only accessible inside the control network.

I think we mean inside the DESY network? The control network is more restricted - I don't think it includes Maxwell.

Other than that, LGTM.

I'm not sure how this worked originally, but when testing it I found that the
KafkaConsumer will connect to the bootstrap server (localhost) correctly with
the AMORE_BROKER variable set, but it will then try to connect to
exflwebstor01.desy.de and hang. I'm guessing that when it connects to the
bootstrap server it gets a list of all brokers and then tries to connect to
them; and since it sees localhost and exflwebstor01.desy.de as different brokers
it'll try to connect to exflwebstor01.desy.de too.
@JamesWrigley
Copy link
Member Author

Oops, yes you're quite right. Fixed in 30095dd.

@JamesWrigley JamesWrigley merged commit 36c355c into master Jul 7, 2023
1 check passed
@JamesWrigley JamesWrigley deleted the kafka-docs branch July 7, 2023 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants