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

go_binary: fix pgoprofile default #24

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Conversation

rickystewart
Copy link

This fixes bazel-contrib#4226.

@rickystewart rickystewart requested a review from rail January 14, 2025 16:51
@rickystewart rickystewart merged commit 59db5f5 into crl-release-0.50 Jan 14, 2025
rickystewart added a commit to rickystewart/cockroach that referenced this pull request Jan 14, 2025
To pull in cockroachdb/rules_go#24

Epic: CRDB-41952
Release note: None
rickystewart added a commit to rickystewart/cockroach that referenced this pull request Jan 14, 2025
To pull in cockroachdb/rules_go#24

Epic: CRDB-41952
Release note: None
craig bot pushed a commit to cockroachdb/cockroach that referenced this pull request Jan 14, 2025
138873: cloudtestutils: refactor CheckExportStore, CheckListFiles, CheckNoPermissions r=RaduBerinde a=RaduBerinde

This PR is taking a first step towards removing `ExternalIODir` from `cluster.Settings`.

#### cloudtestutils: refactor CheckExportStore

This change adds a `cloudtestutils.StoreInfo` structure and refactors
`CheckExportStore` to use it, making the call sites simpler and more
readable. We now pass the external IO dir explicitly through
`StoreInfo` instead of getting it from the settings.

Epic: none
Release note: None

#### cloudtestutils: refactor CheckListFiles

Refactor `CheckListFiles` and `CheckListFilesCanonical` to use
`StoreInfo`.

Epic: none
Release note: None

#### cloudtestutils: refactor CheckNoPermissions

Use `StoreInfo` for `CheckNoPermissions.`

Epic: none
Release note: None


139044: build: update `rules_go` r=jlinder a=rickystewart

To pull in cockroachdb/rules_go#24

Epic: CRDB-41952
Release note: None

Co-authored-by: Radu Berinde <[email protected]>
Co-authored-by: Ricky Stewart <[email protected]>
InManuBytes pushed a commit to InManuBytes/cockroach that referenced this pull request Jan 15, 2025
To pull in cockroachdb/rules_go#24

Epic: CRDB-41952
Release note: None
mohini-crl pushed a commit to mohini-crl/cockroach that referenced this pull request Jan 17, 2025
To pull in cockroachdb/rules_go#24

Epic: CRDB-41952
Release note: None
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.

2 participants