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

[prober] Minor refactoring. #625

Merged
merged 2 commits into from
Nov 6, 2023
Merged

[prober] Minor refactoring. #625

merged 2 commits into from
Nov 6, 2023

Conversation

manugarg
Copy link
Contributor

@manugarg manugarg commented Nov 6, 2023

  • Use read-write mutex, instead of regular mutex, to control access to cloudprober config info.
  • Get raw and parsed config from the config source directly instead of storing it ahead of time.
  • Allow access to the prober instance.

@manugarg
[prober] Minor refactoring.
714e8ae 
- Use read-write locks to control access to cloudprober config info.
- Get raw and parsed config from the config source directly instead of
  storing it ahead of time.
- Allow access to the prober instance.
@manugarg
Add more tests
7fc2f60
@manugarg manugarg merged commit 1e70553 into master Nov 6, 2023
7 of 8 checks passed
@manugarg manugarg deleted the minor_refactoring branch November 6, 2023 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@manugarg