Re: Review Request 64977: UI receives a topology event when turning on/off maintenance mode for the host

2018-01-05 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64977/#review194868 --- Ship it! Ship It! - Eugene Chekanskiy On Січ. 5, 2018, 3:01

Review Request 64952: Disabling Kerberos after enabled during Blueprint install fails with missing data directory error

2018-01-04 Thread Eugene Chekanskiy
86d3ee03a0 ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java 198fdf5e1f Diff: https://reviews.apache.org/r/64952/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 64747: Don't call pprint.pformat unnecessarily in Ambari agent

2017-12-21 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64747/#review194407 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 20, 2017, 12

Re: Review Request 64544: Migrate user data for upgrade to improved user account management

2017-12-18 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64544/#review194074 --- Ship it! Ship It! - Eugene Chekanskiy On Dec. 13, 2017, 10

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-12-13 Thread Eugene Chekanskiy
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64118/#review193576 ----------- On Dec. 13, 2017, 6:14 p.m., Eugene Chekanskiy wrote: > > --

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-12-13 Thread Eugene Chekanskiy
/server/serveraction/upgrades/PreconfigureKerberosActionTest.java a08f7a0a25 Diff: https://reviews.apache.org/r/64118/diff/4/ Changes: https://reviews.apache.org/r/64118/diff/3-4/ Testing --- manual testing, all kind of regenerate keytabs mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 64542: Kerberos support for OneFS

2017-12-13 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64542/#review193700 --- Ship it! Ship It! - Eugene Chekanskiy On Dec. 12, 2017, 3

Re: Review Request 64501: Fix Stacks UT based on secured.json/default.json

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64501/#review193392 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017, 1

Re: Review Request 64499: Reporting host status was broken by merge

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64499/#review193391 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017, 10

Re: Review Request 64498: NFSGateway start failing with error : "ERROR: You must be a privileged user in order to run a secure service."

2017-12-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64498/#review193390 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 11, 2017, 10

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-12-06 Thread Eugene Chekanskiy
/server/serveraction/upgrades/PreconfigureKerberosActionTest.java a08f7a0a25 Diff: https://reviews.apache.org/r/64118/diff/3/ Changes: https://reviews.apache.org/r/64118/diff/2-3/ Testing --- manual testing, all kind of regenerate keytabs mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 64275: Broken Python Unit Tests in branch-2.6

2017-12-02 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64275/#review192627 --- Ship it! Ship It! - Eugene Chekanskiy On Гру. 2, 2017, 10

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-11-30 Thread Eugene Chekanskiy
Map<Key, SomePrimitiveType> only. - Eugene --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64118/#review192141 ------- On Nov. 29, 2017, 5:14 p.m., Eugene Chekanskiy wrote: > > -

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-11-29 Thread Eugene Chekanskiy
/serveraction/upgrades/PreconfigureKerberosActionTest.java a08f7a0a25 Diff: https://reviews.apache.org/r/64118/diff/2/ Testing (updated) --- manual testing, all kind of regenerate keytabs mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-11-29 Thread Eugene Chekanskiy
of regenerate keytabs unit tests updating in progress Thanks, Eugene Chekanskiy

Re: Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-11-29 Thread Eugene Chekanskiy
e.org/r/64118/diff/1/?file=1902432#file1902432line945> > > > > Should host_id be part of this? see comment above, host_id=kkp_id or host_id=null kkp_id=-1. - Eugene --- This is an automatically generated e-

Review Request 64118: Refactor internal code of handling info between kerberos wizard actions

2017-11-28 Thread Eugene Chekanskiy
://reviews.apache.org/r/64118/diff/1/ Testing --- manual testing, all kind of regenerate keytabs unit tests updating in progress Thanks, Eugene Chekanskiy

Re: Review Request 64089: Make Ambari LDAP configuration API consistent with existing API

2017-11-28 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64089/#review192023 --- Ship it! Ship It! - Eugene Chekanskiy On Nov. 27, 2017, 8

Re: Review Request 63907: AMBARI-22463 Removing secure reference configs entries from stack for Ranger KMS service

2017-11-17 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63907/#review191326 --- Ship it! Ship It! - Eugene Chekanskiy On Лис. 17, 2017, 12

Re: Review Request 63707: Simplify Ambari configuration tables from AMBARI-21307

2017-11-13 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63707/#review190840 --- Ship it! Ship It! - Eugene Chekanskiy On Nov. 10, 2017, 11

Re: Review Request 63698: Implement many-to-many relation between keytabs and principals

2017-11-13 Thread Eugene Chekanskiy
stageutils/ResolvedKerberosKeytab.java f66d273665 Diff: https://reviews.apache.org/r/63698/diff/3/ Changes: https://reviews.apache.org/r/63698/diff/2-3/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Review Request 63729: Blueprint deploys failing with missing smoke user keytab file

2017-11-10 Thread Eugene Chekanskiy
: https://reviews.apache.org/r/63729/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 63722: Remove IPA integration from experimental status

2017-11-10 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63722/#review190720 --- Ship it! Ship It! - Eugene Chekanskiy On Nov. 10, 2017, 11

Re: Review Request 63698: Implement many-to-many relation between keytabs and principals

2017-11-09 Thread Eugene Chekanskiy
ice only if host filter is exists. - Eugene --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63698/#review190583 ------- On Nov

Re: Review Request 63698: Implement many-to-many relation between keytabs and principals

2017-11-09 Thread Eugene Chekanskiy
ces/stacks/HDP/2.6/services/SPARK2/kerberos.json 89f19d4927 Diff: https://reviews.apache.org/r/63698/diff/2/ Changes: https://reviews.apache.org/r/63698/diff/1-2/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Review Request 63698: Implement many-to-many relation between keytabs and principals

2017-11-09 Thread Eugene Chekanskiy
beros.json 89f19d4927 Diff: https://reviews.apache.org/r/63698/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 63450: Improve KDC integration

2017-11-02 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63450/#review189916 --- Ship it! Ship It! - Eugene Chekanskiy On Oct. 31, 2017, 9

Re: Review Request 63180: Improve Kerberos principal and keytab accounting

2017-10-25 Thread Eugene Chekanskiy
eatures like many-to-many relation for principals-keytabs will be implemented. - Eugene --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63180/#review189176 ------- On Oct. 24, 2017, 6:02 p.m., Eugene Chekanskiy wrote: &

Re: Review Request 63180: Improve Kerberos principal and keytab accounting

2017-10-24 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63180/#review188838 ------- On Oct. 24, 2017, 6:02 p.m., Eugene Chekanskiy wrote: > > --

Re: Review Request 63180: Improve Kerberos principal and keytab accounting

2017-10-24 Thread Eugene Chekanskiy
enable, kerberos disable, add service Thanks, Eugene Chekanskiy

Review Request 63180: Improve Kerberos principal and keytab accounting

2017-10-20 Thread Eugene Chekanskiy
/java/org/apache/ambari/server/serveraction/upgrades/PreconfigureKerberosActionTest.java a7bf33c775 Diff: https://reviews.apache.org/r/63180/diff/1/ Testing --- mvn clean test, kerberos enable, kerberos disable, add service Thanks, Eugene Chekanskiy

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Eugene Chekanskiy
-services/HIVE/2.1.0.3.0/package/scripts/params_linux.py f6b676b65e Diff: https://reviews.apache.org/r/62866/diff/3/ Changes: https://reviews.apache.org/r/62866/diff/2-3/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62866: Make hive server create directories related to replication

2017-10-11 Thread Eugene Chekanskiy
ambari-server/src/main/resources/common-services/HIVE/2.1.0.3.0/package/scripts/params_linux.py f6b676b65e Diff: https://reviews.apache.org/r/62866/diff/2/ Changes: https://reviews.apache.org/r/62866/diff/1-2/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Review Request 62866: Make hive server create directories related to replication

2017-10-10 Thread Eugene Chekanskiy
/2.1.0.3.0/package/scripts/params_linux.py f6b676b65e Diff: https://reviews.apache.org/r/62866/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62628: Allow for backward compatibility in how identity references are specified

2017-09-28 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62628/#review186573 --- Ship it! Ship It! - Eugene Chekanskiy On Sept. 27, 2017, 8

Review Request 62669: Make dfs.permissions.superusergroup as group property

2017-09-28 Thread Eugene Chekanskiy
/configuration/hadoop-env.xml 4154007 ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/configuration/hdfs-site.xml a4fed0f Diff: https://reviews.apache.org/r/62669/diff/1/ Testing --- mnv clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

2017-09-21 Thread Eugene Chekanskiy
: https://reviews.apache.org/r/62467/diff/2/ Changes: https://reviews.apache.org/r/62467/diff/1-2/ Testing --- mvn clean test, cluster deployment. Thanks, Eugene Chekanskiy

Review Request 62467: Add UID/GID related issue with external users not listed in /etc/passwd

2017-09-21 Thread Eugene Chekanskiy
deployment. Thanks, Eugene Chekanskiy

Review Request 62388: APPLICATION_TIMELINE_SERVER filed to start, coz were already started

2017-09-18 Thread Eugene Chekanskiy
stacks/2.1/YARN/test_apptimelineserver.py b352920 Diff: https://reviews.apache.org/r/62388/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62290: Validate kerberos.json files to ensure they meet the expected schema

2017-09-18 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62290/#review185577 --- Ship it! Ship It! - Eugene Chekanskiy On Sept. 18, 2017, 4

Re: Review Request 62290: Validate kerberos.json files to ensure they meet the expected schema

2017-09-18 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62290/#review18 --- Ship it! Ship It! - Eugene Chekanskiy On Вер. 15, 2017, 2

Re: Review Request 62186: Implement host recovery - backend changes

2017-09-08 Thread Eugene Chekanskiy
-3/ Testing --- mvn clean test; cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62186: Implement host recovery - backend changes

2017-09-08 Thread Eugene Chekanskiy
-2/ Testing --- mvn clean test; cluster deploy Thanks, Eugene Chekanskiy

Review Request 62186: Implement host recovery - backend changes

2017-09-08 Thread Eugene Chekanskiy
/KerberosIdentityDataFileTest.java 323ba8e ambari-server/src/test/java/org/apache/ambari/server/serveraction/kerberos/KerberosServerActionTest.java f63e6b8 Diff: https://reviews.apache.org/r/62186/diff/1/ Testing --- mvn clean test; cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 62035: User can't add node via Ambari UI when being part of both "cluster user" and "cluster admin" roles

2017-09-04 Thread Eugene Chekanskiy
rg/r/62035/diff/1-2/ Testing --- mvn clean test; manual check Thanks, Eugene Chekanskiy

Review Request 62035: User can't add node via Ambari UI when being part of both "cluster user" and "cluster admin" roles

2017-09-01 Thread Eugene Chekanskiy
r/internal/RequestResourceProvider.java 5a0549d ambari-server/src/test/java/org/apache/ambari/server/controller/internal/RequestResourceProviderTest.java c519323 Diff: https://reviews.apache.org/r/62035/diff/1/ Testing --- mvn clean test; manual check Thanks, Eugene Chekanskiy

Re: Review Request 61951: Add keberos service for 3.0.0 stack

2017-08-29 Thread Eugene Chekanskiy
On Aug. 28, 2017, 5:14 p.m., Eugene Chekanskiy wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61951/ > --- &

Review Request 61951: Add keberos service for 3.0.0 stack

2017-08-28 Thread Eugene Chekanskiy
-CREATION Diff: https://reviews.apache.org/r/61951/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 61744: Allow for keytab regeneration to be filtered for hosts

2017-08-22 Thread Eugene Chekanskiy
/3/ Changes: https://reviews.apache.org/r/61744/diff/2-3/ Testing --- mvn clean test. manual tests on deployed cluster. Thanks, Eugene Chekanskiy

Re: Review Request 61744: Allow for keytab regeneration to be filtered for hosts

2017-08-22 Thread Eugene Chekanskiy
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61744/#review183453 ------- On Aug. 21, 2017, 4:31 p.m., Eugene Chekanskiy wrote: > > -

Re: Review Request 61744: Allow for keytab regeneration to be filtered for hosts

2017-08-21 Thread Eugene Chekanskiy
/ Changes: https://reviews.apache.org/r/61744/diff/1-2/ Testing --- mvn clean test. manual tests on deployed cluster. Thanks, Eugene Chekanskiy

Review Request 61744: Allow for keytab regeneration to be filtered for hosts

2017-08-18 Thread Eugene Chekanskiy
abb017b ambari-server/src/test/java/org/apache/ambari/server/controller/KerberosHelperTest.java b220999 Diff: https://reviews.apache.org/r/61744/diff/1/ Testing --- mvn clean test. manual tests on deployed cluster. Thanks, Eugene Chekanskiy

Re: Review Request 61605: Allow WEB alert accept custom HTTP codes

2017-08-16 Thread Eugene Chekanskiy
Thanks, Eugene Chekanskiy

Re: Review Request 61605: Allow WEB alert accept custom HTTP codes

2017-08-14 Thread Eugene Chekanskiy
iew182859 --- On Aug. 14, 2017, 1:23 p.m., Eugene Chekanskiy wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 61605: Allow WEB alert accept custom HTTP codes

2017-08-14 Thread Eugene Chekanskiy
/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 61605: Allow WEB alert accept custom HTTP codes

2017-08-12 Thread Eugene Chekanskiy
/ambari/server/state/alert/AlertUri.java dea105e Diff: https://reviews.apache.org/r/61605/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 61186: Service repos are not updated with "latest" url in repoinfo.xml

2017-07-27 Thread Eugene Chekanskiy
://reviews.apache.org/r/61186/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 60998: ACTIVITY_ANALYZER Install failed: Error: Unable to run the custom hook script

2017-07-20 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60998/#review181036 --- Ship it! Ship It! - Eugene Chekanskiy On Липень 20, 2017, 1

Review Request 60897: Add UID/GID related enhancements

2017-07-16 Thread Eugene Chekanskiy
, Eugene Chekanskiy

Re: Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Eugene Chekanskiy
mber 4 means? no matter, it is removed anyway) - Eugene --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60153/#review178100 --- On June

Review Request 60153: LDAP sync requires user to be root

2017-06-16 Thread Eugene Chekanskiy
Diff: https://reviews.apache.org/r/60153/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 60048: Kafka userprincipal to shortname is not using AUTH_TO_LOCAL rules for authorization

2017-06-13 Thread Eugene Chekanskiy
7500891 contrib/management-packs/hdf-ambari-mpack/src/main/resources/stacks/HDF/2.0/services/KAFKA/kerberos.json e1e6461 Diff: https://reviews.apache.org/r/60048/diff/1/ Testing --- mvn clean test, cluster deploy Thanks, Eugene Chekanskiy

Re: Review Request 59648: Add a flag to indicate NN restart is rolling

2017-05-31 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59648/#review176446 --- Ship it! Ship It! - Eugene Chekanskiy On Травень 31, 2017

Re: Review Request 59102: LDAPS connections to an Active Directory when enabling Kerberos should validate the server's SSL certificate

2017-05-11 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59102/#review174645 --- Ship it! Ship It! - Eugene Chekanskiy On May 9, 2017, 4:38

Re: Review Request 59045: Insecure SSL: Server Identity Verification Disabled

2017-05-08 Thread Eugene Chekanskiy
--- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 58969: Old Status Command Structured Output is Returned on Every Status Command Causing Upgrades to Fail

2017-05-03 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58969/#review173819 --- Ship it! Ship It! - Eugene Chekanskiy On May 3, 2017, 6 p.m

Review Request 58736: After WE is enabled, graphana fails to start with SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version

2017-04-26 Thread Eugene Chekanskiy
d8d0515 ambari-server/src/main/resources/stacks/BIGTOP/0.8/services/HDFS/package/scripts/params.py f85efb0 ambari-server/src/test/python/stacks/2.5/RANGER_KMS/test_kms_server.py 7082a33 Diff: https://reviews.apache.org/r/58736/diff/1/ Testing --- mvn clean test Thanks, Eugene

Re: Review Request 58667: Ambari agents can only connect to the server using TLSv1

2017-04-24 Thread Eugene Chekanskiy
/diff/1-2/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 58667: Ambari agents can only connect to the server using TLSv1

