Re: [VOTE] Apache Ambari 2.5.0 RC2

2017-03-28 Thread Alejandro Fernandez
+1 for Apache Ambari 2.5 RC2 Contents look good. The only difference versus my build using the tag is that some of the images are gzipped. Thank you, Alejandro Fernandez Apache Ambari PMC On 3/28/17, 9:57 AM, "Yusaku Sako" <yus...@hortonworks.com> wrote: >+1 for RC2.

Re: [VOTE] Apache Ambari 2.5.0 RC1

2017-03-27 Thread Alejandro Fernandez
+1 for 2.5.0 RC1 Thanks, Alejandro Fernandez Apache Ambari PMC On 3/23/17, 2:11 PM, "Aravindan Vijayan" <avija...@hortonworks.com> wrote: >Hello all, > >I have created an apache-ambari-2.5.0 release candidate. > >GIT source tag (release-2.5.0-rc1) >https://git-

Re: [VOTE] Apache Ambari 2.5.0 RC0

2017-03-22 Thread Alejandro Fernandez
.gz > apache-ambari-2.5.0-src-mine.tar.gz.sha1 # and compare Folder comparison differed as a lot of the images have a .png.gz extension in uploaded artifacts. Thanks, Alejandro Fernandez On 3/22/17, 12:22 PM, "Aravindan Vijayan" <avija...@hortonworks.com> wrote: >Hel

Hadoop 3.0 - changes to services

2017-02-27 Thread Alejandro Fernandez
to metainfo.xml kerberos.json widgets.json config xml files python files etc will also have to be included in the newer version of the service for Hadoop 3.0. Please keep an eye out for this, especially during code reviews. Thank you, Alejandro Fernandez

Re: Why are AMS and LogSearch not part of RU/EU

2017-02-17 Thread Alejandro Fernandez
Only services that support side-by-side bits can be orchestrated. AMS and LogSearch don't have hdp-select to change the symlink of the version that is current and can only update the existing bits. Thanks, Alejandro On 2/17/17, 8:25 AM, "Jonathan Hurley" wrote:

Re: New service user for custom service

2017-02-15 Thread Alejandro Fernandez
HI Sandeep, you can generate a config-type to store the user but not actually save the config file to the local file system. Thanks, Alejandro On 2/15/17, 6:57 AM, "Sandeep G" wrote: >Thanks for the reply Janne > >I have tried out even that route of adding one xml file

Re: Question about adding our script at pre/post start and pre/post stop actions

2017-01-19 Thread Alejandro Fernandez
I see, you'll need to take a look at the pre-* hooks in the stack, e.g., pre-install, pre-start. E.g., ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START You may potentially need to modify script.py so that calling a start method is allowed to call a pre_start method. You also

Re: Question about adding our script at pre/post start and pre/post stop actions

2017-01-18 Thread Alejandro Fernandez
Adachisan, The usual operations of "restart" also appear in the upgrade packs. E.g., https://github.com/apache/ambari/blob/trunk/ambari-server/src/main/resource s/stacks/HDP/2.4/upgrades/upgrade-2.5.xml Thanks, Alejandro

Re: Question about adding our script at pre/post start and pre/post stop actions

2017-01-17 Thread Alejandro Fernandez
Hi Adachisan, What stack are you using? If you're using HDP, you can modify the upgrade packs (xml files) with additional steps like calling functions in the python files to perform any additional logic. E.g., ... ... Wait for HiveServer2 Connections

New HDP 3.0 stack and new versions of ZK, HDFS, YARN/MR in common-services

2016-12-07 Thread Alejandro Fernandez
nstead overriden the config values in the HDP 3.0 stack. If you have any questions regarding this work, or whether or not you need to include logic in the new versions in common-services, please don't hesitate to ask me. Thanks, Alejandro Fernandez

Re: Question about python scripts

2016-11-30 Thread Alejandro Fernandez
need to do it programmatically. Maybe this could be made as a >property, which is optionally configurable? > > >/usr/bin/hdp-select >/etc/hadoop/conf/topology_script.py > > > >Thanks, > >Ian > > > > > > > >On 11/18/16, 6:44 PM, "Alejandr

Re: Please assign AMBARI-19019 to me

2016-11-29 Thread Alejandro Fernandez
Done Thanks, Alejandro On 11/29/16, 8:48 AM, "Jeffrey Rodriguez" wrote: >https://issues.apache.org/jira/browse/AMBARI-19019 > >Please assign AMBARI-19019 to me or allow me to self assign. ( >jeffr...@us.ibm.com) > >Thanks, > Jeff Rodriguez >

Re: Upgrade mpack stack addon service

2016-11-18 Thread Alejandro Fernandez
Hi Janne, Management Packs are indeed meant to support new stacks, or custom services on top of existing stacks. We are still working on a story of how to do upgrades of a Management Pack. Today, a Management Pack supports its own repo, so the eventual goal is to be able to perform a Patch

Re: Question about python scripts

2016-11-18 Thread Alejandro Fernandez
Hi Ian, Ambari currently supports Python 2.6 and 2.7. Take a look at https://github.com/apache/ambari/blob/trunk/ambari-shell/ambari-python-shel l/conf/unix/ambari-shell to see if you can force it to always use Python 2.# Thanks, Alejandro On 11/17/16, 2:22 PM, "ian.malo...@tdameritrade.com"

Categorizing Unit Tests

2016-10-03 Thread Alejandro Fernandez
stCase { Thank you in advance, Alejandro Fernandez

Re: [VOTE] Apache Ambari 2.4.0 RC0

2016-08-25 Thread Alejandro Fernandez
ratcheck passes cp -R apache-ambari-2.4.0-src-mine apache-ambari-2.4.0-ratcheck cd apache-ambari-2.4.0-ratcheck mvn clean apache-rat:check Thanks, Alejandro Fernandez Apache Ambari PMC On 8/25/16, 2:50 PM, "Venkat" <venkat...@gmail.com> wrote: >+1 > >On Wed, Aug 24, 2016 a

Reminder on Code Review Best Practices

2016-08-04 Thread Alejandro Fernandez
the link above to include developers interested in that area. Thank you, Alejandro Fernandez

Re: Managing stack_version on Ambari 2.4 code

2016-07-26 Thread Alejandro Fernandez
ed in >service metainfo.xml) require the stack version (it used to be wildcards >* on Ambari-2.2.2) > > > zookeeper_${stack_version} > > >How are they connected? > >Once again thanks for your insights Alejandro > >

Re: Managing stack_version on Ambari 2.4 code

2016-07-26 Thread Alejandro Fernandez
Hi Juanjo, in order for the cluster_version to transition to CURRENT as part of the installation, all of the components in the stack that advertise a version (through some means like hdp-select) must actually start up on that version. Can you provide more details? SELECT repo_version_id, version

Re: Add host using REST API

2016-06-21 Thread Alejandro Fernandez
Hi Amruta, If the agent is still heartbeating, it will reinsert itself into the hosts and hoststate tables. Deleting a host does remove it from those, plus a handful of other tables like clusterhostmapping. Thanks, Alejandro On 6/20/16, 9:58 AM, "Amruta Borkar" wrote:

Re: AMBARI-12885 - Adding custom services to Ambari

2016-06-15 Thread Alejandro Fernandez
This is great Tim, custom services are at the core of Ambari and there's a lot of work to make then extensible and pluggable. We eventually need a story to upgrade custom services along with the rest of the stack, which means supporting multiple repos and allowing the upgrade pack orchestration to

Re: Code Review groups

2016-06-02 Thread Alejandro Fernandez
orks.com> wrote: > >>+1. Makes sense. >> >>Thanks. >> >>Regards, >>Swapan. >> >> >> >> >> >> >> >> >> >>On 6/2/16, 1:27 PM, "Robert Levas" <rle...@hortonworks.com> wrote: &

Code Review groups

2016-06-02 Thread Alejandro Fernandez
-in to areas of interest (even outside of their current expertise), should produce better code reviews, and make it easier for new contributors to find the right people. Thank you, Alejandro Fernandez

