[00:08] <nacc> ha nm, it's a bug in the perl
[00:08] <nacc> sigh
[00:11] <nacc> can anyone see why dpkg-parsechangelog says this is an invalid date?
[00:11] <nacc> Tue, 17 May 2008 10:93:55 -0500
[00:11] <nacc> lol, just saw it
[00:11] <nacc> 10:93
[00:31] <nacc> i wonder how that happened? fat-fingered a manual changelog edit?
[00:31] <nacc> smoser: --^ so we could do a parent override for that too
[00:37] <smoser> nacc, shoot. i must have looked at those wrong. sorry. the pcre3 and iscsitarget.
[00:37] <smoser> ohwell
[00:40] <nacc> smoser: it's fine
[00:40] <nacc> the iscsitarget one is fun
[00:40] <nacc> we're stripping the string we get back
[00:40] <nacc> so ' <ogra@ubuntu.com>' which would parse
[00:40] <nacc> gets turned into '<ogra@ubuntu.com>' which does not :)
[00:41] <smoser> need to make usd <subcommand>
[00:41] <nacc> smoser: yep that's on my todo for tmrw
[00:41] <smoser> and convert clone to python
[00:41] <nacc> yep
[00:41] <nacc> i have one already going for that
[00:41] <nacc> a branch i mean
[00:41] <nacc> and for merge
[00:41] <nacc> that's my last main coding change i konw i'd like to be done with for 1.0
[00:41] <nacc> beyond these bugfxies
[00:42] <smoser> the removal of dl-cache...
[00:42] <nacc> hrm?
[00:42] <smoser> it only gets set to true if provided on cmdline
[00:43] <smoser> 57cc02fb81675a2511400ff52f4a21c2a3f1100a
[00:43] <smoser> you didnt need to do that
[00:43] <nacc> yes i understood
[00:43] <smoser> or i dont understand.
[00:43] <nacc> did you read the commit message?
[00:43] <nacc> the cache by default (when not passed as an argument) is in .git
[00:43] <nacc> which will get deleted presuming --no-clean is not passed
[00:43] <smoser> but dl_cache is not set
[00:43] <smoser> workdir is
[00:43] <smoser> dl_cache will still be None
[00:44] <nacc>     if dl_cache is None:
[00:44] <nacc>         workdir = os.path.join(local_repo.git_dir, CACHE_PATH)
[00:44] <smoser> right
[00:44] <nacc> why does it matter if dl_cache is none or not?
[00:44] <smoser> we do not set dl_cache
[00:44] <smoser> if dl_cache:
[00:44] <smoser>   remove stuff
[00:44] <smoser> er..
[00:44] <smoser> wait
[00:45] <nacc> smoser: it's getting late here (and i've been starting at this all day), it's gotta be even later for you! :)
[00:45] <nacc> smoser: yeah, i think what was there before was confusing and a change in behavior -- i think we can do it, but let's do it a distinct change, if that's ok with you
[00:45] <smoser> oh. you're right. never mind.
[00:45] <nacc> smoser: this way, the cache is preserved if the directory is preserved
[00:45] <nacc> which i think is the most correct default
[00:46] <smoser> i suppose, yeah. thats fine.
[00:48] <smoser> nacc, elinks:
[00:48] <smoser>  11/02/2016 23:53:51 - DEBUG:stderr: gbp:error: The orig tarball contains .git metadata - giving up.
[00:48] <smoser> http://paste.ubuntu.com/23418680/
[00:49] <smoser> and docker.io: http://paste.ubuntu.com/23418681/
[00:49] <smoser> those are new failures from pristine-tar
[00:49] <smoser> i'll let you go, and i'm gonna go too
[00:50] <smoser> but basically, it seems to be going really well, this clean restart with the --dl-cache
[00:50] <smoser> 157 total. 16 working. 139 pass. 2 fail.
[01:01] <nacc> smoser: right, import-orig won't like .git-ified upstreams
[01:01] <nacc> which aren't supposed to keep their .git, but some do
[01:02] <nacc> smoser: let's resolve what we're going to do about the orig stuff, because i'm not going to spend time on fixing those if we're not going to merge that before 1.0
[01:02] <smoser> right.
[01:02] <nacc> i think either way the other bugs take priority and we'd need to add a flag like --no-pristine-tar or something
[01:02] <nacc> maybe even default to that, dunno
[01:02] <nacc> smoser: in any case, thanks for your patience and insight, i'm done for the day too -- talk to you tomorrow!
[01:03] <smoser> you must have moved some stuff arodn for usd-build
[01:03] <smoser> before you committed
[01:04] <smoser> it doesn't compile right now
[01:04] <smoser> but tomorrow it will
[01:04] <smoser> good night
[01:04] <smoser> and Go cubs Go!
[05:34] <slangasek> nacc: sounds like debian-devel
[06:37] <TheMuso> Ok, pulseaudio si sitting in zesty-proposed, because pulseaudio-equalizer requires packages from universe, yet it was NEWed into main. Can I do anything to help demote it to universe?
[08:21] <rharper> pitti: sudo mount-image-callback -d install_disk.img chroot _MOUNTPOINT_ /bin/bash
[08:21] <rharper> --system-mounts --system-resolvconf
[08:40] <rharper> http://paste.ubuntu.com/23419821/
[08:43] <pkern> Could someone approve the trusty bug nomination on https://bugs.launchpad.net/ubuntu/+source/bash/+bug/1422795? I suspect that's why it's not shown on http://reqorts.qa.ubuntu.com/reports/sponsoring/index.html
[08:44] <hammed> ANYONER WITH CENTOS OR UBUNTU ROOT
[08:44] <hammed> ANYONER WITH CENTOS OR UBUNTU ROOT
[08:44] <hammed> ANYONER WITH CENTOS OR UBUNTU ROOT
[08:50] <LocutusOfBorg> hammed, no spam
[08:57] <hammed> ya
[08:57] <hammed> i need ubuntu
[08:57] <hammed> root
[08:58] <sladen> hammed: type   sudo command-you-want-to-run-as-root
[08:59] <sladen> hammed: (just the same as on OS X an other modern systems)
[09:00] <hammeds> root
[09:00] <hammeds> where
[09:01] <sladen> hammed: what are you trying to do, that "needs root"?  Perhaps we can help you better, if we understand
[09:44] <Laney> mardy: any chance you can build that package as an sru for yakkety too?
[09:45] <mardy> Laney: I think it has to land on zesty first
[09:45] <Laney> mardy: can be at the same time
[09:46] <Laney> but right now I want one to test
[09:47] <Laney> maybe the zesty binaries are installable
[09:47] <mardy> Laney: ok, but I think you can use the xenial version then, I would expect it to work fine on yakkety
[09:48] <pkern> Laney: As you had been quite helpful on the last trusty SRU I needed (thanks for that!)... Do you happen to know the best way currently to get an SRU to a package in main uploaded?
[09:50] <Laney> hey pkern
[09:50] <Laney> is it in the sponsor queue?
[09:55] <doko> tjaalton: please could you complete the MIR  issue #1547490
[09:57] <hammeds> i need just only ip user plus pass not
[09:57] <pkern> Laney: I think https://bugs.launchpad.net/ubuntu/+source/bash/+bug/1422795 isn't because it needs a coredev to approve the nomination or something.
[09:58] <pkern> Laney: ubuntu-sponsors is subscribed since somewhen in 2015.
[09:58] <Laney> pkern: ah, could be - let me do that
[09:58] <Laney> Maybe it would be a good idea to fix the queue to show bugs with pending nominations
[10:01] <pkern> Laney: Thanks! I was somewhat surprised that it didn't. Or that nominations to packages in main can't be done with usual bugcontrol privs. (Which would be the other recourse here.)
[10:01] <Laney> I think you have to be a "driver" for that - bug control just lets you nominate in the first place
[10:01] <Laney> IIRC
[10:02] <pkern> Ah, right. I think MOTUs are drivers for universe, though. (Which makes sense, so well, meh, maybe.)
[10:04] <Laney> driver *or* uploader
[10:04] <Laney> AIUI
[10:12] <tjaalton> doko: what's missing from -libinput?
[10:12] <doko> tjaalton: the MIR
[10:15] <tjaalton> hmm ok
[13:00] <andrewsh> hmm
[13:00] <andrewsh> latest update broke some of the indicators for me
[13:01] <andrewsh> https://www.dropbox.com/s/lx69qjcc25qnmnb/2016-11-03_14-00-30.png?dl=0
[13:03] <dobey> andrewsh: i think you want #ubuntu instead. it's the support channel
[13:04] <andrewsh> well, I don't think so
[13:04] <andrewsh> I'm pretty sure I don't need support, but I need to report a bug instead
[13:17] <Laney> mardy: I get the notification again instead of the webview with the package from the silo
[13:17] <Laney> thanks!
[13:17] <mardy> Laney: \o/
[13:21] <seb128> andrewsh, if you report a bug include some useful details like the version of ubuntu you are using and what you updated and what indicators are buggy and in which way
[13:22] <dobey> andrewsh: ubuntu-bug <indicator-package-name> for each indicator
[13:22] <seb128> no
[13:22] <dobey> that is buggy
[13:22] <seb128> well, if it's an issue in the lib no point opening a bug on each indicator using it
[13:23] <seb128> would be nice to describe what is buggy to start
[13:24] <dobey> well the "broken" indicators look like they are application indicators anyway, not standard system indicators
[13:24] <seb128> how do you know?
[13:24] <seb128> he didn't even say which ones
[13:25] <dobey> 09:00 < andrewsh> https://www.dropbox.com/s/lx69qjcc25qnmnb/2016-11-03_14-00-30.png?dl=0
[13:25] <dobey> because of their position on the panel
[13:25] <seb128> oh, weird url is a screenshot
[13:25] <seb128> could be bug #1635625
[13:25] <seb128> which is new in 16.10
[13:25] <seb128> but not due to a recent update
[13:25] <seb128> so that doesn't really match the IRC description
[13:26] <dobey> yeah, and nothing has really changed in the library or indicator-application recently either
[13:32] <andrewsh> seb128: the indicators are from skype and owncloud
[14:28] <rbasak> nacc, smoser: let me know when you're both available to talk about the importer
[14:29] <smoser> rbasak, ok.
[14:30] <smoser> i'm fine whenever. just when the west coast wakes up.
[14:30] <rbasak> OK
[14:30]  * rbasak has a hard stop today
[14:36] <rbasak> cpaelzer: around? Did you do anything to your branch after the dovecot merge? It would do with tagging and pushing, and then I can run the importer to bring it up to date with your work. Maybe this isn't documented?
[14:39] <cpaelzer> rbasak: I'm here
[14:40] <cpaelzer> rbasak: I haven't touched it since the upload with the modifications to the MP that you reviewed
[14:40] <cpaelzer> rbasak: but I might miss the point of your question
[14:40] <cpaelzer> rbasak: since I'm curious - short HO to sync on that?
[14:40] <rbasak> cpaelzer: please could you tag the thing that you uploaded? Should be upload/<version> (with substitutions if needed)
[14:41] <rbasak> cpaelzer: sure!
[14:41] <cpaelzer> rbasak: yeah in the past the uploader did that, since that now is me yeah I should add the tag
[14:42] <cpaelzer> rbasak: I'm in the team HO that we have later
[14:42] <rbasak> Oh
[14:56] <jbicha> cyphermox: I noticed you rebuilt devscripts at the start of the yakkety cycle; could you do the same for zesty now so that it picks up zesty as the current dev release?
[14:56] <cyphermox> sure
[14:56] <cyphermox> but I thought this already worked
[14:57] <cyphermox> or not
[14:59] <cyphermox> jbicha: I might as well do the merge at the same time, so it will take a little while
[14:59] <jbicha> cyphermox: that's fine, thanks
[15:05] <nacc> rbasak: i'm around now
[15:05] <nacc> smoser: --^
[15:06] <rbasak> nacc, smoser: usual team hangout?
[15:06] <smoser> k. yeah.
[15:06] <nacc> rbasak: smoser: sure
[15:12] <juliank> Whoa, I spent an hour on cleaning up a bugs now, I got 19.
[15:13] <juliank> There are still 446 in LP, if anyone else wants to play.
[15:13] <juliank> :)
[15:18] <Laney> mardy: oh no!
[15:18] <Laney> got a webview out of the blue
[15:20] <Laney> lots of signon stuff in syslog /me attaches
[15:20] <mardy> Laney: and you still have the signon-ui from that silo?
[15:20] <Laney> yeah
[15:20] <Laney> didn't restart
[15:21] <Laney> I notice that the title is "Web authentication" instead of "Web authentication for Google"
[15:21] <mardy> mmm
[15:21] <mardy> Laney: and what are the contents? Still google?
[15:22] <Laney> yep
[15:22] <Laney> sec, just attaching
[15:23] <cyphermox> jbicha: done.
[15:23] <Laney> ahh, I forgot to remove URLs and stuff
[15:23] <Laney> mardy: it's there, no massive rush on investigating though
[15:24] <mardy> Laney: thanks, I'll have a quick look now just to see if I need to ask you for some more
[15:31] <doko> cyphermox: missing -v<version> for devscripts merge
[15:32] <mardy> Laney: could it be that you removed some lines coming from com.nokia.singlesignonui[11563] ?
[15:32] <Laney> mardy: not removed, but I maybe missed them when copying (just selected a range)
[15:32] <Laney> lemme see
[15:35] <Laney> mardy: There's stuff from earlier on when I logged in
[15:37] <mardy> Laney: no, that's not important
[15:37] <mardy> Laney: I guess I need to add some debugging statements here and there
[15:40] <mardy> Laney: eds is really spamming signond with 28 requests in a second (and apparently, for a single account), maybe it's triggering some race condition
[15:41] <mardy> Laney: I see that the first request is going well, reusing the previously stored token without invoking signon-ui
[15:48] <cyphermox> doko: oh, you're right
[17:41] <nacc> smoser: i don't know what happened to your fetchmail import, it worked fine here -- let me finish testing the fixes for LP: #1638614 andi'll push to master
[17:41] <smoser> nacc, but it didnt stack trace
[17:42] <nacc> smoser: yeah, i have no idea -- did the system kill it?
[17:43] <nacc> smoser: it also doesn't show any errors in that paste
[17:44] <smoser> dont think so
[17:44] <smoser> well, it exted non-zero was all
[17:44] <smoser> but yeah, it was odd to me too
[17:46] <nacc> smoser: right, but we always exit with 1 on failure
[17:46] <nacc> iirc
[17:46] <nacc> so if it was a different exit code
[17:46] <nacc> then maybe it was sigkill'd?
[17:46] <nacc> i just had one die to too many open files
[17:47] <nacc> which i'm not sure how htat happened yet
[17:47] <smoser> i think system is kind of just really not happy
[17:48] <smoser> so possible that something happened
[17:48] <nacc> yeah, i'm not sure how to debug it at this point
[17:49] <nacc> smoser: ok, so git doesn't like '' as an author :)
[17:49] <nacc> smoser: suggestions of how to resolve this?
[17:50] <nacc> i could i think walk the lp package_maintainer_link maybe
[17:51] <smoser> nacc, yeah, not worry about the fail if not reproducible http://paste.ubuntu.com/23421781/ is my dmesg. that thing does look scary
[17:52] <smoser> wrt git not liking ''...
[17:52] <smoser>  if author == "":
[17:52] <smoser>     commit_author = "NotAvailable"
[17:52] <smoser> or somethign to that affect ?
[17:53] <nacc> right, but then we are putting that in the commit hash :)
[17:53] <nacc> and that's not obtainable without the importer
[17:53] <nacc> i'm trying to think if there's anything better we can do
[17:55] <smoser> meh.
[17:55] <smoser> lots of things are not obtainable without the importer.
[17:56] <nacc> i guess that's true
[17:56] <nacc> so i do think launchpad figures this out
[18:02] <nacc> testing again
[19:24] <nacc> smoser: ok, was able to fallback to launchpad data for the name, at least; if the email fails to parse its a hard fail as you need to be logged into lp for that data
[19:24] <nacc> but fixed the code to fail cleaner in that case, at least
[19:37] <nacc> smoser: so i think there migth be some underlying leak with the commit_msg_file thing
[19:37] <nacc> smoser: i just checked with lsof and even though the files are listed as deleted, they are showing as still open
[19:37] <nacc> any idea why that might be?
[19:50] <cjwatson> nacc: (also some people have their email addresses hidden in LP, so you need a fallback anyway)
[19:50] <nacc> cjwatson: ack
[19:51] <nacc> smoser: we probably need to handle that properly at some point (email failing to parse fully)
[19:51] <nacc> smoser: i wonder if we could just insert <no-email-found@launchpad.do-not-mail.net> or something
[19:52] <smoser> hm.. nacc reading for commit_msg_fiel comment
[19:53] <nacc> smoser: thanks, i don't see it yet, unless it's a side-effect of the with and unlink
[19:53] <nacc> smoser: is it possible the run is holding on to a reference somehow?
[19:53] <smoser> nah.
[19:53] <smoser> so...
[19:54] <smoser> tempfile.mkstemp() returns a 'os' filehandle and the file name
[19:54] <smoser> we open the file name on our own (with open ...) and close that
[19:54] <smoser> but i suspect possibly the os file handle needs to be closed
[19:55] <nacc> ah, so it's the first filehandle still being open
[19:55] <nacc> that would make sense
[19:55] <smoser> os.close(_)
[19:55] <smoser> essentilally
[19:55] <cjwatson> perhaps you should use os.fdopen instead
[19:55] <cjwatson> (closing a file object created by fdopen closes the underlying file descriptor too, which is probably what you want here)
[19:56] <cjwatson> so you wouldn't need a separate os.close if you did that
[19:56] <cjwatson> (I have not read the code here, I just know fdopen)
[19:57] <smoser> possibly, yeah.
[19:57] <cjwatson> or else just use a higher-level thing like tempfile.TemporaryFile or tempfile.NamedTemporaryFile rather than messing around with mkstemp
[19:58] <cjwatson> which is implemented with mkstemp + fdopen anyway, but why bother doing it yourself when you don't have to
[19:58] <nacc> smoser: is it not possible to do fp, _ = tempfile.mkstemp()
[19:58] <nacc> fp.write(msg)
[19:58] <nacc> fp.close()?
[19:58] <nacc> or as cjwatson said
[19:58] <cjwatson> nacc: that's:
[19:58] <smoser> you need os.write()
[19:58] <cjwatson> with tempfile.TemporaryFile() as fp:
[19:58] <smoser> to that thing. as its hot a file.
[19:58] <cjwatson>     fp.write(msg)
[19:59] <cjwatson> (though obviously you would need to do something more as a temporary file that immediately vanishes isn't very useful)
[19:59] <nacc> ah right
[19:59] <smoser> we want a name to it also . which is why i used mkstemp
[19:59] <cjwatson> smoser: then use NamedTemporaryFile.
[19:59] <cjwatson> you're fighting the stdlib unnecessarily here.
[19:59] <nacc> smoser: right, so there is an issue here, can you spin up a fix? :)
[20:00] <nacc> smoser: it seems fatal only for libvirt so far :)
[20:00] <smoser> sure. i'm not sure why i didnt use it. but sure.
[20:00] <nacc> smoser: thanks!
[20:01] <cjwatson> you may need delete=False and a later unlink, or else do fp.flush() and put the stuff that uses the tempfile inside the with statement
[20:02] <nacc> anyone have a suggestion on how to commit the following: https://launchpad.net/ubuntu/+source/iscsitarget/0.4.15+svn148-2.1ubuntu1
[20:02] <nacc> note the date is wrong
[20:02] <nacc> as in, invalid -- so should i fallback to the publishing information?
[20:13] <smoser> nacc, i suspect http://paste.ubuntu.com/23422293/ fixes
[20:13] <nacc> smoser: looks reasonable
[20:13] <smoser> actually.
[20:15] <smoser> http://paste.ubuntu.com/23422302/
[20:16] <nacc> makes sense
[20:17] <smoser> you want a MP?
[20:17] <nacc> smoser: i can commit it here if that's ok by you
[20:17] <cjwatson> I think I'd just put the run() and whatever's below it inside the with NamedTemporaryFile block in that case and not have to bother with delete=False
[20:17] <cjwatson> assuming it's small
[20:17] <cjwatson> yeah, that seems much cleaner
[20:17] <smoser> thats fine.
[20:17] <nacc> smoser: ack, commiting now
[20:18] <smoser> cjwatson, i probably need the flush, right ?
[20:18] <cjwatson> yeah
[20:25] <smoser> nacc, that "only" leaks one file handle per git commit though.
[20:26] <smoser> so its not great, but this glibc import i have running for 18 hours now only has 417 open files.
[20:27] <nacc> smoser: committed & pushed
[20:27] <nacc> cjwatson: thanks!
[20:27] <nacc> yeah, it's not a huge deal, just an oddity when i was hopeing to use `lsof -p <importer pid> | grep /tmp` to quickly see where it was running without having passed --verbose
[20:30] <nacc> smoser: thoughts on what to use as a fallback for the author date if it is unparseable? i am adding a strptime check on the extracted value from the changelog, but what should i use if it's bad? published date if possible? and if neither is present ...
[20:31] <smoser> if neither is present i suggest my birthday
[20:31] <smoser> i dont know. publish date shoudl be present always, shoudlnt' it ?
[20:31] <nacc> it's not always in my expereicne
[20:32] <nacc> i can't recall if that's because of superseded or deleted values
[20:32] <nacc> let me see if the underlying pacakge upload date always is
[20:52] <nacc> smoser: do you think it would be trivial to disconnect the run logging from the verbosity? that is, it'd be really nice to run with --verbose and get just the tracing we have from the importer code, not the shell commands being run. To get those, maybe -vv or something?
[20:53] <nacc> smoser: when debugging one change that will fix an import, i really can't read the output anymore :)
[20:53] <nacc> smoser: and i get all the cache hits too
[20:54] <smoser> its not terribly hard. really we just want to set up different loging i think (at lesat thats one way to do it).
[20:54] <smoser> -v = info
[20:54] <smoser> -vv = debug
[20:54] <smoser> -vvv = loud
[20:54] <nacc> right
[20:55] <nacc> but run and co use logging.debug
[20:55] <nacc> as does the importer
[20:55] <nacc> so just changing the loglevel globally won't work
[20:55] <nacc> i guess we could switch those around
[20:56] <nacc> yeah i think that's what we should do
[20:56] <smoser> well, we have to add a 'loud'
[20:56] <smoser> which is just
[20:56] <smoser>  logging.log(logging.DEBUG - 10, msg)
[20:56] <nacc> or logging.addLevelname
[20:56] <smoser> oh i didnt know this.
[20:57] <smoser> probably its and
[20:57] <smoser> not or
[21:00] <nacc> reading about it now
[21:00] <nacc> i think that would be more maintainable
[21:02] <nacc> err, right, it'd be something like logging.addLevelName(logging.DEBUG - 5)
[21:02] <nacc> as DEBUG is 10 and 0 is NOTSET
[21:02] <nacc> i'll play with that
[21:02] <smoser> sure. ijust picked a number
[21:03] <smoser> i've been annoyned before of the "top heavy" ness of python logging.
[21:03] <nacc> yep
[21:03] <smoser> info , warn, error, critical
[21:03] <smoser> it doesn't lend itself to usage like we're wanting.
[21:04] <smoser> debug, debug2, debug3
[21:04] <smoser> so cloud-init just gives you a firehose at debug
[21:04] <smoser> there really seems like there shoudl be a trace
[21:05] <nacc> yep
[23:02] <nacc> smoser: fudge, found a reason *not* to use a cache
[23:02] <nacc> smoser: https://launchpad.net/debian/+source/pcre3/1:8.35-3.2
[23:02] <nacc> https://launchpad.net/debian/+source/pcre3/2:8.35-3.2
[23:02] <nacc> note that the .dsc files are named the same
[23:02] <nacc> but are different
[23:03] <nacc> i believe pull-debian-source doesn't detect that
[23:05] <nacc> urp, they *should* be different
[23:05] <nacc> but they aren't