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.

Sign up for GitHub

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

refactor(kupo): split storage classes per network #193

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

refactor(kupo): split storage classes per network #193

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ccbb4c7
refactor: split kupo storage classes per network
overcookedpanda Feb 11, 2025
e221d85
update preview and preprod to use ssd disk
overcookedpanda Feb 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions stage3/kupo.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ module "ext_cardano_kupo" {
"cell1" = {
pvc = {
storage_size = var.kupo_v1_storage_size_preview
storage_class_name = var.kupo_v1_storage_class_name
storage_class_name = var.kupo_v1_storage_class_name_preview
access_mode = "ReadWriteOnce"
}
instances = {
Expand All @@ -91,7 +91,7 @@ module "ext_cardano_kupo" {
effect = "NoSchedule"
key = "demeter.run/compute-profile"
operator = "Equal"
value = "mem-intensive"
value = "disk-intensive"
},
{
effect = "NoSchedule"
Expand All @@ -118,7 +118,7 @@ module "ext_cardano_kupo" {
"cell2" = {
pvc = {
storage_size = var.kupo_v1_storage_size_preprod
storage_class_name = var.kupo_v1_storage_class_name
storage_class_name = var.kupo_v1_storage_class_name_preprod
access_mode = "ReadWriteOnce"
}
instances = {
Expand All @@ -144,7 +144,7 @@ module "ext_cardano_kupo" {
effect = "NoSchedule"
key = "demeter.run/compute-profile"
operator = "Equal"
value = "mem-intensive"
value = "disk-intensive"
},
{
effect = "NoSchedule"
Expand All @@ -171,7 +171,7 @@ module "ext_cardano_kupo" {
"cell3" = {
pvc = {
storage_size = var.kupo_v1_storage_size_mainnet
storage_class_name = var.kupo_v1_storage_class_name
storage_class_name = var.kupo_v1_storage_class_name_mainnet
access_mode = "ReadWriteOnce"
}
instances = {
Expand All @@ -197,7 +197,7 @@ module "ext_cardano_kupo" {
effect = "NoSchedule"
key = "demeter.run/compute-profile"
operator = "Equal"
value = "mem-intensive"
value = "disk-intensive"
},
{
effect = "NoSchedule"
Expand Down
14 changes: 12 additions & 2 deletions stage3/variables.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,19 @@ variable "kupo_v1_storage_size_mainnet" {
default = "50Gi"
}

variable "kupo_v1_storage_class_name" {
variable "kupo_v1_storage_class_name_preview" {
description = "name of the storage class for the kupo extension"
default = "hyperdisk-balanced-immediate"
default = "premium-rwo"
}

variable "kupo_v1_storage_class_name_preprod" {
description = "name of the storage class for the kupo extension"
default = "premium-rwo"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The idea is that defaults are set for cluster_provider: aws. We should add variables to env.auto.tfvars.example and override for personal setup.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, in that case the current single variable is incorrect also. I'll push a commit tomorrow to share the example for override and update the values to be default for aws.

}

variable "kupo_v1_storage_class_name_mainnet" {
description = "name of the storage class for the kupo extension"
default = "premium-rwo"
}

# Ogmios extension
Expand Down