Bug 854229 - swapsize requires partition element
Summary: swapsize requires partition element
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: lab controller
Version: 0.9
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 19.1
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard: Kickstart
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-09-04 12:57 UTC by Raymond Mancy
Modified: 2014-12-18 00:24 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-12-17 08:40:14 UTC
Embargoed:


Attachments (Terms of Use)

Description Raymond Mancy 2012-09-04 12:57:31 UTC
Description of problem:

the KS meta var 'swapsize' has no effect unless a <partition/> 
is specified

Version-Release number of selected component (if applicable):

0.9

How reproducible:

Always

Steps to Reproduce:
1.Insert swapsize as ks meta var
2.
3.
  
Actual results:

It's ignored unless you have also specified a partition

Expected results:

It should create the swap partition with the value from the swapsize variable

Additional info:

Perhaps we could just do this

diff --git a/Server/bkr/server/snippets/rhts_partitions b/Server/bkr/server/snippets/rhts_partitions
index 0d6d813..11f8fb9 100644
--- a/Server/bkr/server/snippets/rhts_partitions
+++ b/Server/bkr/server/snippets/rhts_partitions
@@ -8,7 +8,7 @@ clearpart --drives {{ ondisk }} --all --initlabel
 clearpart --all --initlabel
 {% endif %}
 
-{% if partitions or ondisk %}
+{% if partitions or ondisk or swapsize %}
 
 {#
 ## Arch Specific Boot requirements

Comment 1 Nick Coghlan 2012-10-17 04:35:54 UTC
Bulk reassignment of issues as Bill has moved to another team.

Comment 3 Dan Callaghan 2014-11-20 06:36:44 UTC
http://gerrit.beaker-project.org/3497

Comment 4 Dan Callaghan 2014-11-25 05:47:53 UTC
In testing this I discovered that swapsize does not actually do anything, I think because we are also passing --recommended at the same time so Anaconda ignores our requested size.

Comment 5 Dan Callaghan 2014-11-25 06:08:39 UTC
Patch updated.

A better test case is probably:
1. Schedule a job that requests 10GB swap, on a system with >20GB disk:
<recipe ks_meta="swapsize=10240"...
<hostRequires>
  <disk><size op="gt;" size="20" units="GB"/></disk>
  ...
<reservesys/>
...
2. Wait for system to be reserved
3. Log in and check size of swap partition, using fdisk -l or free

Expected results:
Swap partition is 10GB.

# free
             total       used       free     shared    buffers     cached
Mem:        873556     399172     474384          0      20228     203272
-/+ buffers/cache:     175672     697884
Swap:     10485752          0   10485752

Comment 12 Dan Callaghan 2014-12-17 08:40:14 UTC
Beaker 19.1 is released.


Note You need to log in before you can comment on or make changes to this bug.