Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Commit

Permalink
process all expired nodes rather than those not already marked for de…
Browse files Browse the repository at this point in the history
…letion (#1103)

This makes sure debug_keep_node is reset and the rest of the reimage processing occurs regardless of reimage_requested and delete_requested being set.

Without this, nodes that are marked `debug_keep_node` do not get reimaged/deleted.
  • Loading branch information
bmc-msft authored Jul 27, 2021
1 parent 0e27256 commit 9ec7e7a
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/api-service/__app__/onefuzzlib/workers/nodes.py
Original file line number Diff line number Diff line change
Expand Up @@ -456,12 +456,9 @@ def reimage_long_lived_nodes(cls, scaleset_id: UUID) -> None:
time_filter = "Timestamp lt datetime'%s'" % (
(datetime.datetime.utcnow() - NODE_REIMAGE_TIME).isoformat()
)
# skip any nodes already marked for reimage/deletion
for node in cls.search(
query={
"scaleset_id": [scaleset_id],
"reimage_requested": [False],
"delete_requested": [False],
},
raw_unchecked_filter=time_filter,
):
Expand Down

0 comments on commit 9ec7e7a

Please sign in to comment.