default remote repo build mode = true

Author: johnstcn

internal/provider/cached_image_resource.go

@@ -339,7 +339,8 @@ func (r *CachedImageResource) Read(ctx context.Context, req resource.ReadRequest
 		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_GIT_PASSWORD", data.GitPassword)
 	}
 	if !data.RemoteRepoBuildMode.IsNull() {
-		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_REMOTE_REPO_BUILD_MODE", data.RemoteRepoBuildMode)
+		// Default to remote build mode.
+		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_REMOTE_REPO_BUILD_MODE", types.BoolValue(true))
 	}
 
 	resp.Diagnostics.Append(resp.State.Set(ctx, &data)...)
@@ -391,7 +392,8 @@ func (r *CachedImageResource) Create(ctx context.Context, req resource.CreateReq
 		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_GIT_PASSWORD", data.GitPassword)
 	}
 	if !data.RemoteRepoBuildMode.IsNull() {
-		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_REMOTE_REPO_BUILD_MODE", data.RemoteRepoBuildMode)
+		// Default to remote build mode.
+		data.Env = appendKnownEnvToList(data.Env, "ENVBUILDER_REMOTE_REPO_BUILD_MODE", types.BoolValue(true))
 	}
 
 	// Save data into Terraform state

internal/provider/cached_image_resource_test.go

@@ -38,6 +38,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "id", uuid.Nil.String()),
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "exists", "false"),
 					resource.TestCheckResourceAttrSet("envbuilder_cached_image.test", "env.0"),
+					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "remote_repo_build_mode", "true"),
 					// Cached image should be set to the builder image.
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "image", deps.BuilderImage),
 					// Inputs should still be present.
@@ -59,6 +60,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "id", uuid.Nil.String()),
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "exists", "false"),
 					resource.TestCheckResourceAttrSet("envbuilder_cached_image.test", "env.0"),
+					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "remote_repo_build_mode", "true"),
 					// Cached image should be set to the builder image.
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "image", deps.BuilderImage),
 					// Inputs should still be present.
@@ -83,6 +85,7 @@ func TestAccCachedImageDataSource(t *testing.T) {
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "cache_repo", deps.CacheRepo),
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "extra_env.FOO", "bar"),
 					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "git_url", deps.Repo.URL),
+					resource.TestCheckResourceAttr("envbuilder_cached_image.test", "remote_repo_build_mode", "true"),
 					// Should be empty
 					resource.TestCheckNoResourceAttr("envbuilder_cached_image.test", "git_username"),
 					resource.TestCheckNoResourceAttr("envbuilder_cached_image.test", "git_password"),