2017-04-24 Thread Eugene Chekanskiy
/services/HDFS/package/scripts/params.py 49cfa86 ambari-server/src/main/resources/stacks/BIGTOP/0.8/services/HDFS/package/scripts/service_check.py 81d7ca5 Diff: https://reviews.apache.org/r/58667/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 58346: AMBARI-20733 /var/log/krb5kdc.log is growing rapidly on the KDC server

2017-04-21 Thread Eugene Chekanskiy
/stacks/2.5/ATLAS/test_atlas_server.py cd2fac8 Diff: https://reviews.apache.org/r/58346/diff/3/ Changes: https://reviews.apache.org/r/58346/diff/2-3/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 58346: AMBARI-20733 /var/log/krb5kdc.log is growing rapidly on the KDC server

2017-04-13 Thread Eugene Chekanskiy
/python/stacks/2.5/ATLAS/test_atlas_server.py cd2fac8 Diff: https://reviews.apache.org/r/58346/diff/2/ Changes: https://reviews.apache.org/r/58346/diff/1-2/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 58398: Remove multiprocessing-based StatusCommandsExecutor

2017-04-12 Thread Eugene Chekanskiy
142e7ca Diff: https://reviews.apache.org/r/58398/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Review Request 58346: AMBARI-20733 /var/log/krb5kdc.log is growing rapidly on the KDC server

2017-04-11 Thread Eugene Chekanskiy
to be reimplemented in something more reliable. Diffs - ambari-agent/src/main/python/ambari_agent/ActionQueue.py 75880c6 Diff: https://reviews.apache.org/r/58346/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 58073: With multi-process StatusCommandsExecutor, Status commands are taking too long to report back

2017-04-03 Thread Eugene Chekanskiy
/StatusCommandsExecutor.py 04a3e85 ambari-agent/src/main/python/ambari_agent/main.py 923c570 Diff: https://reviews.apache.org/r/58073/diff/2/ Changes: https://reviews.apache.org/r/58073/diff/1-2/ Testing --- mvn clean test, manual tests to check if queues really drained fast enough. Thanks, Eugene

Re: Review Request 58073: With multi-process StatusCommandsExecutor, Status commands are taking too long to report back

2017-03-30 Thread Eugene Chekanskiy
eviews.apache.org/r/58073/#review170584 --- On March 30, 2017, 1:32 p.m., Eugene Chekanskiy wrote: > > --- > This is an automatically generated e-mail. To re

Review Request 58073: With multi-process StatusCommandsExecutor, Status commands are taking too long to report back

2017-03-30 Thread Eugene Chekanskiy
/python/ambari_agent/main.py 923c570 Diff: https://reviews.apache.org/r/58073/diff/1/ Testing --- mvn clean test, manual tests to check if queues really drained fast enough. Thanks, Eugene Chekanskiy

Re: Review Request 57968: Prune files contained in jcepolicyinfo.jar to a minimal set

2017-03-28 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57968/#review170317 --- Ship it! Ship It! - Eugene Chekanskiy On March 28, 2017, 3

Re: Review Request 57875: In case of IBM Power, test_CompressionTool[lzo] fails with 'lzo.GPLNativeCodeLoader: Could not load native gpl library'

2017-03-23 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57875/#review169868 --- Ship it! Ship It! - Eugene Chekanskiy On March 23, 2017, 1

Review Request 57782: Ambari-agent log rotation is broken

2017-03-20 Thread Eugene Chekanskiy
: https://reviews.apache.org/r/57782/diff/1/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 57685: Security status commands fill ambari-agent.log with stack traces

2017-03-16 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57685/#review169145 --- Ship it! Ship It! - Eugene Chekanskiy On Березень 16, 2017

Re: Review Request 57612: Some random status command takes more than 5 seconds to execute

2017-03-14 Thread Eugene Chekanskiy
/main/python/ambari_agent/StatusCommandsExecutor.py 3b23f1c Diff: https://reviews.apache.org/r/57612/diff/1/ Testing (updated) --- mvn clean test, manual test Thanks, Eugene Chekanskiy

Review Request 57612: Some random status command takes more than 5 seconds to execute

