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.

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

Make system orchestrator context threadsafe #75

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Make system orchestrator context threadsafe #75

wants to merge 6 commits into from

Conversation

aaronchongth
Copy link
Collaborator

@aaronchongth aaronchongth commented Feb 28, 2025 •
edited
Loading

Resolves #65

@aaronchongth
Threadsafe getters and setters
8232731 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth
Revert tests
9e9548d 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth
Remove unused functions
09a91ae 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth
Isolate with a single test
35f7c46 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth
Checking that only the feedbacks are failing
863e4dd 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth
Set instead of emplace, fix test
fb097e3 
Signed-off-by: Aaron Chong <aaronchongth@gmail.com>
@aaronchongth aaronchongth marked this pull request as ready for review March 3, 2025 10:07
@aaronchongth aaronchongth requested a review from Yadunund March 3, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yadunund Yadunund Awaiting requested review from Yadunund

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make nexus::system_orchestrator::Context thread safe
1 participant
@aaronchongth