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

ENDOC-801 7.3.1-new env vars EntApp CRD #827

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

ENDOC-801 7.3.1-new env vars EntApp CRD #827

wants to merge 2 commits into from

Conversation

jyunmitch
Copy link
Collaborator

***** Will not merge until 7.3.1 Release *****

|`spec.dbms` | Allowed values are: MySQL, PostgreSQL (default), Oracle, or embedded. Oracle is only supported as an external database.|
|`spec.ecrGitSshSecretName`| The configuration used by the Entando Component Manager to download bundles from authenticated Git repositories. It's a Secret containing a private key file named `rsa_id` that matches a public key configured in the authenticated Git repository.|
|`spec.environmentVariables`| A map of environment variables to pass to the EntandoApp Docker image. These variables pass to all components, and can sometimes be used as a mechanism to override any of the default environment variables that need customization.|

Choose a reason for hiding this comment

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

@jyunmitch

There is no EntandoApp docker image.

An EntandoApp is a only custom resource that is used to configure and manage 3 sub-modules (ComponentManager, AppEngine, AppBuilder) and their respective docker images.

The variables set under this property are propagated to all the 3 modules.

Choose a reason for hiding this comment

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

P.S.
unless you were referring to the EntandoApp controller

Copy link
Collaborator Author

@jyunmitch jyunmitch Mar 26, 2024

Choose a reason for hiding this comment

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

@w-caffiero-entando, that confused me a bit as well but it was there previously so I didn't mess with it. Will correct it now.

|`spec.ecrGitSshSecretName`| The configuration used by the Entando Component Manager to download bundles from authenticated Git repositories. It's a Secret containing a private key file named `rsa_id` that matches a public key configured in the authenticated Git repository.|
|`spec.environmentVariables`| A map of environment variables to pass to the EntandoApp Docker image. These variables pass to all components, and can sometimes be used as a mechanism to override any of the default environment variables that need customization.|
|`spec.environmentVariables`| A set of environment variables mapped to the Docker images of the App Builder, App Engine, and Component Manager. They can be used as a mechanism to override any of the default environment variables that need customization.|
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@w-caffiero-entando, Is this better? can you approve this PR if this is okay?

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.

3 participants