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/19 14:18:28 UTC
[GitHub] [airflow] potiuk opened a new issue, #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
potiuk opened a new issue, #23084:
URL: https://github.com/apache/airflow/issues/23084
There is the whole "kind" set of commands that should be implemented in the new Breeze.
There are few sub-commands there that can be implemented gradually.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] Bowrna commented on issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
Bowrna commented on issue #23084:
URL: https://github.com/apache/airflow/issues/23084#issuecomment-1117529669
yes i will pick this up :) thanks @potiuk
--
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] Bowrna commented on issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
Bowrna commented on issue #23084:
URL: https://github.com/apache/airflow/issues/23084#issuecomment-1117124632
@potiuk I am new to k8s / helm chart. Do you think I can pick up this task or Do I need to understand more about Helm to pick this task?
--
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 issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #23084:
URL: https://github.com/apache/airflow/issues/23084#issuecomment-1117372359
I assigned it to you just in case :)
--
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 issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #23084:
URL: https://github.com/apache/airflow/issues/23084#issuecomment-1117372001
Not too much. It's more about converting bash parallelism to Python one in this case. It's mostly to rewrite what's being currently run in https://github.com/apache/airflow/blob/main/.github/workflows/ci.yml#L1477 - eventually it's just running a sequence of commands and making sure they are "ok" :)
it does have a good deal of parallelism involved - we are running tests in parallel in self-hosted runners so there is chance to learn a bit about that. I have alreeady impleemented some paralellism and it is WAY simpler in Python than it was in Bash (the bash way is almost incomprehensible even for me :D and Python parallelism how I implemented it is smpler - but it still handles adjusting parallelism level to the number of CPUs available etc.).
You can see how i implemented "pull" in parallel.
Start here: https://github.com/apache/airflow/blob/673f8940786b33663a5eb47047af2d7c2a14df99/dev/breeze/src/airflow_breeze/commands/ci_image_tools.py#L248
I decided that everything that we run evenually in CI will have a corresponding `breeze` command - this makes everyone's live easier, so in this case we need to have a
`breeze run-helm-upgrade-tests` command with some options that will allow it to run in paralllel.
--
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 closed issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
potiuk closed issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
URL: https://github.com/apache/airflow/issues/23084
--
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 issue #23084: Breeze: Add running Helm Install/Upgrade tests to Breeze
Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #23084:
URL: https://github.com/apache/airflow/issues/23084#issuecomment-1235922107
This is done now with #25678
--
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