dstandish commented on PR #28848: URL: https://github.com/apache/airflow/pull/28848#issuecomment-1379293959
> > Backward compatibility. Previously, KPO did not use airflow conn. If we now use default, this will break things for users. We can do but must be major release. > > So it was using the bult-in "local" kubernetes (basically whatever kubectl sees ?) if None is used? > > Maybe we can check for existence of the connection and fallback to default behaviour when it is not available? What's the likelihood someone will have default k8s connection set and would not want to use it ? (still likely requring major version bump and technically breaking but with far less radius blast) i think quite likely: https://github.com/apache/airflow/blob/6d09fc7b88623919b01cfbdee5566598cefba021/airflow/utils/db.py#L364-L370 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
