Skip to content

Commit

Permalink
Disable adjust_by_ram for swap at all products (#1804)
Browse files Browse the repository at this point in the history
## Problem

As part of #1111 we enabled `adjust_by_ram` for the swap. Only in the
case of Tumbleweed and to (quoting from that PR description) "raise
awareness and get feedback".

But somehow that setting ended up spreading over all products, even
those in which linking the swap size to the RAM size by default doesn't
seem to make much sense.

Very few people is installing the distributions in a setup where
suspending to RAM is needed. On the other hand, many testers want a
default set of settings that work out-of-the-box since they are not
really interested in the storage setup.

## Solution

The original goal of getting feedback is achieved. So let's switch the
feature back off to ease testing while we decide what the next steps
will be regarding `adjust_by_ram` and its default value.
  • Loading branch information
ancorgs authored Dec 3, 2024
2 parents 449f5cb + 22b20e0 commit d516cc1
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 25 deletions.
6 changes: 6 additions & 0 deletions products.d/agama-products.changes
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
-------------------------------------------------------------------
Tue Dec 3 11:22:09 UTC 2024 - Ancor Gonzalez Sosa <ancor@suse.com>

- Temporarily disable adjusting swap to RAM size for all products
(mitigation for gh#agama-project/agama/#1159).

-------------------------------------------------------------------
Tue Nov 12 08:19:51 UTC 2024 - Gustavo Yokoyama Ribeiro <gyribeiro@suse.com>

Expand Down
7 changes: 2 additions & 5 deletions products.d/leap_160.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -130,12 +130,9 @@ storage:
- mount_path: "swap"
filesystem: swap
size:
auto: true
min: 1 GiB
max: 2 GiB
outline:
auto_size:
base_min: 1 GiB
base_max: 2 GiB
adjust_by_ram: true
required: false
filesystems:
- swap
Expand Down
7 changes: 2 additions & 5 deletions products.d/sles_160.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -133,12 +133,9 @@ storage:
- mount_path: "swap"
filesystem: swap
size:
auto: true
min: 1 GiB
max: 2 GiB
outline:
auto_size:
base_min: 1 GiB
base_max: 2 GiB
adjust_by_ram: true
required: false
filesystems:
- swap
Expand Down
7 changes: 2 additions & 5 deletions products.d/sles_sap_160.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -133,12 +133,9 @@ storage:
- mount_path: "swap"
filesystem: swap
size:
auto: true
min: 1 GiB
max: 2 GiB
outline:
auto_size:
base_min: 1 GiB
base_max: 2 GiB
adjust_by_ram: true
required: false
filesystems:
- swap
Expand Down
7 changes: 2 additions & 5 deletions products.d/slowroll.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -121,12 +121,9 @@ storage:
- mount_path: "swap"
filesystem: swap
size:
auto: true
min: 1 GiB
max: 2 GiB
outline:
auto_size:
base_min: 1 GiB
base_max: 2 GiB
adjust_by_ram: true
required: false
filesystems:
- swap
Expand Down
7 changes: 2 additions & 5 deletions products.d/tumbleweed.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -185,12 +185,9 @@ storage:
- mount_path: "swap"
filesystem: swap
size:
auto: true
min: 1 GiB
max: 2 GiB
outline:
auto_size:
base_min: 1 GiB
base_max: 2 GiB
adjust_by_ram: true
required: false
filesystems:
- swap
Expand Down

0 comments on commit d516cc1

Please sign in to comment.