8000 Fix for issue #109, Monitor unit test failing non-deterministically by mathisrichter · Pull Request #110 · lava-nc/lava · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix for issue #109, Monitor unit test failing non-deterministically #110

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 27, 2021
Merged

Fix for issue #109, Monitor unit test failing non-deterministically #110

merged 1 commit into from
Nov 27, 2021

Conversation

mathisrichter
Copy link
Contributor

Signed-off-by: Mathis Richter mathis.richter@intel.com

@mathisrichter
Fix for issue lava-nc#109, where a unit test for the Monitor failed n…
544119d 
…on-deterministically

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
@mathisrichter mathisrichter added the 1-bug Something isn't working label Nov 26, 2021
@mathisrichter mathisrichter added this to the Release v0.2.0 milestone Nov 26, 2021
@mathisrichter mathisrichter self-assigned this Nov 26, 2021
@mathisrichter mathisrichter linked an issue Nov 26, 2021 that may be closed by this pull request
Monitor unit test non-deterministically fails in CI/CD #109
Closed
awintel
awintel approved these changes Nov 26, 2021
View reviewed changes
Copy link
Contributor
@awintel awintel left a comment

Choose a reason for hiding this comment

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

Looks good.

@mgkwill mgkwill merged commit 1fac102 into lava-nc:main Nov 27, 2021
monkin77 pushed a commit to monkin77/thesis-lava that referenced this pull request Jul 12, 2024
@mathisrichter
Fix for issue lava-nc#109, where a unit test for the Monitor failed n…
52b431f 
…on-deterministically (lava-nc#110)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mgkwill mgkwill mgkwill approved these changes

@awintel awintel awintel approved these changes

@srrisbud srrisbud Awaiting requested review from srrisbud

@elvinhajizada elvinhajizada Awaiting requested review from elvinhajizada

@PhilippPlank PhilippPlank Awaiting requested review from PhilippPlank

Assignees

@mathisrichter mathisrichter

Labels
1-bug Something isn't working
Projects
None yet
Milestone
Release v0.2.0
Development

Successfully merging this pull request may close these issues.

Monitor unit test non-deterministically fails in CI/CD
3 participants
@mathisrichter @mgkwill @awintel
0