Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-12 Thread Pradeep Agrawal
> On April 8, 2022, 2:12 p.m., Pradeep Agrawal wrote: > > Currently python2 and python3 both are supported, why we want to remove > > python2 support ? Also What issues are you getting when you execute all > > these script in python3 ? > > Abhishek Kumar wrote: > Article explaining the ne

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-11 Thread Kirby Zhou
> On 四月 8, 2022, 2:12 p.m., Pradeep Agrawal wrote: > > Currently python2 and python3 both are supported, why we want to remove > > python2 support ? Also What issues are you getting when you execute all > > these script in python3 ? > > Abhishek Kumar wrote: > Article explaining the need

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-10 Thread Pradeep Agrawal
> On April 8, 2022, 2:12 p.m., Pradeep Agrawal wrote: > > Currently python2 and python3 both are supported, why we want to remove > > python2 support ? Also What issues are you getting when you execute all > > these script in python3 ? > > Abhishek Kumar wrote: > Article explaining the ne

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-08 Thread Abhishek Kumar
> On April 7, 2022, 10:14 p.m., Ramesh Mani wrote: > > security-admin/scripts/change_password_util.py > > Lines 1 (patched) > > > > > > Deleting file changepasswordutil.py and creating with a new file name > > change_

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-08 Thread Abhishek Kumar
> On April 8, 2022, 2:12 p.m., Pradeep Agrawal wrote: > > Currently python2 and python3 both are supported, why we want to remove > > python2 support ? Also What issues are you getting when you execute all > > these script in python3 ? Article explaining the need to remove python2: https://ww

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-08 Thread Pradeep Agrawal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73937/#review224282 --- Currently python2 and python3 both are supported, why we want to r

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-07 Thread Kirby Zhou
> On 四月 7, 2022, 10:14 p.m., Ramesh Mani wrote: > > security-admin/scripts/change_password_util.py > > Lines 1 (patched) > > > > > > Deleting file changepasswordutil.py and creating with a new file name > > change_pas

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-07 Thread Kirby Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73937/#review224266 --- "#!/usr/bin/python" should be replace to "#!/usr/bin/env python3

Re: Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-07 Thread Ramesh Mani
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73937/#review224265 --- security-admin/scripts/change_password_util.py Lines 1 (patched)

Review Request 73937: RANGER-3697: Python3 migration of all ranger python scripts

2022-04-07 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73937/ --- Review request for ranger, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Sailaj