Development #50075
tests: nom de base de données trop long
Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
12 Jan 2021
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
malgré le [:63] dans tests/settings.py on peut avoir :
E django.core.exceptions.ImproperlyConfigured: The database name 'test_combo-test-wip-50067-lingo-can-pay-only-once-with-remote-regie' (67 characters) is longer than PostgreSQL's limit of 63 characters. Supply a shorter NAME in settings.DATABASES.
parce qu'il y a un test_ ajouté (je ne sais pas d'où il vient, mais il est là).
Files
Associated revisions
tests: shorten database name, again (#50075)
History
Updated by Thomas Noël 14 days ago
- Patch proposed changed from No to Yes
- Status changed from Nouveau to Solution proposée
- File 0001-tests-shorten-database-name-50075.patch 0001-tests-shorten-database-name-50075.patch added
On passe de
'NAME': 'combo-test-%s' % os.environ.get("BRANCH_NAME", "").replace('/', '-')[:63]
à
'NAME': 'combo-test-%s' % os.environ.get("BRANCH_NAME", "").replace('/', '-')[:55]
parce qu'en plus le :63 ne s'appliquait qu'au nom de la branche, alors que va y être ajouté "test_combo-test-" donc on coupe à 55.
Updated by Emmanuel Cazenave 14 days ago
- Status changed from Solution proposée to Solution validée
Updated by Thomas Noël 14 days ago
- Status changed from Solution validée to Résolu (à déployer)
commit 37eebfff08294d5d435f0ecf492d57df87b05847
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Tue Jan 12 14:04:22 2021 +0100
tests: shorten database name (#50075)
Updated by Thomas Noël 14 days ago
et quel imbécile, erreur de calcul, c'était :45 ... je pousse la triviale correction.
commit 38c7ec1bd606d019a4add99e8e9472f45594dba3
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Tue Jan 12 14:28:59 2021 +0100
tests: shorten database name, again (#50075)
Updated by Frédéric Péters 7 days ago
- Status changed from Résolu (à déployer) to Solution déployée
tests: shorten database name (#50075)