-
Notifications
You must be signed in to change notification settings - Fork 53
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
Pretasks #70
Pretasks #70
Conversation
Name: gluster-ansible-roles | ||
Version: 1.0.0 | ||
Release: 2%{?dist} | ||
Name: gluster-ansible |
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.
why is this part of pre-tasks change?
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 was a earlier commit which was done to do a build. And when the PR is created it included earlier commits as well.
- name: Check if /var/log has enough disk space | ||
shell: df -m /var/log | awk '/[0-9]%/ {print $4}' | ||
register: disk_size | ||
failed_when: disk_size.stdout|int < gluster_features_min_disk|default(15360) |
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.
are you changing the GiB to GB check here?
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.
Yes. I have done that.
when: gluster_infra_volume_groups is defined and | ||
item.pvname is not search("/dev/mapper") and | ||
gluster_features_512B_check|default(true) | ||
with_items: "{{ gluster_infra_volume_groups }}" |
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.
can you also add assert statements to the task failures?
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.
Done.
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.
Looks good to me
No description provided.