Skip to content

chore(asf.yaml): route notifications to Airflow PMC commits@ / jobs@; retire dev-magpie@#282

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:chore/asf-notifications-airflow-pmc
May 25, 2026
Merged

chore(asf.yaml): route notifications to Airflow PMC commits@ / jobs@; retire dev-magpie@#282
potiuk merged 1 commit into
apache:mainfrom
potiuk:chore/asf-notifications-airflow-pmc

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented May 25, 2026

Summary

Re-points all nine GitHub-event notification schemes in .asf.yaml
from the dedicated dev-magpie@airflow.apache.org sub-list back to
the Airflow PMC's own lists (commits@airflow.apache.org for
everything except CI runs, which go to jobs@airflow.apache.org).

Scheme Before After
jobs dev-magpie@ jobs@airflow.apache.org
commits dev-magpie@ commits@airflow.apache.org
issues dev-magpie@ commits@airflow.apache.org
issues_status dev-magpie@ commits@airflow.apache.org
issues_comment dev-magpie@ commits@airflow.apache.org
pullrequests dev-magpie@ commits@airflow.apache.org
pullrequests_status dev-magpie@ commits@airflow.apache.org
pullrequests_comment dev-magpie@ commits@airflow.apache.org
discussions dev-magpie@ commits@airflow.apache.org

Why

Traffic volume on the Airflow PMC's lists is already strong and
noisy enough that adding this repo's events does not materially
change the signal/noise profile. The cost of subscribers having to
follow a separate sub-list to see what was happening here turned
out to outweigh the value of keeping that traffic sequestered.

When the framework eventually moves to its own TLP it will inherit
its own commits@ list and the schemes will be re-pointed at that
list in the same change.

Where this differs from Airflow's own .asf.yaml

Airflow only sets the five "main" schemes (jobs, commits,
issues, pullrequests, discussions) explicitly and lets the
four _status / _comment schemes default to
dev@airflow.apache.org.

This config sets all nine explicitly to commits@ so that PR
review comments, CI-status flips, and label / milestone changes
land on the same list as lifecycle events instead of polluting
Airflow's dev@. The framework's "everything on one list"
principle is preserved; only the list changes.

commits_by_path remains the only intentionally-unset scheme (an
optional path-specific override, not a default-suppression risk).

Cleanup

No cross-references to dev-magpie@ remain in the repo (grepped
across all .md / .yaml / .yml; only .asf.yaml referenced
it). Whether the dev-magpie@ list itself stays open or is
retired on the Infra side is a separate request.

Test plan

  • prek run --files .asf.yaml — yaml hooks pass (typos /
    trailing-whitespace / line-ending / private-key).
  • Post-merge: confirm the next push to main lands on
    commits@airflow.apache.org and the next CI run lands on
    jobs@airflow.apache.org.
  • Post-merge: confirm dev@airflow.apache.org receives no
    notifications from this repo (the all-explicit routing should
    suppress the ASF Infra default for every scheme).

…-magpie@ sequester

The framework's commit / PR / issue / discussion traffic now lands
on the Airflow PMC's own `commits@` and `jobs@` lists rather than
the dedicated `dev-magpie@airflow.apache.org` sub-list set up in
apache#271.

Rationale: traffic volume on the Airflow PMC's lists is already
strong and noisy enough that adding this repo's events does not
materially change the signal/noise profile. The cost of subscribers
having to follow a separate sub-list to see what was happening
here turned out to outweigh the value of keeping that traffic
sequestered. When the framework eventually moves to its own TLP it
will inherit its own `commits@` list and the schemes will be
re-pointed at that list in the same change.

Routing follows Airflow's `.asf.yaml` destinations:
- `jobs` -> `jobs@airflow.apache.org` (CI run notifications)
- everything else -> `commits@airflow.apache.org`

The four `_status` / `_comment` schemes (`issues_status`,
`issues_comment`, `pullrequests_status`, `pullrequests_comment`)
are set explicitly to `commits@` here rather than left at the ASF
Infra dev@ default, so PR review comments, CI-status flips, and
label / milestone changes land on the same list as the lifecycle
events instead of polluting `dev@airflow.apache.org`. This is
where the routing differs from Airflow's own asf.yaml: Airflow
accepts those events on `dev@`; the framework keeps them on
`commits@` to consolidate.

`commits_by_path` remains the only intentionally-unset scheme (it
is an optional path-specific override, not a default-suppression
risk). If ASF Infra adds a new scheme, set it explicitly in the
same change to keep the `dev@` default suppressed.

No cross-references to `dev-magpie@` remain in the repo (grepped
across all markdown / yaml; only `.asf.yaml` referenced it).

Generated-by: Claude Code (Opus 4.7)
@potiuk potiuk merged commit 57e75b4 into apache:main May 25, 2026
13 checks passed
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.

1 participant