Skip to content

Commit

Permalink
Clean region variable for tests
Browse files Browse the repository at this point in the history
  • Loading branch information
d3vil-st committed Nov 21, 2024
1 parent 5a18cda commit 26de355
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 22 deletions.
1 change: 1 addition & 0 deletions .github/workflows/terraform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ jobs:

env:
TF_VAR_subnet_id: vpcsubnet-e00dgdntmhgkeej1z3
TF_VAR_region: eu-north1
TF_VAR_loki_access_key_id: ${{ secrets.SA_ACCESS_KEY_ID }}
TF_VAR_loki_secret_key: ${{ secrets.SA_SECRET_KEY }}

Expand Down
4 changes: 0 additions & 4 deletions k8s-inference/tests/main.tftest.hcl
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ run "k8s_inference_apply" {
]
}
variables {
region = "eu-north1"
etcd_cluster_size = 1
}
}
Expand All @@ -20,15 +19,13 @@ run "k8s_node_groups_inference_apply" {
]
}
variables {
region = "eu-north1"
etcd_cluster_size = 1
}
}

run "full_inference_apply" {
command = apply
variables {
region = "eu-north1"
etcd_cluster_size = 1
}
}
Expand All @@ -37,7 +34,6 @@ run "test_mode_k8s_inference_apply" {
command = apply

variables {
region = "eu-north1"
etcd_cluster_size = 1
test_mode = true
}
Expand Down
3 changes: 0 additions & 3 deletions k8s-training/tests/k8s-training-kuberay.tftest.hcl
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ run "k8s_training_kuberay_apply" {
]
}
variables {
region = "eu-north1"
etcd_cluster_size = 1
}
}
Expand All @@ -20,7 +19,6 @@ run "k8s_node_groups_training_kuberay_apply" {
]
}
variables {
region = "eu-north1"
etcd_cluster_size = 1
}
}
Expand All @@ -29,7 +27,6 @@ run "full_training_kuberay_apply" {
command = apply

variables {
region = "eu-north1"
etcd_cluster_size = 1
enable_loki = false # TODO: Disabling Loki since not possible to delete non-empty storage bucket
enable_kuberay = true
Expand Down
12 changes: 0 additions & 12 deletions k8s-training/tests/main.tftest.hcl
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
run "k8s_training_apply" {
command = apply

variables {
region = "eu-north1"
enable_loki = false # TODO: Disabling Loki since not possible to delete non-empty storage bucket
}

plan_options {
target = [
nebius_mk8s_v1_cluster.k8s-cluster
Expand All @@ -16,11 +11,6 @@ run "k8s_training_apply" {
run "k8s_node_groups_training_apply" {
command = apply

variables {
region = "eu-north1"
enable_loki = false # TODO: Disabling Loki since not possible to delete non-empty storage bucket
}

plan_options {
target = [
nebius_mk8s_v1_node_group.cpu-only,
Expand All @@ -33,7 +23,6 @@ run "full_training_apply" {
command = apply

variables {
region = "eu-north1"
enable_loki = false # TODO: Disabling Loki since not possible to delete non-empty storage bucket
}
}
Expand All @@ -42,7 +31,6 @@ run "test_mode_k8s_training_apply" {
command = apply

variables {
region = "eu-north1"
enable_loki = false # TODO: Disabling Loki since not possible to delete non-empty storage bucket
test_mode = true
}
Expand Down
3 changes: 0 additions & 3 deletions slurm/tests/main.tftest.hcl
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ run "slurm_master_apply" {
command = apply

variables {
region = "eu-north1"
cluster_workers_count = 2
}

Expand All @@ -17,7 +16,6 @@ run "slurm_full_apply" {
command = apply

variables {
region = "eu-north1"
cluster_workers_count = 2
}
}
Expand All @@ -26,7 +24,6 @@ run "test_mode_slurm_apply" {
command = apply

variables {
region = "eu-north1"
cluster_workers_count = 2
test_mode = true
}
Expand Down

0 comments on commit 26de355

Please sign in to comment.