Skip to content

Commit 2f47684

Browse files
authored
feat: Security fix and pre-commit fixes (#906)
Signed-off-by: vara-bonthu <vara.bonthu@gmail.com>
1 parent 5b34654 commit 2f47684

File tree

9 files changed

+93
-81
lines changed

9 files changed

+93
-81
lines changed

.github/workflows/plan-examples.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ concurrency:
77
group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
88
cancel-in-progress: true
99

10+
permissions:
11+
contents: read
12+
id-token: write
13+
1014
jobs:
1115
getExampleDirectories:
1216
name: Get example directories

.github/workflows/pre-commit.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,10 @@ concurrency:
1919
group: '${{ github.workflow }} @ ${{ github.event.pull_request.head.label || github.head_ref || github.ref }}'
2020
cancel-in-progress: true
2121

22+
permissions:
23+
contents: read
24+
pull-requests: read
25+
2226
jobs:
2327
collectInputs:
2428
name: Collect workflow inputs

.github/workflows/website-deploy.yaml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,19 @@ on:
77
# Review gh actions docs if you want to further define triggers, paths, etc
88
# https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#on
99

10+
permissions:
11+
contents: read
12+
pages: write
13+
id-token: write
14+
1015
jobs:
1116
deploy:
1217
name: Deploy to GitHub Pages
1318
runs-on: ubuntu-latest
19+
permissions:
20+
contents: read
21+
pages: write
22+
id-token: write
1423
defaults:
1524
run:
1625
shell: bash

.github/workflows/website-test-deploy.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,15 @@ on:
55
branches:
66
- main
77

8+
permissions:
9+
contents: read
10+
811
jobs:
912
test-deploy:
1013
name: Test deployment
1114
runs-on: ubuntu-latest
15+
permissions:
16+
contents: read
1217
defaults:
1318
run:
1419
shell: bash

distributed-databases/cloudnative-postgres/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@ Checkout the [documentation website](https://awslabs.github.io/data-on-eks/docs/
4747
| [kubernetes_storage_class.ebs_csi_encrypted_gp3_storage_class](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/storage_class) | resource |
4848
| [random_password.grafana](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/password) | resource |
4949
| [random_string.random](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/string) | resource |
50-
| [aws_ami.eks](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ami) | data source |
5150
| [aws_availability_zones.available](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/availability_zones) | data source |
5251
| [aws_caller_identity.current](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/caller_identity) | data source |
5352
| [aws_eks_cluster_auth.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/eks_cluster_auth) | data source |

distributed-databases/cloudnative-postgres/data.tf

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -13,21 +13,6 @@ data "aws_availability_zones" "available" {
1313
data "aws_caller_identity" "current" {}
1414

1515

16-
data "aws_ami" "eks" {
17-
owners = ["amazon"]
18-
most_recent = true
19-
20-
filter {
21-
name = "name"
22-
values = ["amazon-eks-node-al2023-arm64-standard-${var.eks_cluster_version}-*"]
23-
}
24-
25-
filter {
26-
name = "architecture"
27-
values = ["arm64"]
28-
}
29-
}
30-
3116
data "aws_iam_policy_document" "irsa_backup_policy" {
3217
statement {
3318
sid = ""

streaming/emr-eks-flink/README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ Checkout the [documentation website](https://awslabs.github.io/data-on-eks/docs/
1818
| Name | Version |
1919
|------|---------|
2020
| <a name="provider_aws"></a> [aws](#provider\_aws) | ~> 5.95 |
21-
| <a name="provider_aws.ecr"></a> [aws.ecr](#provider\_aws.ecr) | ~> 5.95 |
2221
| <a name="provider_helm"></a> [helm](#provider\_helm) | ~> 2.17 |
2322
| <a name="provider_kubernetes"></a> [kubernetes](#provider\_kubernetes) | >= 2.10 |
2423

@@ -50,7 +49,6 @@ Checkout the [documentation website](https://awslabs.github.io/data-on-eks/docs/
5049
| [kubernetes_namespace_v1.flink_team_a](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/namespace_v1) | resource |
5150
| [aws_availability_zones.available](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/availability_zones) | data source |
5251
| [aws_caller_identity.current](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/caller_identity) | data source |
53-
| [aws_ecrpublic_authorization_token.token](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ecrpublic_authorization_token) | data source |
5452
| [aws_eks_cluster_auth.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/eks_cluster_auth) | data source |
5553
| [aws_iam_policy_document.flink_sample_job](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/iam_policy_document) | data source |
5654

streaming/emr-eks-flink/main.tf

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,6 @@ data "aws_eks_cluster_auth" "this" {
1414
name = module.eks.cluster_name
1515
}
1616

17-
data "aws_ecrpublic_authorization_token" "token" {
18-
provider = aws.ecr
19-
}
20-
2117
data "aws_availability_zones" "available" {}
2218

2319
data "aws_caller_identity" "current" {}

website/package-lock.json

Lines changed: 71 additions & 59 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)