LP_DEBUG_SQL broken
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Gary Poster |
Bug Description
With Gary's changes for ++profile++ to include data akin to LP_DEBUG_
$ LP_DEBUG_SQL=1 make harness
bin/harness
execute_
Reading $PYTHONSTARTUP...
Initializing storm...
Creating the factory...
>>> from lp.bugs.model.bug import Bug
>>> Bug.get(1)
<Bug at 0x807f350>
>>>
Reverting r13771 made them re-appear for me again:
$ bzr merge -r13771..13770 .
$ LP_DEBUG_SQL=1 make harness
bin/harness
execute_
Reading $PYTHONSTARTUP...
Initializing storm...
Creating the factory...
>>> from lp.bugs.model.bug import Bug
>>> Bug.get(1)
SELECT Bug.date_
-------
<Bug at 0x711f090>
>>>
Now, this is a regression, but not a user facing regression, so I am not sure if it should be marked Critical.
Related branches
- Benji York (community): Approve (code)
-
Diff: 432 lines (+151/-73)4 files modifiedlib/canonical/launchpad/webapp/adapter.py (+30/-9)
lib/lp/services/profile/profile.py (+13/-4)
lib/lp/services/stacktrace.py (+56/-42)
lib/lp/services/tests/test_stacktrace.py (+52/-18)
Changed in launchpad: | |
assignee: | Gary Poster (gary) → Данило Шеган (danilo) |
status: | Triaged → In Progress |
Changed in launchpad: | |
assignee: | Данило Шеган (danilo) → Gary Poster (gary) |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in launchpad: | |
status: | Fix Committed → Fix Released |
Gary, assigning to you just so you can share your opinion.