//console
This message is automatically generated.
Thanks,
Juanjo Marron
> On June 7, 2016, 6:45 p.m., Alejandro Fernandez wrote:
> > Ship It!
>
> Juanjo Marron wrote:
> I dont have privileges to commit. Could someone deliver it to trunk so I
> can mark the JIRA https://issues.apache.org/jira/browse/AMBARI-16101 as
> complete, please?
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48184/#review136516
---
On June 7, 2016, 4:09 p.m., Juanjo Mar
/7006//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/7006//console
This message is automatically generated.
Thanks,
Juanjo Marron
generated.
Thanks,
Juanjo Marron
/Ambari-trunk-test-patch/7000//console
This message is automatically generated.
Thanks,
Juanjo Marron
generated.
Reply
Thanks,
Juanjo Marron
> On June 1, 2016, 2:33 a.m., Jayush Luniya wrote:
> > Patch doesnt apply cleanly. Can you rebase the patch and resolve conflicts?
>
> Juanjo Marron wrote:
> New patch submitted to https://issues.apache.org/jira/browse/AMBARI-16672.
> Waiting for HAdoop QA review
is automatically generated.
Thanks,
Juanjo Marron
-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/6985//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6985//console
This message is automatically generated.
Thanks,
Juanjo Marron
is automatically generated.
Reply
Thanks,
Juanjo Marron
iew135757
---
On May 27, 2016, 4:36 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
Diffs
-
ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/package/scripts/nodemanager.py
038ccb9
Diff: https://reviews.apache.org/r/48143/diff/
Testing
---
Added after Jayush Luniya comment on https://reviews.apache.org/r/47963
Thanks,
Juanjo Marron
://reviews.apache.org/r/48143/diff/
Testing
---
Added after Jayush Luniya comment on https://reviews.apache.org/r/47963
Thanks,
Juanjo Marron
t;
> > @Juanjo, it looks like check_process_status is not imported for YARN
> > nodemanager and will cause problems. Can you submit an addendum patch with
> > this change?
>
> Juanjo Marron wrote:
> Hi Jayush,
> Yes, ill work on fixing it. Can you tell me when you hit th
it this issue for
verification, please?
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47963/#review135820
-----------
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47653/#review135750
---
On May 20, 2016, 6:14 p.m., Juanjo Mar
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47923/#review135754
---
On May 26, 2016, 11:04 p.m., Juanjo Marron wrote:
>
> ---
> This i
> On June 1, 2016, 2:53 a.m., Jayush Luniya wrote:
> > Patch doesnt apply cleanly. Can you rebase the patch?
>
> Juanjo Marron wrote:
> Patch submitted to https://issues.apache.org/jira/browse/AMBARI-16797.
> Waiting for Hadoop QA review
Patch applied cleanly by
es not increase the total number of javac
compiler warnings.
+1 release audit. The applied patch does not increase the total number of
release audit warnings.
+1 core tests. The patch passed unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//console
This message is automatically generated.
Thanks,
Juanjo Marron
gt;
> > We should probably do a checked_call here to give an exception with a
> > good information in case we cannot do this.
> >
> > Also why to we need timeout here?
>
> Juanjo Marron wrote:
> Hi Andrew!
> Thanks for the quick review
>
>
return mysql_pid_file
I tested start, stop, and restart and seems to be working fine.
Please, let me know what do you think and I will upload the patch if your
feedback is positive
- Juanjo
---
This is an automatically generated
ndrew Onischuk wrote:
> Juanjo sorry for bothering you so much with this issues. I just feel like
> this is something which we should fix to be able debug problems in this place
> more simply in future.
>
> Juanjo Marron wrote:
> No problem Andrew! I totally understand
audit warnings.
+1 core tests. The patch passed unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//console
This message is automatically generated.
Thanks,
Juanjo Marron
mail. To reply, visit:
https://reviews.apache.org/r/48766/#review137885
-----------
On June 15, 2016, 10:39 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically gener
gt;
> > We should probably do a checked_call here to give an exception with a
> > good information in case we cannot do this.
> >
> > Also why to we need timeout here?
>
> Juanjo Marron wrote:
> Hi Andrew!
> Thanks for the quick review
>
>
gt; can create a json file at HDP/2.0.6/properties/stack_features.json and
> > reference it in the config property xml file. I will be committing those
> > changes soon.
I saw the example of stack_tools.json in https://reviews.apache.org/r/44835/.
It would work even if the stack_fetaure
rg/r/44933/diff/
Testing
---
Zookeeper fresh installation has been tested using HDP stack 2.3 and
AMBARI-13364 branch code.
Thanks,
Juanjo Marron
-
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45338/#review125695
---
On March 28, 2016, 4:09 p.m., Juanjo Marron wrote:
>
> --
;
> >
> > Can we remove the import *?
>
> Juanjo Marron wrote:
> Most of the service scripts still include the from resource_management
> import * line.
> I found 123 matches searching in common-services.
> It would be great to clean up the ext
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45470/#review126192
---
Ship it!
Ship It!
- Juanjo Marron
On March 30, 2016, 5:15
pport",
"min_version": "2.3.0.0"
}
I think rolling upgrade and config versioning could be added as features
later on when supported
- Juanjo Marron
On March 30, 2016, 5:15 a.m., Jayush Luniya wrote:
>
> ---
[INFO]
[root@localhost ambari]#
Thanks,
Juanjo Marron
] Finished at: Wed Apr 06 01:38:42 PDT 2016
[INFO] Final Memory: 112M/721M
Thanks,
Juanjo Marron
ly, visit:
https://reviews.apache.org/r/45778/#review127295
-------
On April 5, 2016, 9:46 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically gen
-2434' != 'hive --config
/etc/hive/conf.server --service hiveserver2 --deregister 1.2.1.2.3.0.0-2434'
--
Total run:933
Total errors:0
Total failures:1
Thanks,
Juanjo Marron
EATION
Diff: https://reviews.apache.org/r/44933/diff/
Testing
---
Zookeeper fresh installation has been tested using HDP stack 2.3 and
AMBARI-13364 branch code.
Thanks,
Juanjo Marron
iff: https://reviews.apache.org/r/45141/diff/
Testing
---
Zookeeper fresh installation on top of latest trunk
Thanks,
Juanjo Marron
/main/resources/common-services/TEZ/0.4.0.2.1/package/scripts/service_check.py
c0c66af
ambari-server/src/main/resources/common-services/TEZ/0.4.0.2.1/package/scripts/tez_client.py
e770d9b
Diff: https://reviews.apache.org/r/45254/diff/
Testing
---
TEZ fresh installation
Thanks,
Juanjo
l. To reply, visit:
https://reviews.apache.org/r/45254/#review125316
---
On March 24, 2016, 6:22 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mai
INFO]
[INFO] BUILD SUCCESS
[INFO]
[INFO] Total time: 6:20.634s
[INFO] Finished at: Thu Mar 25 11:22:15 PDT 2016
[INFO] Final Memory: 112M/531M
Thanks,
Juanjo Marron
eviews.apache.org/r/45522/#review126277
---
On March 31, 2016, 2:50 a.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To re
]
[INFO] Total time: 7:18.415s
[INFO] Finished at: Tue Mar 29 16:21:24 PDT 2016
[INFO] Final Memory: 111M/707M
[INFO]
Thanks,
Juanjo Marron
t;
> >
> > Let me check if this restriction for 2.2.1.0 is Oozie specific.
>
> Juanjo Marron wrote:
> Maintained for now in the new patch. Please, let me know if it needs to
> be edited
>
> Jayush Luniya wrote:
> @Juanjo. I didnt get a chance to look into this.
ver/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml
5f77f48
ambari-server/src/main/resources/stacks/HDP/2.0.6/properties/stack_features.json
PRE-CREATION
Diff: https://reviews.apache.org/r/45141/diff/
Testing
---
Zookeeper fresh installation on top of latest trunk
Thanks,
Juanjo Marron
e-mail. To reply, visit:
https://reviews.apache.org/r/44933/#review124636
-----------
On March 22, 2016, 1:03 a.m., Juanjo Marron wrote:
>
> ---
> This is an aut
time: 6:03.806s
[INFO] Finished at: Fri Apr 22 15:48:12 PDT 2016
[INFO] Final Memory: 124M/686M
Additionally Build #6552 was successful:
https://builds.apache.org/job/Ambari-trunk-test-patch/6552/
Thanks,
Juanjo Marron
MBARI-16014 which has been committed
yesterday
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46445/#review130152
----------
:
https://builds.apache.org/job/Ambari-trunk-test-patch/6906//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6906//console
Thanks,
Juanjo Marron
ated e-mail. To reply, visit:
https://reviews.apache.org/r/47560/#review134050
---
On May 18, 2016, 10:14 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically gen
results:
https://builds.apache.org/job/Ambari-trunk-test-patch/6920//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6920//console
This message is automatically generated.
Thanks,
Juanjo Marron
eviews.apache.org/r/47475/#review133768
-------
On May 18, 2016, 4:48 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To re
it:
https://reviews.apache.org/r/47477/#review133767
---
On May 18, 2016, 4:49 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To rep
it:
https://reviews.apache.org/r/47560/#review133823
---
On May 18, 2016, 10:14 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To rep
che.org/r/47475/#review133824
---
On May 18, 2016, 4:48 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
che.org/r/47541/#review133825
---
On May 18, 2016, 4:41 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
in snamenode
check_process_status(status_params.snamenode_pid_file)
NameError: global name 'check_process_status' is not defined
It was not captured on the UTs, fresh installation and service_check tests
Thanks,
Juanjo Marron
> On April 29, 2016, 10:34 p.m., Jayush Luniya wrote:
> > I think we need more testing on this apart from unit tests to ensure all
> > imports are there.
>
> Juanjo Marron wrote:
> Hi Jayush,
>
> Thanks for the review.
> I'll add this coup
unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/6869//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6869//console
This message is automatically generated.
Thanks,
Juanjo Marron
-test-patch/6870//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6870//console
This message is automatically generated.
Thanks,
Juanjo Marron
tests by Hadoop QA
Thanks,
Juanjo Marron
)
---
Sucessfull tests by Hadoop QA
Manual fresh installation and service checks
Thanks,
Juanjo Marron
> On April 29, 2016, 10:34 p.m., Jayush Luniya wrote:
> > I think we need more testing on this apart from unit tests to ensure all
> > imports are there.
>
> Juanjo Marron wrote:
> Hi Jayush,
>
> Thanks for the review.
> I'll add this coup
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46853/#review131430
---
On April 30, 2016, 12:51 a.m., Juanjo Marron wrote:
>
> ---
> This i
[INFO] Final Memory: 124M/560M
Additionally Build #6552 was successful:
https://builds.apache.org/job/Ambari-trunk-test-patch/6552/
Thanks,
Juanjo Marron
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49895/#review141961
---
Ship it!
Ship It!
- Juanjo Marron
On July 12, 2016, 4:48
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49930/#review141805
---
Ship it!
Ship It!
- Juanjo Marron
On July 11, 2016, 10:59
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49735/#review141106
---
Ship it!
Ship It!
- Juanjo Marron
On July 6, 2016, 11:45
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48766/#review144569
---
On Aug. 5, 2016, 9:49 p.m., Juanjo Marron wrote:
>
ds.apache.org/job/Ambari-trunk-test-patch/7367//console
This message is automatically generated.
Thanks,
Juanjo Marron
ndrew Onischuk wrote:
> Juanjo sorry for bothering you so much with this issues. I just feel like
> this is something which we should fix to be able debug problems in this place
> more simply in future.
>
> Juanjo Marron wrote:
> No problem Andrew! I totally understand
ndrew Onischuk wrote:
> Juanjo sorry for bothering you so much with this issues. I just feel like
> this is something which we should fix to be able debug problems in this place
> more simply in future.
>
> Juanjo Marron wrote:
> No problem Andrew! I totally understand
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53103/#review157438
---
Ship it!
Ship It!
- Juanjo Marron
On Nov. 30, 2016, 6:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57429/#review169028
---
Ship it!
Ship It!
- Juanjo Marron
On March 15, 2017, 4:33
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56673/#review171354
---
Ship it!
Ship It!
- Juanjo Marron
On April 6, 2017, 11:04
cluster with permission issues
Thanks,
Juanjo Marron
://reviews.apache.org/r/58671/diff/1/
Testing
---
Manual testing on a 2.4.2 cluster with permission issues
Thanks,
Juanjo Marron
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60844/#review180470
---
- Juanjo Marron
On July 13, 2017, 4:03 p.m., Robert Levas
> On July 13, 2017, 8:52 p.m., Juanjo Marron wrote:
> >
>
> Juanjo Marron wrote:
> Ship it!
Reviewd with Tuong Truong.
This is similar to the logic we were supporting in IOP for LDAP to PAM
migration.
Thanks for workin
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60844/#review180473
---
Ship it!
Ship It!
- Juanjo Marron
On July 13, 2017, 4:03
> On July 13, 2017, 8:52 p.m., Juanjo Marron wrote:
> >
Ship it!
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/60844/#rev
included as part of IOP
supporting QuickLinks through Knox.
Can we make sure all the tests run by HadoopQA pass, even if they are not
related to this change?
- Juanjo Marron
On July 21, 2017, 6:57 p.m., Chandana Mirashi wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61813/#review183494
---
Ship it!
Ship It!
- Juanjo Marron
On Aug. 22, 2017, 1:42
83 matches
Mail list logo