Merge pull request #29015 from edx/revert-28885-iamsobanjaved/check-django32-make-upgrade

Revert "feat: upgrade Django version to 3.2"
This commit is contained in:
Feanil Patel
2021-10-13 16:05:09 -04:00
committed by GitHub
8 changed files with 45 additions and 14 deletions

View File

@@ -106,7 +106,13 @@ compile-requirements: $(COMMON_CONSTRAINTS_TXT) ## Re-compile *.in requirements
# This is a temporary solution to override the real common_constraints.txt
# In edx-lint, until the pyjwt constraint in edx-lint has been removed.
# See BOM-271 for more details.
sed 's/Django<2.3//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
sed 's/pyjwt\[crypto\]<2.0.0//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
mv requirements/common_constraints.tmp requirements/common_constraints.txt
sed 's/social-auth-core<4.0.3//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
mv requirements/common_constraints.tmp requirements/common_constraints.txt
sed 's/edx-drf-extensions<7.0.0//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
mv requirements/common_constraints.tmp requirements/common_constraints.txt
sed 's/edx-auth-backends<4.0.0//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
mv requirements/common_constraints.tmp requirements/common_constraints.txt
sed 's/drf-jwt<1.19.1//g' requirements/common_constraints.txt > requirements/common_constraints.tmp
mv requirements/common_constraints.tmp requirements/common_constraints.txt
@@ -126,6 +132,17 @@ compile-requirements: $(COMMON_CONSTRAINTS_TXT) ## Re-compile *.in requirements
sed '/^[dD]jango==/d' requirements/edx/testing.txt > requirements/edx/testing.tmp
mv requirements/edx/testing.tmp requirements/edx/testing.txt
# to avoid multiple entries remove it from django and django30 first and add later.
sed -i '/^[dD]jango-cookies-samesite==/d' requirements/edx/django.txt
sed -i '/^[dD]jango-cookies-samesite==/d' requirements/edx/django30.txt
grep -e "^django-cookies-samesite==" requirements/edx/base.txt >> requirements/edx/django.txt
grep -e "^django-cookies-samesite==" requirements/edx/base.txt >> requirements/edx/django30.txt
# removing django-cookies-samesite from all testing.txt file. It is not require for django32.
# at the time of django32 final upgrade job remove this package from base.in
sed '/^[dD]jango-cookies-samesite==/d' requirements/edx/testing.txt > requirements/edx/testing.tmp
mv requirements/edx/testing.tmp requirements/edx/testing.txt
upgrade: pre-requirements ## update the pip requirements files to use the latest releases satisfying our constraints
$(MAKE) compile-requirements COMPILE_OPTS="--upgrade"

View File

@@ -18,10 +18,10 @@
# using LTS django version
Django<2.3
# latest version is causing e2e failures in edx-platform.
# See pyjwt[crypto]<2.0.0 comment.
# See comment.
# elasticsearch>=7.14.0 includes breaking changes in it which caused issues in discovery upgrade process.

View File

@@ -16,7 +16,7 @@
celery<5.0.0
# edx-platform currently only supported for Django 2.2.x
django<3.3
django<2.3
# Newer versions need celery >= 5.0
django-celery-results<2.1

View File

@@ -43,6 +43,7 @@ defusedxml
Django # Web application framework
django-appconf
django-celery-results # Only used for the CacheBackend for celery results
django-cookies-samesite # Middleware which allows SameSite=None flag for session and csrf cookies in Django<3.0.5
django-config-models # Configuration models for Django allowing config management with auditing
django-cors-headers # Used to allow to configure CORS headers for cross-domain requests
django-countries # Country data for Django forms and model fields

View File

@@ -44,8 +44,6 @@ aniso8601==9.0.1
# via edx-tincan-py35
appdirs==1.4.4
# via fs
asgiref==3.4.1
# via django
async-timeout==3.0.1
# via aiohttp
attrs==21.2.0
@@ -163,8 +161,9 @@ defusedxml==0.7.1
# social-auth-core
deprecated==1.2.13
# via jwcrypto
django==3.2.8
django==2.2.24
# via
# -c requirements/edx/../common_constraints.txt
# -c requirements/edx/../constraints.txt
# -r requirements/edx/base.in
# django-appconf
@@ -242,6 +241,8 @@ django-config-models==2.2.0
# edx-enterprise
# edx-name-affirmation
# lti-consumer-xblock
django-cookies-samesite==0.9.0
# via -r requirements/edx/base.in
django-cors-headers==3.10.0
# via -r requirements/edx/base.in
django-countries==7.2.1
@@ -990,6 +991,8 @@ tqdm==4.62.3
# via nltk
typing-extensions==3.10.0.2
# via aiohttp
ua-parser==0.10.0
# via django-cookies-samesite
unicodecsv==0.14.1
# via
# -r requirements/edx/base.in

View File

@@ -57,7 +57,6 @@ appdirs==1.4.4
asgiref==3.4.1
# via
# -r requirements/edx/testing.txt
# django
# uvicorn
astroid==2.6.6
# via
@@ -239,8 +238,9 @@ distlib==0.3.3
# via
# -r requirements/edx/testing.txt
# virtualenv
django==3.2.8
django==2.2.24
# via
# -c requirements/edx/../common_constraints.txt
# -c requirements/edx/../constraints.txt
# -r requirements/edx/testing.txt
# django-appconf
@@ -323,6 +323,8 @@ django-config-models==2.2.0
# edx-enterprise
# edx-name-affirmation
# lti-consumer-xblock
django-cookies-samesite==0.9.0
# via -r requirements/edx/testing.txt
django-cors-headers==3.10.0
# via -r requirements/edx/testing.txt
django-countries==7.2.1
@@ -1436,6 +1438,10 @@ typing-extensions==3.10.0.2
# gitpython
# mypy
# pydantic
ua-parser==0.10.0
# via
# -r requirements/edx/testing.txt
# django-cookies-samesite
unicodecsv==0.14.1
# via
# -r requirements/edx/testing.txt

View File

@@ -1 +1,2 @@
django==3.2.8
django==2.2.24
django-cookies-samesite==0.9.0

View File

@@ -53,10 +53,7 @@ appdirs==1.4.4
# -r requirements/edx/base.txt
# fs
asgiref==3.4.1
# via
# -r requirements/edx/base.txt
# django
# uvicorn
# via uvicorn
astroid==2.6.6
# via
# pylint
@@ -229,6 +226,7 @@ diff-cover==4.0.0
distlib==0.3.3
# via virtualenv
# via
# -c requirements/edx/../common_constraints.txt
# -c requirements/edx/../constraints.txt
# -r requirements/edx/base.txt
# django-appconf
@@ -310,6 +308,7 @@ django-config-models==2.2.0
# edx-enterprise
# edx-name-affirmation
# lti-consumer-xblock
# via -r requirements/edx/base.txt
django-cors-headers==3.10.0
# via -r requirements/edx/base.txt
django-countries==7.2.1
@@ -1325,6 +1324,10 @@ typing-extensions==3.10.0.2
# aiohttp
# gitpython
# pydantic
ua-parser==0.10.0
# via
# -r requirements/edx/base.txt
# django-cookies-samesite
unicodecsv==0.14.1
# via
# -r requirements/edx/base.txt