Skip to content

Commit f0e65a7

Browse files
chrisRedwinebryantbiggs
chrisRedwine
and
bryantbiggs
authored
fix: Add required S3 PutObjectTagging permission to Velero IRSA policy (#517)
Co-authored-by: Bryant Biggs <[email protected]>
1 parent c15f9dd commit f0e65a7

File tree

33 files changed

+106
-88
lines changed

33 files changed

+106
-88
lines changed

‎.github/workflows/pre-commit.yml

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ on:
77
- master
88

99
env:
10-
TERRAFORM_DOCS_VERSION: v0.16.0
11-
TFLINT_VERSION: v0.50.3
10+
TERRAFORM_DOCS_VERSION: v0.19.0
11+
TFLINT_VERSION: v0.53.0
1212

1313
jobs:
1414
collectInputs:
@@ -75,10 +75,27 @@ jobs:
7575
# https://github.com/orgs/community/discussions/25678#discussioncomment-5242449
7676
- name: Delete huge unnecessary tools folder
7777
run: |
78+
df -h
7879
rm -rf /opt/hostedtoolcache/CodeQL
7980
rm -rf /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk
8081
rm -rf /opt/hostedtoolcache/Ruby
8182
rm -rf /opt/hostedtoolcache/go
83+
# And a little bit more
84+
sudo apt-get -qq remove -y 'azure-.*'
85+
sudo apt-get -qq remove -y 'cpp-.*'
86+
sudo apt-get -qq remove -y 'dotnet-runtime-.*'
87+
sudo apt-get -qq remove -y 'google-.*'
88+
sudo apt-get -qq remove -y 'libclang-.*'
89+
sudo apt-get -qq remove -y 'libllvm.*'
90+
sudo apt-get -qq remove -y 'llvm-.*'
91+
sudo apt-get -qq remove -y 'mysql-.*'
92+
sudo apt-get -qq remove -y 'postgresql-.*'
93+
sudo apt-get -qq remove -y 'php.*'
94+
sudo apt-get -qq remove -y 'temurin-.*'
95+
sudo apt-get -qq remove -y kubectl firefox powershell mono-devel
96+
sudo apt-get -qq autoremove -y
97+
sudo apt-get -qq clean
98+
df -h
8299
83100
- name: Checkout
84101
uses: actions/checkout@v4

‎.pre-commit-config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
repos:
22
- repo: https://github.com/antonbabenko/pre-commit-terraform
3-
rev: v1.92.0
3+
rev: v1.96.1
44
hooks:
55
- id: terraform_fmt
66
- id: terraform_wrapper_module_for_each

‎examples/iam-account/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -46,4 +46,4 @@ No inputs.
4646
|------|-------------|
4747
| <a name="output_caller_identity_account_id"></a> [caller\_identity\_account\_id](#output\_caller\_identity\_account\_id) | The ID of the AWS account |
4848
| <a name="output_iam_account_password_policy_expire_passwords"></a> [iam\_account\_password\_policy\_expire\_passwords](#output\_iam\_account\_password\_policy\_expire\_passwords) | Indicates whether passwords in the account expire. Returns true if max\_password\_age contains a value greater than 0. Returns false if it is 0 or not present. |
49-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
49+
<!-- END_TF_DOCS -->

‎examples/iam-assumable-role-with-oidc/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -51,4 +51,4 @@ No inputs.
5151
| <a name="output_iam_role_name"></a> [iam\_role\_name](#output\_iam\_role\_name) | Name of IAM role |
5252
| <a name="output_iam_role_path"></a> [iam\_role\_path](#output\_iam\_role\_path) | Path of IAM role |
5353
| <a name="output_iam_role_unique_id"></a> [iam\_role\_unique\_id](#output\_iam\_role\_unique\_id) | Unique ID of IAM role |
54-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
54+
<!-- END_TF_DOCS -->

‎examples/iam-assumable-role-with-saml/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -54,4 +54,4 @@ No inputs.
5454
| <a name="output_iam_role_name"></a> [iam\_role\_name](#output\_iam\_role\_name) | Name of IAM role |
5555
| <a name="output_iam_role_path"></a> [iam\_role\_path](#output\_iam\_role\_path) | Path of IAM role |
5656
| <a name="output_iam_role_unique_id"></a> [iam\_role\_unique\_id](#output\_iam\_role\_unique\_id) | Unique ID of IAM role |
57-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
57+
<!-- END_TF_DOCS -->

‎examples/iam-assumable-role/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ $ terraform apply
1616

1717
Run `terraform destroy` when you don't need these resources.
1818

19-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
19+
<!-- BEGIN_TF_DOCS -->
2020
## Requirements
2121

2222
| Name | Version |
@@ -61,4 +61,4 @@ No inputs.
6161
| <a name="output_iam_role_path"></a> [iam\_role\_path](#output\_iam\_role\_path) | Path of IAM role |
6262
| <a name="output_iam_role_unique_id"></a> [iam\_role\_unique\_id](#output\_iam\_role\_unique\_id) | Unique ID of IAM role |
6363
| <a name="output_role_requires_mfa"></a> [role\_requires\_mfa](#output\_role\_requires\_mfa) | Whether admin IAM role requires MFA |
64-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
64+
<!-- END_TF_DOCS -->

‎examples/iam-assumable-roles-with-saml/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -64,4 +64,4 @@ No inputs.
6464
| <a name="output_readonly_iam_role_name"></a> [readonly\_iam\_role\_name](#output\_readonly\_iam\_role\_name) | Name of readonly IAM role |
6565
| <a name="output_readonly_iam_role_path"></a> [readonly\_iam\_role\_path](#output\_readonly\_iam\_role\_path) | Path of readonly IAM role |
6666
| <a name="output_readonly_iam_role_unique_id"></a> [readonly\_iam\_role\_unique\_id](#output\_readonly\_iam\_role\_unique\_id) | Unique ID of IAM role |
67-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
67+
<!-- END_TF_DOCS -->

‎examples/iam-assumable-roles/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -60,4 +60,4 @@ No inputs.
6060
| <a name="output_readonly_iam_role_path"></a> [readonly\_iam\_role\_path](#output\_readonly\_iam\_role\_path) | Path of readonly IAM role |
6161
| <a name="output_readonly_iam_role_requires_mfa"></a> [readonly\_iam\_role\_requires\_mfa](#output\_readonly\_iam\_role\_requires\_mfa) | Whether readonly IAM role requires MFA |
6262
| <a name="output_readonly_iam_role_unique_id"></a> [readonly\_iam\_role\_unique\_id](#output\_readonly\_iam\_role\_unique\_id) | Unique ID of IAM role |
63-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
63+
<!-- END_TF_DOCS -->

‎examples/iam-eks-role/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ $ terraform apply
1414

1515
Run `terraform destroy` when you don't need these resources.
1616

17-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
17+
<!-- BEGIN_TF_DOCS -->
1818
## Requirements
1919

2020
| Name | Version |
@@ -59,4 +59,4 @@ No inputs.
5959
| <a name="output_iam_role_name"></a> [iam\_role\_name](#output\_iam\_role\_name) | Name of IAM role |
6060
| <a name="output_iam_role_path"></a> [iam\_role\_path](#output\_iam\_role\_path) | Path of IAM role |
6161
| <a name="output_iam_role_unique_id"></a> [iam\_role\_unique\_id](#output\_iam\_role\_unique\_id) | Unique ID of IAM role |
62-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
62+
<!-- END_TF_DOCS -->

‎examples/iam-github-oidc/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ $ terraform apply
1717

1818
Run `terraform destroy` when you don't need these resources.
1919

20-
<!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
20+
<!-- BEGIN_TF_DOCS -->
2121
## Requirements
2222

2323
| Name | Version |
@@ -60,4 +60,4 @@ No inputs.
6060
| <a name="output_iam_role_unique_id"></a> [iam\_role\_unique\_id](#output\_iam\_role\_unique\_id) | Unique ID of IAM role |
6161
| <a name="output_provider_arn"></a> [provider\_arn](#output\_provider\_arn) | The ARN assigned by AWS for this provider |
6262
| <a name="output_provider_url"></a> [provider\_url](#output\_provider\_url) | The URL of the identity provider. Corresponds to the iss claim |
63-
<!-- END OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
63+
<!-- END_TF_DOCS -->

0 commit comments

Comments
 (0)