这是indexloc提供的服务，不要输入任何密码
Skip to content
This repository was archived by the owner on Feb 13, 2025. It is now read-only.

bosun: Requiring no schedule lock to render dashboard. #1197

Merged
merged 1 commit into from
Jul 30, 2015
Merged

bosun: Requiring no schedule lock to render dashboard. #1197

merged 1 commit into from
Jul 30, 2015
+55 −11

Conversation

@captncraig
Copy link
Contributor

@captncraig captncraig commented Jul 23, 2015

Two main fixes:

  1. Reading silences does not require schedule lock, but a more localized silence lock, which is a RWMutex that should take very few Writes.
  2. Schedule now has a readStatus that is a copy of the authoritative status. This should be copied in full at the end of each RunHistory.

@captncraig captncraig mentioned this pull request Jul 23, 2015
Closed
6 tasks
@kylebrandt
Copy link
Member

kylebrandt commented Jul 27, 2015

Need to pass tests of course...

captncraig added a commit that referenced this pull request Jul 30, 2015
@captncraig
Merge pull request #1197 from bosun-monitor/noLocksForDash
27b3332 
bosun: Requiring no schedule lock to render dashboard.
@captncraig captncraig merged commit 27b3332 into master Jul 30, 2015
@captncraig captncraig deleted the noLocksForDash branch July 30, 2015 21:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@captncraig captncraig

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@captncraig @kylebrandt