Skip to content

Fix CI issues - #1018

Merged
foodprocessor merged 39 commits into
mainfrom
fix-ci-issues
Sep 10, 2026
Merged

Fix CI issues#1018
foodprocessor merged 39 commits into
mainfrom
fix-ci-issues

Conversation

@foodprocessor

@foodprocessor foodprocessor commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Mostly AI changes aimed at improving CI reliability

foodprocessor and others added 15 commits August 21, 2026 17:43
@foodprocessor
Display capacity cannot exceed bucket size
424c9dd
@foodprocessor
Don't let log output errors pass as successful writes, since this cou…
0c0a935 
…ld cascade into false rotations, destroying the log.
@foodprocessor
drop stale TODO
2846975
@foodprocessor
Stop pending op uploads on any cloud error.
904320b
@foodprocessor
back off on failures when uploading pending ops
950076d
@foodprocessor
Add failed uploads to pendingOps
9dd1530
@foodprocessor
Fix race condition in upload test
933c2fa
@foodprocessor
Fix windows permissions (#1017)
e252013 
* Windows does not have a compatible permissions system, so don't let file permissions be set
* Set SDDL in a way that makes sense to explorer (add an owner and group)
* Include code to repair permissions on existing file cache files on Windows systems ("rogue read-only")
* Update copilot instructions
@foodprocessor
Make updateObject return whether to continue uploads, rather than its…
4c65bfe 
… error
@foodprocessor
Fix transient I/O test failures
d6af6ca
@foodprocessor
Improve GetRelease CI reliability
920b01f
@foodprocessor
Use a mock test server to avoid GH rate limits and network issues in CI
f50fcc0
@foodprocessor
Run a weekly API test to follow any changes
261f0fe
@foodprocessor
Set permissions
d470898
@foodprocessor
Lint
e15588f
jfantinhardesty
jfantinhardesty approved these changes Sep 10, 2026
View reviewed changes

@jfantinhardesty jfantinhardesty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, besides some minor things

Comment thread component/azstorage/block_blob_test.go Outdated
Comment thread cmd/root.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jfantinhardesty jfantinhardesty jfantinhardesty approved these changes
@Dabnsky Dabnsky Awaiting requested review from Dabnsky Dabnsky is a code owner
@Ferelith-maker Ferelith-maker Awaiting requested review from Ferelith-maker Ferelith-maker is a code owner
@brayan-trejo brayan-trejo Awaiting requested review from brayan-trejo brayan-trejo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@foodprocessor @jfantinhardesty