Andrew Onischuk created AMBARI-15468:
----------------------------------------

             Summary: file_system get_mount_point_for_dir works incorrect
                 Key: AMBARI-15468
                 URL: https://issues.apache.org/jira/browse/AMBARI-15468
             Project: Ambari
          Issue Type: Bug
            Reporter: Andrew Onischuk
            Assignee: Andrew Onischuk
             Fix For: 2.2.2
         Attachments: AMBARI-15468.patch

**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





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to