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 2021/09/01 09:40:43 UTC
[GitHub] [airflow] potiuk opened a new pull request #17883: Avoid race condition when setting experimental docker flag
potiuk opened a new pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-909169603
It were bad - downgraded constraints (the downgrade should be fixed by #17939 )
--
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] uranusjr closed pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
uranusjr closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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] mrbaguvix commented on a change in pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
mrbaguvix commented on a change in pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#discussion_r702332249
##########
File path: scripts/ci/images/ci_prepare_prod_image_on_ci.sh
##########
@@ -29,10 +29,13 @@ export VERBOSE="true"
# Builds or waits for the PROD image in the CI environment
function build_prod_images_on_ci() {
+ build_images::configure_docker_registry
build_images::prepare_prod_build
if [[ ${GITHUB_REGISTRY_WAIT_FOR_IMAGE} == "true" ]]; then
- build_images::wait_for_image_tag "${AIRFLOW_PROD_IMAGE}" ":${GITHUB_REGISTRY_PULL_IMAGE_TAG}"
+ local image_name_with_tag="${AIRFLOW_PROD_IMAGE}:${GITHUB_REGISTRY_PULL_IMAGE_TAG}"
+ push_pull_remove_images::wait_for_image "${image_name_with_tag}"
+ push_pull_remove_images::pull_image "${image_name_with_tag}"
Review comment:
Good stuff!
--
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] uranusjr closed pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
uranusjr closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-909169603
It were bad - downgraded constraints (the downgrade should be fixed by #17939 )
--
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 #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-923007468
All Green. This will remove another source of random CI failures.
--
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 pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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] uranusjr closed pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
uranusjr closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 pull request #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 pull request #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-922262917
I removed completely the "experimental" flag @uranusjr - this only caused troubles and I actually need to pull the image anyway (for verification) once it is ready - so rather than checking if image is there via `manifest` I am pulling it straight away now - this will be cleaner, less experimental and simpler (and it will work better I hope - sometimes the config.json got irreparably broken.
--
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 #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-922382916
Just MSSQL tests wich has to be fixed anywyay. But it's good
--
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 #17883: Avoids race condition on parallel builds
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-907662897
One more fix to accidentally failing tests . Such a :facepalm: on me.
--
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 #17883: Avoid race condition when setting experimental docker flag
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-909169603
It were bad - downgraded constraints (the downgrade should be fixed by #17939 )
--
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 merged pull request #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk merged pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 pull request #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk closed pull request #17883:
URL: https://github.com/apache/airflow/pull/17883
--
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 #17883: Stop using docker manifest to check for image presence
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #17883:
URL: https://github.com/apache/airflow/pull/17883#issuecomment-922345567
Rebuilding to see if the failures are transient.
--
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