bugutil-vserver - Bugs: bug #27134, [PATCH] vsomething does not work...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #27134: [PATCH] vsomething does not work when called without VSOMETHING_PKGMGMT set

Submitter:  Matthijs Kooijman <matthijs>
Submitted:  Wed 29 Jul 2009 11:35:01 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Component Version:  alpha
Release:  trunk (r2844)

Wed 29 Jul 2009 11:42:14 AM UTC, comment #1: 

Hmm, on second thought, this might not be exactly the right fix. I was updating the --help output, I noticed that the behaviour this patch "fixes" is exactly what is documented:

    "Execute <cmd> <vserver> <params>* foreach vserver."

It is mostly not what I had expected, and not what happens when VSOMETHING_PKGMGMT is set, which still makes vsomething a bit inconsistent. However, it does make it more powerful, since it allows running custom scripts in the host for each vserver, but still allows for running things inside a vserver by using "vserver" as the <cmd> and "exec" as the first <param>.

So, I guess we can close this report... I do feel that the stuff that happens in vsomething related to package management would better be extracted in some other script(s)...

Matthijs Kooijman <matthijs>
Wed 29 Jul 2009 11:35:01 AM UTC, original submission:  

When trying to use vsomething directly (which is handy, since it supports --all, as opposed to vserver exec), vsomething doesn't work. It runs the command, but inside the host instead of the vserver, and adds the vserver name as the first argument.

Looking at the code, this is really broken (assuming this weird behaviour is not intentional, in which case it is really not documented anywhere). It seems this has been broken since the initial checkin in 2005.

The attached patch fixes this issue by calling vserver exec when VSOMETHING_PKGMGMT is not set. It slightly restructures the control flow to have only two cases instead of three left.

Matthijs Kooijman <matthijs>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #18492:  vsomething.diff added by matthijs (1KiB - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by matthijs (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2009-07-29 matthijs Attached File- Added vsomething.diff, #18492

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code