astronomer / astro-cli

CLI that makes it easy to create, test and deploy Airflow DAGs to Astronomer
https://www.astronomer.io
Other
334 stars 70 forks source link

Add component Flags to astro deployment logs #1633

Closed ichung08 closed 2 months ago

ichung08 commented 2 months ago

Description

Describe the purpose of this pull request.

Users need to see logs for a specific component (webserver, scheduler, triggerer, worker) of a Deployment. This change will allow users to set specific flags (--webserver) to parse a specific component's flags.

๐ŸŽŸ Issue(s)

Related #1590

๐Ÿงช Functional Testing

List the functional testing steps to confirm this feature or fix.

  • Updated unit tests and added new units
  • CLI Testing:
  • no flags set: Screenshot 2024-04-24 at 2 17 41 PM

๐Ÿ“ธ Screenshots

Add screenshots to illustrate the validity of these changes.

๐Ÿ“‹ Checklist

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 86.74%. Comparing base (1a67fff) to head (d15ea33).

Files Patch % Lines
cloud/deployment/deployment.go 84.61% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1633 +/- ## ========================================== - Coverage 86.75% 86.74% -0.01% ========================================== Files 114 114 Lines 16661 16671 +10 ========================================== + Hits 14454 14462 +8 - Misses 1318 1319 +1 - Partials 889 890 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.