ou
> > 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 sta
it:
https://reviews.apache.org/r/44933/#review124298
---
On March 16, 2016, 9:24 p.m., Juanjo Marron wrote:
>
> ---
> This is an automatically generated e-mail. To reply, vis
rg/r/44933/diff/
Testing
---
Zookeeper fresh installation has been tested using HDP stack 2.3 and
AMBARI-13364 branch code.
Thanks,
Juanjo Marron
e Attachments (updated)
AMBARI-14435.patch
https://reviews.apache.org/media/uploaded/files/2016/03/22/ca5af0ea-dcdc-4ded-9e36-eda14e7b6ecf__AMBARI-14435.patch
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
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, express_upgrade and config_versioning
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44933/#review124634
-------
01834line25>
> >
> > FYI, in https://reviews.apache.org/r/44835/ I added a feature to
> > support external properties file for complete structures like these. So you
> > can create a json file at HDP/2.0.6/properties/stack_features.json and
> > reference
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
eper_service.py
6fe0772
ambari-server/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
es
- Juanjo
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45141/#review124702
---
On March 22, 2016, 1:50 a.m., Juanjo Marron wrote:
>
> -
: https://reviews.apache.org/r/45141/diff/
Testing
---
Zookeeper fresh installation on top of latest trunk
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
/scripts/tez_client.py
e770d9b
Diff: https://reviews.apache.org/r/45254/diff/
Testing
---
TEZ fresh installation
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
1 argument (0 given)
--
Total run:928
Total errors:1
Total failures:0
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
] Finished at: Thu Mar 25 11:22:15 PDT 2016
[INFO] Final Memory: 112M/531M
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:
>
> --
]
[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
;
> >
> > 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
on() takes exactly 1 argument (0 given)
--
Total run:928
Total errors:1
Total failures:0
Thanks,
Juanjo Marron
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
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:
]
[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
]
[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
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
---
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
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:
>
> ---
---
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
[INFO]
[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
]
[root@localhost ambari]#
Thanks,
Juanjo Marron
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
>
> > 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.
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
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
] Finished at: Wed Apr 06 01:38:42 PDT 2016
[INFO] Final Memory: 112M/721M
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
[INFO] Final Memory: 124M/560M
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
----------
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
/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
---
Sucessfull tests by Hadoop QA
Thanks,
Juanjo Marron
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.,
> 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
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
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
> 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
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:
https://builds.apache.org/job/Ambari-trunk-test-patch/6878//console
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
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
ge 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
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
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
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
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
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
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
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
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
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
ge is automatically generated.
Thanks,
Juanjo Marron
ssage is automatically generated.
Reply
Thanks,
Juanjo Marron
g/job/Ambari-trunk-test-patch/7000//console
This message is automatically generated.
Thanks,
Juanjo Marron
cally generated.
Reply
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.
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
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:
>
> -
://reviews.apache.org/r/48143/diff/
Testing
---
Added after Jayush Luniya comment on https://reviews.apache.org/r/47963
Thanks,
Juanjo Marron
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
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
- 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
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
> 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
> 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
/7156//console
This message is automatically generated.
Thanks,
Juanjo Marron
- 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
> 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?
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
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
;
> > 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
>
>
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
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/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/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/49895/#review141961
---
Ship it!
Ship It!
- Juanjo Marron
On July 12, 2016, 4:48
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
:
https://builds.apache.org/job/Ambari-trunk-test-patch/7367//console
This message is automatically generated.
Thanks,
Juanjo Marron
1 - 100 of 112 matches
Mail list logo