Skip to content

ci: add collect-modules action#96

Merged
babinskiy merged 1 commit intomasterfrom
ci/collect_modules_action
Mar 21, 2025
Merged

ci: add collect-modules action#96
babinskiy merged 1 commit intomasterfrom
ci/collect_modules_action

Conversation

@babinskiy
Copy link
Contributor

@babinskiy babinskiy commented Mar 21, 2025

This pull request includes several changes to automate the collection of Terraform modules and update configuration files. The most important changes include adding a custom GitHub action to find Terraform modules, modifying the CI workflow to use this action, and updating configuration files.

Automation of Terraform module collection:

  • Added a custom GitHub action in .github/actions/collect-modules/action.yaml to find Terraform modules in the repository and output their paths.
  • Updated the CI workflow in .github/workflows/ci.yml to include a new job collect-modules that uses the custom action to find Terraform modules and outputs their paths.
  • Modified the terraform-docs job in .github/workflows/ci.yml to depend on the collect-modules job and use the collected module paths.

Configuration file updates:

  • Added default and profile-based configuration settings in asset-account/terraform/config.ini.
  • Added default and profile-based credentials in asset-account/terraform/credentials.ini.
  • Created a .module.toml file in asset-account/terraform/stack-set to define the module type and metadata.

@babinskiy babinskiy self-assigned this Mar 21, 2025
@babinskiy babinskiy requested a review from Veetaha March 21, 2025 17:13
@babinskiy babinskiy marked this pull request as ready for review March 21, 2025 17:13
@babinskiy babinskiy force-pushed the ci/collect_modules_action branch from 450e4c5 to 7a98ae2 Compare March 21, 2025 17:15
@babinskiy babinskiy merged commit cdc7046 into master Mar 21, 2025
7 checks passed
@babinskiy babinskiy deleted the ci/collect_modules_action branch March 21, 2025 18:16
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.

2 participants