----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36171/#review90501 -----------------------------------------------------------
ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py (line 257) <https://reviews.apache.org/r/36171/#comment143616> Can you add repository_version param to the pydoc? ambari-server/src/main/resources/custom_actions/scripts/install_packages.py (line 183) <https://reviews.apache.org/r/36171/#comment143612> Can you add a comment that it's important to use the normalized version here that doesn't contain the build number? If self.repository_version does contain the build number, it will have to be removed. E.g., first call is for 2.3.0.0-1234 (wrong build number, which actually installs 2.3.0.0-5678) Then we should save to the history file, 2.3.0.0,2.3.0.0-5678 This can then update the DB record. On the next call, the repository_version will contain the correct build number, but the first element in the history file will not. ambari-server/src/main/resources/custom_actions/scripts/install_packages.py (line 242) <https://reviews.apache.org/r/36171/#comment143614> I believe we should always call self.compute_actual_version, even when an exception is thrown above. We should remove this else block and underindent it. - Alejandro Fernandez On July 6, 2015, 5:06 p.m., Dmitro Lisnichenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36171/ > ----------------------------------------------------------- > > (Updated July 6, 2015, 5:06 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Nate > Cole. > > > Bugs: AMBARI-12278 > https://issues.apache.org/jira/browse/AMBARI-12278 > > > Repository: ambari > > > Description > ------- > > 2015-07-01 08:07:10,323 - Failure while computing actual version. Error: > Could not determine actual version. This stack's root directory (/usr/hdp) > exists but was not able to determine the actual repository version installed. > Try reinstalling packages again. > Traceback (most recent call last): > File > "/var/lib/ambari-agent/cache/custom_actions/scripts/install_packages.py", > line 299, in install_packages > self.compute_actual_version() > File > "/var/lib/ambari-agent/cache/custom_actions/scripts/install_packages.py", > line 254, in compute_actual_version > raise Fail(msg) > Fail: Could not determine actual version. This stack's root directory > (/usr/hdp) exists but was not able to determine the actual repository version > installed. Try reinstalling packages again. > Traceback (most recent call last): > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/hdp_select.py > 5fdc35d > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py > b028852 > ambari-server/src/test/python/custom_actions/TestInstallPackages.py 1e6dcea > > Diff: https://reviews.apache.org/r/36171/diff/ > > > Testing > ------- > > Performed E2E tests > > > OK > ---------------------------------------------------------------------- > Total run:796 > Total errors:0 > Total failures:0 > OK > > > Thanks, > > Dmitro Lisnichenko > >
