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

Fix Flakey Test - AspNetCore.IntegrationTests.TraceTest.Trace_QPS #966

Closed
iantalarico opened this issue Apr 4, 2017 · 1 comment
Closed

Fix Flakey Test - AspNetCore.IntegrationTests.TraceTest.Trace_QPS #966

iantalarico opened this issue Apr 4, 2017 · 1 comment
Assignees
@iantalarico
Labels
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.

Comments

@iantalarico
Copy link
Contributor

This test is continually flakey.
@iantalarico iantalarico self-assigned this Apr 4, 2017
iantalarico added a commit to iantalarico/google-cloud-dotnet that referenced this issue Apr 4, 2017
@iantalarico
Decrease the QPS allowed for the Trace QPS test.
3134db9 
This is in an attempt to fix a flake where both requests get traced.

Fixes issue googleapis#966
@iantalarico iantalarico mentioned this issue Apr 4, 2017
Merged
iantalarico added a commit that referenced this issue Apr 4, 2017
@iantalarico
Decrease the QPS allowed for the Trace QPS test. (#967)
b4eb645 
This is in an attempt to fix a flake where both requests get traced.

Fixes issue #966
iantalarico added a commit to iantalarico/google-cloud-dotnet that referenced this issue Apr 12, 2017
@iantalarico
Disable AspNetCore.IntegrationTests.TraceTest.Trace_QPS.
ed7f037 
The test is about 50% flaky on our CI but passes almost 100% locally.  It is causing a lot of noise in the CI.  Skip until it is fixed.

See issue googleapis#966
@iantalarico iantalarico mentioned this issue Apr 12, 2017
Merged
iantalarico added a commit that referenced this issue Apr 12, 2017
@iantalarico
Disable qps test (#986)
eca56e1 
* Disable AspNetCore.IntegrationTests.TraceTest.Trace_QPS.

The test is about 50% flaky on our CI but passes almost 100% locally.  It is causing a lot of noise in the CI.  Skip until it is fixed.

See issue #966

* Fix typo
@iantalarico iantalarico mentioned this issue Jun 13, 2017
Merged
@jskeet jskeet added 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 Jun 21, 2017
@jskeet
Copy link
Collaborator

jskeet commented Jun 21, 2017

I'll have a look and see what our test weather is like now...

gcf-owl-bot bot added a commit that referenced this issue Dec 10, 2024
@gcf-owl-bot
feat: Update Compute Engine API to revision 20241201 (#966)
d7402aa 
Source-Link: googleapis/googleapis@5d5b1bf

Source-Link: googleapis/googleapis-gen@c735b61
Copy-Tag: eyJwIjoiYXBpcy9Hb29nbGUuQ2xvdWQuQ29tcHV0ZS5WMS8uT3dsQm90LnlhbWwiLCJoIjoiYzczNWI2MTkzZGFlYmFhOWM0NWMyZTIzMTUzYWYxNmFlZTczMmZlNiJ9
gcf-owl-bot bot added a commit that referenced this issue Dec 10, 2024
@gcf-owl-bot
feat: Update Compute Engine API to revision 20241201 (#966)
bf272aa 
Source-Link: googleapis/googleapis@5d5b1bf

Source-Link: googleapis/googleapis-gen@c735b61
Copy-Tag: eyJwIjoiYXBpcy9Hb29nbGUuQ2xvdWQuQ29tcHV0ZS5WMS8uT3dsQm90LnlhbWwiLCJoIjoiYzczNWI2MTkzZGFlYmFhOWM0NWMyZTIzMTUzYWYxNmFlZTczMmZlNiJ9
gcf-owl-bot bot added a commit that referenced this issue Dec 10, 2024
@gcf-owl-bot
feat: Update Compute Engine API to revision 20241201 (#966)
bdcd4b5 
Source-Link: googleapis/googleapis@5d5b1bf

Source-Link: googleapis/googleapis-gen@c735b61
Copy-Tag: eyJwIjoiYXBpcy9Hb29nbGUuQ2xvdWQuQ29tcHV0ZS5WMS8uT3dsQm90LnlhbWwiLCJoIjoiYzczNWI2MTkzZGFlYmFhOWM0NWMyZTIzMTUzYWYxNmFlZTczMmZlNiJ9
amanda-tarafa pushed a commit that referenced this issue Dec 10, 2024
@gcf-owl-bot @amanda-tarafa
feat: Update Compute Engine API to revision 20241201 (#966)
7239f50 
Source-Link: googleapis/googleapis@5d5b1bf

Source-Link: googleapis/googleapis-gen@c735b61
Copy-Tag: eyJwIjoiYXBpcy9Hb29nbGUuQ2xvdWQuQ29tcHV0ZS5WMS8uT3dsQm90LnlhbWwiLCJoIjoiYzczNWI2MTkzZGFlYmFhOWM0NWMyZTIzMTUzYWYxNmFlZTczMmZlNiJ9
@jskeet jskeet mentioned this issue Dec 12, 2024
Merged
amanda-tarafa pushed a commit that referenced this issue Dec 12, 2024
@jskeet @amanda-tarafa
Release Google.Cloud.Compute.V1 version 3.2.0
e8016f4 
Changes in this release:

### New features

- Update Compute Engine API to revision 20241201 ([issue 966](#966)) ([commit 7239f50](7239f50))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@iantalarico iantalarico

Labels
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.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jskeet @iantalarico