You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Vishal Suvagia via Review Board <no...@reviews.apache.org> on 2022/10/10 10:02:24 UTC
Re: Review Request 74127: Ranger-3880 : Ranger setup fails for newer MySQL versions
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/74127/#review224771
-----------------------------------------------------------
kms/scripts/dba_script.py
Lines 167 (patched)
<https://reviews.apache.org/r/74127/#comment313571>
Justus, Thank-you for the fix, the issue is reported for 2.3 version, but the fix is already in place for 2.3 version.
kms : https://github.com/apache/ranger/blob/ranger-2.3/kms/scripts/db_setup.py#L132-L134
can you kindly share which branch is the fix being applied to, do update the review request to reflect the same ?
security-admin/scripts/dba_script.py
Lines 194 (patched)
<https://reviews.apache.org/r/74127/#comment313572>
Same for this fix, it is already available in the 2.3 version:
security-admin : https://github.com/apache/ranger/blob/ranger-2.3/security-admin/scripts/db_setup.py#L850-L852
- Vishal Suvagia
On Oct. 6, 2022, 8:24 a.m., Justus wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/74127/
> -----------------------------------------------------------
>
> (Updated Oct. 6, 2022, 8:24 a.m.)
>
>
> Review request for ranger, Madhan Neethiraj and Selvamohan Neethiraj.
>
>
> Bugs: RANGER-3880
> https://issues.apache.org/jira/browse/RANGER-3880
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Ranger Setup fails when using newer MySQL which requires to set useSSL to false explicitly
>
>
> Diffs
> -----
>
> kms/scripts/dba_script.py 544c1201b
> security-admin/scripts/dba_script.py 0ba396944
>
>
> Diff: https://reviews.apache.org/r/74127/diff/1/
>
>
> Testing
> -------
>
> The setup works fine.
>
>
> Thanks,
>
> Justus
>
>