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

MQTT role #38

Merged
merged 8 commits into from
Feb 28, 2025
Merged

MQTT role #38

merged 8 commits into from
Feb 28, 2025

Conversation

codyro
Copy link
Member

@codyro codyro commented Oct 31, 2024

Waiting on information from the build team about the configuration required to finish.

  • Restrict 1883 depending on network requirements (VPC/Overlay)
  • Bind to proper interface (priv/pub/tap)
  • Tweak configuration & distribute acl/psk file(s)

@codyro codyro self-assigned this Oct 31, 2024
@codyro codyro requested a review from jonathanspw November 4, 2024 18:11
@codyro
Copy link
Member Author

codyro commented Nov 4, 2024

Should be good sans any tweaking regarding configs/acls as per build teams needs.

Has been ran on the instance.

@codyro codyro marked this pull request as ready for review November 4, 2024 18:27
@codyro codyro changed the title DRAFT: MQTT role MQTT role Nov 12, 2024
@@ -0,0 +1,32 @@
---
albs_web_ip: 10.0.18.39
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is fine for now but long term it should get wrapped into a larger ALBS role at which point we may want to rename the vars.

@jonathanspw jonathanspw merged commit 6c4d1fa into master Feb 28, 2025
0 of 2 checks passed
@jonathanspw jonathanspw deleted the mqtt-role branch February 28, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants