---
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
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
> 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
---
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
://reviews.apache.org/r/58671/diff/1/
Testing
---
Manual testing on a 2.4.2 cluster with permission issues
Thanks,
Juanjo Marron
cluster with permission issues
Thanks,
Juanjo Marron
---
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
---
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/53103/#review157438
---
Ship it!
Ship It!
- Juanjo Marron
On Nov. 30, 2016, 6:43
heck_process_status will handle it as
ComponentIsNotRunning
- Juanjo
-------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48766/#review144569
-
:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//console
This message is automatically generated.
Thanks,
Juanjo Marron
rn mysql_pid_file
>
> Andrew 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
rn mysql_pid_file
>
> Andrew 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
---
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
rn mysql_pid_file
>
> Andrew 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
ql_pid_file is None:
raise Fail("Output of '{0}' didn't contain pid_file. Output:
'{1}'").format(cmd, output)
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
;
> > 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
>
>
javac. The applied patch does 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
;
> > 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
>
>
nerated e-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 automatic
f
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
> 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
/7156//console
This message is automatically generated.
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
> 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
his 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
- 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
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
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
it this issue for
verification, please?
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47963/#review135820
-----------
On May 27, 2016, 4:33 p.m., Juanjo Marron wrote:
>
> -
iew135757
---
On May 27, 2016, 4:36 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
cally generated.
Thanks,
Juanjo Marron
patch/7006//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/7006//console
This message is automatically generated.
Thanks,
Juanjo Marron
cally generated.
Reply
Thanks,
Juanjo Marron
g/job/Ambari-trunk-test-patch/7000//console
This message is automatically generated.
Thanks,
Juanjo Marron
ssage is automatically generated.
Reply
Thanks,
Juanjo Marron
ge is automatically generated.
Thanks,
Juanjo Marron
mbari-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
https://builds.apache.org/job/Ambari-trunk-test-patch/6946//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/6946//console
This message is automatically generated.
Thanks,
Juanjo Marron
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
Test 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
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
sults:
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
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
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
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
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
ge is automatically generated.
Thanks,
Juanjo Marron
trunk-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
assed 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
https://builds.apache.org/job/Ambari-trunk-test-patch/6878//console
Thanks,
Juanjo Marron
e.org/r/46853/#review133259
---
On May 3, 2016, 6:12 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> 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
> 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
)
---
Sucessfull tests by Hadoop QA
Manual fresh installation and service checks
Thanks,
Juanjo Marron
tests by Hadoop QA
Thanks,
Juanjo Marron
his 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
> 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
versions
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46853/#review131209
---
On April 30, 2016, 12:51 a.m.,
---
Sucessfull tests by Hadoop QA
Thanks,
Juanjo Marron
/common-services/GANGLIA/3.5.0/package/scripts/params.py
30f8ee4
ambari-server/src/main/resources/common-services/GANGLIA/3.5.0/package/scripts/status_params.py
0c69ca9
Diff: https://reviews.apache.org/r/46853/diff/
Testing
---
Sucessfull tests by Hadoop QA
Thanks,
Juanjo Marron
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
----------
[INFO] Final Memory: 124M/560M
Additionally Build #6552 was successful:
https://builds.apache.org/job/Ambari-trunk-test-patch/6552/
Thanks,
Juanjo Marron
]
[INFO] Total time: 5:29.110s
[INFO] Finished at: Thu Apr 07 10:42:41 PDT 2016
[INFO] Final Memory: 114M/778M
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
r 1.2.1.2.3.0.0-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
>
> > 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.
>
> > 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.
45522/#review126274
-------
On March 31, 2016, 2:50 a.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.o
]
[root@localhost ambari]#
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]
[root@localhost ambari]#
Thanks,
Juanjo Marron
---
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:
>
> ---
gt; > The point of this function was to have a dictionary with the key being
> > each possible stack name. If we always return "hadoop-client", might as
> > well just return it as a string no matter what.
>
> Juanjo Marron wrote:
> The didct value is
MBARI-14451: Stack Featurize HDFS service (Juanjo Marron via jluniya)
Thanks Jayush!
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45328/#review126003
---
]
[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
]
[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
last trunk version
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45328/#review125895
---
On March 29, 2016, 8:
ated e-mail. To reply, visit:
https://reviews.apache.org/r/45328/#review125898
-----------
On March 29, 2016, 8:06 p.m., Juanjo Marron wrote:
>
> ---
> This is a
on() takes exactly 1 argument (0 given)
--
Total run:928
Total errors:1
Total failures:0
Thanks,
Juanjo Marron
;
> >
> > 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 extra/non
]
[INFO] Total time: 4:57.086s
[INFO] Finished at: Mon Mar 28 16:29:18 PDT 2016
[INFO] Final Memory: 111M/759M
[INFO] --
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:
>
> --
] Finished at: Thu Mar 25 11:22:15 PDT 2016
[INFO] Final Memory: 112M/531M
Thanks,
Juanjo Marron
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
1 argument (0 given)
--
Total run:928
Total errors:1
Total failures:0
Thanks,
Juanjo Marron
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
/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
/scripts/tez_client.py
e770d9b
Diff: https://reviews.apache.org/r/45254/diff/
Testing
---
TEZ fresh installation
Thanks,
Juanjo Marron
ssue
> > during zookeeper_server::security_status checks as only status_params is
> > imported in security_status()
> >
> > def security_status(self, env):
> > import status_params
> > env.set_params(status_params)
> > if s
1 - 100 of 112 matches
Mail list logo