stop chowning everything

Bug #1896163 reported by Paul Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Mirror Charm
Fix Released
Undecided
Paul Collins

Bug Description

```python
def configure_directories(conf, hostname):
    if not os.path.isdir("/srv/ftp.root"):
        mkdir("/srv/ftp.root")
    chownr("/srv/ftp.root", conf.mirror_user(), conf.mirror_user(), follow_links=False)
```

This seems unnecessary and, as we host more and more content on these machines, upgrades become slower and slower.

The charm already creates the root of the mirror path with the correct ownership, and the sync scripts run as that user.

Finally, chownr does not even change the ownership of the path from which it begins its recursive chown.

Paul Collins (pjdc)
description: updated
Paul Collins (pjdc)
Changed in ubuntu-mirror-charm:
status: New → Fix Released
assignee: nobody → Paul Collins (pjdc)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.