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

CI (Buildkite): in the Buildkite cache, don't cache the "registries" directory in the default depot #42475

Merged
merged 1 commit into from
Oct 3, 2021
Merged

CI (Buildkite): in the Buildkite cache, don't cache the "registries" directory in the default depot #42475

merged 1 commit into from
Oct 3, 2021

Conversation

DilumAluthge
Copy link
Member

No description provided.

@DilumAluthge DilumAluthge added ci Continuous integration backport 1.6 Change should be backported to release-1.6 backport 1.7 labels Oct 2, 2021
@DilumAluthge DilumAluthge requested a review from a team as a code owner October 2, 2021 21:50
@DilumAluthge DilumAluthge removed the request for review from a team October 2, 2021 21:51
@DilumAluthge DilumAluthge added the merge me PR is reviewed. Merge when all tests are passing label Oct 2, 2021
@tkf tkf mentioned this pull request Oct 2, 2021
Merged
@DilumAluthge DilumAluthge merged commit be28eb3 into master Oct 3, 2021
@DilumAluthge DilumAluthge deleted the dpa/buildkite-registries-cache branch October 3, 2021 02:15
@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Oct 3, 2021
@staticfloat
Copy link
Member

Was this to fix something? We rarely need a bleeding-edge registry, so this seems like it would just increase the amount of work we do, right?

@staticfloat
Copy link
Member

Ah, right, we do this because we interrupt the process every now and then, and we don't recover from intermediate state very well.

KristofferC pushed a commit that referenced this pull request Oct 5, 2021
@DilumAluthge @KristofferC
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
bd7ec68 
…directory in the default depot (#42475)

(cherry picked from commit be28eb3)
@KristofferC KristofferC mentioned this pull request Oct 5, 2021
Merged
84 tasks
KristofferC pushed a commit that referenced this pull request Oct 19, 2021
@DilumAluthge @KristofferC
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
3a7f6a4 
…directory in the default depot (#42475)

(cherry picked from commit be28eb3)
@KristofferC KristofferC mentioned this pull request Oct 19, 2021
Merged
95 tasks
@KristofferC KristofferC mentioned this pull request Nov 8, 2021
Merged
66 tasks
KristofferC pushed a commit that referenced this pull request Nov 11, 2021
@DilumAluthge @KristofferC
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
3151f26 
…directory in the default depot (#42475)

(cherry picked from commit be28eb3)
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Nov 13, 2021
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
@DilumAluthge @LilithHafner
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
36b2b4d 
…directory in the default depot (JuliaLang#42475)
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
@DilumAluthge @LilithHafner
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
5af68b3 
…directory in the default depot (JuliaLang#42475)
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
@DilumAluthge @staticfloat
CI (Buildkite): in the Buildkite cache, don't cache the "registries" …
007cc7e 
…directory in the default depot (#42475)

(cherry picked from commit be28eb3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@staticfloat staticfloat Awaiting requested review from staticfloat

Assignees
No one assigned
Labels
ci Continuous integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DilumAluthge @staticfloat @KristofferC