Conversation
sghosh23
left a comment
There was a problem hiding this comment.
Looks good and I left few comments
src/how-to/install/demo-wiab.md
Outdated
| ### What will be installed? | ||
|
|
||
| - Wire-server (API) | ||
| - user accounts, authentication, conversations |
There was a problem hiding this comment.
| - user accounts, authentication, conversations | |
| - user accounts | |
| - authentication | |
| - conversations |
There was a problem hiding this comment.
they are core features and are clubbed together as basic APIs, carry forwarded from old documentation
There was a problem hiding this comment.
That's okay, since we are listing then why grouping them together with comma that was my point
There was a problem hiding this comment.
added them as core - user accounts, authentication, conversations
src/how-to/install/demo-wiab.md
Outdated
| - Email relay service i.e. demo-smtp | ||
| - Group calling component i.e. coturn | ||
| - Ephemeral datastores in k8s | ||
| - A cert-manager with `letsencrypt` as `issuser`. As an alternative to providing your own certificate, you may want to allow for automated certificate issuing through Let’s Encrypt. |
There was a problem hiding this comment.
We do need to phrase this As an alternative to providing your own certificate, you may want to allow for automated certificate issuing through Let’s Encrypt. here on the list?
A good candidate to be in its own section such as Benefits of using letsencrypt
There was a problem hiding this comment.
SSL/TLS certificates:
- a way to create SSL/TLS certificates for your domain name (to allow connecting via https://)
- To ease out the process of managing certs, we recommend using Let's Encrypt &
cert-manager
--- it is being talked in planning.md file which is starting point of this documentation:
https://github.com/wireapp/wire-docs/pull/60/files#diff-c73153d5e1b44f4727a457aae0f635e2922253659db169e31ca57b55609b5ff9R22
There was a problem hiding this comment.
Yes, may be then we can omit this extra info regarding letsencrypt here as it feels like a repetition. When I am reading this i am asking this question to myself, is there a way to disallow the letsencrypt here and bring my own cert.
And the title says What will be installed? So I would just expect a list here and if the items on the list deserve description there can be another description section (that you already have in the planning to some extent) which can be linked to the list items.
There was a problem hiding this comment.
Got the idea, removed the description and keeping it simple as
A cert-manager withletsencryptasissuser.
src/how-to/install/demo-wiab.md
Outdated
| - highly available k8s cluster | ||
|
|
||
| ### Diagram | ||
| A demo installation will look a bit like this: |
There was a problem hiding this comment.
| A demo installation will look a bit like this: | |
| The flow diagram of the Demo setup |
src/how-to/install/demo-wiab.md
Outdated
|
|
||
| This guide provides detailed instructions for deploying a Demo Wire-in-a-Box (WIAB) using Ansible on an Ubuntu 24.04 system. | ||
|
|
||
| The deployment process is structured into multiple blocks within the Ansible playbook, offering flexibility in execution. It is designed to configure a remote node, such as example.com (referred to as deploy_node), to install Wire with a custom domain, example.com (referred to as target_domain). |
There was a problem hiding this comment.
is it required for the user to know how the ansible playbook is designed? If so please rephrase this sentence:
It is designed to configure a remote node, such as example.com (referred to as deploy_node), to install Wire with a custom domain, example.com (referred to as target_domain).
Is the example.com referred as both deploy_node and target_domain ?
There was a problem hiding this comment.
yes, I am trying to give an idea on what to expect when running the ansible-playbook.
src/how-to/install/demo-wiab.md
Outdated
|
|
||
| These variables must be verified in the file [ansible/inventory/demo/host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) before running the pipeline. | ||
|
|
||
| Typically, the deployment process runs seamlessly without requiring any external flags. However, if needed, you have the option to skip certain tasks based on their conditional flags. |
There was a problem hiding this comment.
| Typically, the deployment process runs seamlessly without requiring any external flags. However, if needed, you have the option to skip certain tasks based on their conditional flags. | |
| Typically, the deployment process runs seamlessly without requiring any external flags. However, if needed, you have the option to skip certain tasks by passing the conditional flags defined in the task. |
src/how-to/install/demo-wiab.md
Outdated
|
|
||
| ## Deployment requirements: | ||
| - Clone of [wire-server-repository](https://github.com/wireapp/wire-server-deploy) | ||
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) (post cloning the previous repo) to update and verify the following default variables: |
There was a problem hiding this comment.
(post cloning the previous repo) is confusing here. is it wire-server-repository repo? May be omit this part from the sentence. The link of the hosts.yaml takes to that file. Or can be rephrased as:
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) (post cloning the previous repo) to update and verify the following default variables: | |
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) in the wire-server-deploy needs to be verified and updated with the following default variables: |
src/how-to/install/demo-wiab.md
Outdated
| ## Deployment requirements: | ||
| - Clone of [wire-server-repository](https://github.com/wireapp/wire-server-deploy) | ||
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) (post cloning the previous repo) to update and verify the following default variables: | ||
| - ansible_host: aka **deploy_node** i.e. IP address or hostname of VM where Wire will be deployed (Mandatory) |
There was a problem hiding this comment.
| - ansible_host: aka **deploy_node** i.e. IP address or hostname of VM where Wire will be deployed (Mandatory) | |
| - ansible_host: aka **deploy_node** i.e. IP address or hostname of the VM where Wire will be deployed (Mandatory) |
src/how-to/install/demo-wiab.md
Outdated
| ### Ansible run selective tasks | ||
| - You can use ^skip_ variables as environment variables to control the execution flow of the playbook. If these variables are passed, they will skip specific groups of tasks as explained in the [Deployment Flow](#deployment-flow) section. By default, if no variables are passed, all tasks will run in sequence. | ||
|
|
||
| In case of timeouts or any failures, you can skip tasks that have already been completed by passing the appropriate flags. For example, if the Wire artifact download task fails due to a timeout or disk space issue, you can skip the tasks that come before the Wire Artifact Download by using the following command: |
There was a problem hiding this comment.
| In case of timeouts or any failures, you can skip tasks that have already been completed by passing the appropriate flags. For example, if the Wire artifact download task fails due to a timeout or disk space issue, you can skip the tasks that come before the Wire Artifact Download by using the following command: | |
| In case of timeouts or other failures, you can skip tasks that have already been completed by passing the appropriate flags. For example, if the Wire artifact download task fails due to a timeout or a disk space issue, you can skip the preceding tasks by using the following command: |
src/how-to/install/planning.md
Outdated
| - a way to create SSL/TLS certificates for your domain name (to allow connecting via https://) | ||
| - To ease out the process of managing certs, we recommend using [Let\'s Encrypt](https://letsencrypt.org/getting-started/) & | ||
| [cert-manager](https://cert-manager.io/docs/tutorials/acme/http-validation/) | ||
| - Find a detailed explanation |
| ### What will not be installed? | ||
|
|
||
| - notifications over native push notifications via [FCM](https://firebase.google.com/docs/cloud-messaging/)/[APNS](https://developer.apple.com/notifications/) | ||
| - persistent datastores in k8s |
src/how-to/install/demo-wiab.md
Outdated
|
|
||
| - notifications over native push notifications via [FCM](https://firebase.google.com/docs/cloud-messaging/)/[APNS](https://developer.apple.com/notifications/) | ||
| - persistent datastores in k8s | ||
| - highly available k8s cluster |
There was a problem hiding this comment.
| - highly available k8s cluster | |
| - high availablity |
src/how-to/install/demo-wiab.md
Outdated
| For more detailed instructions on each task, please refer to the [Deployment Flow section](#deployment-flow). | ||
|
|
||
| ## Deployment requirements: | ||
| - Clone of [wire-server-repository](https://github.com/wireapp/wire-server-deploy) |
There was a problem hiding this comment.
| - Clone of [wire-server-repository](https://github.com/wireapp/wire-server-deploy) | |
| - Clone of [wire-server-deploy repository](https://github.com/wireapp/wire-server-deploy) |
src/how-to/install/demo-wiab.md
Outdated
|
|
||
| ## Deployment requirements: | ||
| - Clone of [wire-server-repository](https://github.com/wireapp/wire-server-deploy) | ||
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) in the wire-server-deploy needs to be verified and updated with the following default variables: |
There was a problem hiding this comment.
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) in the wire-server-deploy needs to be verified and updated with the following default variables: | |
| - The inventory file [host.yml](https://github.com/wireapp/wire-server-deploy/blob/master/ansible/inventory/demo/host.yml) in the wire-server-deploy needs to be verified and updated with the following default variables: |
|
|
||
| ### Iptables behaviour | ||
|
|
||
| - All the iptables rules are not persisted after reboots, but they can be regenerated by running the entire pipeline or restored from the `/home/ansible_user/wire-iptables-rules/rules_post_wire.v4` directory. Optionally, we can skip everything else when generating the rules again. |
There was a problem hiding this comment.
restored from the... so, just CDing to that directory restores iptables rules, eh? :)
There was a problem hiding this comment.
yes, this is for the restart behavior, in case someone want to preserve the state post restart. They can use manual command or re-run the ansible-playbook to restore iptables rules.
Change type
Basic information
Testing
Tracking