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

tests: roachtest that creates overload when doing intent resolution #108916

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

sumeerbhola
Copy link
Collaborator

Informs #97108

Epic: CRDB-25458

Release note: None

@sumeerbhola sumeerbhola requested review from a team as code owners August 17, 2023 16:35
@sumeerbhola sumeerbhola requested review from herkolategan and DarrylWong and removed request for a team August 17, 2023 16:35
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@sumeerbhola
Copy link
Collaborator Author

ignore the first commit -- it is from #108873

@sumeerbhola
Copy link
Collaborator Author

L0 sublevels goes over the threshold due to the lack of AC
Screenshot 2023-08-17 at 12 40 21 PM

Timeout: time.Hour,
Benchmark: true,
Tags: registry.Tags(`weekly`),
// Second node is solely for Prometheus.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to BTW - even running prometheus on the same node, or not at all, is ok. These runs happen on GCE and we now have that global prometheus instance that knows how to scrape every endpoint.

Owner: registry.OwnerAdmissionControl,
Timeout: time.Hour,
Benchmark: true,
Tags: registry.Tags(`weekly`),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run nightly for a while to shake out flakes before reducing to weekly frequency?

Copy link
Collaborator Author

@sumeerbhola sumeerbhola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TFTR!

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @DarrylWong, @herkolategan, and @irfansharif)


pkg/cmd/roachtest/tests/admission_control_intent_resolution.go line 39 at r2 (raw file):

Previously, irfansharif (irfan sharif) wrote…

Run nightly for a while to shake out flakes before reducing to weekly frequency?

Done


pkg/cmd/roachtest/tests/admission_control_intent_resolution.go line 40 at r2 (raw file):

Previously, irfansharif (irfan sharif) wrote…

We don't need to BTW - even running prometheus on the same node, or not at all, is ok. These runs happen on GCE and we now have that global prometheus instance that knows how to scrape every endpoint.

I added this only because someone could do --cloud aws. I've left this unchanged.

@sumeerbhola
Copy link
Collaborator Author

bors r=irfansharif

@craig
Copy link
Contributor

craig bot commented Aug 18, 2023

Build succeeded:

@craig craig bot merged commit f3c92a0 into cockroachdb:master Aug 18, 2023
@sumeerbhola sumeerbhola deleted the intent_roachtest branch September 5, 2023 13:41
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.

3 participants