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
> 
>