Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add section about clearing the sessions store during OMERO.web upgrade #2312

Merged
merged 14 commits into from
Jul 7, 2023

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented May 4, 2023

See ome/omero-web#435, ome/omero-web#459 and ome/omero-web#383 for more context about the requirement for this new optional section
See also https://docs.djangoproject.com/en/3.2/topics/http/sessions/ for some reference documentation about the different types of Django session store backends.

Motivated by some of the changes introduced in OMERO.web 5.18.0. It is expected the same issues will arise when switching to Django 4.2 and this could happen anytime if the session serialization format is modified.

A new paragraph about common issues when OMERO.server fails to start is also added to the troubleshooting section and this PR should create a link that can be reference from UPGRADING.md in the source code repo.

Motivated by some of the changes introduced in OMERO.web 5.18.0. It is
expected the same issues will arise when switching to Django 4.2 and
this could happen anytime if the session serialization format is modified.

A new paragraph about common issues when OMERO.server fails to start is
also added to the troubleshooting section and this PR should create a
link that can be reference from UPGRADING.md in the source code repo.
@sbesson
Copy link
Member Author

sbesson commented May 4, 2023

Co-authored-by: jean-marie burel <j.burel@dundee.ac.uk>
@jburel
Copy link
Member

jburel commented May 10, 2023

The failure is not related to the change. It started to happen on other repositories. I fixed the issue in ome/omero-guide-python#58
I will go over the repositories using rtd

@jburel jburel closed this May 10, 2023
@jburel jburel reopened this May 10, 2023
@jburel
Copy link
Member

jburel commented May 10, 2023

@sbesson you will have to rebase the PR in order to fix the error

@sbesson
Copy link
Member Author

sbesson commented Jul 3, 2023

This should ready for review and integration in the next documentation release. With the upcoming move to Django 4.2, I expect the session clearing step will be mandatory (unless people have switched to the JSON serializer)

@sbesson sbesson requested review from will-moore and jburel July 3, 2023 16:39
Clarify the two main supported session backends and their configuration
Provide the example for the default configuration in both case
Add final sentence about additional backend configuration
@sbesson
Copy link
Member Author

sbesson commented Jul 4, 2023

@sbesson sbesson requested a review from jburel July 4, 2023 08:05
omero/sysadmins/omeroweb-upgrade.rst Outdated Show resolved Hide resolved
omero/sysadmins/omeroweb-upgrade.rst Outdated Show resolved Hide resolved
@jburel jburel merged commit 9ca8f55 into ome:develop Jul 7, 2023
@sbesson sbesson deleted the omeroweb_clearcache branch July 7, 2023 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants