GNIP 98: Django upgrade to 4.2 LTS

[mattiagiupponi]

GNIP 98 - Django upgrade to 4.2 LTS

Overview

As pointed out by the community, soon Django 3.2 will be deprecated in April 2024 in favor of 4.2

Proposed By

@mattiagiupponi
@cesar-benjamin

Assigned to Release

This proposal is for GeoNode 4.2/master.

State

• Under Discussion
• In Progress
• Completed
• Rejected
• Deferred

Motivation

Django 3.2 will be no longer supported.

Proposal

This upgrade can be quite challenging due to the different geonode dependencies as also mentioned in #11722.
Many geonode dependencies are not maintained and this is blocking the upgrade.
So far the libraries that are raising issues with django 4.2 (list in progress):

• dj-pagination
• dynamic-rest
• geonode-announcements
• geonode-django-activity-stream
• geonode-OAuth-toolkit
• geonode-user-messages
• pinax-ratings

For the above libraries, we should evaluate if:

• Fork the original repo and maintain it
• remove the functionality in geonode or find a library maintained that provides the same functionality

Other than the libraries, there are some fixes to be done:

• The {% ifequal %} and {% ifnotequal %} template tags are removed.
• django.conf.urls.url() is removed.

More info here https://docs.djangoproject.com/en/4.2/releases/4.0/#features-removed-in-4-0

Notes

An initial work has been done by @cesar-benjamin and is available here

Feedback

Update this section with relevant feedback, if any.

Voting

Project Steering Committee:

• Alessio Fabiani:
• Francesco Bartoli:
• Giovanni Allegri:
• Simone Dalmasso:
• Toni Schoenbuchner: ✓
• Florian Hoedt:

Links

Remove unused links below.

• Email Discussion
• Pull Request
• Mail Discussion
• Linked Issue

[mattiagiupponi]

For who is interested, i was able to upgrade geonode by changing the minimum as possible.
A draft PR is available here #11829

In the end, the main changes are:

• Upgrade django to 4.2.9
• Upgrade resframework to 3.14.0
• upgrade django-autocomplete-light to 3.9.5
• Dynamic rest is not compatible (neither the master version) to make some test I used another fork. Ofc it will be needed to fork the main repo under GeoNode
• pinax-ratings is not mantained anymore, i used the upgraded version done by @cesar-benjamin Ofc it will be needed to fork the main repo under GeoNode
• pinax-notifications is not mantained anymore, i used an upgraded fork already available Ofc it will be needed to fork the main repo under GeoNode
• geonode-mapstore-client and geonode-importer are already on Geonode, so i just update them
• geonode-user-messages and geonode-announcements are already internal forks, so i just updated them
• geonode-django-activity-stream was not providing anything new to the original library, so i upgrade to the original library instead: django-activity-stream==2.0.0

What is missing:

• Keyword and Thesaurus select in metadata editor are not working anymore
• Evaluate to upgrade also some other libraries
• Complete test run

[cesarbenjamindotnet]

Thanks, I made a lot of forks to upgrade the broken deps and so on. I did saw that keywords and thesaurus will not work because some strange in autocomplete light, but regions uses autocomplete too but in another way, maybe is a good point to standardize all the forms in a clean way. And dropping old dependencies like jQuery etc. I see in mapstore client at user profile a thing that calls AngularJS... Why there is at the same time react AngularJS and jQuery? This can become be unmaintainable...

[mattiagiupponi]

maybe is a good point to standardize all the forms in a clean way. And dropping old dependencies like jQuery etc. I see in mapstore client at user profile a thing that calls AngularJS... Why there is at the same time react AngularJS and jQuery? This can become be unmaintainable...

Starting from version 4.0, we moved the FE from Geonode to the mapstore client.
The wizard upgrade is tracked here #11511.

I suggest leaving this matter out of the issue/PR and instead focusing on upgrading Django to a newer version without disrupting backward compatibility.

[mattiagiupponi]

sa decided with @giohappy we will remove pinax-ratings

[mattiagiupponi]

as decided with @giohappy we will remove django-haystack==3.2.1 and elasticsearch