---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64827/#review194509
---
Ship it!
Ship It!
- Andrew Onischuk
On Dec. 23, 2017, 12:43
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64552/#review194316
---
Ship it!
Ship It!
- Andrew Onischuk
On Dec. 12, 2017, 7:36
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64747/#review194235
---
Ship it!
Ship It!
- Andrew Onischuk
On Dec. 20, 2017, 12:18
214 (original), 215 (patched)
<https://reviews.apache.org/r/64747/#comment272963>
Not a issue. Just wondering if that does make a change in calls number or
just a refactoring.
- Andrew Onischuk
On Dec. 20, 2017, 12:18 p.m., Attila Magyar
/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
fede57b075
ambari-server/src/test/python/stacks/2.6/configs/default_kafka_sasl_plaintext.json
953e44462a
ambari-server/src/test/python/stacks/2.6/configs/default_kafka_sasl_ssl.json
c00528c499
Diff: https://reviews.apache.org/r/64709/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew
://reviews.apache.org/r/64682/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64547/#review193546
---
Ship it!
Ship It!
- Andrew Onischuk
On Dec. 12, 2017, 3:49
ambari-server/src/test/resources/TestAmbaryServer.samples/dummy_stack/HIVE/package/scripts/hive.py
f9ad166aef
Diff: https://reviews.apache.org/r/64501/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
-22623
https://issues.apache.org/jira/browse/AMBARI-22623
Repository: ambari
Description
---
Diffs
-
ambari-agent/src/main/python/ambari_agent/HostStatusReporter.py c27ee552f6
Diff: https://reviews.apache.org/r/64499/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew
raise ExecutionFailed(err_msg, code, out, err)
resource_management.core.exceptions.ExecutionFailed: Execution of
'ambari-sudo.sh su hdfs -l -s /bin/bash -c 'ulimit -c unlimited ;
/usr/hdp/3.0.0.0-555/hadoop/bin/hdfs --config /usr/hdp/3.0.0.0-555/hadoop/conf
--daemon start nfs3'' returned 1. WARNING: HADOOP_PRIVILEGED_NFS_USER has been
replaced by HDFS_NFS3_SECURE_USER. Using value of HADOOP_PRIVILEGED_NFS_USER.
WARNING: HADOOP_NFS3_OPTS has been replaced by HDFS_NFS3_OPTS. Using value
of HADOOP_NFS3_OPTS.
ERROR: You must be a privileged user in order to run a secure service.
Diffs
-
ambari-server/src/main/resources/common-services/HDFS/3.0.0.3.0/package/scripts/status_params.py
153f9a6ca6
Diff: https://reviews.apache.org/r/64498/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
libraries.script import Script
return RepositoryUtil(Script.get_config(), set()).create_repo_files()
...
- Andrew Onischuk
On Dec. 5, 2017, 11:04 a.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To reply
://reviews.apache.org/r/64209/diff/2/
Changes: https://reviews.apache.org/r/64209/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
es:0
Diffs
-
ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py
10a83d6c23
ambari-server/src/test/python/stacks/2.2/configs/oozie-upgrade.json
29cbddca55
Diff: https://reviews.apache.org/r/64275/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
ustom repo handling
> in any case, we do not have a repo-tag -> license-tag mapping anywhere so
> wouldn't it be equally ugly?
>
> Sid Wagle wrote:
> By license-tag I meant some lookup to figure out whether license was
> accepted.
>
> Andrew Onischuk wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64209/#review192365
-------
On Nov. 30, 2017, 1:28 p.m., Andrew Onischuk wrote:
>
> -
iew192335
---
On Nov. 30, 2017, 1:28 p.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revie
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64186/#review192331
---
Ship it!
Ship It!
- Andrew Onischuk
On Nov. 30, 2017, 6:36
ambari-server/src/test/python/stacks/utils/RMFTestCase.py 80712c5ebe
Diff: https://reviews.apache.org/r/64209/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
> On Nov. 30, 2017, 8:19 a.m., Andrew Onischuk wrote:
> > ambari-server/src/main/python/ambari_server/serverConfiguration.py
> > Line 1162 (original), 1157 (patched)
> > <https://reviews.apache.org/r/64186/diff/3/?file=1904146#file1904146line1163>
> >
> >
> On Nov. 30, 2017, 8:19 a.m., Andrew Onischuk wrote:
> > ambari-server/src/main/python/ambari_server/serverConfiguration.py
> > Line 1162 (original), 1157 (patched)
> > <https://reviews.apache.org/r/64186/diff/3/?file=1904146#file1904146line1163>
> >
> >
> On Nov. 30, 2017, 8:19 a.m., Andrew Onischuk wrote:
> > ambari-server/src/main/python/ambari_server/serverConfiguration.py
> > Line 1162 (original), 1157 (patched)
> > <https://reviews.apache.org/r/64186/diff/3/?file=1904146#file1904146line1163>
> >
> >
/serverConfiguration.py
Line 1162 (original), 1157 (patched)
<https://reviews.apache.org/r/64186/#comment270281>
I think you might have broken silent flag to enable gpl by doing this.
- Andrew Onischuk
On Nov. 30, 2017, 12:20 a.m., Sid Wagle
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64186/#review192244
---
Ship it!
Ship It!
- Andrew Onischuk
On Nov. 30, 2017, 12:20
ckage/scripts/params_linux.py
0e6179b8a7
Diff: https://reviews.apache.org/r/64153/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
833fdbc8ac
Diff: https://reviews.apache.org/r/64117/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
Diff: https://reviews.apache.org/r/64117/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/stack_packages.json
PRE-CREATION
Diff: https://reviews.apache.org/r/64082/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
://reviews.apache.org/r/63982/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63916/#review191492
---
Thanks for reviews
- Andrew Onischuk
On Nov. 20, 2017, 10:35
-server/src/test/python/stacks/2.2/configs/oozie-upgrade.json
86ca03a4a3101e646f265b5dca6e32bb179c9856
Diff: https://reviews.apache.org/r/63916/diff/2/
Changes: https://reviews.apache.org/r/63916/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
Also - don't forget to merge into 3.0 stack in trunk as well ... anything
> Python changes in branch-2.6 need extra-work done when going into trunk.
>
> Andrew Onischuk wrote:
> Sure @Jonathan.
> Do you guys think a separate jira sho
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63916/#review191346
-------
On Nov. 17, 2017, 2:40 p.m., Andrew Onischuk wrote:
>
> -
ow if you think
differently
- Andrew
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63916/#review191345
---
On Nov. 17,
eviews.apache.org/r/63916/#review191345
---
On Nov. 17, 2017, 2:40 p.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To re
enerated e-mail. To reply, visit:
https://reviews.apache.org/r/63916/#review191344
---
On Nov. 17, 2017, 2:40 p.m., Andrew Onischuk wrote:
>
> ---
> This is an automati
it for easier
review.
- Andrew Onischuk
On Nov. 17, 2017, 2:40 p.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
Thanks,
Andrew Onischuk
/ambari_server/serverUpgrade.py 421adfc4a1
ambari-server/src/test/python/TestAmbariServer.py 840b9a6579
Diff: https://reviews.apache.org/r/63876/diff/2/
Changes: https://reviews.apache.org/r/63876/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
accidentally
committed old version of the patch (from here)
- Andrew Onischuk
On Nov. 16, 2017, 6:26 a.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
/63876/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/role_command_order.json
78a31f1797
Diff: https://reviews.apache.org/r/63872/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63838/#review191162
---
Thanks for your reviews guys.
- Andrew Onischuk
On Nov. 16
ambari-server/src/main/python/ambari_server/serverConfiguration.py c7ac13476f
ambari-server/src/main/python/ambari_server/serverSetup.py c1643f2621
Diff: https://reviews.apache.org/r/63838/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/ambari_agent/dummy_files/stomp/execution_commands.json
76dac1b975
Diff: https://reviews.apache.org/r/63755/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/diff/2/
Changes: https://reviews.apache.org/r/63464/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
ark2
- Andrew
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63464/#review190596
---
On Nov. 1, 2017, 12:43 p.m., Andrew
,
Andrew Onischuk
-----
mvn clean test
Thanks,
Andrew Onischuk
ly, visit:
https://reviews.apache.org/r/63618/#review190309
-------
On Nov. 7, 2017, 1:48 p.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail.
/TestAgentStompResponses.py
2154fca8d3
Diff: https://reviews.apache.org/r/63618/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/main/python/ambari_agent/alerts/script_alert.py 301e4401f9
ambari-agent/src/main/python/ambari_agent/alerts/web_alert.py 0e400f75e3
Diff: https://reviews.apache.org/r/63444/diff/2/
Changes: https://reviews.apache.org/r/63444/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
nt/src/main/python/ambari_agent/main.py 3948290519
ambari-agent/src/test/python/ambari_agent/TestAgentStompResponses.py
2154fca8d3
Diff: https://reviews.apache.org/r/63504/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/script_alert.py 301e4401f9
ambari-agent/src/main/python/ambari_agent/alerts/web_alert.py 0e400f75e3
Diff: https://reviews.apache.org/r/63444/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/main/resources/stacks/PERF/2.0/repos/repoinfo.xml
5c3b40b7e5
contrib/utils/perf/deploy-gce-perf-cluster.py 7431ae9d7a
Diff: https://reviews.apache.org/r/63412/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/stacks/PERF/2.0/repos/repoinfo.xml
5c3b40b7e5
ambari-server/src/main/resources/stacks/PERF/2.0/vdf.xml PRE-CREATION
Diff: https://reviews.apache.org/r/63411/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
tacks/2.3/SPARK/test_spark_thrift_server.py
fbe5403a35
Diff: https://reviews.apache.org/r/63289/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
l be used to report the version of
the component which was installed
Command failed after 1 tries
Diffs
-
ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/package/scripts/params_linux.py
289a1d6571
Diff: https://reviews.apache.org/r/63243/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
rc/main/resources/custom_actions/scripts/ru_execute_tasks.py
23a0cfb7f5
Diff: https://reviews.apache.org/r/63240/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
ns.Fail: Configuration parameter
'fs.defaultFS' was not found in configurations dictionary!
Diffs
-
ambari-server/src/main/resources/stacks/HDP/2.0.6/hooks/before-START/scripts/params.py
55a609328e
Diff: https://reviews.apache.org/r/63080/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/scripts/params.py
466837c5cd
Diff: https://reviews.apache.org/r/63071/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/python/custom_actions/test_ru_set_all.py de83f7e07b
Diff: https://reviews.apache.org/r/62932/diff/2/
Changes: https://reviews.apache.org/r/62932/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/test_ru_set_all.py de83f7e07b
Diff: https://reviews.apache.org/r/62932/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/stacks/HDP/2.0.6/properties/stack_packages.json
2db0a74195
Diff: https://reviews.apache.org/r/62888/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/utils/RMFTestCase.py 7091298cbc
Diff: https://reviews.apache.org/r/62808/diff/3/
Changes: https://reviews.apache.org/r/62808/diff/2-3/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/python/stacks/utils/RMFTestCase.py c10ff64e0b
Diff: https://reviews.apache.org/r/62808/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/test/python/stacks/utils/RMFTestCase.py c10ff64e0b
Diff: https://reviews.apache.org/r/62808/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
8e07
ambari-agent/src/test/python/ambari_agent/TestSecurity.py c9a7fbe865
Diff: https://reviews.apache.org/r/62771/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
correctly, during rewritting.
This happened only once on perf cluster. But worth a preventive fix.
Diffs
-
ambari-agent/src/main/python/ambari_agent/ClusterCache.py 40b32886cf
Diff: https://reviews.apache.org/r/62750/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/UpgradeCatalog300Test.java
95a04deb6b
ambari-web/app/mixins/common/persist.js e69de29bb2
Diff: https://reviews.apache.org/r/62674/diff/3/
Changes: https://reviews.apache.org/r/62674/diff/2-3/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
://reviews.apache.org/r/62674/diff/2/
Changes: https://reviews.apache.org/r/62674/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
://reviews.apache.org/r/62674/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
credentials",)
Diffs
-
ambari-server/src/main/resources/common-services/ZEPPELIN/0.7.0/package/scripts/master.py
af5758a6c5
Diff: https://reviews.apache.org/r/62667/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
eviews.apache.org/r/62563/#review186227
---
On Sept. 26, 2017, 10:58 a.m., Andrew Onischuk wrote:
>
> ---
> This is an automatically generated e-mail. To re
,
Andrew Onischuk
,
Andrew Onischuk
/main/java/org/apache/ambari/server/events/listeners/alerts/AlertReceivedListener.java
3607df4
Diff: https://reviews.apache.org/r/62563/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
-
ambari-server/src/main/resources/scripts/configs.py 639f28e
ambari-server/src/main/resources/scripts/configs.sh 5fc96fd
Diff: https://reviews.apache.org/r/62501/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
rg/r/62500/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
Upgrade.py a10841d
Diff: https://reviews.apache.org/r/62431/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/TestAgentStompResponses.py 3c571ab
Diff: https://reviews.apache.org/r/62402/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
> > Sounds like a very dirty hack. Does that reproduce in latest Ambari
> > versions? Maybe it would be better to check status() before attempt to run
> > start() ?
>
> Andrew Onischuk wrote:
> +1. Checking output might break with a simple change to ats, also i
--
>
> (Updated Sept. 18, 2017, 6 p.m.)
>
>
> Review request for Ambari, Andrew Onischuk, Dmitro Lisnichenko, and Robert
> Levas.
>
>
> Bugs: AMBARI-21984
> https://issues.apache.org/jira/browse/AMBARI-21984
>
>
> Repository: ambari
output in exception text. To give debugging information for failures of ATS.
- Andrew Onischuk
On Sept. 18, 2017, 6 p.m., Eugene Chekanskiy wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
ambari-agent/src/main/python/ambari_agent/InitializerModule.py 4fbef65
Diff: https://reviews.apache.org/r/62379/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/params_linux.py
956438e
ambari-server/src/main/resources/stacks/HDP/2.0.6/configuration/cluster-env.xml
eb16778
Diff: https://reviews.apache.org/r/62278/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
966254a
Diff: https://reviews.apache.org/r/62215/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/python/ambari_agent/ComponentStatusExecutor.py 4ccd815
Diff: https://reviews.apache.org/r/62151/diff/2/
Changes: https://reviews.apache.org/r/62151/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
test
Thanks,
Andrew Onischuk
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62121/#review184672
---
Ship it!
Ship It!
- Andrew Onischuk
On Sept. 6, 2017, 1:41
have now sends
port property as "defaultPort" and "port" instead of "default_port".
Is that expected to happen? Should I do the change in python code to reflect
this?
- Andrew Onischuk
On Sept. 5, 2017, 12:3
hon/resource_management/libraries/functions/copy_tarball.py
795160c
Diff: https://reviews.apache.org/r/62058/diff/2/
Changes: https://reviews.apache.org/r/62058/diff/1-2/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
> On Sept. 5, 2017, 12:44 p.m., Andrew Onischuk wrote:
> > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py
> > Line 149 (original), 149 (patched)
> > <https://reviews.apache.org/r/62081/diff/1/?file=1814401#file1814401line149>
> >
> >
Line 149 (original), 149 (patched)
<https://reviews.apache.org/r/62081/#comment260693>
Can we remove this line?
- Andrew Onischuk
On Sept. 5, 2017, 12:43 p.m., Attila Doroszlai wrote:
>
> ---
> This is an automatically g
.py
795160c
Diff: https://reviews.apache.org/r/62058/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
ain/python/ambari_server/serverConfiguration.py f6f3f43
ambari-server/src/main/python/ambari_server/serverUpgrade.py 07f5097
Diff: https://reviews.apache.org/r/62026/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
, line 279, in do_handshake
self._sslobj.do_handshake()
Diffs
-
ambari-agent/src/main/python/ambari_agent/HeartbeatThread.py b6e1aaf
ambari-common/src/main/python/ambari_ws4py/client/__init__.py 89598ab
Diff: https://reviews.apache.org/r/62012/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
ds
Diffs
-
ambari-server/src/main/resources/stacks/BigInsights/4.2/upgrades/config-upgrade.xml
d85c7ee
Diff: https://reviews.apache.org/r/61945/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
/resource_management/libraries/providers/msi.py
7f4deeb
Diff: https://reviews.apache.org/r/61941/diff/1/
Testing
---
mvn clean test
Thanks,
Andrew Onischuk
1 - 100 of 674 matches
Mail list logo