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

mupPR snapd#10675 opened: tests: migrate tests that are only executed on xenial to bionic <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10675>01:55
mupPR snapd#10676 opened: cmd/snap-update-ns/change.go: sort needed, desired and not reused mount entries <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10676>03:10
mardy'morning!05:27
mardyquestion of the day: what is the snappy-hub? https://code.launchpad.net/~alexmurray/snappy-hub/update-for-snapd-2.51.6/+merge/40765105:28
mardyis it the base-set of apparmor templates, starting from which snapd composes the final profiles?05:28
mborzeckimorning05:32
mborzeckimardy: look like amurray's private of snappy-debug snap05:34
mborzeckiprivate copy ofc05:34
mborzeckimardy: fwiw the base template which snapd uses is right here: https://github.com/snapcore/snapd/blob/master/interfaces/apparmor/template.go05:34
mardymborzecki: good morning! Thanks, so I guess it's not for us to review?05:42
mborzeckimardy: no, i don't think so, i've never reviewed anythign there, but you can always ask amurray about snappy-debug05:44
zygagood morning :)06:18
zygahello mvo06:18
mborzeckizyga: hey06:19
mvohey zyga and mborzecki 06:19
zygamvo, it's gotten cold here06:19
zygawhen I was leaving the house today it was just 9C outside06:20
zygabrr06:20
pstolowskimorning06:21
zygahey pawel!06:21
mborzeckimvo: pstolowski and hey06:21
mborzeckiyeah, it's cold here too, 10C at most06:21
mborzeckiand it's not even september :/06:21
mardyhi zyga, mvo, pstolowski :-)06:22
zygahey mardy 06:22
mardymborzecki: ah! it's 11C here, so we win today :-)06:22
mvogood morning pstolowski and mardy 06:34
mvozyga: yeah, same here (but I don't think 9C just yet), feels like the summer is on the way out06:34
zygaindeed :/06:35
mupPR snapd#10644 closed: o/snapstate: don't hold some snaps if not all snaps can be held by the given gating snap <Needs Samuele review> <Refresh control> <Created by stolowski> <Merged by stolowski> <https://github.com/snapcore/snapd/pull/10644>06:51
mupPR snapd#10673 closed: .github/workflows/test.yaml: fix logic <Simple 😃> <Created by anonymouse64> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10673>06:51
mupPR snapd#10647 closed: o/snapstate: fail remove with invalid snap names <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10647>06:56
mupPR snapd#10669 closed: o/hookstate/ctlcmd: correct err message if missing root <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10669>06:56
mvomardy: 10606 is ready it seems, just needs a master merge to resolve a conflict and (maaaaaybe) consider my diff in there but that would be one more merged PR :)07:15
mupPR snapd#10107 closed: packaging: stop snapd.{socket,service} before purging data <Precious Logs> <Test Robustness> <Created by mvo5> <Closed by mvo5> <https://github.com/snapcore/snapd/pull/10107>07:16
mardymvo: I'll see if I can apply your suggestion without major changes (but I can't foresee any)07:19
mvomardy: I tested locally and it seems unit tests are still happy07:21
mvomardy: but I only looked very briefly :)07:22
mardymvo: thanks, I applied it and also rebased+squashed the branch on top of the latest master, since it already got a few approvals07:29
mardyit's "funny", that on github you can make any changes to your branch, and you do not lose the approvals. On Gitlab one generally loses them (though it can also be configured to retain them)07:31
mupPR snapd#10677 opened: snapstate: fix misleading `assumes` error message <Simple 😃> <Skip spread> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10677>07:31
mupPR snapd#10671 closed: o/assertstate,snapstate: refresh validation set assertions with snap declarations <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <Closed by stolowski> <https://github.com/snapcore/snapd/pull/10671>07:46
mupPR snapd#10678 opened: o/snapstate: enforce validation sets assertions when removing snaps <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10678>08:31
pstolowskimardy: yes it's a bit controversial08:33
pstolowskiwe should use common sense and if changes are not trivial, re-request reviews for already approved08:33
mborzeckimvo: quick question about remodel since pedronis isn't around yet and maybe you remember what the agreement was, we update the tracking channel of kernel and gadget snaps, do you think the tracking channel of application snaps should be updated too?08:38
mvomborzecki: I think so, if it has changed we should probably do the same 08:42
mborzeckimvo: well, i extended the uc20 managers test that i have and we're not updating it ;)08:43
mborzeckii don't think that we do it for 18 remodels either08:43
mborzeckimvo: fwiw, we only add install tasks, but otoh it wasn't really possible to specify tracking channel in 18 remodels08:44
mvomborzecki: but I also have a question for you: in https://github.com/snapcore/snapd/pull/10634 I made some progress and debian/ubuntu seems to be in reasonable shape with go-modules. but the rpm is giving me a bit of trouble, it looks when I run "spread fedora-34-64:tests/main/listing" locally that it still tries to GO111MODULES=off things even though in this branch there should be no GO111MODULE=off left. so it seems this is somehow implicit from the rpm macros?08:44
mupPR #10634: many: move to go modules gradually  <â›” Blocked> <Created by mvo5> <https://github.com/snapcore/snapd/pull/10634>08:44
mvomborzecki: woah, thanks for finding this08:44
mvomborzecki: I guess that is something we need to fix eventually08:45
mvomborzecki: :/08:45
mborzeckimvo: it may be the case that the go rpm packaging sets that, let me check08:45
mborzeckimvo: go packaging in fedora sets that08:52
mborzeckimvo: https://paste.ubuntu.com/p/CKTWyCGD2W/08:53
mvomborzecki: hm, so it looks like this needs some deeper dive :/08:58
mvomborzecki: thanks, good to know where to look08:58
mborzeckimvo: it's part of the distro marcos https://paste.ubuntu.com/p/hdKZZZvGYs/09:02
mborzeckiso unless you redefine the marco it's not possible to drop it?09:03
mborzeckistill, do we need to drop that?09:05
zygahey miguelpires 09:06
miguelpiresHi =)09:08
mborzeckimvo: any clue who can fix/update the docs index while degville is away? some links are bad eg. https://snapcraft.io/docs/snapcraft-extensions09:23
pstolowskimiguelpires: hi, if you could address the last remaining comment from https://github.com/snapcore/snapd/pull/10584 that would be great, would love to see it landed today09:24
mupPR #10584: o/assertstate: fix missing 'scheduled' header when auto refreshing assertions <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10584>09:24
miguelpires@psto09:26
miguelpirespstolowski pushing it now =)09:26
pstolowskimiguelpires: awesome! thanks09:27
degvillemborzecki / mvo: having a look at that issue now (I can spare some time).09:54
mborzeckidegville: hey, no worries, you're on vac 😉 do you know who i can poke to take a look at this?09:54
degvillemborzecki: first glance, and it looks like the webteam have swapped out the old  table of contents again.09:55
degvillemborzecki: Fran on the webteam made the change.09:55
degvillethey're now using this as the source for all pages: https://forum.snapcraft.io/t/snap-documentation-pages/2575009:56
mborzeckidegville: cool, i'll ping jkfran in the web channel09:57
degvillemborzecki: thanks! worst case is he'll need to revert to the old style. It's very annoying they make these changes like without tests or letting us know.09:59
pstolowskimiguelpires: +1, but I think there is a small vet problem09:59
mborzeckidegville: yeah, thank you for looking into this 🙂10:00
miguelpirespstolowski Yeah, the static checks are failing but I can't reproduce it locally. Strange10:02
degvillemborzecki: no problem, sorry you had to get involved. Thank you!10:02
pstolowskimiguelpires: different go vet versions probably10:03
zygahey degville :)10:25
zygalong time no see10:25
mborzeckipstolowski: can https://github.com/snapcore/snapd/pull/10243 be closed or do you plan to update the test?10:35
mupPR #10243: tests: failure of prereqs on content interface doesn't prevent install <Created by stolowski> <https://github.com/snapcore/snapd/pull/10243>10:35
degvillezyga: hello!! yeah, it has been a while. Hope you're doing well.10:37
zyga:)10:38
zygayeah, I'm fine10:38
zygaI miss sprints10:38
zygaand the people10:38
degvilleyeah. I think we all do, although we can hopefully get back to them next year. I really miss travelling too - stayed at home this year, which has been fun too, but I'm already looking forward to exploring further afield.10:40
zygano Frankfurt in Sept?10:42
pstolowskimborzecki: no, it was unexpected discovery to me but works as designed, if you mean updating XXX comment then yes, i should update it. i think it makes sense to have this test10:52
mborzeckiok, sgtm10:53
pstolowskilemme do this right now10:53
pstolowskimborzecki: updated PR description as well10:58
mborzeckianyone using `hub` tool? damn thing doesn't want to checkout PRs anymore, getting 40410:58
pstolowskimborzecki: i do from time to time, let me check11:00
pstolowskimborzecki: works, hub 2.14.211:01
mborzeckipstolowski: hm same version :/11:01
pstolowskimborzecki: hub pr checkout 10676, that's what i tried11:02
mborzeckipstolowski: ok, i'm a muppet, was calling that in a different repository11:14
pstolowskilol11:14
pstolowskimvo: can you land https://github.com/snapcore/snapd/pull/10617? random failures11:21
mupPR #10617: cmd/snap: add Size column to refresh --list <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10617>11:21
pstolowskii saw some PRs failing with https://codecov.io: 404 on CodeCov11:45
mupPR snapd#10679 opened: tests: fix restore in snapfuse spread tests <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10679>11:52
mborzeckimiguelpires: hm something went south a bit: https://paste.ubuntu.com/p/9z2R25ZtSX/ can you check whether this happens on master too?12:19
mborzecki(that's after mergin master to #10241)12:19
mupBug #10241: readline5: FTBFS: Missing Build-Depends on 'texinfo' <readline5 (Ubuntu):Fix Released by doko> <readline5 (Debian):Fix Released> <https://launchpad.net/bugs/10241>12:19
mupPR #10241: tests: set memory limit for snapd <Simple 😃> <Created by bboozzoo> <https://github.com/snapcore/snapd/pull/10241>12:19
miguelpiresmborzecki: do you mean the error itself or the weird whitespace? I opened a PR to fix the spread test: https://github.com/snapcore/snapd/pull/10679 12:23
mupPR #10679: tests: fix restore in snapfuse spread tests <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10679>12:23
mborzeckimiguelpires: the spread test, thanks for pushing a fix!12:24
pstolowskimiguelpires: haha, funny how ---purge went unnoticed because of that other problem you fixed12:29
mardyare your PRs also affected by spread timeouts, or is it only mine? :-) https://github.com/snapcore/snapd/pull/1060612:38
mupPR #10606: o/hookstate/ctlcmd: unify the error message when context is missing <Squash-merge> <Simple 😃> <Created by mardy> <https://github.com/snapcore/snapd/pull/10606>12:38
miguelpireshaha, I guess now we know the remove validation really works!12:42
mvomiguelpires: \o/12:44
pstolowski:)12:49
ares1256HI, I am working on a project to setup a hotspot and do a lot more things. So, I am using wifi-ap to create hotspot. I add it to stage snaps. How am I supposed to run a snap installed in my snap?12:53
ares1256i try running the executable in the $SNAP/bin folder but it does not work, am I missing something?12:53
miguelpiresmardy: I noticed the same  "cannot provision XYZ: quota 'CPUS' exceeded"12:55
zygaares1256, do you want to run an exported snap application from the system or an internal binary from a snap command 12:55
ares1256yeah I want to run an exported snap from my snap command12:56
zyga?12:56
zygawait, what is an exported snap?12:56
zygado you want to run an app published by your own snap from another app in your own snap?12:56
ares1256like a snap which is already on the store'12:56
zygaah12:56
zygathat's not possible unless your snap is using classic confinement12:56
ares1256ohh12:56
zygathat would introduce a dependency12:56
zygaand those must be modeled with interfaces 12:57
zygabut technically it's just hard/impossible to run an app from another app 12:57
zygamainly because seccomp, by design, does not allow a process to shed past restrictions12:57
ares1256and that would also mean that the snap should have interfaces available to interact with it right?like slots availble to connect to...12:58
zygaand technically your snap sandbox would combine with the sandbox of the other app, resulting in breakage at best12:58
zygaright but even then you have to explicitly design for that12:58
ares1256ohh...12:58
zygae.g. by offering a socket and having an interface that lets one of the snaps talk to the other12:58
zygaservices are hard12:58
zygathe socket model is really the way forward but it requires some preparation12:59
zygaas each service runs with its own confinement12:59
ares1256so lets say there is a snap wifi-ap12:59
zygaand technically the interface is not about snap-to-snap communication12:59
zygabut about a snap-to-service12:59
ares1256and i can start wifi-sp by command start12:59
zygaand the service _can_ be replaced by another snap12:59
ares1256so i cant have a shell file saying "start"?12:59
ares1256to begin that wifi-ap snap?13:00
zygano13:00
ares1256ohh13:00
zygawell, I can explain why13:00
zygabut no13:00
zygafor both technical and design reasons13:00
zygayou can run any command from the unconfined host process13:00
zygabut once you are inside a snap application, you are confined and you cannot, by design, execute apps from other snaps13:01
zyga(or most of the host apps for that matter)13:01
zygaon ubuntu-core devices this is strict, on classic devices you may use classic confinement to do more free-form snaps but those are notoriously hard to get right13:01
zyga(with apologies to sergiusens)13:01
ares1256yeah i was able to do it in devmode...but when i came to starict.....nothing was working13:02
zygaright13:02
zygathat's expecte13:02
zyga*expected13:02
zygaeven in devmode you could easily hit a wall where it breaks though13:02
zygabecause unless both snaps are in devmode13:02
zygathe "other" snap is still using normal confinement13:03
ares1256so my only option is to create my own hotspot snap?13:03
zygaor bundle both into one larger snap13:03
ares1256if i need full control over it13:03
ograares1256, have a look at https://github.com/ogra1/nm-wifi-hotspot-snap13:03
ogra(also in the store already) 13:04
zygaor work with the author of the other snap to create a control interface13:04
zyga(a socket and a matching snapd interface that allows you to use it)13:04
zygahey ogra :-)13:04
ograyo !13:04
zygahow have you been?13:04
ograplaying with kiosks recently again 13:04
ares1256thanks, i wil look into this @o13:05
ogra(trying to convince a browser to use accelerated video playback on a PI ... with a fun 5000 line patch to chromium 😛 )13:05
ares1256thanks ogra and zyga i will look into this13:05
zygaares1256, good luck!13:06
zygaogra, raspberry pi, how we all have to deal with it :)13:06
ograares1256, also, in case you are interested in a captive-portal setup https://github.com/ogra1/opennds-snap ... (https://snapcraft.io/opennds and https://snapcraft.io/nm-wifi-hotspot are the store pages for them)13:06
ograzyga, yeah ... its a blessing and a curse at the same time 🙂13:07
ares1256Alright(y) 13:07
zygaogra, I was looking for CM4 modules but availability is well into the Q2 202213:07
ograyeah, chip shortage 13:07
zygaat some point I can switch to pico :)13:08
zygaand play with zephyr 13:08
zygabut not yet13:08
ograi luckily got my CM4 half a year ago ... running fine, booting off NVME PCIe 😉13:08
zygaogra, I was looking for a model that doesn't suck, some ram, emmc and wifi13:09
zygano luck13:09
zygabase boards are plentiful though13:09
ograyeah13:09
ograi guess the chip shortage will still last a while 😞13:10
zygahow come we have chip shortage and not beer shortage13:11
zygacan people really consume more silicon than alcohol? ;-)13:11
ogralol13:12
ares1256hi ogra , just to be sure. If i can use nm-wifi-hotspot. I have to include the app inside my snap right?13:25
ares1256*inside my snapcraft.yaml13:25
ograares1256, as you like, you can just use the existing snap but will need to manually set the SSID and password13:25
ares1256yeah thats fine13:26
ares1256ok thanks13:26
ograor you can include my snap via stage-snaps 13:26
ograor you can just copy/paste the part and app bits from my snapcraft.yaml into yours 13:26
ograall three should work13:26
ares1256if i stage snaps, can i run this command then " snap set nm-wifi-hotspot ssid="My Cool SSID""13:26
ograit will be slghtly different since you likely wont call it nm-wifi-hotspot ... but yeah, the basics should be the same13:27
ares1256ohk(y) 13:27
ograif you have any issues down the road, feel free to ask 13:28
ares1256ok, thanks13:29
ares1256just another small question of a different topic13:32
ares1256i want to modify a config file that i stored in $snap_data directory13:32
ares1256so i add a plug13:32
ares1256  config-files:13:32
ares1256    interface: system-files13:32
ares1256    read:13:32
ares1256    - $SNAP_DATA/AP_config13:32
ares1256    - $SNAP_DATA/lte_config13:32
ares1256when i compile this, it always gives an erro that it requires "/" first13:33
ograyou do not need an interface for accessing $SNAP_DATA ... 13:34
ograthat is always rw for the snap 13:34
ares1256ohh13:35
ares1256ok13:35
ograand yes, system-files requires a full path on the filesystem, it is usually used to i.e. access something on the host filesystem you normally can not get access to with any existing interface13:35
pstolowskimvo: can you merge https://github.com/snapcore/snapd/pull/10584 ?13:41
mupPR #10584: o/assertstate: fix missing 'scheduled' header when auto refreshing assertions <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10584>13:41
mvopstolowski: sure13:48
mvothanks pstolowski and miguelpires 13:48
mupPR snapd#10584 closed: o/assertstate: fix missing 'scheduled' header when auto refreshing assertions <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10584>13:52
mupPR snapd#10680 opened: o/assertstate,daemon: refresh validation sets assertions with snap declarations <Needs Samuele review> <validation-sets :white_check_mark:> <Created by stolowski> <https://github.com/snapcore/snapd/pull/10680>14:22
mvoijohnson[m]: a second review for 10666 would be great14:28
mardymvo: could you please use your magic powers on https://github.com/snapcore/snapd/pull/10606 ?14:29
mupPR #10606: o/hookstate/ctlcmd: unify the error message when context is missing <Squash-merge> <Simple 😃> <Created by mardy> <https://github.com/snapcore/snapd/pull/10606>14:29
pstolowskimvo: can you merge #10617 as well?14:29
mupPR #10617: cmd/snap: add Size column to refresh --list <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10617>14:29
mupBug #10617: Multiple cd burning problems <Ubuntu:Fix Released> <https://launchpad.net/bugs/10617>14:29
pstolowskimy old PR that is test-only needs a 2nd review: https://github.com/snapcore/snapd/pull/1024314:42
mupPR #10243: tests: failure of prereqs on content interface doesn't prevent install <Created by stolowski> <https://github.com/snapcore/snapd/pull/10243>14:42
pstolowskicachio_: could you merge master into #10443 to ensure it still passes (i was last executed 1 month ago)14:47
mupPR #10443: tests: check files and dirs are cleaned for each test <Created by sergiocazzolato> <https://github.com/snapcore/snapd/pull/10443>14:47
mupBug #10443: X pointer persistent in middle of screen on login <xfree86 (Ubuntu):Invalid by daniels> <https://launchpad.net/bugs/10443>14:47
miguelpiresmvo: can you merge #10679 please?15:49
mupPR #10679: tests: fix restore in snapfuse spread tests <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/10679>15:49
mupBug #10679: Use xdelta binary diffs in Ubuntu unstable release? <apt (Ubuntu):Invalid by mdz> <https://launchpad.net/bugs/10679>15:49
mvomiguelpires: sure15:50
mupPR snapd#10679 closed: tests: fix restore in snapfuse spread tests <Created by MiguelPires> <Merged by mvo5> <https://github.com/snapcore/snapd/pull/10679>15:53
mupPR snapcraft#3571 opened: [WIP] offline <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3571>16:46
mupPR snapcraft#3572 opened: cli & providers: pass part names for lifecycle commands <Created by cjp256> <https://github.com/snapcore/snapcraft/pull/3572>20:36
* cachio_ afk20:46
mupPR snapd#10681 opened: interfaces/hardware-observe: add some dmi properties <Simple 😃> <Needs security review> <Created by anonymouse64> <https://github.com/snapcore/snapd/pull/10681>23:59

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