-
Notifications
You must be signed in to change notification settings - Fork 70
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
HTML report file when using Gaggle #426
Comments
I can confirm that I am getting the same error. Will investigate.... |
Thanks @slashrsm, the error is gone. However, new problems occur with your branch:
Also, maybe this is unrelated, but this branch doesn't work with goose_eggs. Will update the comment with details. |
It turns out that manager won't write the report (which is what I expected), but the workers will (they seem to inherit the report file configuration from the manager). I think that the correct way to solve this would be to allow
I see the metrics printed on the manager and workers. I started the test with these options manager:
worker:
Could you share your run configuration? |
You're right, the metrics show up correctly. I ran the tests for a too short amount of time and also had a random waiting time built in. So, works as expected, thanks! However, I have already teased that the branch doesn't work with goose_eggs anymore. My code:
The error:
Not related to the HTML report at all, but this works with goose v0.15.2 and not on your fork branch. |
Is it solved? In which release was it merged? |
Unfortunately this is still a lingering bug. And the current release of Goose also currently doesn't support Gaggles, the functionality still needs to be added back: #548 |
How would I generate an HTML report when using a manager node and multiple workers? Is this even possible?
--report-file
flag--run-time
), it throws this error:The text was updated successfully, but these errors were encountered: