Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #6657

closed

Better error messages if collection field doesn't fit DB field length

Added by osapryki over 4 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
fao89
Sprint/Milestone:
0.2.0
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 73
Quarter:

Description

Collection import task should produce human readable error message instead of the traceback below, in case when collection metadata doesn't match database model requirements, such as URL field is bigger than max_length for that field.

Actual behavior:

value too long for type character varying(128)
  File "/venv/lib64/python3.6/site-packages/rq/worker.py", line 884, in perform_job
    rv = job.perform()
  File "/venv/lib64/python3.6/site-packages/rq/job.py", line 664, in perform
    self._result = self._execute()
  File "/venv/lib64/python3.6/site-packages/rq/job.py", line 670, in _execute
    return self.func(*self.args, **self.kwargs)
  File "/venv/lib64/python3.6/site-packages/pulp_ansible/app/tasks/collections.py", line 162, in import_collection
    collection_version.save()
  File "/venv/lib64/python3.6/site-packages/pulp_ansible/app/models.py", line 185, in save
    super().save(*args, **kwargs)
  File "/venv/lib64/python3.6/site-packages/pulpcore/app/models/base.py", line 107, in save
    return super().save(*args, **kwargs)
  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py", line 741, in save
    force_update=force_update, update_fields=update_fields)
  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py", line 779, in save_base
    force_update, using, update_fields,
  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py", line 870, in _save_table
    result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
  File "/venv/lib64/python3.6/site-packages/django/db/models/base.py", line 908, in _do_insert
    using=using, raw=raw)
  File "/venv/lib64/python3.6/site-packages/django/db/models/manager.py", line 82, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/venv/lib64/python3.6/site-packages/django/db/models/query.py", line 1186, in _insert
    return query.get_compiler(using=using).execute_sql(return_id)
  File "/venv/lib64/python3.6/site-packages/django/db/models/sql/compiler.py", line 1375, in execute_sql
    cursor.execute(sql, params)
  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py", line 76, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/venv/lib64/python3.6/site-packages/django/db/utils.py", line 89, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/venv/lib64/python3.6/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
Actions
Copy link
 #1

Updated by bmbouter over 4 years ago

  • Project changed from 34 to Ansible Plugin
Actions
Copy link
 #2

Updated by fao89 over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to fao89
  • Sprint set to Sprint 72
Actions
Copy link
 #3

Updated by fao89 over 4 years ago 

In [1]: CollectionImport.objects.count()                                                                                                                                                      
Out[1]: 1

