Skip to content

Conversation

VincentGoeckeritz
Copy link

Fixes:

  • fix the airflow CLI command (as far as I can determine from the airflow docs even in versions 1.x it was always airflow pools, never pool)
  • rename the project_id variable to project (this is what the gcloud command uses and using _id when actually the name is required is misleading)

New:

  • add optional variable include_deferred (defaults to false) to allow users to specify whether deferred tasks should be included in the pool's occupied slot calculation
  • add a section in the README for how to correctly destroy a pool when using this module

Updates:

  • update used the gcloud command terraform module from version ~> 3.1 to ~> 4.0

@VincentGoeckeritz VincentGoeckeritz requested review from a team and imrannayer as code owners October 9, 2025 12:43
Copy link

google-cla bot commented Oct 9, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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.

1 participant