zygaGood morning mborzecki05:06
mborzeckizyga: hey05:07
zygaRies and shine,05:07
mupPR snapcraft#2563 closed: Release changelog for 3.5 <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2563>05:40
=== pstolowski|afk is now known as pstolowski
mborzeckipstolowski: hey07:12
mborzeckihmm mirrors.fedoraproject.org does not resolve07:36
mborzeckipstolowski: can you try nslookup fedoraproject.org ?07:39
Chipacabongiorno, principesse e principi08:01
pstolowskiChipaca: hey!08:02
mborzecki_Chipaca: hey, thought you're swapping today08:02
ChipacaI was talked into not doing so :-)08:03
Chipacaas in, not needing to swap08:03
Chipacanot as in not swapping08:03
Chipacahad a very lazy weekend (where 'lazy' includes 'personal best 10k woo' but w/e)08:04
mborzecki_Chipaca: congrats08:09
=== mborzecki_ is now known as mborzecki
Chipacapstolowski: on Friday you said maybe we should add a flag to Done() to say whether the script had run or not, and I said maybe instead just don't run things (and we add the flag if/when we need the "run Done even with no script" feature instead), but I didn't understand your answer -- and didn't want to slow down the standup further08:10
mborzeckiChipaca: don't tell mvo though :)08:10
Chipacamborzecki: about which part? =)08:11
mborzeckiChipaca: 10k personal best08:11
Chipacamborzecki: :-) I'm more competitive than he is, but we both go to great efforts not to be competitive with each other08:12
Chipacajust don't play board games with me and you'll be fine08:12
* Chipaca does not want to show that side of himself ever08:12
Chipacapstolowski: could you tell me why the "don't run it" option is worse?08:13
Chipacato me it seems like a win, there's a lot of stuff we'd avoid doing in the common case of 'no script'08:14
Chipaca(everything from the xxx on)08:14
pstolowskiChipaca: yes, sorry, I think I didn't understand your suggestion either. I think as long as we keep an option to run Done() even if there is no script, it's fine, that was my main point. Probably your idea is better to save some cycles indeed08:16
Chipacapstolowski: I'll push a pr to add this, then08:17
Chipacawith a flag on the hook setup08:17
Chipacajust in case (tm)08:17
pstolowskiChipaca: thanks08:25
mupPR snapd#6862 opened: overlord/hookstate: don't run handler unless hooksup.Always <Created by chipaca> <https://github.com/snapcore/snapd/pull/6862>08:31
Chipacapstolowski: ^^^08:31
ackkhi, I'm seeing a weird behavior with python deps building a snap non non-intel machines10:07
ackkhttps://launchpadlibrarian.net/423189074/buildlog_snap_ubuntu_bionic_arm64_maas_BUILDING.txt.gz is failing because cffi pulled from pipy differs from the one in bionic, but https://launchpadlibrarian.net/423191401/buildlog_snap_ubuntu_bionic_amd64_maas_BUILDING.txt.gz doesn't10:08
ackkand they have the same versions10:08
=== ricab is now known as ricab|bbl
Chipacaackk: ports.u.c. differing from the main archive?10:20
Chipacaor is that what you mean about the same versions10:21
ackkChipaca, well it seems the python3-cffi-backend deb that get installed is the same version in both cases10:21
ackk1.11.5, whereas the one from pypi is 1.12.310:22
ackkso I'm not sure why it's failing in one case and not in the other10:22
ackkactually it fails on arm64 and ppc64 arches, works in i386 and amd6410:23
Chipacaackk: so the deb is the same version, but the one in pipy is different, for those arches?10:24
ackkChipaca, no, deb is 1.11.5 for all and pypi is 1.12.3 for all10:24
ackkbut it only complains that the versions are different on non-intel10:24
Chipacaackk: could it be because pipy does not have non-intel linux binaries of cffi?10:25
ackkChipaca, ah. that's a good point10:26
ackkindeed it seems so10:26
Chipacaackk: so maybe on non-intel it's trying to build it from source, and needs more stuff for that?10:26
* Chipaca does not know10:27
ackkChipaca, well I guess I can try to force cffi in requrements to match the ubuntu one10:27
Chipacazyga: 'sup11:08
pstolowskizyga: hey!11:11
pstolowskizyga: how was the travel? how is the sprint going?11:11
=== xnox_weechat is now known as xnox
zygapstolowski: the travel was very good, the sprint is going on slowly, nothing eventful yet11:33
* Chipaca lunches11:34
zygapstolowski: some meetings about core20 but early early discussions11:36
Saviqzyga: https://git.launchpad.net/~saviq/+git/miral-framework?h=master is my test - didn't work with edge FWIW11:51
mborzeckicmatsuoka: pstolowski: please take another look at #6849, i've pushed an implementation that matches what libblkid does12:09
mupPR #6849: many: introduce a gadget helper for locating device matching given structure <Gadget update> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6849>12:09
mborzecki(or systemd/udev for that matter)12:09
mborzeckioff to pick up the kids12:09
cmatsuokamborzecki: ack12:12
pstolowskimborzecki: k, will do after lunch12:17
=== ricab|bbl is now known as ricab
mborzeckicmatsuoka: pstolowski: thx!12:43
Chipacadegville: what would be better: "Talk to apt via an apt hook", "Run as an apt hook", "Run in 'apt hook' mode", or "Be an apt hook" ?12:55
zygamborzecki: quick one https://github.com/snapcore/snapd/pull/686312:58
mupPR #6863: testutil: support sharing-related mount flags <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6863>12:58
mupPR snapd#6863 opened: testutil: support sharing-related mount flags <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6863>12:58
degvilleChipaca: I think the second, or - 'via an apt hook'? (difficult without context). if we can link to something apt hook related, that would be good too.12:58
Chipacadegville: this is for 'snap advise-snap --from-apt', which is a bit backwards but also currently is wrong and also *also* currently upsets localisers because trying to fix it breaks our static lowercase-description rule12:59
=== joedborg_ is now known as joedborg
Chipacadegville: it's a command that's hidden (so nobody ever sees this string! until we fix go-flags) and the option is only used from integration scripts13:00
mupPR snapd#6861 closed:  packaging/fedora: Merge changes from Fedora Dist-Git and drop EOL Fedora releases <Created by Conan-Kudo> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6861>13:00
Chipacabut, we smacked an i18n.G on it to be conscientious about it or sth13:01
Chipacadegville: apt runs this as a hook for suggestions, basically13:02
Chipaca"Run under apt" could also work, not mention hooks at all13:03
GirtabluluSave data of snap "retroarch" in automatic snapshot set #27 (cannot create archive: runuser: failed to user credentials: <-- someone knows what's the issue is?13:03
degvilleChipaca: I'm just in our standup if you wanted to chat about it.13:03
Chipacawas just about to ask about the standup :)13:03
zygamborzecki: another one please https://github.com/snapcore/snapd/pull/686413:04
mupPR #6864: cmd/snap-update-ns: fix golint complaints about variable names <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6864>13:04
mupPR snapd#6864 opened: cmd/snap-update-ns: fix golint complaints about variable names <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6864>13:04
ChipacaGirtablulu: ouch13:06
ChipacaGirtablulu: is there anything interesting about our user setup?13:07
Girtabluludefine interesting? :), installment works fine just doesn't let me remove stuff :D13:08
ChipacaGirtablulu: anything other than the default /etc/passwd user13:08
Girtablulunope, not that I'm aware of, using Solus13:09
ChipacaGirtablulu: on remove snapd now tries to archive the user data before deleting it, and for some reason your system doesn't support that13:09
ChipacaGirtablulu: you'll probably get the same error from 'snap save'13:09
Girtabluluyep correct13:10
ChipacaGirtablulu: you can disable these snapshots, to unblock you; pstolowski will know how to do that (it's in the forum otherwise)13:10
ChipacaGirtablulu: but something's funny with your system fwiw13:10
Girtabluluk thanks for the info13:12
degvilleChipaca: if the way apt is probed doesn't matter to whoever is asking for help on advise-snap (would it help to know, for eg., when troubleshooting an integration script), I'd go with "Run under apt."13:17
Chipacadegville: the super-verbose one would be "This flag tells snap-advise that it is being run under apt, as a hook", or something like that13:18
degvilleChipaca: +1 - sounds good to me!13:19
Chipacadegville: but that's too long :-D13:19
Chipacabah,not really, but we usually get away with it being shorter13:20
Chipacathe "this flag tells <the command> that" is usually implied13:20
Chipacabut it's a bit weird, this one13:20
degvilleChipaca: right, sorry. Thought there was going to be both a short and verbose version.l13:20
Chipacaomg no :)13:21
* Chipaca suddenly imagined having to come up with short _and_ verbose versions of every flag help text13:21
degvilleChipaca: ahaha! "Run  under apt using a hook", "Run via an apt hook."?13:23
GirtabluluChipaca: thanks found the command13:23
ChipacaGirtablulu: snap set system somethingsomethingsomething?13:24
Chipacadegville: this is a flag apt uses to tell 'snap advise-snap' it is running under apt13:24
Girtablulusnap set core snapshots.automatic.retention=no13:24
ChipacaGirtablulu: that's what i said :-p (j/k, but note core and system are synonymous there)13:24
Girtabluluah k :)13:25
ChipacaGirtablulu: out of curiosity if you could explore why runuser doesn't work for you I'd be grateful13:25
ChipacaGirtablulu: if you can't you can't :)13:25
GirtabluluI gonna poke the solus cores devs regarding this, for me is this a bit confusing :)13:27
ChipacaGirtablulu: it's a new feature so it's entirely possible we're needing to do something extra -- but it might be some wonky integration on solus's side. I really don't know.13:27
ChipacaGirtablulu: you can quote me on that (especially the last sentence is always true)13:27
Girtabluluthat's why first poking solus devs :D13:28
Girtabluluquite possible that ikey build the snap package not correctly for solus and now it shows13:28
Chipacadegville: everything is terrible. I'm going with "Run as an apt hook" and hoping for the best.13:29
* Chipaca misses ikey13:29
degvilleChipaca: ok. :)13:29
Chipacamborzecki: fedora 30 not having snap/bin on path was fixed somewhere, right?13:36
mborzeckiChipaca: is it about the warning in snap install?13:37
Chipacamborzecki: ye s13:39
Chipacamborzecki: see https://forum.snapcraft.io/t/how-to-fix-snap-binaries-not-found/946913:39
Chipacalots of federees13:40
mborzeckiChipaca: iirc, plain sudo would not load the whole profile, so no PATH entry13:40
mborzeckiChipaca: sudo -i worked though13:40
mupPR snapd#6865 opened: cmd/snap: allow option descriptions to start with the command <Created by chipaca> <https://github.com/snapcore/snapd/pull/6865>13:40
zygamborzecki: https://github.com/snapcore/snapd/pull/686613:53
mupPR #6866: cmd/snap-update-ns: allow changing mount propagation <Created by zyga> <https://github.com/snapcore/snapd/pull/6866>13:53
zygaI'll review with jdstrand locally13:54
mupPR snapd#6863 closed: testutil: support sharing-related mount flags <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6863>13:54
mupPR snapd#6866 opened: cmd/snap-update-ns: allow changing mount propagation <Created by zyga> <https://github.com/snapcore/snapd/pull/6866>13:54
zygamborzecki: I force pushed because git add -p typo :P13:55
mupPR snapd#6858 closed: cmd/snap: unit tests for debug timings <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6858>14:34
Chipacalol of course the only test that failed is the connectivity check15:09
Chipacaogra: nethack 3.6.2 has been out for _ages_ and there's been no snap update! what is this! rabble rabble rabble15:14
ograChipaca, oh, geez ... thanks for the ping, will update it15:16
Chipacaogra: :-) i wasn't looking for it, i just noticed the news15:17
ograwhat is it that all of a sudden they make regular releases ? ...15:17
ograone revision per dacade should be enough for everyone !15:17
Chipacaogra: next up: nethack LTS15:17
Chipacapstolowski: is there anything that sets hooksetup.revision? I somehow got it in there at _some_ point which led me to believe it worked, but i can't reproduce that now15:28
pstolowskiChipaca: it seems to be only used in tests if looking for all references doesn't lie15:31
=== pstolowski is now known as pstolowski|afk
mupPR snapd#6864 closed: cmd/snap-update-ns: fix golint complaints about variable names <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6864>16:27
* cachio lunch16:45

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