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-525 Create new ent CLI Services page #541

Merged
merged 4 commits into from
Aug 1, 2022
Merged

ENDOC-525 Create new ent CLI Services page #541

merged 4 commits into from
Aug 1, 2022

Conversation

Lyd1aCla1r3
Copy link
Contributor

No description provided.

@Lyd1aCla1r3 Lyd1aCla1r3 requested review from jyunmitch and nshaw July 28, 2022 19:31
- The username and password are derived from the bundle name, e.g. YOUR-BUNDLE has the username YOUR-BUNDLE-USER and the password YOUR-BUNDLE-PASSWORD
- A database is automatically created and given the bundle name, e.g. YOUR-BUNDLE

- `keycloak` contains the Entando-related configuration
Copy link
Collaborator

Choose a reason for hiding this comment

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

this phrase is left hanging? Colon?

Entando-related configuration >> Entando configuration

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i didn't use colons because the first keycloak bullet provides general info but the indents provide unrelated, additional details. a colon fits for mysql/postgresql (indents expand upon first bullet) but i wanted to be consistent.

this one bothered me. both seem wrong for different reasons. @nshaw does the revision work?

Copy link
Contributor

Choose a reason for hiding this comment

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

Formatting looks okay when I view the full file


## Commands

Use `ent bundle svc` to enable, disable, start, stop, restart, print the logs of, or list services. Services must be formatted as Docker Compose YAML files in the 'svc' bundle folder.
Copy link
Collaborator

Choose a reason for hiding this comment

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

this seems unnecessary as it is listed all below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

doesn't do any harm and introduces the ent bundle svc command


**Command details:**

- `svc enable`: Adds the service(s) to the 'svc' attribute of the bundle descriptor
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think this description belongs in the table above and no additional details are necessary. This one is more clear what it does.

Same for disable. The aux services is not clear> they have to go looking for what is aux services

Copy link
Contributor Author

Choose a reason for hiding this comment

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

?? we're talking about aux services. these are all aux services, but only if they're added to the attribute and follow the required file format / dir placement. otherwise they may be services but they're not an aux service of the bundle. did the reverse because the intro paragraph calls out the svc attribute

Copy link
Collaborator

Choose a reason for hiding this comment

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

was just pointing out that you end up having to read the table and the details below. I think the details section would be unnecessary if those descriptions were put in the table.

Maybe I'm confused, but are there other services besides aux services

Copy link
Contributor Author

Choose a reason for hiding this comment

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

? i already edited this

services is a very broad term, so absolutely


- `svc disable`: Removes the service(s) from the 'svc' attribute of the bundle descriptor

- `svc logs`: The log of a running service will continue to update until exited via `Ctrl+C`
Copy link
Collaborator

Choose a reason for hiding this comment

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

until exited via Ctrl+C>> sounds awkward and don't think we need to back tick Ctrl-C

Copy link
Contributor Author

Choose a reason for hiding this comment

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

disagree

we decided to backtick Ctrl+C a few months ago, and to use + instead of -

Copy link
Collaborator

Choose a reason for hiding this comment

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

got the sign wrong, meant to do + . Pretty sure we just decided on Ctrl+C but the backtick is not a big deal. (4-5-22)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that decision doesn't make any mention of backticking, so i'm going off of how ctrl+c is usually formatted on the developer site. i have no strong feelings about this. just aiming for consistency.

- All roles and permissions are preset, based on the Keycloak realm-config file
- The `keycloak.yaml`, realm-config and user config files are located in the 'keycloak' folder inside 'svc'

A custom service can be added to a bundle's 'svc' folder as a Docker Compose YAML file. The file and service must have the same name, e.g. if the service filename is `mysql.yml`, then the service name must be `mysql`:
Copy link
Collaborator

Choose a reason for hiding this comment

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

a custom service can be added 'as' a Docker Compose... >> as sounds odd, maybe with?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the service is the file

Copy link
Contributor

Choose a reason for hiding this comment

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

A Docker Compose file can contain multiple things. In this case it looks like Eng has a convention of 1 Docker Compose YAML File defining a single named service. So 'as' works for me.


Use `ent bundle svc` to enable, disable, start, stop, restart, print the logs of, or list services. Services must be formatted as Docker Compose YAML files in the 'svc' bundle folder.

To run auxiliary operations, a service must be enabled. Enabled services are listed under the 'svc' attribute of the bundle descriptor, `entando.json`. Common auxiliary service commands are detailed below.
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure about 'auxiliary operations' here >> simplify >> a service must be enabled before it can be used

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the service performs auxiliary operations. the current wording is precise and connects the dots.

Copy link
Contributor

Choose a reason for hiding this comment

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

This explanation does help me connect it all.

- All roles and permissions are preset, based on the Keycloak realm-config file
- The `keycloak.yaml`, realm-config and user config files are located in the 'keycloak' folder inside 'svc'

A custom service can be added to a bundle's 'svc' folder as a Docker Compose YAML file. The file and service must have the same name, e.g. if the service filename is `mysql.yml`, then the service name must be `mysql`:
Copy link
Contributor

Choose a reason for hiding this comment

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

A Docker Compose file can contain multiple things. In this case it looks like Eng has a convention of 1 Docker Compose YAML File defining a single named service. So 'as' works for me.


Use `ent bundle svc` to enable, disable, start, stop, restart, print the logs of, or list services. Services must be formatted as Docker Compose YAML files in the 'svc' bundle folder.

To run auxiliary operations, a service must be enabled. Enabled services are listed under the 'svc' attribute of the bundle descriptor, `entando.json`. Common auxiliary service commands are detailed below.
Copy link
Contributor

Choose a reason for hiding this comment

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

This explanation does help me connect it all.

- The username and password are derived from the bundle name, e.g. YOUR-BUNDLE has the username YOUR-BUNDLE-USER and the password YOUR-BUNDLE-PASSWORD
- A database is automatically created and given the bundle name, e.g. YOUR-BUNDLE

- `keycloak` contains the Entando-related configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatting looks okay when I view the full file


# Auxiliary Services

The following sections discuss auxiliary services and commands.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it was listed elsewhere but I think we need to call out that these "auxiliary" services are purely for a local development environment. In Kubernetes these services are provided by the platform (e.g. keycloak or RedHat SSO) or configuration settings can be enabled to point to externally managed services (like mysql or postgresql). It took me a minute to sort that out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i really don't remember reading that anywhere and it gives some missing context/explanation

@@ -4,7 +4,9 @@ sidebarDepth: 1

# Auxiliary Services

The following sections discuss auxiliary services and commands.
Auxiliary services extend the capabilities of a local development environment. Kubernetes can be configured to point to externally managed services, such as MySQL or PostgreSQL, or services are provided by the platform (e.g. Keycloak or RedHat SSO).
Copy link
Contributor

Choose a reason for hiding this comment

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

Getting closer. Maybe this is even clearer "Auxiliary services add capabilities to a local environment to help accelerate application development. In a full Kubernetes setting, by comparison, the equivalent services are either provisioned by the Entando platform or the application can be configured to point to externally managed services."

@nshaw nshaw merged commit 12f53aa into ENDOC-522 Aug 1, 2022
@nshaw nshaw deleted the ENDOC-525 branch August 1, 2022 16:45
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