Skip to content

Add support for GitURL and Dockerfile in Execution Container Images#234

Merged
mcasperson merged 6 commits into
mainfrom
mattc/execution_container_update
Jul 21, 2026
Merged

Add support for GitURL and Dockerfile in Execution Container Images#234
mcasperson merged 6 commits into
mainfrom
mattc/execution_container_update

Conversation

@mcasperson

Copy link
Copy Markdown
Contributor

This PR exposes the git_url and dockerfile properties on the container images.

@mcasperson
mcasperson marked this pull request as ready for review July 21, 2026 21:02
@mcasperson
mcasperson enabled auto-merge (squash) July 21, 2026 21:02
@mcasperson
mcasperson merged commit 071c732 into main Jul 21, 2026
19 checks passed
@mcasperson
mcasperson deleted the mattc/execution_container_update branch July 21, 2026 22:40
@wlthomson
wlthomson requested a review from Copilot July 21, 2026 22:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds Terraform surface area for Octopus “execution container image” fields by exposing git_url and dockerfile alongside existing feed_id/image, across both the legacy SDKv2 schemas and the newer Plugin Framework resources.

Changes:

  • Extend the SDKv2 DeploymentActionContainer expand/flatten + schema to include git_url and dockerfile, with a unit test update.
  • Extend Framework process step container schema/model and mapping logic to round-trip git_url/dockerfile (including updated mapping tests).
  • Bump github.com/OctopusDeploy/go-octopusdeploy/v2 dependency and update generated docs to include the new attributes.

Reviewed changes

Copilot reviewed 31 out of 32 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
octopusdeploy/schema_deployment_action_container.go Adds git_url/dockerfile to SDKv2 container expand/flatten and schema.
octopusdeploy/schema_deployment_action_container_test.go Updates unit test coverage for the SDKv2 container expansion.
octopusdeploy_framework/schemas/process_step.go Adds git_url/dockerfile to Framework container model + schema defaults.
octopusdeploy_framework/resource_process_templated_step.go Passes git_url/dockerfile into container mapping when building API model.
octopusdeploy_framework/resource_process_templated_child_step.go Passes git_url/dockerfile into container mapping when building API model.
octopusdeploy_framework/resource_process_step.go Maps git_url/dockerfile to/from state for process steps.
octopusdeploy_framework/resource_process_step_mapping_test.go Updates mapping tests for the expanded container model (one initialization needs adjustment).
octopusdeploy_framework/resource_process_child_step.go Passes git_url/dockerfile into container mapping when building API model.
octopusdeploy_framework/resource_process_child_step_mapping_test.go Updates mapping tests for child steps with the expanded container model.
go.mod Bumps go-octopusdeploy/v2 version to a release that supports the new container fields.
go.sum Updates checksums for the bumped go-octopusdeploy/v2 dependency.
docs/resources/ssh_connection_deployment_target.md Documents new container fields in resource docs.
docs/resources/process_templated_step.md Documents new container fields in resource docs (with descriptions).
docs/resources/process_templated_child_step.md Documents new container fields in resource docs (with descriptions).
docs/resources/process_step.md Documents new container fields in resource docs (with descriptions).
docs/resources/process_child_step.md Documents new container fields in resource docs (with descriptions).
docs/resources/polling_tentacle_deployment_target.md Documents new container fields in resource docs.
docs/resources/offline_package_drop_deployment_target.md Documents new container fields in resource docs.
docs/resources/kubernetes_cluster_deployment_target.md Documents new container fields in resource docs.
docs/resources/cloud_region_deployment_target.md Documents new container fields in resource docs.
docs/resources/azure_web_app_deployment_target.md Documents new container fields in resource docs.
docs/resources/azure_service_fabric_cluster_deployment_target.md Documents new container fields in resource docs.
docs/resources/azure_cloud_service_deployment_target.md Documents new container fields in resource docs.
docs/data-sources/ssh_connection_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/polling_tentacle_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/offline_package_drop_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/kubernetes_cluster_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/cloud_region_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/azure_web_app_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/azure_service_fabric_cluster_deployment_targets.md Documents new container fields in data source docs.
docs/data-sources/azure_cloud_service_deployment_targets.md Documents new container fields in data source docs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ExcludedEnvironments: types.SetValueMust(types.StringType, []attr.Value{}),
Channels: types.SetValueMust(types.StringType, []attr.Value{}),
Container: &schemas.ProcessStepActionContainerModel{FeedID: types.StringValue(""), Image: types.StringValue("")},
Container: &schemas.ProcessStepActionContainerModel{FeedID: types.StringValue(""), Image: types.StringValue(""), GitUrl: types.StringValue(""), Dockerfile: types.StringNull()},
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants