[00:03] <tsimonq2> slangasek: Just cron.daily or cron.daily-live too?
[00:04] <slangasek> don't know yet
[00:04] <slangasek> :)
[00:04] <slangasek> https://launchpad.net/~ubuntu-cdimage/+livefs/ubuntu/artful/lubuntu/
[00:04] <tsimonq2> ack
[00:18] <tsimonq2> slangasek: So why does it show as successful then? http://people.canonical.com/~ubuntu-archive/cd-build-logs/lubuntu/artful/daily-20180105.log
[00:18] <tsimonq2> This is on Nusakan itself, not an LP builder, correct?
[00:24] <wxl> if i read slangasek's email correctly, we will be re-releasing 17.10 to fix the SPI bug, but the Meltdown/Spectre fixes will come in kernel updates, right?
[00:24] <tsimonq2> Correct.
[00:25] <slangasek> tsimonq2: it's successful because I manually hit enter at the prompt.  But it should not prompt interactively just because I ran the command with a controlling tty
[00:25] <tsimonq2> slangasek: ohhh, ic now...
[00:26] <slangasek> and yes, it also affects lubuntu live ;)
[00:26] <nacc> wxl: yeah
[00:27] <wxl> thx release team for your hard work on this! (and to the kernel team as well)
[00:27] <tsimonq2> ^^^^
[00:31] <valorie> amen to that
[00:31] <valorie> thank you thank you thank you
[00:31] <valorie> my emails asking for testing have gone out
[00:31] -queuebot:#ubuntu-release- Builds: Lubuntu Desktop amd64 [Artful Dot One] (20180105) has been added
[00:31] -queuebot:#ubuntu-release- Builds: Lubuntu Desktop i386 [Artful Dot One] (20180105) has been added
[00:51] <tsimonq2> slangasek: So I've figured out what needs to happen. Would something like this be acceptable? https://paste.ubuntu.com/26329048/
[00:51] <tsimonq2> slangasek: As long as that env var is set, it won't prompt for anything, but I'm unsure if setting an env var in that way will ensure it gets passed to git from germinate...
[00:55] <tsimonq2> slangasek: (er, I can test this locally, my point is, what way would you prefer I set the env var?)
[00:56] <slangasek> tsimonq2: rather, you should pass an env argument to proxy_check_call (which is a wrapper around subprocess.check_call).
[00:56] <slangasek> tsimonq2: cheatsheet: https://stackoverflow.com/questions/2231227/python-subprocess-popen-with-a-modified-environment
[00:57] <tsimonq2> slangasek: thanks
[01:01] -queuebot:#ubuntu-release- Builds: Lubuntu Next Desktop amd64 [Artful Dot One] (20180106) has been added
[01:02] -queuebot:#ubuntu-release- Builds: Lubuntu Next Desktop i386 [Artful Dot One] (20180106) has been added
[01:03] <tsimonq2> slangasek: incoming: https://code.launchpad.net/~tsimonq2/ubuntu-cdimage/make-git-noninteractive/+merge/335789
[01:03] <tsimonq2> slangasek: Also, Lubuntu Next shouldn't get 17.10.1 builds ;)
[01:06] <slangasek> tsimonq2: ack, I forgot that directory was empty; how about I delete the directory
[01:06] <tsimonq2> slangasek: sure np
[01:11] <slangasek> tsimonq2: did you run 'run-tests'?
[01:13] <tsimonq2> slangasek: er, good point... let me fix these...
[01:17] <tsimonq2> slangasek: (nothing's broken except the tests fwiw)
[01:36] -queuebot:#ubuntu-release- Builds: Ubuntu Kylin Desktop amd64 [Artful Dot One] (20180106) has been added
[01:36] -queuebot:#ubuntu-release- Builds: Ubuntu Kylin Desktop i386 [Artful Dot One] (20180106) has been added
[01:52] <smoser> hey. just want to make sure this is expected..
[01:53] <smoser> ah. never mind.
[01:53] <smoser> no builds yet. was wondering why a package let in yesterday wasnt in archive.
[01:53] <smoser> thanks for listening :)
[01:58] <slangasek> :)
[02:01] <tsimonq2> slangasek: I'm probably really overthinking this but I keep getting an error code of 256 and I'm having trouble figuring out how to fix it
[02:01] <tsimonq2> I'll keep trying things but any pointers would really be appreciated
[02:02] <slangasek> tsimonq2: after modifying the germinate_command() function to include the same env logic?
[02:03] <tsimonq2> slangasek: Yes.
[02:04] <tsimonq2> slangasek: I tried this:
[02:04] <tsimonq2> -                    ], cwd=os.path.join(germinate_output, arch))
[02:04] <tsimonq2> +                    ], cwd=os.path.join(germinate_output, arch),
[02:04] <tsimonq2> +                       env=dict(os.environ, GIT_TERMINAL_PROMPT="false"))
[02:04] <tsimonq2> Because that's the same logic, no?
[02:05] <slangasek> yes
[02:05] <tsimonq2> Specifically: AssertionError: Calls not found. -- that's the error
[02:06] <slangasek> when I apply that and run it here, I see the same failure; if I compare the calls, I see the 'actual' list includes an additional lockfile call?
[02:08] <tsimonq2> Hmm.
[02:08] <slangasek> don't know anything beyond that and am afk for now, sorry
[02:08] <tsimonq2> Alright, thanks anyways
[02:09] -queuebot:#ubuntu-release- Builds: Ubuntu Budgie Desktop amd64 [Artful Dot One] (20180106) has been added
[02:09] -queuebot:#ubuntu-release- Builds: Ubuntu Budgie Desktop i386 [Artful Dot One] (20180106) has been added
=== ypwong- is now known as ypwong
=== joedborg_ is now known as joedborg
=== mpontillo_ is now known as mpontillo
[02:30] <tsimonq2> slangasek: So it ended up being a *one* *line* *fix* - but what you said did help. But I did learn something about Python unit testing in the progress...
[05:04] <nacc_> tsimonq2: fyi, the documented value is 0
[05:05] <nacc_> tsimonq2: not "false" (see `man git`)
[05:05] <nacc_> tsimonq2: I mentioned that value (0) earlier
[05:06] <tsimonq2> nacc_: Oh, thanks
[05:06] <tsimonq2> Gah, where'd he go? :)
[05:07] <nacc> tsimonq2: np :) (and now i'm afk for real too :)
[05:07] <tsimonq2> o/
[09:46] <flocculant> slangasek: I don't appear to be able to get iso's from Artful Dot One ... both 32 and 64 bit give me a 404
[09:47] <flocculant> checked a couple of other flavours and ubuntu - all 404
[09:52] <flocculant> they 'can' be grabbed ok from cdimage.ubuntu.com/xubuntu/artful/daily-live/current/  at least - but I suspect everyone else points their testers to the tracker, I know I did ;)
[09:58] <acheronuk> flocculant: same for kubuntu. the qa tracker url points to /kubuntu/daily-live/20180105.2/ folder, whereas the actua images are in /kubuntu/artful/daily-live/20180105.2/
[09:58] <acheronuk> *actual
[09:58] <flocculant> acheronuk: yea - you were one of the other flavours I looked at :p
[10:01] <acheronuk> not done any announcing yet, so good catch
[10:04] <acheronuk> tsimonq2: lubuntu is also 404, so guess they all are (which is logical)
[10:04] <flocculant> acheronuk: that was the other flavour :D
[17:24] <wxl> flocculant: can't we just go on the tracker and change the URLs?
[17:31] <flocculant> wxl - no idea, never seen that anywhere
[17:32] <flocculant> wxl - just point people at the cd.image url for the time being
[17:32] <wxl> well it looks like it should just work http://iso.qa.ubuntu.com/admin/config/services/qatracker/products/497/downloads
[17:33] <flocculant> never seen that
[17:33] <wxl> with great power comes great curiousity XD
[17:36] <flocculant> :)
[17:46] <wxl> well anyways i can't figure it out if it is possible so oh well
[17:52] <flocculant> probably is - on the other hand it could go horribly wrong
[17:52] <flocculant> for you - cos I would wait ;)
[18:18] <wxl> hm are bionic dailies halted?
[18:19] <wxl> everyone's stuck at 20180103 it seems.. except for Lubuntu alternates which are ahead and the rest of the Lubuntu images which are behind but rebuilding
[18:47] <jbicha> wxl: I assume that's because LP isn't building anything now (see the /topic )
[18:48] <wxl> yeah i guess that's true, jbicha, except for the fact that it's clear that's not true XD
[18:49] <wxl> again, lubuntu has a fresh release
[18:49] <wxl> not to mention artful dot one
[18:51] <jbicha> bionic hasn't changed in a few days anyway
[18:53] <wxl> it has for lubuntu
[18:53] <jbicha> I'm guessing it's all related to Meltdown/Spectre; there's no need for bionic dailies right now, but we do need a 17.10 release since 17.04 is nearly EOL
[19:39] <slangasek> flocculant: sorry, fixes for the code on the tracker are welcome, but in the meantime people will just have to find the correct URLs manually
[19:41] <flocculant> slangasek: no problem - have a good weekend :)
[19:41] <wxl> ouch that sucks