Skip to content

Commit

Permalink
Added more checks on Redis-Container versions.
Browse files Browse the repository at this point in the history
  • Loading branch information
gwright99 committed Jan 9, 2025
1 parent 1d40267 commit de19c34
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 4 deletions.
27 changes: 23 additions & 4 deletions scripts/installer/validation/check_configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -541,15 +541,34 @@ def verify_alb_settings(data: SimpleNamespace):
"Use of a reverse proxy (`flag_use_custom_docker_compose_file = true`, cannot be combined with `flag_make_instance_private_behind_alb = true`. Please set only one of the options to true."
)


def verify_redis_version(data: SimpleNamespace):
"""Warn that versions of Seqera Platform >= 24.2 will default to Redis v7.0 container image."""

if (
data.tower_container_version >= "v24.2.0"
):
if data.tower_container_version >= "v24.2.0":

if data.flag_use_container_redis:

logger.warning(
"Seqera Platform version >= 24.2.0 uses a Redis v7.0 container (previously Redis v6.0)."
)

if data.flag_create_external_redis:
# TO DO
# As of Jan 9/25, the external redis are hardcoded into the `003-database.tf` file.
# This uses a redis 7.0 as baseline and thus is compliant to the needs of 24.2.2.
# In future this may need to change if Tower demands require a version > Redis 7.0 OR we
# choose to make the external redis values configurable.
logger.warning(
"The external Elasticache instance is hardcoded to use Redis 7.0."
)

else:
logger.warning(
"Seqera Platform version >= 24.2.0 will use the required Redis v7.0 (previously Redis v6.0)."
"When you upgrade to Seqera Platform >= v24.2, a Redis version >= 6.2 will be required."
)


# ------------------------------------------------------------------------------------
# MAIN
# ------------------------------------------------------------------------------------
Expand Down
2 changes: 2 additions & 0 deletions templates/TEMPLATE_terraform.tfvars
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,8 @@ flag_use_existing_external_db = false
flag_use_container_db = true

# Only one of these can be true.
# NOTE: Redis versions and ports are hard-coded in their respective files (docker-compose.yaml &
# 003-database.tf)
flag_create_external_redis = false
flag_use_container_redis = true

Expand Down

0 comments on commit de19c34

Please sign in to comment.