=== chihchun is now known as chihchun_afk
=== epod is now known as luk3yx
=== chihchun_afk is now known as chihchun
=== chihchun is now known as chihchun_afk
=== chihchun_afk is now known as chihchun
mborzeckizyga: hey05:26
zygaHow are you?05:27
mborzeckizyga: are we goign to be equally unlucky today? i've restarted some travis jobs yesterday evening, and all are red :/05:27
zygaWhat is failing?05:28
mborzeckizyga: one failed with prepare image, another in go-build:s390x, the last one probably maanged to hit some mirror sync05:29
zygaAssorted bad luck05:30
mborzeckizyga: prepare-image was probably some issue fetching snaps from the store05:30
zygaIā€™m with the dog, outside05:30
mborzeckiand the 4th job got 502 when pulling gopkg.in/yaml.v2 :P05:31
zygaIs the store wonky today?05:31
mborzeckifeels like i should play some euro jackpot today05:31
zygaYesterday I learned that apple support should be split in two05:33
mborzeckiis #6667 waiting for something else?05:33
mupPR #6667: tests: enable tests that write /etc/{hostname,timezone} on core18 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6667>05:33
mborzeckizyga: sup and port?05:34
zyga90% of the people came with a shattered phone05:34
zygaThat took forever05:34
zygaLooking at 666705:34
mborzeckizyga: many people are clumsy then :)05:34
zygaMaybe ok to land?05:35
mborzecki#6659 can probably land too05:35
mupPR #6659: snapcraft: build static fontconfig in the snapd snap <Created by mvo5> <https://github.com/snapcore/snapd/pull/6659>05:35
zygaMvo will be around soon05:35
zyga+1 there05:35
mborzeckithere will be a followup but somewhere else in the code05:36
zygaback in the office05:45
mupPR snapd#6502 closed: dirs,overlord/snapstate: add Soft and Hard refresh checks <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6502>05:46
zygaeh, I feel even more sick than yesterday now05:47
zygamborzecki: can you please review https://github.com/snapcore/snapd/pull/664305:48
mupPR #6643: tests: deny ioctl - TIOCSTI with garbage in high bits <Created by zyga> <https://github.com/snapcore/snapd/pull/6643>05:48
zygait's something that should have landed weeks ago but was under embargo05:48
zygahello mvo05:48
mvozyga: good morning05:51
mborzeckimvo: hey05:51
mvozyga: how are you? how is the testing situation?05:51
mvohey mborzecki05:51
mvomborzecki: anything on the suspected memleak?05:52
zygamvo: I have a running nose and something in my lungs :/ I probably will skip japanese classes today05:52
mborzeckimvo: no, Chipaca asked some questions in the LP bug05:52
zygamvo: tests are grim, failing on assorted collection of random annoyances05:52
zygafrom network to more network to random stuff05:52
mvozyga: :(05:53
mvozyga: I saw that archive.u.c had some issues last evening05:53
mvomborzecki: cool, keen to learn what john figured out, I will check the bug05:53
mborzeckizyga: mvo: do you use auditd and ausearch on ubuntu?06:03
zygaauditd maybe06:03
zygabut not ausearch06:03
mborzeckiso ausearch does not properly report denials from apparmor on arch, thought that ubuntu may carry some patches that fix that, but apparently not, doesn't work on ubuntu either06:07
mborzeckiso only uses has extra patches to fix that06:07
mborzeckiMalformed event skipped, rc=9. type=AVC msg=audit(1554271576.256:61): apparmor="DENIED" operation="open" profile="snap.hello-world.sh" name="/home/guest/" pid=13765 comm="bash" requested_mask="r" denied_mask="r" fsuid=1000 ouid=100006:07
mborzeckithat's what I see on ubuntu (and arch for that matter too) when runnig ausearch -m AVC --debug06:07
mborzeckijust ausearch -m AVC shows <no matches>06:07
mborzeckii suggested name tweaks in #6656 but i'm not sure it's worth risking another travis run at this point, maybe we should just land it06:10
mupPR #6656: tests: split travis spread execution in 2 jobs for ubuntu and non ubuntu systems <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6656>06:10
mupPR snapd#6674 closed: tests: use apt via eatmydata <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6674>06:11
mborzeckihm https://bugs.launchpad.net/ubuntu/+source/audit/+bug/111780406:15
mupBug #1117804: ausearch doesn't show AppArmor denial messages <apparmor> <AppArmor:Confirmed> <audit (Ubuntu):Confirmed> <linux (Ubuntu):Incomplete> <https://launchpad.net/bugs/1117804>06:15
mborzeckizyga: yeah, suse carries a patch that was submitted upstream, but then the discussion went to apparmor not using some common audit ids and thus being wrongly interpreted in userspace or sth06:16
mborzeckizyga: oh, and suse patch just works aroudn that in the userspace parser :P06:17
zygakernel stuff is hard06:17
mupPR snapd#6672 closed: metautil, snap: extract yaml value normalization to a helper package <Simple šŸ˜ƒ> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6672>06:20
mborzeckiehh, opensuse this time: Some of the repositories have not been refreshed because of an error.06:21
mborzeckimvo: zyga: what about #6677, do we want to keep golang.org/x/sys/unix pinned?06:22
mupPR #6677: vendor: pin golang.org/x/sys/unix to a revision before SYS_CLOCK_GETTIME on OSX <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6677>06:22
mvomborzecki: now that things are fixed again upstream I would say we don't need to pin until the next issue like this? the risk of pinning is that stay behind forever and don't get e.g. security fixes06:23
zygamborzecki: I would unpin it06:34
mborzeckimvo: zyga: ack, closed the PR06:36
zygabecause this gives us error visibility at the cost of ... well ... errors06:36
mupPR snapd#6677 closed: vendor: pin golang.org/x/sys/unix to a revision before SYS_CLOCK_GETTIME on OSX <Created by bboozzoo> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/6677>06:36
mvothanks mborzecki06:39
* zyga prepares the next refresh app awareness batch06:53
zyganot a big one, but functional one06:53
pedronismvo: hi,  need to decided what to do with #6659, follow up needs  to be tracked, and #6667 is green06:56
mupPR #6659: snapcraft: build static fontconfig in the snapd snap <Created by mvo5> <https://github.com/snapcore/snapd/pull/6659>06:56
mupPR #6667: tests: enable tests that write /etc/{hostname,timezone} on core18 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6667>06:56
=== pstolowski|afk is now known as pstolowski
zygahey pedronis, hey pawel07:17
mborzeckipstolowski: pedronis: hello guys07:18
pstolowskipedronis: hey, #6665 has 2 +1s; would you like to take a look or can it land?07:26
mupPR #6665: overlord/ifacestate: implement String() method of HotplugDeviceInfo for better logs/messages <Hotplug šŸ”Œ> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6665>07:26
pedronispstolowski: I need to take a look07:27
pedronispstolowski: we don't use "<...>" style of representation anywhere so far07:29
pedronisso I need to think a bit actually07:29
pstolowskipedronis: ok; i think it'd make sense to somehow wrap this representation up as it's a bit long07:30
pedronispstolowski: yes, they are bit long; too long?07:31
pstolowskipedronis: maybe; maybe vendor/model should be omitted07:33
pedronispstolowski: well model seems important,  are vendor usually that long?07:34
mupPR core18#125 closed: hooks: create snapd directory skeleton <Created by zyga> <Merged by sil2100> <https://github.com/snapcore/core18/pull/125>07:38
pstolowskipedronis: not really; but models can be long, three examples from my VM:07:38
pstolowskiES1371/ES1373 / Creative Labs CT2518 (Audio PCI 64V/128/5200 / Creative CT4810/CT5803/CT5806 [Sound Blaster PCI])07:38
pstolowski82545EM Gigabit Ethernet Controller (Copper) (PRO/1000 MT Single Port Adapter)07:38
pstolowski53c1030 PCI-X Fusion-MPT Dual Ultra320 SCSI (LSI Logic Parallel SCSI Controller)07:38
pedronispstolowski: so model usually contains also the vendor name?07:39
pstolowskipedronis: it seems so, we could get rid of vendor07:40
zygapedronis: usually the rules are: there are no rules :)07:40
pedroniszyga: that's fine, we still need to do something reasonable and consistent with the rest of our code07:41
zygaI totally agree07:41
pedronisthat PR is not there yet from my POV07:42
zygaI think, while this may be odd, that a lookaside table (in the store perhaps) may be the best outcome07:42
zygafor real usability07:42
pedronisthat is over ambitious considering were we are now :)07:43
zygais it? we can just reply with what is in the usb database already07:44
zygaand we can fix ugly things one-by-one as encountered07:44
pedroniszyga: ?07:44
zygaI mean, it's not out of reach07:44
pedroniszyga: what are you proposing?  (I'm missing something here, given that you mentioned the store)07:45
pedroniszyga: I notice that you gave +1 to that PR without further comments07:45
zygapedronis: that using a simple solution now (no transformations) and planning to use the store for nice descriptions should be done07:45
pedronispstolowski: anyway the closest thing we have to this sort of display problem atm is this:  https://github.com/snapcore/snapd/blob/master/asserts/asserts.go#L196 (though assertions are much more structured than what we have here)07:47
pstolowskizyga: #6605 has conflicts07:53
mupPR #6605: cmd/libsnap,osutil: fix parsing of mountinfo <Created by zyga> <https://github.com/snapcore/snapd/pull/6605>07:53
pstolowskipedronis: yes, in fact i was looking at assertions for inspiration ;)07:54
mborzeckipstolowski: you could skip serial for sure07:54
mborzeckidevpath can be quite long usually since, but devname is preferred anyway07:55
zygapstolowski: I know, waiting for a review from Jamie though07:56
pedronishe said he should be back reviewing things, now that he draft the daemon user stuff07:57
mborzeckiheh fedora 30 beta released, fedora repos back to the usual not-responding-when-release mode07:58
mwhudsonwe just get that when we release kernels08:34
mupPR snapd#6682 opened: snap, gadget: move gadget read/validation into separate package, tweak naming <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6682>08:53
mborzeckipedronis: mvo: gadget package split ^^08:56
mvomborzecki: nice, I have a look08:56
mborzeckimvo: twekaed the naming too, so snap.GadgetSize is now gadget.Size and such08:56
mborzeckiaaand damn go 1.9 fmt08:57
mvomborzecki: nice09:00
mborzeckihope this lands soon09:03
mvomborzecki: yeah, it will be a massive source of conflicts, right?09:03
mborzeckimvo: yeah09:03
mborzeckimvo: merging or rebasing changes on top is super annoying09:04
pedronispstolowski: some comments in 6665, happy to get feedback back on what I propose09:05
=== chihchun is now known as chihchun_afk
mupPR core18#123 closed: hooks: remove /etc/apt/sources.list.d/proposed.list <Created by mvo5> <Merged by sil2100> <https://github.com/snapcore/core18/pull/123>09:18
pstolowskipedronis: thanks09:19
zygamvo: now that https://github.com/snapcore/core18/pull/125 is merged, how soon can we get core18 build?09:24
mupPR core18#125: hooks: create snapd directory skeleton <Created by zyga> <Merged by sil2100> <https://github.com/snapcore/core18/pull/125>09:24
pedronisthat reminds  me, is there a deep reason why we kept firstboot in snapd.dirs,  is not used anymore since long time09:26
mvozyga: let me see09:27
mvopedronis: that sound like an oversight (90% confident)09:27
mvozyga: I triggered a new core18 build, should be ready in ~30min or so (depending on the buildds)09:38
zygasuper, thanks!09:38
pedronismborzecki: assuming all the renames are relatively obvious I don't need to review 668209:39
=== ricab is now known as ricab|bbl
mupPR snapd#6436 closed: interfaces: add system-backup interface <ā›” Blocked> <Created by jdstrand> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/6436>09:44
pedronismvo: I did another pass on some of the remodel PRs, the main one needs 2nd reviews09:44
mborzeckipedronis: ack09:45
mborzeckihttps://github.com/snapcore/snapd/pull/6656 is green, are we moving forward with the plan to split the travis jobs to ubuntu and non-ubuntu?09:46
mupPR #6656: tests: split travis spread execution in 2 jobs for ubuntu and non ubuntu systems <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6656>09:46
pedronismborzecki: sounds best to re-discuss it quickly at today standup09:47
mborzeckipedronis: ok09:47
pedronismvo: any reasons not to merge #6667 ?09:47
mupPR #6667: tests: enable tests that write /etc/{hostname,timezone} on core18 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6667>09:47
=== chihchun_afk is now known as chihchun
pstolowskizyga: added a few comments to #664309:52
mupPR #6643: tests: deny ioctl - TIOCSTI with garbage in high bits <Created by zyga> <https://github.com/snapcore/snapd/pull/6643>09:52
mvopedronis: the SRU is not out yet officially, we just have the fixes in the ppa09:56
mupPR snapd#6654 closed: packaging/fedora, tests/upgrade/basic: patch existing mount units with SELinux context on upgrade <SELinux> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6654>10:06
pedronismvo: so  verifications will fail in some cases?10:07
pedronismvo: should it be marked blocked then?10:07
mvopedronis: I think so, let me do that and add a small explaination10:11
pedronispstolowski: I reivewed #666010:17
mupPR #6660: cmd/debug: integrate new task timings with "snap debug timings" <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6660>10:17
mvosil2100: re 121 for core18 - I'm on it, I think I know what is going on10:20
Chipacahmm, hmm10:24
Chipacamborzecki: this maybeShellcheck has broken a lot of tests here10:24
Chipacagrmbl grmbl10:24
pstolowskipedronis: ty10:25
mborzeckiChipaca: which tests?10:28
Chipacamborzecki: the unit tests10:29
mborzeckiChipaca: master seems fine10:30
Chipacamborzecki: this is on master10:30
Chipacamborzecki: pastebinning in a bit10:31
mborzeckiChipaca: hm all fine here10:31
Chipacamborzecki: is your shellcheck from the snap10:31
mborzeckiChipaca: no, let me try that10:31
Chipacamborzecki: http://paste.ubuntu.com/p/9R5g6QFTtd/10:32
mborzeckiChipaca: see the same here10:34
mborzeckihaha, it's /tmp10:34
Chipacayou done fucked up, son10:34
mborzeckishellcheck from snap cannot access that probably10:34
mborzeckibecause it gets a private tmp10:34
mborzeckihmm hmm, not sure there's a workaround for that, unless shellcheck becomes classic10:36
mborzeckior check uses a different tmp directory10:37
Chipacamborzecki: you've got the script10:38
Chipacamborzecki: why not just feed it to shellcheck10:38
mborzeckiChipaca: ah right, that's another option all right10:38
mborzeckiChipaca: fix coming up10:50
mupPR snapd#6683 opened: testutil: make mocked command work with shellcheck from snaps <Simple šŸ˜ƒ> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6683>10:56
mborzeckiChipaca: ^^ can you do the honors?10:56
Chipacamborzecki: for extra fun, go could be running from a snap as well :-)10:57
mborzeckiChipaca: all on ubuntu core :P10:57
Chipacawell, no, because go is classic10:57
mborzeckiah, right10:57
pstolowskipedronis: ok to drop serial in #6665 ?10:59
mupPR #6665: overlord/ifacestate: implement String() method of HotplugDeviceInfo for better logs/messages <Hotplug šŸ”Œ> <ā›” Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6665>10:59
mborzeckiChipaca: could have made it more fancy, like os.Open(), io.Copy(), but meh11:00
Chipacamborzecki: i thought the same on reading it11:00
pedronispstolowski: is it useful or not?11:00
pedronispstolowski: how long do they tend to be?11:00
pstolowskipedronis: only if you have more instances of same device i suppose, to somehow distinguish them11:01
pedronispstolowski: yea,  as I said we could truncate them11:01
pedronisbut I'm not sure how long they tend to be11:01
mborzeckiback to zyga's PR11:02
mvosergiusens: ideas about lp 1822988 would be great11:04
mvosil2100: -^11:04
pedronispstolowski: my current feeling it to leave it, not do truncating yet, and see how it works out in practice11:10
pedronispstolowski: we also need to deal with the fact that hotplug keys are long/unwieldy of their own11:11
pstolowskipedronis: ok, will only truncate model/vendor11:13
pedronispstolowski: but as you said, it seems we should indeed swap SHORT and non-SHORT for serial, I mean prefer SHORT if it exists11:13
pedronisfor this use11:13
pstolowskipedronis: yes11:13
=== ricab|bbl is now known as ricab
mborzeckipstolowski: can you take a look at #6682?11:28
mupPR #6682: snap, gadget: move gadget read/validation into separate package, tweak naming <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6682>11:28
pstolowskimborzecki: yes!11:29
mborzeckipstolowski: thanks!11:29
=== chihchun is now known as chihchun_afk
pstolowskipedronis: updated #666511:39
mupPR #6665: overlord/ifacestate: implement String() method of HotplugDeviceInfo for better logs/messages <Hotplug šŸ”Œ> <ā›” Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6665>11:39
mvo6577 needs a second review11:43
mupPR snapd#6683 closed: testutil: make mocked command work with shellcheck from snaps <Simple šŸ˜ƒ> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6683>11:44
* Chipaca ā‡ lunch11:44
zygaChipaca: I would love your review of 668411:45
mupPR snapd#6684 opened: overlord,tests: perform soft refresh check in doInstall <Created by zyga> <https://github.com/snapcore/snapd/pull/6684>11:45
Chipacazyga: ack11:45
zygait's deceptively short :)11:45
zygaand works in spread11:45
vidal72[m]is there a way to list snaps that use core18?11:51
Chipacazyga: +1, with a silly suggestion you can ignore11:52
Chipacavidal72[m]: installed ones?11:52
zygaChipaca: wow, that's quick :)11:53
jdstrandmborzecki: re ausearch, I can't remember otoh, but either jjohansen1, ChrisCoulson or tyhicks would have more info on ausearch and apparmor11:53
zygaso... I need a 2nd review :)11:53
* cachio afk11:54
jdstrandmborzecki: ah yes, I see now in backscroll you found the bug and extra context11:54
vidal72[m]Chipaca: no, those avalaible on store11:55
Chipacavidal72[m]: no11:55
jdstrandzyga: if that 2nd review was for me, please remember I was asked to focus on something ahead of all others, a queue formed and I will get through it. I promise11:55
zygajdstrand: no no :)11:55
jdstrandok :)11:56
zygajdstrand: there are lots of people who can review that11:56
zygaand it's not security oriented11:56
Chipacavidal72[m]: why do you care?11:56
jdstrandI thought this was a 6605 reminder :)11:56
mborzeckijdstrand: yeah, it seems the bug got deprioritized, which is ok, there's workarounds11:56
jdstrandsee, it is so much in the queue I typed that from memory :)11:56
jdstrandI'm currently a bit stretched thin. joe is trying to help with that11:58
jdstrandmborzecki: you could ask the priority in #apparmor on OFTC. ChrisCoulson was assigned that at one point, but, yes, it got deprioritzed for various reasons11:59
vidal72[m]Chipaca: I don't want to install ones using older base12:00
Chipacavidal72[m]: why?12:00
mborzeckizyga: nice review from opensuse again, pretty thorough12:01
mborzeckianyone wants to take a look at #6661?12:03
mupPR #6661: data/selinux, tests/main/selinux-clean: fine tune the policy, make sure that no denials are raised <SELinux> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6661>12:03
vidal72[m]Chipaca: they are built without additional compiler hardening12:03
mborzeckihopefully that's the last one in the selinux series12:03
Chipacavidal72[m]: what?12:03
pstolowskizyga: i raised one general question to #6643 ; my mistake for not adding that to review summary, it's kinda easy to miss and github makes it annoying to answer this kind of comments (no Reply underneath)12:07
mupPR #6643: tests: deny ioctl - TIOCSTI with garbage in high bits <Created by zyga> <https://github.com/snapcore/snapd/pull/6643>12:07
zygamvo: I'm not feeling very well and I'd like to take the 2nd half of the day off12:07
zygamvo: my highlights for the day are https://github.com/snapcore/snapd/pull/6684 which gives us the first part of working refresh app awareness12:08
mupPR #6684: overlord,tests: perform soft refresh check in doInstall <Created by zyga> <https://github.com/snapcore/snapd/pull/6684>12:08
zygamvo: and the suse review from https://bugzilla.suse.com/show_bug.cgi?id=112736612:08
zygamvo: both will need more work as you can suspect12:08
vidal72[m]Chipaca: https://wiki.ubuntu.com/Security/Features12:09
mborzeckipstolowski: whdyt about https://github.com/snapcore/snapd/pull/6682#discussion_r271708445 ?12:10
mupPR #6682: snap, gadget: move gadget read/validation into separate package, tweak naming <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6682>12:10
vidal72[m]Chipaca: some mitigations are enabled only since bionic12:10
mborzeckipstolowski: the catch is that we'll probably end up extracting snaps (instead of mounting them) like ubuntu-image does once we start to lear how to assemble an image12:11
vidal72[m]Chipaca: https://forum.snapcraft.io/t/build-snaps-with-hardened-toolchain-by-default/544412:11
Chipacavidal72[m]: gotcha12:12
Chipacavidal72[m]: 'snap info --verbose' will tell you the base of the most-stable channel of a snap12:13
Chipacafor arbitrary value of most-stable (but it's usually what you want)12:13
vidal72[m]ok, thx12:13
Chipacavidal72[m]: otherwise, e.g., curl -s -H Snap-Device-Series:16 'https://api.snapcraft.io/v2/snaps/info/gnome-calculator?fields=base' | jq -r '.["channel-map"][] | .channel.name + "/" + .channel.architecture + ": " + .base'12:15
pstolowskimborzecki: works for me12:20
pstolowskimborzecki: perhaps mvo can vote as well12:21
=== chihchun_afk is now known as chihchun
pedroniszyga: what's the status of #6583 ?12:52
mupPR #6583: cmd/snap-confine: move ubuntu-core fallback checks <Created by zyga> <https://github.com/snapcore/snapd/pull/6583>12:52
zygaI think it needs a 2nd review and I just kicked tests that may acutally pass now12:53
zygamvo: ^ can you finish your review there please12:53
* zyga is skipping the standup and officially EODing now13:00
mvothanks for the update zyga, enjoy eod13:00
zygathanks (cough, literally)13:01
=== ricab is now known as ricab|lunch
mvozyga: in this case, get well!13:05
mupPR snapd#6656 closed: tests: split travis spread execution in 2 jobs for ubuntu and non ubuntu systems <Created by sergiocazzolato> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6656>13:12
sergiusensmvo: let me take a look13:21
mvosergiusens: thank you!13:26
sergiusensmvo: commented on bug and PR13:28
mvosergiusens: yay, thank you - I think scriptlet is what we will do \o/13:29
mvosergiusens: we really just need control over the ENV and that gives it13:29
mupPR snapd#6682 closed: snap, gadget: move gadget read/validation into separate package, tweak naming <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/6682>13:32
mborzeckisil2100: do you prefer a force push to https://github.com/CanonicalLtd/ubuntu-image/pull/168 or another commit on top of the current one?13:41
mupPR CanonicalLtd/ubuntu-image#168: ubuntu_image: parser improvements <Created by bboozzoo> <https://github.com/CanonicalLtd/ubuntu-image/pull/168>13:41
mupPR # closed: core-build#11, core-build#22, core-build#26, core-build#3714:09
mupPR # opened: core-build#11, core-build#22, core-build#26, core-build#3714:10
=== ricab|lunch is now known as ricab
=== chihchun is now known as chihchun_afk
sergiusensmvo:  hey, here's more context on why I want the "second part" https://forum.snapcraft.io/t/snap-try-messaging-and-user-experience/10667/215:30
kyrofaI have a user tracking the stable channel, but is months out of date. `snap changes` shows nothing. Manual refreshes work. Does this ring any bells?15:33
=== chihchun_afk is now known as chihchun
pedronisChipaca: you should probably look at #6679 (at least the daemon bits), there are some questions for you there15:43
mupPR #6679: many: implement user removal <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/6679>15:43
pedroniscmatsuoka: I did a pass on ^ as well15:44
cmatsuokapedronis: thanks samuele, I'll provide a commit addressing those issues15:47
ijohnsonkyrofa: are they using a brand store, such as on a dell gateway for example?15:51
kyrofaijohnson, I don't believe so, no15:53
kyrofaOtherwise refresh control, you're thinking?15:53
ijohnsonyeah I had a similar issue recently with a user trying to install a snap on a dell gateway and the snap was gated (because of refresh-control in the default dell brand store) and that particular snap update had not been verified yet15:55
pedronisjdstrand: hi, I added one more snapd PR to your queue, I think you have 3 now, plus something landed I think where you were asked for a post-review today15:56
* cachio lunch15:57
=== pstolowski is now known as pstolowski|afk
jdstrandpedronis: ack, yep. I'm going to go through various store requests, emails then circle back around to reviews17:46
cachiomvo, https://travis-ci.org/snapcore/spread-cron/builds/51512934817:53
cachiomvo, I saw that error on revert test17:53
cachiomvo, perhaps the trace helps to understand17:54
cachiomvo, https://travis-ci.org/snapcore/spread-cron/builds/515129348#L220117:56
mvocachio: thanks, looking in a bit18:01
cachioniemeyer, hey, when you have time could you please take a look to https://github.com/snapcore/spread/pull/75 ?18:45
mupPR spread#75: Make spread tests for spread project run on google backend <Created by sergiocazzolato> <https://github.com/snapcore/spread/pull/75>18:45
WimpressSnapcraft Live starts in a few minutes - https://twitter.com/snapcraftio/status/111351276688996352218:46
* cachio afk19:04
mupPR snapd#6685 opened: image: prefer local for snapd/core snaps <Created by mvo5> <https://github.com/snapcore/snapd/pull/6685>19:12
Chipacazyga: gif is pronounced with the g as in gnat, the i as in suit, and the f as the second one in fifth19:12
zygaChipaca: isn't gif the endless topic of discussion on how to pronounce it? :-)19:13
Chipacazyga: yes :-)19:21
mupPR snapd#6686 opened: testutil: fix MockCmd for shellcheck 0.5 <Created by mvo5> <https://github.com/snapcore/snapd/pull/6686>19:25
niemeyercachio: Need to finish something else before coming back into it, but it's on my list for after that19:42
cachioniemeyer, np20:08
cachioniemeyer, thankd20:36
cachioniemeyer, thanks20:36
mupPR snapcraft#2520 opened: snap: set core as a base <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2520>23:09
mupPR snapcraft#2521 opened: cli: cleanup environment detection <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2521>23:24

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