Project

General

Profile

Actions

Fix #12348

open

clean up modify_fstab

Added by Dan Mick almost 9 years ago.

Status:
New
Priority:
Normal
Assignee:
Category:
ansible
% Done:

0%

Source:
Development
Tags:
Backport:
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

I don't see any reason why modify_fstab can't run on any machine; true, it's not necessary to preserve fs mount options on VMs that will be destroyed before they are rebooted, but ... someone might reboot the VM without destroying, and editing fstab should not be harmful, so the special case is unnecessary.

Figure out why this was deemed necessary and justify it better or remove it.

Other outstanding weirdnesses around this code:

1) it's crazy Perl; if the only reason we need the Perl is because we need
the mountpoint and the UUID, as the comment in roles/testnode/tasks/apt_systems.yml claims, those should be available through facts.

2) why is this for apt systems only? If it's necessary, surely it's necessary regardless of the package type of the system?

No data to display

Actions

Also available in: Atom PDF