Skip to content

Publish cockroach-sql in publish bleeding edge#79796

Merged
craig[bot] merged 2 commits intocockroachdb:masterfrom
rail:cockroachsql-pbe
Apr 11, 2022
Merged

Publish cockroach-sql in publish bleeding edge#79796
craig[bot] merged 2 commits intocockroachdb:masterfrom
rail:cockroachsql-pbe

Conversation

@rail
Copy link
Copy Markdown
Member

@rail rail commented Apr 11, 2022

Previously, the cockroach-sql tool was published only as a part of the
release workflow.

The first commit refactors the way how we publish and test. This is very similar to what we use in publish-provisional-artifacts.

The second commit adds the cockroach-sql binary to the list of artifacts to publish.

Release note: None

rail added 2 commits April 11, 2022 15:40
Previously, the tests in publish-artifacts were marked as "slow",
because they used actual compilation.

This patch changes the logic and uses the same approach we use in
publish-provisional-artifacts.

Release note: None
Previously, the `cockroach-sql` tool was published only as a part of the
release workflow.

This patch enables publishing of the `cockroach-sql` binary as a part of
the Publish Bleeding Edge build configuration.

Release note: None
@rail rail added C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-dev-inf A-ci Continuous Integration labels Apr 11, 2022
@rail rail requested a review from rickystewart April 11, 2022 19:50
@rail rail self-assigned this Apr 11, 2022
@rail rail requested a review from a team as a code owner April 11, 2022 19:50
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@rail
Copy link
Copy Markdown
Member Author

rail commented Apr 11, 2022

bors r=rickystewart

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 11, 2022

Build succeeded:

@craig craig bot merged commit 6de7a18 into cockroachdb:master Apr 11, 2022
@rail rail deleted the cockroachsql-pbe branch March 14, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci Continuous Integration C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-dev-inf

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants