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

Initial NanoMQ/EMQX configuration #91

Merged
merged 8 commits into from
Feb 14, 2024

Conversation

jr1221
Copy link
Contributor

@jr1221 jr1221 commented Feb 12, 2024

Changes

Adds basic EMQX and NanoMQ conf files. Includes sqlite backup on TPU, bridging with the IP addresses of TPU and base station, etc. Does not include acl. Also does not include way for one to subscribe to # on EMQX, that can be added later as part of emqx acl.

Nanomq remains masked at startup.

Notes

Below summary of nanomq issues and possible resolutions:

  • Sqlite persistence failure (workaround applied in PR: optimziations disabled)
  • [NanoMQ] - NanoMQ dies when detached #78 (workaround applied in prev. PR: assign nanomq PID with start-stop-daemon)
  • [NanoMQ] - NanoMQ dies on boot when hostapd is present #79 (caused by hostapd, more investigation needed)**
  • Nanomq dies when bridge not up (no workaround, fixed upstream and awaiting release)
  • EMQX soft crashes when MQTTX is used to susbcribe to the same topic from both sides of the bridge (no workaround, issue not a big deal)

Test Cases

  • Send any topic from nanomq to emqx
  • Observe messages sent over bridge
  • Observe sqlite db logging all messages

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please reach out to your Project Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • All commits are tagged with the ticket number
  • No merge conflicts
  • [n/a] All checks passing
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #76

@jr1221 jr1221 self-assigned this Feb 12, 2024
@jr1221 jr1221 linked an issue Feb 12, 2024 that may be closed by this pull request
Copy link
Contributor

@nwdepatie nwdepatie left a comment

Choose a reason for hiding this comment

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

I think I'm gonna trust you on this config lol, seems to pass the sniff test for what we want. We might want to play around with things like the number of threads and such potentially but we can get to that later during testing.

@@ -0,0 +1,7 @@
## EMQX Base Station

Clone repo and cd to this directory.
Copy link
Contributor

Choose a reason for hiding this comment

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

What is this for? Aren't we just going to deploy using the web deployment interface?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't use the web interface simply bc then I knew exactly what command line parameters were being included. I can try this out with the web interface it is just a matter of copying the various settings over. However we still need to clone the repo.

@jr1221 jr1221 merged commit d8d652e into develop/initial-hw-validation Feb 14, 2024
@jr1221 jr1221 deleted the 76-init-nanomq-conf branch February 14, 2024 16:47
jr1221 added a commit that referenced this pull request Mar 2, 2024
* initial config, not yet operable

* basic quic works, still loops

* initial config, not yet operable

* basic quic works, still loops

* include quic file

* more conf fixups, sqlite?

* Update nanomq.mk
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.

[Validation] - NanoMQ configuration
2 participants