Skip to content

reftracker: return snapshot from AppsForRef/RefsForApp to fix concurrent-map panic (#1812)#1820

Closed
SAY-5 wants to merge 2 commits into
carvel-dev:developfrom
SAY-5:say5/reftracker-snapshot-1812-r2
Closed

reftracker: return snapshot from AppsForRef/RefsForApp to fix concurrent-map panic (#1812)#1820
SAY-5 wants to merge 2 commits into
carvel-dev:developfrom
SAY-5:say5/reftracker-snapshot-1812-r2

Conversation

@SAY-5

@SAY-5 SAY-5 commented May 3, 2026

Copy link
Copy Markdown

Refiles a previously-closed PR (#1817) that was closed in error after the force-push. Same fix; new branch.

Fixes #1812.

@carvel-bot carvel-bot added this to Carvel May 3, 2026
@SAY-5 SAY-5 force-pushed the say5/reftracker-snapshot-1812-r2 branch from abae973 to 16648c8 Compare May 27, 2026 04:34
SAY-5 added 2 commits June 2, 2026 14:04
…ent-map panic (carvel-dev#1812)

Signed-off-by: SAY-5 <saiasish.cnp@gmail.com>
Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@SAY-5 SAY-5 force-pushed the say5/reftracker-snapshot-1812-r2 branch from 16648c8 to e278c53 Compare June 2, 2026 21:05
@SAY-5

SAY-5 commented Jun 9, 2026

Copy link
Copy Markdown
Author

Consolidating into #1825, same fix with the PR template filled out.

@SAY-5 SAY-5 closed this Jun 9, 2026
@github-project-automation github-project-automation Bot moved this to Closed in Carvel Jun 9, 2026
@github-actions github-actions Bot added the carvel-triage This issue has not yet been reviewed for validity label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

carvel-triage This issue has not yet been reviewed for validity

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

Race Condition in AppRefTracker: Concurrent Map Iteration and Write

2 participants