Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

EX-1476 Create step to create and populate ldev config #2154

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

EX-1476 Create step to create and populate ldev config #2154

wants to merge 5 commits into from

Conversation

johnsonw
Copy link
Contributor

@johnsonw johnsonw commented Aug 12, 2020 •
edited by jgrund
Loading

Create the step that will populate the ldev config.

Signed-off-by: johnsonw [email protected]

This change is Reviewable

@johnsonw johnsonw added this to the IML EX V3 milestone Aug 12, 2020
@johnsonw johnsonw self-assigned this Aug 12, 2020
@johnsonw
Copy link
Contributor Author

image

@johnsonw johnsonw force-pushed the add-mgs-fs-to-stats-service branch 2 times, most recently from f205d34 to cd80377 Compare August 13, 2020 18:58
@johnsonw
Copy link
Contributor Author

Output on /etc/ldev.conf on MDS2 (the mgs) after running the command:

oss2.local oss1.local fs-OST0002 /mnt/ost2
oss2.local oss1.local fs-OST0004 /mnt/ost4
oss2.local oss1.local fs-OST0008 /mnt/ost8
oss2.local oss1.local fs-OST0000 /mnt/ost0
oss2.local oss1.local fs-OST0006 /mnt/ost6
mds2.local mds1.local fs-MDT0001 /mnt/mdt1
mds2.local mds1.local MGS /mnt/mgs
oss1.local oss2.local fs-OST0007 /mnt/ost7
oss1.local oss2.local fs-OST0009 /mnt/ost9
oss1.local oss2.local fs-OST0005 /mnt/ost5
oss1.local oss2.local fs-OST0003 /mnt/ost3
oss1.local oss2.local fs-OST0001 /mnt/ost1
mds1.local mds2.local fs-MDT0000 /mnt/mdt0

One thing I noticed is that the file already existed and was a bit different so the initial parsing failed. This still needs to be addressed. Here is a copy of the original file:

# example /etc/ldev.conf
#
#local  foreign/-  label       [md|zfs:]device-path   [journal-path]/- [raidtab]
#
#zeno-mds1 - zeno-MDT0000 zfs:lustre-zeno-mds1/mdt1
#
#zeno1 zeno5 zeno-OST0000 zfs:lustre-zeno1/ost1
#zeno2 zeno6 zeno-OST0001 zfs:lustre-zeno2/ost1
#zeno3 zeno7 zeno-OST0002 zfs:lustre-zeno3/ost1
#zeno4 zeno8 zeno-OST0003 zfs:lustre-zeno4/ost1
#zeno5 zeno1 zeno-OST0004 zfs:lustre-zeno5/ost1
#zeno6 zeno2 zeno-OST0005 zfs:lustre-zeno6/ost1
#zeno7 zeno3 zeno-OST0006 zfs:lustre-zeno7/ost1
#zeno8 zeno4 zeno-OST0007 zfs:lustre-zeno8/ost1

@johnsonw johnsonw marked this pull request as ready for review August 18, 2020 02:48
@mkpankov
Copy link
Contributor

Looks like some checks are failing

@johnsonw johnsonw force-pushed the add-mgs-fs-to-stats-service branch 8 times, most recently from fb33d6d to a1f7903 Compare August 21, 2020 18:00
@johnsonw johnsonw force-pushed the EX-1476 branch 3 times, most recently from 7d25504 to f88cf9e Compare August 21, 2020 20:44
@johnsonw
Copy link
Contributor Author

Updates:
image

[root@mds2 etc]# cat /etc/ldev.conf
oss2.local oss1.local fs-OST0000 /mnt/ost0
oss2.local oss1.local fs-OST0002 /mnt/ost2
oss2.local oss1.local fs-OST0004 /mnt/ost4
oss2.local oss1.local fs-OST0006 /mnt/ost6
mds1.local mds2.local fs-MDT0000 /mnt/mdt0
mds2.local mds1.local fs-MDT0001 /mnt/mdt1
oss1.local oss2.local fs-OST0001 /mnt/ost1
oss1.local oss2.local fs-OST0003 /mnt/ost3
oss1.local oss2.local fs-OST0005 /mnt/ost5
oss1.local oss2.local fs-OST0007 /mnt/ost7
mds2.local mds1.local MGS /mnt/mgs
oss2.local oss1.local fs-OST0008 /mnt/ost8
oss1.local oss2.local fs-OST0009 /mnt/ost9

@johnsonw johnsonw force-pushed the EX-1476 branch 3 times, most recently from 1ef21bc to 3298f45 Compare August 26, 2020 02:23
@johnsonw johnsonw force-pushed the EX-1476 branch 3 times, most recently from 1e7e460 to b039434 Compare October 21, 2020 21:01
@johnsonw johnsonw changed the base branch from master to add-fs-type October 21, 2020 21:02
@johnsonw johnsonw force-pushed the EX-1476 branch 2 times, most recently from a09f3a3 to 017d91e Compare October 22, 2020 14:02
@johnsonw johnsonw force-pushed the add-fs-type branch 8 times, most recently from d02e1a7 to 724e50e Compare October 27, 2020 17:42
@johnsonw
Add fs_type field to targets table
07a7a72 
The fs_type data should be included in the targets table. This will be
used when creating the ldev conf and likely other things as well.

Signed-off-by: johnsonw <[email protected]>
@johnsonw
CR Updates
2c8d67e 
Signed-off-by: johnsonw <[email protected]>
@johnsonw
Make sure target table exists before adding the fs_type field.
e33feba 
Signed-off-by: johnsonw <[email protected]>
@johnsonw
Don't assume ldiskfs
dfa96f3 
Signed-off-by: johnsonw <[email protected]>
@johnsonw
EX-1476 Create step to create and populate ldev config
99b352a 
Create the step that will populate the ldev config.

Signed-off-by: johnsonw <[email protected]>
@johnsonw johnsonw force-pushed the add-fs-type branch 2 times, most recently from 45d4cd6 to df6e8ba Compare November 17, 2020 01:26
Base automatically changed from add-fs-type to master November 18, 2020 16:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jgrund jgrund jgrund requested changes

@ip1981 ip1981 ip1981 left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees

@johnsonw johnsonw

Labels
enhancement
Projects
None yet
Milestone
IML EX V3
Development

Successfully merging this pull request may close these issues.
4 participants
@johnsonw @mkpankov @ip1981 @jgrund