vandonr-amz opened a new pull request, #31191:
URL: https://github.com/apache/airflow/pull/31191
I added some code earlier to get a description of the pod when k8s system
tests are failing (#30134)
I thought I had tested this code, but apparently not well enough, because
there was 2 problems:
1. the way the bash command was written was incorrect, the arguments were
passed to `/bin/bash` instead of the command it was running (so it was failing
on the first command because it didn't "see" the `--reinstalled` and returned
an error saying aws cli was already installed)
2. name of the pod, needed to be able to describe it, is only pushed to
xcom at the end of the `execute`, so if there is an error in there, it's not
pushed, and we don't know what to describe.
I fixed the bash by using `;` instead of `&&` to chain commands, and moved
the xcom parameter setting higher in the operator.
Also, while I was at it, I realized it was a bit stupid to force reinstall
each time just to have no error, and replaced that with a `|| true` to just
ignore the error if there is one.
--
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]
