Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Added volume specification #56

Open
rohan47 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added volume specification #56

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions ansible-playbook-base/roles/configure-gluster-volume/tasks/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
---
# Tasks sets specification for gluster volume

- name: Include gluster vol specification details
include_vars:
file: "{{ vs_definition }}"
run_once: true

Copy link
Collaborator

@ShyamsundarR ShyamsundarR Jan 23, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

Add a play to reset the existing options to defaults, before setting the new options.

- name: Set gluster volume specification
gluster_volume:
state: "present"
name: "{{ volume }}"
Copy link
Collaborator

@ShyamsundarR ShyamsundarR Jan 23, 2019

Choose a reason for hiding this comment

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

Volume name comes from vc_definition

options: "{{ options }}"
run_once: true
6 changes: 6 additions & 0 deletions ansible-playbook-base/site.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,12 @@
tags:
- glustervolume

- name: Set gluster volume specification
hosts: servers
gather_facts: false
roles:
- configure-gluster-volume

- name: Mount Gluster volume
hosts: clients
gather_facts: false
Expand Down
14 changes: 14 additions & 0 deletions volume-specifications/vs-0000-0010.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
---
volume: gbenchvol
Copy link
Collaborator

@ShyamsundarR ShyamsundarR Jan 24, 2019

Choose a reason for hiding this comment

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

Volume name should be from vc_defn so that we do not have to repeat it.. (mentioning here as well so that we do not miss it)

options:
server.allow-insecure: 'on'
server.event-threads: '4'
client.event-threads: '4'
features.cache-invalidation: 'on'
features.cache-invalidation-timeout: '600'
performance.stat-prefetch: 'on'
performance.cache-invalidation: 'on'
performance.md-cache-timeout: '600'
network.inode-lru-limit: '90000'
cluster.lookup-optimize: 'on'
Copy link
Collaborator

@ShyamsundarR ShyamsundarR Jan 23, 2019

Choose a reason for hiding this comment

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

I would like group options to also be a part of this, not sure if this is handled here, will check and get back on this. IOW, the comment is this yml should allow setting volume group options as well (I suspect that is the one we would be testing with most often)