Merge ~kajiya/ubuntu/+source/gce-compute-image-packages:change-udev-rules-for-HDDs-mantic into ubuntu/+source/gce-compute-image-packages:ubuntu/mantic-devel

[Chloé Smith]

Reduce udev rule scope and change I/O scheduler

The previous udev rule was drastically reducing the bootspeed on
SSD backed instances. Experiments found that swapping HDDs to "none"
in this file yielded the best results.

[Utkarsh Gupta (utkarsh)]

Hey, I know we agreed to do this together but didn't get enuf time during the sprints to actually do it.

On a quick note, if you want to SRU this to any of the stable releases, you'll need a bug associated with an upload - with an SRU template. Could you please work that out and I can then sponsor the upload?

I'll do Noble -> Mantic. And further if need be. Let me know if you have any questions or concerns.

review: Needs Fixing

[Chloé Smith (kajiya)]

Rejected in favour of https://code.launchpad.net/~kajiya/ubuntu/+source/gce-compute-image-packages/+git/gce-compute-image-packages/+merge/455766

review: Disapprove

Unmerged commits

000e991... by Chloé Smith on 2023-10-18

d/changelog entry

b9fe69b... by Chloé Smith on 2023-10-16

Reduce udev rule scope and change I/O scheduler

The previous udev rule was drastically reducing the bootspeed on
SSD backed instances. Experiments found that swapping HDDs to "none"
in this file yeilded the best results.

diff --git a/debian/99-gce.rules b/debian/99-gce.rules
index fa98786..d321cde 100644
--- a/debian/99-gce.rules
+++ b/debian/99-gce.rules
@@ -1,5 +1,5 @@
 # Give syslog write access to the kernel logs
 KERNEL=="console", GROUP="syslog", MODE="0620"
 
-# Switch to using NOOP as the default scheduler per GCE request
-SUBSYSTEM=="block", ENV{DEVTYPE}!="partition", ACTION=="add|change", ENV{ID_VENDOR}=="*Google*", ATTR{queue/scheduler}="noop"
+# Force "NONE" I/O scheduler on HDDs only to enhance boot speeds
+SUBSYSTEM=="block", ENV{DEVTYPE}!="partition", ACTION=="add|change", ENV{ID_VENDOR}=="*Google*", ATTR{queue/rotational}=="1", ATTR{queue/scheduler}="none"
diff --git a/debian/changelog b/debian/changelog
index 968b98f..bbd3b79 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,13 @@
+gce-compute-image-packages (20230808.00-0ubuntu2) mantic; urgency=medium
+
+  * Changes made to d/99-gce.rules
+    * The previous rule forced an I/O scheduler on all disk types. This
+      has now been re-scoped to only affect HDDs
+    * The I/O scheduler has also been changed from "NOOP" to "NONE"
+      which performed much better in testing.  
+
+ -- Chloé 'kajiya' Smith <[e-mail]>  Mon, 16 Oct 2023 22:59:06 +0100
+
 gce-compute-image-packages (20230808.00-0ubuntu1) mantic; urgency=medium
 
   * New upstream version 20230808.00. (LP: #2033061)