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.

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

Storage integration test failing in nightly since Dec. 21 #4265

Closed
bmccutchon opened this issue Dec 27, 2018 · 5 comments · Fixed by #4936
Closed

Storage integration test failing in nightly since Dec. 21 #4265

bmccutchon opened this issue Dec 27, 2018 · 5 comments · Fixed by #4936
Assignees
Labels
api: storage Issues related to the Cloud Storage API. type: process A process-related concern. May include testing, release, or the like.

Comments

@sduskis sduskis added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: storage Issues related to the Cloud Storage API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Dec 27, 2018
@JustinBeckwith JustinBeckwith added the triage me I really want to be triaged. label Dec 28, 2018
@JustinBeckwith JustinBeckwith removed the triage me I really want to be triaged. label Dec 28, 2018
@ajaaym
Copy link
Contributor

ajaaym commented Dec 28, 2018

Fixed in 4266

@ajaaym ajaaym closed this as completed Dec 28, 2018
@frankyn frankyn reopened this Jan 11, 2019
@frankyn
Copy link
Member

frankyn commented Jan 11, 2019

Hi all,

This is a bug in the GCS service and one the TL's is now aware of it.

I'm reopening this issue as it didn't resolve the underlying issue raised.

Internal bug: 122739863

@andrey-qlogic
Copy link

andrey-qlogic commented Jan 14, 2019

@frankyn
Nightly link says one test is failed
testRotateFromCustomerEncryptionToKmsKeyWithCustomerEncryption

why does it runs?

https://source.cloud.google.com/results/invocations/fd413181-7414-4e8e-9d62-13cfd8fe7a7d/targets/cloud-devrel%2Fclient-libraries%2Fjava%2Fgoogle-cloud-java%2Fnightly%2Fstorage-it/log

which has to be is ignored

public void testRotateFromCustomerEncryptionToKmsKeyWithCustomerEncryption() {

@frankyn
Copy link
Member

frankyn commented Jan 14, 2019

Apologies for the confusion @andrey-qlogic. I'm following up with the GCS team to fix this validation issue. The test breaking found an issue where the service doesn't validate when both CSEK and CMEK are provided in a rewrite request.

I reopened this issue to track that this is still an open bug. I'll close this when the GCS service no longer has this bug and the test is no longer ignored.

@andrey-qlogic
Copy link

@frankyn , thanks for the explanation.

@JustinBeckwith JustinBeckwith added the 🚨 This issue needs some love. label Jan 22, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Jan 22, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Feb 1, 2019
@sduskis sduskis added type: process A process-related concern. May include testing, release, or the like. and removed 🚨 This issue needs some love. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Feb 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the Cloud Storage API. type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
8 participants