Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  closed
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:  needsinfo
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Dominik George):

 Nope, it's not a duplicate.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.b854452cb8ec9d775a6aac124ff08903%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  closed
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:  needsinfo
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--
Changes (by Tim Graham):

 * status:  new => closed
 * resolution:   => needsinfo


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.e7d2949985b9e47e6ea3bb47940a068f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Tim Graham):

 I'm skeptical because I'd think that if this were a common issue, we'd
 have seen it reported before now.

 I don't know what can be done. Are you able to offer a patch?

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.882d14fa9b6eb1975e9f08ee715de19a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Dominik George):

 Yeah. I am not saying the CMS isn't part of the problem, but I don't need
 a backtrace to tell that the code in Django is neither thread-safe nor
 otherwise good practice ☺.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.7a8ee3264f3aa920b5c96403ee667263%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Tim Graham):

 I'm not sure. The fact that the traceback involves `multisite/hacks.py`
 (which is not part of Django) looks a bit suspicious.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.a1cb4dfbe04fe7a10471fdd4122d5112%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
+--
 Reporter:  Dominik George  |Owner:  nobody
 Type:  Bug |   Status:  new
Component:  contrib.sites   |  Version:  1.10
 Severity:  Normal  |   Resolution:
 Keywords:  | Triage Stage:  Unreviewed
Has patch:  0   |  Needs documentation:  0
  Needs tests:  0   |  Patch needs improvement:  0
Easy pickings:  0   |UI/UX:  0
+--

Comment (by Dominik George):

 After a discussion at Chemnitzer Linux-Tage with some Django people, we
 agreed it is a bug, but maybe related to Apache mpm_prefork's multi-
 threading behaviour.

 We reduced the threads in each process to 1 in the mod_wsgi config, but
 the error still occured after that.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/070.0f27b28d09c3c19b198f4bda932275fb%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #29202: KeyError in django-sites caching (race condition?)

[Django]

#29202: KeyError in django-sites caching (race condition?)
--+
   Reporter:  Dominik George  |  Owner:  nobody
   Type:  Bug | Status:  new
  Component:  contrib.sites   |Version:  1.10
   Severity:  Normal  |   Keywords:
   Triage Stage:  Unreviewed  |  Has patch:  0
Needs documentation:  0   |Needs tests:  0
Patch needs improvement:  0   |  Easy pickings:  0
  UI/UX:  0   |
--+
 Django Version: 1.10.8
 Python Version: 3.5.3

 Relevant backtrace:

 {{{
 File
 "/srv/www/teckids-website/venv/lib/python3.5/site-
 packages/django/contrib/sites/models.py"
 in _get_site_by_id
   37. return SITE_CACHE[site_id]

 File "/srv/www/teckids-website/venv/lib/python3.5/site-
 packages/multisite/hacks.py" in
 __getitem__
   124. raise KeyError(key)
 }}}

 In theory, there cannot be a KeyError there, unless the cache is cleared
 while the method is running. The code does not seem to be thread-safe.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/055.c52bb260cb200bedbf571c13eb462e32%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.