> On Sept. 15, 2016, 2:49 a.m., Jonathan Hurley wrote: > > I think we can do better. Why not just use `os.path.exists` to check the > > `current_dir` structure. In the case of atlas, the psuedo code woudl read: > > > > if "/etc/atlas/conf" is a directory and if "/usr/hdp/current/atlas-client" > > is a valid link > > then seed stuff > > > > Basically we just want to seed IFF both the `conf_dir` is a phyiscal > > directory and the `current_dir` is valid (indicates installed)
The seeding happens when we install the new HDP 2.5 bits and before Atlas RPM is installed, so /etc/atlas/conf will not exist and /usr/hdp/current/atlas-client will not be a valid link (still points to /usr/hdp/2.4/atlas-client which DNE) . Line 361 is needed because we cannot create symlink /etc/atlas/conf -> /usr/hdp/current/atlas-client/conf since the source will be needed once the Atlas RPM is installed. Line 618 is needed because Atlas RPM brings in artifacts not known to Ambari that are first stored in /etc/atlas/conf, which then gets copied to /etc/atlas/conf.backup because we will need to make /etc/atlas/conf be a symlink to /usr/hdp/current/atlas-client/conf - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51903/#review149019 ----------------------------------------------------------- On Sept. 15, 2016, 1:57 a.m., Alejandro Fernandez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51903/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2016, 1:57 a.m.) > > > Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan > Hurley, and Nate Cole. > > > Bugs: AMBARI-18368 > https://issues.apache.org/jira/browse/AMBARI-18368 > > > Repository: ambari > > > Description > ------- > > Steps to Reproduce: > > * Install Ambari 2.2.2 with HDP 2.4 with HBase, Solr, and Hive (this is > important) > * Perform EU/RU to HDP 2.5 > * Add Atlas Service > > Atlas Server log contains, > > Caused by: > org.apache.solr.client.solrj.impl.HttpSolrClient$RemoteSolrException: Error > from server at > http://natu146-ehbs-dgm10toeriesec-u14-1.openstacklocal:8886/solr: Can not > find the specified config set: vertex_index > > Fix: > The Hive RPM installs /usr/$stack/$version/atlas with some partial packages > that contain Hive hooks, while the Atlas RPM is responsible for installing > the full content. > If the user does not have Atlas currently installed on their stack, then > /usr/$stack/current/atlas-client will be a broken symlink, and we should not > create the symlink /etc/atlas/conf -> /usr/$stack/current/atlas-client/conf . > If we mistakenly create this symlink, then when the user performs an EU/RU > and then adds Atlas service then the Atlas RPM will not be able to copy its > artifacts into /etc/atlas/conf directory and therefore prevent Ambari from by > copying those unmanaged contents into /etc/atlas/$version/0 > > Further, when installing Atlas service, we must copy the artifacts from > /etc/atlas/conf.backup/* to /etc/atlas/conf (which is now a symlink to > /usr/hdp/current/atlas-client/conf/) with the no-clobber flag. > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/conf_select.py > c60b324 > > ambari-server/src/test/python/stacks/2.0.6/hooks/after-INSTALL/test_after_install.py > 06a366e > > Diff: https://reviews.apache.org/r/51903/diff/ > > > Testing > ------- > > ---------------------------------------------------------------------- > Total run:1125 > Total errors:0 > Total failures:0 > OK > > > Need to perform more tests on a live cluster. > > > Thanks, > > Alejandro Fernandez > >
