/srv/irclogs.ubuntu.com/2022/02/18/#ubuntu-server.txt

cpaelzergood morning05:51
mirespacegood morning08:27
mirespacehi, I'm creating the logical tag manually when merging amavisd-new, but version contains ':'  ... could I use lp1946841/logical/1_2.11.1-5ubuntu1 instead of lp1946841/logical/1:2.11.1-5ubuntu1?10:34
ahasenackmorning11:26
athosgood morning!12:08
mirespacehi athos, andreas12:22
ahasenackhello miraspace12:23
ahasenackhello athos12:23
ahasenacksergiodj: hi, morning, around yet?12:52
ahasenackthis test stared failing, it's like it's not setting the admin password correctly anymore. It used to work: https://pastebin.ubuntu.com/p/BSRMJtDswk/12:55
ahasenackhm, something fishy is going on13:04
paridemirespace, you probably already sorted that out by now :) anyway yes, but I think it's more common to use a '%' to replace ':' in those cases14:05
paridethe advantage of '%' is that it isn't a valid version string character, so it's made clear that the original version string has been modified14:07
paridebut that's it14:07
rbasakFWIW, it's the standard for escaping Debian version strings into git tag names: https://dep-team.pages.debian.net/deps/dep14/14:13
sergiodjahasenack: hi14:15
ahasenacksergiodj: troubleshooting this weird problem, it's like debconf-set-selections stopped working14:16
=== Nafallo_ is now known as Nafallo
sergiodjahasenack: does this seem like an openldap problem?  because the test was passing normally before14:18
sergiodjand no changes have been made to the package since then14:18
ahasenackstill troubleshooting14:21
ahasenackthe one thing I spotted so far was that the domain (example.com) had a leading space ( example.com)14:22
ahasenackthat might have tripped a regexp check14:22
ahasenackbut also can't say why it became a problem now14:22
ahasenack /var/cache/debconf/passwords.dat has no password set, from the user (password{1,2}) or internally generated14:22
ahasenack /var/cache/debconf/config.dat has the domain name and org correct14:23
ahasenacksergiodj: the set-selection run was fine: https://pastebin.ubuntu.com/p/9zQv43jpnb/14:24
ahasenackwhy didn't this work: info: Trying to set 'slapd/password1' [password] to 'secret'14:24
ahasenackslapcat confirms the example.com domain was used: olcSuffix: dc=example,dc=com14:25
ahasenackand *some* password was set, but it's a hash, I can't tell what it is14:26
ahasenackof course14:26
ahasenackolcRootDN: cn=admin,dc=example,dc=com14:26
ahasenackolcRootPW:: e1NTSEF9b1phRGhxWVpzNk1qWGljMm9PNzYzb0RKUlZ6OVArWWI=14:26
* ahasenack scratches head14:26
ahasenackit's lost after the reconfigure: https://pastebin.ubuntu.com/p/m6sWqVvy37/14:38
ahasenackbut just the passwords14:38
ahasenackthe rest (domain, org) are still set14:39
ahasenackoh, ok14:51
ahasenack    # Now that we created the new directory we don't need the passwords in the14:51
ahasenack    # debconf database anymore. So wipe them.14:51
ahasenack    wipe_admin_pass14:51
ahasenackso that's a, what is it called again, red herring?14:52
ahasenackanyway14:52
ahasenackproblem is not there14:52
ahasenackworked after a slapd restart... :(14:55
ahasenackthat's... unexpected14:56
* ahasenack has lunch14:57
* oden heads home15:01
sergiodjahasenack: I'm trying to debug multipath-tools here, but I'm keeping an eye on your progress15:02
mirespaceparide: yuo're right :), I got that a while a go ... but thanks a lot for answering!15:35
ahasenacksergiodj: I bet it's that debhelper bug with restarts15:50
ahasenackwhere was that email with the list of packages that needed rebuilding15:51
sergiodjlet me find it15:51
ahasenackit's in the bug15:52
ahasenack#195905415:52
sergiodjyeah15:52
ahasenackand openldap is in the list15:52
sergiodjthere you go15:53
ahasenackopenldap (2.5.11+dfsg-1~exp1ubuntu3) jammy; urgency=medium15:53
ahasenack  * No-change rebuild to update maintainer scripts, see LP: 195905415:53
ubottuLaunchpad bug 1959054 in libvirt (Ubuntu Jammy) "debhelper restarts services marked --no-restart-on-upgrade" [Undecided, Triaged] https://launchpad.net/bugs/195905415:53
ahasenack1u3 should be fine then15:53
ahasenackbut it's what I'm using15:53
ahasenackso back to square 115:53
ahasenackor maybe square 1.515:53
cpaelzerahasenack: sergiodj: as mentioned in standup - those rebuilds were without -v so if you had a merge in proposed the old changelogs will be gone15:54
cpaelzerno auto-close of your LP bugs anymore :-)15:54
ahasenackthe changelog mentions the rebuild explicitly15:55
ahasenackI have to see what postinst does15:55
ahasenackit's the most complicated postinst I have ever seen in a package15:55
sergiodjI had to hack on it a lot during last cycle15:56
sergiodjI'll try to look into it later15:56
sergiodjstill working on multipath-tools/postgresql now, so my plate is currently full :(15:57
ahasenackit does call this at the very end15:58
ahasenack+ update-rc.d slapd defaults 19 8015:58
ahasenack+ invoke-rc.d --skip-systemd-native slapd start15:58
ahasenackbut it's running, so that should be working15:58
=== genii-core is now known as genii
bryycerbasak, https://git.launchpad.net/~bryce/+git/excuses-kicker/ is the tool I've been using.  16:23
ahasenacksergiodj: got it16:30
ahasenackwhat's happening, I mean, not the cause yet16:30
ahasenackopenldap from the release pocket works16:30
ahasenackthe one from proposed doesn't16:30
ahasenackso good test, caught a real problem16:30
ahasenackwaveform: around? I might need you and your debhelper restart thingy fix16:30
waveformahasenack, I'm here16:30
waveformthe fix for the debhelper restart problem *should* be in proposed IIRC16:31
ahasenackwaveform: sergiodj how it works in the release pocket for openldap: https://pastebin.ubuntu.com/p/FF4JytPQmq/16:31
ahasenackkey thing is ldapwhoami working with that password16:31
ahasenackand we can see that slapd was restarted after dpkg-reconfigure16:31
ahasenacknow with proposed:16:31
ahasenackhttps://pastebin.ubuntu.com/p/xgMRrzGX9g/16:32
ahasenackno restart after dpkg-reconfigure, ldapwhoami is not working until I restart slapd myself16:32
ahasenack openldap | 2.5.11+dfsg-1~exp1ubuntu1 | jammy           | source16:33
ahasenack openldap | 2.5.11+dfsg-1~exp1ubuntu3 | jammy-proposed  | source16:33
ahasenacku3 and u2 are no-change rebuilds16:33
ahasenackwaveform: what is the binary package that has your fix? debhelper?16:35
waveformthat's correct -- the debhelper 13.6ubuntu1 merge also incorporated the changes to restart behaviour16:35
ahasenackPreparing to unpack .../35-debhelper_13.6ubuntu1_all.deb ...16:36
waveformI'll have a look at the what's been generated in the openldap maintscripts in proposed16:36
ahasenackand it was used in the openldap u3 build16:36
waveformhmmm ... I can see the start line in the generated postinst, which is correct ... let's see the prerm16:39
ahasenackwaveform: this is my quick diff: https://pastebin.ubuntu.com/p/GmYFjfHQR9/16:41
ahasenacks/restart/start/ ?16:42
waveformno, that much is correct -- that was the root of the restart issue16:42
waveformno, I see the issue16:42
ahasenackdpkg-reconfigure runs that16:42
waveformit's only *stopping* the service on removal, rather than unconditionally16:42
waveform(it should be unconditional in the case of something using --no-restart-after-upgrade which openldap is)16:43
waveformdamn16:43
ahasenackfirst, let me thank you for working on this very thorny issue :)16:44
ahasenackit's the kind of thing that would keep me awake at night after I uploaded16:44
ahasenackwaveform: is that enough to file a bug?16:48
waveformhang on, I'll dig out the original bug -- it may as well go there as this'll need to be fixed upstream in debian too and that's got the link16:49
waveformLP: #195905416:49
ubottuLaunchpad bug 1959054 in libvirt (Ubuntu Jammy) "debhelper restarts services marked --no-restart-on-upgrade" [Undecided, Triaged] https://launchpad.net/bugs/195905416:49
ahasenackok, I'll paste the diff there (or attach)16:49
waveformyeah, the relevant bit is the prerm one16:50
waveformdoubly annoying because I knew openldap was one of the affected packages (not affected by the restart bug in that issue *directly*, but by the fix that caused the restart bug in that issue, and which I had to undo and redo differently to fix that issue)16:50
waveformthought I'd checked the maintscripts were being generated correctly, but evidently not16:51
ahasenackthe openldap maintscripts are very complicated16:51
waveformoh, hang on -- no, the prerm was *meant* to only stop on upgrade; it's the pre*inst* that's meant to handle the stop in the --no-restart-after-upgrade scenario now ... just a mo17:02
waveformstop on remove even ... bah17:02
waveformthat was the whole crux of the openldap issue (sorry, too many things on fire today -- only just swapping back all the debhelper stuff into my head)17:04
ahasenackcareful with those juggling knives, they are sharp :)17:04
waveformthis dpkg-reconfigure sequence ... doesn't seem to be running all the maintscripts? Theoretically what *should* be happening with the scripts generated by the new debhelper (for openldap specifically) is: prerm does nothing (previously it stopped the service but it shouldn't anymore); preinst stops the service (if $1 == upgrade, which it presumably should); postinst starts the service again (if $1 == configure which it should do)17:08
rbasakbryyce: no docs :-/17:09
ahasenackwaveform: if you want I can add set -x to all of them and run dpkg-reconfigure17:11
waveformahasenack, that could be helpful -- I have a sneaking suspicion it's running the postinst (which now just "starts" the service instead of "restart" -- it can't do the latter as that was the crux of the restart-on-no-restart bug), but it's skipping the preinst (which would stop the service) or it's running it with some parameter set which is causing it to skip stopping the service17:12
ahasenackok17:19
bryycerbasak, yeah sorry, there is `excuses-kicker -h` but probably would help to have some better docs17:29
ahasenackwaveform: attached17:40
waveformahasenack, thanks -- I'll take a look this evening17:41
ahasenackcheers17:41
mirespacehi, I need review on the MP for amavisd-new merge https://code.launchpad.net/~mirespace/ubuntu/+source/amavisd-new/+git/amavisd-new/+merge/41580818:13
mirespacethanks in advance, of course18:35
ahasenackmirespace: I'm taking a look18:54
sergiodjahasenack: thanks for the time spent on openldap19:18
ahasenacknp, an sssd test is failing because of it, and I need sssd to pass due to nfs-utils (it's a rdeps) :)19:18
waveformahasenack, okay -- figured it out. To my way of thinking this is a bug in dpkg-reconfigure which has been exposed as a result of the debhelper fix. I'll write up the details in the original bug (and upstream in Debian), and propose a (trivial) patch for dpkg-reconfigure (and then wait and see what breaks as a result of *that* ... urgh, what a rabbit hole!)19:34
ahasenackugh, nasty19:35
ahasenackthanks for digging in19:35
waveformthe crux of it, though, is that dpkg-reconfigure runs d/prerm (incorrectly IMO because that's meant to be from the *prior* version of the package, but it's where the "service stop" went historically so that's probably why), skips the d/preinst (again, incorrectly, but historically almost nothing used it and it's only now it's used as the new location for "service stop"), runs d/config (correctly), skips d/postrm (correctly), then runs d/19:36
waveformpostinst (which runs "service start")19:36
waveform(so the "correct" fix would be "change out running d/prerm for d/preinst" but that'll almost certainly break a ton of stuff so I'll propose adding d/preinst to the list of things run and leaving d/prerm alone for now)19:37
bryyceI'd like to start jotting down docs on "how to transition php".  Where would be the best place to put it?20:13
bryycefor now I'll work on adding it to the UMH, unless anyone has a better idea20:14
ahasenackpart of me says wiki.ubuntu.com20:17
ahasenackbut then another part says nope20:17
bryyceheh20:17
ahasenackmaybe in the future the server guide could grow a development component20:17
ahasenackbut UMH works20:17
ahasenack(and it's markdown)20:18
ahasenackbiiig plus20:18
bryyceyeah I used the wiki up til now - https://wiki.ubuntu.com/ServerTeam/Transition/Php8.120:18
sarnoldumh?20:18
bryycebut yeah markdown and being able to edit with $EDITOR would be nice20:18
sbeattieif'n y'all figure out a place to publish markdown for developer guides, I'd love to do the same for the security team's processes documentation.20:19
sarnoldohai sbeattie, funny seeing you here.. :)20:19
bryyceumh = https://github.com/canonical/ubuntu-maintainers-handbook20:19
ahasenackis there an ubuntu namespace in github?20:22
ahasenackhttps://github.com/ubuntu/ yep20:22
bryycesbeattie, github's been working well for us.  We can RTFM to brand new hires even before their creds are set up :-)20:23
sbeattiebryyce: ah yeah, nice.20:26
mirespaceahasenack: thank you for the review :) 21:03
mirespaceHave you all a nice weekend21:03
bryycecya mirespace 21:19
waveformooh, that's some nice documentation (re: umh) -- is it built anywhere? (i.e. rtd-ish?)22:03
=== JanC is now known as Guest5415
=== JanC_ is now known as JanC

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