builtins.PermissionError: [Errno 1] Operation not permitted: '/var/lib/maas/.maas-proxy.conf.wwu2aw6z.tmp'
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Medium
|
Felipe Reyes | ||
2.7 |
Fix Committed
|
Medium
|
Felipe Reyes | ||
2.8 |
Fix Released
|
Medium
|
Felipe Reyes |
Bug Description
It was found an environment where there were about ~50 GB of files /var/lib/
2020-06-16 15:34:46 maasserver.
Traceback (most recent call last):
File "/usr/lib/
File "/usr/lib/
return target()
File "/usr/lib/
task()
File "/usr/lib/
task()
--- <exception caught here> ---
File "/usr/lib/
result = inContext.theWork()
File "/usr/lib/
File "/usr/lib/
return self.currentCon
File "/usr/lib/
return func(*args,**kw)
File "/usr/lib/
return func(*args, **kwargs)
File "/usr/lib/
result = func(*args, **kwargs)
File "/usr/lib/
return func_outside_
File "/usr/lib/
return func(*args, **kwargs)
File "/usr/lib/
return func(*args, **kwds)
File "/usr/lib/
File "/usr/lib/
result = func(*args, **kwargs)
File "/usr/lib/
File "/usr/lib/
[Test case]
1. Install MAAS from deb packages (tested with 2.7 and 2.8).
2. sudo chown postgres:postgres /var/lib/
3. echo "# This shouldn't be here" | sudo tee -a /var/lib/
4. sudo systemctl restart maas-regiond
5. ls /var/lib/
Expected result:
The failure of chown() gets logged and the temporary file removed.
Actual result:
chown()'s PermissionError is not handled and leaving the temporary file behind.
Related branches
- Alberto Donato (community): Approve
- MAAS Lander: Approve
-
Diff: 55 lines (+25/-1)2 files modifiedsrc/provisioningserver/utils/fs.py (+10/-1)
src/provisioningserver/utils/tests/test_fs.py (+15/-0)
- Alberto Donato (community): Approve
- MAAS Lander: Approve
-
Diff: 55 lines (+25/-1)2 files modifiedsrc/provisioningserver/utils/fs.py (+10/-1)
src/provisioningserver/utils/tests/test_fs.py (+15/-0)
- Adam Collard (community): Approve
- Dougal Matthews (community): Approve
- MAAS Lander: Approve
-
Diff: 55 lines (+25/-1)2 files modifiedsrc/provisioningserver/utils/fs.py (+10/-1)
src/provisioningserver/utils/tests/test_fs.py (+15/-0)
Changed in maas: | |
assignee: | nobody → Felipe Reyes (freyes) |
Changed in maas: | |
status: | New → In Progress |
Changed in maas: | |
importance: | Undecided → Medium |
milestone: | none → 2.9.0b1 |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
status: | Fix Committed → Fix Released |