Remove variable SESSION_SERIALIZER from settings.py

Bug #1349463 reported by Christian Berendt
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Wishlist
Matthias Runge

Bug Description

Check if it is possible to remove the variable SESSION_SERIALIZER from settings.py and test/settings.py during the K-cycle.

Julie Pichon (jpichon)
Changed in horizon:
importance: Undecided → Wishlist
Revision history for this message
Dirk Mueller (dmllr) wrote :

Can you give a rationale on why removing this is a good idea?

Revision history for this message
Julie Pichon (jpichon) wrote :

I believe this is a related discussion: http://lists.openstack.org/pipermail/openstack-dev/2013-October/016876.html

We don't really want to stick with pickle, but I don't think the work required to switch to the JsonSerializer (which is the Django default since 1.6) has been investigated yet.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/161117

Changed in horizon:
assignee: nobody → Matthias Runge (mrunge)
status: New → In Progress
Revision history for this message
Matthias Runge (mrunge) wrote :

Turns out, it's not that a brilliant idea to change this right now.

Changed in horizon:
assignee: Matthias Runge (mrunge) → nobody
status: In Progress → New
Matthias Runge (mrunge)
Changed in horizon:
status: New → Fix Released
assignee: nobody → Matthias Runge (mrunge)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Ivan Kolodyazhny (<email address hidden>) on branch: master
Review: https://review.openstack.org/574310
Reason: This review is > 4 months without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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.