| === georgelorch2 is now known as georgelorch | ||
| mvo | is there something in https://launchpad.net/ubuntu/yakkety/+queue?queue_state=0&queue_text=golang-gopkg-macaroon.v1 that I could do to make NEW processing easier for you? | 13:53 |
|---|---|---|
| mvo | infinity: who should I ask about https://launchpad.net/ubuntu/yakkety/+queue?queue_state=0&queue_text=golang-gopkg-macaroon.v1 ? would be nice to get a NEW review of this, its going to be a new dependency of snapd | 15:19 |
| infinity | mvo: I was just now clearing the NEW queue, I'll have a look at it this morning. | 15:22 |
| seb128 | mvo, I can trade you reviews :p | 15:22 |
| infinity | Or seb can do it! | 15:23 |
| mvo | infinity: thank you! | 15:24 |
| mvo | seb128: hm, tell me more ;) | 15:24 |
| seb128 | mvo, want to have a look to https://code.launchpad.net/~seb128/update-notifier/logs-no-ctime/+merge/295806 in exchange? ;-) | 15:24 |
| seb128 | I'm waiting for your feedback for a while | 15:24 |
| seb128 | that's the one that mades update-manager never being auto-opened on xenial | 15:24 |
| mvo | seb128: hm, indeed, I remember now | 15:27 |
| seb128 | :-) | 15:27 |
| mvo | seb128: and your patch works? | 15:28 |
| mvo | seb128: i.e. with that it opens? | 15:28 |
| seb128 | mvo, from my local testing yes | 15:28 |
| infinity | If it's in yakkety, I'd say it's working, cause I get u-m annoying me in Y. | 15:29 |
| seb128 | rotating the log updates the mtime and not the ctime | 15:29 |
| seb128 | no, it's not | 15:29 |
| infinity | Oh. Then I'm not seeing the bug in the first place. :) | 15:29 |
| seb128 | weird | 15:29 |
| mvo | seb128: this is the part that I find confusing, I can't quite explain how it fixes it :/ | 15:30 |
| mvo | seb128: maybe we need to make the code smarter to actually look at the timestamps | 15:30 |
| mvo | seb128: inside the file | 15:30 |
| seb128 | mvo, what do you mean how it fixes it? | 15:30 |
| seb128 | you say that the log rotation shouldn't update the ctime? | 15:31 |
| mvo | seb128: I'm just surprised that ctime is updated but mtime remains, but maybe I'm just missing something, would you mind adding the stat output of a file in question | 15:31 |
| mvo | seb128: actually I think I get it now | 15:31 |
| mvo | seb128: sorry, long day and all that | 15:31 |
| seb128 | I don't have one atm, I installed things today | 15:31 |
| seb128 | no worry | 15:31 |
| seb128 | I might be overlooking something | 15:32 |
| seb128 | but I had a machine where I didn't touch packages for > 1 week | 15:32 |
| seb128 | and the log had been rotated | 15:32 |
| seb128 | the .1.gz ctime was the rotation date | 15:32 |
| seb128 | the mtime was the last dpkg use | 15:32 |
| seb128 | the .log was empty | 15:32 |
| seb128 | which the code already had a case for/ignores | 15:32 |
| mvo | seb128: yeah, logrotate will keep the mtime when it rotates but ctime will be updated, so I think it all makes sense | 15:32 |
| seb128 | :-) | 15:33 |
| mvo | seb128: sorry, a bit dense, I will followup in the LP | 15:33 |
| seb128 | mvo, thanks | 15:33 |
| mvo | seb128: thank you | 15:34 |
| seb128 | mvo, golang-gopkg-macaroon.v1 looks good, NEWed | 15:34 |
| mvo | seb128: ha! | 15:35 |
| * mvo hugs seb128 | 15:35 | |
| * seb128 hugs mvo back ;-) | 15:35 | |
| ogra_ | get a room ! | 15:38 |
| ogra_ | :) | 15:38 |
| ChrisTownsend | Hello, I have an autopackage test that claims its running, but it is not: https://requests.ci-train.ubuntu.com/static/britney/xenial/landing-051/excuses.html | 20:55 |
| ChrisTownsend | Could somebody please kick that so I can get this completed? | 20:56 |
| ChrisTownsend | infinity: slangasek: Are either of you around to help me out?^^^^ | 21:01 |
| tvoss | slangasek, ping | 21:08 |
| slangasek | tvoss: hi - are you pinging about https://requests.ci-train.ubuntu.com/static/britney/xenial/landing-051/excuses.html or about something else? | 21:15 |
| slangasek | ChrisTownsend: ^^ looking now ;) | 21:15 |
| tvoss | slangasek, something else, see pm | 21:15 |
| ChrisTownsend | slangasek: Thanks | 21:16 |
| slangasek | ok | 21:16 |
| slangasek | ChrisTownsend: http://autopkgtest.ubuntu.com/running.shtml | 21:20 |
| slangasek | pitti: btw, should silo-triggered autopkgtest results be indexed under e.g. http://autopkgtest.ubuntu.com/packages/c/content-hub/xenial/amd64/ for https://requests.ci-train.ubuntu.com/static/britney/xenial/landing-051/excuses.html ? | 21:22 |
| ChrisTownsend | slangasek: Thanks again! | 21:26 |
| ChrisTownsend | slangasek: Ugh, looks like the content-hub job is hung, maybe waiting on a reboot???? | 21:40 |
| ChrisTownsend | slangasek: Not sure if you did anything, but it looks like it's running again. | 21:42 |
| slangasek | ChrisTownsend: I only poked it the once :) did it complete successfully now? | 23:43 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!