Skip to content

Commit

Permalink
Merge pull request #73 from gonzalezzfelipe/fix/playbook-error
Browse files Browse the repository at this point in the history
fix: Playbook error
  • Loading branch information
scarmuega authored Nov 25, 2024
2 parents c65e1e0 + 446c16a commit 9fcdc30
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 2 deletions.
12 changes: 12 additions & 0 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -134,8 +134,17 @@ jobs:
matrix:
include:
- region: us-east-1
region_prefix: a
autoscaler_high_watermark: 5
autoscaler_low_watermark: 1
- region: eu-central-1
region_prefix: b
autoscaler_high_watermark: 5
autoscaler_low_watermark: 1
- region: us-west-2
region_prefix: c
autoscaler_high_watermark: 5
autoscaler_low_watermark: 1

env:
# Secrets
Expand All @@ -157,6 +166,9 @@ jobs:
TF_VAR_frontend_image: ghcr.io/cardano-scaling/hydra-doom-frontend:sha-a786f53
TF_VAR_referee_image: ghcr.io/cardano-scaling/hydra-doom-referee:sha-f68f1d4 # latest?
TF_VAR_ai_image: ghcr.io/cardano-scaling/hydra-doom-ai:sha-f68f1d4 # latest?
TF_VAR_autoscaler_high_watermark: ${{ matrix.autoscaler_high_watermark }}
TF_VAR_autoscaler_low_watermark: ${{ matrix.autoscaler_low_watermark }}
TF_VAR_autoscaler_region_prefix: ${{ matrix.region_prefix }}

steps:
- name: Filter regions
Expand Down
2 changes: 1 addition & 1 deletion crates/operator/src/controller.rs
Original file line number Diff line number Diff line change
Expand Up @@ -483,7 +483,7 @@ impl K8sContext {
"{}{}{}",
self.config.autoscaler_region_prefix,
"0", // 1 for online, 0 for offline
random_name()
random_name().to_lowercase()
);
let new_node = HydraDoomNode {
spec: HydraDoomNodeSpec::default(),
Expand Down
2 changes: 1 addition & 1 deletion playbook/doom-dev/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ variable "autoscaler_low_watermark" {
default = 1
}

variable "autoscaler_low_watermark" {
variable "autoscaler_high_watermark" {
type = number
default = 5
}
Expand Down

0 comments on commit 9fcdc30

Please sign in to comment.