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

DMVP-0000: Initial PR #320

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

DMVP-0000: Initial PR #320

wants to merge 19 commits into from

Conversation

viktoryathegreat
Copy link
Contributor

Terraform generated PR for best practices changes

@github-actions
Copy link

💰 Infracost estimate: monthly cost will not change

This comment will be updated when the cost estimate changes.

@what-the-diff
Copy link

what-the-diff bot commented Oct 13, 2023

PR Summary

  • Daily update of GitHub-Actions and Terraform Packages
    A new schedule has been put in place with the introduction of the .github/dependabot.yaml file. This will ensure that github-actions and terraform packages are updated every day.

  • Enhanced Checks for Branch Names on Pull Requests
    A new addition is the .github/workflows/branch-name-check.yaml file, which enforces certain guidelines on branch names submitted during pull requests. This includes checks to ensure the branch names use specific prefixes, match a predetermined pattern and satisfy length requirements.

  • Improved Error Handling in Project Analysis
    The .github/workflows/checkov.yaml and .github/workflows/terraform-test.yaml files have been modified to apply changes to the project's root directory and to continue in case of errors, offering smoother processing of jobs.

  • Cost Estimation for Changes
    The new .github/workflows/infracost.yaml file sets up a system for cost estimates. This is triggered every time there is a new pull request or push event, providing financial transparency for made changes.

  • Enhanced Checks on Pull Request Titles
    A check system has been implemented in the new .github/workflows/pr-title-checker.yaml file. It verifies that titles of pull requests satisfy certain norms, such as specific prefixes, patterns, and length requirements.

  • Inclusion of AWS Credentials
    The .github/workflows/pre-commit.yaml and .github/workflows/tflint.yaml files have been edited to include AWS access and secret key as environment variables. This ensures secure access to AWS services.

  • Updated Version of Setup-Node Action
    The .github/workflows/semantic-release.yaml file has been updated to utilize the latest version of setup-node action.

  • Addition of Check-YAML as a Pre-Commit Hook
    The .github/workflows/tfsec.yaml and .pre-commit-config.yaml files have been updated to include check-yaml as a pre-commit hook. This helps maintain the quality of the code by detecting potential issues before a commit is made.

@github-actions
Copy link

Output

[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
[INFO] Initializing environment for https://github.com/qoomon/git-conventional-commits.
[INFO] Initializing environment for https://github.com/zricethezav/gitleaks.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/zricethezav/gitleaks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check for added large files..............................................Passed
check for merge conflicts................................................Passed
check vcs permalinks.....................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing .github/workflows/checkov.yaml
Fixing .github/workflows/infracost.yaml
Fixing .github/workflows/tflint.yaml
Fixing .github/workflows/terraform-test.yaml

check yaml...............................................................Failed
- hook id: check-yaml
- exit code: 1

while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 16

check for merge conflicts................................................Passed
check for case conflicts.................................................Passed
mixed line ending........................................................Passed
detect aws credentials...................................................Passed
detect private key.......................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Passed
Detect hardcoded secrets.................................................Passed

Copy link

github-actions bot commented Nov 3, 2023

Output

[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Initializing environment for https://github.com/antonbabenko/pre-commit-terraform.
[INFO] Initializing environment for https://github.com/qoomon/git-conventional-commits.
[INFO] Initializing environment for https://github.com/zricethezav/gitleaks.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/zricethezav/gitleaks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check for added large files..............................................Passed
check for merge conflicts................................................Passed
check vcs permalinks.....................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing .github/workflows/tflint.yaml
Fixing .github/workflows/terraform-test.yaml
Fixing .github/workflows/checkov.yaml

check yaml...............................................................Failed
- hook id: check-yaml
- exit code: 1

while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 16

check for merge conflicts................................................Passed
check for case conflicts.................................................Passed
mixed line ending........................................................Passed
detect aws credentials...................................................Passed
detect private key.......................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Passed
Detect hardcoded secrets.................................................Passed

Copy link

github-actions bot commented Nov 3, 2023

Output

check for added large files..............................................Passed
check for merge conflicts................................................Passed
check vcs permalinks.....................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing .github/workflows/tflint.yaml
Fixing .github/workflows/terraform-test.yaml
Fixing .github/workflows/checkov.yaml

check yaml...............................................................Failed
- hook id: check-yaml
- exit code: 1

while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 16

check for merge conflicts................................................Passed
check for case conflicts.................................................Passed
mixed line ending........................................................Passed
detect aws credentials...................................................Passed
detect private key.......................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Passed
Detect hardcoded secrets.................................................Passed

1 similar comment
Copy link

github-actions bot commented Nov 3, 2023

Output

check for added large files..............................................Passed
check for merge conflicts................................................Passed
check vcs permalinks.....................................................Passed
fix end of files.........................................................Passed
trim trailing whitespace.................................................Failed
- hook id: trailing-whitespace
- exit code: 1
- files were modified by this hook

Fixing .github/workflows/tflint.yaml
Fixing .github/workflows/terraform-test.yaml
Fixing .github/workflows/checkov.yaml

check yaml...............................................................Failed
- hook id: check-yaml
- exit code: 1

while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/pvc.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/serviceaccount.yaml", line 1, column 16
while scanning a plain scalar
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 5
found unexpected ':'
  in "modules/aws-efs/backup/helm/jobs/charts/base-cronjob/templates/cronjob.yaml", line 1, column 16

check for merge conflicts................................................Passed
check for case conflicts.................................................Passed
mixed line ending........................................................Passed
detect aws credentials...................................................Passed
detect private key.......................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Passed
Detect hardcoded secrets.................................................Passed

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.

None yet

1 participant