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

UPSTREAM: 36248: Fix possible race in operationNotSupportedCache #11791

Merged
merged 1 commit into from
Nov 7, 2016
Merged

UPSTREAM: 36248: Fix possible race in operationNotSupportedCache #11791

merged 1 commit into from
Nov 7, 2016

Conversation

ncdc
Copy link
Contributor

@ncdc ncdc commented Nov 4, 2016

Because we can run multiple workers to delete namespaces simultaneously, the
operationNotSupportedCache needs to be guarded with a mutex to avoid concurrent
map read/write errors.

@liggitt

Fixes #11584

UPSTREAM: 36248: Fix possible race in operationNotSupportedCache
0b30b0d 
Because we can run multiple workers to delete namespaces simultaneously, the
operationNotSupportedCache needs to be guarded with a mutex to avoid concurrent
map read/write errors.
@ncdc
Copy link
Contributor Author

ncdc commented Nov 4, 2016

[test]

@liggitt
Copy link
Contributor

liggitt commented Nov 4, 2016

LGTM

@liggitt liggitt added this to the 1.4.0 milestone Nov 4, 2016
@ncdc
Copy link
Contributor Author

ncdc commented Nov 4, 2016

#11452 [test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 0b30b0d

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11155/) (Base Commit: da2c207)

@liggitt
Copy link
Contributor

liggitt commented Nov 5, 2016

[merge]

@ncdc
Copy link
Contributor Author

ncdc commented Nov 5, 2016

[merge]

On Saturday, November 5, 2016, OpenShift Bot notifications@github.com
wrote:

continuous-integration/openshift-jenkins/merge FAILURE (
https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11175/) (Base
Commit: 046fad4
046fad4
)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#11791 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABYq2qfpe-xpcpzW-W-iH33lN0HX5Aks5q7Fx6gaJpZM4Kp17F
.

@ncdc
Copy link
Contributor Author

ncdc commented Nov 5, 2016

[merge]

On Saturday, November 5, 2016, OpenShift Bot notifications@github.com
wrote:

continuous-integration/openshift-jenkins/merge FAILURE (
https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11176/) (Base
Commit: 046fad4
046fad4
)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#11791 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABYvluUFOjV_xuZkT5vIv2wAYhx0uYks5q7HiTgaJpZM4Kp17F
.

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 5, 2016 •
edited
Loading

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11155/) (Image: devenv-rhel7_5328)

@ncdc
Copy link
Contributor Author

ncdc commented Nov 7, 2016

#11452 [merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 0b30b0d

@openshift-bot openshift-bot merged commit 65fca62 into openshift:master Nov 7, 2016
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
1.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@ncdc @liggitt @openshift-bot