| zyga[m] | mvo: hey, are you using spread with lxd backend anywhere in snapd? | 10:04 |
|---|---|---|
| zyga[m] | mvo: I filed https://github.com/snapcore/spread/issues/151 because I think this part has bit-rotted | 10:04 |
| mvo | zyga[m]: we are not currently, only gce and qemu | 10:05 |
| zyga[m] | oh, speaking of lxd | 10:15 |
| zyga[m] | I sent a patch a while ago | 10:15 |
| * zyga[m] looks | 10:15 | |
| zyga[m] | did samuele have a chance to review that? | 10:15 |
| zyga[m] | yes! | 10:16 |
| mup | PR snapd#11912 opened: features: disable refresh-app-awarness by default again <Created by mvo5> <https://github.com/snapcore/snapd/pull/11912> | 10:20 |
| zyga[m] | mvo: can you or samuele perhaps ping me once ther is an agreement on how to introduce the new refresh mode? | 10:22 |
| zyga[m] | the changes are simple but my time budget does not allow for back and forth exploration with full tests and all the expected changes | 10:22 |
| zyga[m] | I'm happy to do that once it's confirmed if the new refresh-mode is the way forward | 10:22 |
| mvo | zyga[m]: sorry, meetings - I am inclined to just revert refresh-app-awareness for now | 10:43 |
| mvo | zyga[m]: and then look at this without time pressure | 10:44 |
| zyga[m] | mvo: I saw, that a bit big footed but I understand | 10:44 |
| zyga[m] | mvo: perhaps just disable it for lxd and nothing else? | 10:44 |
| zyga[m] | it's a regression for browsers | 10:44 |
| zyga[m] | the patch I offered now handles this without a regression as well, with new optional syntax | 10:45 |
| mvo | zyga[m]: I discussed just disabling for lxd with samuele but I feel refresh-app-awareness has other issues too, like the UX is really not there yet and we need a bit of time to fix that. i.e. right now it tells you "please close your apps" but when you do nothing in snapd picks that up and does the refresh, one needs to do it mnaully. it's just not great | 11:04 |
| mvo | zyga[m]: and for firefox - it will keep working, the bug in the layouts that we used ot have here got fixed | 11:04 |
| zyga[m] | oh, that's nice | 11:05 |
| zyga[m] | sure, if you think this is the way forward that's good | 11:05 |
| zyga[m] | so a new version of snapd without r-a-a by default | 11:05 |
| zyga[m] | and the (perhaps) new design from samuele on refresh-mode | 11:05 |
| zyga[m] | and lxd can eventually roll that out with a proper assumes | 11:06 |
| mvo | zyga[m]: precisely | 11:07 |
| mup | PR snapd#11913 opened: cmd/snap: fix test failing due to timezone differences <Simple 😃> <Skip spread> <Created by MiguelPires> <https://github.com/snapcore/snapd/pull/11913> | 15:01 |
| mup | PR snapcraft#3790 closed: lifecyle: root level build-packages and build-snaps <Created by sergiusens> <Merged by sergiusens> <https://github.com/snapcore/snapcraft/pull/3790> | 16:55 |
| === ahasenack__ is now known as ahasenack | ||
| mup | PR snapcraft#3810 opened: extensions: method to get data dir <Created by sergiusens> <https://github.com/snapcore/snapcraft/pull/3810> | 20:26 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!