In [2]: CollectionImport.objects.first().__dict__                                                                                                                                             
Out[2]: 
{'_state': <django.db.models.base.ModelState at 0x7fef92ac5c50>,
 'task_id': UUID('f2a48504-46f3-41ac-9706-44f282da822e'),
 'messages': [{'time': 1589484812.6272595,
   'level': 'INFO',
   'message': 'Getting doc strings via ansible-doc'},
  {'time': 1589484813.1112957,
   'level': 'INFO',
   'message': 'Finding content inside collection'},
  {'time': 1589484813.118735,
   'level': 'INFO',
   'message': 'Loading role minishift_up_role'},
  {'time': 1589484813.1287265,
   'level': 'INFO',
   'message': 'Linting role minishift_up_role via ansible-lint...'},
  {'time': 1589484816.1007335,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:2: [E201] Trailing whitespace'},
  {'time': 1589484816.107979,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:4: [E201] Trailing whitespace'},
  {'time': 1589484816.1112132,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:18: [E201] Trailing whitespace'},
  {'time': 1589484816.1143317,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:20: [E201] Trailing whitespace'},
  {'time': 1589484816.1178918,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:45: [E201] Trailing whitespace'},
  {'time': 1589484816.12133,
   'level': 'WARNING',
   'message': 'minishift_up_role/meta/main.yml:47: [E201] Trailing whitespace'},
  {'time': 1589484816.1250782,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:3: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.1286356,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:7: [E201] Trailing whitespace'},
  {'time': 1589484816.1320767,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:8: [E201] Trailing whitespace'},
  {'time': 1589484816.1355245,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:11: [E201] Trailing whitespace'},
  {'time': 1589484816.1389334,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:13: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.1427157,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:17: [E302] chown used in place of argument owner to file module'},
  {'time': 1589484816.1463225,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:21: [E201] Trailing whitespace'},
  {'time': 1589484816.149592,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:23: [E302] chmod used in place of argument mode to file module'},
  {'time': 1589484816.1529021,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:24: [E201] Trailing whitespace'},
  {'time': 1589484816.1558628,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Darwin_driver.yml:27: [E201] Trailing whitespace'},
  {'time': 1589484816.1591582,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:5: [E201] Trailing whitespace'},
  {'time': 1589484816.1621182,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:7: [E201] Trailing whitespace'},
  {'time': 1589484816.167183,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:10: [E201] Trailing whitespace'},
  {'time': 1589484816.171719,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:19: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.1772206,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:21: [E201] Trailing whitespace'},
  {'time': 1589484816.1828034,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:27: [E201] Trailing whitespace'},
  {'time': 1589484816.185793,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:30: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.1886654,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:30: [E305] Use shell only when shell functionality is required'},
  {'time': 1589484816.1917927,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/Debian_driver.yml:32: [E201] Trailing whitespace'},
  {'time': 1589484816.1945646,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:10: [E201] Trailing whitespace'},
  {'time': 1589484816.1975818,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:16: [E201] Trailing whitespace'},
  {'time': 1589484816.2005541,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:19: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.2034,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:21: [E201] Trailing whitespace'},
  {'time': 1589484816.2061791,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:27: [E201] Trailing whitespace'},
  {'time': 1589484816.2089698,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:30: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.2118006,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:30: [E305] Use shell only when shell functionality is required'},
  {'time': 1589484816.2144017,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/RedHat_driver.yml:31: [E201] Trailing whitespace'},
  {'time': 1589484816.2171144,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/add_hostname.yml:1: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.219762,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/add_hostname.yml:8: [E201] Trailing whitespace'},
  {'time': 1589484816.2226574,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/add_hostname.yml:9: [E201] Trailing whitespace'},
  {'time': 1589484816.2258966,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/add_hostname.yml:11: [E201] Trailing whitespace'},
  {'time': 1589484816.2288508,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/cluster_up.yml:6: [E502] All tasks should be named'},
  {'time': 1589484816.231656,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/cluster_up.yml:7: [E601] Don't compare to literal True/False"},
  {'time': 1589484816.2346747,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/cluster_up.yml:15: [E502] All tasks should be named'},
  {'time': 1589484816.2377443,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/cluster_up.yml:16: [E601] Don't compare to literal True/False"},
  {'time': 1589484816.2408257,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/copy_oc_client.yml:21: [E201] Trailing whitespace'},
  {'time': 1589484816.2437854,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:1: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.2468894,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:1: [E303] mktemp used in place of tempfile module'},
  {'time': 1589484816.249949,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:9: [E201] Trailing whitespace'},
  {'time': 1589484816.2530699,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:10: [E201] Trailing whitespace'},
  {'time': 1589484816.2567987,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:12: [E201] Trailing whitespace'},
  {'time': 1589484816.2602284,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:14: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.2639635,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:15: [E201] Trailing whitespace'},
  {'time': 1589484816.2680395,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/create_objects.yml:17: [E201] Trailing whitespace'},
  {'time': 1589484816.2715418,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_assets.yml:7: [E201] Trailing whitespace'},
  {'time': 1589484816.2749438,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/get_ms_assets.yml:7: [E602] Don't compare to empty string"},
  {'time': 1589484816.2781801,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_assets.yml:12: [E201] Trailing whitespace'},
  {'time': 1589484816.2814295,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/get_ms_assets.yml:12: [E602] Don't compare to empty string"},
  {'time': 1589484816.284856,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_assets.yml:18: [E201] Trailing whitespace'},
  {'time': 1589484816.2880492,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/get_ms_assets.yml:18: [E602] Don't compare to empty string"},
  {'time': 1589484816.2918286,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_assets.yml:23: [E201] Trailing whitespace'},
  {'time': 1589484816.2951405,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/get_ms_assets.yml:23: [E602] Don't compare to empty string"},
  {'time': 1589484816.2991982,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_darwin_asset.yml:1: [E201] Trailing whitespace'},
  {'time': 1589484816.308954,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/get_ms_linux_asset.yml:1: [E201] Trailing whitespace'},
  {'time': 1589484816.315386,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:1: [E201] Trailing whitespace'},
  {'time': 1589484816.320533,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:1: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3252313,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:4: [E201] Trailing whitespace'},
  {'time': 1589484816.3292437,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:4: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3328657,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:7: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3362055,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:9: [E201] Trailing whitespace'},
  {'time': 1589484816.3396056,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:10: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3430452,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/grant_admin.yml:11: [E201] Trailing whitespace'},
  {'time': 1589484816.3461611,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:3: [E201] Trailing whitespace'},
  {'time': 1589484816.3494494,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:17: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3529058,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:17: [E303] mktemp used in place of tempfile module'},
  {'time': 1589484816.3561723,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:24: [E201] Trailing whitespace'},
  {'time': 1589484816.3597164,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:26: [E303] unzip used in place of unarchive module'},
  {'time': 1589484816.3629677,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:27: [E201] Trailing whitespace'},
  {'time': 1589484816.3661678,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:32: [E303] tar used in place of unarchive module'},
  {'time': 1589484816.3696184,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:33: [E201] Trailing whitespace'},
  {'time': 1589484816.3730192,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:38: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3762379,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:38: [E306] Shells that use pipes should set the pipefail option'},
  {'time': 1589484816.3796587,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:39: [E206] Variables should have spaces before and after: {{ var_name }}'},
  {'time': 1589484816.3831143,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/install_minishift.yml:47: [E201] Trailing whitespace'},
  {'time': 1589484816.3861964,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/isrunning.yml:1: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.3896391,
   'level': 'WARNING',
   'message': "minishift_up_role/tasks/isrunning.yml:12: [E601] Don't compare to literal True/False"},
  {'time': 1589484816.3931148,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/isrunning.yml:30: [E201] Trailing whitespace'},
  {'time': 1589484816.3966193,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/main.yml:23: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.4001596,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/main.yml:30: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.403435,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/main.yml:30: [E306] Shells that use pipes should set the pipefail option'},
  {'time': 1589484816.4071002,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/main.yml:48: [E301] Commands should not change things if nothing needs doing'},
  {'time': 1589484816.4105835,
   'level': 'WARNING',
   'message': 'minishift_up_role/tasks/main.yml:48: [E306] Shells that use pipes should set the pipefail option'},
  {'time': 1589484816.4533432,
   'level': 'INFO',
   'message': 'Loading module k8s'},
  {'time': 1589484816.4573016,
   'level': 'INFO',
   'message': 'Loading module k8s_scale'},
  {'time': 1589484816.4609346,
   'level': 'INFO',
   'message': 'Loading module k8s_service'},
  {'time': 1589484816.464483,
   'level': 'INFO',
   'message': 'Loading module k8s_facts'},
  {'time': 1589484816.4682703,
   'level': 'INFO',
   'message': 'Loading module k8s_auth'},
  {'time': 1589484816.471888,
   'level': 'INFO',
   'message': 'Loading module_utils raw'},
  {'time': 1589484816.4758008,
   'level': 'INFO',
   'message': 'Loading module_utils scale'},
  {'time': 1589484816.479419,
   'level': 'INFO',
   'message': 'Loading module_utils common'},
  {'time': 1589484816.4831336,
   'level': 'INFO',
   'message': 'Loading filter k8s'},
  {'time': 1589484816.4888449,
   'level': 'INFO',
   'message': 'Loading inventory k8s'},
  {'time': 1589484816.4972575,
   'level': 'INFO',
   'message': 'Collection validation and loading complete'}]}

In [3]: Task.objects.all()                                                                                                                                                                    
Out[3]: <QuerySet [<Task: pulpcore.app.tasks.orphan.orphan_cleanup>, <Task: pulp_ansible.app.tasks.collections.import_collection>]>

In [4]: Task.objects.all()[1].__dict__                                                                                                                                                        
Out[4]: 
{'_state': <django.db.models.base.ModelState at 0x7fef92a3e590>,
 'pulp_id': UUID('f2a48504-46f3-41ac-9706-44f282da822e'),
 'pulp_created': datetime.datetime(2020, 5, 14, 19, 33, 32, 491513, tzinfo=<UTC>),
 'pulp_last_updated': datetime.datetime(2020, 5, 14, 19, 33, 36, 521167, tzinfo=<UTC>),
 'state': 'failed',
 'name': 'pulp_ansible.app.tasks.collections.import_collection',
 'started_at': datetime.datetime(2020, 5, 14, 19, 33, 32, 594439, tzinfo=<UTC>),
 'finished_at': datetime.datetime(2020, 5, 14, 19, 33, 36, 520760, tzinfo=<UTC>),
 'error': {'traceback': '  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/rq/worker.py", line 886, in perform_job\n    rv = job.perform()\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/rq/job.py", line 664, in perform\n    self._result = self._execute()\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/rq/job.py", line 670, in _execute\n    return self.func(*self.args, **self.kwargs)\n  File "/home/vagrant/devel/pulp_ansible/pulp_ansible/app/tasks/collections.py", line 163, in import_collection\n    collection_version.save()\n  File "/home/vagrant/devel/pulp_ansible/pulp_ansible/app/models.py", line 185, in save\n    super().save(*args, **kwargs)\n  File "/home/vagrant/devel/pulpcore/pulpcore/app/models/base.py", line 110, in save\n    return super().save(*args, **kwargs)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/base.py", line 741, in save\n    force_update=force_update, update_fields=update_fields)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/base.py", line 779, in save_base\n    force_update, using, update_fields,\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/base.py", line 870, in _save_table\n    result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/base.py", line 908, in _do_insert\n    using=using, raw=raw)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/manager.py", line 82, in manager_method\n    return getattr(self.get_queryset(), name)(*args, **kwargs)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/query.py", line 1186, in _insert\n    return query.get_compiler(using=using).execute_sql(return_id)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/models/sql/compiler.py", line 1375, in execute_sql\n    cursor.execute(sql, params)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/backends/utils.py", line 67, in execute\n    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/backends/utils.py", line 76, in _execute_with_wrappers\n    return executor(sql, params, many, context)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute\n    return self.cursor.execute(sql, params)\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/utils.py", line 89, in __exit__\n    raise dj_exc_value.with_traceback(traceback) from exc_value\n  File "/usr/local/lib/pulp/lib64/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute\n    return self.cursor.execute(sql, params)\n',
  'description': 'value too long for type character varying(2000)\n'},
 'worker_id': UUID('6eeeaca5-1091-411f-a860-34550e3ecc79'),
 'parent_task_id': None,
 'task_group_id': None}

Added by Fabricio Aguiar over 4 years ago

Revision ff4d9572 | View on GitHub

Readable error messages on user facing logger

https://pulp.plan.io/issues/6657 closes #6657

Actions
Copy link
 #4

Updated by pulpbot over 4 years ago

  • Status changed from ASSIGNED to POST
Actions
Copy link
 #5

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 72 to Sprint 73
Actions
Copy link
 #6

Updated by alikins over 4 years ago

The original posted traceback seems to imply that there is some missing data validation happening in the serializers?

Actions
Copy link
 #7

Updated by Anonymous over 4 years ago

  • Status changed from POST to MODIFIED
Actions
Copy link
 #8

Updated by bmbouter over 4 years ago

alikins wrote:

The original posted traceback seems to imply that there is some missing data validation happening in the serializers?

@alikins I agree. I filed this issue to resolve that aspect. https://pulp.plan.io/issues/6772

Actions
Copy link
 #9

Updated by fao89 over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Copy link
 #10

Updated by fao89 over 4 years ago

  • Sprint/Milestone set to 0.2.0
Actions
Send by e-mail Copy link

Also available in: Atom PDF