/srv/irclogs.ubuntu.com/2006/09/18/#upstart.txt

=== Keybuk [n=scott@syndicate.netsplit.com] has joined #upstart
=== wasabi__ [n=wasabi@c-67-166-172-239.hsd1.tx.comcast.net] has joined #upstart
=== wasabi__ [n=wasabi@c-67-166-172-239.hsd1.tx.comcast.net] has joined #upstart
=== mbiebl [n=michael@dslb-084-056-226-102.pools.arcor-ip.net] has joined #upstart
mbieblKeybuk, hi02:28
Keybukello02:29
mbieblI currently studying the code and noticed, that "pid file" does not yet have any effect.02:30
mbieblinit/job.c -> job_run_process always uses the pid returned from fork.02:31
mbieblIf job->pidfile is specified it should parse the given file for the pid file instead.02:33
mbieblI also don't understand, why "pid file" can only be specified together with respawn.02:33
Keybukright, it's not implemented02:34
mbieblImho it would also make sense together with exec.02:34
Keybukwas supposed to only make sense with daemon :p02:34
mbieblYeah, but a daemon can also be started with exec, or not?02:34
Keybukyes02:34
mbieblI also think, that a reload and maybe also a restart utility would be great.02:35
Keybukthere's no implementation for reload yet02:35
mbieblIt should also be possible to specify in the job description file a custom reload function.02:36
Keybukprobably yes02:37
Keybukactions are in the todo somewhere02:37
Keybukgot to get events and jobs right first though02:37
mbieblSure ;-)02:37
mbieblOh, another thing, more from the packaging perspective:02:38
mbieblin postinst you call kill directly. Maybe it would be safer to specify the complete path as kill is a bash builtin02:38
mbiebliirc lintian also moans about that.02:38
Keybukwhy would you need to do that?02:39
Keybukwhat's wrong with the builtin?02:39
mbieblThey could behave differently and you don't notice as long as you run bash.02:39
Keybukthey'd at least behave consistently02:39
Keybukit's /bin/sh02:39
Keybukso kill must behave like POSIX kill02:39
Keybukkill -TERM 1 is therefore legal02:40
mbieblIs it guaranteed that the bash builtin kill behaves exactly as /bin/kill?02:40
mbieblI'm not sure.02:40
Keybukyes02:40
Keybukwhat's bash got to do with it anyway?02:40
Keybuk/bin/sh on Ubuntu is dash02:40
mbieblWell, if the builtin and /bin/kill behave exactly the same, then the point is moot.02:42
Keybukindeed02:42
mbieblYou seem confident so I take this point as resolved ;-)02:43
Keybukwell, if a shell installs itself as /bin/sh and doesn't implement the POSIX-specified behaviour of kill, then that's a bug in that shell02:43
Keybukif the user replaces /bin/sh with a non-POSIX shell like zsh, and complains; then the bug can just be rejected :p02:43
mbieblAs you already have feature freeze for edgy I was wondering if you also plan to migrate initscripts or parts of initscripts to upstart jobs.02:46
Keybuknot in edgy02:46
Keybukbeta freeze is only a few days away! :P02:48
Keybukand have other bugs to be fixed for that02:48
=== wasabi__ [n=wasabi@c-67-166-172-239.hsd1.tx.comcast.net] has joined #upstart
mbieblyeah, bug fixing should always have top priority, even if it's not so much fun :-)02:55
Keybukfortunately there don't seem to be any upstart bugs02:56
mbieblThat speaks for your coding skills03:01
Keybukwell, no major ones anywhere03:02
=== Seveaz [n=seveas@ubuntu/member/seveas] has joined #upstart
=== Seveaz is now known as Seveas
=== janm [n=jmalonzo@ppp4592.dsl.pacific.net.au] has joined #upstart
=== Md [i=md@freenode/staff/md] has joined #upstart
=== mbiebl [n=michael@dslb-084-056-219-235.pools.arcor-ip.net] has joined #upstart
Admiral_Chicagoi'm on Knot 3 and i did an apt-get update11:19
Admiral_Chicagoinitscript is a package that is being updated11:19
Admiral_Chicagois that right?11:19
=== Keybuk [n=scott@quest.netsplit.com] has joined #upstart
=== nmbooker [n=nick@80-47-232-107.lond-th.dynamic.dial.as9105.com] has joined #upstart
=== nmbooker [n=nick@80-47-232-107.lond-th.dynamic.dial.as9105.com] has left #upstart ["Ex-Chat"]
=== jbailey [n=jbailey@montreal.canonical.com] has joined #upstart
jbaileyKeybuk: The only two problems I had were 1) The upgrade also included glibc and I was still running the hack'd upstart.  It wasn't taking signals, glibc locked in postinst. =)01:27
jbailey2) In single user mode, terminal type seems to be not set correctly, and "reboot" command didn't work.01:27
jbaileyAlthough from a command prompt now, C-M-Del seems to get registered correctly now.01:27
Keybukjbailey: isn't glibc supposed to work unconfigured? :)01:31
Keybukhmm, will test the single user mode thing01:32
Keybukreboot should work even better than sysvinit (no pesky "unable to obtain runlevel" type thing)01:32
jbaileyKeybuk: It does.  It was that the postinst I think HUPs pid 1 or something like that.01:32
jbaileyKeybuk: So I killall'd that, then dpkg --force-depends -i upstart*deb01:32
jbaileyReboot, finished upgrade ;)01:32
Keybukoh, and you had the hack'd upstart that didn't work? :p01:32
pepsimanKeybuk: single user doesn't run bash01:33
jbaileyRight. =)01:33
Keybukpepsiman: err, yes?01:34
pepsimanit did in dapper01:34
Keybukactually it ran sulogin in dapper ;)01:34
pepsimanyes, and sulogin looked at /etc/passwd in dapper01:34
Keybukright01:34
Keybukcan probably hack that01:34
Keybukor just use the real sulogin01:35
pepsimanbug 6096501:35
jbaileyKeybuk: Path of least resistance: use the real sulogin.01:35
Keybukpepsiman: yes, I saw it01:35
jbaileySolve that problem sometime in +1 =)01:35
Keybukjbailey: yeah, was planning to move that to sysvutils too, just hadn't got around to it yet01:35
Keybukwas gonna do that along with fixing check*01:35
jbaileyCool.01:36
jbaileySo for ppc64, I think it's properly cooked now.  I should test on the older kernel as well, I s'pose.01:37
jbaileyActually, I'll focus on getting my HD working with the stock edgy kernel so I don't have to roll my own.  Easier.01:37
Keybukjbailey: oh, I see why shutdown doesn't work in single-user mode01:39
jbaileyKeybuk: Cool.  Need something in Malone, or is this good enough?01:40
Keybuknot sure it's actually fixable :-/01:40
Keybuknot for edgy, anyway01:40
jbaileyErr...01:40
jbaileyNeed to have *some* way of getting back to a full state.01:40
jbaileyexit didn't work either, IIRC01:40
Keybukexit should work01:41
Keybukit's making exit work that's causing shutdown to not work01:41
Keybukoh, hmm01:41
Keybukexit doesn't work either01:41
Keybukmeh01:41
Keybukyes, bug me01:41
jbaileyTwo bugs or one?01:41
Keybukonly one of them will be fixable01:42
Keybukreboot is more important, so I'll fix that01:42
pepsimaninit: sulogin process (3353) killed by signal 901:44
pepsimanSystem halted.01:44
=== pepsiman wonders why poweroff wasn't killed that time
Keybukhmm? poweroff shouldn't get killed?01:47
pepsimanthat was the 4th attempt at poweroff01:48
Keybukjbailey: the reboot bug is that there's currently no way for upstart to tell the difference between sulogin being killed by a signal or the user typing "exit"01:48
pepsimanhow did sysvinit tell the difference?01:49
Keybukrunlevel change01:49
pepsimanah01:50
Keybukit's a bug, or at least a lack of an implementation in upstart that causes this bug01:51
Keybukbut it's not something I can implement before Beta Freeze01:51
jbaileyKeybuk: rewind a sec for me?  what kills sulogin with a signal?01:51
Keybukwhich is IN THREE DAYS TIME01:51
KeybukPANIC!!!!01:51
jbaileyIs that a sideeffect of runlevel change?01:51
Keybukjbailey: sendsigs ;)01:51
Keybukand killing sulogin forces a runlevel change, stopping the running reboot01:51
jbaileyOh.  Weird design.01:51
Keybukwell, how would you have done it?01:52
jbaileyI'm not sure. =)01:54
KeybukI couldn't remember what happened with sysvinit if you ran telinit from the shutdown process :p01:54
=== theCore [n=alex@modemcable106.200-70-69.mc.videotron.ca] has joined #upstart
=== Keybuk [n=scott@quest.netsplit.com] has joined #upstart
=== mbiebl [n=michael@tecov6131.teco.uni-karlsruhe.de] has joined #upstart
theCoreupstart is so boring... there is barely any bugs to find :)02:37
mbieblKeybuk: debian/control: system-services: Is there a reason why you explicitely depend on util-linux?02:41
mbieblutil-linux is essential, so it shouldn't be necessary to add this dep.02:41
KeybukI don't remember, no02:45
mbieblWould make one lintian error less.02:48
Mdmbiebl: are there any news about the sysvinit split?02:49
Keybukheh, I entirely got out of the habit of running lintian when I was maintaining dpkg02:53
Keybukdon't think I've ever used it since02:53
mbieblMd: It's currently sitting in NEW.02:53
Keybukjbailey: looks like there's already a bug for the single-user problem03:07
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #upstart
=== juergbi [n=juerg@80-219-26-249.dclient.hispeed.ch] has joined #upstart
=== mbiebl [n=michael@dslb-084-056-242-121.pools.arcor-ip.net] has joined #upstart
=== wasabi [n=wasabi@ubuntu/member/wasabi] has joined #upstart
=== j_ack [n=rudi@p508D92C8.dip0.t-ipconnect.de] has joined #upstart
=== Admiral_Chicago [n=freddy@st0660990722.monm.edu] has joined #upstart
!Rez:*! :Hi, xFallenAngel would like to inform you that they have created ##google-apis for discussion of GAPIs (http://code.google.com/apis.html), if you are interested we would urge you to join. Thank you for using freenode and have a good day!09:02
=== mbiebl [n=michael@dslb-084-056-242-121.pools.arcor-ip.net] has joined #upstart
=== theCore [n=alex@modemcable106.200-70-69.mc.videotron.ca] has joined #upstart
=== che [n=che@unaffiliated/che] has joined #upstart
=== j_ack [n=rudi@p508D92C8.dip0.t-ipconnect.de] has joined #upstart

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!