Activity log for bug #1144612

Date Who What changed Old value New value Message
2013-03-04 15:39:36 Louis Bouchard bug added bug
2013-03-04 15:39:52 Louis Bouchard puppet (Ubuntu): status New Confirmed
2013-03-04 15:39:57 Louis Bouchard puppet (Ubuntu): assignee Louis Bouchard (louis-bouchard)
2013-03-04 16:18:35 Louis Bouchard bug added subscriber Canonical Support
2013-03-04 16:18:47 Louis Bouchard bug added subscriber Sustaining Engineering
2013-04-26 12:17:40 Louis Bouchard puppet (Ubuntu): status Confirmed In Progress
2013-04-26 12:37:45 Louis Bouchard affects puppet (Ubuntu) cifs-utils (Ubuntu)
2013-04-26 12:50:27 Dimitri John Ledkov nominated for series Ubuntu Precise
2013-04-26 12:50:27 Dimitri John Ledkov bug task added cifs-utils (Ubuntu Precise)
2013-04-26 12:50:27 Dimitri John Ledkov nominated for series Ubuntu Quantal
2013-04-26 12:50:27 Dimitri John Ledkov bug task added cifs-utils (Ubuntu Quantal)
2013-04-26 12:50:36 Dimitri John Ledkov cifs-utils (Ubuntu): status In Progress Fix Released
2013-04-26 12:50:38 Dimitri John Ledkov cifs-utils (Ubuntu Quantal): status New Fix Released
2013-04-26 12:50:41 Dimitri John Ledkov cifs-utils (Ubuntu Precise): status New Confirmed
2013-04-26 12:50:43 Dimitri John Ledkov cifs-utils (Ubuntu Precise): importance Undecided Medium
2013-04-26 12:50:54 Dimitri John Ledkov cifs-utils (Ubuntu Precise): assignee Louis Bouchard (louis-bouchard)
2013-04-26 13:39:21 Louis Bouchard description Create a file with the following contents, making sure that the samba location is available and is mountable by guests and that the mountpoint also exists and is a directory: # cat /etc/puppet/manifests/tst.pp package{'samba-client': } mount{'home directory mount': name => '/mnt/target', atboot => 'true', device => '//localhost/test', ensure => 'mounted', fstype => 'cifs', require => [Package['samba-client']] } 4.2- Apply the puppet config: # puppet apply /etc/puppet/manifests/tst.pp 4.3- Verify that the mounted FS shows twice: root@ubuntu:~# mount -t cifs //localhost/test on /mnt/target type cifs (rw) //localhost/test on /mnt/target type cifs (rw) SRU justification : During a remount of a cifs filesystem, the mtab file is not properly updated, which leads to a doubled entry of the same filesystem in the /etc/mtab file. Impact : df and other tools incorrectly report the CIFS file system as mounted twice Fix : This has been fixed upstream in version 5.3 which is present in Quantal and more recent releases. The following upstream commit provides the fix : https://git.samba.org/?p=cifs-utils.git;a=commitdiff_plain;h=f46dd7661cfb87257c95081fc2071c934bfbbb16 Test Case : - An accessible SAMBA share is required on host 'sambahost' - the user=,password= options are only required to avoid password prompt $ mkdir /mnt/target $ mount -o user=,password= //sambahost/test /mnt/target $ mount -o remount /mnt/target $ df | grep target //sambahost/test 7033648 1278124 5402656 20% /mnt/target //sambahost/test 7033648 1278124 5402656 20% /mnt/target With the proposed fix, there would be only one /mnt/target entry Regression : This fix is already present in Q & R. It only introduce a del_mtab function that is only called when MS_REMOUNT is set on the file system. Original description of the problem : Create a file with the following contents, making sure that the samba location is available and is mountable by guests and that the mountpoint also exists and is a directory: # cat /etc/puppet/manifests/tst.pp package{'samba-client': } mount{'home directory mount':  name => '/mnt/target',  atboot => 'true',  device => '//localhost/test',  ensure => 'mounted',  fstype => 'cifs',  require => [Package['samba-client']] } 4.2- Apply the puppet config: # puppet apply /etc/puppet/manifests/tst.pp 4.3- Verify that the mounted FS shows twice: root@ubuntu:~# mount -t cifs //localhost/test on /mnt/target type cifs (rw) //localhost/test on /mnt/target type cifs (rw)
2013-04-26 13:39:37 Louis Bouchard summary puppet double mounts CIFS mounts mount.cifs double mounts CIFS mounts
2013-04-26 13:40:24 Louis Bouchard summary mount.cifs double mounts CIFS mounts mount.cifs incorrectly update /etc/mtab when -o remount is used
2013-04-26 13:40:32 Louis Bouchard cifs-utils (Ubuntu Precise): status Confirmed In Progress
2013-04-26 13:41:05 Louis Bouchard attachment added cifs-utils-lp1144612.debdiff https://bugs.launchpad.net/ubuntu/+source/cifs-utils/+bug/1144612/+attachment/3654742/+files/cifs-utils-lp1144612.debdiff
2013-04-26 13:41:22 Louis Bouchard bug added subscriber Ubuntu Stable Release Updates Team
2013-05-07 13:59:17 Adam Stokes bug added subscriber Ubuntu Sponsors Team
2013-05-15 11:38:43 Sebastien Bacher removed subscriber Ubuntu Sponsors Team
2013-05-16 21:54:50 Brian Murray cifs-utils (Ubuntu Precise): status In Progress Fix Committed
2013-05-16 21:54:54 Brian Murray bug added subscriber SRU Verification
2013-05-16 21:54:56 Brian Murray tags verification-needed
2013-05-16 22:07:55 Launchpad Janitor branch linked lp:ubuntu/precise-proposed/cifs-utils
2013-05-17 09:27:04 Eduardo Damato tags verification-needed verification-done
2013-05-24 04:36:39 Adam Conrad removed subscriber Ubuntu Stable Release Updates Team
2013-05-24 04:37:14 Launchpad Janitor cifs-utils (Ubuntu Precise): status Fix Committed Fix Released
2014-02-07 16:04:44 Curtis Hovey removed subscriber Registry Administrators