potiuk commented on PR #35813:
URL: https://github.com/apache/airflow/pull/35813#issuecomment-1826801193

   The part that fixes the problem you had is here:
   
   
https://github.com/apache/airflow/pull/35862/files#diff-6ef5b756e45d03d3723ba4a5c4b0f1417b115fe25560f293351c30290ec67b31R127
   
   ```
       """Starts building attempt. Returns false if we should not continue"""
       result = run_command(
           ["docker", "inspect", ci_image_params.airflow_image_name_with_tag],
           capture_output=True,
           text=True,
           check=False,
       )
       if result.returncode != 0:
           return True
       if ci_image_params.skip_image_upgrade_check:
           return False
       if not ci_image_params.force_build and not 
ci_image_params.upgrade_to_newer_dependencies:
           if not should_we_run_the_build(build_ci_params=ci_image_params):
               return False
       return True
   ```
   
   But it's done together with renaming of e "image_check" property in the 
object - that's why it's easier to keep it together with the rename.
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to