potiuk commented on pull request #12511:
URL: https://github.com/apache/airflow/pull/12511#issuecomment-731763675


   There is a bug in the code - result of one of the latest refactors (and main 
reason why I opened a support ticket to GitHub). IT's extremely difficult to 
see such bugs in GA when you have a typo in name of an output because this is 
silently ignored:
   
   The "Set upgrade to latest constraints" is not executed because the output 
has earlier version of the job name,
   `steps.cancel.outputs.sourceEvent ` should be 
`steps.source-run-info.outputs.sourceEvent` similarly as in the previous step!
   
   In this code:
   
   ```
         - name: "Set upgrade to latest constraints"
           id: upgrade-constraints
           run: |
             if [[ ${{ steps.cancel.outputs.sourceEvent == 'push' ||
                 steps.cancel.outputs.sourceEvent == 'scheduled' }} == 'true' 
]]; then
                 echo "::set-output name=upgradeToLatestConstraints::${{ 
github.sha }}"
             else
                 echo "::set-output name=upgradeToLatestConstraints::false"
             fi
   ```


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

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


Reply via email to