mysql reconnect does not catch 1047 errors

Bug #1312212 reported by Sergey Vasilenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Critical
Max Rasskazov
Mirantis OpenStack
Fix Released
Critical
Dina Belova
Sahara
Fix Released
Undecided
Unassigned
oslo.db
Fix Released
Medium
Davanum Srinivas (DIMS)

Bug Description

Any service using oslo db reconnect feature does not catch 1047 mysqldb error which happens after galera cluster is in non-primary state. this can lead cluster being non-functional even after galera functioning is restored.

Revision history for this message
Sergey Vasilenko (xenolog) wrote :
Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

it shows that our db reconnect patch does not catch 1047 mysqlDB error. we need to fix it

summary: - nova-scheduler can't reconnect to the database
+ mysql reconnect does not catch 1047 errors
Changed in fuel:
importance: Medium → Critical
status: New → Confirmed
description: updated
Changed in fuel:
milestone: none → 5.0
assignee: nobody → Fuel OSCI Team (fuel-osci)
Revision history for this message
Vladimir Kuklin (vkuklin) wrote :

this should be used done fuel-hardening team.

Revision history for this message
Roman Vyalov (r0mikiam) wrote :

need get patches of old git branch (openstack-ci/fuel/havana) and push to fuel-5.0 branch

Changed in fuel:
assignee: Fuel OSCI Team (fuel-osci) → Max Rasskazov (mrasskazov)
Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Max Rasskazov (mrasskazov) wrote :
Revision history for this message
Max Rasskazov (mrasskazov) wrote :
Roman Vyalov (r0mikiam)
Changed in sahara:
status: New → In Progress
Roman Vyalov (r0mikiam)
Changed in fuel:
status: In Progress → Fix Committed
Changed in sahara:
status: In Progress → Fix Committed
Mike Scherbakov (mihgen)
tags: added: release-notes
Revision history for this message
Meg McRoberts (dreidellhasa) wrote :

Added to fixed issues in 5.0 Release Notes.

Thierry Carrez (ttx)
Changed in sahara:
milestone: none → juno-1
status: Fix Committed → Fix Released
tags: added: db
Thierry Carrez (ttx)
Changed in sahara:
milestone: juno-1 → 2014.2
Revision history for this message
Alexander Ignatov (aignatov) wrote :

Reopened this bug because patch https://review.fuel-infra.org/#/c/93/ could not be applied since all components switched to oslo.db . Thus we need to check if oslo.db contains this bug or not.

Changed in mos:
importance: Undecided → Critical
milestone: none → 6.0
status: New → Confirmed
assignee: nobody → MOS Oslo (mos-oslo)
tags: added: ceilometer neutron
tags: added: glance heat keystone
tags: added: cinder murano
Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable-220/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0-stable-220/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable-220/ubuntu

Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0-stable-220/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
RPM Repository URL: http://osci-obs.vm.mirantis.net:82/centos-fuel-6.0-stable/centos

Revision history for this message
OSCI Robot (oscirobot) wrote :

Package oslo.db has been built from changeset: https://review.fuel-infra.org/220
DEB Repository URL: http://osci-obs.vm.mirantis.net:82/ubuntu-fuel-6.0-stable/ubuntu

Changed in mos:
assignee: MOS Oslo (mos-oslo) → Dina Belova (dbelova)
Revision history for this message
Viktor Serhieiev (vsergeyev) wrote :

Can someone provide steps to reproduce this bug?

Revision history for this message
Roman Podoliaka (rpodolyaka) wrote :
Changed in mos:
status: Confirmed → Fix Committed
Revision history for this message
Alexander Gubanov (ogubanov) wrote :

I checked code on mos6.0 (build 56) - fixed

[root@nailgun ~]# fuel --f
api: '1.0'
astute_sha: 16b252d93be6aaa73030b8100cf8c5ca6a970a91
auth_required: true
build_id: 2014-12-18_01-32-01
build_number: '56'
feature_groups:
- mirantis
fuellib_sha: 73332192a257ea02c40a39885c502ad1ebdf3eda
fuelmain_sha: 45caacadb878abfbd9d60e134d72229698b469c9
nailgun_sha: 5f91157daa6798ff522ca9f6d34e7e135f150a90
ostf_sha: a9afb68710d809570460c29d6c3293219d3624d4
production: docker
release: '6.0'
release_versions:
  2014.2-6.0:
    VERSION:
      api: '1.0'
      astute_sha: 16b252d93be6aaa73030b8100cf8c5ca6a970a91
      build_id: 2014-12-18_01-32-01
      build_number: '56'
      feature_groups:
      - mirantis
      fuellib_sha: 73332192a257ea02c40a39885c502ad1ebdf3eda
      fuelmain_sha: 45caacadb878abfbd9d60e134d72229698b469c9
      nailgun_sha: 5f91157daa6798ff522ca9f6d34e7e135f150a90
      ostf_sha: a9afb68710d809570460c29d6c3293219d3624d4
      production: docker
      release: '6.0'

root@node-7:~# grep '1047' /usr/lib/python2.7/dist-packages/oslo/db/sqlalchemy/exc_filters.py
         r".*\((?:2002|2003|2006|2013|1047)")

Changed in mos:
status: Fix Committed → Fix Released
Tom Fifield (fifieldt)
Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/oslo.db (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Dina Belova <email address hidden>
Review: https://review.fuel-infra.org/8272

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote :
Changed in oslo.db:
status: New → In Progress
assignee: nobody → Davanum Srinivas (DIMS) (dims-v)
importance: Undecided → Medium
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/oslo.db (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/8272
Submitter: Victor Sergeyev <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: fd761f6916f7111ded68768e27dcdf29bc551bc7
Author: Dina Belova <email address hidden>
Date: Thu Jul 23 08:32:54 2015

Added catching of errors 1047 (Galera) for MySQL oslo db reconnect

Fixed H238 flake8 errors, which were raised because of the new hacking

Change-Id: I03ae2ca87dfe02a7838576b17dc707b80d199c2c
Closes-Bug: #1312212

Changed in oslo.db:
status: In Progress → Fix Committed
Changed in oslo.db:
milestone: none → 2.5.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.