Add fullname to owner on logging

[arjunanan6]

closes: #29662

This change adds the owner's full name in brackets next to the owner username.

[arjunanan6]

@potiuk Seems like 1 test out of 40 fails, but I cannot see which. Would you be able to help?

[malthe]

This looks great, but it would be nice to have a means of providing a formatting function in $AIRFLOW_HOME/webserver_config.py.

[uranusjr]

Instead of changing the meaning of user, it may be better to add a new field for the full name.

[arjunanan6]

@uranusjr That's also a solid option. But, would that not require changes to the log table to add in a new column referencing the full name?

[uranusjr]

Yes it does. Adding a column (with NULL default) is quite cheap, so the question is whether we want to do it. Speaking of that, why do we want the full name here in the first place? Maybe it is easier to change the UI to JOIN the user table instead?

[arjunanan6]

@uranusjr There is a screenshot in my linked issue, the reason is to just make it clear in the Audit logs as to who has done what. At least in our case, it just references the azure ID which doesn't say anything.

Could you give me a hint on where to introduce the JOIN? I could take a look at it then.

[arjunanan6]

@uranusjr Actually, figured out the addition of a new field to the log table, I was just overthinking it. I'll put some more into it and return :)

This is what it looks like:

[arjunanan6]

Hi @uranusjr, I've commited the suggested change. Is it good to go?

[uranusjr]

Doc building failure is unrelated.