{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/18630","repository_url":"https://api.github.com/repos/saltstack/salt","labels_url":"https://api.github.com/repos/saltstack/salt/issues/18630/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/18630/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/18630/events","html_url":"https://github.com/saltstack/salt/issues/18630","id":50596677,"number":18630,"title":"Forced remount because options changed when no options changed (2014.7 regression)","user":{"login":"nvx","id":551727,"avatar_url":"https://avatars.githubusercontent.com/u/551727?v=3","gravatar_id":"","url":"https://api.github.com/users/nvx","html_url":"https://github.com/nvx","followers_url":"https://api.github.com/users/nvx/followers","following_url":"https://api.github.com/users/nvx/following{/other_user}","gists_url":"https://api.github.com/users/nvx/gists{/gist_id}","starred_url":"https://api.github.com/users/nvx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nvx/subscriptions","organizations_url":"https://api.github.com/users/nvx/orgs","repos_url":"https://api.github.com/users/nvx/repos","events_url":"https://api.github.com/users/nvx/events{/privacy}","received_events_url":"https://api.github.com/users/nvx/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/saltstack/salt/labels/Bug","name":"Bug","color":"e11d21"},{"url":"https://api.github.com/repos/saltstack/salt/labels/High%20Severity","name":"High Severity","color":"ff9143"},{"url":"https://api.github.com/repos/saltstack/salt/labels/P2","name":"P2","color":"155fb5"},{"url":"https://api.github.com/repos/saltstack/salt/labels/Platform","name":"Platform","color":"fef2c0"},{"url":"https://api.github.com/repos/saltstack/salt/labels/Regression","name":"Regression","color":"e11d21"},{"url":"https://api.github.com/repos/saltstack/salt/labels/State%20Module","name":"State Module","color":"006b75"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/saltstack/salt/milestones/13","html_url":"https://github.com/saltstack/salt/milestones/Approved","labels_url":"https://api.github.com/repos/saltstack/salt/milestones/13/labels","id":9265,"number":13,"title":"Approved","description":"All issues that are ready to be worked on, both bugs and features.","creator":{"login":"thatch45","id":507599,"avatar_url":"https://avatars.githubusercontent.com/u/507599?v=3","gravatar_id":"","url":"https://api.github.com/users/thatch45","html_url":"https://github.com/thatch45","followers_url":"https://api.github.com/users/thatch45/followers","following_url":"https://api.github.com/users/thatch45/following{/other_user}","gists_url":"https://api.github.com/users/thatch45/gists{/gist_id}","starred_url":"https://api.github.com/users/thatch45/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thatch45/subscriptions","organizations_url":"https://api.github.com/users/thatch45/orgs","repos_url":"https://api.github.com/users/thatch45/repos","events_url":"https://api.github.com/users/thatch45/events{/privacy}","received_events_url":"https://api.github.com/users/thatch45/received_events","type":"User","site_admin":false},"open_issues":2858,"closed_issues":3300,"state":"open","created_at":"2011-05-14T04:00:56Z","updated_at":"2016-02-06T00:29:05Z","due_on":null,"closed_at":null},"comments":55,"created_at":"2014-12-02T00:13:59Z","updated_at":"2016-02-07T16:58:14Z","closed_at":null,"body":"Running a highstate on minions with some NFS mounts results in the mount being remounted every time. This did not occur under 2014.1\r\n\r\n```\r\n----------\r\n[INFO    ] Completed state [/nfsmnt] at time 17:21:20.145018\r\n[INFO    ] Running state [/nfsmnt] at time 17:21:20.146726\r\n[INFO    ] Executing state mount.mounted for /nfsmnt\r\n[INFO    ] Executing command 'mount -l' in directory '/root'\r\n[INFO    ] Executing command 'mount -l' in directory '/root'\r\n[INFO    ] Executing command 'mount -o rw,tcp,bg,hard,intr,remount -t nfshost:/nfsmnt /nfsmnt ' in directory '/root'\r\n[INFO    ] {'umount': 'Forced remount because options changed'}\r\n[INFO    ] Completed state [/nfsmnt] at time 17:21:20.267764\r\n...\r\n          ID: /nfsmnt\r\n    Function: mount.mounted\r\n      Result: True\r\n     Comment:\r\n     Started: 10:04:16.078806\r\n    Duration: 68.802 ms\r\n     Changes:\r\n              ----------\r\n              umount:\r\n                  Forced remount because options changed\r\n```\r\n\r\nRunning mount -l shows the following:\r\n\r\n```\r\n...\r\nnfshost:/nfsmnt on /nfsmnt type nfs (rw,remount,tcp,bg,hard,intr,addr=x.x.x.x)\r\n```\r\n\r\nI can only assume it's breaking due to the addr option (which is automatically filled by the OS by the looks of it, it was never manually specified as a mount option) or the ordering.\r\n\r\nThe mount.mounted state looks as follows:\r\n\r\n```\r\n/nfsmnt:\r\n  mount.mounted:\r\n    - device: nfshost:/nfsmnt\r\n    - fstype: nfs\r\n    - opts: rw,tcp,bg,hard,intr\r\n```\r\n\r\n```\r\n# salt-call --versions-report\r\n           Salt: 2014.7.0\r\n         Python: 2.6.8 (unknown, Nov  7 2012, 14:47:45)\r\n         Jinja2: 2.5.5\r\n       M2Crypto: 0.21.1\r\n msgpack-python: 0.1.12\r\n   msgpack-pure: Not Installed\r\n       pycrypto: 2.3\r\n        libnacl: Not Installed\r\n         PyYAML: 3.08\r\n          ioflo: Not Installed\r\n          PyZMQ: 14.3.1\r\n           RAET: Not Installed\r\n            ZMQ: 4.0.4\r\n           Mako: Not Installed\r\n```"},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/181049806","html_url":"https://github.com/saltstack/salt/issues/18630#issuecomment-181049806","issue_url":"https://api.github.com/repos/saltstack/salt/issues/18630","id":181049806,"user":{"login":"garethgreenaway","id":718525,"avatar_url":"https://avatars.githubusercontent.com/u/718525?v=3","gravatar_id":"","url":"https://api.github.com/users/garethgreenaway","html_url":"https://github.com/garethgreenaway","followers_url":"https://api.github.com/users/garethgreenaway/followers","following_url":"https://api.github.com/users/garethgreenaway/following{/other_user}","gists_url":"https://api.github.com/users/garethgreenaway/gists{/gist_id}","starred_url":"https://api.github.com/users/garethgreenaway/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garethgreenaway/subscriptions","organizations_url":"https://api.github.com/users/garethgreenaway/orgs","repos_url":"https://api.github.com/users/garethgreenaway/repos","events_url":"https://api.github.com/users/garethgreenaway/events{/privacy}","received_events_url":"https://api.github.com/users/garethgreenaway/received_events","type":"User","site_admin":false},"created_at":"2016-02-07T16:58:14Z","updated_at":"2016-02-07T16:58:14Z","body":"@pkruithof Perfect.  Can you also provide the output after running the mount command?  I'm curious to see what options are making it into the actual mounted file system vs. what they look like in the salt state.  As mentioned, I saw a discrepancy when testing an NFSv4 filesystem which caused Salt to try to remount it during each run. "}}