Skip to content

fix: Allow optional initProcessEnabled when enable_execute_command is true #319

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

Conversation

magreenbaum
Copy link
Member

Description

Reverts to previous logic here: https://github.com/terraform-aws-modules/terraform-aws-ecs/pull/136/files
This allows user to override optional initProcessEnabled as needed.

Motivation and Context

Closes: #311

Breaking Changes

No.

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

Copy link
Member

@bryantbiggs bryantbiggs left a comment

Choose a reason for hiding this comment

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

thank you!

@bryantbiggs bryantbiggs changed the title feat: Allow optional initProcessEnabled when enable_execute_command is true fix: Allow optional initProcessEnabled when enable_execute_command is true Jul 18, 2025
@bryantbiggs bryantbiggs merged commit 99f8edd into terraform-aws-modules:master Jul 18, 2025
12 checks passed
antonbabenko pushed a commit that referenced this pull request Jul 18, 2025
## [6.0.6](v6.0.5...v6.0.6) (2025-07-18)

### Bug Fixes

* Allow optional `initProcessEnabled` when `enable_execute_command` is `true` ([#319](#319)) ([99f8edd](99f8edd))
@antonbabenko
Copy link
Member

This PR is included in version 6.0.6 🎉

@magreenbaum magreenbaum deleted the fitx/init_process_enabled branch July 18, 2025 22:52
@nZac
Copy link

nZac commented Jul 20, 2025

Thank you @antonbabenko and @magreenbaum!

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.

initProcessEnabled should be optional for enable_execute_command
4 participants