Skip to content

Cleanup multi-team docs for team_name tagging - #70146

Merged
o-nikolas merged 2 commits into
apache:mainfrom
Tegh25:feature/multi-team_metrics_doc_cleanup
Jul 28, 2026
Merged

Cleanup multi-team docs for team_name tagging#70146
o-nikolas merged 2 commits into
apache:mainfrom
Tegh25:feature/multi-team_metrics_doc_cleanup

Conversation

@Tegh25

@Tegh25 Tegh25 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

As discussed in PR #69943, the unrelated doc changes are being moved to their own branch + PR instead of being attached to the other changes.

Relevant issue: #68294


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@Tegh25

Tegh25 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@ferruzzi tagging you for visibility. Please review when you get the chance 🙂

@ferruzzi

Copy link
Copy Markdown
Contributor

Ah, I just meant to move out the docs that weren't directly related to that PR. There were two that you updated because you saw they were wrong while you were in there.

The connection tests bullet and the scheduler.executor_heartbeat_duration / triggers.blocked_main_thread additions to the existing triggerer/executor bullets were directly related to that PR and should have stayed.

@Tegh25
Tegh25 force-pushed the feature/multi-team_metrics_doc_cleanup branch 2 times, most recently from 59d29b9 to 469357d Compare July 20, 2026 22:41
@Tegh25

Tegh25 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback. I amended the commit, should be correct now.

@ferruzzi ferruzzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved pending CI passing

Tegh25 added 2 commits July 27, 2026 12:12
As discussed in PR apache#69943, these doc changes are being moved to
their own branch + PR instead of being attached to the other changes.
@Tegh25
Tegh25 force-pushed the feature/multi-team_metrics_doc_cleanup branch from 469357d to a41044e Compare July 27, 2026 17:20
@Tegh25

Tegh25 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

Completed a rebase since #69943 was merged. This PR is ready for review.

@o-nikolas
o-nikolas merged commit f5d7c8a into apache:main Jul 28, 2026
70 checks passed
@Tegh25
Tegh25 deleted the feature/multi-team_metrics_doc_cleanup branch July 28, 2026 19:19
dabla pushed a commit to dabla/airflow that referenced this pull request Aug 14, 2026
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