amoghrajesh commented on PR #51738:
URL: https://github.com/apache/airflow/pull/51738#issuecomment-2979024394
@jscheffl valid points, thanks.
In a happy scenario it should never fail yes, and no, i didnt see it failing
on mac either.
I was just being careful since we are touching the API layers here and would
not want airflow running on a badly configured setup to break things.
For example, i ran a python:3.11 container with a random uid: `99999` and
that's not defined and could lead to a failure like so:
```
(airflow) ➜ airflow git:(fix_helm_chart_release_guide) ✗ docker run --rm
-it --user 99999:99999 python:3.11 bash
I have no name!@cbe5b8d5aeb3:/$ python3 -c "import getpass;
print(getpass.getuser())"
Traceback (most recent call last):
File "<string>", line 1, in <module>
File "/usr/local/lib/python3.11/getpass.py", line 169, in getuser
return pwd.getpwuid(os.getuid())[0]
^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'getpwuid(): uid not found: 99999'
```
Point being -- adding a try / except would do no harm, but I'll let you take
a call on that
--
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]
