|
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
|