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

chore: change Docker broker configs #2353

Closed
wants to merge 8 commits into from
Closed

Conversation

harbu
Copy link
Contributor

@harbu harbu commented Feb 2, 2024

Summary

Replace 10.200.10.1 with container hostnames.

Changes

Provide a bullet list of individual changes. Leave this section out if change
set is small and obvious from summary.

Limitations and future improvements

Provide a bullet list or description of known omissions or limitations of the
solution and/or any ideas for future improvement. Leave section ouf if
not applicable.

Checklist before requesting a review

  • Is this a breaking change? If it is, be clear in summary.
  • Read through code myself one more time.
  • Make sure any and all TODO comments left behind are meant to be left in.
  • Has reasonable passing test coverage?
  • Updated changelog if applicable.
  • Updated documentation if applicable.

@github-actions github-actions bot added the broker Related to Broker Package label Feb 2, 2024
@harbu harbu changed the title change graphUrl hostname adjust broker configs Feb 5, 2024
@harbu harbu changed the title adjust broker configs chore: change Docker broker configs Feb 6, 2024
@harbu
Copy link
Contributor Author

harbu commented Feb 12, 2024

Doesn't seem to be needed after all

@harbu harbu closed this Feb 12, 2024
@harbu harbu deleted the adjust-broker-docker-configs branch July 4, 2024 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
broker Related to Broker Package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant