You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/04/29 08:44:04 UTC

[GitHub] [airflow] ephraimbuddy opened a new pull request, #23355: Fix `breeze --backend mssql`

ephraimbuddy opened a new pull request, #23355:
URL: https://github.com/apache/airflow/pull/23355

   `breeze --backend mssql` is not working and this fixes it
   
   


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


[GitHub] [airflow] jedcunningham commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
jedcunningham commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113419291

   Should we symlink it then instead?


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


[GitHub] [airflow] ephraimbuddy commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
ephraimbuddy commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113117228

   Another option would be to have `breeze --backend mssql` use `backend-mssql-bullseye.yml` as the file is the same as this `backend-mssql.yml`


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


[GitHub] [airflow] potiuk commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113548699

   Better fix here: https://github.com/apache/airflow/pull/23368


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


[GitHub] [airflow] potiuk commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113553791

   The problem was that we neede to differentiate between buster and bullseye MSSQL clients as they have slightly different support for SSL and I had to be creative and introduce different compose files for those two.
   
   The change in #23368 adds back the capability to choose different compose for mssql based on the debian version used.


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


[GitHub] [airflow] potiuk commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113501377

   They actually were different - let me fix it really quickly.


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


[GitHub] [airflow] jedcunningham commented on pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
jedcunningham commented on PR #23355:
URL: https://github.com/apache/airflow/pull/23355#issuecomment-1113676658

   Closing this as #23368 has been merged.


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


[GitHub] [airflow] jedcunningham closed pull request #23355: Fix `breeze --backend mssql`

Posted by GitBox <gi...@apache.org>.
jedcunningham closed pull request #23355: Fix `breeze --backend mssql`
URL: https://github.com/apache/airflow/pull/23355


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