Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the worker_pool_id field to apply_terraform_template_action steps #603

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

mcasperson
Copy link
Contributor

@mcasperson mcasperson commented Feb 13, 2024

The apply_terraform_template_action step does not have a worker_pool_id attribute, and setting the attribute in the properties does not work.

This PR adds the worker_pool_id attribute to bring apply_terraform_template_action inline with other steps like run_kubectl_script_action.

Fixes #601

@mcasperson mcasperson enabled auto-merge (squash) February 13, 2024 21:11
@rain-on
Copy link
Collaborator

rain-on commented Feb 13, 2024

It may be worth updating the data-sources docs to remove "name" as part of the worker_pools query

Copy link
Collaborator

@rain-on rain-on left a comment

Choose a reason for hiding this comment

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

Looks great, the terraform setup files are lovely - happy for it to go in.

@@ -524,6 +524,14 @@ func flattenApplyTerraformTemplateAction(action *deployments.DeploymentAction) m
}
}

if len(action.WorkerPool) > 0 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there value in this function calling "flattenDeploymentAction", rather than flattenAction on line 497 - or is this a more correct fix?

take = 1
}

data "octopusdeploy_feeds" "feed_octopus_server__built_in_" {
Copy link
Collaborator

Choose a reason for hiding this comment

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

should feed_octopus_server__built_in_ have double underscore, (between server and built) and ending "_"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 I can clean that up

@mcasperson mcasperson requested a review from rain-on February 13, 2024 22:17
@mcasperson mcasperson merged commit a58b130 into main Feb 13, 2024
21 checks passed
@mcasperson mcasperson deleted the mattc/worker_pool_tf_fix branch February 13, 2024 22:19
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.

Woker pool is not persisted for Terraform Apply steps
2 participants