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-533 Revamp Build and Publish a Simple Bundle #545

Merged
merged 8 commits into from
Aug 16, 2022
Merged

Conversation

Lyd1aCla1r3
Copy link
Contributor

No description provided.

3. Apply the bundle custom resource to Kubernetes
4. Install the bundle into an application

The sections below detail the steps required to generate simple examples of each Entando bundle type.
Copy link
Collaborator

Choose a reason for hiding this comment

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

what bundle type? may be good to supply idea about the types

detail the steps >> slightly awkward

Copy link
Contributor Author

@Lyd1aCla1r3 Lyd1aCla1r3 Aug 11, 2022

Choose a reason for hiding this comment

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

this is a leftover fragment from copy/paste; i'd started integrating v5 into this page before we changed course.

touch widgets/example-widget.yaml
```

3. Populate the widget descriptor file `example-widget.yaml` with a simple definition. Retain the correct YAML indentation of 2 or 4 spaces.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Retain the correct YAML indentation of 2 or 4 spaces. >> I know what it is saying but it feels like some context is needed. What does YAML indentation in and of itself mean? YAML indentation format/rules?

widgets:
- widgets/example-widget.yaml
```
Component descriptor file names and locations (e.g. `widgets/example-widget.yaml`) are arbitrary because the bundle descriptor explicitly points to those files. Convention is to group components by type with all widgets in one directory, all page templates in another, etc.
Copy link
Collaborator

Choose a reason for hiding this comment

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

would prefer this info before the example. easier to follow. Same above. Prefer all the information clumped together before the example. Otherwise it feels broken up. But not a big deal, probably personal preference

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 think so. each step is a directive. supplementary details not critical to the execution of the instruction are noted afterwards. i had thought we decided to write each step clearly/concisely to provide the more savvy readers with quick one-liners; generally developers want to see only the pertinent info and not wade through paragraphs.

Copy link
Collaborator

Choose a reason for hiding this comment

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

No we never decided to divide up the instructions like that. We did agree to write the steps and everything else clearly & concisely

``` sh
ent bundler from-git --name=example-bundle --namespace=entando --repository=https://YOUR-REMOTE-REPOSITORY.git --dry-run > example-bundle.yaml
```
A bundle must be published to Git before you can create a custom resource for it. Provide the URL of the bundle's remote Git repository (ends with ".git"). To include a thumbnail for your bundle, use the option `--thumbnail-file` or `--thumbnail-url`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

(ends with ".git") >> repeated above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no direct repetition. i think this is left over from us wanting to make it really clear that you have to use that particular url form


7. Confirm the presence of your custom resource with the command `kubectl get EntandoDeBundle -n entando`.
2. Click `Install`. A bundle with multiple iterations allows version selection.
Copy link
Collaborator

Choose a reason for hiding this comment

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

A bundle with multiple iterations allows version selection. >> would prefer a simpler statement about selecting versions. Unless they already know the process, I would worry that non-english speakers would be confused by this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this note is good to know before the user hits "install" so i kept it with the instruction, which should be short and sweet. "iteration" is a well-known term/concept in software.


## Install the Bundle into an Application
Your bundle should appear in `App Builder` → `Component Repository` in your Entando instance. Clicking `Install` should allow version selection if your bundle has multiple iterations.
3. Verify the `Install` button changes to `Uninstall` to indicate that the installation completed successfully
Copy link
Collaborator

Choose a reason for hiding this comment

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

simplify >> the installation is successful when the button reads Uninstall

Copy link
Contributor Author

Choose a reason for hiding this comment

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

each step should be an instruction, telling the user to do something

@Lyd1aCla1r3 Lyd1aCla1r3 marked this pull request as ready for review August 12, 2022 16:03
@Lyd1aCla1r3 Lyd1aCla1r3 requested a review from nshaw August 12, 2022 16:03
This tutorial describes how to create a simple Entando Bundle and deploy it into the [Entando Component Repository](../../../docs/getting-started/concepts-overview.md#entando-component-repository) (ECR). This process consists of 4 essential steps:

1. Define the bundle component(s)
2. Check the bundle artifacts into a Git repository
Copy link
Member

Choose a reason for hiding this comment

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

This step is incorrect for 7.1. Change to "Publish the bundle artifacts to a Docker registry."


2. Add the following definition to `descriptor.yaml`:
``` yaml
code: example-bundle
Copy link
Member

Choose a reason for hiding this comment

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

I think it had to be an underscore in the name in the final version but we haven't finished testing with a widget...

@Lyd1aCla1r3 Lyd1aCla1r3 merged commit 9eb8971 into main Aug 16, 2022
@nshaw nshaw deleted the ENDOC-533 branch August 26, 2022 18:16
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