storage: TestIntegration_PredefinedACLs failed

[flaky-bot]

Note: #7653 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.

---

commit: fbfaf21
buildURL: Build Status, Sponge
status: failed

Test output

    integration_test.go:4028: Object.ACL: expected acl to contain: {entity:allAuthenticatedUsers role:READER}, got acl: [{Entity:allUsers EntityID: Role:READER Domain: Email: ProjectTeam:} {Entity:user-kokoro@dulcet-port-762.iam.gserviceaccount.com EntityID: Role:OWNER Domain: Email:kokoro@dulcet-port-762.iam.gserviceaccount.com ProjectTeam:}]

[flaky-bot]

Test passed for commit 05a1484 (Build Status, Sponge)! Closing this issue.

[flaky-bot]

Looks like this issue is flaky. 😟

I'm going to leave this open and stop commenting.

A human should fix and close this.

---

commit: 1a928ae
buildURL: Build Status, Sponge
status: failed

Test output

    integration_test.go:4028: Object.ACL: expected acl to contain: {entity:allAuthenticatedUsers role:READER}, got acl: [{Entity:allUsers EntityID: Role:READER Domain: Email: ProjectTeam:} {Entity:user-kokoro@dulcet-port-762.iam.gserviceaccount.com EntityID: Role:OWNER Domain: Email:kokoro@dulcet-port-762.iam.gserviceaccount.com ProjectTeam:}]

[BrennaEpp]

This is for the second Object.ACL check in the test (currently L4370). It's already wrapped in a retry loop; this needs some investigation.

[BrennaEpp]

So the object is created with w.PredefinedACL = "authenticatedRead" set. It seems that instead of allAuthenticatedUsers , allUsers was set in the acl. The retry loop seems fine.

This seems unlikely to be an issue in the client library. Since this is quite old, I will close this for now, will see if it comes up again.