wrangler containers: wrangler deploy should be able to deploy container

applications

We are adding a new step into wrangler deploy that allows the user to
deploy container applications through wrangler deploy.

The main changes on this commit allow:
 1. cloudchamber/build.ts to accomodate the needs of the new deploy path
    of containers.
 2. cloudchamber/apply.ts to create a Cloudchamber rollout.
 3. deploy.ts to call a method when containers option is defined.

 This method in cloudchamber/deploy.ts does the following:

  1. Call fetchVersion to know what's the namespace_id of the target
     DOs.
  2. Deploy each container application.
     Each container application might specify a Dockerfile or an image
     URI. If it's a Dockerfile, we will read it and build it.

Author: gabivlj

.changeset/sad-taxis-kick.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+wrangler deploy is able to deploy new container versions

packages/wrangler/src/__tests__/cloudchamber/apply.test.ts

@@ -65,6 +65,25 @@ function mockModifyApplication(
 		response = res;
 	});
 
+	// Add when we start creating rollouts in `apply.ts`
+	//
+	// msw.use(
+	// 	http.post(
+	// 		"*/applications/:id/rollouts",
+	// 		async ({ request }) => {
+	// 			const json = await request.json();
+	// 			if (expectedRollout !== undefined) {
+	// 				expect(json).toMatchObject(expectedRollout);
+	// 			}
+	//
+	// 			expect((json as CreateApplicationRequest).name).toBeUndefined();
+	// 			response(json as ModifyApplicationRequestBody);
+	// 			return HttpResponse.json(json);
+	// 		},
+	// 		{ once: true }
+	// 	)
+	// );
+
 	msw.use(
 		http.patch(
 			"*/applications/:id",
@@ -73,7 +92,7 @@ function mockModifyApplication(
 				if (expected !== undefined) {
 					expect(json).toEqual(expected);
 				}
-				console.log(json);
+
 				expect((json as CreateApplicationRequest).name).toBeUndefined();
 				response(json as ModifyApplicationRequestBody);
 				return HttpResponse.json(json);
@@ -166,6 +185,7 @@ describe("cloudchamber apply", () => {
 				name: "my-container-app",
 				instances: 3,
 				created_at: new Date().toString(),
+				version: 1,
 				account_id: "1",
 				scheduling_policy: SchedulingPolicy.REGIONAL,
 				configuration: {
@@ -198,6 +218,8 @@ describe("cloudchamber apply", () => {
 			├ Do you want to apply these changes?
 			│ yes
 			│
+			├ Loading
+			│
 			│
 			│  SUCCESS  Modified application my-container-app
 			│
@@ -239,6 +261,7 @@ describe("cloudchamber apply", () => {
 				instances: 3,
 				created_at: new Date().toString(),
 				account_id: "1",
+				version: 1,
 				scheduling_policy: SchedulingPolicy.REGIONAL,
 				configuration: {
 					image: "./Dockerfile",
@@ -281,6 +304,8 @@ describe("cloudchamber apply", () => {
 			├ Do you want to apply these changes?
 			│ yes
 			│
+			├ Loading
+			│
 			│
 			│  SUCCESS  Modified application my-container-app
 			│
@@ -336,6 +361,7 @@ describe("cloudchamber apply", () => {
 				id: "abc",
 				name: "my-container-app",
 				instances: 3,
+				version: 1,
 				created_at: new Date().toString(),
 				account_id: "1",
 				scheduling_policy: SchedulingPolicy.REGIONAL,
@@ -408,6 +434,8 @@ describe("cloudchamber apply", () => {
 			├ Do you want to apply these changes?
 			│ yes
 			│
+			├ Loading
+			│
 			│
 			│  SUCCESS  Modified application my-container-app
 			│
@@ -461,6 +489,7 @@ describe("cloudchamber apply", () => {
 				id: "abc",
 				name: "my-container-app",
 				instances: 3,
+				version: 1,
 				created_at: new Date().toString(),
 				account_id: "1",
 				scheduling_policy: SchedulingPolicy.REGIONAL,
@@ -601,8 +630,8 @@ describe("cloudchamber apply", () => {
 		};
 
 		mockGetApplications([
-			completeApp,
-			{ ...completeApp, name: "my-container-app-2", id: "abc2" },
+			{ ...completeApp, version: 1 },
+			{ ...completeApp, version: 1, name: "my-container-app-2", id: "abc2" },
 		]);
 		await runWrangler("cloudchamber apply --json");
 		/* eslint-disable */

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

@@ -63,6 +63,7 @@ describe("containers list", () => {
 			        \\"created_at\\": \\"123\\",
 			        \\"account_id\\": \\"test-account\\",
 			        \\"name\\": \\"Test-app\\",
+			        \\"version\\": 1,
 			        \\"configuration\\": {
 			            \\"image\\": \\"test-registry.cfdata.org/test-app:v1\\",
 			            \\"network\\": {
@@ -81,6 +82,7 @@ describe("containers list", () => {
 			        \\"created_at\\": \\"123\\",
 			        \\"account_id\\": \\"test-account\\",
 			        \\"name\\": \\"Test-app\\",
+			        \\"version\\": 1,
 			        \\"configuration\\": {
 			            \\"image\\": \\"test-registry.cfdata.org/test-app:v10\\",
 			            \\"network\\": {
@@ -99,6 +101,7 @@ describe("containers list", () => {
 			        \\"created_at\\": \\"123\\",
 			        \\"account_id\\": \\"test-account\\",
 			        \\"name\\": \\"Test-app\\",
+			        \\"version\\": 1,
 			        \\"configuration\\": {
 			            \\"image\\": \\"test-registry.cfdata.org/test-app:v2\\",
 			            \\"network\\": {

packages/wrangler/src/__tests__/deploy.test.ts

@@ -10,13 +10,15 @@ import { http, HttpResponse } from "msw";
 import dedent from "ts-dedent";
 import { File } from "undici";
 import { vi } from "vitest";
+import { type Application } from "../cloudchamber/client";
 import {
 	printBundleSize,
 	printOffendingDependencies,
 } from "../deployment-bundle/bundle-reporter";
 import { clearOutputFilePath } from "../output";
 import { sniffUserAgent } from "../package-manager";
 import { writeAuthConfigFile } from "../user";
+import { mockAccount as mockContainersAccount } from "./cloudchamber/utils";
 import { mockAccountId, mockApiToken } from "./helpers/mock-account-id";
 import { mockAuthDomain } from "./helpers/mock-auth-domain";
 import { mockConsoleMethods } from "./helpers/mock-console";
@@ -8602,6 +8604,48 @@ addEventListener('fetch', event => {});`
 			});
 
 			it("should support durable object bindings to SQLite classes with containers", async () => {
+				function mockGetVersion(versionId: string) {
+					msw.use(
+						http.get(
+							`*/accounts/:accountId/workers/scripts/:scriptName/versions/${versionId}`,
+							async () => {
+								return HttpResponse.json(
+									createFetchResult({
+										id: versionId,
+										metadata: {},
+										number: 2,
+										resources: {
+											bindings: [
+												{
+													type: "durable_object_namespace",
+													namespace_id: "1",
+													class_name: "ExampleDurableObject",
+												},
+											],
+										},
+									})
+								);
+							},
+							{ once: true }
+						)
+					);
+				}
+
+				function mockGetApplications(applications: Application[]) {
+					msw.use(
+						http.get(
+							"*/applications",
+							async () => {
+								return HttpResponse.json(applications);
+							},
+							{ once: true }
+						)
+					);
+				}
+
+				mockGetVersion("Galaxy-Class");
+
+				mockContainersAccount();
 				writeWranglerConfig({
 					durable_objects: {
 						bindings: [
@@ -8625,6 +8669,31 @@ addEventListener('fetch', event => {});`
 						{ tag: "v1", new_sqlite_classes: ["ExampleDurableObject"] },
 					],
 				});
+
+				mockGetApplications([]);
+				function mockCreateApplication(expected?: Partial<Application>) {
+					msw.use(
+						http.post(
+							"*/applications",
+							async ({ request }) => {
+								const json = await request.json();
+								if (expected !== undefined) {
+									expect(json).toMatchObject(expected);
+								}
+
+								return HttpResponse.json(json);
+							},
+							{ once: true }
+						)
+					);
+				}
+
+				mockCreateApplication({
+					name: "my-container",
+					instances: 10,
+					durable_objects: { namespace_id: "1" },
+				});
+
 				fs.writeFileSync(
 					"index.js",
 					`export class ExampleDurableObject {}; export default{};`

packages/wrangler/src/__tests__/helpers/mock-cloudchamber.ts

@@ -191,6 +191,7 @@ export const MOCK_APPLICATIONS: Application[] = [
 		created_at: "123",
 		account_id: "test-account",
 		name: "Test-app",
+		version: 1,
 		configuration: {
 			image: "test-registry.cfdata.org/test-app:v1",
 			network: {
@@ -207,6 +208,7 @@ export const MOCK_APPLICATIONS: Application[] = [
 		created_at: "123",
 		account_id: "test-account",
 		name: "Test-app",
+		version: 1,
 		configuration: {
 			image: "test-registry.cfdata.org/test-app:v10",
 			network: {
@@ -223,6 +225,7 @@ export const MOCK_APPLICATIONS: Application[] = [
 		created_at: "123",
 		account_id: "test-account",
 		name: "Test-app",
+		version: 1,
 		configuration: {
 			image: "test-registry.cfdata.org/test-app:v2",
 			network: {