[ https://issues.apache.org/jira/browse/AMBARI-21744?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16130789#comment-16130789 ]
Hudson commented on AMBARI-21744: --------------------------------- SUCCESS: Integrated in Jenkins build Ambari-trunk-Commit #7898 (See [https://builds.apache.org/job/Ambari-trunk-Commit/7898/]) AMBARI-21744. package_regex in get_package_from_available() can match (dlysnichenko: [http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=f2cc609b0033d59130cb00d14ff8d2c8db6e1392]) * (edit) ambari-common/src/main/python/resource_management/libraries/script/script.py > package_regex in get_package_from_available() can match wrong pkg > ----------------------------------------------------------------- > > Key: AMBARI-21744 > URL: https://issues.apache.org/jira/browse/AMBARI-21744 > Project: Ambari > Issue Type: Bug > Components: ambari-server > Reporter: Dmitry Lysnichenko > Assignee: Dmitry Lysnichenko > Priority: Critical > Attachments: AMBARI-21744.patch > > > Due to the issue with regex (missing ^ and $ boundaries), > resource_management.libraries.script.script.Script#get_package_from_available > may return wrong package. > {code} > >>> list=['hbase_3_0_0_0_229-master', 'hbase_3_0_0_0_229'] > >>> if re.match('hbase_(\d|_)+', 'hbase_3_0_0_0_229-master'): > ... print 'YES' > ... > YES > >>> if re.match('hbase_(\d|_)+', 'hbase_3_0_0_0_229'): > ... print 'YES' > ... > YES > {code} > In this case, the first package name from a list of available packages will > be returned. > The impact of bug is that we may install a wrong package if it's simillary > named and goes first at list. Patch is a single-line fix. -- This message was sent by Atlassian JIRA (v6.4.14#64029)