Skip to content

Conversation

ipcrm
Copy link

@ipcrm ipcrm commented Nov 16, 2023

Summary

Large environments will take hours to deploy if we do not enable parallelization.

How did you test this change?

Manual execution:

  • creation
  • updates
  • deletion

Update/deletes tested

Issue

https://lacework.atlassian.net/browse/GROW-2583

versions.tf Outdated
aws = {
source = "hashicorp/aws"
version = ">= 3.35.0"
version = ">= 5.11.0"
Copy link
Author

Choose a reason for hiding this comment

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

NOTE: the change here, in it's own commit. Please rebase/merge not squash.

@ipcrm ipcrm changed the title fix(GROW-2583): enable parrallel stackset operations fix(GROW-2583): enable parallel stackset operations Nov 16, 2023
PengyuanZhao
PengyuanZhao previously approved these changes Nov 16, 2023
Copy link
Contributor

@PengyuanZhao PengyuanZhao left a comment

Choose a reason for hiding this comment

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

LGTM. Do we know how faster the parallel mode is? Any pitfalls?

@ipcrm
Copy link
Author

ipcrm commented Nov 17, 2023

No pitfalls that I'm aware of, but much faster; for both creation as well as update/deletes

@ipcrm ipcrm force-pushed the ipcrm/GROW-2583/operating-params branch from 9aafe5f to 8e5d13a Compare November 17, 2023 01:11
@afiune afiune changed the title fix(GROW-2583): enable parallel stackset operations fix: enable parallel stackset operations Nov 17, 2023
afiune
afiune previously approved these changes Nov 17, 2023
Copy link
Contributor

@afiune afiune left a comment

Choose a reason for hiding this comment

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

Can we not update the aws provider version?

* chore: downgrade Python version to 3.9

Signed-off-by: Salim Afiune Maya <afiune@lacework.net>

* style: create a local variable

Signed-off-by: Salim Afiune Maya <afiune@lacework.net>

---------

Signed-off-by: Salim Afiune Maya <afiune@lacework.net>
Copy link
Contributor

@afiune afiune left a comment

Choose a reason for hiding this comment

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

@ipcrm ipcrm merged commit 06f3c0c into main Nov 17, 2023
@ipcrm ipcrm deleted the ipcrm/GROW-2583/operating-params branch November 17, 2023 20:03
@lacework-releng lacework-releng mentioned this pull request Nov 17, 2023
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