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/07/14 23:21:54 UTC

[GitHub] [airflow] jedcunningham opened a new pull request, #25077: Update `check_files.py` to support many provider releases

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

   This allows you to put many provider releases in `packages.txt` and
   check them all in one go. In fact, you can copy/paste the list of pypi
   urls from the vote email!


-- 
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 #25077: Update `check_files.py` to support many provider releases

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

   Nice. But it does not solve everything. You need to verify signatures and checksums and those are not in PyPI - only in SVN :) 


-- 
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 #25077: Update `check_files.py` to support many provider releases

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

   True, but those steps work in bulk as-is.
   
   Checking that all the files are present required you to do them one by one, or transform the list of urls into something like `echo {name} > packages.txt && python check_packages.py -t providers -v {version} -p {path}`, and that gets annoying quickly. This is the one step that I dreaded when voting on providers and now it'll be easy 🙂.


-- 
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 #25077: Update `check_files.py` to support many provider releases

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

   I see. Yeah. I forgot that I (I think never) run the verification of providers myself, so I do not know what it involves :)


-- 
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 merged pull request #25077: Update `check_files.py` to support many provider releases

Posted by GitBox <gi...@apache.org>.
jedcunningham merged PR #25077:
URL: https://github.com/apache/airflow/pull/25077


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