Skip to content

Commit

Permalink
[7820] Attempt to fix potential issue with tab screenshots not being …
Browse files Browse the repository at this point in the history
…cleaned up properly. Needs some additional investigation. (#17438) (#17442)

(cherry picked from commit 385859f)

Co-authored-by: mattreaganmozilla <[email protected]>
  • Loading branch information
mergify[bot] and mattreaganmozilla authored Nov 27, 2023
1 parent cb79404 commit f9374f9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Storage/DiskImageStore.swift
Original file line number Diff line number Diff line change
Expand Up @@ -86,13 +86,13 @@ public actor DefaultDiskImageStore: DiskImageStore {
}

public func clearAllScreenshotsExcluding(_ keys: Set<String>) async throws {
let keysToDelete = keys.subtracting(keys)
let keysToDelete = self.keys.subtracting(keys)

for key in keysToDelete {
let url = URL(fileURLWithPath: filesDir).appendingPathComponent(key)
try FileManager.default.removeItem(at: url)
}
self.keys = keys.intersection(keys)
self.keys = keys
}

public func deleteImageForKey(_ key: String) async {
Expand Down

1 comment on commit f9374f9

@firefoxci-taskcluster
Copy link

Choose a reason for hiding this comment

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

Uh oh! Looks like an error! Details

InterpreterError at template.tasks[0].extra[0].treeherder[1].symbol: unknown context value cron

Please sign in to comment.