[GitHub] [airflow] kiwy42 commented on pull request #17342: From feature request #17314: add support to enable kinit options [-f|-F] and [-a|-A]

[GitBox <gi...@apache.org>]

kiwy42 commented on pull request #17342:
URL: https://github.com/apache/airflow/pull/17342#issuecomment-889981595


   > 
   > 
   > If all the tests pass, I think the change should be safe - but this might be a good opportunity to live the world a little better after the change than it was before :).
   > 
   > Happy to guide you. I think we do not have (historically) a lot of tests about kerberos, but this might be a nice opportunity to add some unit tests to "renew_kt" function. Also it might be a nice opportunity to review and improve kerberos integration documentation - which might need some polishing (or even adding).
   > 
   > Would you like to try ?
   > 
   > It is a bit involved and requires to learn about mocking (but I can point you to a few examples that you can learn from).
   
   Amazing reply, at the moment I'm in the rush to finish my airflow integration at work and can work with my custom builded version, when things get smoother I'd be delighted to help in incorporating a bit more than just that.  
   Howerver cleary I'm not a very good Python writer (in fact I really do not like Python as a langage) and not really confortable with continus integration  and unit testing in Python. So I have a lot to work on before being able to improve kerberos integration.  
   I'll considering helping anyway.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org