November 2009 - YaST Development - openSUSE Mailing Lists

[yast-devel] WebYaST status call 30-Nov-2009
by Jiri Srain 30 Nov '09

30 Nov '09

mschmidkunz - catching up with current status kkaempf - testing specially of registration, seems to work now - further planning for SP1 mvidner - GMC: all packages are in place, waiting for build, currently broken and being fixed - discovered missing product file on RC4, need to test that WebYaST is updateable dmacvicar - SP1 planning, trying to implement autocopletion in users module to find changes needed in back-end and REST jsuchome - testing latest RC and proposed solutions for bugs jreidinger - limited language selection for SLE11-SP1, now works - to do: fixing moved to branch schubi - testing RC4, concentrated on patch evaluation mkudlvasr - testing RC4, investigating bug sending random data to Language back-end jdsn - testing, debugging product file issue jsrain - planning for SP1 -- Regards, Jiri Srain YaST Team Leader --------------------------------------------------------------------- SUSE LINUX, s.r.o. e-mail: jsrain(a)suse.cz Lihovarska 1060/12 tel: +420 284 028 959 190 00 Praha 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] testing multiple vbox machines with network
by Martin Kudlvasr 27 Nov '09

27 Nov '09

Hi, I've just wanted to test working with several webyast appliances at the same time. This is how it worked for me: - using yast - create a bridge interface - create 2 tap interfaces - edit the bridge and include the ethX and the tapX - using virtualbox, select bridged networking and the appropriate tap interfaces. - when booting both VMs at the same time, you'll probably need to restart networking (dhcpcd of the second one kind of stalls). Enjoy, Martin -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] WebYaST status call 27-Nov-2009
by Josef Reidinger 27 Nov '09

27 Nov '09

kkaempf: SP1 planning duncanmv: no update schubi: bug fixing, testing vendor UI jdsn: test-case fot registration lslezak: testing and writing documentation for automatic testing mvidner: release RC5, with ke release doc as open source jsuchome: testing RC4 jreidinger: wrote automatic tests for the latest status bug fixes, writing remaining problems to bugzilla jsrain: no update blocker bugs - status timming issue, registration problem (no update channel for webyast ) -- Josef Reidinger YaST team maintainer of perl-Bootloader, YaST2-Repair, webyast (language,time,basesystem,ntp) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] ruby extensions for SLES11 GA?
by Martin Vidner 26 Nov '09

26 Nov '09

Hi, 2 weeks ago rubygem-mocha was updated in devel:languages:ruby:extensions and as a result, our SLES11 GA build of WebYaST broke. I solved it by including also the updated rubygems-1.3.5 there. But SLE SDK has rubygems-1.3.1. So I will modify Devel:WebYaST to link to the previous version of rubygem-mocha so that I can remove the updated rubygems again. -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 1

[yast-devel] Re: [yast-commit] <web-client> master : * fixed css problem at status wrapper * dont now, where the other files come from
by Ladislav Slezak 26 Nov '09

26 Nov '09

On 26.11.2009 10:41, Robert Lihm wrote: > ref: refs/heads/master > commit 62649a1d7c26ce7259551417f6a3b0a49ad74a59 > Author: Robert Lihm <rlihm(a)lihmedia.org> > Date: Thu Nov 26 15:11:49 2009 +0530 > > * fixed css problem at status wrapper > * dont now, where the other files come from > --- > plugins/language/app/views/language/web-client | 1 + > webclient/public/stylesheets/.style.css.swp | Bin 0 -> 24576 bytes > webclient/public/stylesheets/style.css | 8 ++++++-- You have added a *.swp file by mistake. I'll remove it and add *.swp to .gitignore files. -- Best Regards Ladislav Slezák Yast Developer ------------------------------------------------------------------------ SUSE LINUX, s.r.o. e-mail: lslezak(a)suse.cz Lihovarská 1060/12 tel: +420 284 028 960 190 00 Prague 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz/ -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] WebYaST status call 25-Nov-2009
by Josef Reidinger 26 Nov '09

26 Nov '09

