-
Notifications
You must be signed in to change notification settings - Fork 45
wpb-22439 wiab-staging: ansible prepare host for vm provisioning #846
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-21356-update-vm-management
Are you sure you want to change the base?
wpb-22439 wiab-staging: ansible prepare host for vm provisioning #846
Conversation
…pulated by wiab-staging-provision.yml playbook
…ible-VM-provisioning
ansible/wiab-staging-provision.yml
Outdated
| - lsof | ||
| - net-tools | ||
| - rsyslog | ||
| - screen |
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.
we don't really use screen anymore, you can remove it
| @@ -0,0 +1,315 @@ | |||
| - hosts: deploy_node | |||
| vars: | |||
| artifact_hash: d5c1e25ac25d9cf4ade2f00c12f0cd05371b8616 | |||
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.
this is hardcoded, is there a documentation in other PRs that states this should be changed/updated ?
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.
This is mentioned here in this documentation:
https://github.com/wireapp/wire-server-deploy/blob/wpb-21356-7-docs-wiab-staging/offline/wiab-staging.md#requirements
| register: gateway_name | ||
|
|
||
| - name: Generate inventory.yaml with actual VM IPs from .vm-env | ||
| shell: | |
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.
this can probably go into its own script and then execute it here, will be more readable
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.
I kept it here as an ansible task as it removes the dependency from the script being available on the node and, this script isn't used for any other purpose so kept it as task here.
|
|
||
| - name: set permissions inside wire-server-deploy via shell command (fails when using ansible directive) | ||
| shell: | ||
| cmd: sudo chmod -R 0775 "{{ deploy_dir }}"; sudo chown -R demo:demo /home/demo |
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 put ssh keys into wire directory in the previous steps and you are giving it 775 permissions with this
also, why are you changing ownership on the /home/demo directory?
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.
I am changing the permissions for the deploy_dir=/home/demo/wire-server-deploy, for virsh to be able to access it and ensure when extracting the bundle, all files have accessible permissions. The SSH keys would be created post this task in Start VMs on the node task.
…ansible playbook
|



Rename ansible templates for wiab-staging to make them generic
Add wiab-staging VMs ansible inventory template to be populated by wiab-staging-provision.yml playbook
Add ansible playbook for wiab-staging VM provisioning
Add ansible inventory for playbook wiab-staging-provision.yml
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