containers: add option to allow all tiers (#10258)

* containers: add option to not set a constraint on tier

* changeset

* Update packages/containers-shared/src/types.ts

Co-authored-by: emily-shen <[email protected]>

* fix diff

* update comment

---------

Co-authored-by: emily-shen <[email protected]>

Author: nikitassharma

.changeset/famous-coats-chew.md

@@ -0,0 +1,6 @@
+---
+"@cloudflare/containers-shared": patch
+"wrangler": patch
+---
+
+Add the option to allow all tiers when creating a container

packages/containers-shared/src/types.ts

@@ -68,7 +68,7 @@ export type SharedContainerConfig = {
 	constraints: {
 		regions?: string[];
 		cities?: string[];
-		tier: number;
+		tier: number | undefined;
 	};
 	observability: { logs_enabled: boolean };
 } & InstanceTypeOrLimits;

packages/wrangler/src/__tests__/containers/config.test.ts

@@ -542,4 +542,33 @@ describe("getNormalizedContainerOptions", () => {
 		expect(result[0]).toHaveProperty("image_build_context");
 		expect(result[0]).not.toHaveProperty("image_uri");
 	});
+
+	it("should be able to specify all tiers", async () => {
+		writeFileSync("Dockerfile", "FROM scratch");
+		const config: Config = {
+			name: "test-worker",
+			containers: [
+				{
+					class_name: "TestContainer",
+					image: `${getCloudflareContainerRegistry()}/test:latest`,
+					name: "test-container",
+					constraints: {
+						tier: -1,
+					},
+				},
+			],
+			durable_objects: {
+				bindings: [
+					{
+						name: "TEST_DO",
+						class_name: "TestContainer",
+					},
+				],
+			},
+		} as Partial<Config> as Config;
+
+		const result = await getNormalizedContainerOptions(config);
+		expect(result).toHaveLength(1);
+		expect(result[0].constraints.tier).toBeUndefined();
+	});
 });

packages/wrangler/src/containers/config.ts

@@ -58,7 +58,13 @@ export const getNormalizedContainerOptions = async (
 			scheduling_policy: (container.scheduling_policy ??
 				SchedulingPolicy.DEFAULT) as SchedulingPolicy,
 			constraints: {
-				tier: container.constraints?.tier ?? 1,
+				// if the tier is -1, then we allow all tiers
+				// Wrangler will default an input value to 1. The API, however, will
+				// treat an undefined value to mean no constraints on tier (i.e. "all tiers")
+				tier:
+					container.constraints?.tier === -1
+						? undefined
+						: container.constraints?.tier ?? 1,
 				regions: container.constraints?.regions?.map((region) =>
 					region.toUpperCase()
 				),