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

ci: artifacts of FIPS-enabled roachtests should be separate #100679

Merged
merged 1 commit into from
Apr 5, 2023

Conversation

srosenberg
Copy link
Member

When running FIPS-enabled roachtests, we want to separate their artifacts from other roachtests. This removes the confounding factor for performance tests wherein FIPS may result in performance degradation.

Epic: none

Release note: None

@srosenberg srosenberg requested a review from a team as a code owner April 5, 2023 02:49
@srosenberg srosenberg requested a review from rail April 5, 2023 02:49
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@srosenberg srosenberg removed the request for review from a team April 5, 2023 02:49
When running FIPS-enabled roachtests, we want to separate their artifacts
from other roachtests. This removes the confounding factor for performance
tests wherein FIPS may result in performance degradation.

Epic: none

Release note: None
@srosenberg srosenberg force-pushed the sr/ci_fips_artifacts branch from b027651 to a874f1e Compare April 5, 2023 02:59
@srosenberg srosenberg added branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 backport-23.1.x Flags PRs that need to be backported to 23.1 and removed branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 labels Apr 5, 2023
@srosenberg
Copy link
Member Author

TFTR!

bors r=rail

@craig
Copy link
Contributor

craig bot commented Apr 5, 2023

Build succeeded:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants