Return-Path: X-Original-To: apmail-ambari-dev-archive@www.apache.org Delivered-To: apmail-ambari-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5576E10D49 for ; Wed, 5 Mar 2014 15:30:02 +0000 (UTC) Received: (qmail 90375 invoked by uid 500); 5 Mar 2014 15:30:01 -0000 Delivered-To: apmail-ambari-dev-archive@ambari.apache.org Received: (qmail 90236 invoked by uid 500); 5 Mar 2014 15:30:01 -0000 Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@ambari.apache.org Delivered-To: mailing list dev@ambari.apache.org Received: (qmail 90189 invoked by uid 99); 5 Mar 2014 15:30:00 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 05 Mar 2014 15:30:00 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 543A81D44CF; Wed, 5 Mar 2014 15:29:58 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============8408960925193454921==" MIME-Version: 1.0 Subject: Re: Review Request 18772: After configuring NNHA, nn process alerts don't work From: "Nate Cole" To: "Nate Cole" , "Dmitro Lisnichenko" Cc: "Vitalyi Brodetskyi" , "Ambari" Date: Wed, 05 Mar 2014 15:29:58 -0000 Message-ID: <20140305152958.6955.42755@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Nate Cole" X-ReviewGroup: Ambari X-ReviewRequest-URL: https://reviews.apache.org/r/18772/ X-Sender: "Nate Cole" References: <20140305152245.10089.71771@reviews.apache.org> In-Reply-To: <20140305152245.10089.71771@reviews.apache.org> Reply-To: "Nate Cole" X-ReviewRequest-Repository: ambari --===============8408960925193454921== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18772/#review36228 ----------------------------------------------------------- ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/templates/hadoop-services.cfg.j2 I realize it's sometimes convenient to put logic into template files, but it makes it harder to maintain. Can we keep the logic in the params.py script and just "resolve the port" so to speak there, and just use a single value in the template file? - Nate Cole On March 5, 2014, 10:22 a.m., Vitalyi Brodetskyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18772/ > ----------------------------------------------------------- > > (Updated March 5, 2014, 10:22 a.m.) > > > Review request for Ambari, Dmitro Lisnichenko and Nate Cole. > > > Bugs: AMBARI-4954 > https://issues.apache.org/jira/browse/AMBARI-4954 > > > Repository: ambari > > > Description > ------- > > 1) Configure NN HA > 2) stack 2.0.6 (but check stack 2.1 as well) > 3) Two alerts show" check_tcp: Port must be a positive integer" > > NameNode process on c6401.ambari.apache.org > NameNode process on c6402.ambari.apache.org > > 4) Looked at /etc/nagios/objects/hadoop-services.cfg > 5) Saw: > > {code} > define service { > host_name c6401.ambari.apache.org > use hadoop-service > service_description NAMENODE::NameNode process on c6401.ambari.apache.org > servicegroups HDFS > check_command check_tcp_wrapper!//test!-w 1 -c 1 > normal_check_interval 0.5 > retry_check_interval 0.25 > max_check_attempts 3 > } > define service { > host_name c6402.ambari.apache.org > use hadoop-service > service_description NAMENODE::NameNode process on c6402.ambari.apache.org > servicegroups HDFS > check_command check_tcp_wrapper!//test!-w 1 -c 1 > normal_check_interval 0.5 > retry_check_interval 0.25 > max_check_attempts 3 > } > {code} > > Notice in the above *//test* is the name of my nameservice. > > Attaching screen shot of my config. So looks like it's grabbing port from the wrong prop. > > > Diffs > ----- > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/scripts/params.py 813c626 > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/NAGIOS/package/templates/hadoop-services.cfg.j2 5b8fb6d > > Diff: https://reviews.apache.org/r/18772/diff/ > > > Testing > ------- > > > Thanks, > > Vitalyi Brodetskyi > > --===============8408960925193454921==--