Page MenuHomeFreeBSD

release: tweak Oracle Cloud settings
ClosedPublic

Authored by dch on Oct 11 2024, 12:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 2, 12:34 PM
Unknown Object (File)
Sun, Dec 1, 3:09 PM
Unknown Object (File)
Thu, Nov 28, 5:58 AM
Unknown Object (File)
Mon, Nov 25, 5:52 AM
Unknown Object (File)
Mon, Nov 25, 5:52 AM
Unknown Object (File)
Tue, Nov 12, 2:30 AM
Unknown Object (File)
Sat, Nov 9, 5:43 AM
Unknown Object (File)
Wed, Nov 6, 8:45 PM
Subscribers

Details

Summary
  • use raw image disk type and enable zfs, this yields smaller images for upload after using native qcow2 + zstd compression
Test Plan
  • removed bump image size due to python bloat in cloudinit packages because re-testing shows its no longer required
$ qemu-img convert -S 512b -p \
  -O qcow2 -c -o compression_type=zstd  \
  /usr/obj/usr/src/amd64.amd64/release/oracle.zfs.raw \
  /tmp/qc2

yields a 4.0GiB -> ~ 890MiB reduction :-)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dch requested review of this revision.Oct 11 2024, 12:13 PM

Just to confirm, the Oracle upload system doesn't have any preference for disk image formats?

release/Makefile.vm
50

As with OCI->ORACLE rename, we want =? here.

It's interesting that using raw could result smaller image size than qcow2.

This revision is now accepted and ready to land.Oct 14 2024, 8:05 PM
emaste added a subscriber: emaste.

It's interesting that using raw could result smaller image size than qcow2.

I'm guessing this means we were generating an uncompressed qcow2, and converting raw to qcow2 includes compression as a side effect.

The image size bump doesn't seem to be reflected in this review (I'm guessing it got lost in the migration from the other review) but is fine with me.

release/Makefile.vm
50

?=

dch marked 2 inline comments as done.

Make ORACLE_FSLIST? overridable.

This revision now requires review to proceed.Oct 15 2024, 9:10 AM

i.e. raw > mkimg qcow2 > recompressed raw qcow2.

Oracle bills based on raw uploaded size, so saving 2/3 of the image size makes a big difference for everybody.

See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=263029 for more details.

Ed - the VMSIZE change is in the next file.

This revision is now accepted and ready to land.Oct 15 2024, 9:10 AM
dch edited the test plan for this revision. (Show Details)
dch edited the test plan for this revision. (Show Details)

remove VMSIZE change we don't need it anymore

This revision now requires review to proceed.Oct 15 2024, 1:12 PM
dch added inline comments.
release/tools/oracle.conf
29–30 ↗(On Diff #144878)

this isn't required anymore - not sure what's changed, but I bumped this locally over a year ago, and I'm not seeing the bloat that broke images in the past.

This revision is now accepted and ready to land.Oct 16 2024, 12:28 PM
This revision was automatically updated to reflect the committed changes.