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

NOISSUE - Improve MQTT benchmarking tools #828

Merged
merged 6 commits into from
Aug 30, 2019
Merged

Conversation

drasko
Copy link
Contributor

@drasko drasko commented Aug 29, 2019

No description provided.

@codecov-io
Copy link

codecov-io commented Aug 29, 2019

Codecov Report

Merging #828 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #828      +/-   ##
==========================================
- Coverage   85.57%   85.53%   -0.05%     
==========================================
  Files          72       72              
  Lines        4701     4701              
==========================================
- Hits         4023     4021       -2     
- Misses        442      444       +2     
  Partials      236      236
Impacted Files Coverage Δ
things/service.go 86.61% <0%> (-1.41%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bfd1230...557424b. Read the comment docs.

@@ -0,0 +1,25 @@
[mqtt]
[mqtt.broker]
url = "tcp://localhost:1883"
Copy link
Contributor

Choose a reason for hiding this comment

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

tcps://localhost:8883

mteodor
mteodor previously approved these changes Aug 30, 2019
@@ -0,0 +1,25 @@
[mqtt]
[mqtt.broker]
url = "tcp://localhost:1883"
Copy link
Contributor

Choose a reason for hiding this comment

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

tcps://localhost:8883

cd benchmark
go build -o mqtt-bench *.go
drasko@Marx:~/go/src/github.com/mainflux/mainflux/tools/mqtt-bench$ ./mqtt-bench --help
Tool for exctensive load and benchmarking of MQTT brokers used withing Mainflux platform.
Copy link
Contributor

Choose a reason for hiding this comment

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

'withing Mainflux'

The tool supports multiple concurrent clients, publishers and subscribers configurable message size, etc:

```
cd benchmark
go build -o mqtt-bench *.go
drasko@Marx:~/go/src/github.com/mainflux/mainflux/tools/mqtt-bench$ ./mqtt-bench --help
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove your prompt. If really want the path to be there, exchange it with cd $GOPATH/mainflux/mainflux... and then the help command

Signed-off-by: drasko <[email protected]>
@drasko drasko merged commit f090573 into absmach:master Aug 30, 2019
dborovcanin pushed a commit to dborovcanin/supermq that referenced this pull request Sep 2, 2019
* Refactor mqtt-bench

Signed-off-by: drasko <[email protected]>

* Refactor mqtt-bench and provision tools

Signed-off-by: drasko <[email protected]>

* Improve README

Signed-off-by: drasko <[email protected]>

* Remove provision generated toml file

Signed-off-by: drasko <[email protected]>

* Update README

Signed-off-by: drasko <[email protected]>

* Replace `tcp` with `tcps` for mtls

Signed-off-by: drasko <[email protected]>
manuio pushed a commit that referenced this pull request Oct 12, 2020
* Refactor mqtt-bench

Signed-off-by: drasko <[email protected]>

* Refactor mqtt-bench and provision tools

Signed-off-by: drasko <[email protected]>

* Improve README

Signed-off-by: drasko <[email protected]>

* Remove provision generated toml file

Signed-off-by: drasko <[email protected]>

* Update README

Signed-off-by: drasko <[email protected]>

* Replace `tcp` with `tcps` for mtls

Signed-off-by: drasko <[email protected]>
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.

4 participants