mupPR snapcraft#2778 opened: Release changelog for 3.9 <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2778>01:23
mvohey mborzecki06:28
mborzeckimvo: hey06:29
zygagood morning07:18
zygabug triage duty!07:18
mborzeckizyga: hey07:18
zygahey :)07:18
zygaI'm working away from office today07:18
zygaI'll begin the day with some reviews07:21
zygathen go to UX docs07:21
mupPR snapd#7682 opened: [RFC] add kernel remodel undo tests and fix undo <Created by mvo5> <https://github.com/snapcore/snapd/pull/7682>07:30
zygamvo: small -1 comment on the 2.42.1 cherry pick PR07:30
zygamvo: 2.42 PR failed on mount ns test07:32
zygamvo: the failure indicates it is a missing lxd cleanup07:32
zygamvo: I think mborzecki sent a patch to master to fix that07:32
zygamvo: you may want to cherry pick that as well07:32
zygamvo: for context, this is how such failure looks like:07:32
mborzeckizyga: mvo: this one? https://github.com/snapcore/snapd/commit/03f3210eadeb47ee3c286e2c4fb242ce67365c86#diff-413b2f27493a11371053198f7020e23f07:34
zygaI think so07:34
zygaabeato: gentle ping about https://github.com/snapcore/snapd/pull/760307:47
mupPR #7603: interfaces: add dpdk and hugepages interfaces <Created by alfonsosanchezbeato> <https://github.com/snapcore/snapd/pull/7603>07:47
=== pstolowski|afk is now known as pstolowski
zygahey pawel08:04
mborzeckipstolowski: hey08:09
mvozyga, mborzecki thank you, I have a look08:33
=== pedronis_ is now known as pedronis
pedronismvo: hi, is #7624 ready for review?08:39
mupPR #7624: snap: make `snap download` download via snapd if available <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7624>08:40
mvopedronis: yes08:42
mvostore seems to give 503 right now, we have lots of red08:44
mupPR snapd#7663 closed: tests: add 14.04 canonical-livepatch test <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7663>08:45
pedronismvo: I made some initial comments in 768208:49
mvopedronis: yeah, excellent08:50
mvopedronis: I was wondering if deviceCtx is the way and you suggested it for oldModel so I will add it. thank you!08:51
mvopedronis: will make the diff a bit bigger (with test updates) so I may do it as a separate PR (?)08:51
mvopedronis: and *thank you* for the  feedback!08:51
pedronismvo: np08:55
pedronisdid a quick on 7624 too08:55
mvopedronis: thank you!08:56
mupPR snapd#7683 opened: overlord/ifacestate: remove automatic connections if plug/slot missing <Created by stolowski> <https://github.com/snapcore/snapd/pull/7683>09:01
* dot-tobias good morning09:09
mvoackk: did you had a chance to test the performance of snaps inside lxd recently? we fixed some performance issues and iirc the maas team mentioned they were hit by the slow performance inside lxd09:14
ackkmvo, hi, I haven't tried it recently. our issue was with squashfuse being super-slow, and particularly visible with the maas snap since it's quite big09:15
ackkand a lot of small (python) files that get read at startup09:15
mvoackk: yeah, that should be fixed in current core09:15
ackkmvo, nice, I'll give it a try, let you know how it goes09:16
mvoackk: thank you!09:16
ackkmvo, thank you for the update09:17
Chipacamorning folks09:22
Chipacamvo: tried reviewing the download one but all i'm seeing is bad news so i'm going to finish my coffee and look back later, maybe i'm just grumpy09:31
ackkmvo, I'm getting "Run configure hook of "maas" snap if present (run hook "configure": cannot perform readlinkat() on the mount namespace file descriptor of the init process: Permission denied)" when installing the maas snap, could it be something in the configure hook or is it snapd itself?09:33
dot-tobiasmvo: Is there a rough date for the next stable core release? Just to get a sense if it'll be this week, next or later.09:42
Chipacaackk: https://forum.snapcraft.io/t/cannot-perform-readlinkat-on-the-mount-namespace-file-descriptor-of-the-init-process-permission-denied/6743/709:45
mvoChipaca: oh no! I'm ready for bad news, I will steady myself09:46
ackkChipaca, thanks09:46
ackkChipaca, so I need the snapd snap, right?09:47
Chipacaackk: … no?09:48
Chipacaackk: sorry i'm not sure i'm being helpful at all here09:49
Chipacaackk: are you running this on 14.04?09:49
ackkChipaca, no, bionic09:49
Chipacaackk: then it's not that, probably09:49
ackkChipaca, spinned a fresh bionic container, tried snap install --edge maas09:49
Chipacaackk: what's in 'snap version'?09:50
* Chipaca should add a --irc flag to snap version09:50
ackkChipaca, one sec, I'm testing with an up-to-date bionic now since I didn't update snapd (from the deb) there09:51
Chipacaackk: in which case installing core or snapd would've also fixed it via snapd updating itself09:52
ackkChipaca, fwiw https://paste.ubuntu.com/p/2rCzjTtrMM/09:53
ackkI'm trying to install the maas snap but the store is flaky09:53
ackkyeah it's not working09:54
mvoackk: that error rings a bell, we saw this before, probably zyga knows more09:57
Chipacaackk: "yeah it's not working" same error, or now store issues?09:57
Chipacaackk: the store is having some issues now yes09:57
ackkChipaca, store errors, sorry09:57
ackkyeah I can't test right now09:57
ackkmvo, btw is "core" still required for installing core18 snaps?09:58
ackkmvo, installing maas didn't install it09:58
mvodot-tobias: iirc we did cherry pick your fix, right? the plan is to release 2.42.1 to beta this week and stable next week (if testing is all good). worst case stable in the week after that. but there is one pending branch that needs reviews :/09:58
mvoackk: core should not be required anymore09:59
mvoackk: on fresh systems snapd will now install core18 and snapd09:59
mvoackk: if you request a core18 only snap09:59
mupPR snapd#7676 closed: many: cherry pick test updates for 2.42 <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/7676>10:00
dot-tobiasmvo: Yes, it's been merged to master (though all credit to zyga). Thanks for the timeframe, I can work with that 😊10:00
mvodot-tobias: was it cherry-picked into release/2.42?10:01
mvodot-tobias: it will only be in 2.42.1 if it was cherry-picked, if not please point me to the PR10:01
dot-tobiasmvo: Checking10:01
mvodot-tobias: then I can (probably) cherry-pick it, I remember it was a pretty straightforward fix10:01
dot-tobiasmvo: Actually, it doesn't seem to be cherry-picked into release/4.42 yet. https://github.com/snapcore/snapd/pull/7655 is zyga's PR10:02
mupPR #7655: interfaces: allow introspecting network-manager on core <Bug> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/7655>10:02
mvodot-tobias: ok - now it is :) thanks for checking10:04
dot-tobiasmvo: 🙏🎉10:06
mupPR snapd#7684 opened: tests: disable mount-ns test in release/2.42 <Created by mvo5> <https://github.com/snapcore/snapd/pull/7684>10:08
pedronispstolowski: what's the status of #7601 ?10:18
mupPR #7601: overlord/ifacestate: use SetupMany in setupSecurityByBackend <Created by stolowski> <https://github.com/snapcore/snapd/pull/7601>10:18
pstolowskipedronis: i've the tab open and going to address Jamie's comment, will do today10:19
pedronisok, thx10:19
pedronismvo: did you make a card or something about unifying the default-provider extracting code?10:28
mvopedronis: uh, not yet, let me do this now10:29
Chipacamvo: what's that about a pending branch that needs reviews?10:34
Chipacamvo: i see one pr each in both 2.42 and 2.43 milestones, but it looks like the 2.42 one needs input from foundations and the 2.43 has all the reviewage it can handle ;-p10:38
mvoChipaca: the most important one is the foundations input10:43
Chipacaxnox is allegedly off gallivanting though :-p10:43
mvoChipaca: yeah, I can try to get a re-review from vorlon for #766110:44
mupPR #7661: packaging: tweak handling of usr.lib.snapd.snap-confine <Created by mvo5> <https://github.com/snapcore/snapd/pull/7661>10:44
* Chipaca throws flowers at vorlon 10:44
mvoChipaca: should we have a HO about 7624 ? do you think there is something fundamentally wrong with it?10:45
Chipacamvo: just started going through it again10:45
Chipacamvo: download-via-snapd needs to grow a 'resume' thing, but that's a separate pr10:45
mupPR snapd#7685 opened: snapstate,devicestate: make OldModel() available in DeviceContext <Created by mvo5> <https://github.com/snapcore/snapd/pull/7685>10:45
mvoChipaca: oh, yeah, thats a good point10:46
ackkmvo, I still see snapfuse at 100%cpu but perhaps that's expected?10:59
pedronismborzecki: I did a first pass on 766511:01
mborzeckipedronis: thanks11:03
pedronismborzecki: some questions/wonderings there11:03
ackkmvo, yeah, it seems it's still quite slow in the case of maas11:03
mvoackk: hm, ok. it should be quicker than before, can you see what version of snapd is running inside your lxd?11:06
pedronisChipaca: hi, could you answer the question about hidden commands vs autocomplete toward the end of comments in #7589 ?11:10
mupPR #7589: cmd/snap: add ability to register "snap internal" commands <⛔ Blocked> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/7589>11:10
ackkmvo, https://paste.ubuntu.com/p/2rCzjTtrMM/11:12
Chipacapedronis: probaboy11:14
Chipacamvo: review done. Less bad news than this morning early.11:19
Chipacamvo: still a little bad11:19
Chipacatoday's already been a long day11:33
Chipacai think i need more coffee11:33
Chipacastarting to chase shadows11:33
mupPR snapd#7686 opened: systemd: handle preseed mode <Prebaking> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7686>11:35
pstolowskiuhm... 403 via GET to "https://api.snapcraft.io/api/v1/snaps/sections"11:36
mborzeckipstolowski: yeah11:37
mborzeckiin the morning i got some 503s as well11:38
mvoChipaca: sorry for the delay, I was in a meeting - you say this gives me no progress bar for snpa donwload but iirc we don't have one today either. or am I missing something?12:05
mvoChipaca: hm, the code says there is a download progress, I'm just not seeing it, probably an issue on my side, I have a look. nevermind12:10
pedronismvo: Chipaca: I meade a suggestion there12:10
pedronisnot sure it fits or not, it was a quick thought12:11
pedronisChipaca: thanks for the answer, I enquired some more :)12:11
* zyga is at a service center and may miss standup12:19
zygaI'll make up the time in the evening to do all my responsibilities12:20
zygasorry :/12:20
Chipacamvo: i mean, try it: 'snap download core' gives you a nice progress bar12:23
Chipacapedronis: did you enquire, or did you inquire?12:23
mvoChipaca: yeah, I did something wrong, thanks for noticing this12:25
pedronisChipaca: afaict I meant enquire12:25
Chipacapedronis: phew12:26
pedronispstolowski: reviewed #768312:27
mupPR #7683: overlord/ifacestate: remove automatic connections if plug/slot missing <Created by stolowski> <https://github.com/snapcore/snapd/pull/7683>12:27
pedronispstolowski: thank you12:31
pedronispstolowski: I also made a general comment in 768612:31
pstolowskipedronis: ty12:32
sergiusensmvo, jdstrand, zyga: any assistance with https://paste.ubuntu.com/p/KXHFrbYmvZ/ ? This is on an autopkgtest env for 18.04/amd6412:33
sergiusensjournal show Oct 29 05:37:28 autopkgtest snapd[1828]: handlers.go:460: Reported install problem for "gtk3-hello" as 2f8217c0-fa0e-11e9-b619-fa163e102db1 OOPSID12:36
mvosergiusens: are these instances resource constrained? it looks like a malloc() (well, "new") wasn't successful, maybe(?) it requested too much memory12:37
* mvo needs to urgently get lunch though before he can look deeper12:37
sergiusensmvo: get lunch, I am not so worried about the failure itself (as I know our test works on google/spread)12:38
pstolowskipedronis: i've updated #7601, would you like to take a look at this as well or can it land when green?12:46
mupPR #7601: overlord/ifacestate: use SetupMany in setupSecurityByBackend <Created by stolowski> <https://github.com/snapcore/snapd/pull/7601>12:46
pedronispstolowski: no, it's probably fine if zygmunt and jamie are ok with it12:46
pedronispstolowski: I mean, I can look if you think I should12:48
pstolowskipedronis: maybe just check the comment & response re locking there. i don't think there was a deep reason it was done the way it was12:49
pedronispstolowski: I put it in my queue, I will also merge it if everything is fine12:50
pstolowskipedronis: thanks12:55
Chipacapedronis: should I do the 'hidden-but-completed' thing?13:17
pedronisChipaca: if you can, and is not too distracting, yes13:17
Chipacait's not, right now13:17
=== ricab is now known as ricab|lunch
Eighth_Doctorpopey_, zyga, mborzecki, mvo: I've finally introduced `snapd` for EL813:22
Eighth_Doctorthat was a more annoying bringup than I thought it would be13:23
mborzeckibut it's done now :P13:23
Eighth_Doctoryeah, I'll do a PR to snapd upstream with a spec sync soon13:24
Eighth_DoctorI haven't done one of those in a while13:25
zygaEighth_Doctor: thank you for persisting, congratulations!13:30
Eighth_Doctorbetween broken architectures and missing packages due to incomplete imports, I wasn't certain EL8 was ever going to get built13:31
mborzeckiEighth_Doctor: if you take away these problems, there's no more excitement in getting things done13:36
Eighth_DoctorI guess so13:36
Eighth_Doctorbut I didn't enjoy people bitching about updated snapd releases being held up13:37
jdstrandsergiusens (cc mvo, zyga): that is the memory consumption bug13:43
jdstrandsergiusens (cc, mvo, zyga): can you try edge?13:44
Chipacamvo: silly question: i just noticed the completion helper for 'snap' in 1604 is ancient, is the update on its way / stuck somewhere?13:44
jdstrandsergiusens: zyga did excellent PR work to fix that which is committed, and may even end up in 2.42.2 (not sure of the status of that)13:44
jdstrandor is it 2.42.1 (did I say I'm not sure of the status of 2.42.N? :)13:45
mvoEighth_Doctor: e8> nice! thanks so much!13:47
mvojdstrand: 2.42.1 once we get it out, pending one review from xnox or vorlon (7661)13:48
mvoChipaca: could you tell me some more details? is the completion helper part of bash-completion there?13:48
Chipacamvo: I mean data/completion/snap that gets shipped to /usr/share/bash-completion/completions/snap13:49
jdstrandmvo: cool, thanks :)13:49
Chipacamvo: this is not about completion of snapped commands13:50
jdstrandmvo: I wouldn't mind 2.42.1 with that change myself :)13:50
Chipacamvo: but of the snap command itself13:50
Chipacamvo: an easy way to check: snap debug <tab>, vs snap debug model <tab>13:50
* zyga blushes :-)13:50
mvojdstrand: I cherry picked it already :)13:51
Chipacamvo: with the one in-tree, that second one should not offer all debug commands all over again13:51
mvoChipaca: what do we need to do to update it? (sorry, I still don't have the full picture but hopefully I'm getting there :)13:52
Chipacamvo: I don't know :) the one in-tree was fixed in August13:52
Chipacamvo: how does the .deb pick that up? maybe the .deb is older than august?13:53
Chipacaor the release13:53
Chipacai guess august on master means september release so it's not unpossible for this to be alright just lagged13:54
mvoChipaca: aha, got it now - yes, I think we want 2.42.113:54
mvoChipaca: we have no deb update in a while because of various issues13:54
Chipacaa'ight then13:54
mvoChipaca: one of which is this postinst PR review :)13:54
Chipacamvo: are we going to be able to update before 16.04.<whatever>?13:54
Chipacamvo: because images don't ship packages from -updates so snapcraft hates us13:55
Chipacabecause there's all these neat features they can't depend on or sth13:55
Chipacaanyway, snowballing moaning doesn't hep13:55
Chipacai need to get ready for standup13:55
mvoChipaca: woah, let me look13:55
mvoChipaca: me too - I fixed the issues in download but I guess this could need a refactor to unify a bit of this code13:58
mupPR snapd#7687 opened: snap-bootstrap: check gadget versus disk partitions <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/7687>13:59
jdstrandmvo: re cherrypic, yep, thanks! looking forward to 2.42.1 in a channel somewhere :) (not pushing, just enthusiastic :)14:00
mvojdstrand: :)14:01
jdstrandmvo: also, I heartily endorse the approach for snap-confine in 766114:02
jdstrandmvo: that'll be real nice to finally have fixed :)14:02
mvojdstrand: \o/14:06
sergiusensjdstrand: cannot easily try edge on autopkgtests, will try later though, thanks14:06
Chipacacachio: BTW there are 63520800 ways of getting 3 tests out of 400 so the fact that you found a failing combination is quite something14:13
jdstrandsergiusens: with the fact that it is a gtk3-hello demo which I bet is using the content interface, I'm inclined to say, don't worry about it for now since it will be fixed soon. if it is happening after 2.42.1 is in stable, report back (cc mvo)14:18
* mvo had the same thoughts14:18
cachioChipaca, there are pretty good search techniques :)14:26
jdstrandpstolowski: hey, fyi, this is what I meant to say before: https://github.com/snapcore/snapd/pull/7601#discussion_r34010108614:26
mupPR #7601: overlord/ifacestate: use SetupMany in setupSecurityByBackend <Needs Samuele review> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7601>14:26
mupIssue # closed: classic-snap#7, classic-snap#8, classic-snap#14, classic-snap#15, classic-snap#31, classic-snap#32, classic-snap#3314:35
mupPR # closed: classic-snap#24, classic-snap#27, classic-snap#28, classic-snap#3014:35
mupIssue # opened: classic-snap#7, classic-snap#8, classic-snap#14, classic-snap#15, classic-snap#31, classic-snap#32, classic-snap#3314:38
mupPR # opened: classic-snap#24, classic-snap#27, classic-snap#28, classic-snap#3014:38
=== ricab|lunch is now known as ricab
pstolowskijdstrand: ok, i see, right. i was thinking if the unlocking/locking there was deliberate or not. semantically it looks irrelevant to me and if it had any sideffects it would be a bit fishy tbh. either way it's impossible to test / prove. maybe pedronis can recall something14:44
pedronispstolowski: I'm having a break and then I will look at that PR14:53
om26erprivate jobs have taken over https://launchpad.net/builders ...14:57
cachiomborzecki, hey, could yo utake a look to #768115:00
mupPR #7681: tests: add info that could be used to determine why journalctl is failing to restart <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7681>15:00
mupPR snapd#7688 opened: cmd/snap: make completion skip commands (& opts) that ask for it <Created by chipaca> <https://github.com/snapcore/snapd/pull/7688>15:00
cachioIt is confirmed the start-limit-hit issue on core-1815:00
Chipacaom26er: private jobs do that15:01
jdstrandpstolowski: ack, not a blocker or anything. thanks for thinking about it15:05
pstolowskijdstrand: sure, thanks for the review!15:07
pstolowskimvo, pedronis: ah, i forgot about the fix for the issue of services & install hook that we want to fix for pre-seeding, and it needs ijohnson's #734115:15
mupPR #7341: many: introduce package seed and seedtest <Created by pedronis> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7341>15:15
pstolowskii mean #743115:16
mupPR #7431: overlord/snapstate: don't re-enable and start disabled services on refresh, etc <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7431>15:16
ijohnsonpstolowski: I should update that PR with your suggestions later today, I implemented them just need to finish testing that the spread tests still work with my changes15:16
pstolowskiijohnson: that's great, thanks15:17
Chipacamvo: you mind if i push some code to #7624?15:17
mupPR #7624: snap: make `snap download` download via snapd if available <Needs Samuele review> <Created by mvo5> <https://github.com/snapcore/snapd/pull/7624>15:17
Chipacanot saying i will, but i might15:18
ijohnsonhmm it's rather confusing for github's PR diff files view that for a line changed, the "unexpanded" snippet which shows the function/unindented block of code the line is in doesn't account for the fact that the "unexpanded" part itself also changed15:22
cachiozyga, could yo uplease re-review #768115:23
mupPR #7681: tests: add info that could be used to determine why journalctl is failing to restart <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7681>15:23
cachioI added a fix there15:23
pedronispstolowski: mmh, are we losing the timing in 7601 ?15:24
* cachio lunch15:28
pstolowskipedronis: we're passing it to SetupMany, it's measured there15:30
pstolowskiindentation is a bit weird15:30
pedronispstolowski: sorry, yes, I missed it15:34
pedronispstolowski: I commented a bit more on the lock thing15:34
pedronispstolowski: and gave my +115:35
pedronispstolowski: it can land when it is green15:35
pedronisChipaca: I commented on 768815:36
Chipacapedronis: i saw15:36
Chipacapedronis: we can always think more :)15:36
pedronisChipaca: I can see a couple of ways out15:36
zygacachio: doing so now15:36
pstolowskipedronis: thank you, yes, exactly what i was thinking15:36
zygacachio: interesting15:38
zygacachio: does this mean we can make journalctl fail if we just restart the service in a loop?15:38
pedronisChipaca: we could add code to stick (hidden) at the end of any description of hidden command that don't have it on their own. And have a flag to stop that15:38
zygacachio: if we could do that without any snapd stuff in the way, we could report a bug upstream15:38
zygacachio: it feels like explicit requests should not be counted towards the number of after which the service is not started15:38
Chipacapedronis: a contra to that is that we'd have to reflect the structs ourselves15:39
pedronisChipaca: ?15:40
Chipacato get the other flag15:40
Chipacaunless you meant a flag to addCommand15:40
Chipacanot a struct tag15:40
Chipaca(my first read was that you meant a struct tag, like hidden:"true" completed:"true"15:41
pedronisChipaca: we have our own flags on cmdInfo15:41
zygakenvandine: hey, personal question15:41
zygakenvandine: where can I report a bug on gnome-shell where someone I can talk to will look15:42
kenvandinezyga: 715:42
pedronisChipaca: can we key on Item somehow? can it be partial, how does that bit work?15:42
zygakenvandine: I have this pet bug of mine that is very frustrating15:42
zygakenvandine: and probably something silly somewhere15:42
kenvandineping Trevinho :)15:42
zygakenvandine: high-dpi in vmware results in white background15:42
zygalow-dpi is ok,15:42
zygabackground shows15:42
zygakenvandine: all you need to do is to toggle 200% zoom15:42
zygaand you get white background15:42
pedronisChipaca: I mean, we could also keep global maps Item -> flags, or Description -> flags (assuming they are unique enough)15:42
kenvandinethat's annoying15:42
zygathis bug was around for about a year now15:42
zygabut I never reported it, hoping someone would notice and it wouldn't be just my machine15:43
zygabut now I just want to report it15:43
zygabecause maybe my machine has something useful on it15:43
zygait happens in vmware fusion on both my macbook and my imac15:43
zygait happens in not just ubuntu15:43
Trevinhomhmh, ah so the problem is the guest15:43
zygaall gnome shell is affected15:43
zyga(opensuse, fedora, debian and ubuntu for sure)15:43
zygaperhaps it's some texture size limit15:43
zygaor something of this nature15:43
zygait's a new thing15:44
zygait did work great about a year ago, using same older VMs works good now15:44
Trevinhozyga: always in x11 or wayland?15:44
zygalet me check15:44
zygabut double checking now15:44
Chipacapedronis: go-flags figures out what things to complete, and we then get the list of whole names even if it's completing a partial15:44
Trevinhozyga: and also using new vmware + old ubuntu?15:44
zygaTrevinho: it's just new gnome/kernel perhaps that causes this15:44
zygaone sec15:45
zygaTrevinho: it happens in x11, let me log out into wayland15:45
pedronisChipaca: ok, so it sound we could make a map in addCommand as well, instead of looking at the description15:45
zygait's also 100% reproducible on my machines using latest version of everything, I can toggle back and forth15:45
pedronisthough in principle I have nothing agaisnt sticking (hidden) there,  it's true15:45
zygaTrevinho: it affects x11 and wayland equally15:46
zygaeverything works great at 100% zoom in 5K15:46
zygain 200% zoom everything but background displays fine15:46
zygalooking at logs now15:46
om26erIs https://snapcraft.io/blog/handy-snapcraft-features-remote-build in the stable channel now ? Also can we use that to build private projects and how is the code actually pushed to launchpad builders ?15:46
Chipacapedronis: yes, we could. Assuming you mean a map of just things to skip, and there weren't synonyms where we wanted to skip only one of them, then it'd probably work15:46
pedronisChipaca: yes, just the one to skip15:47
om26erIOW is it secure ?15:47
Chipacapedronis: note we only get the last command15:47
zygaTrevinho: in wayland, toggling 200% zoom gives me15:47
Chipacaom26er: it is secure but as it says there your build is public15:47
Chipacapedronis: 'snap debug foo' → you get "foo"15:47
zygaTrevinho: wayland journal fragment after setting 200% zoom https://www.irccloud.com/pastebin/beonJbDn/15:47
pedronisChipaca: I see, so we cannot use item15:48
pedronisbut we can use description15:48
pedronisassuming they are unique15:48
pedronisand they should be really15:48
pedronisexcept for a few degenerate cases that we can accept15:48
pedronislike "Internal (hidden)"15:48
zygaTrevinho: x11 journal fragment after setting 200% zoom https://www.irccloud.com/pastebin/6Wm8NA5u/15:49
zygaTrevinho: I can report that, just point me the way15:49
pedronisChipaca: shortHelp is description ?15:49
zygathank you for looking and I'd love to know if you have any ideas15:49
pedronisfor commands15:49
Chipacapedronis: yes15:49
Trevinhozyga: don't think journal is saying anything wrong, unless edid, but shouldn't matter for this15:50
zygayeah, apart from the background there are no problems15:50
Trevinhozyga: better if you report it at https://gitlab.gnome.org/GNOME/mutter/issues15:50
zygaTrevinho: nothing fails, 3D works, all the apps work flawlessly15:50
zygaTrevinho: sure thing15:50
om26er@chipaca that's not a problem, we want to release the software for everyone but the source code model is closed for now15:50
Chipacaom26er: but if the build is public, the source is public15:51
Chipacaom26er: as i read it at least15:51
Trevinhozyga: also when setting 200 the WM is using native 2x or zooming?15:52
Chipacaom26er: maybe confirm with somebody that knows :)15:52
Trevinho xprop -root RESOURCE_MANAGER15:52
zygaTrevinho: I don't know the difference15:52
Trevinhoand maybe you can -spy that whiele changing the value15:52
zygalet me get that output for you :)15:52
Trevinho(just use -spy :))15:52
Trevinhozyga: well, it's not stretched I imagine, right?15:52
Trevinhoit's just asking the WM to use proper scaling I assuem15:53
zygaTrevinho: no, it's all nice and pretty all the time15:53
pstolowskipedronis: was your suggestion to also avoid 'preseed' term in the apparmor/security backend?15:53
om26erChipaca not sure who works on the build service these days, is Colin (cj watson) the right person ?15:53
pedronispstolowski: no, there is fine I think15:54
pedronisI mean it's ok if you want to use a different term15:54
pedronislike DiskOnly or something15:54
pedronisbut preseed would work too15:54
pstolowskipedronis: ok, cause emulation indeed wouldn't fit15:54
zygaTrevinho: RESOURCE_MANAGER(STRING) = "*customization:\t-color\nXft.dpi:\t96\nXft.antialias:\t1\nXft.hinting:\t1\nXft.hintstyle:\thintslight\nXft.rgba:\trgb\nXcursor.size:\t24\nXcursor.theme:\tYaru\n"15:54
zygaTrevinho: not sure how to use -spy15:54
Trevinhojust pass -spy, it will print you something if it changes15:55
Trevinhozyga: that's at 200%?15:55
zygaTrevinho: yes15:55
zygaTrevinho: let me give you output at both15:55
zygaI'm in x11 now15:55
zygathough that matters for xprop15:55
Trevinhoweird, so... mh not really using internal scaling...mhmh15:55
Trevinhoa scaled system should have Xft.dpi = scale*9615:56
zygathis ix xprop -root -spy15:56
zyga*this is xprop -root spy15:56
zygawhen going from low to high dpi15:56
zygaTrevinho: let me double check if I got those initial output bits right15:56
Trevinhoah, ok it changes to 192, it makes sense then15:56
TrevinhoI saw already, yeah... it's just showing things at 2%15:56
zygaTrevinho: so at 200% it is: RESOURCE_MANAGER(STRING) = "*customization:\t-color\nXft.dpi:\t192\nXft.antialias:\t1\nXft.hinting:\t1\nXft.hintstyle:\thintslight\nXft.rgba:\trgb\nXcursor.size:\t48\nXcursor.theme:\tYaru\n"15:57
Trevinhoso, weird. Cause there's should no difference between that and normal 2x scaling (normal = native)15:57
zygaTrevinho: and at 100% it is15:57
zygaRESOURCE_MANAGER(STRING) = "*customization:\t-color\nXft.dpi:\t96\nXft.antialias:\t1\nXft.hinting:\t1\nXft.hintstyle:\thintslight\nXft.rgba:\trgb\nXcursor.size:\t24\nXcursor.theme:\tYaru\n"15:57
Trevinhoyeah makes sense15:57
pedronisChipaca: I played a bit and indeed our description are basically unique, so that idea would work15:57
zygaTrevinho: anything else I can try?15:57
Chipacapedronis: is it worth it?15:57
Trevinhozyga: mh, no in the top of my head, should probably do some deeper debugging.15:58
zygalet me report the issue15:58
pedronisChipaca: less magic, so yes15:58
zygawhat to include in the report?15:58
pedronisChipaca: it's fairly simple I think15:58
Trevinhozyga: I use vmware player here, and I don't think that has the option visible15:58
pedronisChipaca: anyway probably simple enough that I should pick up your PR and try to fit it in there15:59
Trevinhobut I assume I can have the same by changing manually the conf fike15:59
cmatsuokazyga: patriciadomin needs more space for snap data, would it work if she just moves /var/snap to a different filesystem?15:59
zygaTrevinho: I think it's a fusion thing, on a high-dpi display you get to choose to give the VM the full resolution or to give it a scaled down version (e.g. for older OSes)15:59
zygacmatsuoka: yes, it should work perfectly15:59
pedronisChipaca: but maybe your point is that you think that by default hidden commands should auto complete?15:59
cmatsuokazyga: thanks!16:00
Trevinhozyga: if you instead change the option internally from the VM only?16:00
ijohnsonpedronis: reviewed #767916:00
mupPR #7679: many: changes to testing in preparation of Core 20 seed consuming code <Created by pedronis> <https://github.com/snapcore/snapd/pull/7679>16:00
Trevinhoi.e from g-c-c, as that's what I'm doing here all the times, but never had such issue16:00
zygaTrevinho: this is internally in the VM16:00
zygaTrevinho: I didn't change the vmware fusion settings16:00
zygaTrevinho: (having that high-dpi screen really makes me want to use the high-dpi for daily work)16:00
Trevinhoah, but well fusion I suppose is doing something different the16:00
zygaTrevinho: so to be clear, the VM on the host has a setting toggled to enable "retina" resolution (native resolution of the panel)16:01
zygaTrevinho: if you toggle that down the VM gets told that resolution is half in each dimension16:01
Trevinhoalthough I've not used this in a very high dpi screen, but simulating it shoudl be the same16:01
zygaTrevinho: and there's no 200% zoom to choose from16:01
Chipacapedronis: no, i'm not sure what my point is, but that is not it :)16:01
pedronisChipaca: ok, I'll try to push a PR based on yours and then we can discuss something concrete16:01
Chipacapedronis: ok16:02
Trevinhozyga: ah, I see then. But well in the normal case it should just give the same display size so...16:02
pedronisChipaca: in a bit16:02
Trevinhobasically the same of the player16:02
Trevinhozyga: by luck changing the bg image rebuilds anything and makes it appear or no?16:02
zygaTrevinho: let me check16:02
zygabut I think not16:02
Trevinhoin theory should not be the same of what happens on resolution changes, but.. sometimes.. :)16:03
zygaTrevinho: nope, no luck16:03
Trevinhozyga: this seems close https://gitlab.gnome.org/GNOME/gnome-shell/issues/117516:03
Trevinhocould be texture size problem indeed16:05
zygaTrevinho: https://gitlab.gnome.org/GNOME/mutter/issues/89416:05
TrevinhoI suppose in some cases it should be tiled16:05
zygaTrevinho: I'll grab lunch (starving) and be back soon16:05
zygaTrevinho: oh actually, I think I could set a background *color* correctly16:05
zyganow this option seems gone16:05
* Chipaca ⇝ tea16:06
Trevinhozyga: I've marked as duplicate, feel free to copy&paste that text to the original bug so we don't loose the comment :)16:06
Trevinhosorry, I linked it too late16:06
Trevinholooks like something introduced during the 3.32 cycle though, so if running a machine with 3.30 is working, maybe would be nice to bisect 3.32 branches16:08
om26erI have python3.8 in a snap and a snapcraft plugin allowing other snaps to use python3.8 https://forum.snapcraft.io/t/introducing-python3-8-as-a-snap/1392316:12
pedronisijohnson: thansk for the reviews, I tried to answer all the questions16:15
ijohnsonpedronis: cool looking now16:15
ijohnsonpedronis: thanks for the explanations, lgtm still16:17
pedronisthank you16:18
cachiozyga, hey, so, we can make it fail in a loop but just on ubuntu core16:24
cachioit is not reproduced on classic16:24
vorlonmvo, Chipaca: sorry, my slowness in reviewing that PR (aside from being sick yesterday) is that the PR is of course only showing me the delta vs the latest commit, and what I actually care about is the delta vs the current version of the packaging in bionic.  I'll get to it today16:26
mvovorlon: thank you, sorry for being so pushy about it16:30
Chipacamvo: do you know the tag of most-recently-in-bionic ?16:30
Chipacagit tag i mean16:30
mvoChipaca: one sec, looking16:31
mvoChipaca: commit adee89d789e452c9d12a90625865954cae448d42 (tag: 2.40)16:33
zygaTrevinho: thank you16:35
zygacachio: interesting, wonder why there16:35
zygacachio: and if you copy the config to a regular system, can we make it fail there?16:35
cachiozyga, didn0't try that16:36
cachiolet me try16:36
zygacachio: perhaps it's something specific to the config we use on core16:36
zygacachio: cool, good luck16:36
Chipacamvo: 2.40, or release/2.40?16:47
mvoChipaca: tag is "2.40"16:51
Chipacagah, github does not help with this16:56
Chipacavorlon: sorry :) best i can do is say git diff 2.40 mvo5/snap-confine-conffile -- packaging/ubuntu-16.0416:56
Chipacabut that's not particularly useful :)16:57
mupPR snapd#7689 opened: client,daemon: pass sha3-384 in /v2/download to the client <Created by mvo5> <https://github.com/snapcore/snapd/pull/7689>16:57
Chipacait's http://paste.ubuntu.com/p/ZZ4nK4Byq9/ fwiw16:57
cachiozyga, well I found that it also fails on classic16:58
zygacachio: that's good, closer to reproducer for a bug report16:59
cachiozyga, https://paste.ubuntu.com/p/jqdCqX7sTn/16:59
mvoChipaca: nice!16:59
zygacachio: why "not systemctl restart"?17:00
mvoChipaca: I pushed 7689 that probably should land before we land 7624 (tweaks the api a little bit for things we need)17:00
cachiozyga, we restart until it fails17:00
zygaah, I see17:01
cachiowhat we see is that when the tests run fast17:01
cachiothe service fails to be restarted17:01
cachioand it is because we are reaching the start limit17:01
zygacachio: would be worth to report a bug on upstream systemd project with the question if manual restarts should count towards the failure counter17:02
zygacachio: perhaps we should reset the failure state17:02
zygacachio: and try again?17:02
cachioreset the failut state?17:03
cachiozyga, how can I do that?17:03
zygacachio: one sec17:03
zygathere's a command for it17:03
zygait's pretty rarely used17:03
cachiozyga, ahh, nice17:04
zygacachio: systemctl reset-failed17:05
zygacachio: note that we could do something else17:05
zygacachio: run systemctl _stop_ to stop the journal17:05
zygato let various IO buffers stabilize17:05
zygacachio: then start again17:05
zygacachio: the sync in the middle may give everything enough time to actually not choke17:06
zygacachio: but if stop or start fails, we could run the reset-failed command to see if we can try again17:06
cachiozyga, let me try it17:06
zygacachio: good luck, I have not tried that, maybe it won't lead anywhere17:06
zygabut maybe it will17:06
cachiousing stop / sync / start17:08
cachioworks well17:09
cachiorexec 20 times without error17:09
zygacachio: worth giving it a shot17:09
cachiozyga, fix pushed17:12
zygacachio: super17:12
zygacachio: thank you!17:12
cachiozyga, to you17:12
mvopedronis: you may want to check https://github.com/snapcore/snapd/pull/7689 just to make sure that the chosen name (client.DownloadInfo) is ok. happy to adjust as needed17:19
mupPR #7689: client,daemon: pass sha3-384 in /v2/download to the client <Created by mvo5> <https://github.com/snapcore/snapd/pull/7689>17:19
mupPR snapd#7690 opened: cmd/snap: make completion skip hidden commands (unless overridden) <Created by pedronis> <https://github.com/snapcore/snapd/pull/7690>17:19
=== pstolowski is now known as pstolowski|afk
pedronisChipaca: #7690 is what I had in mind17:19
mupPR #7690: cmd/snap: make completion skip hidden commands (unless overridden) <Created by pedronis> <https://github.com/snapcore/snapd/pull/7690>17:19
pedronismvo: seems fine17:20
pedronisChipaca: ?17:24
Chipacapedronis: reviewed17:25
Chipacapedronis: "hehe" because you called it 'autocomplete', and the flag 'autocompletHidden' (sic)17:25
Chipacapedronis: when it's completion, not autocompletion17:26
pedronisChipaca: it's funny because people call it autocompletion but is not the correct term?17:26
Chipacapedronis: on the phone it is17:27
Chipacapedronis: who calls it autocompletion on the terminal?17:27
* Chipaca googles17:27
pedronisme apparently17:27
pedronisliving in the future17:27
Chipacapedronis: you and 190k+ people17:27
Chipacapedronis: vs 6M+ for just 'completion'17:27
Chipacapedronis: you're the 3%17:28
zygaChipaca: this autocompletes this problem17:28
pedronisChipaca: the flag is the override, so don't need to set on first boot, I need to set it on any hidden command we still want to auto complete17:28
pedronismostly the key stuff I think17:29
Chipacazyga: https://i.imgur.com/9HY53al.gif17:29
Chipacapedronis: d'oh! obvs17:29
zygamy day is autocomplete now17:29
Chipacazyga: https://i.imgur.com/NJZgC1W.gif17:30
mupPR snapd#7691 opened: builtin/browser_support.go: allow monitoring process memory utilizati… <Created by Erick555> <https://github.com/snapcore/snapd/pull/7691>17:30
pedronisChipaca: fixed and marked some commands back17:39
vidal72[m]what "disabled" mean under "notes" in "snap list" outpout?17:39
Chipacavidal72[m]: that the snap is not enabled17:40
Chipacavidal72[m]: i.e. that it is not even mounted17:40
Chipacavidal72[m]: or, if you're seeing 'snap list --all', that it is not current17:40
Chipacavidal72[m]: why?17:40
vidal72[m]Chipaca: it seems my snap was during self-update, now "disabled" is gone17:42
pedronisChipaca: wonder if this means we would like: snap debug commands  to list them all anyway17:48
Chipacapedronis: say again?17:48
pedronisChipaca: do we need a way to show all the commands implemented17:49
pedronisbefore this PR you could17:49
pedronis"snap " <TAB> away to that17:49
Chipacapedronis: i don't think we do17:49
pedronisnow not anymore17:49
pedronisChipaca: ok17:49
Chipacapedronis: «GO_FLAGS_COMPLETION=1 snap ""» fwiw17:49
pedronisChipaca: that will not show the hidden ones now17:50
Chipacapedronis: i know, i'm just saying you to get it non-interactively17:50
pedronisI mean that command to list commands, for us, not in general17:50
Chipacapedronis: i don't think we need it, but we can add it if we do17:54
Chipacai'ma EOD17:58
mupBug #1584590 changed: Snap install download progress neatening up <snapd:Confirmed> <https://launchpad.net/bugs/1584590>18:07
mupBug #1616629 changed: could not unmarshal state entry "snap-type" <canonical-bootstack> <snapd:Confirmed> <https://launchpad.net/bugs/1616629>18:07
mupBug #1574103 changed: Raspberry Pi 2 image LEDs are swapped. <snapd:In Progress by maciek-borzecki> <https://launchpad.net/bugs/1574103>18:10
mupBug #1630789 changed: normal users can't run snaps inside of LXD containers <verification-needed> <snap-confine:Fix Released by jdstrand> <snapd:Fix Released by tyhicks> <snap-confine (Ubuntu):Fix Released by jdstrand> <snapd (Ubuntu):Fix Released by tyhicks> <snap-confine (Ubuntu Xenial):Fix18:10
mupCommitted> <snap-confine (Ubuntu Yakkety):Fix Committed> <https://launchpad.net/bugs/1630789>18:10
mupBug #1664383 changed: running snap list <snap> on a system that has no snaps installed exits 0 <conjure-up> <snapd:Fix Released> <https://launchpad.net/bugs/1664383>18:13
zygasil2100: hello, can you please enqueue looking at https://bugs.launchpad.net/snappy/+bug/1701018 tomorrow18:14
mupBug #1701018: Splash screen is not enabled in kernel <Snappy:Confirmed> <https://launchpad.net/bugs/1701018>18:14
zygathere are open PRs and I think someone should have a look at them, decide which way to go and update the bug report18:15
mupBug #1721676 changed: implement errno action logging in seccomp for strict mode with snaps  <verification-done-xenial> <verification-done-zesty> <snapd:Fix Released by tyhicks> <linux (Ubuntu):Fix Released by tyhicks> <linux (Ubuntu Xenial):Fix Released by tyhicks> <linux (Ubuntu Zesty):Fix18:16
mupReleased by tyhicks> <linux (Ubuntu Artful):Fix Released by tyhicks> <https://launchpad.net/bugs/1721676>18:16
mupBug #1708527 changed: Add /proc/partitions to system-observe <snapd-interface> <snapd:Fix Released> <https://launchpad.net/bugs/1708527>18:19
mupPR snapd#7688 closed: cmd/snap: make completion skip commands (& opts) that ask for it <Created by chipaca> <Closed by pedronis> <https://github.com/snapcore/snapd/pull/7688>18:23
mupBug #1583975 changed: ubuntu IoT - webcam-demo 1.0.2 - apparmor="DENIED" operation="open" profile="webcam-demo.canonical_webcam-demo_1.0.2" name="/dev/video0" pid=3898 comm="fswebcam" requested_mask="wr" denied_mask="wr" fsuid=0 ouid=0 <snapd:Fix Released> <https://launchpad.net/bugs/1583975>18:25
mupBug #1648431 changed: Allow snaps to shadow mounts from core <snapd:Fix Released by kalikiana> <https://launchpad.net/bugs/1648431>18:25
mupBug #1649837 changed: Can remove required-snaps <snapd:In Progress by pedronis> <https://launchpad.net/bugs/1649837>18:25
zygapedronis: kind request to look at the bug https://bugs.launchpad.net/snapd/+bug/1649837 and check if the state can be updated now18:27
mupBug #1649837: Can remove required-snaps <snapd:In Progress by pedronis> <https://launchpad.net/bugs/1649837>18:27
mupBug #1650689 changed: Channel switching (track new channel) does not work if the two channels happen to have identical snap packages <lxd> <Snappy:Fix Released> <https://launchpad.net/bugs/1650689>18:28
pedroniszyga: we never implemented a patch so it's still not finished, now that we have idempotent patches maybe we can finish it18:30
zygapedronis: thank you, I'll update the bug18:31
zygamvo: are you still working by any chance?18:35
mupBug #1659534 changed: userdel doesn't supports extrausers <patch> <verification-done-bionic> <verification-done-xenial> <Snappy:Fix Released> <shadow (Ubuntu):Fix18:40
mupReleased> <shadow (Ubuntu Xenial):Fix Released> <shadow (Ubuntu Bionic):Fix Released> <shadow (Ubuntu Cosmic):Confirmed> <https://launchpad.net/bugs/1659534>18:40
mupBug #1660879 changed: snap refresh with more than one argument produces poor error message <snapd:Fix Released> <https://launchpad.net/bugs/1660879>18:40
mupBug #1661436 changed: snap download can't find snaps from a branded store <snapd:In Progress> <https://launchpad.net/bugs/1661436>18:40
mvozyga: ish, what can I do for you?18:53
* cachio afk19:02
zygaah, sorry, I went upstairs for some tea19:07
mupBug #1661590 changed: GNOME Software only supports running one application from a snap <verification-done-xenial> <GNOME Software:Fix Released> <Snappy:Fix Released> <gnome-software (Ubuntu):Fix Released by robert-ancell> <gnome-software (Ubuntu Xenial):Fix Released> <gnome-software (Ubuntu19:11
mupArtful):Won't Fix> <gnome-software (Ubuntu Bionic):Fix Released by robert-ancell> <https://launchpad.net/bugs/1661590>19:11
mupBug #1693423 changed: initramfs error while loading Custom ubuntu core Image on X86 but it works fine in KVM <Snappy:Invalid> <https://launchpad.net/bugs/1693423>19:14
mupPR snapcraft#2778 closed: Release changelog for 3.9 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2778>19:25
* zyga managed to put Lucy to bed20:23
zygaback to bug review20:23
mupPR snapcraft#2779 opened: ci: switch to travis workspaces <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2779>20:31
mupPR snapd#7692 opened: tests: opensuse tumbleweed has similar issue than arch linux with snap --strace <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/7692>21:06
mupPR snapd#7601 closed: overlord/ifacestate: use SetupMany in setupSecurityByBackend <Needs Samuele review> <Created by stolowski> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/7601>21:54
om26erHi! I landed a pull request for a project that has been setup for snapcraft automatic builds but it seems the build didn't start22:16
om26erHere is the relevent PR https://github.com/crossbario/crossbar/pull/164922:17
mupPR crossbario/crossbar#1649: Base snap on python 3.8 <Created by om26er> <Merged by om26er> <https://github.com/crossbario/crossbar/pull/1649>22:17
* zyga EODs22:19
sil2100zyga: ACK! Will look!23:50

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