Skip to content

chore: Relax the GKE WIF principal:// restriction#36

Merged
davidtorres merged 2 commits intogoogleapis:mainfrom
benjamin-maynard:relax-gke-wif-restriction
Jul 14, 2025
Merged

chore: Relax the GKE WIF principal:// restriction#36
davidtorres merged 2 commits intogoogleapis:mainfrom
benjamin-maynard:relax-gke-wif-restriction

Conversation

@benjamin-maynard
Copy link
Member

Submit once cl/749106195 is in prod.

Relax GKE WIF limitation. b/385138184.

Running tool: /usr/local/go/bin/go test -timeout 30s -run ^(TestNewMechanismWithTokenSource|TestMechanism_Name|TestMechanism_Start)$ github.com/googleapis/managedkafka/sasl-plain-access-token/segmentio/saslplainoauthmechanism

=== RUN   TestNewMechanismWithTokenSource
=== RUN   TestNewMechanismWithTokenSource/Valid_Token_and_Email
--- PASS: TestNewMechanismWithTokenSource/Valid_Token_and_Email (0.00s)
=== RUN   TestNewMechanismWithTokenSource/Valid_Token,_Bad_Email
--- PASS: TestNewMechanismWithTokenSource/Valid_Token,_Bad_Email (0.00s)
=== RUN   TestNewMechanismWithTokenSource/Token_Error
--- PASS: TestNewMechanismWithTokenSource/Token_Error (0.00s)
--- PASS: TestNewMechanismWithTokenSource (0.00s)
=== RUN   TestMechanism_Name
--- PASS: TestMechanism_Name (0.00s)
=== RUN   TestMechanism_Start
=== RUN   TestMechanism_Start/Valid_Token_and_Email
--- PASS: TestMechanism_Start/Valid_Token_and_Email (0.00s)
--- PASS: TestMechanism_Start (0.00s)
PASS
ok      github.com/googleapis/managedkafka/sasl-plain-access-token/segmentio/saslplainoauthmechanism    0.496s

@benjamin-maynard benjamin-maynard requested a review from a team April 19, 2025 09:58
@benjamin-maynard benjamin-maynard force-pushed the relax-gke-wif-restriction branch from 1314c58 to ec81054 Compare April 19, 2025 10:10
@davidtorres davidtorres changed the title Relax the GKE WIF principal:// restriction chore: Relax the GKE WIF principal:// restriction Jul 14, 2025
@davidtorres davidtorres merged commit 88e81d7 into googleapis:main Jul 14, 2025
3 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.

2 participants