[2.3] 500 error when uploading a new commissioning script
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Critical
|
Lee Trager |
Bug Description
This bug is not related to #1712423 afaik as the steps to reproduce are dissimilar.
Steps to reproduce:
1. Go to Setting tab and click on 'Upload script' in the 'Commissioning scripts' section.
2. Click on 'Choose File' to upload a script file.
3. Pick the script file you would like to upload.
4. Click on the 'Upload' button.
500 stacktrace is produced:
2017-08-22 17:23:40 maasserver.
Traceback (most recent call last):
File "/usr/lib/
response = wrapped_
File "/usr/lib/
return view_atomic(*args, **kwargs)
File "/usr/lib/
return func(*args, **kwds)
File "/usr/lib/
return view_func(request, *args, **kwargs)
File "/usr/lib/
return self.dispatch(
File "/usr/lib/
return handler(request, *args, **kwargs)
File "/usr/lib/
return super(BaseCreat
File "/usr/lib/
if form.is_valid():
File "/usr/lib/
if not self._form.
File "/usr/lib/
self.
File "/usr/lib/
(self.
AssertionError: VersionedTextFile object can't be deleted because its id attribute is set to None.
2017-08-22 17:23:40 regiond: [info] ::1 POST /MAAS/commissio
Related branches
- Blake Rouse (community): Approve
-
Diff: 673 lines (+117/-82)6 files modifiedsrc/maasserver/api/tests/test_commissioning.py (+3/-3)
src/maasserver/api/tests/test_scripts.py (+4/-4)
src/maasserver/forms/script.py (+18/-4)
src/maasserver/forms/tests/test_script.py (+70/-69)
src/maasserver/testing/factory.py (+21/-1)
src/maasserver/views/tests/test_settings_commissioning_scripts.py (+1/-1)
Changed in maas: | |
milestone: | 2.3.0alpha2 → 2.3.0 |
Changed in maas: | |
status: | New → Triaged |
tags: | added: commissioning scripts |
Changed in maas: | |
status: | Triaged → In Progress |
assignee: | nobody → Lee Trager (ltrager) |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
milestone: | 2.3.0 → 2.3.0alpha3 |
Changed in maas: | |
status: | Fix Committed → Fix Released |