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/08/17 01:19:10 UTC
[GitHub] [airflow] uranusjr opened a new pull request, #25751: Fix SystemsManagerParameterStoreBackend test
uranusjr opened a new pull request, #25751:
URL: https://github.com/apache/airflow/pull/25751
This has been failing since b9b8841ef38d94edcb05f75d5883e5b37ac07c54, which moved `use_ssl` to a separated attribute instead of bundling it in `kwargs`.
@Taragolis
--
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 #25751: Fix SystemsManagerParameterStoreBackend test
Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #25751:
URL: https://github.com/apache/airflow/pull/25751#issuecomment-1217788162
I looked at why it was not caught in the PR. The problem is that the system manager tests are not run when only providers change. I we should move them to "always" part to catch similar kind of problems.
--
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] eladkal merged pull request #25751: Fix SystemsManagerParameterStoreBackend test
Posted by GitBox <gi...@apache.org>.
eladkal merged PR #25751:
URL: https://github.com/apache/airflow/pull/25751
--
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 #25751: Fix SystemsManagerParameterStoreBackend test
Posted by GitBox <gi...@apache.org>.
potiuk commented on PR #25751:
URL: https://github.com/apache/airflow/pull/25751#issuecomment-1217797991
Fix preventing it from happening in the future in #25760
--
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