Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Revert "AIP-72: Move Secrets Masker to task SDK (#46375)" #46488

Closed
wants to merge 1 commit into from
Closed

Revert "AIP-72: Move Secrets Masker to task SDK (#46375)" #46488

wants to merge 1 commit into from

Conversation

jedcunningham
Copy link
Member

This reverts commit e78b5b4.

Reverts #46375, which is causing k8s intergration test failures.

@jedcunningham jedcunningham added the full tests needed We need to run full set of tests for this PR to merge label Feb 5, 2025
@jedcunningham jedcunningham reopened this Feb 5, 2025
@jedcunningham jedcunningham added the legacy api Whether legacy API changes should be allowed in PR label Feb 5, 2025
@jedcunningham jedcunningham reopened this Feb 5, 2025
@jedcunningham
Copy link
Member Author

Well, this isn't the culprit!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mobuchowski mobuchowski Awaiting requested review from mobuchowski mobuchowski will be requested when the pull request is marked ready for review mobuchowski is a code owner

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy will be requested when the pull request is marked ready for review ephraimbuddy is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun will be requested when the pull request is marked ready for review pierrejeambrun is a code owner

@potiuk potiuk Awaiting requested review from potiuk potiuk will be requested when the pull request is marked ready for review potiuk is a code owner

@ashb ashb Awaiting requested review from ashb ashb will be requested when the pull request is marked ready for review ashb is a code owner

@bolkedebruin bolkedebruin Awaiting requested review from bolkedebruin bolkedebruin will be requested when the pull request is marked ready for review bolkedebruin is a code owner

@XD-DENG XD-DENG Awaiting requested review from XD-DENG XD-DENG will be requested when the pull request is marked ready for review XD-DENG is a code owner

Assignees
No one assigned
Labels
area:API Airflow's REST/HTTP API area:CLI area:dev-tools area:logging area:providers area:secrets area:serialization area:webserver Webserver related Issues full tests needed We need to run full set of tests for this PR to merge kind:documentation legacy api Whether legacy API changes should be allowed in PR provider:amazon AWS/Amazon - related issues provider:cncf-kubernetes Kubernetes provider related issues provider:openlineage AIP-53
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jedcunningham