Re: Confluence edit access

2016-05-31 Thread Alejandro Fernandez
I added Tim Thorpe and Di Li. Thank you, Alejandro On 5/30/16, 8:41 AM, "Tim Thorpe" < wrote: > >I'd like to get edit privileges on the Ambari Wiki pages in order to >contribute to the new features that have been added for service >definitions. > >My confluence ID is ttho...@ca.ibm.com. >

Re: how is the data directory prefix determined?

2016-05-31 Thread Alejandro Fernandez
Hi Martin, That code exists in Stack Advisor. If data dir is not specified, it should default to /hadoop/hdfs/data Make sure to check your mount points. Which stack and version are you using? Thanks, Alejandro On 5/31/16, 7:58 AM, "Martin Serrano" wrote: >Hello, > >I'm

Re: HDP upgrade failed on Finalize Upgrade Pre-Check using ambari.

2016-05-20 Thread Alejandro Fernandez
Hi Anandha, Which version of Ambari are you on? If it's 2.2.0 or higher, you may find a button to "Pause" the upgrade and restart ZK on your own. On all of the ZK hosts, make sure to run this, hdp-select set zookeeper-server If ZK is actually started but Ambari doesn't recognize that it's on

[VOTE] Apache Ambari 2.2.2 RC1

2016-04-28 Thread Alejandro Fernandez
z.sha1 # and compare to the file on the server # I made sure ratcheck passes cp -R apache-ambari-2.2.2-src apache-ambari-2.2.2-ratcheck cd apache-ambari-2.2.2-ratcheck mvn clean apache-rat:check Thank you, Alejandro Fernandez Apache Ambari PMC On 4/25/16, 6:00 PM, "Nate Cole" <nc...@hor

Re: Missed commit to 2.2.2 branch

2016-04-20 Thread Alejandro Fernandez
When was branch-2.2.2 created? I thought that release 2.2.2 was coming from commits to branch-2.2 I want to make sure no commits were missed. Thanks, Alejandro On 4/20/16, 7:30 AM, "Dmitro Lysnichenko" wrote: >Hi all, > >Some time ago I've committed patch for

Re: Ambari Plugin

2016-03-24 Thread Alejandro Fernandez
Hi Vladimir and Michael, Srimanth wrote a great wiki on how to add a service to Ambari, https://cwiki.apache.org/confluence/display/AMBARI/How-To+Define+Stacks+and +Services How is Apache Ignite packaged? You may be able to install it via RPMs or wget. What are the requirements of your service?

Design & Code Review Guidelines

