In future, please past flows here directly. In order to make code readable and usable it is necessary to surround your code with three backticks (also known as a left quote or backquote ```)
```
code goes here
```
See this post for more details - How to share code or flow json
Is your mqtt broker running on the same machine as node-red? If so then you should have localhost as the broker address.
Do you mean the the mqtt node is showing 'connected' as its status. If so then that is correct.
