You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ambari.apache.org by "Di Li (JIRA)" <ji...@apache.org> on 2016/11/22 19:39:58 UTC

[jira] [Updated] (AMBARI-18962) Make check_package_condition in script.py overridable by services in stacks

     [ https://issues.apache.org/jira/browse/AMBARI-18962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Di Li updated AMBARI-18962:
---------------------------
    Attachment: AMBARI-18962.patch

> Make check_package_condition in script.py overridable by services in stacks
> ---------------------------------------------------------------------------
>
>                 Key: AMBARI-18962
>                 URL: https://issues.apache.org/jira/browse/AMBARI-18962
>             Project: Ambari
>          Issue Type: Bug
>          Components: ambari-server
>    Affects Versions: trunk, 2.5.0
>            Reporter: Di Li
>            Assignee: Di Li
>         Attachments: AMBARI-18962.patch
>
>
> check_package_condition method in script.py was added via AMBARI-14739 to make the conditionally installing RPMs controlled jointly by metainfo.xml and functions defined in package_conditions.py.  For example, with the following metainfo.xml section. The package_conditions.py has the should_install_ams_collector function implementation.
>             <package>
>               <name>ambari-metrics-collector</name>
>               <skipUpgrade>true</skipUpgrade>
>               <condition>should_install_ams_collector</condition>
>             </package>
> The issue is that this is not easily extendable by services extensions (installed via mpack), as any third party service needs to add functions to package_conditions.py in resource_management library.
> This JIRA proposes to make check_package_condition overwritable at the service level ( by changing it from a static method to a instance method).
> Then each service can still maintain the metainfo.xml section, while implement its own check_package_condition that pulls in its own package_condition util.



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