2016-03-24 Thread Alejandro Fernandez
already have the necessary +1. This encourages others to take an interest in your review and helps us find more bugs (it's ok to slow down in order to speed up). If you have any suggestions, feel free to share. Thank you, Alejandro Fernandez

Re: TypeError: getpwnam() argument 1 must be string, not None

2016-03-21 Thread Alejandro Fernandez
Hi Juntao, thank you for finding this. Can you add the Ambari version to the Jira.? Any steps to reproduce? Thanks, Alejandro From: zhang juntao > Reply-To: "u...@ambari.apache.org"

Re: Blueprints - RCO - Related question.

2016-03-14 Thread Alejandro Fernandez
lt;smoha...@hortonworks.com<mailto:smoha...@hortonworks.com>>, Alejandro Fernandez <afernan...@hortonworks.com<mailto:afernan...@hortonworks.com>> Subject: Re: Blueprints - RCO - Related question. Thank you very much Robert for the detailed explanation. It helps to understa

Re: Blueprints - RCO - Related question.

2016-03-11 Thread Alejandro Fernandez
+others who have more insight into BluePrints On 3/11/16, 3:24 PM, "Bhuvnesh Chaudhary" wrote: >Hello Sebastian, Alejandro, Andrew, > >Referring to the discussion on RB: https://reviews.apache.org/r/43948 >, it appears

Apache Mentors for Google Summer of Code

2016-03-08 Thread Alejandro Fernandez
, Alejandro Fernandez

[jira] [Updated] (AMBARI-15330) Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-15330?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez updated AMBARI-15330: - Attachment: AMBARI-15330.trunk.patch > Bubble up errors during RU

[jira] [Updated] (AMBARI-15330) Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-15330?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez updated AMBARI-15330: - Attachment: (was: AMBARI-15330.trunk.patch) > Bubble up errors during RU

Re: Review Request 44492: [DRAFT] AMBARI-15330. Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez
https://reviews.apache.org/r/44492/diff/ Testing --- Verified on RU and EU by introducing a failure. Still need to write unit tests. == Thanks, Alejandro Fernandez

Review Request 44492: [DRAFT] AMBARI-15330. Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez
ommandDAO.java b48ffa8 ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 05d49c4 Diff: https://reviews.apache.org/r/44492/diff/ Testing --- Verified on RU and EU by introducing a failure. Still need to write unit tests. == Thanks, Alejandro Fernandez

[jira] [Updated] (AMBARI-15330) Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-15330?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez updated AMBARI-15330: - Attachment: AMBARI-15330.trunk.patch > Bubble up errors during RU

[jira] [Created] (AMBARI-15330) Bubble up errors during RU/EU

2016-03-07 Thread Alejandro Fernandez (JIRA)
Alejandro Fernandez created AMBARI-15330: Summary: Bubble up errors during RU/EU Key: AMBARI-15330 URL: https://issues.apache.org/jira/browse/AMBARI-15330 Project: Ambari Issue Type

Re: Review Request 44457: AMBARI-15310: Host Service Summary Page does not display long service names well

2016-03-07 Thread Alejandro Fernandez
tps://reviews.apache.org/r/44457/#comment184355> In the image for "Updated Fix (Mar-07).tiff" the component name of "ZOOKEEPER SERVER" is missing. - Alejandro Fernandez On March 7, 2016, 9

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-03-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44148/#review122373 --- Ship it! Ship It! - Alejandro Fernandez On March 6, 2016

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-03-07 Thread Alejandro Fernandez
> On March 7, 2016, 6:22 p.m., Alejandro Fernandez wrote: > > ambari-web/app/controllers/main/admin/kerberos.js, line 35 > > <https://reviews.apache.org/r/44148/diff/8/?file=1282155#file1282155line35> > > > > Do you plan on adding an #experimental flag that c

Re: Review Request 44387: AMBARI-15292: Text update for the Hive Server 2 port switch manual step in rolling upgrade

2016-03-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44387/#review122371 --- Ship it! Ship It! - Alejandro Fernandez On March 4, 2016, 1

Re: Review Request 42032: [AMBARI-12916] Make server hostname configurable

2016-03-07 Thread Alejandro Fernandez
t; <greg.h...@rackspace.com> wrote: > > >> On March 4, 2016, 7:36 p.m., Alejandro Fernandez wrote: >> > >>ambari-server/src/main/java/org/apache/ambari/server/configuration/Config >>uration.java, line 953 >> > >><https://reviews.apache.org/

Re: Review Request 42031: [AMBARI-12915] Make agent hostname configurable

2016-03-07 Thread Alejandro Fernandez
> On March 4, 2016, 4:19 p.m., Robert Levas wrote: > > Thinking more about this, the Hadoop services as well as some other > > services, like Oozie, internally perform the `_HOST` replacment with the > > local host's hostname. This mechanism is not getting the hostname value > > from the

Re: Review Request 44272: Install & Manage Zeppelin with Ambari

2016-03-07 Thread Alejandro Fernandez
first-time setup was successful. ambari-server/src/main/resources/common-services/ZEPPELIN/0.6.0.2.5.0.0/package/scripts/params.py (line 70) <https://reviews.apache.org/r/44272/#comment184316> spark_version is not defined in case of an exception.

Re: Review Request 42031: [AMBARI-12915] Make agent hostname configurable

2016-03-07 Thread Alejandro Fernandez
> On March 4, 2016, 8:43 p.m., Alejandro Fernandez wrote: > > ambari-agent/src/main/python/ambari_agent/hostname.py, line 40 > > <https://reviews.apache.org/r/42031/diff/1/?file=1186440#file1186440line40> > > > > There's already a script that can

Re: Review Request 42032: [AMBARI-12916] Make server hostname configurable

2016-03-07 Thread Alejandro Fernandez
> On March 4, 2016, 7:36 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/java/org/apache/ambari/server/configuration/Configuration.java, > > line 953 > > <https://reviews.apache.org/r/42032/diff/1/?file=1186442#file1186442line953> > > > &

Re: Review Request 44413: AMBARI-15308 UI: ability to perform bulk add host components

2016-03-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44413/#review122339 --- Ship it! Ship It! - Alejandro Fernandez On March 7, 2016, 6

Re: Review Request 44406: Move Hive Server Interactive related files from HDP 2.4 stack to HDP 2.6.

2016-03-07 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44406/#review122336 --- Ship it! Ship It! - Alejandro Fernandez On March 4, 2016, 9

Re: Review Request 44445: kafka commands throw an error after upgrading to HDP 2.4 from HDP 2.3

2016-03-07 Thread Alejandro Fernandez
/package/scripts/kafka.py (line 134) <https://reviews.apache.org/r/5/#comment184286> Why is the owner root? - Alejandro Fernandez On March 7, 2016, 11:31 a.m., Dmitro Lisnichenko wrote: > > --- > This is a

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-03-07 Thread Alejandro Fernandez
) <https://reviews.apache.org/r/44148/#comment184282> Do you plan on adding an #experimental flag that can be used to enable FreeIPA? This will have to undergo testing so I want to make sure it's visible after it has been fully vetted. Thank you - Alejandro Fernandez On

Re: Process for creating and running unit tests for patch

2016-03-05 Thread Alejandro Fernandez
Thank you for your contribution Kevin. I commented on https://issues.apache.org/jira/browse/AMBARI-13271 Cheers, Alejandro On 3/4/16, 9:06 PM, "Kevin Vasko" wrote: >Hello, > >I am trying to fix my first bug and contribute back. The issue is really >silly but I figured it

[jira] [Commented] (AMBARI-13271) Error: str() function only 1 takes parameter

2016-03-05 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-13271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15181941#comment-15181941 ] Alejandro Fernandez commented on AMBARI-13271: -- [~kvasko], thank you for your contribution

Re: How do outsiders actually get stuff committed?

2016-03-04 Thread Alejandro Fernandez
Dev mailing list doesn't allow attachments. I've uploaded my shell script here https://drive.google.com/folderview?id=0By8U4hLO7mEBQ2dvNVBHSHJTTkk=sha ring Thanks, Alejandro On 3/4/16, 1:07 PM, "Alejandro Fernandez" <afernan...@hortonworks.com> wrote: >I agree about includin

Re: How do outsiders actually get stuff committed?

2016-03-04 Thread Alejandro Fernandez
I agree about including several reviewers that have worked on that same area (git blame), maybe ping them if they don't reply within 24 hours. I often wait for 2 reviewers to say "Ship It" even though we only require 1. It helps to let someone know that they should submit it once you give the

Re: Review Request 43940: AMBARI-15153 : Change Ranger's default value for LDAP Group Sync Case Conversion properties to "none"

2016-03-04 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43940/#review122150 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 29, 2016, 3

Re: Review Request 42031: [AMBARI-12915] Make agent hostname configurable

2016-03-04 Thread Alejandro Fernandez
) <https://reviews.apache.org/r/42031/#comment184048> There's already a script that can be called to change the hostname that is reported to Ambari. - Alejandro Fernandez On Jan. 7, 2016, 8:17 p.m., Greg Hill

Re: Review Request 44109: RBAC : Background Jobs window doesn't show when login user role is : CLUSTER.ADMINISTRATOR, SERVICE.OPERATOR, SERVICE.ADMINISTRATOR and CLUSTER.OPERATOR.

2016-03-04 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44109/#review122148 --- Ship it! Ship It! - Alejandro Fernandez On March 4, 2016, 7

