Skip to content
This repository has been archived by the owner on Jun 18, 2021. It is now read-only.

prefixed the vars, added resizefs, since first post PXE task is alway… #11

Merged
merged 3 commits into from
Oct 9, 2015

Conversation

kennethgillen
Copy link
Member

…s resize the PXE created LVs and FS.

Feel free to test with infra_testpr. There's a snapshot ready for reverting-to (name of snap is self documenting).

dev: /dev/{{ vgname }}/{{ lvname }}
fstype: "{{ lvm_lvfilesystem }}"
dev: /dev/{{ lvm_vgname }}/{{ lvm_lvname }}
resizefs: yes

- name: storage | mount scratch
Copy link
Member

Choose a reason for hiding this comment

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

Since you're cleaning up this file, can you change the name to storage | mount?

@manics
Copy link
Member

manics commented Sep 24, 2015

I've opened #12 which can be tested alongside this. Overall there should be no change since these are either variable renames, rearrangements of tasks, or variable changes which should not have an effect on existing systems.

@sbesson
Copy link
Member

sbesson commented Oct 9, 2015

@manics: okay with the last changes?

vg: "{{ lvm_vgname }}"
lv: "{{ lvm_lvname }}"
size: "{{ lvm_lvsize }}"
opts: "{{ lvm_lvopts }}"
Copy link
Member

Choose a reason for hiding this comment

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

While testing I realized lvol opts is a 2.0+ feature. Are you using development ansible or a beta version @kennethgillen @manics ?

Copy link
Member

Choose a reason for hiding this comment

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

development. Note lvm_lvopts will need another fix but I can rebase #21: https://github.com/openmicroscopy/infrastructure/pull/21/files#diff-ed13ce14137be8371d9a6e4bf1420595

manics added a commit that referenced this pull request Oct 9, 2015
prefixed the vars, added resizefs, since first post PXE task is alway…

Further refinements can be made in a future PR
@manics manics merged commit 348b08d into ome:master Oct 9, 2015
manics added a commit to manics-archive/ome-infrastructure that referenced this pull request Nov 24, 2016
prefixed the vars, added resizefs, since first post PXE task is alway…

Further refinements can be made in a future PR
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants