Skip to content

Conversation

@jgraham
Copy link
Member

@jgraham jgraham commented Jan 20, 2026

Checklist for reviewer:

  • Commits should reference a bug or github issue, if relevant (if a bug is
    referenced, the pull request should include the bug number in the title)
  • Scan the PR and verify that no changes (particularly to
    .circleci/config.yml) will cause environment variables (particularly
    credentials) to be exposed in test logs
  • Ensure the container image will be using permissions granted to
    telemetry-airflow
    responsibly.

Note for deployments: In order to push images built by this PR, the user who merges the PR
must be in the telemetry Github team.
This is because deploys depend on the
data-eng-airflow-gcr CircleCI context.
See DENG-8850 for additional discussion.

@jgraham jgraham requested a review from ksy36 January 20, 2026 12:00
@jgraham jgraham requested a review from a team as a code owner January 20, 2026 12:00
@ksy36 ksy36 merged commit 7b86b2c into main Jan 21, 2026
3 checks passed
@ksy36 ksy36 deleted the crux_202512 branch January 21, 2026 17:30
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.

3 participants