storage: TestIntegration_ACL failed #6579
Assignees
Labels
api: storage
Issues related to the Cloud Storage API.
flakybot: flaky
Tells the Flaky Bot not to close or comment on this issue.
flakybot: issue
An issue filed by the Flaky Bot. Should not be added manually.
priority: p2
Moderately-important priority. Fix may not be included in next release.
type: bug
Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Comments
flaky-bot
bot
added
flakybot: issue
product-auto-label
bot
added
the
api: storage
flaky-bot
bot
added
the
flakybot: flaky
|
Looks like this issue is flaky. 😟 I'm going to leave this open and stop commenting. A human should fix and close this. When run at the same commit (c220ca2), this test passed in one build (Build Status, Sponge) and failed in another build (Build Status, Sponge). |
|
Oops! Looks like this issue is still flaky. It failed again. 😬 I reopened the issue, but a human will need to close it again. commit: 9f3c334 Test output integration_test.go:2377: hasRule: object ACL missing {Entity:domain-google.com EntityID: Role:READER Domain:google.com Email: ProjectTeam:}
integration_test.go:2381: object ACL: could not delete entity domain-google.com |
BenWhitehead
added
priority: p2
tritone
added a commit
to tritone/google-cloud-go
that referenced
this issue
Sep 6, 2022
Previously the docs said that these operations would take effect within 10s, but I noticed that at some point this was loosened up to "at least 30s" and "about a minute" depending on the location. I think these tests will still usually run in a similar amount of time, but hopefully extending this timeout will avoid some flakes. https://cloud.google.com/storage/docs/consistency#eventually_consistent_operations https://cloud.google.com/storage/docs/access-control/lists#modification-rules (see note above) Fixes googleapis#6579
tritone
added a commit
that referenced
this issue
Sep 7, 2022
Previously the docs said that these operations would take effect within 10s, but I noticed that at some point this was loosened up to "at least 30s" and "about a minute" depending on the location. I think these tests will still usually run in a similar amount of time, but hopefully extending this timeout will avoid some flakes. https://cloud.google.com/storage/docs/consistency#eventually_consistent_operations https://cloud.google.com/storage/docs/access-control/lists#modification-rules (see note above) Fixes #6579
|
Oops! Looks like this issue is still flaky. It failed again. 😬 I reopened the issue, but a human will need to close it again. commit: 5733a42 Test output integration_test.go:2377: hasRule: object ACL missing {Entity:domain-google.com EntityID: Role:READER Domain:google.com Email: ProjectTeam:}
integration_test.go:2381: object ACL: could not delete entity domain-google.com |
|
Latest failure was from a commit before my fix was merged. Therefore I'm going to close this again. |
|
Oops! Looks like this issue is still flaky. It failed again. 😬 I reopened the issue, but a human will need to close it again. commit: 1ad1aba Test output integration_test.go:2377: hasRule: object ACL missing {Entity:domain-google.com EntityID: Role:READER Domain:google.com Email: ProjectTeam:}
integration_test.go:2381: object ACL: could not delete entity domain-google.com |
tritone
added a commit
to tritone/google-cloud-go
that referenced
this issue
Sep 9, 2022
I realized that this section had been slightly messed up in the integration test refactor for retries; we need the object creation to be retried as well if the check fails. Also removed superfluous unused object name. Fixes googleapis#6579
tritone
added a commit
that referenced
this issue
Sep 9, 2022
I realized that this section had been slightly messed up in the integration test refactor for retries; we need the object creation to be retried as well if the check fails. Also removed superfluous unused object name. Fixes #6579
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Note: #6379 was also for this test, but it was closed more than 10 days ago. So, I didn't mark it flaky.
commit: c220ca2
buildURL: Build Status, Sponge
status: failed
Test output
integration_test.go:2373: hasRule: object ACL missing {Entity:domain-google.com EntityID: Role:READER Domain:google.com Email: ProjectTeam:} integration_test.go:2377: object ACL: could not delete entity domain-google.comThe text was updated successfully, but these errors were encountered: