vine: correctly assign to cache_level when file is reused (#4075)

btovar · web-flow · commit a27e4c5c4c89 · 2025-02-26T12:47:55.000-05:00
diff --git a/taskvine/src/manager/vine_manager_put.c b/taskvine/src/manager/vine_manager_put.c
@@ -427,7 +427,7 @@ static vine_result_code_t vine_manager_put_input_file_if_needed(struct vine_mana
 	if (replica) {
 		if (f->cache_level < VINE_CACHE_LEVEL_WORKFLOW) {
 			debug(D_VINE, "File %s is not marked as a cachable file, but it is used by more than one task. Marking as cachable.", f->source);
-			f->flags |= VINE_CACHE_LEVEL_WORKFLOW;
+			f->cache_level = VINE_CACHE_LEVEL_WORKFLOW;
 		}
 
 		return VINE_SUCCESS;

Original file line number	Diff line number	Diff line change
`@@ -427,7 +427,7 @@ static vine_result_code_t vine_manager_put_input_file_if_needed(struct vine_mana`
`427`	`427`	`if (replica) {`
`428`	`428`	`if (f->cache_level < VINE_CACHE_LEVEL_WORKFLOW) {`
`429`	`429`	`debug(D_VINE, "File %s is not marked as a cachable file, but it is used by more than one task. Marking as cachable.", f->source);`
`430`		`- f->flags \|= VINE_CACHE_LEVEL_WORKFLOW;`
	`430`	`+ f->cache_level = VINE_CACHE_LEVEL_WORKFLOW;`
`431`	`431`	`}`
`432`	`432`
`433`	`433`	`return VINE_SUCCESS;`