[Bug 893755] New: The "obsstoragesetup" script ignores the size of the "OBS/server" logical volume.
https://bugzilla.novell.com/show_bug.cgi?id=893755 https://bugzilla.novell.com/show_bug.cgi?id=893755#c0 Summary: The "obsstoragesetup" script ignores the size of the "OBS/server" logical volume. Classification: Internal Novell Products Product: openSUSE Build Service Version: 2.5 Platform: x86-64 OS/Version: openSUSE 13.1 Status: NEW Severity: Major Priority: P5 - None Component: General AssignedTo: adrian@suse.com ReportedBy: DOlsson@WEB.de QAContact: adrian@suse.com Found By: --- Blocker: --- Created an attachment (id=603835) --> (http://bugzilla.novell.com/attachment.cgi?id=603835) Patch fixing the size calculation of the "root" LV (now allowing for the "server" LV). User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0 During the installation of the OBS Appliance (e.g. using the instructions found in http://en.opensuse.org/openSUSE:Build_Service_Appliance or http://openbuildservice.org/download/), it is possible to make use of LVM by having a volume group (VG) named "OBS". According to the instructions, it is even possible to have the "/srv/obs" server directory mounted on a logical volume (LV) under "OBS" by creating a LV named "[/dev/OBS/]server". Unfortunately, though, when doing so, the "obsstoragesetup" script ignores the size of the "server" LV, resulting in not being able to create any of the worker "root" and "swap" LVs as well as the worker "cache" LV, because the formula used to calculate the size of the "root" LVs are depending on the total size of the VG minus the size of the "cache" LV and the sizes used for the "swap" LVs, but not the size of the "server" LV, resulting in a broken setup of the OBS Appliance Server. Reproducible: Always Steps to Reproduce: 1. Download the OBS Appliance server of your choice. 2. Before installing the OBS Appliance, create a volume group named "OBS" and a logical volume in this VG named "server" (the sizes do not matter, but use e.g. 50 GiB for the VG size and 10 GiB for the LV size). 3. Install the OBS Appliance. 4. After the installation logon to the system and notice that the "/srv/obs" directory has been mount onto "/dev/obs/server", as expected, but also notice that not all of the worker root directories have not been created respectively mounted as expected. Notice also that the "cache" LV not neither been created nor mounted. 5. When looking through the start-up logs, you will also notice that the "obsstoragesetup" script has never output "Looking for OBS Worker Cache LVM Volume". This has not happened, because the script has bailed out when trying to create the "cache" LV, which was not possible due to lack of space in the VG (the bail out has happened in line #324. Actual Results: Various number of worker "root" and "swap" LVs as well as the "cache" LVs havenot been setup and mounted properly. Expected Results: All worker "root", "swap" and "cache" LVs have been corrected created and mounted. As work around, when using the OBS Appliance v2.5.4.2, one should not create the "server" LV in the "OBS" VG, as the installation instructions suggests, but instead after having installed the OBS Appliance manually move the "/srv/obs" directory to another disk or LV of VG not being the "OBS" VG! -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=893755 https://bugzilla.novell.com/show_bug.cgi?id=893755#c1 --- Comment #1 from Dennis Olsson <DOlsson@WEB.de> 2014-08-27 12:33:52 CEST --- Sorry for the typos. The sentence: Notice also that the "cache" LV not neither been created nor mounted. should read: Notice also that the "cache" LV has neither been created nor mounted. The sentence: (the bail out has happened in line #324. should read: (the bail out has happened in line #324 of "obsstoragesetup"). The sentence: directory to another disk or LV of VG not being the "OBS" VG! should read: directory to another disk or a LV of a VG not being the "OBS" VG! -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=893755 https://bugzilla.novell.com/show_bug.cgi?id=893755#c2 Dennis Olsson <DOlsson@WEB.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #603835|0 |1 is obsolete| | --- Comment #2 from Dennis Olsson <DOlsson@WEB.de> 2014-09-04 17:17:38 CEST --- Created an attachment (id=605052) --> (http://bugzilla.novell.com/attachment.cgi?id=605052) Revised patch that also takes care of handling missing LVs in VG "OBS". Also found an annoying bug regarding not being able to handle the existence of the "OBS" VG, but it not having any LVs assigned to it. Attached new patch with fix of this problem, too. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=893755 https://bugzilla.novell.com/show_bug.cgi?id=893755#c3 Adrian Schröter <adrian@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #3 from Adrian Schröter <adrian@suse.com> 2014-09-05 06:47:18 UTC --- thanks, I have applied it to git master now. I can also backport to 2.5 branch if it is important for you. JFYI, we are a bit faster in appling patches when using github pull request, in case you care :) https://github.com/openSUSE/open-build-service thanks again -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=893755 https://bugzilla.novell.com/show_bug.cgi?id=893755#c4 Dennis Olsson <DOlsson@WEB.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #4 from Dennis Olsson <DOlsson@WEB.de> 2014-09-07 20:43:26 CEST --- Yes, it would nice to have it in the 2.5 branch, too. After all, this is the current version, AFAIK, just as it is the version that gets published as an appliance. Thanks. PS: Hope it was correct of me to "reopen" this bug, due to the above request for backport to the 2.5 branch. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com