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

Migrate XFS configuration files. #2431

Merged
merged 2 commits into from Jun 22, 2020
Merged

Conversation

gozora
Copy link
Member

@gozora gozora commented Jun 20, 2020

# lsblk -ipo NAME,KNAME,PKNAME,TRAN,TYPE,FSTYPE,SIZE,MOUNTPOINT
NAME                        KNAME     PKNAME    TRAN   TYPE  FSTYPE             SIZE MOUNTPOINT
/dev/sda                    /dev/sda            sata   disk                       8G 
|-/dev/sda1                 /dev/sda1 /dev/sda         part  vfat               200M /boot/efi
|-/dev/sda2                 /dev/sda2 /dev/sda         part  xfs                  1G /boot
`-/dev/sda3                 /dev/sda3 /dev/sda         part  LVM2_member        6.8G 
  |-/dev/mapper/centos-root /dev/dm-0 /dev/sda3        lvm   xfs                  6G /
  `-/dev/mapper/centos-swap /dev/dm-1 /dev/sda3        lvm   swap               820M [SWAP]
/dev/sdb                    /dev/sdb            sata   disk  mpath_member         8G 
`-/dev/mapper/disk_2        /dev/dm-3 /dev/sdb         mpath linux_raid_member    8G 
  `-/dev/md0                /dev/md0  /dev/dm-3        raid1 xfs                  8G /data
/dev/sdc                    /dev/sdc            sata   disk  mpath_member         8G 
`-/dev/mapper/disk_1        /dev/dm-2 /dev/sdc         mpath linux_raid_member    8G 
  `-/dev/md0                /dev/md0  /dev/dm-2        raid1 xfs                  8G /data
/dev/sr0                    /dev/sr0            ata    rom                     1024M 
  • Brief description of the changes in this pull request:
    When in MIGRATION_MODE, migrate/rename XFS configuration files so they follow disk mapping set by user.

When in MIGRATION_MODE, migrate/rename XFS configuration files so
they follow disk mapping set by user.
@gozora gozora requested review from jsmeix, rmetrich and gdha June 20, 2020 21:54
@gozora gozora self-assigned this Jun 20, 2020
@gozora gozora added this to the ReaR v2.7 milestone Jun 20, 2020
@gozora gozora linked an issue Jun 20, 2020 that may be closed by this pull request
Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

@gozora
please have a look at my
#2431 (comment)
and
#2431 (comment)

Perhaps I am wrong but at least from plain looking at the code
it seems things could go wrong as I described.
If all is OK perhaps a comment that explains how those sed substitutions
behave fail-safe could help to better understand how things actually work.

@gozora
Copy link
Member Author

gozora commented Jun 22, 2020

Hello @jsmeix,

Thanks for your comments!
I've made some improvements in code which should cover also problem when mapping multipath device to ordinary disk. Which previously created entry like meta-data=/dev/mapper/sdd.

V.

Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

I don't know the XFS configuration file syntax
so I cannot actually verify the sed substitute commands to
"Replace old device name in meta-data= option in XFS configuration file"
but because @gozora you now double-checked that part
I trust that now things work sufficiently well.

@jsmeix
Copy link
Member

jsmeix commented Jun 22, 2020

@gozora
since we are currently in early ReaR 2.7 development mode
I think you could "just merge" it e.g. tomorrow
provided @gdha does not object until then?

@jsmeix
Copy link
Member

jsmeix commented Jun 22, 2020

... or "just merge" it right now ;-)

@gozora gozora merged commit 01249e1 into rear:master Jun 22, 2020
@gozora
Copy link
Member Author

gozora commented Jun 22, 2020

Thanks for your reviews!

V.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mapping disks doesn't map corresponding XFS information file
4 participants