2017-03-14 Thread Eugene Chekanskiy
/StatusCommandsExecutor.py 3b23f1c Diff: https://reviews.apache.org/r/57612/diff/1/ Testing --- Thanks, Eugene Chekanskiy

Re: Review Request 57559: python UT fail

2017-03-13 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57559/#review168765 --- Ship it! Ship It! - Eugene Chekanskiy On March 13, 2017, 3

Re: Review Request 57558: Add a property to enable Status commands on separate process (non-default)

2017-03-13 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57558/#review168761 --- Ship it! Ship It! - Eugene Chekanskiy On March 13, 2017, 2

Re: Review Request 57410: When SPNEGO authentication is enabled for Hadoop in a cluster with NN HA, PXF Process alert fails

2017-03-09 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57410/#review168458 --- Ship it! Ship It! - Eugene Chekanskiy On March 8, 2017, 4

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-09 Thread Eugene Chekanskiy
; > and `not self.worker_process.is_alive()` as 3rd OR condition, this way > > omitting `self.worker_process and`? done - Eugene --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-09 Thread Eugene Chekanskiy
Thanks, Eugene Chekanskiy

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-07 Thread Eugene Chekanskiy
Thanks, Eugene Chekanskiy

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-07 Thread Eugene Chekanskiy
Thanks, Eugene Chekanskiy

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-07 Thread Eugene Chekanskiy
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57344/#review168101 ------- On March 7, 2017, 3:28 p.m., Eugene Chekanskiy wrote: > >

Re: Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-07 Thread Eugene Chekanskiy
Thanks, Eugene Chekanskiy

Review Request 57344: Commands timed-out on ambari host without any error logs

2017-03-06 Thread Eugene Chekanskiy
/ambari_agent/TestController.py 663e215 ambari-agent/src/test/python/ambari_agent/TestMain.py 97c448b Diff: https://reviews.apache.org/r/57344/diff/1/ Testing --- mvn clean test, manual testing Thanks, Eugene Chekanskiy

Re: Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-02 Thread Eugene Chekanskiy
] Final Memory: 242M/2931M [INFO] Thanks, Eugene Chekanskiy

Review Request 57204: After regenerate keytabs post Ambari upgrade yarn.nodemanager.linux-container-executor.cgroups.mount-path property got added with blank value

2017-03-01 Thread Eugene Chekanskiy
] [INFO] Total time: 31:15.402s [INFO] Finished at: Wed Mar 01 17:58:13 EET 2017 [INFO] Final Memory: 242M/2931M [INFO] Thanks, Eugene Chekanskiy

Re: Review Request 57145: Services fail to start due to incorrect permissions on spnego.service.keytab

2017-02-28 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57145/#review167079 --- Ship it! Ship It! - Eugene Chekanskiy On Лют. 28, 2017, 1

Re: Review Request 56980: Adding spark2 to a kerberized cluster with spark1 fails with keytab file does not exist error

2017-02-28 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56980/#review167078 --- Ship it! Ship It! - Eugene Chekanskiy On Лют. 27, 2017, 2

Re: Review Request 57087: HDFS service check fails after Stopping one Namenode in HA cluste

2017-02-28 Thread Eugene Chekanskiy
eviews.apache.org/r/57087/#review166902 --- On Feb. 27, 2017, 4:19 p.m., Eugene Chekanskiy wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57087/ > ---

Re: Review Request 57087: HDFS service check fails after Stopping one Namenode in HA cluste

2017-02-27 Thread Eugene Chekanskiy
, Eugene Chekanskiy

Review Request 57087: HDFS service check fails after Stopping one Namenode in HA cluste

2017-02-27 Thread Eugene Chekanskiy
/resources/common-services/HDFS/2.1.0.2.0/package/scripts/service_check.py dffa077 ambari-server/src/test/python/stacks/2.0.6/HDFS/test_service_check.py bbc1b3a Diff: https://reviews.apache.org/r/57087/diff/ Testing --- mvn clean test Thanks, Eugene Chekanskiy

Re: Review Request 56800: Finalize Operations stage fails when Enabling Kerberos using the manual option

2017-02-21 Thread Eugene Chekanskiy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56800/#review166174 --- Ship it! Ship It! - Eugene Chekanskiy On Лют. 17, 2017, 8

  1   2   >