| mup | PR snapd#8260 opened: tests: enable nested on core20 and test current branch <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/8260> | 04:52 |
|---|---|---|
| pstolowski | morning | 08:03 |
| mup | PR snapd#8257 closed: tests: backport master test fixes <Created by mvo5> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8257> | 08:46 |
| mup | PR snapd#8241 closed: interfaces: work around apparmor_parser slowness affecting uio <Bug> <Reviewed> <Squash-merge> <Created by zyga> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8241> | 08:50 |
| mup | PR snapd#8261 opened: interfaces: work around apparmor_parser slowness affecting uio (2.44) <Created by pedronis> <https://github.com/snapcore/snapd/pull/8261> | 08:55 |
| pedronis | pstolowski: hi, I asked a question in #8235 | 08:58 |
| mup | PR #8235: cmd/snap-preseed: handle --reset flag <Preseeding 🍞> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8235> | 08:58 |
| pstolowski | pedronis: yes, i saw it, thanks, will get to it soon | 08:59 |
| mup | PR snapd#8255 closed: cmd/snap: make the portal-info command search for the network-status interface <Reviewed> <Created by jhenstridge> <Merged by pedronis> <https://github.com/snapcore/snapd/pull/8255> | 09:13 |
| pedronis | pstolowski: can you look at #8248 if you have a moment, is not super urgent, OTOH is small | 09:20 |
| mup | PR #8248: snap: introduce Container.RandomAccessFile <Created by pedronis> <https://github.com/snapcore/snapd/pull/8248> | 09:20 |
| pstolowski | ok | 09:20 |
| pedronis | also it relates (indirectly) to UC20 needs | 09:21 |
| mup | PR snapd#8003 closed: o/ifacestate, api: implementation of snap disconnect --forget <Needs Samuele review> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/8003> | 09:24 |
| pedronis | pstolowski: thanks for landing that ^ | 09:26 |
| pstolowski | i'm equally happy about landing it finnaly | 09:27 |
| === epod is now known as luk3yx | ||
| pedronis | pstolowski: thansk for checking, we might do something also about fonts, but maybe we don't see changes because there are not fonts in the image? | 10:29 |
| pstolowski | pedronis: checking | 10:29 |
| pedronis | pstolowski: ah, no, on ubuntu the cache for those is in var (it's in usr for some other distros) | 10:36 |
| pstolowski | yes, that's right | 10:36 |
| pedronis | pstolowski: it's /var/cache/fontconfig but I'm not even sure we need to undo if we do something there, or whether we can | 10:36 |
| pedronis | it's a cache and it's not just touched by snapd | 10:37 |
| pstolowski | pedronis: that's right. i don't think re-running fc-cache makes any sense | 10:39 |
| pedronis | pstolowski: yea, agreed | 10:39 |
| pstolowski | / so yeah, spread test picked change to /usr.../completions | 10:39 |
| pedronis | yes, that we need to fix | 10:40 |
| pedronis | thanks for checking as I said | 10:40 |
| pstolowski | yep, on it | 10:40 |
| * zyga waves | 10:45 | |
| zyga | everyone at home but me has a stomach bug | 10:45 |
| zyga | I suspect I got "lucky" because first symptoms started before I returned from Fra | 10:45 |
| zyga | so I may be off early next week as well, depending on how the situation develops | 10:46 |
| pstolowski | zyga: ironic..\ | 10:46 |
| zyga | right? | 10:46 |
| alvesadrian | Hi, squashfs is read only. We are trying to make gluu snap package. As our previous experience showed that we may change any file in the snap for security or bug fixes. | 11:03 |
| alvesadrian | Currently we trying to figure out any file/directory that gluu services want to write. | 11:03 |
| alvesadrian | But this won't be enough for us. We may need to edit/replace any file inside the snap | 11:03 |
| alvesadrian | We are using containers and everything is writable, when we switch to snap it will be read only | 11:04 |
| zyga | alvesadrian: that's pretty hard to do | 11:04 |
| zyga | alvesadrian: can you just ship a new snap revision? | 11:04 |
| zyga | alvesadrian: everyone will update anyway | 11:04 |
| alvesadrian | snap reivsion? | 11:04 |
| alvesadrian | wharts that? | 11:04 |
| zyga | alvesadrian: if you make a snap you get a blob which is the squashfs filesystem | 11:04 |
| zyga | alvesadrian: instead of making changes to the files at runtime (application files, not data) you can just make another blob | 11:05 |
| zyga | alvesadrian: and put that to the store | 11:05 |
| zyga | alvesadrian: delta downloads should be fast and efficient | 11:05 |
| zyga | alvesadrian: so instead of shipping a snap with a self-update mechanism | 11:05 |
| zyga | alvesadrian: just upload subsequent revisions of the snap | 11:05 |
| alvesadrian | we still playing we dont have a full app ready | 11:06 |
| alvesadrian | we found this squashfs issue | 11:06 |
| alvesadrian | and we are not sure how to sort it | 11:06 |
| alvesadrian | for writeable files | 11:06 |
| zyga | alvesadrian: my advice: disable the self-update | 11:06 |
| alvesadrian | we dont even finish to build the app | 11:06 |
| zyga | alvesadrian: and let snapd handle that aspect | 11:06 |
| zyga | alvesadrian: how are you building the app? | 11:10 |
| alvesadrian | zyga snapcrft command | 11:10 |
| zyga | alvesadrian: when you said "we dont even finish to build the app" does that mean you still are in progress of working on the packaging? | 11:11 |
| alvesadrian | we are playing with parts of our big app | 11:12 |
| alvesadrian | is a java app | 11:12 |
| zyga | I see | 11:12 |
| alvesadrian | we are using parts to packafe it and see how it works | 11:12 |
| alvesadrian | just a war file | 11:12 |
| alvesadrian | with jetty | 11:12 |
| alvesadrian | @zyga one momment a coworker is comming online | 11:14 |
| alvesadrian | with more details | 11:14 |
| alvesadrian | zyga : ^^^^^ | 11:14 |
| zyga | alvesadrian: I'm off today, just helping while I'm in the office | 11:14 |
| alvesadrian | one moment | 11:14 |
| zyga | alvesadrian: I'm afraid I won't be around much longer | 11:14 |
| alvesadrian | please | 11:14 |
| zyga | sure, | 11:14 |
| zyga | just saying I can respond more slowly | 11:15 |
| alvesadrian | slvn_ mustafa is that u? | 11:16 |
| alvesadrian | damn | 11:17 |
| alvesadrian | zyga this is mbaser my co worker | 11:26 |
| alvesadrian | zyga he has some concerns | 11:26 |
| alvesadrian | zyga can u help us? | 11:26 |
| alvesadrian | zyga can he ask u a few things? | 11:27 |
| alvesadrian | zyga are you still around? | 11:27 |
| alvesadrian | mbaser drop ur concern to zyga | 11:27 |
| zyga | just ask the questions please | 11:29 |
| alvesadrian | mbaser are u reding us? | 11:30 |
| alvesadrian | reading us? | 11:30 |
| zyga | perhaps forum.snapcraft.io might be easier? | 11:30 |
| alvesadrian | zyga no worries we can back later | 11:32 |
| alvesadrian | zyga he is having some connection issues | 11:32 |
| zyga | sure | 11:32 |
| alvesadrian | thanks | 11:32 |
| zyga | I'm online 24/7 but not always reading | 11:32 |
| zyga | so try asking your questions and just check for answers later | 11:33 |
| zyga | the forum is easier to use | 11:33 |
| zyga | and better for asynchronous conversations | 11:33 |
| pedronis | pstolowski: I made a comment in #8217 | 12:42 |
| mup | PR #8217: o/devicestate: delay the creation of mark-seeded task until asserts are loaded <Bug> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8217> | 12:42 |
| pstolowski | pedronis: ty. updated #8235 | 12:42 |
| mup | PR #8235: cmd/snap-preseed: handle --reset flag <Preseeding 🍞> <Squash-merge> <Created by stolowski> <https://github.com/snapcore/snapd/pull/8235> | 12:42 |
| pedronis | pstolowski: thx, made a comment about the helper you are using there, it's actually a bit odd | 12:55 |
| pstolowski | pedronis: by that you mean to fix the helper to literally match on "snapd/complete.sh" right? | 12:59 |
| pedronis | pstolowski: to match that the end of target is like that, yes | 12:59 |
| pstolowski | ok, makes sense | 13:00 |
| pedronis | I mean /snapd/complete.sh | 13:00 |
| pedronis | really | 13:00 |
| pstolowski | right | 13:00 |
| pedronis | I don't know why the helper is like that | 13:00 |
| pedronis | is used in a couple of places | 13:00 |
| pedronis | but haven't closely at them | 13:00 |
| pstolowski | indeed, it's a bit naive | 13:00 |
| pedronis | but I looked at the helper in dirs that makes the targets | 13:00 |
| pedronis | and afaict they will always have snapd in them | 13:01 |
| mup | PR snapd#8248 closed: snap: introduce Container.RandomAccessFile <UC20> <Created by pedronis> <Merged by cmatsuoka> <https://github.com/snapcore/snapd/pull/8248> | 13:02 |
| pedronis | cmatsuoka: thank for merging that | 13:07 |
| cmatsuoka | pedronis: thanks for the patch, that should help moving secboot ahead | 13:08 |
| pedronis | we'll need it for our own cross checking bits as well I think | 13:08 |
| mup | PR snapcraft#2972 opened: [rfc] catkin plugins: remove bash workaround for old catkin bug <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2972> | 13:13 |
| mup | PR snapcraft#2970 closed: providers: match build provider flag keys to envvar <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2970> | 13:31 |
| pedronis | cmatsuoka: I made a maybe annoying comment in #8159, I'm just a bit unsure why we need more passes on things | 15:11 |
| mup | PR #8159: snap-bootstrap: remove created partitions on reinstall <UC20> <Created by cmatsuoka> <https://github.com/snapcore/snapd/pull/8159> | 15:11 |
| cmatsuoka | pedronis: thanks, I'll check that right after lunch | 15:13 |
| pedronis | cmatsuoka: let me know if you have questions | 15:13 |
| pstolowski | pedronis: search v2 is in production | 15:45 |
| mup | PR snapcraft#2973 opened: specifications: add core20 plugin specification <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/2973> | 16:01 |
| mup | PR snapd#8262 opened: interfaces/udisks2: also allow Introspection on /org/freedesktop/UDisks/** <Simple 😃> <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8262> | 16:06 |
| mup | PR snapd#8263 opened: interfaces/udisks2: also allow Introspection on /org/freedesktop/UDisks2/** - 2.44 <Created by jdstrand> <https://github.com/snapcore/snapd/pull/8263> | 16:43 |
| thresh | when I do snapcraft snap, would environment variables I set in my shell propagate to override-build: section in the snapcraft.yaml ? | 17:31 |
| ackk | thresh, well if you use override-build you can just export variables as part of that. otherwise you can use build-environment (see https://snapcraft.io/docs/parts-environment-variables) | 17:36 |
| thresh | ackk, I probably need to add some more context. I export some variables in the CI that I would like to check when I'm doing snapcraft snap (inside the yaml, so inside the override-build). | 17:37 |
| thresh | I guess my question is "does snapcraft snap sanitize env(1) when doing the builds?" | 17:38 |
| ackk | thresh, AFAIK it does | 17:39 |
| thresh | there's only one way to check. (which I'm doing right now) :-) | 17:39 |
| thresh | ok, I think it actually works and the variable is there | 17:53 |
| thresh | as in I can see it in "env" output when doing the build | 17:53 |
| sergiusens | thresh: if you are in --destructive-mode or not using bases, the env should make it through | 18:21 |
| mup | PR snapcraft#2973 closed: specifications: add core20 plugin specification <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/2973> | 19:08 |
| mup | PR snapcraft#2960 closed: specification: base plugin and plugins for core20 <Created by sergiusens> <Closed by sergiusens> <https://github.com/snapcore/snapcraft/pull/2960> | 19:26 |
| thresh | sergiusens, I'm not using destructive mode, and my snapcraft.yaml has base: core18; env makes it through | 19:38 |
| thresh | so, eh. I'm on snapcraft 2.43.1+18.04 (from .deb) | 19:39 |
| sergiusens | thresh: oh, so no real support for bases then, environment should just passthrough | 20:12 |
| thresh | sergiusens, will it change when I upgrade? | 20:13 |
| thresh | I'm building inside (a somewhat outdated) 18.04 non-privileged docker container. | 20:13 |
| mup | PR snapcraft#2974 opened: project_loader: use -isystem instead of -I for system include paths <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/2974> | 21:14 |
| sergiusens | thresh: if you are not using a base (in snapcraft.yaml), then I would recommend 16.04 as the container | 21:47 |
| sergiusens | thresh: this might help with what bases are https://snapcraft.io/docs/base-snaps (but you need snapcraft 3.x) | 21:48 |
| sergiusens | thresh: and docs on snapcraft 3.x with docker https://snapcraft.io/docs/build-on-docker | 21:49 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!