Skip to content

Conversation

@jason810496
Copy link
Member

related: #52929 (comment)

Why

We need to backport #52929 but it depends on #52860

What

Backport #52860

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI is failing.

jason810496 and others added 2 commits July 18, 2025 21:09
* Remove gunicorn daemonize for api-server

* Rename _CommonCLIUvicornTestClass

* Add test for api-server with daemon option

* Fix setproctitle on MacOS

* Replace api/access_logfile with api/log_config

* Fix [api/log_config] cli_config

* Remove logging and unsed option in api_server_command

- the logging before daemonization is not logging the correct PID so
  remove it
- should_setup_logging is default as False

* Fix test_run_command_daemon

* Add significant rst

(cherry picked from commit ab1ee72)
@jason810496 jason810496 force-pushed the backport-ab1ee72-v3-0-test branch from 1321e05 to b2fb97c Compare July 18, 2025 13:10
@potiuk
Copy link
Member

potiuk commented Jul 19, 2025

Yeah - some doc issues.

@jason810496
Copy link
Member Author

Just fixed the CI by adding [api/log_config] to deprecated_options.

f6a7b1f

@potiuk potiuk merged commit 7212c88 into apache:v3-0-test Jul 20, 2025
54 checks passed
jedcunningham added a commit to astronomer/airflow that referenced this pull request Jul 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants