Skip to content

Fix: Show AF2 login hint on start for standalone mode - #2192

Merged
pritt20 merged 1 commit into
mainfrom
fix_linux_af2_standalone_messaging
Jul 1, 2026
Merged

Fix: Show AF2 login hint on start for standalone mode#2192
pritt20 merged 1 commit into
mainfrom
fix_linux_af2_standalone_messaging

Conversation

@pritt20

@pritt20 pritt20 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Description

  • Fixes a UX gap on Linux/WSL where astro dev start (standalone mode) reports "Airflow is ready!" but never tells the user how to log in. AF2 uses FAB auth, and on Linux upstream airflow standalone generates a random admin password into $AIRFLOW_HOME/standalone_admin_password.txt — users hit the login screen, guess admin/admin, and get "Invalid login."
  • Adds printLoginHint() to the standalone startup output. Prints:
    • AF2 macOSLogin: admin / admin (matches the Darwin shim's seeded credentials)
    • AF2 Linux/WSLLogin: username admin, password in .astro/standalone/standalone_admin_password.txt
    • AF3 → nothing (Simple Auth Manager accepts any username, no hint needed)
  • Wired into both startForeground and startBackground paths so the hint appears whether the user runs with or without -f.

🎟 Issue(s)

Related #XXX

🧪 Functional Testing

Tested locally on both MacOS and Linux:

  • Linux
astro dev start --standalone
Note: Standalone mode is experimental. Report issues at https://github.com/astronomer/astro-cli/issues

✔ Environment ready

Starting Airflow in standalone mode…

✔ Airflow is ready! (PID 90711)
➤ Airflow UI: http://af2.localhost:6563/
➤ View logs: astro dev logs -f
➤ Stop:      astro dev stop
➤ Login:      username admin, password in .astro/standalone/standalone_admin_password.txt
  • MacOS
astro dev start --standalone
Note: Standalone mode is experimental. Report issues at https://github.com/astronomer/astro-cli/issues

✔ Environment ready

Starting Airflow in standalone mode…

✔ Airflow is ready! (PID 75754)
➤ Airflow UI: http://af2.localhost:6563
➤ View logs: astro dev logs -f
➤ Stop:      astro dev stop
➤ Login:      admin / admin

📸 Screenshots

Add screenshots to illustrate the validity of these changes.

📋 Checklist

  • Rebased from the main (or release if patching) branch (before testing)
  • Ran make test before taking out of draft
  • Ran make lint before taking out of draft
  • Added/updated applicable tests
  • Tested against Astro-API (if necessary).
  • Tested against Houston-API and Astronomer (if necessary).
  • Communicated to/tagged owners of respective clients potentially impacted by these changes.
  • Updated any related documentation

@pritt20
pritt20 requested a review from a team as a code owner July 1, 2026 18:11
@coveralls-official

Copy link
Copy Markdown

Coverage Report for CI Build 28538154178

Coverage increased (+0.007%) to 45.317%

Details

  • Coverage increased (+0.007%) from the base build.
  • Patch coverage: 3 uncovered changes across 1 file (9 of 12 lines covered, 75.0%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
airflow/standalone.go 12 9 75.0%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 55527
Covered Lines: 25163
Line Coverage: 45.32%
Coverage Strength: 8.12 hits per line

💛 - Coveralls

@pritt20
pritt20 merged commit d5f149d into main Jul 1, 2026
7 checks passed
@pritt20
pritt20 deleted the fix_linux_af2_standalone_messaging branch July 1, 2026 18:22
pritt20 added a commit that referenced this pull request Jul 2, 2026
Co-authored-by: Pritesh Arora <pritt@Priteshs-MacBook-Pro.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants