[AMBARI-15468] file_system get_mount_point_for_dir works incorrect

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.2.2
• Assignee: Andrew Onischuk
• Reporter: Andrew Onischuk
• Created: 17/Mar/16 16:02
• Updated: 18/Mar/16 14:58
• Resolved: 18/Mar/16 12:59

Description

*Case 1*:

[root@c6403 ~]# mount
/dev/mapper/VolGroup-lv_root on / type ext4 (rw)
/newFs on /newHdfsDir type ext3 (rw,loop=/dev/loop1)

[root@c6403 ~]# python
>>> from resource_management import *
>>> from resource_management.libraries.functions import file_system
>>> with Environment() as env:
... print file_system.get_mount_point_for_dir('/newHdfsDir/some_dir')
... print file_system.get_mount_point_for_dir('/newHdfsDir')
...
2016-03-17 13:00:33,640 - Mount point for directory /newHdfsDir/some_dir is /
2016-03-17 13:00:33,640 - Mount point for directory /newHdfsDir is /

It should be /newHdfsDir, not /

*Reason*
/newHdfsDir and / have the same count of os separators.

*Case 2*:

[root@c6403 ~]# mount
/dev/mapper/VolGroup-lv_root on / type ext4 (rw)
/newFs on /a/a type ext3 (rw,loop=/dev/loop2)
/newFs on /a/a1 type ext3 (rw,loop=/dev/loop3)

[root@c6403 ~]# python
>>> from resource_management import *
>>> from resource_management.libraries.functions import file_system
>>> with Environment() as env:
>>> from resource_management import *
>>> from resource_management.libraries.functions import file_system
>>> with Environment() as env:
... print file_system.get_mount_point_for_dir('/a/a1/some_dir')
... print file_system.get_mount_point_for_dir('/a/a1')
...
2016-03-17 13:07:38,899 - Mount point for directory /a/a1/some_dir is /a/a
2016-03-17 13:07:38,900 - Mount point for directory /a/a1 is /a/a

It should be /a/a1, not /a/a

Issue Links

• links to: ReviewBoard