| mup | PR snapd#6625 opened: tests: system disable ssh for config defaults in gadget <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6625> | 00:59 |
|---|---|---|
| mwhudson | is there a way via the snapd api to see if refreshing to a particular risk (in particular: including a branch) will perform an upgrade? | 01:01 |
| wgrant | mwhudson: "risk (in particular: including a branch)" is just a channel | 01:07 |
| mwhudson | wgrant: well ok my question still stands | 01:07 |
| wgrant | mwhudson: snap refresh --list SNAP --channel=CHANNEL should do it, but it's not supported | 01:07 |
| wgrant | Not sure why | 01:08 |
| wgrant | --list doesn't take the normal refresh args | 01:08 |
| mwhudson | it's a very different path inside snapd for whatever reason | 01:09 |
| mwhudson | you can say snap switch --channel=foo SNAP, snap refresh --list | 01:09 |
| wgrant | Ah, --list uses find | 01:09 |
| wgrant | That's weird | 01:09 |
| mwhudson | but that seems to silently succeed if you use a bogus channel | 01:09 |
| wgrant | Yeah, snap refresh --channel=foo SNAP checks that the target channel exists, but snap switch doesn't | 01:10 |
| wgrant | It doesn't look like snapd provides a sufficient API :( | 01:11 |
| mwhudson | heh switch to a missing channel and refresh --list does end up with a complaint in the logs | 01:12 |
| mwhudson | but i don't think i should be that evil :) | 01:12 |
| mwhudson | snap refresh --no-wait --channel=foo SNAP and assume i can abort the job in time? :) | 01:13 |
| wgrant | Heh | 01:15 |
| mborzecki | morning | 05:58 |
| === chihchun_afk is now known as chihchun | ||
| mborzecki | shall we land #6620? | 06:23 |
| mup | PR #6620: tests/main/remodel: clean up before reverting the state <Created by chipaca> <https://github.com/snapcore/snapd/pull/6620> | 06:23 |
| zyga | Good morning | 06:51 |
| zyga | Hey Maciej | 06:51 |
| zyga | mborzecki: I will likely have a slower day today | 06:52 |
| zyga | Looking at 6620 | 06:52 |
| mborzecki | zyga: hey hey | 06:52 |
| zyga | Yeah | 06:53 |
| zyga | Let’s land it | 06:53 |
| zyga | good morning mvo | 07:32 |
| zyga | I just merged the fix from chipaca | 07:32 |
| zyga | let's see some green today :-) | 07:32 |
| mup | PR snapd#6620 closed: tests/main/remodel: clean up before reverting the state <Created by chipaca> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6620> | 07:33 |
| mvo | hey zyga | 07:33 |
| zyga | I, selfishly restarted https://github.com/snapcore/snapd/pull/6622 - let's see if it goes green now | 07:33 |
| zyga | hey :) | 07:33 |
| mvo | zyga: thank you | 07:33 |
| mup | PR #6622: cmd/libsnap: rename C enum for feature flag <Simple 😃> <Created by zyga> <https://github.com/snapcore/snapd/pull/6622> | 07:33 |
| mvo | zyga: lets focus on the 2.38 tagged one(s) initially :) | 07:36 |
| zyga | I'll wait and see if anything turns green | 07:37 |
| zyga | mvo: https://github.com/snapcore/snapd/pull/6617 needs a review from You | 07:37 |
| mup | PR #6617: cmd/snap: fix regression of snap saved command <Squash-merge> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6617> | 07:37 |
| mup | PR snapd#6607 closed: cmd: typedef mountinfo structures <Simple 😃> <Created by zyga> <Merged by zyga> <https://github.com/snapcore/snapd/pull/6607> | 07:47 |
| === pstolowski|afk is now known as pstolowski | ||
| pstolowski | mornings | 08:02 |
| mvo | hey pstolowski | 08:06 |
| zyga | hey pawel :) | 08:16 |
| === chihchun is now known as chihchun_afk | ||
| === chihchun_afk is now known as chihchun | ||
| zyga | hmm | 08:35 |
| zyga | tests still red? | 08:35 |
| mup | PR snapd#6626 opened: strutil: make SplitUnit public, allow negative numbers <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6626> | 08:36 |
| mborzecki | should be a simple one ^^ | 08:36 |
| mborzecki | pstolowski: mvo: hey | 08:36 |
| mvo | mborzecki: good morning | 08:37 |
| zyga | mvo: https://github.com/snapcore/snapd/pull/6613 needs a 2nd review | 08:38 |
| mup | PR #6613: interfaces/builtin: add dev/pts/ptmx access to docker_support (for 2.38) <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/6613> | 08:38 |
| mvo | zyga: on it | 08:38 |
| zyga | mborzecki: is there anything that unicode.IsDigit returns true to that is not in range '0'..'9'? | 08:39 |
| zyga | like any funky numerals? | 08:39 |
| mborzecki | zyga: iirc that's what IsNumber could do | 08:40 |
| mborzecki | afaik IsDigit is 0-9 | 08:40 |
| zyga | mborzecki: https://golang.org/src/unicode/digit.go | 08:40 |
| zyga | mborzecki: https://golang.org/src/unicode/letter.go line 170 | 08:41 |
| zyga | mborzecki: I think it does more than you think it does :) | 08:41 |
| zyga | at least my naive understanding would suggest that there are more characters that match | 08:42 |
| zyga | can you run a simple loop over 65K runes and see what matches? | 08:42 |
| zyga | mborzecki: https://golang.org/src/unicode/tables.go line 2308 | 08:43 |
| zyga | there's a lot of digits | 08:43 |
| mborzecki | pfff | 08:43 |
| mborzecki | ৬ hello there little digit | 08:44 |
| zyga | whaat? what is that? | 08:45 |
| zyga | google says that is 6 | 08:45 |
| zyga | well | 08:45 |
| zyga | I think the branch needs some more changes | 08:45 |
| mborzecki | 'bengali digit one' | 08:45 |
| zyga | 一 | 08:46 |
| zyga | I wonder if that also matches | 08:46 |
| mborzecki | zyga: but it doesn't parse as a number | 08:46 |
| zyga | 二 | 08:46 |
| mborzecki | "߁kB" -> cannot parse \"߁kB\": \"\\xdf\" is not a number | 08:47 |
| zyga | hmm | 08:47 |
| zyga | that's not what I sent, perhaps IRC client mangles that | 08:47 |
| zyga | anyway :) | 08:48 |
| mup | PR snapd#6613 closed: interfaces/builtin: add dev/pts/ptmx access to docker_support (for 2.38) <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6613> | 08:50 |
| zyga | tests are still ungreen | 08:52 |
| zyga | fatal: unable to access 'https://github.com/snapcore/squashfuse/': gnutls_handshake() failed: Error in the pull function. | 08:52 |
| zyga | eh | 08:52 |
| zyga | hey chihchun | 09:02 |
| zyga | hey Chipaca | 09:02 |
| zyga | hey tab completion :/ | 09:03 |
| Chipaca | hey zygurat | 09:03 |
| Chipaca | :-p | 09:03 |
| Chipaca | zyga: 'sup? | 09:03 |
| zyga | Chipaca: it's a sunny day | 09:03 |
| zyga | that's always good | 09:03 |
| * Chipaca looks outside | 09:03 | |
| Chipaca | lies | 09:03 |
| Chipaca | zyga: mborzecki: i agree about the unicode.IsDigit, saw that the other day and added a todo to fix it | 09:07 |
| Chipaca | zyga: mborzecki: but in my case i was going to change it to return uints everywhere | 09:07 |
| Chipaca | (because we weren't using it from anywhere that needed negative sizes anyway) | 09:08 |
| Chipaca | mborzecki: but | 09:08 |
| Chipaca | mborzecki: we rely on the behaviour of never returning negative | 09:08 |
| Chipaca | ah but it now checks for that explicitly | 09:09 |
| Chipaca | ok :-) | 09:09 |
| mborzecki | hm have to fix the slicing too, otherwise it'll split up the unicode chars incorrecty | 09:13 |
| Chipaca | mborzecki: ? | 09:14 |
| mborzecki | Chipaca: just antoher bug in the PR that i see now | 09:14 |
| Chipaca | mborzecki: did you change the loop from a range over the string? | 09:15 |
| Chipaca | mborzecki: ranging over a string yields full runes | 09:15 |
| mup | PR snapd#6627 opened: devicestate: deal correctly with the "required" flag on Remodel <Created by mvo5> <https://github.com/snapcore/snapd/pull/6627> | 09:16 |
| Chipaca | mborzecki: https://play.golang.org/p/SUc1sdPpeUd fwiw | 09:20 |
| Chipaca | mborzecki: or https://play.jsgo.io/8e1f7704efa0f1ca794f16e88134281cf870fe93 :-p | 09:22 |
| * Chipaca stops having fun and gets to work | 09:23 | |
| mup | PR snapd#6628 opened: overlord/snapshotstate: support auto flag (automatic snapshots 1/N) <Created by stolowski> <https://github.com/snapcore/snapd/pull/6628> | 09:23 |
| mborzecki | jsgo.io | 09:24 |
| mborzecki | that's quite fancy | 09:24 |
| mborzecki | and 3rd party imports work too | 09:24 |
| Chipaca | mborzecki: they even tweaked the limits so we could import snapd in there | 09:25 |
| Chipaca | (i mentioned this when it happened) | 09:25 |
| mborzecki | Chipaca: zyga: updated the PR, please take a look | 09:25 |
| zyga | enqueued, doing another review now | 09:25 |
| mborzecki | zyga: #6605 needs master merge | 09:26 |
| mup | PR #6605: cmd/libsnap,osutil: fix parsing of mountinfo <Created by zyga> <https://github.com/snapcore/snapd/pull/6605> | 09:26 |
| pedronis | mvo: hi, could you re-review #6575 when you have some time | 09:27 |
| mup | PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575> | 09:27 |
| zyga | mborzecki: thank you, I will de-conflict that soon | 09:27 |
| mborzecki | need to grab some coffee | 09:27 |
| mborzecki | relative offsets in gadget.yaml are so fancy | 09:29 |
| Chipaca | mborzecki: I'd expect you to make a ParseByteSize analogue, ParseNonISOByteSize (or something), instead of exporting the helper, fwiw | 09:31 |
| Chipaca | mborzecki: but I don't mind (obviously) | 09:32 |
| * Chipaca not that worried | 09:32 | |
| * zyga could use a coffee too but first a quick merge with master | 09:35 | |
| mborzecki | Chipaca: would like that, but gadget takes sizes in format 1/1M/1G, SplitUnit is all i need | 09:35 |
| zyga | https://github.com/snapcore/snapd/pull/6614 needs a 2nd review now | 09:42 |
| mup | PR #6614: cmd/snap-confine: use fixed private tmp directory <Created by zyga> <https://github.com/snapcore/snapd/pull/6614> | 09:42 |
| zyga | mborzecki: conflict resolved, thanks | 09:42 |
| mborzecki | wodner how did google come up with 'stadia' as the name | 09:43 |
| zyga | mborzecki: https://www.dailymotion.com/video/x1afila | 09:45 |
| zyga | mborzecki: (seek to the end unless you know this one ;) | 09:47 |
| mborzecki | heh | 09:49 |
| * zyga looks at https://forum.snapcraft.io/t/problems-installing-a-base-core18-snap-on-bionic/10084/2 | 09:53 | |
| zyga | but first coffee, it's so cold in the office still | 09:53 |
| zyga | I need to print something again, it's always cozy when that thing is on | 09:53 |
| pedronis | Chipaca: can we have a chat about 6624, everything is more complicated than we remembered | 09:59 |
| Chipaca | pstolowski: #6628 includes #6617 ? | 10:01 |
| mup | PR #6628: overlord/snapshotstate: support auto flag (automatic snapshots 1/N) <Created by stolowski> <https://github.com/snapcore/snapd/pull/6628> | 10:01 |
| mup | PR #6617: cmd/snap: fix regression of snap saved command <Squash-merge> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6617> | 10:01 |
| Chipaca | pedronis: sure | 10:01 |
| pstolowski | Chipaca: no, i'm waiting for 6617 to become green, will merge and deconflict | 10:01 |
| Chipaca | pedronis: but if it really is, that means it isn't 2.38 material, so I vote we refactor now and fix it properly | 10:01 |
| pedronis | Chipaca: well the issue is not so much the change to autorefresh, is the store changes | 10:02 |
| pstolowski | and maybe i'm very optimistic hoping for green.. | 10:03 |
| Chipaca | pedronis: i'm listening | 10:03 |
| Chipaca | pstolowski: what's red now? | 10:03 |
| pedronis | Chipaca: can we do a quick HO? | 10:03 |
| Chipaca | pedronis: sure, in 5? | 10:03 |
| pedronis | Chipaca: works for me, thanks | 10:03 |
| pstolowski | Chipaca: just saying, still a couple of red PRs from there, maybe just need restarting | 10:05 |
| pstolowski | d/from/ | 10:05 |
| * Chipaca searches for his headset | 10:06 | |
| Chipaca | pedronis: ready when you are | 10:09 |
| pedronis | Chipaca: omw | 10:10 |
| mvo | pedronis: sure, will re-review. thank you | 10:16 |
| pstolowski | oh noes | 10:44 |
| pstolowski | Chipaca: you asked about what's red... #6617 went red | 10:45 |
| mup | PR #6617: cmd/snap: fix regression of snap saved command <Squash-merge> <⚠ Critical> <Created by stolowski> <https://github.com/snapcore/snapd/pull/6617> | 10:45 |
| Chipaca | fatal: unable to access 'https://go.googlesource.com/net/': Failed to connect to go.googlesource.com port 443: Connection timed out | 10:45 |
| Chipaca | pstolowski: ^ | 10:46 |
| Chipaca | lots of timeouts from there | 10:46 |
| Chipaca | pstolowski: restarted | 10:46 |
| Chipaca | pstolowski: thank you | 10:46 |
| Chipaca | pedronis: maybe i should've split the the backoff test, but it exercises the bits we want it to now | 10:57 |
| pedronis | Chipaca: if you think two tests make more sense, yes let's do that | 10:57 |
| Chipaca | pedronis: i think one is fine, even though the name could use a tweak | 10:58 |
| Chipaca | as it's not just about the backoff | 10:58 |
| Chipaca | but the comments cover what's going on | 10:58 |
| Chipaca | and they're all about backoff in one way or another, so maybe it's fine | 10:58 |
| pedronis | Chipaca: maybe it should have Error in the name? | 10:58 |
| pedronis | is is about errors after all | 10:59 |
| pstolowski | Chipaca: ty! | 11:00 |
| pedronis | zyga: I need to have lunch, we should probably have a chat about 6597, seems we are a bit talking across each other | 11:15 |
| mup | PR snapd#6622 closed: cmd/libsnap: rename C enum for feature flag <Simple 😃> <Created by zyga> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6622> | 11:24 |
| zyga | pedronis: gladly, I would prefer to do it before the standup if you can because I need to break early today | 11:25 |
| pedronis | zyga: 20 mins before the standup? | 11:44 |
| === chihchun is now known as chihchun_afk | ||
| mvo | zyga, pedronis quick question abut https://github.com/snapcore/snapd/pull/6575#discussion_r264631829 then I can continue with the review | 11:47 |
| mup | PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575> | 11:47 |
| mvo | zyga: but I'm puzzled by this one | 11:47 |
| * mvo probably miss something obvious | 11:47 | |
| zyga | mvo: looking | 11:51 |
| zyga | mvo: replied | 11:52 |
| zyga | pedronis: sounds good | 11:52 |
| pedronis | mvo: the move make the unit test linking fail, I suppose it's .h bit of the move | 11:54 |
| pedronis | s/linking/compiling/ | 11:54 |
| * mvo nods | 11:56 | |
| zyga | pedronis: we could move it again but I would have to rework snap-confine.c to effectively move main somewhere else | 11:59 |
| mup | PR snapd#6629 opened: overlord/snapshotstate: helpers for dealing with expiration times <Created by stolowski> <https://github.com/snapcore/snapd/pull/6629> | 12:10 |
| cachio | pedronis, hey, I see this error on the new test to disable ssh | 12:10 |
| cachio | https://paste.ubuntu.com/p/d9ckjJWMPk/ | 12:10 |
| pedronis | cachio: we don't mask ssh, we use a file instead | 12:11 |
| pedronis | mvo: ^ | 12:11 |
| cachio | pedronis, mvo which file? | 12:12 |
| pedronis | one sec | 12:13 |
| pedronis | cachio: /etc/ssh/sshd_not_to_be_run | 12:15 |
| cachio | pedronis, nice, thanks | 12:15 |
| pedronis | cachio: code is here: https://github.com/snapcore/snapd/blob/master/overlord/configstate/configcore/services.go#L59 | 12:15 |
| pedronis | we do unmask in case when (re)enabling, but our own disabling logic is based on that file | 12:16 |
| pedronis | there's a conditional in the unit | 12:16 |
| cachio | pedronis, reading, thanks for the info | 12:18 |
| mvo | pedronis, cachio its complicated, we used to "mask" but had bugs becaus of ssh/sshd aliases and the recommended way was to use this file | 12:20 |
| pstolowski | pedronis: re your comment to measurements PR about capturing task status, that's a good idea; does "task-status" tag sound ok? | 12:21 |
| mvo | zyga: reviewed, sorry for the delay, had some questions | 12:22 |
| pedronis | pstolowski: yes | 12:22 |
| cachio | mvo, it is ok, I with that code I can adjust the test case | 12:22 |
| zyga | mvo: no worries | 12:22 |
| pedronis | pstolowski: like task-kind | 12:22 |
| mvo | cachio: thanks | 12:22 |
| pstolowski | pedronis: ty | 12:27 |
| pstolowski | and now google:fedora-29-64:tests/main/degraded failed. fun. | 12:34 |
| * pstolowski lunch | 12:37 | |
| zyga | mvo: updated https://github.com/snapcore/snapd/pull/6575 - I left one thing out (cleanups memset 0) and I would prefer to do it in a separate branch | 12:44 |
| mup | PR #6575: cmd/snap-confine: pass sc_invocation instead of numerous args around <Created by zyga> <https://github.com/snapcore/snapd/pull/6575> | 12:44 |
| ddstreet | mvo what's your plans for lp #1819728? i have a couple/few systemd patches i'm planning to sru soon, i can include your patches if you think they will be ready this or next week | 12:48 |
| mup | Bug #1819728: Please backport "fix race between daemon-reload and other commands #8803" to 16.04 (for UC16) and 18.04 (for UC18) <patch> <verification-failed-bionic> <verification-failed-xenial> <verification-needed> <systemd (Ubuntu):Fix Released> <systemd (Ubuntu Xenial):Triaged> <systemd (Ubuntu | 12:48 |
| mup | Bionic):Fix Committed> <https://launchpad.net/bugs/1819728> | 12:48 |
| ddstreet | also i assume the current systemd bionic upload will be rejected due to failed verification, so you'll need to include lp #1778936 patch in the next upload as well | 12:49 |
| mup | Bug #1778936: please re-add Support-system-image-read-only-etc.patch <patch> <verification-needed> <verification-needed-bionic> <systemd (Ubuntu):Fix Released> <systemd (Ubuntu Bionic):Fix Committed> <systemd (Ubuntu Cosmic):Fix Released> <https://launchpad.net/bugs/1778936> | 12:49 |
| mborzecki | off to pick up the kids, may be a bit late to the standup | 13:12 |
| zyga | mborzecki: https://github.com/snapcore/snapd/pull/6626 is green | 13:21 |
| mup | PR #6626: strutil: make SplitUnit public, allow negative numbers <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6626> | 13:21 |
| === ricab is now known as ricab|lunch | ||
| mvo | ddstreet: that sounds reasonable, I should have things ready by then. its not at all straightforward unfortunately because its 239->229 and there are a bunch of changes which cause conflicts etc. plus its a delicate area of the code. but I think i have something ready | 13:48 |
| mvo | ddstreet: do you have a link to your things? just to double check that we don't conflict? | 13:48 |
| ddstreet | mvo so far we have lp #1812760 and lp #1818282 which we're getting patches ready for, but also 2 more issues without bugs yet that we're investigating | 13:51 |
| mup | Bug #1812760: networkd: [Route] PreferredSource not working in *.network files <sts> <systemd:Unknown> <systemd (Ubuntu):Fix Released by xnox> <systemd (Ubuntu Bionic):In | 13:51 |
| mup | Progress by ddstreet> <systemd (Ubuntu Cosmic):In Progress by ddstreet> <systemd (Ubuntu Disco):Fix Released by xnox> <https://launchpad.net/bugs/1812760> | 13:51 |
| mup | Bug #1818282: systemd-networkd - RoutingPolicyRule does not apply correctly <patch> <patch-accepted-upstream> <patch-forwarded-debian> <sts> <sts-sponsor> <sts-sponsor-ddstreet> <systemd (Ubuntu):Fix Released by joalif> <systemd (Ubuntu Bionic):In Progress by joalif> <systemd (Ubuntu Cosmic):In | 13:51 |
| mup | Progress by joalif> <systemd (Ubuntu Disco):Fix Released by joalif> <systemd (Debian):New> <https://launchpad.net/bugs/1818282> | 13:51 |
| ddstreet | mvo also, there may be a systemd security release coming soon so we might be waiting until after that to upload | 13:52 |
| mvo | ddstreet: thanks, that looks like a totally different area, so that should be fine | 13:53 |
| mup | PR snapd#6617 closed: cmd/snap: fix regression of snap saved command <Squash-merge> <⚠ Critical> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/6617> | 14:13 |
| Chipaca | cachio: spread -list google:[^u]...:tests/... vs spread -list google:[u]...:tests/... | 14:34 |
| zyga | pedronis: updated https://github.com/snapcore/snapd/pull/6597 | 14:51 |
| mup | PR #6597: cmd/snap-update-ns: refactor of profile application (1/N) <Created by zyga> <https://github.com/snapcore/snapd/pull/6597> | 14:51 |
| mup | Bug #1821023 opened: core18 base on core 16 missing firmware <Snappy:Triaged by zyga> <https://launchpad.net/bugs/1821023> | 14:54 |
| tobikoch | Is there an API call to determine which base a snap has before installation? | 15:06 |
| zyga | tobikoch: yes, see what happens when you `snap info --verbose jq-core18` | 15:08 |
| === ricab|lunch is now known as ricab | ||
| tobikoch | zyga: I see, I have to explicitly request the field. | 15:22 |
| cachio | Chipaca, nice, thanks | 15:22 |
| zyga | yes, I believe so | 15:22 |
| * zyga needs to break, need to study before classes | 15:22 | |
| === mborzeck1 is now known as mborzecki | ||
| Chipaca | tobikoch: request it where? | 15:31 |
| tobikoch | Chipaca: refresh API endpoint | 15:33 |
| Chipaca | tobikoch: on api.snapcraft.io? | 15:33 |
| tobikoch | Chipaca: yes | 15:33 |
| Chipaca | tobikoch: refresh endpoint, or info endpoint? | 15:34 |
| tobikoch | Chipaca: refresh | 15:34 |
| Chipaca | strange, but ok | 15:34 |
| Chipaca | tobikoch: what're you doing? | 15:34 |
| Chipaca | (out of pure unadulterated curiosity) | 15:35 |
| tobikoch | Updating the snap download tool in livecd-rootfs | 15:35 |
| Chipaca | tobikoch: 'snap download' not cutting it? | 15:36 |
| tobikoch | Chipaca: snap download was fine, but it doesn't allow passing along a cohort key. | 15:37 |
| tobikoch | So until it does, there is this script. | 15:37 |
| * cachio lunch | 15:37 | |
| Chipaca | pedronis: ^ were you aware that livecd-rootfs was using cohorts already? | 15:38 |
| Chipaca | tobikoch: fascinating :-) | 15:38 |
| tobikoch | heh | 15:38 |
| Chipaca | tobikoch: if you're doing bases like that, how're you doing other prerequisites (content providers, particularly)? | 15:38 |
| pedronis | Chipaca: yes | 15:38 |
| pedronis | I was aware | 15:39 |
| tobikoch | Chipaca: not sure I understand, so far the script does what it needs, I may be missing out on the wider scope. | 15:39 |
| Chipaca | tobikoch: if it's not a proble, it's not a problem :-) | 15:39 |
| tobikoch | :) | 15:39 |
| tobikoch | Chipaca: this is only for download, it does nothing else. | 15:40 |
| tobikoch | It's used for snap-preseeding in our images. | 15:40 |
| pedronis | Chipaca: they might need changes, but that's really a discussion to have with the store for them | 15:40 |
| Chipaca | k | 15:41 |
| tobikoch | pedronis: actually I had mentioned it here: https://forum.snapcraft.io/t/managing-cohorts/8995/2 | 15:43 |
| pedronis | zyga: could you re-review #6485 (it has two reviews but it had a complicated history so a 3rd will not hurt) | 16:00 |
| mup | PR #6485: interfaces/seccomp: regenerate changed profiles only <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/6485> | 16:00 |
| zyga | pedronis: certainly! | 16:00 |
| pedronis | thx | 16:04 |
| iTommix | Hi. maybe someone can help me: my apache user www-data should run a snap app (like /snap/bin/vlc) but gives me „cannot create user data directory: /var/www/snap/vlc/770: Read-only file system“ - first: the directoy is created by this app (its there), but second: could it be that there are missing permissions (solved with snap connect) ? | 16:16 |
| zyga | iTommix: hey, just a partial answer, read only file system means that you are on a read only file system, permissions cannot fix this | 16:17 |
| zyga | iTommix: I will let others help you, I'm partially AFK / commutting | 16:17 |
| iTommix | zyga: thats not right… the FS is writable. | 16:18 |
| zyga | is the path realy /var/www/snap/vlc/770? | 16:18 |
| zyga | that's unusual | 16:18 |
| iTommix | yes, because of the user www-data home-dir its /var/www | 16:19 |
| iTommix | i have an own „snap“-folder in my homedir | 16:19 |
| zyga | iTommix: please seek help on the forum, I'm sorry I cannot help you more now | 16:20 |
| iTommix | will wait till others answer maybe… thanks anyway :) | 16:20 |
| mup | PR snapd#6610 closed: interfaces/builtin: add add exec "/" to docker-support <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6610> | 16:24 |
| mup | PR snapd#6630 opened: cmd/snap: make 'snap warnings' output yamlish <Created by chipaca> <https://github.com/snapcore/snapd/pull/6630> | 16:25 |
| mvo | pedronis: do you want to squash merge 6624 and tweak the commit message or shall I do it? the current commit message does not match the reality of the PR anymore (or maybe Chipaca can do it when he is back?) | 16:47 |
| pedronis | mvo: I'm about to go afk | 16:47 |
| Chipaca | ah | 16:47 |
| Chipaca | mvo: i'll do it now | 16:47 |
| pedronis | so I think either you or John | 16:47 |
| pedronis | Chipaca: thx | 16:48 |
| mvo | Chipaca: thank you! later is fine if you are on the run | 16:48 |
| Chipaca | mvo: done | 16:48 |
| Chipaca | about to disappear but here still :-) | 16:49 |
| mvo | Chipaca: \o/ | 16:51 |
| mup | PR snapd#6624 closed: overlord/snapstate: retry less for auto-stuff <Squash-merge> <Created by chipaca> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/6624> | 16:52 |
| mup | PR snapd#6631 opened: tests: split travis spread execution in 2 jobs for ubuntu and non ubuntu systems <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/6631> | 17:20 |
| mvo | pedronis, Chipaca just FYI, I added a comment in https://bugs.launchpad.net/snapd/+bug/1768419/comments/4 | 17:23 |
| mup | Bug #1768419: Pass through snapd client User-Agent to store requests snapd <snapd:Triaged> <https://launchpad.net/bugs/1768419> | 17:23 |
| === pstolowski is now known as pstolowski|afk | ||
| mup | PR snapd#6615 closed: spread: some debug info to catch /var/snap/.. not being empty <⛔ Blocked> <Created by bboozzoo> <Closed by chipaca> <https://github.com/snapcore/snapd/pull/6615> | 17:46 |
| mup | PR snapd#6626 closed: strutil: make SplitUnit public, allow negative numbers <Simple 😃> <Created by bboozzoo> <Merged by chipaca> <https://github.com/snapcore/snapd/pull/6626> | 17:48 |
| mup | PR # closed: core-build#11, core-build#22, core-build#26, core-build#37 | 18:10 |
| mup | PR # opened: core-build#11, core-build#22, core-build#26, core-build#37 | 18:11 |
| === jacekn_ is now known as jacekn | ||
| Wimpress | Snapcraft Live! starts in about 10 minutes - https://www.youtube.com/watch?v=X_U-pcvBFrU | 19:51 |
| mup | PR # closed: core-build#11, core-build#22, core-build#26, core-build#37 | 20:50 |
| mup | PR # opened: core-build#11, core-build#22, core-build#26, core-build#37 | 20:51 |
| mup | PR snapcraft#2506 opened: schema, tests: add more detail wrt numeric version errors <Created by adanhawth> <https://github.com/snapcore/snapcraft/pull/2506> | 20:54 |
| mup | Issue # closed: core18#56, core18#86, core18#89, core18#117 | 23:57 |
| mup | PR # closed: core18#43, core18#63, core18#72, core18#90, core18#98, core18#120 | 23:57 |
| mup | Issue # opened: core18#56, core18#86, core18#89, core18#117 | 23:58 |
| mup | PR # opened: core18#43, core18#63, core18#72, core18#90, core18#98, core18#120 | 23:58 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!