-
Notifications
You must be signed in to change notification settings - Fork 45
wpb-22439 7 docs wiab staging #851
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
base: wpb-22439-postgresql-repmgr_node_config
Are you sure you want to change the base?
wpb-22439 7 docs wiab staging #851
Conversation
594b922 to
28b9f70
Compare
ea69e50 to
6dea77e
Compare
7b39bdd to
9e1ee11
Compare
6dea77e to
be20a6c
Compare
21281f9 to
a9032df
Compare
be20a6c to
6ded955
Compare
a9032df to
7f9ddf0
Compare
668a27c to
7c8f95f
Compare
| Added: terraform resources for wiab-staging | ||
| Added: cd_staging script to verify the default build bundle | ||
| Changed: changed the flow of offline.yml - introduced wiab-staging build and split bundle processing with default-build | ||
| Added: wiab-staging documetation to wire-server-deploy and fix coturn port ranges |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo
offline/wiab-staging.md
Outdated
| - One physical machine with hypervisor support: | ||
| - **Memory:** 55 GiB RAM | ||
| - **Compute:** 29 vCPUs | ||
| - **Storage:** 550 GB disk space (thin-provisioned) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
inconsistent requirements #1
| | assethost | Asset/Storage Server | 4 GiB | 2 | 100 GB | | ||
| | kubenode1 | Kubernetes Node 1 | 9 GiB | 5 | 150 GB | | ||
| | kubenode2 | Kubernetes Node 2 | 9 GiB | 5 | 150 GB | | ||
| | kubenode3 | Kubernetes Node 3 | 9 GiB | 5 | 150 GB | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
inconsistent requirements #2
| - Source the `bin/offline-env.sh` shell script to set up a `d` alias that runs commands inside a Docker container with all necessary tools for offline deployment. | ||
|
|
||
| - **[Generating secrets](docs_ubuntu_22.04.md#generating-secrets)** | ||
| - Run `./bin/offline-secrets.sh` to generate fresh secrets for Minio and coturn services. This creates two secret files: `ansible/inventory/group_vars/all/secrets.yaml` and `values/wire-server/secrets.yaml`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
values/wire-server/secrets.yaml does not get generated anymore, now its prod-secrets.example.yaml, which also brings the question, why ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#858 - it is generated here and this PR came post creating this documentation, and the idea is to create a copy first from prod-secrets.example.yaml to secrets.yaml for each chart and make changes directly to this secrets.yaml and values.yaml, so that an admin can understand what changes has been done by our scripts. As of now, the changes are going to prod-secrets.example.yaml in #858 I will patch it to make offline-secrets run post creating the copies.
| - **Deploying Helm charts** | ||
| - **[Deploying stateless services and other dependencies](docs_ubuntu_22.04.md#deploying-stateless-dependencies)** | ||
| - Deploy cassandra-external, elasticsearch-external, minio-external, and databases-ephemeral helm charts to set up connections to external data services and stateless database dependencies. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you are missing rabbitmq here, which causes the deployment to fail
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rabbitmq is coming from the helm chart, In the recent failure, i was trying to enable it again. As of this PR, rabbitmq is getting deployed using helm chart.
https://github.com/wireapp/wire-server-deploy/blob/master/bin/offline-cluster.sh#L55
offline/wiab-staging.md
Outdated
|
|
||
| - **Deploying Helm charts** | ||
| - **[Deploying stateless services and other dependencies](docs_ubuntu_22.04.md#deploying-stateless-dependencies)** | ||
| - Deploy cassandra-external, elasticsearch-external, minio-external, and databases-ephemeral helm charts to set up connections to external data services and stateless database dependencies. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reaper does not deploy
Failed to pull image "docker.io/bitnamilegacy/kubectl:1.32.9": rpc error: code = NotFound desc = failed to pull and unpack image "docker.io/bitnamilegacy/kubectl:1.32.9": failed to resolve reference "docker.io/bitnamilegacy/kubectl:1.32.9": docker.io/bitnamilegacy/kubectl:1.32.9: not found
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is it happening, can you please share the run?
| ### Network & Security | ||
|
|
||
| - **[Enabling emails for wire](smtp.md)** | ||
| - Configure SMTP for user onboarding via email. Deploy either a temporary SMTP service included in the bundle or integrate with your existing SMTP relay, and ensure proper network configuration for email delivery. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can remove this, I dont see why anybody would be bothering with setting up DKIM for a test environment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
smtp can be deployed without setting up DKIM and we would need some kind of mail service to enable user creation and testing
Veki301
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
documentation incomplete
| @@ -0,0 +1,220 @@ | |||
| # Scope | |||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
documentation is incomplete, as we have discussed in our meeting
d11d557 to
d56c938
Compare
7c8f95f to
a9e1137
Compare
fdb2c12 to
0254dc9
Compare
18d8d0c to
6350fce
Compare
31d4a93 to
de86aa0
Compare
6350fce to
bddee18
Compare
de86aa0 to
425c518
Compare
bddee18 to
bafba56
Compare
425c518 to
7728269
Compare
…gle_hetzner_machine_installation
bafba56 to
e63a276
Compare
|



Summary of changes :
This ticket is part of cascading PRs:
#852 - clean old wiab staging script and workflows
#853 - Testing wiab-staging solution for wire-server-5.14
No labels:
The changes in the PR will be verified in #853
Change type
Basic information
Testing
Offline Build CI (label-based)
Add one or more labels to trigger offline builds:
build-default- Full production build (ansible, terraform, all packages)build-demo- Demo/WIAB buildbuild-min- Minimal build (fastest, essential charts only)build-all- Run all three buildsNote: No builds run by default. Add a label to trigger CI.
Tracking
changelog.dKnowledge Transfer
Motivation
Objective
Reason
Use case