From: Todd K. <tod...@gm...> - 2009-06-26 03:52:47
|
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 |
From: Jamie C. <jca...@we...> - 2009-06-26 07:51:45
|
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 [1]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 |
From: Todd K. <tod...@gm...> - 2009-06-26 13:22:26
|
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. 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. On Fri, Jun 26, 2009 at 3:51 AM, Jamie Cameron <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 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 web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > |
From: Jamie C. <jca...@we...> - 2009-06-28 06:50:40
|
On 26/Jun/2009 06:22 Todd Kennedy <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 <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 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 web...@we... > > To remove yourself from this list, go to > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > |
From: Todd K. <tod...@gm...> - 2009-06-28 23:34:20
|
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 <jca...@we...> wrote: > On 26/Jun/2009 06:22 Todd Kennedy <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 <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 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 web...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > > ------------------------------------------------------------------------------ > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > |
From: Jamie C. <jca...@we...> - 2009-06-29 02:14:04
|
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 <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 <jca...@we...> > wrote: > On 26/Jun/2009 06:22 Todd Kennedy <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 > <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 > 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 webmin- > de...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > --- > --- > --- > --------------------------------------------------------------------- > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > --- > --- > --- > --------------------------------------------------------------------- > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel |
From: Todd K. <tod...@gm...> - 2009-06-30 21:57:14
Attachments:
webmin_solaris_zone.patch
|
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. 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... 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. 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... Havin' Fun with Code, -Todd- On Sun, Jun 28, 2009 at 10:13 PM, Jamie Cameron <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 <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 < <jca...@we...> > jca...@we...> wrote: > >> On 26/Jun/2009 06:22 Todd Kennedy < <tod...@gm...> >> 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 < <jca...@we...> >> 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 <jca...@we...> >> 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 <web...@we...> >> web...@we... >> > > To remove yourself from this list, go to >> > > <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > > >> > > >> >> >> ------------------------------------------------------------------------------ >> - >> Forwarded by the Webmin development list at <web...@we...> >> web...@we... >> To remove yourself from this list, go to >> <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > > > ------------------------------------------------------------------------------ > > - > Forwarded by the Webmin development list at <web...@we...> > web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > ------------------------------------------------------------------------------ > > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > |
From: Jamie C. <jca...@we...> - 2009-07-01 05:47:54
|
On 30/Jun/2009 14:57 Todd Kennedy <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 <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 <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 < <jca...@we...> > > jca...@we...> wrote: > > > >> On 26/Jun/2009 06:22 Todd Kennedy < <tod...@gm...> > >> 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 < <jca...@we...> > >> 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 <jca...@we...> > >> 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 <web...@we...> > >> web...@we... > >> > > To remove yourself from this list, go to > >> > > <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > >> > > > >> > >> > >> ------------------------------------------------------------------------------ > >> - > >> Forwarded by the Webmin development list at <web...@we...> > >> web...@we... > >> To remove yourself from this list, go to > >> <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > > > > ------------------------------------------------------------------------------ > > > > - > > Forwarded by the Webmin development list at <web...@we...> > > web...@we... > > To remove yourself from this list, go to > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > ------------------------------------------------------------------------------ > > > > - > > Forwarded by the Webmin development list at web...@we... > > To remove yourself from this list, go to > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > |
From: Todd K. <tod...@gm...> - 2009-07-02 18:12:29
|
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 <jca...@we...> wrote: > On 30/Jun/2009 14:57 Todd Kennedy <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 <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 <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 < <jca...@we...> > > > jca...@we...> wrote: > > > > > >> On 26/Jun/2009 06:22 Todd Kennedy < <tod...@gm...> > > >> 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 < < > jca...@we...> > > >> 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 <jca...@we... > > > > >> 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 < > web...@we...> > > >> web...@we... > > >> > > To remove yourself from this list, go to > > >> > > <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > >> > > > > >> > > > > >> > > >> > > >> > ------------------------------------------------------------------------------ > > >> - > > >> Forwarded by the Webmin development list at <web...@we...> > > >> web...@we... > > >> To remove yourself from this list, go to > > >> <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > >> > > > > > > > > > > ------------------------------------------------------------------------------ > > > > > > - > > > Forwarded by the Webmin development list at <web...@we...> > > > web...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > > > > - > > > Forwarded by the Webmin development list at web...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > > ------------------------------------------------------------------------------ > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > |
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 |
From: Todd K. <tod...@gm...> - 2009-07-03 00:52:35
Attachments:
solaris_zone_patch2.patch
|
Here's the new patch...I fixed the index.cgi return that I missed in the save_zone.cgi and also I found that if you uninstall the zone but don't delete it and reinstall..."which I do allot of" you have to perform the manual sysidcfg. Instead of moving the created sysidcfg to the newzone prior to booting I replaced that with a copy command instead. This way if you do uninstall and reinstall without recreating the zone it's not an issue. Then when you delete the zone the sysidcfg in the /etc/webmin/zones directory will get removed. So it seems to work out. I have some more ideas that I want to do but might take some time, like install from a flar image and possibly watch the console messages as the zone boots. On Thu, Jul 2, 2009 at 3:00 PM, Jamie Cameron <jca...@we...> wrote: > 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 <jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom> > > wrote: > >> On 30/Jun/2009 14:57 Todd Kennedy <tod...@gm...<http://reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom>> >> 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 <jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom>> >> 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 <tod...@gm...<http://reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom>> >> 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 < <jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom> >> > >> > > jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom>> >> wrote: >> > > >> > >> On 26/Jun/2009 06:22 Todd Kennedy < <tod...@gm...<http://reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom> >> > >> > >> tod...@gm...<http://reply_mail.cgi?new=1&to=toddkenn%40gmail%2Ecom>> >> 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 < < >> jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom> >> > >> > >> jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom>> >> 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 < >> jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom> >> > >> > >> jca...@we...<http://reply_mail.cgi?new=1&to=jcameron%40webmin%2Ecom>, 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 < >> web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > >> > >> web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > >> > > To remove yourself from this list, go to >> > >> > > <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> >> > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > >> > > >> > >> > > >> > >> >> > >> >> > >> >> ------------------------------------------------------------------------------ >> > >> - >> > >> Forwarded by the Webmin development list at <web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > >> > >> web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > >> To remove yourself from this list, go to >> > >> <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> >> > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > >> >> > > >> > > >> > > >> ------------------------------------------------------------------------------ >> > > >> > > - >> > > Forwarded by the Webmin development list at <web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > >> > > web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > > To remove yourself from this list, go to >> > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > > >> > > >> > > >> > > >> ------------------------------------------------------------------------------ >> > > >> > > - >> > > Forwarded by the Webmin development list at web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> > > To remove yourself from this list, go to >> > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > > >> > > >> >> >> ------------------------------------------------------------------------------ >> - >> Forwarded by the Webmin development list at web...@we...<http://reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom> >> To remove yourself from this list, go to >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel >> > > > > > ------------------------------------------------------------------------------ > > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > |
From: Jamie C. <jca...@we...> - 2009-07-03 18:44:06
|
Awesome, thanks again .. I will include this in Webmin 1.490. On 02/Jul/2009 17:20 Todd Kennedy wrote .. Here's the new patch...I fixed the index.cgi return that I missed in the save_zone.cgi and also I found that if you uninstall the zone but don't delete it and reinstall..."which I do allot of" you have to perform the manual sysidcfg. Instead of moving the created sysidcfg to the newzone prior to booting I replaced that with a copy command instead. This way if you do uninstall and reinstall without recreating the zone it's not an issue. Then when you delete the zone the sysidcfg in the /etc/webmin/zones directory will get removed. So it seems to work out. I have some more ideas that I want to do but might take some time, like install from a flar image and possibly watch the console messages as the zone boots. On Thu, Jul 2, 2009 at 3:00 PM, Jamie Cameron <[1]jca...@we...> wrote: 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 <[2]jca...@we...> wrote: On 30/Jun/2009 14:57 Todd Kennedy <[3]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 <[4]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 <[5]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 < <[6]jca...@we...> > > [7]jca...@we...> wrote: > > > >> On 26/Jun/2009 06:22 Todd Kennedy < <[8]tod...@gm...> > >> [9]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 < <[10]jca...@we...> > >> [11]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 <[12]jca...@we...> > >> [13]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 <[14]web...@we...> > >> [15]web...@we... > >> > > To remove yourself from this list, go to > >> > > <[16]http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> [17]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > >> > > > >> > >> > >> ---------------------------------------------------------------------------- -- > >> - > >> Forwarded by the Webmin development list at <[18]web...@we...> > >> [19]web...@we... > >> To remove yourself from this list, go to > >> <[20]http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > >> [21]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > >> > > > > > > ---------------------------------------------------------------------------- -- > > > > - > > Forwarded by the Webmin development list at <[22]web...@we...> > > [23]web...@we... > > To remove yourself from this list, go to > > [24]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > ---------------------------------------------------------------------------- -- > > > > - > > Forwarded by the Webmin development list at [25]web...@we... > > To remove yourself from this list, go to > > [26]http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > ---------------------------------------------------------------------------- -- - Forwarded by the Webmin development list at [27]web...@we... To remove yourself from this list, go to [28]http://lists.sourceforge.net/lists/listinfo/webadmin-devel -------------------------------------------------------------------------- ---- - Forwarded by the Webmin development list at [29]web...@we... To remove yourself from this list, go to [30]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 29. file://localhost/home/webmail/homes/jcameron/.tmp/reply_mail.cgi?new=1&to=webmin%2Ddevel%40webmin%2Ecom |
From: Todd K. <tod...@gm...> - 2009-07-03 04:11:37
|
Yeah I saw the sysidcfg entries in the module...When I boot the zone for the first time the only question it asked was which version of NFS did I want to use...NFS4, etc .... So maybe just the sysidcfg needs to be updated. I'll look at it and see what I can come up with. On Wed, Jul 1, 2009 at 1:46 AM, Jamie Cameron <jca...@we...> wrote: > On 30/Jun/2009 14:57 Todd Kennedy <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 <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 <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 < <jca...@we...> > > > jca...@we...> wrote: > > > > > >> On 26/Jun/2009 06:22 Todd Kennedy < <tod...@gm...> > > >> 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 < < > jca...@we...> > > >> 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 <jca...@we... > > > > >> 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 < > web...@we...> > > >> web...@we... > > >> > > To remove yourself from this list, go to > > >> > > <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > >> > > > > >> > > > > >> > > >> > > >> > ------------------------------------------------------------------------------ > > >> - > > >> Forwarded by the Webmin development list at <web...@we...> > > >> web...@we... > > >> To remove yourself from this list, go to > > >> <http://lists.sourceforge.net/lists/listinfo/webadmin-devel> > > >> http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > >> > > > > > > > > > > ------------------------------------------------------------------------------ > > > > > > - > > > Forwarded by the Webmin development list at <web...@we...> > > > web...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > > > > - > > > Forwarded by the Webmin development list at web...@we... > > > To remove yourself from this list, go to > > > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > > > > > > > > > ------------------------------------------------------------------------------ > - > Forwarded by the Webmin development list at web...@we... > To remove yourself from this list, go to > http://lists.sourceforge.net/lists/listinfo/webadmin-devel > |