From: Jamie C. <jca...@we...> - 2009-07-02 19:34:41
|
Send me another patch that I can put "on top" of the previous one, as I have already applied it.. Thanks! - Jamie On 02/Jul/2009 11:11 Todd Kennedy wrote .. Hey Jamie, I sent that patch to you a bit prematurely. There was an additional add_footer missing an index.cgi and also it looks like the saving of the sysidcfg file isn't getting pushed into the zone's /etc/sysidcfg file. I am investigating it now. Hopefully I can track that down quickly. As far as the patch I sent you, do you want me to send an additional patch with just the new changes or do you want a full patch including all the changes? On Wed, Jul 1, 2009 at 1:46 AM, Jamie Cameron <[1]jca...@we...> wrote: On 30/Jun/2009 14:57 Todd Kennedy <[2]tod...@gm...> wrote .. > Hey Jamie, > > I finished making some changes to the Solaris Zones module. The biggest > change is so that it could support branding. I also fixed a few additional > issues: > > 1. Return link in the edit create and save pages were missing someplace to > return to, so I added index.cgi as a return point. Cool, thanks! > 2. The get_confirm_page subroutine in forms-lib.pl is using > Webmin::ConfirmPage with only one button. This produced the regular sized > button with the correct action label and also another button which should be > cancel but there wasn't any text defined. So the cancel button was really > small. Kind of looked like a radio button. So I fixed that... That's a bug .. thanks for fixing it. > 3. When using the ctrun command to boot a zone was causing webmin to > continue polling for a return. This was not a webmin problem, more of how > ctrun was invoked. The command { ctrun zoneadm -z some_zone boot } when run > from a console never exits for some reason. So I added an option to end when > the child command ends thus fixing the problem inside of webmin. This used > to drive me nutz. I'd boot a zone through webmin and it would never return > "done". Sometimes if I let it run for a long time in my browser, the browser > would hang and I'd have to force kill the browser. Awesome, thanks! > So anyway the patch is attached. > > There's one caveat to using zones in webmin. The first time you boot the > zone it runs sysconfig so you have to use zlogin in console mode to make the > selections before all the services will start properly. If you don't do this > you can still start the zone but it's in single user mode "hard to tell" and > inetd is waiting on sysconfig to finish. So I'm not sure if there is a way > to allow webmin to interact with zlogin. I think this would be like running > a telnet session from within webmin. not sure how to do that... Actually, I thought I had avoided this in Webmin by having it setup a sysidcfg file before booting the zone, so that Solaris wouldn't ask all those questions. But perhaps there is some entry from that file I am missing .. - Jamie > Havin' Fun with Code, > -Todd- > > > On Sun, Jun 28, 2009 at 10:13 PM, Jamie Cameron <[3]jca...@we...> wrote: > > > I see the bug - the ui_select functions 3rd arg needs to be an array ref, > > you you need to surround the list_brands call with [ ] > > > > - Jamie > > > > On Jun 28, 2009, at 4:34 PM, Todd Kennedy <[4]tod...@gm...> wrote: > > > > Jamie, > > > > maybe you can help me with something. I'm trying to add the branding > > functionality of zones to the Solaris Zone module. For some reason my select > > box in the create_form.cgi keeps coming up as empty. Here's the code that I > > added below. I know readdir only returns single file / directory > > entries...so I'm not sure it I'm populating the @rv array correctly. The @rv > > array should look like this I'm assuming: > > > > Since the directory entries under /usr/lib/brand are natvie, solaris8, > > solaris9 and the parent and current. I'm filtering out the parent and > > current which should leave me with the three remaining directories. So maybe > > you can point something out to me that I'm missing.... > > > > @rv = (native,solaris8,solaris9) > > > > ------------------create_form.cgi------------------- > > print &ui_table_row($text{'create_brand'}, > > &ui_select("brand",undef, &list_brands(), 0, 0, $value ? 1 : > > 0)); > > ----------------------------------------------------------- > > > > ------------------forms-lib.pl------------------------- > > local $brand = new Webmin::Select("brand",undef, &list_brands(), 0, 0, > > $value ? 1 : 0); > > $section->add_input($text{'create_brand'}, $brand); > > ----------------------------------------------------------- > > > > ------------------zones-lib.pl------------------------ > > #list brands() > > #returns a list of valid brands > > sub list_brands > > { > > local @rv; > > opendir(BRND, "/usr/lib/brand"); > > foreach (readdir(BRND)) { > > if ($_ !~ /^\./){ > > push(@rv, $_); > > } > > } > > close(BRND); > > return @rv; > > } > > ------------------------------------------------------------ > > > > > > > > > > > > On Sun, Jun 28, 2009 at 2:50 AM, Jamie Cameron < <[5]jca...@we...> > > [6]jca...@we...> wrote: > > > >> On 26/Jun/2009 06:22 Todd Kennedy < <[7]tod...@gm...> > >> [8]tod...@gm...> wrote .. > >> > Yeah I can add them to SVN....I guess once I have access I'll download > >> the > >> > files I need to change from SVN then upload them once the changes have > >> been > >> > made. > >> > >> Actually, it might be simpler if you just download Webmin 1.480, make your > >> changes, then send me a patch. > >> > >> > Also - There's a problem with the The breadcrumb link in Solaris Zones > >> that > >> > is suppose to get you back to the previous page...but what happens is > >> that > >> > when you click it it reruns the previous selection. So if you created a > >> zone > >> > and then selected the return to zones list link at the bottom of the > >> page it > >> > tries to create the zone again....I can hunt that down but might take me > >> > longer since I'm still trying to familiarize myself with that module. > >> > >> That sounds like a bug. Let me know if you find the cause.. > >> > >> - Jamie > >> > >> > On Fri, Jun 26, 2009 at 3:51 AM, Jamie Cameron < <[9]jca...@we...> > >> [10]jca...@we...> wrote: > >> > > >> > > On 25/Jun/2009 17:40 Todd Kennedy wrote .. > >> > > > >> > > Hey guys, > >> > > > >> > > I've been doing some work on the Solaris Zones module and have added a > >> few > >> > > features and plan on making a few more. Like the ability to define a > >> spare > >> > > root zone or whole root zone before the config, plus I plan adding the > >> > > template feature of zonecfg to the creation process as well. This > >> would help > >> > > out allot since Solaris zones support branding of sol8 and sol9. Once > >> I am > >> > > finished what is the best way to get patches to you Jamie? I've never > >> > > submitted anything before so I'm new to that part of it. > >> > > > >> > > Thanks > >> > > > >> > > Hi Todd, > >> > > > >> > > You can just email a patch to me directly at <[11]jca...@we...> > >> [12]jca...@we... , and I > >> > > will apply it for inclusion in the next Webmin release. The zones mode > >> > > hasn't been touched for a while, and so is overdue for some new > >> features. > >> > > > >> > > If you plan to do a lot of development, I can grant you commit access > >> to > >> > > the Webmin SVN repository.. > >> > > > >> > > - Jamie > >> > > > >> > > > >> > > > >> > > > >> ---------------------------------------------------------------------------- -- > >> > > > >> > > - > >> > > Forwarded by the Webmin development list at <[13]web...@we...> > >> [14]web...@we... > >> > > To remove yourself from this list, go to > >> > > <[15]http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> [16]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > >> > > > >> > >> > >> ---------------------------------------------------------------------------- -- > >> - > >> Forwarded by the Webmin development list at <[17]web...@we...> > >> [18]web...@we... > >> To remove yourself from this list, go to > >> <[19]http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> [20]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > > > > ---------------------------------------------------------------------------- -- > > > > - > > Forwarded by the Webmin development list at <[21]web...@we...> > > [22]web...@we... > > To remove yourself from this list, go to > > [23]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > ---------------------------------------------------------------------------- -- > > > > - > > Forwarded by the Webmin development list at [24]web...@we... > > To remove yourself from this list, go to > > [25]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > ---------------------------------------------------------------------------- -- - Forwarded by the Webmin development list at [26]web...@we... To remove yourself from this list, go to [27]http://lists.sourceforge.net/lists/listinfo/webadmin-devel 1. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 2. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom 3. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 4. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom 5. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 6. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 7. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom 8. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom 9. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 10. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 11. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 12. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom 13. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 14. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 17. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 18. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 21. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 22. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 24. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom 26. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom |