907985 - [traceback] IntegrityError: (1062, "Duplicate entry '4918855-127' for key 'user_id'")

Status: VERIFIED FIXED

(Marketplace Graveyard :: Reviewer Tools, defect, P1)

Description

[krupa raj[:krupa]]

steps to reproduce:
1. Load https://marketplace-dev.allizom.org/reviewers/apps/review/test-webapi-permissions-16
2. Add a comment and save changes

sentry: http://sentry.dmz.phx1.mozilla.com/addons/marketplace-dev/group/16626/

traceback details:


IntegrityError: (1062, "Duplicate entry '4918855-127' for key 'user_id'")

Stacktrace (most recent call last):

  File "django/core/handlers/base.py", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "newrelic/api/object_wrapper.py", line 216, in __call__
    self._nr_instance, args, kwargs)
  File "newrelic/hooks/framework_django.py", line 475, in wrapper
    return wrapped(*args, **kwargs)
  File "django/db/transaction.py", line 224, in inner
    return func(*args, **kwargs)
  File "amo/decorators.py", line 32, in wrapper
    return func(request, *args, **kw)
  File "amo/decorators.py", line 62, in wrapper
    return f(request, *args, **kw)
  File "addons/decorators.py", line 33, in wrapper
    return f(request, addon, *args, **kw)
  File "mkt/reviewers/views.py", line 379, in app_review
    resp = _review(request, addon, version)
  File "mkt/reviewers/views.py", line 308, in _review
    form.helper.process()
  File "mkt/reviewers/utils.py", line 558, in process
    return self.actions[action]['method']()
  File "mkt/reviewers/utils.py", line 370, in process_comment
    self.create_comm_thread(action='comment')
  File "mkt/reviewers/utils.py", line 271, in create_comm_thread
    thread=thread, user=moz_contact)
  File "django/db/models/manager.py", line 137, in create
    return self.get_query_set().create(**kwargs)
  File "django/db/models/query.py", line 377, in create
    obj.save(force_insert=True, using=self.db)
  File "django/db/models/base.py", line 463, in save
    self.save_base(using=using, force_insert=force_insert, force_update=force_update)
  File "django/db/models/base.py", line 551, in save_base
    result = manager._insert([self], fields=fields, return_id=update_pk, using=using, raw=raw)
  File "django/db/models/manager.py", line 203, in _insert
    return insert_query(self.model, objs, fields, **kwargs)
  File "django/db/models/query.py", line 1593, in insert_query
    return query.get_compiler(using=using).execute_sql(return_id)
  File "django/db/models/sql/compiler.py", line 912, in execute_sql
    cursor.execute(sql, params)
  File "django/db/backends/mysql/base.py", line 114, in execute
    return self.cursor.execute(query, args)
  File "newrelic/hooks/database_dbapi2.py", line 36, in execute
    return self._nr_cursor.execute(sql, *args, **kwargs)
  File "MySQLdb/cursors.py", line 173, in execute
    self.errorhandler(self, exc, value)
  File "MySQLdb/connections.py", line 36, in defaulterrorhandler
    raise errorclass, errorvalue

Comment 1

[krupa raj[:krupa]]

same traceback seen on Escalating the app

Comment 2

[Andrew Williamson [:eviljeff]]

these seem to be same traceback:
http://sentry.dmz.phx1.mozilla.com/addons/marketplace-dev/group/16631/
http://sentry.dmz.phx1.mozilla.com/addons/marketplace-dev/group/16629/

Comment 4

[Ashish Dubey [:ashish_d]]

fixed https://github.com/mozilla/zamboni/commit/4ad48f5

Comment 5

[Victor Carciu]

Verified as fixed : I was able to reject or add multiple comments as a reviewer