| mup | PR snapcraft#2877 closed: [3.9 backport] meta: handle plug & slot string objects <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2877> | 00:46 |
|---|---|---|
| mup | PR snapcraft#2879 closed: [3.9-backport] meta: ensure Snap's `assumes` is initialized as a set <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2879> | 00:46 |
| mup | PR snapcraft#2881 opened: meta: fix missing provider case for get_provider_content_directories() <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2881> | 02:43 |
| mborzecki | morning | 06:29 |
| mvo | hey mborzecki | 06:32 |
| mborzecki | mvo: hey | 06:33 |
| mup | PR snapd#7998 closed: httputil: use shorter timeout in TestRetryRequestTimeoutHandling <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/7998> | 06:40 |
| zyga-laptop | o/ | 06:52 |
| zyga-laptop | mborzecki hey | 06:52 |
| zyga-laptop | I will be back shortly, the office is still super cold after the night | 06:52 |
| mborzecki | zyga-laptop: hey | 06:52 |
| zyga-laptop | I need to catch up with things from yesterday | 07:06 |
| zyga | oh, iCloud is back! | 07:08 |
| zyga | ircloud | 07:08 |
| * zyga builds 2.43.1 update | 07:17 | |
| zyga | mborzecki: we have a new dependency on Suse? | 07:24 |
| zyga | we need fake root for tests | 07:24 |
| mborzecki | zyga: hm? | 07:24 |
| zyga | mborzecki: without fake root dependency | 07:26 |
| zyga | https://www.irccloud.com/pastebin/08hHDzfx/ | 07:26 |
| zyga | I'll send a patch shortly | 07:26 |
| zyga | https://github.com/snapcore/snapd/pull/8009 | 07:28 |
| mborzecki | zyga: yeah, we shuld porbably skip those tests | 07:28 |
| mup | PR #8009: packaging/opensuse: build-depend on fakeroot <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/8009> | 07:29 |
| mup | PR snapd#8009 opened: packaging/opensuse: build-depend on fakeroot <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/8009> | 07:29 |
| mborzecki | zyga: otoh, fakeroot is mocked in those tests, the test will fail if it's called and not mocked by some useful code | 07:31 |
| zyga | hmm? | 07:32 |
| zyga | what? | 07:32 |
| mborzecki | zyga: 2 lines up from the first point it failed | 07:32 |
| mborzecki | zyga: https://github.com/snapcore/snapd/blob/master/gadget/mkfs_test.go#L60 | 07:33 |
| zyga | dunno | 07:34 |
| mborzecki | zyga: was this on obs? | 07:35 |
| zyga | no, in "osc build" | 07:36 |
| mborzecki | zyga: can you run those tests with -check.work and inspect the test dir afterwards? | 07:37 |
| zyga | mborzecki: in osc build or manually? | 07:37 |
| zyga | (in either case, later, updating TW now) | 07:39 |
| zyga | I'll grab some food, see you after breakfast | 07:46 |
| pstolowski | morning | 08:02 |
| mborzecki | mvo: side note, does it make sense to start console-conf in install mode at all? | 08:02 |
| mvo | mborzecki: it does not | 08:09 |
| mvo | pstolowski: good morning | 08:09 |
| mborzecki | pstolowski: hey ;) | 08:09 |
| pstolowski | o/ | 08:10 |
| sdhd-sascha | good morning, | 08:14 |
| sdhd-sascha | zyga: yesterday, i solved it. i found in glibc, that dlvsym is like dlsym with version equal null. So i add a PR for snapcraft-preload. And it works for sway :-) | 08:14 |
| mvo | mborzecki: do you remember if we had a forum top to the parallel seccomp setup? | 08:19 |
| mborzecki | mvo: let me check | 08:21 |
| mvo | mborzecki: no need to create it if it's not there, I'm just updating the snapd roadmap with interessting bits of 2.43 | 08:22 |
| mborzecki | mvo: no, i don't think we had a topic for that | 08:22 |
| mvo | that's fine, thanks for checking | 08:23 |
| mborzecki | hm wonder whether we could build an image with systemd.debug-shell=1 out of the box | 08:24 |
| zyga | back now | 08:41 |
| zyga | sdhd-sascha: cool, I'm glad you solved it :) | 08:41 |
| zyga | hey mvo :) | 08:41 |
| zyga | hey pawel :) | 08:41 |
| sdhd-sascha | zyga: thank you. Just talk about the problem with you, was a great help to choose a solution ;-) | 08:42 |
| zyga | mvo: I published the render code and will try to sync with chipaca about it | 08:42 |
| zyga | mvo: I need to send some tweaks that I thought about while sleeping but it's mostly enough what I need for explain | 08:42 |
| zyga | mvo: I published 2.43.1 in suse, shall I do fedora/debian before jumping into regular PR work? | 08:43 |
| Chipaca | zyga: your implementation of RuneWidth is rather optimistic :) | 08:44 |
| zyga | Chipaca: I was sure you'd be interested in that one :) | 08:44 |
| zyga | Chipaca: yeah, that's why it is in the heuristic package | 08:45 |
| zyga | Chipaca: happy to take feedback on every part | 08:45 |
| zyga | Chipaca: if you are reading it already, the two changes I was thinking about are: relative positioning (d'oh, why did I do absolute) and changing Pre widget to Paragraph that I also implemented but left out - paragraph does text folding | 08:46 |
| zyga | I think it's just generally better than a fixed text thing | 08:47 |
| pstolowski | hey zyga | 08:47 |
| zyga | hey pawel :) | 08:47 |
| mvo | hey zyga, good morning | 08:47 |
| Chipaca | zyga: does it support truncation / elision of text? | 08:47 |
| zyga | Chipaca: no, it supports folding into available width - the whole design of this is like printer paper | 08:48 |
| zyga | you have a fixed with | 08:48 |
| zyga | but all the height you want | 08:48 |
| zyga | Chipaca: it supports clipping though, so you can force it to occupy some space | 08:48 |
| Chipaca | zyga: thinking of things we are doing that would benefit from something like this, find and list come directly to mind | 08:48 |
| Chipaca | but both of them require elision | 08:48 |
| zyga | Chipaca: I think the thing you should look at is the signature of Widget.Pack | 08:49 |
| mvo | zyga: release> not super urgent to release 2.43.1, I think it's fine to release when it hits our candidate channel | 08:49 |
| zyga | if we give it information about what the constraints are (currently it's just the remaining width) we could do what you just described | 08:49 |
| zyga | mvo: ok | 08:49 |
| zyga | Chipaca: do you have time now? we could jump into a HO and talk about this | 08:52 |
| Chipaca | zyga: I'm having breakfast, now :) | 08:52 |
| zyga | Chipaca: ah, sorry | 08:52 |
| Chipaca | zyga: for 'snap list' one problem you have is that almost every field needs shrinking, and some are more 'important' than others | 08:53 |
| zyga | indeed, I was thinking about tabular widget | 08:53 |
| Chipaca | doing that in a generic way is doable, but you might need to be knuth | 08:53 |
| zyga | right now vbox and hbox are insufficient to perfectly replicate a table | 08:53 |
| zyga | Chipaca: heh, I remember reading that paper a few years ago - about table layout problems | 08:54 |
| zyga | it's a surprisingly hard problem | 08:54 |
| mup | PR snapd#8010 opened: snap-bootstrap: add support for "recover" mode <UC20> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8010> | 08:54 |
| Chipaca | zyga: the good news is fixed-width makes it a little simpler :) | 08:55 |
| Chipaca | the bad news is double-width characters make it hard again | 08:55 |
| mup | Bug #1655376 changed: Add support for android/touch type images <personal> <Canonical System Image:Invalid by pat-mcgowan> <Snappy:Won't Fix by ogra> <https://launchpad.net/bugs/1655376> | 09:09 |
| mup | PR snapd#7997 closed: overlord: increase settle timeout for slow machines <Created by mvo5> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/7997> | 09:22 |
| mborzecki | zyga: heh, pstolowski correctly spotted that those tests are in snap-bootstrap | 09:37 |
| zyga | mborzecki: hmm? | 09:38 |
| mborzecki | zyga: https://github.com/snapcore/snapd/pull/8009#pullrequestreview-343774644 | 09:38 |
| mup | PR #8009: packaging/opensuse: build-depend on fakeroot <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/8009> | 09:38 |
| zyga | pstolowski: thanks pawel! | 09:38 |
| sdhd-sascha | What is you opinion. What should happened if a symlink couldn't created. I think the snap shouldn't abort installation. | 09:39 |
| sdhd-sascha | `cannot update snap namespace: cannot create symlink in "/etc/sway": existing file in the way` | 09:39 |
| zyga | hmm | 09:39 |
| zyga | so do we use fakeroot in the real code? | 09:39 |
| zyga | sdhd-sascha: layouts are required, if it cannot be created and there are install hooks or configure hooks then installation is aborted | 09:39 |
| pstolowski | yw, good! | 09:40 |
| sdhd-sascha | I have to handle two case's. Case 1: no desktop packages are installed on the host. Case 2: a desktop installation exists on the host. | 09:40 |
| sdhd-sascha | E. g. openGl drivers are on the host. Then the snap don't need to deliver them. | 09:41 |
| sdhd-sascha | ... | 09:41 |
| sdhd-sascha | Would it make sense, to add "try-symlink" to layouts ? | 09:42 |
| sdhd-sascha | And/Or "try-bind" mount | 09:43 |
| mborzecki | zyga: yes, we do, but that's relevant only for uc20 atm | 09:44 |
| mborzecki | zyga: got a fix, can open a PR | 09:46 |
| mborzecki | it's a single patch, so you should be able to cherry pick it without trouble | 09:46 |
| zyga | mborzecki: cool, thank you! | 09:46 |
| zyga | sdhd-sascha: I don't know, can you give me an example please? | 09:47 |
| zyga | sdhd-sascha: but it feels like unrelated to layouts | 09:47 |
| zyga | sdhd-sascha: layouts cannot grab data from the host | 09:47 |
| zyga | sdhd-sascha: what you are describing seems more like interface connections | 09:48 |
| sdhd-sascha | zyga: currently i used this: | 09:50 |
| sdhd-sascha | layout: | 09:50 |
| sdhd-sascha | /etc/sway: | 09:50 |
| sdhd-sascha | symlink: $SNAP/etc/sway | 09:50 |
| mup | PR snapd#8009 closed: packaging/opensuse: build-depend on fakeroot <Simple 😃> <Created by zyga> <Closed by bboozzoo> <https://github.com/snapcore/snapd/pull/8009> | 09:50 |
| mup | PR snapd#8011 opened: cmd/snap-boostrap: add mocking for fakeroot <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8011> | 09:50 |
| sdhd-sascha | zyga: but this is not urgent. | 09:50 |
| mborzecki | mvo: posted a little guide for working with vm cloud images (incl. uc20) and feedimg some quick setup via cloud-init https://gist.github.com/bboozzoo/a1303a1e4deb516c2ad9a1e64835c341 | 09:51 |
| sdhd-sascha | i had another case in the past. But can't remember, so i can give more examples in the next days. Current i test, sway on both case's. | 09:51 |
| zyga | mborzecki: maybe mention the gist on the forum | 09:52 |
| zyga | mborzecki: it will be easier to find | 09:52 |
| mvo | mborzecki: nice! | 09:53 |
| mup | PR snapd#8012 opened: o/devicestate: do not create perfTimings if not needed inside ensureSeed/Operational <Simple 😃> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8012> | 09:54 |
| mvo | mborzecki: did you end up with network in the qemu vm? | 09:56 |
| pstolowski | Chipaca: hey, conflict in #8004 | 09:56 |
| mup | PR #8004: cmd/snap: print full channel in 'snap list', 'snap info' <Created by chipaca> <https://github.com/snapcore/snapd/pull/8004> | 09:56 |
| mvo | mborzecki: I remember there was an issue yesterday? | 09:56 |
| mborzecki | mvo: we got it resolved once the system booted into the run mode and kernel module were mounted at the right location | 09:57 |
| pstolowski | pedronis: boot speed ho? | 10:01 |
| mup | PR snapcraft#2881 closed: [backport] meta: fix missing provider case for get_provider_content_directories() <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2881> | 10:03 |
| zyga | brb | 10:13 |
| zyga | re | 10:22 |
| Chipaca | going to take a break in a few minutes. Nothing particularly noteworthy in triage today. | 10:24 |
| mup | PR snapd#8002 closed: tests/core18/snapd-failover: collect more debug info <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8002> | 10:28 |
| mborzecki | mvo: this will help a little https://i.imgur.com/86JdDRU.png | 10:41 |
| mborzecki | not sure why the quantity got badly formatter, maybe it's not bytes (?) | 10:42 |
| mborzecki | heh, the conversion is wrong ;) | 10:44 |
| mvo | mborzecki: nice! | 10:45 |
| mvo | mborzecki: you rock | 10:45 |
| mborzecki | btw. u-i should probably create an image large enough to fit the layout too | 10:45 |
| mvo | mborzecki: yes, probably something for sil2100 to make ubuntu-image smarter so that it knows how much space to reserve when building a uc20 image (right now it creates an image that is too smalll to fit the partitions) | 10:46 |
| * Chipaca afk | 10:46 | |
| pedronis | mvo: I'm looking at #7992, yes I think the issue is that all the fixture code is recovery oriented but shouldn't for this new stuff | 10:52 |
| mup | PR #7992: bootloader: add ExtractedRunKernelImageBootloader interface, implement in grub <UC20> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/7992> | 10:52 |
| pedronis | mvo: I'll see if I can untangle that a bit | 10:53 |
| mvo | pedronis: nice, thank you! | 11:00 |
| zyga | Chipaca: hey, is this a good time? | 11:23 |
| pedronis | mvo: done, but there is probabably a bit more to do there | 11:30 |
| mborzecki | Chipaca: there's no code for format MiB/GiB quantities in the tree, is there? | 11:32 |
| zyga | pedronis: I'd like to merge https://github.com/snapcore/snapd/pull/7875 as-is | 11:33 |
| mup | PR #7875: interfaces: refactor path() from raw-volume into utils with comments for old <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7875> | 11:33 |
| zyga | do you think it's possible to get your review today? | 11:34 |
| * zyga jumps to another PR | 11:39 | |
| pstolowski | ijohnson: hi, thanks for the insight into the root cause of https://github.com/snapcore/snapd/pull/7871 failures! i got it working (i don't know yet if it passes on all systems, waiting for travis, but as far as this denial is concerned, it should be fine now) | 11:53 |
| mup | PR #7871: tests: add spread test for hook permissions <â›” Blocked> <Created by stolowski> <https://github.com/snapcore/snapd/pull/7871> | 11:53 |
| popey | I have asked this before, sorry. When are we getting the "don't update me please, I'm running" for applications? | 11:56 |
| pstolowski | i think zyga worked on this ^ | 11:57 |
| zyga | popey: it's stuck in review for ... months? | 11:59 |
| zyga | popey: I just resolved a merge conflict https://github.com/snapcore/snapd/pull/7825 | 12:00 |
| mup | PR #7825: many: use transient scope for tracking apps and hooks <Created by zyga> <https://github.com/snapcore/snapd/pull/7825> | 12:00 |
| mup | PR snapd#8013 opened: cmd/snap-bootstrap: check device size before boostrapping and produce a meaningful error <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/8013> | 12:00 |
| pstolowski | zyga: i'll take a look at it later today | 12:01 |
| zyga | pstolowski: super, thanks! | 12:01 |
| zyga | I'll push a small tweak for older merge breaking after test rename | 12:01 |
| mup | PR snapd#8011 closed: cmd/snap-boostrap: add mocking for fakeroot <Simple 😃> <Created by bboozzoo> <Merged by bboozzoo> <https://github.com/snapcore/snapd/pull/8011> | 12:15 |
| mborzecki | zyga: you can cherry-pick the patch now ^^ | 12:16 |
| zyga | mborzecki: I released a variant with fakeroot build-dep | 12:16 |
| zyga | mborzecki: I'll get this for 2.44 :) | 12:16 |
| mborzecki | ah, ok | 12:16 |
| zyga | mborzecki: or I can do a -2 if there's some other things to cherry pick | 12:16 |
| * pstolowski lunch+errand, bbl | 12:19 | |
| ogra | zyga, should all layouts be set up when i use snap run --shell for a daemon ? | 12:25 |
| zyga | ogra: yes | 12:26 |
| ogra | hmm, weird | 12:26 |
| ogra | ls prime/etc/dbus-1/system.d/ | 12:27 |
| ogra | org.freedesktop.DisplayManager.conf | 12:27 |
| ogra | root@localhost:/home/ogra# ls /etc/dbus-1/system.d/ | 12:27 |
| ogra | wpa_supplicant.conf | 12:27 |
| ogra | the above is from the packaging dir ... the below one from snap run --shell ... i have a bind layout defined for /etc/dbus-1 | 12:27 |
| ogra | i dont get any error when installing or anything ... so i'd expect to find that org.freedesktop.DisplayManager.conf file somehow ... | 12:28 |
| mup | Bug #1859974 opened: failing to release devmode revisions to edge/beta channels <Snappy:New> <https://launchpad.net/bugs/1859974> | 12:28 |
| zyga | ogra: can you show me the relevant parts of snap.yaml please | 12:30 |
| * zyga jumps to setgid branch | 12:31 | |
| zyga | looking at those failures now | 12:31 |
| sil2100 | mvo, mborzecki: on it! (soon) | 12:36 |
| mvo | sil2100: thank you | 12:36 |
| mvo | sil2100: not a blocker but certainly will improve the experience :) (all of us ran into this problem) | 12:36 |
| sil2100 | mvo: yeah, when I switched to the 'cleaner' way of handling system-seed, I guess I forgot about the image-size calculation | 12:38 |
| mborzecki | sil2100: thanks! | 12:42 |
| mborzecki | mvo: uc20 rite of passage | 12:42 |
| * zyga sighs at inability to stop spread | 12:43 | |
| mborzecki | zyga: ^\ doesn't work? | 12:48 |
| mborzecki | zylop: i mean sigquit | 12:48 |
| mborzecki | zyga: ^ :) | 12:49 |
| zyga | mborzecki: yeah but that leaves garbage | 12:51 |
| mborzecki | zyga: you need to spread -discard -spread-pid=<..> later | 12:51 |
| zyga | yeah but ... why do I need to :/ | 12:51 |
| mborzecki | cmatsuoka: mvo: do you have any notes on runnnig qemu with tpm? | 12:55 |
| === ricab is now known as ricab|lunch | ||
| cmatsuoka | mborzecki: once you have the tpm simulator installed, you must run qemu the the appropriate parameters | 13:14 |
| cmatsuoka | mborzecki: see them here: https://github.com/snapcore/spike-tools/blob/master/run-test.sh | 13:14 |
| mborzecki | cmatsuoka: thanks | 13:14 |
| cmatsuoka | mborzecki: it's the line with tpms in it | 13:14 |
| zyga | brb | 13:56 |
| ijohnson | morning folks | 13:56 |
| ackk | hi, I'm seeing an issue when using snapcraft candidate. it seems the command-*.wrapper files are not being created. did something change there? | 13:56 |
| Chipaca | sergiusens: ^ | 13:58 |
| ackk | oh I'm not on the latest candidate, I guess it just got udpated | 14:00 |
| zyga | degville: please ping me when you look at the content interface docs, there's some rarely known features there | 14:13 |
| zyga | s/'s/are | 14:13 |
| mup | Bug #1859974 changed: failing to release devmode revisions to edge/beta channels <Snap Store:New> <https://launchpad.net/bugs/1859974> | 14:14 |
| degville | zyga: will do, thank you! | 14:14 |
| sergiusens | ackk: wrappers are now gone, instead what used to be in the wrapper is now part of the command-chain, this allows for "snap run --shell <snap.app>" to have the right command from the get go | 14:24 |
| === ricab|lunch is now known as ricab | ||
| mup | PR pc-amd64-gadget#33 closed: grub.cfg-boot: use $prefix/kernel instead of $ubuntu-boot/kernel <Created by anonymouse64> <Merged by xnox> <https://github.com/snapcore/pc-amd64-gadget/pull/33> | 14:25 |
| ackk | sergiusens, is there a way from inside the snap to run stuff as if it were from the outside? we currently call the wrapper script explicitly since we need to call stuff under snapcraft-runner for the LD_PRELOAD/PATH exports | 14:26 |
| ackk | sergiusens, I guess we can wrap everything with our wrapper otherwise | 14:26 |
| mup | PR core20#18 closed: static: try using /run/mnt/snapd first in run-snapd-from-snap <Created by mvo5> <Merged by xnox> <https://github.com/snapcore/core20/pull/18> | 14:26 |
| sergiusens | ackk: hmm, that wrapper was always an implementation detail, that said, "$snap/command-chain/snapcraft-runner" will have most of what you need | 14:27 |
| mup | PR core20#17 closed: Drop encoding digits in units and code paths. <Created by xnox> <Merged by xnox> <https://github.com/snapcore/core20/pull/17> | 14:28 |
| ackk | sergiusens, I see, thanks | 14:28 |
| ackk | sergiusens, is snapcraft-runner used both for apps and daemons? | 14:30 |
| ackk | ah yeah it is, cool | 14:31 |
| * zyga goes to grab lunch | 14:32 | |
| === mborzeck1 is now known as mborzecki | ||
| sdhd-sascha | sil2100: hi. Is there a tool to use, to generate the changelog entry's for ubuntu-image ? | 14:37 |
| ijohnson | pedronis: thanks for the patches to 7992, I guess I never quite understood what you meant by NoSlashBoot option when you mentioned it in our meetings but I see what you meant now. | 14:43 |
| pedronis | np | 14:44 |
| mup | PR snapd#8007 closed: tests: remove execution of ubuntu 19.04 from google backend <Simple 😃> <Created by sergiocazzolato> <Merged by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8007> | 14:48 |
| mup | PR snapd#8014 opened: tests: run `uc20-snap-recovery-encrypt` test on 20.04-64 as well <Simple 😃> <Created by mvo5> <https://github.com/snapcore/snapd/pull/8014> | 14:52 |
| cachio | mvo, pedronis should I enable ubuntu 20.04 tests? | 14:53 |
| cachio | as 19.04 is gone | 14:53 |
| cachio | I already have an image | 14:53 |
| cachio | or should I make it once I am back from my vacactions | 14:54 |
| cachio | ? | 14:54 |
| sil2100 | sdhd-sascha: hey! I usually just add one with dch for that - but I can also create those for you, as long as everything else is good to go | 14:58 |
| Girtablulu | when I try to start snap-store voa terminal, I get no protocol specified and Unable to init server: Could not connect: Connection refused - what's the issue? | 15:00 |
| sdhd-sascha | sil2100: i cleaned up, rootfs_size ( 182 ) and added one test. | 15:02 |
| Chipaca | Girtablulu: and if you try to launch it in a different way? | 15:07 |
| * cachio lunch | 15:07 | |
| Girtablulu | Start menu and terminal both dont work | 15:08 |
| Chipaca | Girtablulu: sounds like a bug to me | 15:08 |
| Girtablulu | mmmh okay | 15:10 |
| roadmr | Girtablulu: https://launchpad.net/snap-store to report it maybe | 15:11 |
| Girtablulu | yea will do | 15:11 |
| mvo | cachio: yeah, I think we should enable 20.04-64, maybe to unstable at first | 15:11 |
| Chipaca | Girtablulu: please include the output of 'snap version' there | 15:13 |
| Girtablulu | will do | 15:13 |
| cachio | mvo, sure, I'll add this after lunch | 15:13 |
| Chipaca | Girtablulu: thanks | 15:20 |
| zyga | fd | 16:13 |
| zyga | re :) | 16:13 |
| zyga | mvo: can we merge https://github.com/snapcore/snapd/pull/7238 ? | 16:17 |
| mup | PR #7238: usersession: add systemd user instance service control to user session agent <Needs Samuele review> <Security-High> <Created by jhenstridge> <https://github.com/snapcore/snapd/pull/7238> | 16:17 |
| zyga | mvo: +3 and green | 16:18 |
| zyga | I can merge master into it to ensure tests passed recentlty | 16:18 |
| zyga | *recently | 16:18 |
| pedronis | zyga: it can be merged, but yes best to merge master first | 16:24 |
| zyga | ok | 16:24 |
| zyga | jd | 16:25 |
| mvo | zyga: looks like samuele and jamie are happy so I see no reason not to merge it | 16:33 |
| zyga | cool, I'll merge it after it passes one round of tests | 16:33 |
| mvo | ok | 16:33 |
| pstolowski | zyga: i did one pass over #7825, i'll continue tomorrow morning | 17:06 |
| mup | PR #7825: many: use transient scope for tracking apps and hooks <Created by zyga> <https://github.com/snapcore/snapd/pull/7825> | 17:06 |
| pstolowski | cachio, ijohnson can you take another look at #7871 if/when it gets green (just had to restart the job, it seemed to be stuck) | 17:08 |
| mup | PR #7871: tests: add spread test for hook permissions <Created by stolowski> <https://github.com/snapcore/snapd/pull/7871> | 17:08 |
| ijohnson | pstolowski: sure | 17:09 |
| pstolowski | thanks | 17:09 |
| pstolowski | eod, cu | 17:09 |
| mup | PR snapd#8015 opened: [RFC] daemon, store: support download resume from /v2/download <Created by mvo5> <https://github.com/snapcore/snapd/pull/8015> | 17:37 |
| pedronis | zyga: I pushed a tweak to #7875 along the last comments, it can go in if it gets green | 18:04 |
| mup | PR #7875: interfaces: refactor path() from raw-volume into utils with comments for old <Created by jdstrand> <https://github.com/snapcore/snapd/pull/7875> | 18:04 |
| mup | PR snapcraft#2882 opened: extensions: change extension merge-strategy to fix build-environment <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2882> | 19:09 |
| === blackboxsw_ is now known as blackboxsw | ||
| mup | PR snapd#8016 opened: gitignore: ignore snap files <Simple 😃> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/8016> | 20:43 |
| mup | PR snapd#8004 closed: cmd/snap: print full channel in 'snap list', 'snap info' <Created by chipaca> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/8004> | 20:51 |
| cjp256 | on a 16.04 system using the snapd deb (2.34.2), should one be able to just run `sudo snap install snapd` to get the latest? | 20:57 |
| cjp256 | or would there other steps to take? | 20:58 |
| mup | PR snapd#8017 opened: tests: add ubuntu 20.04 to the tests execution and remove tumbleweed from unstable <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8017> | 21:05 |
| ijohnson | cjp256: I'm not sure about 16.04, but I know on 18.04 if you don't have snaps, it will use the snapd snap when you first install a snap | 21:09 |
| ijohnson | cjp256: if you wanted to get the latest on 16.04 I'd just say do `sudo snap refresh core` | 21:10 |
| ijohnson | cjp256: though I did just check on 16.04 and it should be fine to install snapd like you have it | 21:13 |
| cjp256 | thanks ijohnson. i did a similar test and was OK, but recalled a conversation between sergiusens and Chipaca about some flag I can't recall, wanted to make sure it wasn't generally relevant :D | 21:17 |
| cjp256 | my test is on a fresh lxd image, so it's probably much newer base than theirs | 21:17 |
| ijohnson | cjp256: as long as you are doing this fresh without any snaps installed it is okay to install snapd, otherwise you would need to migrate from core -> snapd which does require a flag | 21:18 |
| ijohnson | err "okay to install the snapd snap" is what I meant | 21:18 |
| cjp256 | oh, what flag is that? | 21:18 |
| ijohnson | let me find it, it's an experimental flag | 21:18 |
| cjp256 | ijohnson: https://github.com/snapcore/snapcraft/blob/master/snapcraft/internal/build_providers/_snap.py#L332 ? | 21:20 |
| ijohnson | cjp256: ah yep there it is | 21:20 |
| cjp256 | tried searching for it earlier and couldn't find it because I was checked out on an old tag 🤦 | 21:21 |
| ijohnson | that's the flag you need to use if you have the core snap installed, and want to use the snapd snap instead | 21:21 |
| === Girtablulu is now known as Girtablulu_ | ||
| Chipaca | ijohnson: cjp256: snapcraft needs to work on the un-updated snapd that didn't know about the snapd snap | 22:33 |
| cjp256 | Chipaca: thanks! ijohnson gave me a lot of background info and I just confirmed running an old version of snapd (2.34.2): `error: cannot install "snapd": cannot install snapd snap on a model without a base snap yet` so the two options they have are `apt update; apt install snapd` or `sudo snap install core` to get working. | 22:38 |
| Chipaca | cjp256: that's why snapcraft sets an experimental flag | 22:39 |
| Chipaca | cjp256: but even that needs a new enough snapd | 22:39 |
| cjp256 | if eventually installing core snap anyways, is the apt update/upgrade wasting cycles? | 22:39 |
| Chipaca | cjp256: the experimental.snapd-snap=true | 22:39 |
| Chipaca | cjp256: I can't answer that question with enough confidence one way or the other | 22:40 |
| Chipaca | cjp256: it depends | 22:40 |
| Chipaca | cjp256: an update would bring in apparmor updates also hopefully, which would be good | 22:41 |
| Chipaca | but, given it's a throwaway, maybe it's fine as long as the snapd in the original deb understands that experimental flag | 22:41 |
| Chipaca | because at one point it didn't | 22:41 |
| cjp256 | yeah no dice on that flag: `Run configure hook of "core" snap (run hook "configure": cannot set "core.experimental.snapd-snap": unsupported system option)` | 22:41 |
| cjp256 | not with their version anyhow | 22:42 |
| Chipaca | so you had to install core first to get a new enough snapd to install snapd | 22:42 |
| Chipaca | and that always results in the conflict | 22:42 |
| Chipaca | (#1859469 is a conflict error) | 22:42 |
| mup | Bug #1859469: snapcraft lxd + snapd transition in progress <Snapcraft:Confirmed for sergiusens> <https://launchpad.net/bugs/1859469> | 22:42 |
| Chipaca | so, in that sense, if you use apt to refresh the packaged snapd, and that way get a new enough snapd that it doesn't need the flag (or at least understands the flag!), then it's slower but safe | 22:43 |
| Chipaca | OTOH it's a conflict, all you have to do is retry | 22:43 |
| cjp256 | gotcha, thanks! i'll just stick with the `apt` route and encourage them to update their image :D | 22:45 |
| ijohnson | Thanks Chipaca | 22:46 |
| Chipaca | at the same time, i need to discuss with pedronis about snap returning a particular return value on conflict | 22:46 |
| Chipaca | cjp256: i thought running on that particular image was a requirement, as it's what you got from some clouds and we wanted snapcraft to work well there | 22:47 |
| Chipaca | anyhoo, I shouldn't be on here :-) | 22:47 |
| Chipaca | but it's raining so i didn't take the dog for a walk | 22:48 |
| Chipaca | 👋 | 22:48 |
| ijohnson | cjp256: so did you get it all sorted out? | 22:48 |
| ijohnson | I'm still around for a bit if you need to discuss | 22:48 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
