/srv/irclogs.ubuntu.com/2021/08/31/#snappy.txt

mupPR snapcraft#3570 closed: ROS V2 plugins: resolve conditional dependencies <Created by artivis> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3570>00:08
h1potwhenever i invoke snap, i'm getting main.go:176: description of prepare-image's "<target-dir>" is lowercase in locale "pt_BR": "o directรณrio de destino"01:31
=== Ringtailed_Fox is now known as RingtailedFox
=== RingtailedFox is now known as ItsNeverAWasteOf
=== ItsNeverAWasteOf is now known as Time
=== Time is now known as RingtailedFox
=== RingtailedFox is now known as ElIsWrong
=== ElIsWrong is now known as RingtailedFox
=== RingtailedFox is now known as ZnevnaIsATroll
=== ZnevnaIsATroll is now known as RingtailedFox
mborzeckimorning05:34
=== RingtailedFox is now known as Ringtailed_Fox
mardymborzecki, miguelpires: hi!05:47
miguelpireshi mardy :)05:48
mardygolang question of the day: when importing a package, will the whole package be included in the final binary, or only those parts which are actually used?05:49
mborzeckimardy: if you're thinking about lto, then go compiler isn't that smart afaik05:53
miguelpiresGood morning mvo. Can you merge https://github.com/snapcore/snapd/pull/10685 please?06:24
mupPR #10685: many: fix run-checks gofmt check  <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10685>06:24
mupPR snapd#10696 closed: many: move to yaml.v3 <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/10696>06:30
pstolowskimorning06:32
mardy'morning pstolowski, mvo 06:33
mvogood morning miguelpires, sure, happy to do that06:34
mvomiguelpires: it's early for you, isn't it :) ?06:35
mvogood morning pstolowski 06:35
mvoand good morning mardy too :)06:35
mardymborzecki: I'm not sure how the LTO idea translates to golang, but let me ask a more concrete question: I've added a method here: https://github.com/mardy/snapd/blob/access-portal/usersession/client/client.go#L297-L33206:36
mardyone possibility is to have this method be called by a separate process, not snapd itself06:37
mardynow, it would be a bit unlucky if this process, which does not need to know anything about systemd services and other snapd features, had to carry in its binary the code for serviceControlCall() and all of its depending bits06:38
mardy(though, as a matter of fact, it seems that this particular package does not have many deps in systemd, so in this case it might be quite an acceptable situation)06:39
mardys/systemd/snapd/06:40
mupPR snapd#10685 closed: many: fix run-checks gofmt check  <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10685>06:40
mupPR snapd#10698 opened: wrappers: fix a bunch of duplicated service definitions in tests <Simple ๐Ÿ˜ƒ> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10698>06:40
mborzeckimiguelpires: mvo: pstolowski: hey06:41
mvohey mborzecki 06:42
mborzeckimardy: i suppose easiest solution is to build a minimal program and inspect what symbols show up inside, otoh if it's that minimal than runtime and net/http will take up most of the size, + that magical line/symbol lookup table that go generates, i would not expect anything smaller than 3-4MB really :P06:42
miguelpires@mvo a bit, I look like =.= 06:43
miguelpireshi mborzecki06:43
mborzeckimvo: https://github.com/snapcore/snapd/pull/10634 looks mostly happy06:48
mupPR #10634: many: move to go modules gradually  <โ›” Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10634>06:48
mborzeckimvo: shall we dropped the blocked label then? :)06:51
mvomborzecki: \o/ !!! yes06:52
mvomiguelpires: haha06:52
zyga-mbpgood morning :)07:17
zyga-mbphey mardy, mborzecki, mvo and miguelpires :)07:18
pstolowskihey zyga-mbp 07:19
mborzeckimeh, the remodel code for generating tasks is pretty hairy now07:25
mvogood morning zyga-mbp 07:25
sil2100Hello snapd! 07:39
sil2100So I wanted to finally release snapd 2.51.1 into -updates for all the stable series07:39
sil2100Sadly, focal constantly FTBFS on riscv6407:39
sil2100https://launchpad.net/ubuntu/+source/snapd/2.51.1+20.04/+build/2170192207:39
sil2100I think I already retried the build like 3 times07:39
pstolowskisil2100: hi, looks like a flaky test (managers_test.go:3522: mgrsSuite.TestUpdateManyWithAutoconnect - "Settle is not converging"), i'll see if I can find something or reproduce locally in a moment07:46
mupPR snapd#10699 opened: daemon: allow enabling enforce mode <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10699>08:00
mborzeckiheh and that case of new kernel or base being already installed during remodel was compeltely untested, fun08:10
pstolowskiit seems 3s settle timeout in this test (scaled to 18s on riscv64) may not be enough (a few lines down we've another settle but with much greater timeout  - 6*45s on riscv64)08:12
pstolowskii can bump it, but 2.51.1 is only tagged (not branched, but we have 2.51 branch covering up to .7), so not sure what to patch for?08:14
pstolowskimvo ^ ?08:14
mvopstolowski: in a meeting, sry, will get back to you 08:16
mborzeckipstolowski: hm i see all tags up to 2.51.7?08:23
mvomborzecki: could you please set "security-high" on 10661 once the feedback from alberto/ian is adressed? I would love to get it (more) ready :)08:24
mborzeckimvo: will do, wanted to finish the uc20 remodel since it's very close now08:25
pstolowskimborzecki: yes, tags are there, but can a branch target a tag?08:30
mvomborzecki: totally fine, sorry for being pushy08:33
pstolowskiit doesn't seem possible to target a tag with a PR (which makes sense), so i guess we would need a proper 2.51.1 branch 08:36
mvopstolowski: let's talk in some minutes, I need a bit more context I think08:41
mupPR snapd#10700 opened: tests: use host-scaled timeout to avoid riscv64 test failure <Simple ๐Ÿ˜ƒ> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10700>08:45
pstolowskimvo: we need this tweak ^ (here: for master), but for 2.51.1 which FTBFS on riscv64 now, see above what sil2100 reported08:46
mupPR snapd#10701 opened: many: remove unused parameters <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10701>08:50
mvopstolowski: I see, I can handle this, no worries, we will do a 2.51.1+20.04ubuntu1 or something equally nice outside for the git repo by just applying the patch08:51
pstolowskiok, great08:52
mardystgraber: I'm getting this error when running `snapcraft --use-lxd`: An error occurred with the instance when trying to launch with 'LXD': Failed creating instance from image: Failed to run: zfs clone lxd/images/e076b3174006b985a6298feb03d74f565be43d1f3df21b530a31c8372f6c27be@readonly lxd/containers/snapcraft-snapd: cannot create 'lxd/containers/snapcraft-snapd': dataset already exists.09:29
mardyremoving (with --purge) the lxd snap and reinstalling it helped09:53
sil2100pstolowski: thanks!10:23
sil2100In the meantime I'll try rebuilding the existing package again, maybe the flacky test will pass this time - since I'd like to unblock snapd releases ASAP10:24
pstolowskisil2100: i hope my fix solves it. the 3s timeout we had was very aggressive for riscv64, in other places it was 270s (!) with a comment saying that riscv64 builder is 5x slower than armhf10:29
mborzeckihmm gcc -fanalyzer does not like g_assert_nonull() and concludes it will not stop the program execution10:44
mborzeckiand clang-analyzer isn't coping with the cleanup attribute too well? eh10:49
sil2100pstolowski: wow, 3s? Yeah, that sounds like a bad idea for riscv64 machines that are generally quite slow11:04
mupPR snapd#10702 opened: interfaces: introduce snap-refresh-control interface <Needs Samuele review> <Refresh control> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10702>11:11
pstolowskiwoah, i didn't know we have "Missing high-level test for interface 'snap-refresh-control'. Please add to: ..." test check11:18
pstolowskinice touch11:19
mupPR snapcraft#3577 opened: cli: add experimental --offline option (CRAFT-480) <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3577>11:49
mardywhat do you think of this way of mocking systemd? https://github.com/mardy/snapd/commit/0a524128abd3021f98e6ed1a66d1e1ecd26f0ea012:12
mupPR snapd#10703 opened:  tests/main/security-device-cgroups-strict-enforced: demonstrate device cgroup being enforced <cgroupv2> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10703>12:26
mardyfeedback very welcome: https://github.com/snapcore/snapd/pull/1070412:42
mupPR #10704: systemd: add fake type for easier mocking of systemd functions <Created by mardy> <https://github.com/snapcore/snapd/pull/10704>12:42
mupPR snapd#10704 opened: systemd: add fake type for easier mocking of systemd functions <Created by mardy> <https://github.com/snapcore/snapd/pull/10704>12:46
mvoPR 10634 now needs reviews, it seems to be ready finally13:09
mupPR #10634: many: move to go modules gradually  <Created by mvo5> <https://github.com/snapcore/snapd/pull/10634>13:09
ares1925hi13:18
ares1925i am trying to setup up can adapter mcp2518fd on spi613:19
ares1925so i wrote an overlay for it13:19
ares1925but i cant find what the issue is in this13:19
ares1925 * Device tree overlay for mcp251x/mcp2518fd on spi0.013:19
ares1925 */13:19
ares1925    compatible = "brcm,bcm2835", "brcm,bcm2836", "brcm,bcm2708", "brcm,bcm2709";13:19
ares1925    /* disable spi-dev for spi3.0 */13:19
ares1925    fragment@0{13:19
ares1925target = <&spidev3>;13:19
ares1925__overlay__ {13:19
ares1925    status = "disabled";13:19
ares1925};13:19
ares1925    };13:19
ares1925    /* the interrupt pin of the can-controller */13:19
ares1925    fragment@2 {13:19
ares1925        target = <&gpio>;13:19
ares1925        __overlay__ {13:19
ares1925            can_int_pins: can_int_pins {13:19
ares1925                brcm,pins = <6>;13:19
ares1925                brcm,function = <0>; /* input */13:19
ares1925            };13:19
ares1925    spi6_pins: spi6_pins {13:19
ares1925brcm,pins = <19 20 21>;13:19
ares1925        target = <&spi6>;13:20
ares1925        __overlay__ {13:20
ares1925            /* needed to avoid dtc warning */13:20
ares1925            #address-cells = <1>;13:20
ares1925            #size-cells = <0>;13:20
ares1925    cs-gpios = <&gpio 18 1>;13:20
ares1925     status = "okay";13:20
ares1925            can0: can@0 {13:20
ares1925                reg = <0>;13:20
ares1925                pinctrl-names = "default";13:20
ares1925         pinctrl-0 = <&spi6_pins &spi6_cs_pins>;13:20
ares1925                compatible = "microchip,mcp2518fd";13:20
ares1925                spi-max-frequency = <20000000>;13:20
ares1925                interrupt-parent = <&gpio>;13:20
ares1925                interrupts = <6 8>; /* IRQ_TYPE_LEVEL_LOW */13:20
ares1925                clocks = <&mcp2518fd_osc>;13:20
ares1925            };13:20
ares1925};13:20
ares1925later i want to set it up for ucore too, so first i am trying it on ubuntu server13:22
mupPR snapd#10698 closed: wrappers: fix a bunch of duplicated service definitions in tests <Simple ๐Ÿ˜ƒ> <Skip spread> <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10698>13:56
mupPR snapd#10700 closed: tests: use host-scaled timeout to avoid riscv64 test failure <Simple ๐Ÿ˜ƒ> <Created by stolowski> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10700>13:56
pstolowskiares1925: this is a very detailed and low level question that you should probably ask elsewhere (i'm not sure how is it related to snapd and snaps but if you think it is, you may try to ask on the forum along with some more context). also, please don't paste huge snippets like this on irc, use pastebin.ubuntu.com and post links instead13:56
ares1925ohk sorry14:01
mborzeckihm so the spread test is happy on a remodel branch commit just before i merged master :/14:20
mborzeckiand master passed too14:20
mupPR snapd#8926 closed: interfaces: add microstack-support interface <Squash-merge> <:birthday:> <Created by dshcherb> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/8926>14:21
mupPR snapd#10705 opened: tests: add minimal smoke test for microstack <Created by mvo5> <https://github.com/snapcore/snapd/pull/10705>14:51
* mvo switches network, bbiab14:55
=== itu is now known as Guest2168
mupPR snapd#10524 closed: packaging: add libfuse3-dev build dependency <Created by mvo5> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10524>16:16
=== Ringtailed_Fox is now known as God
=== God is now known as RingtailedFox
mupPR snapd#10706 opened: tests: clean up test tools <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10706>18:02
mupPR snapcraft#3566 closed: ROS 2 plugins v2 misc fixes <Created by artivis> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3566>18:25
mupPR snapcraft#3577 closed: cli: add experimental --offline option (CRAFT-480) <Created by cjp256> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3577>19:35
=== Guest2168 is now known as itu
=== itu is now known as Guest6671
=== Guest6671 is now known as itu

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