> On Dec. 17, 2017, 2:14 p.m., Jonathan Hurley wrote:
> > ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> > Lines 155 (patched)
> > <https://reviews.apache.org/r/64667/diff/1/?file=1919066#file1919066line156>
> >
> > OK - let's just make this a little less hard-coded ...
> >
> > Can we pass in the paramters of livy/livy-server to this method and
> > call it something like "fix_default_links"
> >
> > We should also have some documentation as to why we need this method ...
>
> Jonathan Hurley wrote:
> Also, i think livy2 also had this
>
> Dmitro Lisnichenko wrote:
> This fix will not work for Livy2, because it is also missing from
> stack_packages.json
>
> Dmitro Lisnichenko wrote:
> also, conf-select has not idea about Livy2
>
> /usr/bin/conf-select dry-run-create --package livy2 --stack-version
> 2.6.3.0-235 --conf-version 0
> livy2 not installed or incorrect package name
Sure it does:
```
[root@c6403 livy]# grep livy2 `which conf-select`
"tez", "livy", "livy2", "zookeeper", "zeppelin",
```
stack_packages.json:
```
"livy2": [
{
"conf_dir": "/etc/livy2/conf",
"current_dir": "{0}/current/livy2-client/conf"
}
],
```
- Jonathan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64667/#review194033
-----------------------------------------------------------
On Dec. 16, 2017, 10:53 a.m., Dmitro Lisnichenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64667/
> -----------------------------------------------------------
>
> (Updated Dec. 16, 2017, 10:53 a.m.)
>
>
> Review request for Ambari, Jonathan Hurley, Mahadev Konar, and Nate Cole.
>
>
> Bugs: AMBARI-22522
> https://issues.apache.org/jira/browse/AMBARI-22522
>
>
> Repository: ambari
>
>
> Description
> -------
>
> Adding a small, livy-only workaround into install_packages.py
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/custom_actions/scripts/install_packages.py
> c8497cd
>
>
> Diff: https://reviews.apache.org/r/64667/diff/1/
>
>
> Testing
> -------
>
> mvn clean test,
> live cluster check
>
>
> Thanks,
>
> Dmitro Lisnichenko
>
>
