Skip to content

Commit

Permalink
Set defaults overidden by TC to reduce churn.
Browse files Browse the repository at this point in the history
  • Loading branch information
jschwartzentruber committed Oct 30, 2023
1 parent e508d17 commit 378f017
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -329,6 +329,7 @@ def build_resources(
# give workers 15 minutes to register before assuming they're broken
"registrationTimeout": parse_time("15m"),
"reregistrationTimeout": parse_time("4d"),
"queueInactivityTimeout": parse_time("2h"),
}

# Build the decision task payload that will trigger the new fuzzing tasks
Expand Down Expand Up @@ -516,6 +517,7 @@ def build_resources(
# give workers 15 minutes to register before assuming they're broken
"registrationTimeout": parse_time("15m"),
"reregistrationTimeout": parse_time("4d"),
"queueInactivityTimeout": parse_time("2h"),
}

# Build the decision task payload that will trigger the new fuzzing tasks
Expand Down
7 changes: 6 additions & 1 deletion services/fuzzing-decision/tests/test_pool.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,7 @@ def test_aws_resources(
config["launchConfig"]["InstanceMarketOptions"] = {"MarketType": "spot"}
if platform == "linux":
expected["config"]["lifecycle"] = {
"queueInactivityTimeout": 7200,
"registrationTimeout": 900,
"reregistrationTimeout": 345600,
}
Expand Down Expand Up @@ -403,7 +404,11 @@ def test_gcp_resources(
"zone": "us-west1-a",
},
],
"lifecycle": {"registrationTimeout": 900, "reregistrationTimeout": 345600},
"lifecycle": {
"queueInactivityTimeout": 7200,
"registrationTimeout": 900,
"reregistrationTimeout": 345600,
},
"maxCapacity": 6,
"minCapacity": 0,
},
Expand Down

0 comments on commit 378f017

Please sign in to comment.