[jira] [Updated] (AMBARI-15012) Set "default" database as selected db when hive view loads

2016-03-04 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-15012?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez updated AMBARI-15012: - Fix Version/s: (was: 2.4.0) 2.2.2 > Set "

[jira] [Updated] (AMBARI-15012) Set "default" database as selected db when hive view loads

2016-03-04 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-15012?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez updated AMBARI-15012: - Fix Version/s: 2.4.0 > Set "default" database as selected db w

Re: Review Request 44397: New Alerts Do Not Honor Existing Maintenance Mode Setting

2016-03-04 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44397/#review122136 --- Ship it! Ship It! - Alejandro Fernandez On March 4, 2016, 6

Re: Review Request 44403: HiveServerInteractive. Correcting the 'hive.server2.thrift.port' and 'hive.server2.thrift.http.port' values.

2016-03-04 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44403/#review122133 --- Ship it! Ship It! - Alejandro Fernandez On March 4, 2016, 7

Re: Review Request 44109: RBAC : Background Jobs window doesn't show when login user role is : CLUSTER.ADMINISTRATOR, SERVICE.OPERATOR, SERVICE.ADMINISTRATOR and CLUSTER.OPERATOR.

2016-03-04 Thread Alejandro Fernandez
g/r/44109/#comment184033> Trailing whitespace. - Alejandro Fernandez On March 4, 2016, 1:28 a.m., Swapan Shridhar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Separate Issues Mailing list

2016-03-03 Thread Alejandro Fernandez
+1, dev should be meant for technical discussions while we create another list for jira traffic. Thanks, Alejandro On 3/3/16, 6:49 AM, "Robert Levas" wrote: >Seems line a fine idea to me. +1. > >Rob > > > > > >On 3/3/16, 7:09 AM, "Lars Francke"

Re: Review Request 44264: Upgrade Warning on Non-Versioned Components

2016-03-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44264/#review121703 --- Ship it! Ship It! - Alejandro Fernandez On March 2, 2016

Re: Review Request 43830: AMBARI-15127 : Add retry logic for DB Setup during Ranger upgrade step (to handle upgrade with Ranger HA)

2016-03-02 Thread Alejandro Fernandez
ttps://reviews.apache.org/r/43830/#comment183480> Change "( greater than 3GB )" to " (greater than 3GB) " - Alejandro Fernandez On March 2, 2016, 1:36 p.m., Gautam Borad wrote: > > --- > This is an automaticall

Re: Review Request 44272: Install & Manage Zeppelin with Ambari

2016-03-02 Thread Alejandro Fernandez
22) <https://reviews.apache.org/r/44272/#comment183466> Use explicit imports - Alejandro Fernandez On March 2, 2016, 4:54 p.m., Renjith Kamath wrote: > > --- > This is an automa

Re: Review Request 44267: Updates for initial logsearch stack definition

2016-03-02 Thread Alejandro Fernandez
SEARCH/0.5.0/metrics.json (line 5) <https://reviews.apache.org/r/44267/#comment183455> Why ganglia metrics? - Alejandro Fernandez On March 2, 2016, 3:02 p.m., Oliver Szabo wrote: > > --- > This is an automatically gener

Re: Review Request 44174: Ambari overwrites permissions on HDFS directories

2016-03-02 Thread Alejandro Fernandez
/test_hbase_master.py (line 497) <https://reviews.apache.org/r/44174/#comment183453> Can we declare this as a constant in each file to avoid so much repetition? - Alejandro Fernandez On March 2, 2016, 6:59 p.m., Andrew Onischuk

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-03-02 Thread Alejandro Fernandez
/serveraction/kerberos/IPAKerberosOperationHandler.java (line 36) <https://reviews.apache.org/r/44148/#comment183450> Just import the classes you need. - Alejandro Fernandez On March 2, 2016, 6:48 p.m., Bolke de Bruin

Re: Review Request 43254: [cleanup] Remove old unused component version/status staff from agent

