Skip to content

Commit

Permalink
Merge pull request #149 from seqeralabs/gwright99/verify-24-1-5
Browse files Browse the repository at this point in the history
Added email check for v24.1.4.
  • Loading branch information
gwright99 authored Sep 18, 2024
2 parents 1482692 + ad21a92 commit d677d7e
Showing 1 changed file with 19 additions and 0 deletions.
19 changes: 19 additions & 0 deletions scripts/installer/validation/check_configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -505,6 +505,24 @@ def verify_if_v24_1_3(data: SimpleNamespace):
)


def verify_if_v24_1_4(data: SimpleNamespace):
"""Warn that Tower v24.1.4 has email problem."""
if data.tower_container_version == "v24.1.4":
oidc_flags = [
data.flag_oidc_use_generic,
data.flag_oidc_use_google,
data.flag_oidc_use_github,
]
if any(oidc_flags):
logger.warning(
"Tower version 24.1.4 cannot send emails. You will only be able to use your OIDC option."
)
else:
log_error_and_exit(
"Tower version 24.1.4 cannot send emails. Please use v24.1.5 or higher."
)


def verify_connect_version_tls(data: SimpleNamespace):
"""Warn that Tower v24.1.4 cannot connect to Redis securely."""
if data.data_studio_container_version == "0.7.0":
Expand Down Expand Up @@ -550,6 +568,7 @@ def verify_alb_settings(data: SimpleNamespace):
verify_not_v24_1_1(data)
verify_not_v24_1_2(data)
verify_if_v24_1_3(data)
verify_if_v24_1_4(data)
verify_connect_version_tls(data)

# Verify tfvars fields
Expand Down

0 comments on commit d677d7e

Please sign in to comment.