duncan - no updates, evaluates features kkaempf - fix IE html bug, evaluating features schubi - fixing status module jdns - fixing bugs, fixing registration mvidner - update bugs status, releasing RC4 ( problems with istudio) jsrain - testing lslezak - fixing bugs, suggestion to create branch for final product jreidinger - restarting collectd if time is moved, add translating browser locale identifiers to our identifiers jsuchome - testing mkudlvasr - enhancing eula documentation general: if you want fix something which should not go to release create branch for it and after release we merge it as we release new features also as maintenance update for released appliance deadline on Thursday noon for packages with fixed P2 ( don't forget tests and mentioned bug numbers) -- Josef Reidinger YaST team maintainer of perl-Bootloader, YaST2-Repair, webyast (language,time,basesystem,ntp) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 1

[yast-devel] Re: [yast-commit] <web-client> master : Better indentation
by Josef Reidinger 24 Nov '09

24 Nov '09

Klaus KXXmpf write: > ref: refs/heads/master > commit d1e9f121df16fe85120b62f0d961c276e5013234 > Author: Klaus Kأ¤mpf <kkaempf(a)suse.de> > Date: Tue Nov 24 11:16:12 2009 +0100 > > Better indentation > --- > webclient/app/views/layouts/main.rhtml | 64 > ++++++++++++++++--------------- 1 files changed, 33 insertions(+), 31 > deletions(-) > > diff --git a/webclient/app/views/layouts/main.rhtml > b/webclient/app/views/layouts/main.rhtml index 6e5f44f..cd161be 100644 > --- a/webclient/app/views/layouts/main.rhtml > +++ b/webclient/app/views/layouts/main.rhtml > @@ -48,19 +48,19 @@ > <h1><%= _("My Appliance") %></h1> > > <div id="login-wrapper"> > - <% host = Host.find(session[:host]) rescue nil > - user = session[:user] > - if host.blank? || user.blank? -%> > - <% else -%> > - <ul> > - <li><%=_("Connected host:")%> <strong><%= host.name > -%></strong></li> - <li><%=_("User:")%> <strong><%= user > -%></strong></li> > - </ul> > - <ul class='header-links'> > - <li><em><%= link_to _("Control panel"), > '/controlpanel'%></em></li> - <li><em><%= link_to _("Logout"), > '/logout' %></em></li> - </ul> > - <% end -%> > + <% host = Host.find(session[:host]) rescue nil > + user = session[:user] > + if host.blank? || user.blank? -%> > + <% else -%> Ugh, WTF (I know that klaus doesn't write that code)? use unless and then see if ... which do nothing and immediately else if really not good practice. > + <ul> > + <li><%=_("Connected host:")%> <strong><%= host.name > -%></strong></li> + <li><%=_("User:")%> <strong><%= user > -%></strong></li> + </ul> > + <ul class='header-links'> > + <li><em><%= link_to _("Control panel"), > '/controlpanel'%></em></li> + <li><em><%= link_to _("Logout"), > '/logout' %></em></li> + </ul> > + <% end -%> > </div> > </div> > > @@ -68,33 +68,35 @@ > >  > <div id="flash-messages" class="grid_12 alpha"> > - <div id="language_selection"></div> > -  > + <div id="language_selection"></div> > +  > > - <% flash.each do |flash_type, message| %> > - <% > - flash_icon = case flash_type > - when :error then "ui-icon-alert" > - else "ui-icon-info" > - end > - flash_style = case flash_type > - when :error then "ui-state-error" > - else "ui-state-highlight" > - end > - %> > + <% flash.each do |flash_type, message| %> > + <% > + flash_icon = case flash_type > + when :error then "ui-icon-alert" > + else "ui-icon-info" > + end > + flash_style = case flash_type > + when :error then "ui-state-error" > + else "ui-state-highlight" > + end > + %> > > - <div class="flash-message <%= flash_style %> ui-corner-all > grid_12"> - <p><span class="ui-icon <%= flash_icon > %>"></span><strong><%= message %></strong></p> - </div> > - <% flash.delete flash_type -%> > - <% end %> > + <div class="flash-message <%= flash_style %> ui-corner-all > grid_12"> + <p><span class="ui-icon <%= flash_icon > %>"></span><strong><%= message %></strong></p> + </div> > + <% flash.delete flash_type -%> > + <% end %> > </div>  > + > <div id="progress" style="display: none;"> > <%= image_tag("/images/working.gif") %><%=_("...Wait...")%> > </div> > > <%= yield %> > </div> > + > <div id="footer" class="container_12"> > <div class="copy">© 2009 Novell, Inc.</div> > </div> > -- Josef Reidinger YaST team maintainer of perl-Bootloader, YaST2-Repair, webyast (language,time,basesystem,ntp) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] WebYaST status call 23-Nov-2009
by Josef Reidinger 24 Nov '09

24 Nov '09

kkaempf: discuss SP1 features and bugzilla prioritize schubi: translation check script, fixing collectd problems jdns: generate https certificate during build duncan: no progress mvidner: move firewall definition jsrain: no update jsuchome: working on regular yast jreidinger: fixing time moving bugs general: write comments your P1 bugs find the easiest solution not the nicest one -- Josef Reidinger YaST team maintainer of perl-Bootloader, YaST2-Repair, webyast (language,time,basesystem,ntp) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 1

[yast-devel] Re: [yast-commit] <web-client> master : create an SSL certificate with the best hostname, do not start yastwc without a certificate (bnc#557752), version 0.0.20
by Josef Reidinger 23 Nov '09

23 Nov '09

J.Daniel Schmidt write: > ref: refs/heads/master > commit c2db5a722725ee68bf539b29d8369ed71508d088 > Author: J. Daniel Schmidt <jdsn(a)suse.de> > Date: Mon Nov 23 18:18:20 2009 +0100 > > create an SSL certificate with the best hostname, do not start yastwc > without a certificate (bnc#557752), version 0.0.20 --- > webclient/package/check-create-certificate.pl | 22 > ++++++++++++++++------ webclient/package/yast2-webclient.changes | > 7 +++++++ > webclient/package/yastwc | 8 +++++++- > 3 files changed, 30 insertions(+), 7 deletions(-) > > diff --git a/webclient/package/check-create-certificate.pl > b/webclient/package/check-create-certificate.pl index e63c328..8fb27a2 > 100755 > --- a/webclient/package/check-create-certificate.pl > +++ b/webclient/package/check-create-certificate.pl > @@ -70,9 +70,9 @@ emailAddress = root\@$fqdn > chomp $CNF; > chomp $CERT; > chomp $KEY; > - if ( not defined $CNF || $CNF =~ /^$/ || > - not defined $CERT || $CERT =~ /^$/ || > - not defined $KEY || $KEY =~ /^$/ ) > + if ( (not defined $CNF ) || $CNF =~ /^$/ || > + (not defined $CERT) || $CERT =~ /^$/ || > + (not defined $KEY ) || $KEY =~ /^$/ ) Hi, few notes to your script. I think that this condition needn't use so complex statements. I think helper is better ( or lambda function - http://perl.plover.com/lambda/). But I think that you need it on more place so use helper. sub is_blank($){ my $val = shift; return not (defined $val && $val ne ""); } unless ( is_blank($CNF) || is_blank($CERT) || is_blank($KEY)) { > { > print STDERR "Could not create temporary files. Aborting.\n"; > return 0; > @@ -178,10 +178,20 @@ if (defined $create) > > $hostname = `hostname --fqdn` unless defined $hostname; > chomp $hostname if defined $hostname; > - unless (defined $hostname) > + if ( (not defined $hostname) || $hostname =~ /^$/) > { > - print STDERR "Hostname missing or invalid. Aborting.\n"; > - exit 1; > + # do not abort, just create a certificate (bnc#557752) > + #print STDERR "Hostname missing or invalid. Aborting.\n"; > + #exit 1; ^^^ we use source version system, so code changes doesn't loose. Just remove lines instead commenting out unless it is temporary or pseudo-code to describe function. > + > + print STDERR "No fully qualified domain name can be found. Please > fix your DNS setup.\n"; + print STDERR "Using only the hostname for > SSL certificate.\n"; + $hostname = `hostname`; > + chomp $hostname if defined $hostname; > + if ( (not defined $hostname) || $hostname =~ /^$/) ^ use helper defined above. My helper is, that If I must write same code twice then something is bad and my code is duplicated. > + { > + $hostname = 'localhost'; > + } > } > > if ( create_certificate( $hostname, $certfile, $keyfile, $combinedfile > ) ) diff --git a/webclient/package/yast2-webclient.changes > b/webclient/package/yast2-webclient.changes index c5f0686..5dbf1cb 100644 > --- a/webclient/package/yast2-webclient.changes > +++ b/webclient/package/yast2-webclient.changes > @@ -1,4 +1,11 @@ > ------------------------------------------------------------------- > +Mon Nov 23 18:15:24 CET 2009 - jdsn(a)suse.de > + > +- create an SSL certificate with the best hostname (bnc#557752) > +- do not start yastwc without a certificate (bnc#557752) > +- 0.0.20 > + > +------------------------------------------------------------------- > Mon Nov 23 12:19:02 CET 2009 - jdsn(a)suse.de > > - log check-create-certificate output to logfile > diff --git a/webclient/package/yastwc b/webclient/package/yastwc > index 99acd07..f25a61a 100755 > --- a/webclient/package/yastwc > +++ b/webclient/package/yastwc > @@ -179,7 +179,13 @@ case "$1" in > if [ ! -e $COMBINEDCERTFILE ] > then > echo "No certificate found. Creating one now." > - /usr/sbin/check-create-certificate.pl -c -C $CERTIFICATEFILE -K > $CERTKEYFILE -B $COMBINEDCERTFILE > >/srv/www/yast/log/check-create-certificate.log 2>&1 + if ! > /usr/sbin/check-create-certificate.pl -c -C $CERTIFICATEFILE -K > $CERTKEYFILE -B $COMBINEDCERTFILE > >/srv/www/yast/log/check-create-certificate.log 2>&1 + then > + echo -n "Can not create certificate. Please see > /srv/www/yast/log/check-create-certificate.log for details." + > rc_failed > + rc_status -v > + rc_exit > + fi > chown lighttpd:lighttpd $CERTIFICATEFILE $CERTKEYFILE > $COMBINEDCERTFILE fi > -- Josef Reidinger YaST team maintainer of perl-Bootloader, YaST2-Repair, webyast (language,time,basesystem,ntp) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] Re: [yast-commit] <web-client> master : create ssl certificate when starting webclient the first time (bnc#550395)
by Martin Vidner 23 Nov '09

23 Nov '09

On Fri, Nov 20, 2009 at 05:06:36PM +0100, J.Daniel Schmidt wrote: > ref: refs/heads/master > commit d911631162b51d8145bcf1c9d1c53eb69aa50332 > Author: J. Daniel Schmidt <jdsn(a)suse.de> > Date: Fri Nov 20 17:06:36 2009 +0100 > > create ssl certificate when starting webclient the first time (bnc#550395) > --- > webclient/config/lighttpd.conf | 2 +- > webclient/package/yast2-webclient.spec | 7 +++---- > webclient/package/yastwc | 12 ++++++++++++ > 3 files changed, 16 insertions(+), 5 deletions(-) > ... > diff --git a/webclient/package/yastwc b/webclient/package/yastwc > index d97c005..207e12b 100755 > --- a/webclient/package/yastwc > +++ b/webclient/package/yastwc > @@ -128,6 +128,11 @@ test -r $LIGHTTPD_CONFIG || { echo "$LIGHTTPD_CONFIG not existing"; > PID_FILE=/var/run/yastwc.pid > IGNORE_FILE=/var/run/yastws.pid > > + > +CERTIFICATEFILE=/etc/lighttpd/certs/webyast.pem > +CERTKEYFILE=/etc/lighttpd/certs/webyast.key > +COMBINEDCERTFILE=/etc/lighttpd/certs/webyast-combined.pem > + > # Source LSB init functions > # providing start_daemon, killproc, pidofproc, > # log_success_msg, log_failure_msg and log_warning_msg. > @@ -171,6 +176,13 @@ rc_reset > > case "$1" in > start) > + if [ ! -e $COMBINEDCERTFILE ] > + then > + echo "No certificate found. Creating one now." > + /usr/sbin/check-create-certificate.pl -c -C $CERTIFICATEFILE -K $CERTKEYFILE -B $COMBINEDCERTFILE >/dev/null 2>&1 Do not throw away errors, especially if they can affect the operation of the entire web client. Others will not thank you after wasting time on trying to figure out what went wrong. How about >/srv/www/yast/log/check-create-certificate.log 2>&1 > + chown lighttpd:lighttpd $CERTIFICATEFILE $CERTKEYFILE $COMBINEDCERTFILE > + fi > + > echo -n "Starting yastwc " > startproc -p $PID_FILE -i $IGNORE_FILE $LIGHTTPD_BIN -f $LIGHTTPD_CONFIG > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0