2016-03-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43254/#review121688 --- Ship it! Ship It! - Alejandro Fernandez On March 2, 2016, 5

Re: Review Request 44210: AMBARI-15226 - The stack advisor should be pushed down to the services

2016-03-02 Thread Alejandro Fernandez
(line 35) <https://reviews.apache.org/r/44210/#comment183451> Please include Srimanth Gunturi in the code review, thanks! - Alejandro Fernandez On March 1, 2016, 4:53 p.m., Tim Thorpe wrote: > > --- > This is a

Re: Review Request 44057: AMBARI-15197 : Make use of log dir properties while installing Ranger

2016-03-02 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44057/#review121684 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 26, 2016, 6

Re: Review Request 43830: AMBARI-15127 : Add retry logic for DB Setup during Ranger upgrade step (to handle upgrade with Ranger HA)

2016-03-01 Thread Alejandro Fernandez
t value ambari-server/src/main/resources/common-services/RANGER_KMS/0.5.0.2.3/package/scripts/kms.py (line 81) <https://reviews.apache.org/r/43830/#comment183163> This method doesn't actually accept stack_version but the caller in the file above passes it. - Alejandro Fernandez On

[jira] [Resolved] (AMBARI-14597) Create Rolling and Express Upgrade Packs for HDP 2.2->2.4

2016-02-29 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-14597?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez resolved AMBARI-14597. -- Resolution: Delivered Delivered via several other checkins. AMBARI-14618

[jira] [Reopened] (AMBARI-14597) Create Rolling and Express Upgrade Packs for HDP 2.2->2.4

2016-02-29 Thread Alejandro Fernandez (JIRA)
[ https://issues.apache.org/jira/browse/AMBARI-14597?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alejandro Fernandez reopened AMBARI-14597: -- > Create Rolling and Express Upgrade Packs for HDP 2.2-&

Re: How hdp.version is calculated?

2016-02-29 Thread Alejandro Fernandez
A lot of services rely on how the process is started. It analysis its current working directory to find the version of HDP from the path. E.g., /usr/hdp/current/$component-name symlink to /usr/hdp/#.#.#.#-###/$component /usr/hdp/#.#.#.#-###/$component/bin/start.sh Thanks, Alejandro On 2/29/16,

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-02-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44148/#review121325 --- +1, please also get approval from Robert Levas - Alejandro

Dev Tip: Config Changes

2016-02-29 Thread Alejandro Fernandez
Hi all, For anyone contributing changes to the service configs, I'd like to highlight a couple of important cases that affect Rolling/Express Upgrade. In the past, there have been several bugs during RU/EU because we forgot to make changes to the Config Packs whenever editing stack configs, so

Re: Review Request 44178: Cluster Stack Version returning wrong state

2016-02-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44178/#review121306 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 29, 2016, 7

Re: Review Request 43355: Tez Ambari View: Add protocol configuration for YARN

2016-02-29 Thread Alejandro Fernandez
/ambari/Services.java (line 277) <https://reviews.apache.org/r/43355/#comment182987> What if the value is null. - Alejandro Fernandez On Feb. 9, 2016, 10:21 a.m., Sreenath Somarajapuram wrote: > > --- > This is a

Re: Review Request 44178: Cluster Stack Version returning wrong state

2016-02-29 Thread Alejandro Fernandez
tps://reviews.apache.org/r/44178/#comment182980> Was amazon2015 intentional? - Alejandro Fernandez On Feb. 29, 2016, 7:16 p.m., Nate Cole wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 44174: Ambari overwrites permissions on HDFS directories

2016-02-29 Thread Alejandro Fernandez
g/r/44174/#comment182977> We need to ensure this property exists during RU/EU. - Alejandro Fernandez On Feb. 29, 2016, 6:39 p.m., Andrew Onischuk wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > ht

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-02-29 Thread Alejandro Fernandez
. Thank you for adding IPA. - Alejandro Fernandez On Feb. 29, 2016, 10:09 a.m., Bolke de Bruin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 44148: Add FreeIPA support to Ambari.

2016-02-29 Thread Alejandro Fernandez
; May want to log this. ambari-web/app/controllers/main/admin/kerberos/step2_controller.js (line 257) <https://reviews.apache.org/r/44148/#comment182961> Can we convert to lower - Alejandro Fernandez On

Re: Review Request 44086: Registering 700 hosts gets stuck without timeout on 8 hosts

2016-02-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44086/#review121279 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 29, 2016, 5

Re: Review Request 44167: Express Upgrade config change files are in the wrong locations

2016-02-29 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44167/#review121278 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 29, 2016, 1

Re: Review Request 43940: AMBARI-15153 : Change Ranger's default value for LDAP Group Sync Case Conversion properties to "none"

2016-02-29 Thread Alejandro Fernandez
/configuration/usersync-properties.xml (line 128) <https://reviews.apache.org/r/43940/#comment182949> Does this also need to be set during RU/EU in certain HDP versions? - Alejandro Fernandez On Feb. 29, 2016, 3:40 a.m., Gautam Borad

Re: Review Request 43948: RM fails to start: IOException: /ats/active does not exist

2016-02-26 Thread Alejandro Fernandez
> On Feb. 24, 2016, 6:56 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py, > > line 123 > > <https://reviews.apache.org/r/43948/diff/1/?file=1267791#file1267791line123> > &g

Re: Review Request 44074: Authorization for Auto start services

2016-02-26 Thread Alejandro Fernandez
/ambari/server/controller/internal/ComponentResourceProviderTest.java (line 623) <https://reviews.apache.org/r/44074/#comment182463> Can we a description in Javadoc for this? - Alejandro Fernandez On Feb. 26, 2016, 4:05 p.m., Robert Levas

Re: Review Request 44086: Registering 700 hosts gets stuck without timeout on 8 hosts

2016-02-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44086/#review120933 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 26, 2016, 5

Re: Review Request 44066: Add pre-upgrade check that warns about components with VERSION_MISMATCH

2016-02-26 Thread Alejandro Fernandez
ent182455> Please add some javadoc for the description. - Alejandro Fernandez On Feb. 26, 2016, 3:30 p.m., Dmitro Lisnichenko wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 44064: Unusable configs after creating override

2016-02-26 Thread Alejandro Fernandez
f-else blocks ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java (line 1380) <https://reviews.apache.org/r/44064/#comment182447> This statement is not needed. - Alejandro Fernandez On Feb. 26, 2016, 12:29 p.m., Sebastian

Re: Review Request 44086: Registering 700 hosts gets stuck without timeout on 8 hosts

2016-02-26 Thread Alejandro Fernandez
/BSRunner.java (line 403) <https://reviews.apache.org/r/44086/#comment182438> Under what conditions would the file not exist? - Alejandro Fernandez On Feb. 26, 2016, 5:07 p.m., Andrew Onischuk wrote: > > --- > This is a

Re: Review Request 43967: Express Upgrade Stuck At Manual Prompt Due To HRC Status Calculation Cache Problem

2016-02-26 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43967/#review120910 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 26, 2016, 5

Re: Review Request 44006: column "upgrade_id" does not exist upgrading to Ambari 2.4.0

2016-02-25 Thread Alejandro Fernandez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44006/#review120736 --- Ship it! Ship It! - Alejandro Fernandez On Feb. 25, 2016, 6

Re: Review Request 43948: RM fails to start: IOException: /ats/active does not exist

2016-02-25 Thread Alejandro Fernandez
> On Feb. 24, 2016, 6:56 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py, > > line 123 > > <https://reviews.apache.org/r/43948/diff/1/?file=1267791#file1267791line123> > &g

Re: Review Request 43948: RM fails to start: IOException: /ats/active does not exist

2016-02-24 Thread Alejandro Fernandez
> On Feb. 24, 2016, 6:56 p.m., Alejandro Fernandez wrote: > > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/resourcemanager.py, > > line 123 > > <https://reviews.apache.org/r/43948/diff/1/?file=1267791#file1267791line123> > &g

  1   2   3   4   5   6   7   8   9   10   >