,
Laszlo Puskas
is configured with sudo on agents.
Thanks,
Laszlo Puskas
/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing (updated)
---
Unit tests passed.
Manual testing underway.
Thanks,
Laszlo Puskas
/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing
---
Unit tests passed.
Manual testing underway.
Thanks,
Laszlo Puskas
if ls is configured with sudo on agents.
Thanks,
Laszlo Puskas
-
ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing (updated)
---
Unit tests passed.
Thanks,
Laszlo Puskas
/TopologyManager.java
(line 162)
<https://reviews.apache.org/r/48266/#comment201269>
I'd rather perform this check out of this method, in order to always have a
boolean return value for this method.
- Laszlo Puskas
On June 6, 2016, 8:47 a.m., Daniel Gergely
t can be /usr/hdp.
Using the stack_root to the log commands.
- Laszlo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48044/#review135943
---------
)
-
ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing
---
Manual testing underway.
Thanks,
Laszlo Puskas
/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing (updated)
---
Unit tests passed.
Manual testing underway please don't commit yet.
Thanks,
Laszlo Puskas
/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing (updated)
---
Unit tests passed.
Manual testing underway.
Thanks,
Laszlo Puskas
/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing (updated)
---
Unit tests passed.
Manual testing underway please don't commit yet.
// found some major differences between the branches / investigating it
Thanks,
Laszlo Puskas
/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing
---
Unit tests passed.
Thanks,
Laszlo Puskas
----------
On June 13, 2016, 8:28 a.m., Laszlo Puskas wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48044/
> ---
information.
The patch only applies for 2.2-next versions.
Diffs (updated)
-
ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py
9a3201e
Diff: https://reviews.apache.org/r/48044/diff/
Testing
---
Unit tests passed.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48691/#review137527
---
Ship it!
Ship It!
- Laszlo Puskas
On June 14, 2016, 2:51
t.java
(line 87)
<https://reviews.apache.org/r/48266/#comment202953>
The member annotated with @TestSubject is instantiated by the framework.
Why do we need two instances here?
- Laszlo Puskas
On June 15, 2016, 11:33 a.m., Daniel G
/diff/
Testing
---
Unit tests running;
Manually tested on local env.
Thanks,
Laszlo Puskas
, visit:
https://reviews.apache.org/r/48044/#review135565
-------
On June 2, 2016, 12:55 p.m., Laszlo Puskas wrote:
>
> ---
> This is an automatically gener
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48169/#review135950
---
Ship it!
Ship It!
- Laszlo Puskas
On June 2, 2016, 2:38 p.m
: https://reviews.apache.org/r/48732/diff/
Testing (updated)
---
Unit tests succeeded
Manually tested on local env.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48722/#review138587
---
Ship it!
Ship It!
- Laszlo Puskas
On June 16, 2016, 4:43
/HeartBeatHandler.java
(line 532)
<https://reviews.apache.org/r/48722/#comment203792>
This call here is superfluous.
- Laszlo Puskas
On June 16, 2016, 4:43 p.m., Sebastian Toader wrote:
>
> ---
> This is an automatically g
,
Laszlo Puskas
changed the test framework from Mockito to EasyMock. The test "logic"
didn't change at all.
- Laszlo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49260/#review139909
--------
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46765/#review130897
---
Ship it!
Ship It!
- Laszlo Puskas
On April 28, 2016, 9:06
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46654/#review130606
---
Ship it!
Ship It!
- Laszlo Puskas
On April 25, 2016, 7:03
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47596/#review134149
---
Ship it!
Ship It!
- Laszlo Puskas
On May 20, 2016, 8:01 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46899/#review131493
---
Ship it!
Ship It!
- Laszlo Puskas
On May 2, 2016, 8:01 p.m
/org/apache/ambari/server/controller/internal/BlueprintConfigurationProcessorTest.java
2759869
Diff: https://reviews.apache.org/r/46973/diff/
Testing
---
In progress...
Thanks,
Laszlo Puskas
/internal/BlueprintConfigurationProcessorTest.java
2759869
Diff: https://reviews.apache.org/r/46973/diff/
Testing (updated)
---
Tested manually.
Unit tests passed.
Thanks,
Laszlo Puskas
---
Unit test done. (mvn test underway)
Manually tested on local / openstack cluster.
Thanks,
Laszlo Puskas
/TopologyManager.java
(line 395)
<https://reviews.apache.org/r/46496/#comment193458>
Add the host to the log message
- Laszlo Puskas
On April 21, 2016, 3:19 p.m., Sebastian Toader wrote:
>
> ---
> This is an automatica
Diff: https://reviews.apache.org/r/46198/diff/
Testing (updated)
---
Unit test done. Tests ran successfully.
Manually tested on local / openstack cluster.
Thanks,
Laszlo Puskas
://reviews.apache.org/r/46198/diff/
Testing (updated)
---
Unit test done. Tests ran successfully.
Manually tested on local / openstack cluster.
Java Results :
Tests run: 4226, Failures: 0, Errors: 0, Skipped: 32
Python:
Total run:959
Total errors:0
Total failures:0
OK
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47117/#review132266
---
Ship it!
Ship It!
- Laszlo Puskas
On May 9, 2016, 1:01 p.m
bd1bef2
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/nonrolling-upgrade-2.5.xml
065a933
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml
d152f34
Diff: https://reviews.apache.org/r/50336/diff/
Testing
---
Manual testing underway.
Thanks,
Laszlo
underway.
Thanks,
Laszlo Puskas
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
0190478
Diff: https://reviews.apache.org/r/50753/diff/
Testing
---
Manually, in progress.
Thanks,
Laszlo Puskas
)
-
ambari-server/src/main/java/org/apache/ambari/server/state/host/HostImpl.java
a757010
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
0190478
Diff: https://reviews.apache.org/r/50753/diff/
Testing
---
Manually, in progress.
Thanks,
Laszlo Puskas
in progress.
Thanks,
Laszlo Puskas
/server/controller/AmbariManagementControllerTest.java
fd70df5
Diff: https://reviews.apache.org/r/49260/diff/
Testing (updated)
---
Manually tested on dev-env.
Unit tests running
Thanks,
Laszlo Puskas
--
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49260/#review139577
---
On June 28, 2016, 2:09 p.m., Laszlo Puskas wrote:
>
> ---
> This is an automatically generated e-
-
ambari-server/src/main/java/org/apache/ambari/server/controller/AmbariCustomCommandExecutionHelper.java
b60592d
Diff: https://reviews.apache.org/r/49260/diff/
Testing
---
Manually tested on dev-env.
Unit tests running.
Thanks,
Laszlo Puskas
---
Manually tested on dev-env.
Unit tests running.
Thanks,
Laszlo Puskas
)
---
Manually tested on dev-env.
OK
--
Total run:1073
Total errors:0
Total failures:0
Thanks,
Laszlo Puskas
al won't interfere with the
new method.
- Laszlo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50753/#review144920
----------
st and when
> > it's received by your subscriber method.
> >
> > If the problem only happens on registration, I think this solution is OK.
> > But if a heartbeat from an agent can cause problems as well, then there is
> > still a slight chance that the timing her
/java/org/apache/ambari/server/topology/TopologyManager.java
9a6ee94
Diff: https://reviews.apache.org/r/50753/diff/
Testing
---
Suceesfully tested. (Manually)
Unit tests OK
Thanks,
Laszlo Puskas
(updated)
-
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
9a6ee94
Diff: https://reviews.apache.org/r/50753/diff/
Testing
---
Suceesfully tested. (Manually)
Unit tests OK
Thanks,
Laszlo Puskas
-upgrade-2.5.xml
065a933
ambari-server/src/main/resources/stacks/HDP/2.4/upgrades/upgrade-2.5.xml
d152f34
Diff: https://reviews.apache.org/r/50336/diff/
Testing
---
Manual testing underway.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49383/#review140150
---
Ship it!
Ship It!
- Laszlo Puskas
On June 29, 2016, 4:14
seconds)
Diffs
-
ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
e0bda13
Diff: https://reviews.apache.org/r/49455/diff/
Testing
---
Unit tests running.
Thanks,
Laszlo Puskas
: https://reviews.apache.org/r/49455/diff/
Testing
---
Unit tests running.
Thanks,
Laszlo Puskas
.
Thanks,
Laszlo Puskas
e looked up with a single scan ...
- Laszlo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49455/#review140277
-------
On
.
Thanks,
Laszlo Puskas
this shortens the startup time by ~25 seconds)
Diffs (updated)
-
ambari-server/src/main/java/org/apache/ambari/server/controller/ControllerModule.java
e0bda13
Diff: https://reviews.apache.org/r/49455/diff/
Testing
---
Unit tests running.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56175/#review163842
---
Ship it!
Ship It!
- Laszlo Puskas
On Feb. 1, 2017, 4:43 p.m
been added, the related guice
module hasn;t been set into the tool's injector.
The fix is to add the missing module to the db-cleanup tool's injector.
branch-2.5:
commit 8111ba662e849c27e4a9a04305b3ef2260c030e9
Author: Laszlo Puskas <lpus...@hortonworks.com>
Date:?? Fri Jan 20 19:18:10 2017
a382bed7f55be632fd03e1b02bb8a01151234b24
Author: Laszlo Puskas <lpus...@hortonworks.com>
Date: Fri Jan 20 12:41:02 2017 +0100
AMBARI-19613. ZKFC Zookeper connection is not secure. (Laszlo Puskas via
stoader)
branch-2.5
commit 00b2c42ccf6fe68267483a645f6e57e9c921f01b
Author: Laszlo Puskas
user (not root ...)
- opened the generated csv as the newlycreated user
Unit test created.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55983/#review163124
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 26, 2017, 1:31
/post-user-creation-hook.sh ee8d2d1
ambari-server/src/test/java/org/apache/ambari/server/hooks/users/UserHookServiceTest.java
293b22a
Diff: https://reviews.apache.org/r/55987/diff/
Testing
---
Unit tests running.
Manual testing underway
Thanks,
Laszlo Puskas
/users/PostUserCreationHookServerActionTest.java
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing
---
Unit tests running.
Manual testing underway
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56064/#review163483
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 29, 2017, 8:43
Diff: https://reviews.apache.org/r/55762/diff/
Testing
---
Manually.
Thanks,
Laszlo Puskas
been added, the related guice
module hasn;t been set into the tool's injector.
The fix is to add the missing module to the db-cleanup tool's injector.
branch-2.5:
commit 8111ba662e849c27e4a9a04305b3ef2260c030e9
Author: Laszlo Puskas <lpus...@hortonworks.com>
Date:?? Fri Jan 20 19:18:10 2017
/server/serveraction/users/PostUserCreationHookServerActionTest.java
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing
---
Unit tests running.
Manual testing underway
Thanks,
Laszlo Puskas
ambari-server/src/test/java/org/apache/ambari/server/serveraction/users/PostUserCreationHookServerActionTest.java
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing
---
Unit tests running.
Manual testing underway
Thanks,
Laszlo Puskas
/PostUserCreationHookServerActionTest.java
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing
---
Unit tests running.
Manual testing underway
Thanks,
Laszlo Puskas
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing (updated)
---
Unit tests succeeded.
Manual testing underway
Thanks,
Laszlo Puskas
f5cdf48
Diff: https://reviews.apache.org/r/55987/diff/
Testing (updated)
---
Unit tests succeeded.
Manually tested changes.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56054/#review163430
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 29, 2017, 12:50
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56783/#review165942
---
Ship it!
Ship It!
- Laszlo Puskas
On Feb. 17, 2017, 3:16
be notified
about the success/failure of the operation)
Diffs
-
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/namenode.py
123486e
Diff: https://reviews.apache.org/r/57040/diff/
Testing
---
Manually.
Unit tests running.
Thanks,
Laszlo Puskas
/test_namenode.py fae500f
Diff: https://reviews.apache.org/r/57040/diff/
Testing (updated)
---
Manually.
Unit tests running...
Thanks,
Laszlo Puskas
://reviews.apache.org/r/57040/diff/
Testing (updated)
---
Manually.
Unit tests success.
Thanks,
Laszlo Puskas
/UpgradeCatalog250Test.java
(line 1666)
<https://reviews.apache.org/r/56599/#comment237174>
Instead mocking tehe injector itself, add a module where instances are
bound to mocks.
- Laszlo Puskas
On Feb. 13, 2017, 1:48 p.m., Attila Magyar
> On Feb. 13, 2017, 3:08 p.m., Laszlo Puskas wrote:
> > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java,
> > line 1672
> > <https://reviews.apache.org/r/56599/diff/1/?file=1631931#file1631931line1672>
> >
> > In
> On Feb. 13, 2017, 3:08 p.m., Laszlo Puskas wrote:
> > ambari-server/src/test/java/org/apache/ambari/server/upgrade/UpgradeCatalog250Test.java,
> > line 1672
> > <https://reviews.apache.org/r/56599/diff/1/?file=1631931#file1631931line1672>
> >
> > In
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56713/#review165719
---
Ship it!
Ship It!
- Laszlo Puskas
On Feb. 15, 2017, 3:24
.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55833/#review162805
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 24, 2017, 1 p.m
/resources/scripts/post-user-creation-hook.sh 91511a0
Diff: https://reviews.apache.org/r/56600/diff/
Testing
---
Manually, underway.
Thanks,
Laszlo Puskas
/ unrelated test failed though.
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55513/#review161720
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 13, 2017, 6:14
/DefaultTimeSource.java
(line 29)
<https://reviews.apache.org/r/55643/#comment233314>
Why is this abstarction needed? Couldn't we simply use Calendar instead?
- Laszlo Puskas
On Jan. 17, 2017, 9:30 p.m., Balázs Bence Sári
https://reviews.apache.org/r/55680/#review162123
-----------
On Jan. 18, 2017, 3:58 p.m., Laszlo Puskas wrote:
>
> ---
> This is an automatically generated e-ma
e-mail. To reply, visit:
https://reviews.apache.org/r/55680/#review162125
-------
On Jan. 18, 2017, 3:58 p.m., Laszlo Puskas wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55680
the configuration entry: ha.zookeeper.acl - removed
* checked the hadoop-env.sh - doesn't contain the variable export
HADOOP_ZKFC_OPTS
Unit tests running.
Thanks,
Laszlo Puskas
the hadoop-env.sh - doesn't contain the variable export
HADOOP_ZKFC_OPTS
Unit tests running.
Thanks,
Laszlo Puskas
export
HADOOP_ZKFC_OPTS
Unit tests:
Successfully ran on local machine / unrelated test failed though.
Thanks,
Laszlo Puskas
the configuration entry: ha.zookeeper.acl - removed
* checked the hadoop-env.sh - doesn't contain the variable export
HADOOP_ZKFC_OPTS
Unit tests running.
Thanks,
Laszlo Puskas
* checked the hadoop-env.sh - doesn't contain the variable export
HADOOP_ZKFC_OPTS
Unit tests running.
Thanks,
Laszlo Puskas
/ActiveStandbyElectorLock)
Unit tests:
Running in progress for trunk
Thanks,
Laszlo Puskas
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55764/#review162439
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 20, 2017, 4:31
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55777/#review162460
---
Ship it!
Ship It!
- Laszlo Puskas
On Jan. 20, 2017, 6:30
/apache/ambari/server/state/host/HostImpl.java
a757010
ambari-server/src/main/java/org/apache/ambari/server/topology/TopologyManager.java
0190478
Diff: https://reviews.apache.org/r/50753/diff/
Testing (updated)
---
Suceesfully tested. (Manually)
Unit tests OK
Thanks,
Laszlo Puskas
e command output we can't get information about the balancer.
- Laszlo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57147/#review167110
----------
1 - 100 of 234 matches
Mail list logo