| Chipaca | zyga: snap refresh to a bogus channel succeeds when the store is throttling connections, because it returns bogus results that snapd thinks are "ok go for it" | 00:54 |
|---|---|---|
| Chipaca | zyga: https://bugs.launchpad.net/snapd/+bug/1804245 | 00:56 |
| mup | Bug #1804245: empty response from store when throttled allows switch to nonexistent track <snapd:New> <https://launchpad.net/bugs/1804245> | 00:56 |
| RickRNF | Where are the logs for the snap version of snapcraft? I get a "Sorry, an error occurred in Snapcraft.Sending an error report" when running snapcraft cleanbuild | 07:41 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | mornings | 08:04 |
| pedronis | pstolowski: hi, can you try to merge master into #6315 | 08:31 |
| mup | PR #6315: overlord/ifacestate: include interface name in the hotplug-disconnect task summary <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6315> | 08:32 |
| pstolowski | pedronis: will do | 08:33 |
| pstolowski | pushed | 08:40 |
| mup | PR snapd#6265 closed: cmd/snap: attempt to restore SELinux context of snap user directories <SELinux> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6265> | 09:43 |
| mup | PR snapd#6321 closed: spread: show free space in debug output <Simple 😃> <Created by bboozzoo> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6321> | 09:44 |
| mup | PR snapd#6302 closed: wrappers: address review feedback from #6301 <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/6302> | 09:46 |
| pedronis | pstolowski: could I get a review of #6306 , it has a bit of non-nice hack for tests atm but that should go away with go1.9 (go vet in 1.6 is too picky) | 09:55 |
| mup | PR #6306: release: use locking around lazy intialized state <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6306> | 09:55 |
| pstolowski | sure | 09:55 |
| pedronis | pstolowski: #6315 is green | 10:52 |
| mup | PR #6315: overlord/ifacestate: include interface name in the hotplug-disconnect task summary <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6315> | 10:52 |
| pstolowski | ty | 10:53 |
| mup | PR snapd#6315 closed: overlord/ifacestate: include interface name in the hotplug-disconnect task summary <Simple 😃> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6315> | 10:53 |
| oSoMoN | is it a known issue that core18 ships a dbus machine-id file? | 11:18 |
| oSoMoN | $ cat /snap/core18/current/var/lib/dbus/machine-id | 11:18 |
| oSoMoN | a7579438e8b04d97a185d6aeefb83323 | 11:18 |
| oSoMoN | that breaks a number of things for core18-based snaps | 11:19 |
| oSoMoN | including ibus support | 11:19 |
| sparkiegeek | $ cat /snap/core18/current/var/lib/dbus/machine-id | 11:20 |
| sparkiegeek | a7579438e8b04d97a185d6aeefb83323 | 11:20 |
| sparkiegeek | SNAP! | 11:20 |
| pedronis | sil2100: ^ | 11:20 |
| pedronis | that's kind of unexpected | 11:20 |
| sil2100 | Looking | 11:42 |
| sil2100 | eh | 11:45 |
| sil2100 | Indeed, we missed that one | 11:45 |
| sil2100 | The removal of the machine-id was done in livecd-rootfs ;/ | 11:45 |
| oSoMoN | sil2100, would it be useful if I filed a bug to track the issue? (if so, where?) | 11:53 |
| sil2100 | oSoMoN: https://bugs.launchpad.net/snap-core18 | 11:55 |
| sil2100 | I'll be fixing it now | 11:55 |
| oSoMoN | thanks | 11:55 |
| oSoMoN | sil2100, https://bugs.launchpad.net/snap-core18/+bug/1809107 | 11:57 |
| mup | Bug #1809107: core18 contains var/lib/dbus/machine-id <snap-core18:New> <https://launchpad.net/bugs/1809107> | 11:58 |
| sil2100 | pedronis: I don't know much about the implications of that, is this a blocker for core18 release you think? | 12:02 |
| sil2100 | e.g. should I fast-track this through QA to stable and re-spin core18 images today ASAP? | 12:03 |
| sil2100 | Testing the fix | 12:04 |
| pedronis | sil2100: not sure, need to think a bit | 12:04 |
| mup | PR core18#107 opened: As per what we did in core16, remove dbus's machine-id <Created by sil2100> <https://github.com/snapcore/core18/pull/107> | 12:09 |
| pedronis | oSoMoN: sil2100: I'm a bit confused because afaict snaps don't have access to that file anyway by default | 12:09 |
| sil2100 | pedronis, oSoMoN: if anything, the PR above seems to do the trick | 12:12 |
| sil2100 | pedronis, oSoMoN: I'll merge the PR so that the new core18 can go to validation straight away | 12:17 |
| mup | PR core18#107 closed: As per what we did in core16, remove dbus's machine-id <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/107> | 12:18 |
| sil2100 | cachio: hey! Once the new core18 snap appears at the store, could you run your tests on it? | 12:20 |
| pedronis | sil2100: what about /etc/machine-id ? | 12:24 |
| sil2100 | pedronis: I trunkate it | 12:29 |
| sil2100 | We did the same for core | 12:30 |
| === ricab is now known as ricab|lunch | ||
| sil2100 | pedronis: anyway, in case we decide that we want to re-spin for this, could you give Eric a sign? | 12:31 |
| * sil2100 AFK for lunch | 12:31 | |
| cwayne | Another core 18 snap coming? | 12:45 |
| pedronis | cwayne: yes | 12:49 |
| pedronis | sil2100: is this the only change that we would get since the last core18? | 12:50 |
| pedronis | (seems so) | 12:55 |
| sil2100 | pedronis: yeah, I suppose | 12:58 |
| sil2100 | cwayne: could your team pick it up? | 12:58 |
| cwayne | sil2100: the snap test will be automatically picked up yes | 12:59 |
| sil2100 | eh, we missed the auto-import window, I triggered it manually, should be there soon | 13:00 |
| sil2100 | oSoMoN: a new core18 build is in progress, could you test out the new version once it hits the store? | 13:01 |
| * sil2100 now really AFK for lunch + vet | 13:01 | |
| oSoMoN | sil2100, sure, I'll test it as soon as it's out, edge channel IÂ suppose? | 13:46 |
| oSoMoN | could it be that revision 520 is it already? | 13:47 |
| pedronis | oSoMoN: yes, 520 seems correct, I see no /var/lib/dbus/machine-id and an empty one in /etc | 14:25 |
| pedronis | pstolowski: did you foget to submit the commnet in #6306? I don't see anything new there on GH unless I'm confused | 14:26 |
| mup | PR #6306: release: use locking around lazy intialized state <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6306> | 14:26 |
| pstolowski | pedronis: damn, indeed. now | 14:27 |
| pedronis | pstolowski: thanks I see it now | 14:28 |
| oSoMoN | sil2100, pedronis: 520 does the job, the regression with ibus that I was observing with the chromium snap is gone | 15:00 |
| oSoMoN | thanks for fixing that so promptly! | 15:00 |
| oSoMoN | when can I expect 520 to make it to stable? | 15:00 |
| pedronis | oSoMoN: we are trying to QA it quickly | 15:19 |
| zyga | hey | 15:39 |
| zyga | how are things? | 15:39 |
| * zyga was freezing his ... ears off outside today | 15:39 | |
| roadmr | wear earmuffs :) | 15:42 |
| zyga | I was wearing everything I could, just have to admit not used to the cold anymore | 15:43 |
| Saviq | hey all, can you please tell me which is the right image on http://cdimage.ubuntu.com/ubuntu-core/ to use in Multipass for "core"? http://cdimage.ubuntu.com/ubuntu-core/16/current/ is from April last year, http://cdimage.ubuntu.com/ubuntu-core/16/stable/current/ is from August, are there no newer images? | 15:45 |
| roadmr | zyga: hehe well, sometimes it is just too cold. | 15:45 |
| zyga | I ended up walking most of the way | 15:46 |
| sil2100 | Saviq: http://cdimage.ubuntu.com/ubuntu-core/16/stable/current/ is the latest deal | 15:46 |
| zyga | the streets are clogged with traffic jams | 15:46 |
| zyga | sil2100: is core18 healthy? | 15:46 |
| sil2100 | Saviq: we basically only re-create new images with every point-release (not counting some really emergency cases) | 15:46 |
| sil2100 | zyga: core18 is in testing right now | 15:46 |
| sil2100 | zyga: I'll tell you once I have results! | 15:46 |
| sil2100 | ;) | 15:46 |
| zyga | sounds good, thank you | 15:46 |
| sil2100 | zyga: certification team gave +1 on the snap, now just waiting for cachio | 15:48 |
| sil2100 | pedronis: did you talk to Eric? | 15:48 |
| Saviq | sil2100: ack, it's kinda surprising since that means it would like to reboot pretty soon after first boot | 15:48 |
| sil2100 | Indeed | 15:50 |
| mup | PR snapd#6322 opened: overlord/hookstate: apply pending transaction changes onto temporary configuration for snapctl get <Complex> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6322> | 15:51 |
| cachio | sil2100, it is running | 15:55 |
| cachio | it is gonna take about 1~2 hrs to complete | 15:56 |
| cachio | sil2100, so far so good | 15:56 |
| * cachio lunch | 15:58 | |
| sil2100 | eeek | 16:02 |
| sil2100 | Not good | 16:02 |
| cachio | sil2100, perhaps before | 16:21 |
| mup | PR core18#108 opened: Remove build-time openssh-server host keys and add service to auto-generating them on boot <Created by sil2100> <https://github.com/snapcore/core18/pull/108> | 16:52 |
| om26er | Would it be "sane" to share the whole snap tree using content interface ? I have a snap crossbar (WAMP router) and I want my app (also a snap) to start that router. | 16:56 |
| ogra | om26er, i'd call it "unelegant" but there is surely no technical reason to not do that | 17:06 |
| sil2100 | zyga: hey! You around? | 17:08 |
| sil2100 | zyga: could you take a look at https://github.com/snapcore/core18/pull/108 ? | 17:08 |
| mup | PR core18#108: Remove build-time openssh-server host keys and add service to auto-generating them on boot <Created by sil2100> <https://github.com/snapcore/core18/pull/108> | 17:08 |
| === pstolowski is now known as pstolowski|afk | ||
| mup | PR core18#108 closed: Remove build-time openssh-server host keys and add service to auto-generating them on boot <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/108> | 17:27 |
| sil2100 | cwayne, plars: hey! New core18 snaps need validation! | 18:14 |
| sil2100 | cwayne, plars: could someone track that and make sure it's pushed forward? | 18:15 |
| cwayne | sil2100: you don't need to tell us, it's done automatically | 18:15 |
| cwayne | Images are different though :) | 18:15 |
| sil2100 | cwayne: thanks ;p Just poking in case of some infra failure | 18:22 |
| sil2100 | So that someone can poke it with a stick | 18:22 |
| pedronis | cachio: are you testing the new core18 (528 etc) ? | 18:32 |
| cachio | pedronis, yes | 18:33 |
| pedronis | thx | 18:33 |
| cachio | tests being executed | 18:33 |
| cachio | pedronis, I'll keep you updated | 18:33 |
| cachio | sil2100, +1 to go to candidate and stable | 20:32 |
| cachio | snapd tests passed | 20:32 |
| cwayne | cachio: do you guys do any BT testing, or just the bt snapd interfaces? | 20:34 |
| cachio | just interfaces | 20:34 |
| cachio | cwayne, | 20:34 |
| zyga | sil2100: hey, I'm around now | 20:34 |
| cachio | cwayne, I am going to get my son | 20:35 |
| cachio | I'll be back in 30 minutes | 20:35 |
| cachio | cwayne, do you need anything urgent? | 20:35 |
| cwayne | cachio: nope | 20:35 |
| cachio | otherwise I'll be back soon | 20:35 |
| cachio | cwayne, nice, thanks | 20:35 |
| zyga | sil2100: hey | 20:36 |
| zyga | sil2100: if I could I would -1 that pull request | 20:36 |
| zyga | what there ensures that keys are generated only once? | 20:36 |
| sil2100 | Too late | 20:36 |
| sil2100 | That's the mechanism from core16 | 20:36 |
| zyga | do we have *perl* on the image? | 20:36 |
| zyga | there are better ways for first boot systemd units | 20:37 |
| cachio | sil2100, hold on | 20:37 |
| zyga | I guess it is too late | 20:37 |
| cachio | I just saw an error | 20:37 |
| zyga | but this doesn't look great | 20:37 |
| sil2100 | I don't know the details but it works, and works only once | 20:37 |
| zyga | looks like a hack | 20:37 |
| zyga | I believe you | 20:37 |
| sil2100 | cachio: ...uh? | 20:37 |
| cachio | sil2100, https://paste.ubuntu.com/p/NbXvcW6Vtt/ | 20:38 |
| zyga | cachio: interesting, any logs that go with the unit? | 20:38 |
| cachio | sil2100, seems to be related with the last change, right? | 20:38 |
| sil2100 | zyga: yes | 20:39 |
| sil2100 | hmm | 20:39 |
| cachio | zyga, no | 20:39 |
| cachio | no logs | 20:39 |
| cachio | let me try again | 20:39 |
| sil2100 | I meant, cachio: yes | 20:39 |
| cachio | sil2100, https://paste.ubuntu.com/p/DbGGCsc2pt/ | 20:40 |
| cachio | zyga, ~ | 20:40 |
| sil2100 | cachio: what device is that? | 20:40 |
| sil2100 | Crap | 20:40 |
| cachio | it is a vm with ubuntu core i386 | 20:40 |
| cachio | sil2100, zyga https://paste.ubuntu.com/p/Dmj2DmtyVp/ | 20:41 |
| sil2100 | What the heck? | 20:41 |
| zyga | sil2100: sent my review on https://github.com/snapcore/core18/pull/108#pullrequestreview-186738700 | 20:42 |
| mup | PR core18#108: Remove build-time openssh-server host keys and add service to auto-generating them on boot <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/108> | 20:42 |
| zyga | sil2100: did you test this locally? | 20:42 |
| zyga | I'm surprised about the path | 20:42 |
| zyga | specifically /usr/lib/snapd/ssh-host-keygen | 20:42 |
| sil2100 | Why is /usr/lib/snapd/sshd-host-keygen gone? I did and I remembered it worked | 20:42 |
| sil2100 | But now it doesn't | 20:43 |
| zyga | what ships that file? | 20:43 |
| cachio | sil2100, could you reproduce that? | 20:43 |
| sil2100 | I added it in the PR | 20:43 |
| zyga | sil2100: ah I see it now | 20:44 |
| sil2100 | zyga: I took all of this from core-build and live-build hooks | 20:44 |
| sil2100 | Fuck | 20:44 |
| pedronis | core18 | 20:44 |
| sil2100 | zyga: so snapd overrides that directory right? | 20:44 |
| zyga | sil2100: I don't remember from the top of my head | 20:44 |
| zyga | I presume yes | 20:44 |
| pedronis | sil2100: in core18 yes | 20:45 |
| pedronis | likely | 20:45 |
| zyga | sil2100: perhaps that script should ship in /usr/bin | 20:45 |
| sil2100 | How did that work, I got working keys | 20:45 |
| pedronis | timing? | 20:45 |
| zyga | sil2100: we bind mount that whole thing | 20:45 |
| sil2100 | Let me move it | 20:45 |
| sil2100 | Probably timing | 20:45 |
| pedronis | sil2100: the overriding at first boot is complicated | 20:45 |
| pedronis | it invovles services as well | 20:45 |
| zyga | sil2100: can you consider my other comments before landing the next revision (at your discretion) | 20:45 |
| pedronis | in core18 | 20:45 |
| pedronis | core was easier snapd was inside it | 20:45 |
| zyga | hey pedronis :) | 20:45 |
| pedronis | so that dir was fixed | 20:45 |
| sil2100 | zyga: I would prefer to just do it now, we're REALLY late | 20:45 |
| sil2100 | This is like REALLY REALLY REALLY late | 20:46 |
| zyga | sil2100: as I said, at your discretion | 20:46 |
| cachio | sil2100, zyga pedronis it is a fresh install https://paste.ubuntu.com/p/zZYJ5h4ZD6/ | 20:46 |
| zyga | sil2100: I'm super worried about shipping ssh keys | 20:46 |
| zyga | that we didn't notice this | 20:46 |
| zyga | sil2100: can you at least go over the list of files in the core18 snap that is produced | 20:46 |
| sil2100 | Anyway, the script as-is was in core so it won't be worse than it was already at least | 20:46 |
| cachio | zyga, pedronis sil2100 I need to leave, I need to get my son now | 20:47 |
| cachio | I'll be back as soon as possible | 20:47 |
| cachio | please telegran me | 20:47 |
| pedronis | zyga: that script shipped like that in core16, we can improve it, but probably not right now | 20:48 |
| zyga | pedronis: sounds good | 20:49 |
| zyga | pedronis: but I think whoever is responsible for core18 should eyeball the list of files | 20:49 |
| zyga | pedronis: imagine we shipped this | 20:49 |
| zyga | with those keys | 20:49 |
| pedronis | zyga: that's what sil2100 did, that's how we got here, afaiu | 20:50 |
| zyga | that's good | 20:50 |
| sil2100 | Ok, first boot works, no failures here, trying second boot with the path change' | 20:54 |
| sil2100 | zyga: ok, I moved the script to /usr/bin/ and it seems to work | 20:55 |
| zyga | sil2100: sounds very good | 20:55 |
| pedronis | sil2100: it might need a different name there tough, snapd- ? | 20:56 |
| pedronis | when it was in lib/snapd it didn't need a prefix | 20:56 |
| mup | PR core18#109 opened: Move the ssh keygen script to /usr/bin <Created by sil2100> <https://github.com/snapcore/core18/pull/109> | 20:56 |
| sil2100 | pedronis: +1 on that | 20:57 |
| sil2100 | It's good to change that just in case, let me do that | 20:57 |
| sil2100 | Maybe to core-? | 20:57 |
| sil2100 | Since it's not really a snapd thing | 20:58 |
| sil2100 | Sure, it was in the snapd directory but well, not sure | 20:58 |
| sil2100 | zyga, pedronis: PR ready for a quick review | 20:59 |
| zyga | looking | 21:00 |
| sil2100 | Thank you guys | 21:00 |
| zyga | thank *you* :) | 21:00 |
| zyga | +1 | 21:01 |
| sil2100 | \o/ | 21:01 |
| pedronis | looks good | 21:01 |
| sil2100 | I'm doing a quick test build and test run to see if the new name didn't have a typo and then we can build the snap | 21:01 |
| sil2100 | Merging it in the meantime | 21:01 |
| mup | PR core18#109 closed: Move the ssh keygen script to /usr/bin <Created by sil2100> <Merged by sil2100> <https://github.com/snapcore/core18/pull/109> | 21:02 |
| sil2100 | Ok, looks good, kicking new builds | 21:05 |
| zyga | \o/ | 21:05 |
| sil2100 | I need to go AFK for a bit | 22:32 |
| sil2100 | I'll be back in ~30 minutes | 22:32 |
| cachio | ok, it will take to finish | 22:49 |
| sil2100 | Back | 23:16 |
| sil2100 | cachio: how's the testing? | 23:31 |
| cachio | sil2100, so far it is ok | 23:32 |
| cachio | last time failed 1 test which is failing because an env issue I have | 23:32 |
| cachio | the when I checked the error I saw this time it was something different | 23:33 |
| cachio | this time I fixed that issue on the environment but it is running slower | 23:33 |
| cachio | sil2100, but I need to make sure everything went well before the +1 :) | 23:34 |
| sil2100 | cachio: sure! | 23:41 |
| pedronis | cachio: do you have a sense of how much long it will take? | 23:54 |
| cachio | pedronis, it is 60% | 23:54 |
| cachio | I think first suite will finish in 30 minutes maximun | 23:55 |
| cachio | second in 35 | 23:55 |
| cachio | with those we will have a good idea | 23:55 |
| cachio | so far no errors | 23:56 |
| cachio | but last time the last test failed :( | 23:59 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!