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

feat: add execution order group #2178

Merged
merged 2 commits into from
Jun 29, 2022

Conversation

tufitko
Copy link
Contributor

@tufitko tufitko commented Apr 4, 2022

Implementing is #2076, without early aborting

Added execution_order_group. After building projects, projects will be sorted by these groups.

It's useful for projects with dependencies among themselves (Mostly for terragrunt projects, or terraform with remote_states).

I will be able to support parallel apply/plan with these groups in next pr.

@tufitko tufitko requested a review from a team as a code owner April 4, 2022 08:05
@jamengual
Copy link
Contributor

jamengual commented May 13, 2022

@tufitko could you add some docs examples?

@jamengual jamengual added feature New functionality/enhancement waiting-on-review Waiting for a review from a maintainer labels May 13, 2022
@Vladyslav-Miletskyi
Copy link

Such an elegant approach!
Totally love it! I hope this will be added soon.

Terragrunt user :)

to apply/plan by order
@tufitko tufitko force-pushed the execution-order-group branch from e24aa88 to f19e8af Compare May 20, 2022 06:00
@tufitko
Copy link
Contributor Author

tufitko commented May 20, 2022

@jamengual I updated docs
Looks like CI fails because not found DNS record install.goreleaser.com

@tufitko
Copy link
Contributor Author

tufitko commented May 20, 2022

@jamengual also fixed CI lint

@tufitko
Copy link
Contributor Author

tufitko commented Jun 21, 2022

@jamengual any updates?

Copy link
Contributor

@jamengual jamengual left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the contribution

@jamengual jamengual merged commit 569031a into runatlantis:master Jun 29, 2022
krrrr38 pushed a commit to krrrr38/atlantis that referenced this pull request Dec 16, 2022
* feat: add execution order group

to apply/plan by order

* fix: use official golanci-lint script
@nitrocode nitrocode added this to the v0.19.5 milestone Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality/enhancement waiting-on-review Waiting for a review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants