The future of partclone and therefore FOG as it is
-
@Sebastian-Roth OK I’ll hold off until I get the green light from you.
-
I’m preparing a build with Buildroot version 2019.02.4 and one with that Buildroot version + partclone 3.12, will test and post them on Monday probably.
-
@Quazz Here are the edits (not in diff format but george shorthand) that junkhacker recommended to take advantage of new features in pigz and zstd for data dedup systems
vi fog.upload partclone.imager -aX0 -c -s "$hd" -O /tmp/pigz1 -N -f 1 vi funcs.sh pigz --rsyncable $PIGZ_COMP < $fifo | split -a 3 -d -b 200m - ${file}. & pigz --rsyncable $PIGZ_COMP < $fifo > ${file}.000 & zstdmt --rsyncable -B128 --ultra $PIGZ_COMP < $fifo | split -a 3 -d -b 200m - ${file}. & zstdmt --rsyncable -B128 --ultra $PIGZ_COMP < $fifo > ${file}.000 & partclone.$fstype -aX0 -n "Storage Location $storage , Image name $img" -cs $part -O $fifoname -Nf 1 remove --ignore_crc -
@george1421 Thanks, I had already done some of that, got all of it in there now.
-
@george1421 said in The future of partclone and therefore FOG as it is:
OK I’ll hold off until I get the green light from you.
Totally forgot to let you know that it’s been done - weeks ago already (ref).
-
Okay, so I can’t believe I never noticed this, but -aX0 is an invalid parameter, it should be -a0
Source: https://github.com/Thomas-Tsai/partclone/blob/master/src/partclone.c
" -aX --checksum-mode=X Checksum formula to use to add error detection\n" " where X:\n" " 0: No checksum (no slowdown, smallest image)\n" " 1: CRC32 (Fast to compute, basic detection)\n"Hopefully the source of the funky problem I was having, rebuilding now.
-
The error I was getting seems to be originating from zstdmt, it doesn’t have an rsyncable option in the included version. Would have thought that would be bundled by now, grrr.
-
-
@george1421 I decided to go for 1.4.2 since it includes a couple of performance improvements and bug fixes.
Took a while longer because I forgot to include the hash file.
It’s finally compiling successfully, time for some more tests.
-
Currently it fails to capture
msftrespartitions, throwing exit code 139 (presumably zstd’s error code). msftres is captured withpartclone.imagerIt’s difficult to see what’s going on because the screen breaks up, but it seems to not recognize any size on this type of partition on partclone 3.12 which is a regression compared to 2.89
edit: Adding a strategic
debugPausetells me there is a segmentation fault on line 2041 infuncs.shEven stranger is that it is talking about line 2053, yet for some reason mentions line 2041???
Though not sure why this occurs…
edit2: Seems to be down to the fact that it writes to
/tmp/pigz1Direct writing to /images works fine
edit3: Welp, it seems to come and go as it pleases, not sure what the actual cause is!
edit4: My suspicion lies on B128 option being the culprit (initial test is promising), recompiling, will test tomorrow.
-
Removing the
-B128option seems to allow it to capture, finally!64 bit init.xz Buildroot 2019.02.4 + ZSTD 1.4.2 + Partclone 0.3.12 (with APFS support)
32 bit init.xz Buildroot 2019.02.4 + ZSTD 1.4.2 + Partclone 0.3.12 (with APFS support)
Give it a whirl if you’re interested.
Current status:
-
Specifying the
-B128option gives issues in certain scenarios (special partitions/raw/very small ones???), so we can’t reliably use that. -
In order to use
--rsyncable, zstd had to be updated to a minimum of 1.3.8 (chosen the latest version of 1.4.2 to include performance improvements and bug fixes) -
Minor Buildroot update means config doesn’t have to be updated, it’s just bugfixes
-
Added APFS support so that we can offer some better support for newer Macs (if they decide to PXE boot that is) (potentially gptfdisk package should be updated to 1.0.4 (adds typecodes for APFS and others), haven’t tested anything in this direction!)
-
-
I’m also wondering if we should look into updating other packages that FOG either overrides or provides manually. (eg testdisk is on 6.14 as opposed to 7.1 and even pigz is at 2.3.4 instead of 2.4 (although pigz seems to have introduced potentially backwards incompatible changes))
-
@Quazz Great news that you got it to compile. Also APFS support was a current topic wondering if it was going to be supported. So good idea to add that support. To make use of that will the web gui need to be updated to for a different operating system type?
I’ll download the inits a bit later today to try them in my environment. I’m still running FOG 1.5.5 FWIW.
As for updating other packages… (understand this is just my opinion) You installed buildroot 2019.02.1 which is the same as what the developers are using. Every time you deviate from the standard buildroot package you run the risk of introducing unexpected bugs. I would say unless there is a compelling reason (fog needs a feature of the update package or to fix a bug fog is hitting) don’t upgrade packages at random. Let the buildroot devs do that leg work. I also saw that pigz had an updated version and I think I saw the incompatibility statements so I decided to not update that in my build. That was simply because pigz supported the resyncable flag that we were looking to implement. So I would recommend that unless we are trying to solve a problem, I would say hold off on updating any packages beyond the buildroot stream. I would rather want stable over new unneeded features.
Great job on getting the inits compiled and working!! I’ll let you know how it works on my stuff.
-
@george1421 I agree for vital packages that waiting on Buildroot where possible is preferable, but something like Testdisk (which doesn’t come from Buildroot) could surely use a bump since that doesn’t break anything
You installed buildroot 2019.02.1
2019.02.4 actually, which doesn’t fundamentally changes any packages, just a bunch of bugfixes!
-
@Quazz said in The future of partclone and therefore FOG as it is:
which doesn’t come from Buildroot
Ah so its a fog supplied package. Well then that changes the equation a bit. If its fog project supplied and its tested then there is probably no reason why we should not update. My concern is deviating from the norm that buildroot devs create.
-
@Quazz You might want to include the
nvmepackage in your build too. Hopefully that will help us solve the issue with the nvme disk swapping. Its not important for this new init testing, but it may be of some use in the future.Symbol: BR2_PACKAGE_NVME [=n] x x x x Type : bool x x x x Prompt: nvme x x x x Location: x x x x -> Target packages x x x x -> Hardware handling x x -
Had a kernel error of rootfs can’t mount blablabla on a device today, normal init works fine. Ramdisk size is at 275000 currently.
Unsure why this happened.
edit: Might just be a machine issue as it seems to be acting up in other ways too (namely freezing)
-
Would be great if we could come to a satisfactory answer on this matter. Unfortunately I reached the limit on what I can test on my end (all good), but obviously we want to be sure it works at least as well as the older version for everyone.
-
@Quazz Sorry for staying away from this so far. Guess there is not much I can add as I can only test on VirtualBox. I’ll still do that later on an get back to you. Is it still the download links you posted 20 days ago?
-
@Sebastian-Roth Those should still work yes. Also created a PR on github, so through that way people could check it out too I suppose.
We only really have consumer grade hardware here, rarely anything special, so I know it will work in most scenarios, just need to know if it also works in those other scenarios!
