metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From justinleet <>
Subject [GitHub] incubator-metron pull request #518: METRON-799: The MPack should function in...
Date Fri, 07 Apr 2017 13:09:52 GMT
Github user justinleet commented on a diff in the pull request:
    --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/
    @@ -0,0 +1,74 @@
    +Licensed to the Apache Software Foundation (ASF) under one
    +or more contributor license agreements.  See the NOTICE file
    +distributed with this work for additional information
    +regarding copyright ownership.  The ASF licenses this file
    +to you under the Apache License, Version 2.0 (the
    +See the License for the specific language governing permissions and
    +limitations under the License.
    +import os.path
    +from resource_management.core.source import Template
    +from resource_management.core.resources.system import Directory, File
    +from resource_management.core import global_lock
    +from resource_management.core.logger import Logger
    +from resource_management.core.resources.system import Execute
    +from resource_management.libraries.functions import format as ambari_format
    +# Convenience function for ensuring home dirs are setup consistently.
    +def storm_security_setup(params):
    +    if params.security_enabled:
    +        # I don't think there's an Ambari way to get a user's local home dir , so have
Python perform tilde expansion.
    +        # Ambari's Directory doesn't do tilde expansion.
    +        metron_storm_dir_tilde = '~' + params.metron_user + '/.storm'
    +        metron_storm_dir = os.path.expanduser(metron_storm_dir_tilde)
    +        Directory(metron_storm_dir,
    +                  mode=0755,
    +                  owner=params.metron_user,
    +                  group=params.metron_group
    +                  )
    +        File(ambari_format('{client_jaas_path}'),
    +             content=Template('client_jaas.conf.j2'),
    +             owner=params.metron_user,
    +             group=params.metron_group,
    +             mode=0755
    +             )
    +        File(metron_storm_dir + '/storm.yaml',
    +             content=Template('storm.yaml.j2'),
    +             owner=params.metron_user,
    +             group=params.metron_group,
    +             mode=0755
    +             )
    --- End diff --
    Just the properties in here.  Couple thoughts as you bring this up.
    We probably want to have a ticket to make sure turning off Kerberos works correctly in
the future.  The properties in the file (except nimbus.seeds) are essentially set properties.
 We need our own client_jaas, and the storm.thrift.transport has to be there for some reason
and that's pretty much constant on a secure cluster.  That should also be made a property
that flows down from Storm.
    nimbus.seeds is wrong and I need to carry that over from the actual Storm property. And
I even made sure it was in params_linux and forgot to use it.

contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message