Conversation
It was erroring with "unbound" variable, expecting INTEGRATION to be a pre-defined variable.
|
one lowercase too much |
|
Hmm . but this is strange. It shoudl work actually, was it 'undefined' because there was no actual integration after -i ? |
potiuk
left a comment
potiuk
left a comment
There was a problem hiding this comment.
Hmm. It should work let me check it.
|
Changing it definately made it work and behave as expected. |
| -i | --integration) | ||
| local integration=${2} | ||
| local INTEGRATION=${2} | ||
| parameters::check_and_save_allowed_param "INTEGRATION" "integration" "--integration" |
There was a problem hiding this comment.
| parameters::check_and_save_allowed_param "INTEGRATION" "integration" "--integration" | |
| parameters::check_and_save_allowed_param "integration" "integration" "--integration" |
There was a problem hiding this comment.
I thought I tired this, and it then couldn't find the list of allowed values.
I will double check
|
See my suggestion - this was the actual problem -> the "check_and_save_allowed_param" continued checking INTEGRATION variable instead if "integration" |
True, |
potiuk
added
the
type:misc/internal
3 participants
It was erroring with "unbound" variable, expecting INTEGRATION to be a
pre-defined variable.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.