ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Florian Barca" <fba...@hortonworks.com>
Subject Re: Review Request 29983: Refactor the OS-dependent Ambari Server Windows components - Part 1.4
Date Fri, 23 Jan 2015 00:56:39 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29983/
-----------------------------------------------------------

(Updated Jan. 23, 2015, 12:56 a.m.)


Review request for Ambari, Artem Baranchuk, Eugene Chekanskiy, Jonathan Hurley, Jayush Luniya,
Mahadev Konar, and Nate Cole.


Changes
-------

Fixed issues when upgrading to a version beyond 1.7.0.


Bugs: AMBARI-8317
    https://issues.apache.org/jira/browse/AMBARI-8317


Repository: ambari


Description
-------

+Further decomposing ambari-server.py: Moved setup, reset and upgrade into their own files
+Made them OS-independent - now the same routines are being used in Windows too (save upgrade,
which is not active in Windows yet)
+Merged the OS-dependent code for JDBC & JDK setup, database interface creation, firewall
checks
+Refactored the database setup: split the supported DBMSes into distinct objects
+Replaced recursive calls in the user input routines


Diffs (updated)
-----

  ambari-common/src/main/python/ambari_commons/firewall.py 76d9890a8edc2902c1f2c145626b6b6e41836573

  ambari-common/src/main/python/ambari_commons/inet_utils.py 2a54cb6867aec65d6400451ab12e30c146193cfa

  ambari-common/src/main/python/ambari_commons/logging_utils.py af1390cc4643d3b2e80f0cdf3697babcc48eb23a

  ambari-server/conf/unix/ambari.properties e29a6e25f576a8f806a447a4cf03c56f4d0c105b 
  ambari-server/conf/windows/ambari.properties d5c531319e52da10b599fd771d63ca6eee88a39f 
  ambari-server/src/main/python/ambari-server-windows.py 923110546517c1962cfd7c9b20fdd4f0721ff907

  ambari-server/src/main/python/ambari-server.py ea072ab0fddcf48979b830faea86f1ee7b4336aa

  ambari-server/src/main/python/ambari_server/dbConfiguration.py a2008794522fb18fbef745433dfb3aaa63f6368f

  ambari-server/src/main/python/ambari_server/dbConfiguration_linux.py 62853b72787c14be8717e786f6f43c391c8901c8

  ambari-server/src/main/python/ambari_server/dbConfiguration_windows.py ab038db82c8467e877f6822b3401bc6688149114

  ambari-server/src/main/python/ambari_server/serverConfiguration.py 5547e443c7193eb51bf6e2c3f3245ebc72614e7b

  ambari-server/src/main/python/ambari_server/serverSetup.py 7d4544595396ecbe9bdb663deea124d7842bbacd

  ambari-server/src/main/python/ambari_server/serverSetup_linux.py 33241515f6648f8a066f090ec6a98493008ff092

  ambari-server/src/main/python/ambari_server/serverSetup_windows.py dce404d50b98bd4330da8891d9eb0b2e61a76cf7

  ambari-server/src/main/python/ambari_server/serverUpgrade.py PRE-CREATION 
  ambari-server/src/main/python/ambari_server/setupActions.py ee8eaa07fee924ec46a6edc828ea9dc4686fbdc5

  ambari-server/src/main/python/ambari_server/setupSecurity.py 34070391046a0aee680f735a6f3e0e23ba8a7d16

  ambari-server/src/main/python/ambari_server/userInput.py 3e8b2c74b28212b3fcc552d1697087f8101f6c74

  ambari-server/src/main/resources/custom_actions/scripts/check_host.py c33580645311a3962878e1b6f17e90a3e2f1a937

  ambari-server/src/main/resources/stacks/HDPWIN/2.1/hooks/before-ANY/scripts/setup_jdk.py
d48bb523e9ff4d8c81d42fed25d0043e75a0e3cb 
  ambari-server/src/main/resources/stacks/HDPWIN/2.1/services/YARN/package/scripts/mapred_service_check.py
f81408f546a90e07ed3a1434b7d6e8f62fa62966 
  ambari-server/src/test/python/TestAmbariServer.py 44844589953c87e4d495f02883ab9f5067120a69

  ambari-server/src/test/python/TestOSCheck.py 9ec2db32fcf3c786ca6f9a34702966efbae49e6f 

Diff: https://reviews.apache.org/r/29983/diff/


Testing
-------

mvn clean test

ambari-server setup (--silent, --verbose) ->start -> ui ok
ambari-server reset ->start -> ui ok
ambari-server upgrade ->start -> ui ok


Thanks,

Florian Barca


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message