Project

General

Profile

Actions

Bug #69912

closed

pybind/mgr: autocommit creating transactions automatically

Added by Patrick Donnelly about 1 year ago. Updated 5 months ago.

Status:
Resolved
Priority:
Urgent
Target version:
% Done:

0%

Source:
Community (user)
Backport:
squid,reef,quincy
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Tags (freeform):
backport_processed
Fixed In:
v19.3.0-7585-g4b3d91abce
Released In:
v20.2.0~1121
Upkeep Timestamp:
2025-11-01T01:35:33+00:00

Description

First reported here:

https://github.com/ceph/ceph/pull/57199#issuecomment-2484817235

I don't really understand why I didn't catch this in my original work on that PR but here we are. I believe the solution to make the code consistent for both autocommit or isolation_level (when autocommit is unavailable) is to set autocommit=LEGACY_TRANSACTION_CONTROL).

Tests on Ubuntu 24.04 are necessary to confirm this.


Related issues 3 (0 open3 closed)

Copied to cephsqlite - Backport #69959: reef: pybind/mgr: autocommit creating transactions automaticallyResolvedPatrick DonnellyActions
Copied to cephsqlite - Backport #69960: squid: pybind/mgr: autocommit creating transactions automaticallyResolvedPatrick DonnellyActions
Copied to cephsqlite - Backport #69961: quincy: pybind/mgr: autocommit creating transactions automaticallyRejectedPatrick DonnellyActions
Actions #1

Updated by Patrick Donnelly about 1 year ago

  • Description updated (diff)
  • Status changed from New to In Progress
  • Assignee set to Patrick Donnelly
  • Priority changed from Normal to Urgent
  • Target version set to v20.0.0
  • Source set to Community (user)
  • Backport set to squid,reef,quincy
Actions #2

Updated by Patrick Donnelly about 1 year ago

  • Blocks Backport #65730: squid: ceph-mgr critical error: "Module 'devicehealth' has failed: table Device already exists" added
Actions #3

Updated by Patrick Donnelly about 1 year ago

  • Blocks Backport #65731: reef: ceph-mgr critical error: "Module 'devicehealth' has failed: table Device already exists" added
Actions #4

Updated by Patrick Donnelly about 1 year ago

  • Blocks deleted (Backport #65730: squid: ceph-mgr critical error: "Module 'devicehealth' has failed: table Device already exists")
Actions #5

Updated by Patrick Donnelly about 1 year ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 61763
Actions #6

Updated by Patrick Donnelly about 1 year ago

  • Status changed from Fix Under Review to Pending Backport
Actions #7

Updated by Upkeep Bot about 1 year ago

  • Copied to Backport #69959: reef: pybind/mgr: autocommit creating transactions automatically added
Actions #8

Updated by Upkeep Bot about 1 year ago

  • Copied to Backport #69960: squid: pybind/mgr: autocommit creating transactions automatically added
Actions #9

Updated by Upkeep Bot about 1 year ago

  • Copied to Backport #69961: quincy: pybind/mgr: autocommit creating transactions automatically added
Actions #10

Updated by Upkeep Bot about 1 year ago

  • Tags (freeform) set to backport_processed
Actions #11

Updated by Patrick Donnelly about 1 year ago

  • Blocks deleted (Backport #65731: reef: ceph-mgr critical error: "Module 'devicehealth' has failed: table Device already exists")
Actions #12

Updated by Upkeep Bot 9 months ago

  • Status changed from Pending Backport to Resolved
  • Upkeep Timestamp set to 2025-07-08T14:47:19+00:00
Actions #13

Updated by Upkeep Bot 8 months ago

  • Merge Commit set to 4b3d91abcef88514316b8d66452798f4a5562a88
  • Fixed In set to v19.3.0-7585-g4b3d91abce
  • Upkeep Timestamp changed from 2025-07-08T14:47:19+00:00 to 2025-08-01T10:43:41+00:00
Actions #14

Updated by Upkeep Bot 5 months ago

  • Released In set to v20.2.0~1121
  • Upkeep Timestamp changed from 2025-08-01T10:43:41+00:00 to 2025-11-01T01:35:33+00:00
Actions

Also available in: Atom PDF