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.

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

IZE-598 added generation of markdown files with IZE specification #491

Conversation

psihachina
Copy link
Contributor

@psihachina psihachina commented Oct 10, 2022

Change:

added generation of markdown files with IZE specification

Test:

asciicast

Ecs Section

Parameter Required Description
aws_profile no (optional) ECS-specific AWS profile (optional) can be specified here (but normally it should be inherited from a global AWS_PROFILE).
aws_region no (optional) ECS-specific AWS Region of this environment should be specified here. Normally global AWS_REGION is used.
cluster no (optional) ECS cluster can be specified here. By default it's derived from env & namespace
depends_on no (optional) expresses startup and shutdown dependencies between apps
docker_registry no (optional) Docker registry can be set here. By default it uses ECR repo with the name of the service.
icon no (optional) set icon
image no (optional) Docker image can be specified here. By default it's derived from the app name.
path no (optional) Path to ecs app folder can be specified here. By default it's derived from apps path and app name.
skip_deploy no (optional) skip deploy app.
task_definition_revision no (optional) Task definition revision can be specified here. By default latest revision is used to perform a deployment. Normally this parameter can be used via cli during specific deployment needs.
timeout no (optional) ECS deployment timeout can be specified here.
unsafe no (optional) Enables unsafe mode that increases deploy time on a cost of shorter healthcheck.

Main Section

Parameter Required Description
alias no (optional) Alias mode can be enabled here. This can be used to combine various apps via depends_on parameter.
apps_path no (optional) Path to apps directory can be set. By default apps are searched in 'apps' and 'projects' directories. This is needed in case your repo structure is not purely ize-structured (let's say you have 'src' repo in your dotnet app, as an example)
aws_profile yes (optional) AWS Profile can be specified here (but normally it's specified via AWS_PROFILE env var)
aws_region yes (required) AWS Region of this environment should be specified here. Can be overridden by AWS_PROFILE env var or --aws-region flag.
config_file no (optional) Path to ize.toml config file can be specified, but normally it's read from the environment's directory automatically.
custom_prompt no (optional) Custom prompt can be enabled here for all console connections. Default: false.
docker_registry no (optional) Docker registry can be set here. By default it uses ECR repo with the name of the service.
ecs no Ecs apps configuration.
env yes (optional) Environment name can be specified here. Normally it should be passed via ENV variable or --env flag.
env_dir yes (optional) Environment directory can be specified here. Normally it's calculated automatically based on the directory structure convention.
home yes (optional) User home directory can be specified here. Normally $HOME is used.
infra no Infrastructure configuration.
ize_dir yes (optional) Ize directory can be specified here. Normally it's assumed to be .infra or .ize in the current repo.
log_level no (optional) Log level can be specified here. Possible levels: info, debug, trace, panic, warn, error, fatal(default). Can be overridden via IZE_LOG_LEVEL env var or via --log-level flag.
namespace yes (required) Namespace of the project can be specified here. It is used as a base for all naming. It can be overridden by NAMESPACE env var or --namespace flag.
plain_text no (optional) Plain text output can be enabled here. Default is false. Can be overridden by IZE_PLAIN_TEXT env var or --plain-text flag.
prefer_runtime yes (optional) Prefer a specific runtime. (native or docker) (default 'native')
root_dir yes (optional) Project directory can be set here. By default it's the current directory, but in case you prefer to run ize from the outside of repo it may be useful (uncommon).
serverless no Serverless apps configuration.
tag no (optional) Tag can be set statically. Normally it is being constructed automatically based on the git revision.
terraform no Terraform configuration.
terraform_version no (optional) Terraform version can be set here. 1.1.3 by default
tf_log no (optional) Terraform TF_LOG can be set here. Can be TRACE, DEBUG, INFO, WARN or ERROR.
tf_log_path no (optional) TF_LOG_PATH can be set here.
tunnel no Tunnel configuration.

@codecov-commenter
Copy link

Codecov Report

Merging #491 (33654b2) into main (71734a8) will decrease coverage by 0.35%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #491      +/-   ##
==========================================
- Coverage   30.85%   30.50%   -0.36%     
==========================================
  Files          60       60              
  Lines        7545     7619      +74     
==========================================
- Hits         2328     2324       -4     
- Misses       4995     5073      +78     
  Partials      222      222              
Impacted Files Coverage Δ
internal/commands/doc.go 11.86% <0.00%> (-23.14%) ⬇️
internal/schema/schema.go 27.14% <0.00%> (-34.15%) ⬇️
internal/commands/initialize.go 64.13% <0.00%> (-0.60%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

internal/commands/doc.go Outdated Show resolved Hide resolved
Copy link
Contributor

@igorkotof igorkotof left a comment

Choose a reason for hiding this comment

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

LGTM, if @AutomationD's comments are fixed

@psihachina psihachina force-pushed the IZE-598-schema-is-used-to-create-markdown-file-with-ize-toml-structure-in-website-docs branch from 33654b2 to e433190 Compare October 10, 2022 15:05
Copy link
Contributor

@AutomationD AutomationD left a comment

Choose a reason for hiding this comment

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

Should it be like that?

internal/commands/doc.go Outdated Show resolved Hide resolved
@AutomationD
Copy link
Contributor

@psihachina also, remember, @kobrikx will be moving out website directory from ize repo, so maybe something may break if directory won't be existing any more.

Co-authored-by: Dmitry Kireev <dmitry@hazelops.com>
Copy link
Contributor

@AutomationD AutomationD left a comment

Choose a reason for hiding this comment

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

Thank you @psihachina, clean code, as always.

@psihachina psihachina merged commit 8a973ac into main Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants