Skip to content
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

relax case-sensitivity check on unit #55922

Open
wants to merge 3 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@ticapix
Copy link

commented Apr 30, 2019

proposal fix for #37522

SUMMARY

Allow unit to be with capital or not letter

Fixes #37522

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

Fix for parted

ADDITIONAL INFORMATION

It allows in a playbook task to reuse the unit value from a previous parted call.

    - name: collect disk info
      parted:
        device: /dev/sda
      register: disk_info

    - name: create data partition
      parted:
        device: /dev/sda
        number: 2
        part_start: '{{ disk_info.partitions[0].end + 1 }}{{ disk_info.partitions[0].unit }}'
        part_end: 100%
        state: present
      when: disk_info.partitions | length == 1
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Apr 30, 2019

ticapix added some commits Apr 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.