From 3fe754167ebc20fca8221b5262372857ff0d5fbf Mon Sep 17 00:00:00 2001 From: Simone Ruffilli Date: Thu, 23 May 2024 09:43:23 +0200 Subject: [PATCH] FAST: Clarify max prefix len for tenant factory --- fast/stages/1-tenant-factory/README.md | 2 +- tools/check_names.py | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/fast/stages/1-tenant-factory/README.md b/fast/stages/1-tenant-factory/README.md index 5abc3b323..e736feb33 100644 --- a/fast/stages/1-tenant-factory/README.md +++ b/fast/stages/1-tenant-factory/README.md @@ -154,7 +154,7 @@ This stage has only three variables that can be customized: A small number of attributes can be configured for each tenant in `tenant_configs` regardless of its type (simple or FAST-enabled). -The key in the tenant map is used as the tenant shortname, and should be selected with care as it becomes part of resource names. If the tenant plans on using FAST stages it should not exceed 3 characters, unless a custom prefix is also defined for the tenant. +The key in the tenant map is used as the tenant shortname, and should be selected with care as it becomes part of resource names. If the tenant plans on using FAST stages, the total combined length of string `{fast-prefix}-{tenant-shortname}` should not exceed 11 characters combined, unless a custom prefix is also defined for the tenant. `admin_principal` is a IAM-format principal (e.g. `"group:tenant-admins@example.org"`) which is assigned administrative permissions on the tenant environment, and impersonation permissions on the automation service account. diff --git a/tools/check_names.py b/tools/check_names.py index 47f21c8d5..180d72234 100755 --- a/tools/check_names.py +++ b/tools/check_names.py @@ -72,6 +72,7 @@ def get_names(dir_name): @click.command() @click.argument('dirs', type=str, nargs=-1) +# max len(f'{fast prefix max length = 9}-{tenant prefix}') = 11 @click.option('--prefix-length', default=11, type=int) @click.option('--failed-only', is_flag=True, default=False) def main(dirs, prefix_length=None, failed_only=False):