Skip to content

Remove work-queue storage configuration #66985

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

Merged
pelikhan merged 4 commits into from
Oct 8, 2026
Merged

Remove work-queue storage configuration #66985

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7f43e9a
Remove work-queue storage configuration
pelikhan Oct 8, 2026
da3444e
Merge origin/main into pelikhan-remove-work-queue-storage
pelikhan Oct 8, 2026
706ab56
Address work-queue storage removal review feedback
pelikhan Oct 8, 2026
d138185
Remove legacy work-queue storage checks entirely
pelikhan Oct 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge origin/main into pelikhan-remove-work-queue-storage 
Preserve upstream work-queue debug logging while keeping the storage selector removed.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
  • Loading branch information
@pelikhan
pelikhan and Copilot committed Oct 8, 2026
commit da3444e3860785bffe3965d7bbedfda5241e1a35
3 changes: 3 additions & 0 deletions actions/setup/js/work_queue_store.cjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ async function readRef(githubClient, owner, repo, branch) {

/** @param {QueueReadOptions} options */
async function readWorkQueueLog({ githubClient, owner, repo, branch = WORK_QUEUE_BRANCH, core: coreApi = undefined }) {
Comment thread
pelikhan marked this conversation as resolved.
Resolved
log.debug("ledger.read.start");
validateBranch(branch);
const repository = await verifyRepository(githubClient, owner, repo);
const sha = await readRef(githubClient, owner, repo, branch);
Expand Down Expand Up @@ -240,6 +241,7 @@ async function publishWorkQueueRequest({
sleepFn = delay => new Promise(resolve => setTimeout(resolve, delay)),
core: coreApi = undefined,
}) {
log.debug("request.publish.start", { max_retries: maxRetries, initialize_only: initializeOnly });
validateBranch(branch);
integer(maxRetries, 0, 10, "publication retries");
validateTrustedContext(context, actor);
Expand All @@ -254,6 +256,7 @@ async function publishWorkQueueRequest({
let observedPrefix = [];
let lastConflict;
for (let attempt = 0; attempt <= maxRetries; attempt++) {
log.debug("request.publish.attempt", { attempt: attempt + 1 });
let current = await readWorkQueueLog({ githubClient, owner, repo, branch, core: coreApi });
validateExtendingPrefix(observedPrefix, current.transactions, "refreshed queue history does not extend the previously validated prefix");
observedPrefix = current.transactions;
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.