/srv/irclogs.ubuntu.com/2018/07/27/#ubuntu-desktop.txt

=== mgedmin_ is now known as mgedmin
didrocksgood morning05:26
dufluMorning didrocks05:30
dufluAnd nothing personal, but bbl05:30
didrockshey duflu05:34
didrocks;)05:34
oSoMoNgood morning desktoppers05:47
didrocksand a 3rd reboot due to this extension crashing the Shell!06:20
* didrocks does reboot on reboot since 7am :/06:20
dufluMorning oSoMoN06:27
jibelduflu, hi, about bug 1691921 you theory looks right. Yesterday I couldn't open system settings because of "too many open files" so it might be related.06:44
ubot5bug 1691921 in gnome-shell (Ubuntu) "gnome-shell crashed with SIGTRAP in g_wakeup_new() from g_main_context_new() from g_dbus_connection_send_message_with_reply_sync() from g_dbus_connection_call_sync_internal() from g_dbus_connection_call_sync() ["Creating pipes for GWakeup: Too many open files\n"]" [High,Confirmed] https://launchpad.net/bugs/169192106:44
duflujibel, yeah it's less likely that you'd ever hit "Too many open files" without a leak. Although that can happen06:48
dufluThe default limit is still 102406:49
dufluper process06:49
duflujibel, hopefully you can identify it by name in /proc/PID/fd/* ?06:49
jibelduflu, it's set to 2034595 on my system, I'll have a look06:52
dufluulimit -n ?06:52
jibelah true, 102406:53
jibelduflu, 1017 gnome-shell06:57
jibel1017 = files in fd/06:57
duflujibel, right so something transient released some but it clearly bounced off the limit06:57
duflujibel, are they mostly the same name? any name?06:57
dufluor type?06:57
jibelduflu, they are all like  980 -> 'pipe:[8531231]06:59
dufluHmm, tricky06:59
duflujibel, I can only think of spawning external programs (in a buggy way) could cause that. Maybe an extension is running some command periodically?07:01
jibelduflu, https://paste.ubuntu.com/p/WFMNPZ6Bk8/ is the output of lsof for one of the pipe id07:02
dufluOr maybe it's a shell and it's meant to spawn programs ;)07:02
duflujibel, "JS\x20Hel" sounds like "JS Helper" or something?07:03
duflujibel, sounds like an extension is using https://community.algolia.com/algoliasearch-helper-js/ or such in a buggy way07:05
duflugnome-shell doesn't seem to use it07:05
duflujibel, try grepping the extensions *.js for "JS Help"07:06
duflugrep -r "JS Help" /usr/share/gnome-shell/extensions/07:07
jibelduflu, yes it's JS Helper https://paste.ubuntu.com/p/rrbQpW5tqk/07:11
duflujibel, yeah I think it must be an extension using that. Just grep the extensions for it07:11
* duflu assumes most of the other 1017 are the same07:12
jibelduflu, nothing07:12
jibeland nothing in locally installed extensions07:13
duflujibel, grep -ri help.js ?07:13
dufluor helper.js07:13
jibelduflu, still nothing07:14
duflujibel, OK then time to grep everything on disk, including binaries :)07:16
dufluDamn. libmozjs-5207:17
duflujibel, I found one: package mozjs5207:18
dufluwhich I think is used indirectly by gnome-shell07:18
jibelduflu, http://paste.ubuntu.com/p/yKTgtRtJzB/ all the references to JS Helper from lsof07:20
jibeldefinitely not specific to gnome-shell07:20
duflujibel, no that's a reverse lookup you did. That's all files in use by process "JS Helper"07:21
dufluSo maybe not the issue after all07:23
duflujibel, try:  lsof -c gnome-shell07:23
dufluand see what most of the 1017 entries are07:23
* duflu is seeing too many references to a deleted file "/i915"07:24
dufluso that would be mesa-related07:24
jibelduflu, http://paste.ubuntu.com/p/Fqjf2Y8Tbg/07:28
jibelmost of the entries are pipes07:29
duflujibel, I wonder - can you match any of those NODE values to pipes in other processes?07:31
duflugnome-shell 7476 j-lallement  153w     FIFO               0,12      0t0 7855742 pipe07:31
duflugnome-shell 7476 j-lallement  154r     FIFO               0,12      0t0 7855299 pipe07:31
duflugnome-shell 7476 j-lallement  155w     FIFO               0,12      0t0 7855912 pipe07:31
duflugnome-shell 7476 j-lallement  156r     FIFO               0,12      0t0 7853797 pipe07:31
duflu(the second last column)07:31
dufluPossibly not. More likely they are leaked descriptors to dead pipes07:32
dufluI think some basic system-monitor type shell exensions might do stuff like that. So try excluding those07:32
jibelduflu, no, they only exist for process 7476 (gnome-shell)07:33
duflujibel, basically it looks like something in gnome-shell has been spawning external commands and not cleaning up. Could be a system monitor extension07:35
jibelduflu, I had a system-monitor extension enabled. It's removed now and I'll monitor the number of open files to see if it fixes the situation07:36
dufluCool07:36
seb128good morning desktopers07:47
dufluMorning seb12807:49
seb128hey duflu, how are you?07:49
jibelsalut seb12807:50
dufluseb128, going well. How are you?07:51
willcookehi gang07:52
willcookechrisccoulson, the thunderstorms have failed to arrive. again07:52
didrockshey seb128, willcooke07:53
chrisccoulsonwillcooke, we got a some lightning at 4am, and I got up to cover the garden furniture07:53
chrisccoulsonbut the actual storm passed a few miles to our west and it didn't rain in the end07:53
dufluHi willcooke07:55
seb128hey jibel willcooke didrocks07:59
seb128we had like 3 drops yesterday evening07:59
seb128didn't help to lower the temperature at all, still 27°C inside and 37°C forecasted today08:00
willcooke:(08:00
willcooke24 outside here, 28 in the house!!08:00
seb128lucky you!08:00
didrocks28.3°C inside already :(08:00
seb128go work in the garden :)08:01
didrocksit's already 28°C outside here08:01
Laneyyo08:01
seb128hey Laney, how are you?08:01
didrockshey Laney08:01
Laneyhey seb128 didrocks08:03
Laneygood thx, travelling to debconf later today08:03
Laneyyou?08:03
seb128oh right, debconf08:05
seb128safe travel!08:05
seb128I'm good, waiting for the rain that should arrive tonight/tomorrow and drop the temperature08:05
seb128Laney, you made version unhappy :/08:06
Laneyoh how?08:06
LaneyI ran it on my system and it worked08:07
seb128  File "versions.py", line 35, in <module>08:07
seb128    from ssl import CertificateError08:07
seb128ImportError: cannot import name CertificateError08:07
Laneymeh must be some version problem on that box08:07
seb128could have to do with the version on lillypilly08:07
Laneyprecise08:07
LaneyWTFFFFFFFFFFFFFFF08:07
seb128sorry :/08:08
didrockscontainers containers containers containers containers :)08:08
seb128troll day right?08:08
dufluEvery day is troll day08:09
* seb128 not falling into that one today, trying to get some work done while it's not too hot to think :p08:09
duflu(for trolls)08:09
didrocksnot a troll, just a why they exist :)08:10
dufluor channeling Steve Balmer08:10
didrocksok, that part for the troll one ;)08:10
seb128containers are not a justification to not update machines to a modern OS version though08:11
didrockswell, as long as it's supported, from an IS standpoint, I can understand… (and it's not rare)08:12
seb128yeah, well it's easy enough to08:12
seb128yeah, well it's easy enough to make that script work on precise08:12
didrocksyep08:12
seb128containers are a big hammer for that small nail08:12
didrocksright, but for a while, production was broken (which was OK for our use case)08:12
seb128because we don't have a "pull, try, revert if fail" machinery08:13
seb128containers don't protect you either from buggy commits08:13
seb128I mean they have use but are not a megical solution08:14
seb128anyway, I said no trolling :p08:14
Laney'buggy'08:14
didrockssounds like you want to troll ;) but at least testing on Laney's machine == what you deploy in production, that was my point08:14
seb128I'm not saying that one was buggy08:14
LaneyI guess ESM changes that precise is not properly supported08:14
seb128just that containers don't protect you from everything automagically08:14
seb128I was not speaking about that particular commit!08:15
didrocksdid I say that it was magic? ;)08:15
Laneyhaha08:15
seb128tendency of the modern world08:15
didrocksjust highlighting that in that case, it would have prevented that issue08:15
didrocksnothing more :)08:15
Laneyyeah then I could have run the docker script to check the actual environment it was going to be deployed in08:15
Laney;-)))))))))))))))))))))))))))))))))))08:16
seb128yeah, it's fine if that 100 lines script requires a 1GB of disk for the container and 3G of ram to have a webengine and a datacenter to run08:16
didrocksLaney: you are not even sure that -updates are installed on the machine08:16
seb128energy is infinite and why would you want to be efficient08:16
seb128waste for the win!08:16
didrocksseb128: most of containers, if properly done aren't that big :p08:16
didrocksbut it requires effort to streamline it, agreed08:17
seb128still more cpu and resources used that being native code on the host08:17
didrockstrue08:17
seb128but yeah modern world is not about optmization08:17
LaneyCPU used by me fixing this up now08:17
seb128it's about doing the less work and putting the cost on the infra08:17
Laneyand us talking about it08:18
Laney!!!!!!08:18
seb128haha08:18
Laneykeeping this light hearted troll friday08:18
seb128:)08:18
Laneyseems the version on this machine isn't validating SSL certificates then08:19
andyrockseb128: https://code.launchpad.net/~azzar1/ubuntu/+source/file-roller/+git/file-roller/+merge/35140708:39
andyrockseb128: I'm not sure it makes sense doing this in salsa08:39
andyrockdebian does not ship unstable gnome releaes08:40
andyrockand doing this in experimental for a non-DD takes too long08:40
seb128andyrock, thx08:44
seb128sorry channel for the stupid question08:44
seb128but do we really need 3 mps in those cases for the 3 branches (ubuntu / pristine-tar /  upstream)?08:45
andyrocki wanted to ask the same08:45
andyrockLaney: Trevinho ^^^08:45
didrocksI don't think there is any other way for a new release with the pristine-tar workflow08:45
LaneyYou don't need the merge proposal itself to be able to merge something from one place to another08:46
didrocks(at least, needing a way to pull/push pristine-tar & upstream)08:46
LaneyBut you need to do it of course08:46
LaneyMaybe merge requests help with reminding people to do that and maybe they don't, I don't know08:47
seb128well, our contributor workflow needs to be standard08:47
LaneyI'm not arguing with that at all08:47
LaneyI'm replying to your question about whether we need it08:47
andyrockit would be nice if we could have multi-branches MR in launchpad08:47
seb128do we have that in salsa?08:47
andyrockseb128: nope08:47
Laneyno08:47
seb128k08:48
seb128so we could define that contributors mp the ubuntu (or debian) branch08:48
seb128and that the reviewer needs to pull/merge pristine-tar and upstream as well08:48
seb128?08:48
Laneythat would work08:48
Laneyby the way "takes too long" I don't agree with, and certainly not in this case08:49
didrockswe still need to check the content of upstream/ though? If it's an external contributor08:49
seb128Laney, you can't really "disagree" with how people feel though... better to try to figure out what is making them feel like that08:50
seb128andyrock, what is your main issue there?08:50
seb128the fact that you can't branch to experimental in a mp?08:50
Laney:/08:50
seb128didrocks, I guess yes08:50
andyrockdidrocks: even if it's not an external contributor08:50
andyrockseb128: Laney the fact that I cannot use gitlab to propse things08:50
didrocksandyrock: you mean we have to double check on you? :p08:50
Laneythanks for the nitpicking pickup08:50
didrockswe know where you live08:50
Laneybut OK08:51
didrocks"Italy"08:51
didrocksshould be easy to find you :p08:51
andyrockdidrocks: of course you need to check on me08:51
Laneycan you propose on launchpad for non-existing branches?08:51
TrevinhoMorning...08:51
andyrockLaney: I don't think so08:51
didrockshey Trevinho08:52
LaneyI find it strange since I offered to sponsor this stuff08:52
Trevinhodidrocks: hi08:52
Laneybut whatever, getting updates in Debian is a battle I've been fighting for a long time08:52
Laneyjust do it in cosmic, that's fine08:52
seb128Laney, was that comment for me? sorry, I didn't mean to be negative08:52
seb128hey Trevinho08:52
andyrockLaney: my point is that debian never did .x[13579] releases08:53
TrevinhoCiao seb12808:53
andyrockciao Trevinho08:53
Laneyandyrock: you can do those in experimental, that's what I've been doing this week08:53
seb128andyrock, that's a non issue, Laney said he's wanting to sponsor those to experimental08:53
andyrockLaney: and the other point is that doing this in experimental without being a DD is a pain08:53
dufluHappy Friday Trevinho08:54
seb128andyrock, well, until we branched08:54
duflu and andyrock08:54
dufluand Laney08:54
dufluand people I missed08:54
seb128once the branch exists then it 's ok08:54
andyrockhey duflu08:54
andyrockseb128: indeed08:54
andyrockbeing not able to use gitlab MR feels :/ to me08:54
seb128I'm a bit lost as a reviewer there on what needs to be reviewed08:54
andyrockseb128: that's another thing08:55
seb128is there an easy way to check that the upstream branch that has been mp matches real upstream?08:55
seb128like a command/tool to use?08:55
Trevinhoyou too duflu08:55
andyrockseb128: it's not easy to review this kind of stuff :D08:55
seb128right08:55
dufluOr not. It's 5pm on a Friday and two fixes I thought were finished need reworking08:55
Trevinhoanyway I was saying the same to Andrea, last night... Using only ubuntu and in case Laney can pull to debian experimental08:56
seb128I start thinking it's more work to review/sponsor an update with this workflow that to do it themselve for the sponsors08:56
TrevinhoAnd we merge back once both on 3.3008:56
seb128like I don't even know how I would go to verify that the upstream branch hasn't been tricked08:56
andyrockseb128: I guess you can check the SHA https://gitlab.gnome.org/GNOME/file-roller/commits/3.29.108:56
seb128it's less effort to just do the gbp import-orig myself08:56
andyrockat least for upstream/latest08:57
Trevinhoseb128: git diff gnome/master08:57
LaneyTrevinho: why would you though if a DD (me) has said they wanted that in experimental?08:58
Laneyanyway I need to pack my bag, if it's easier for everyone to do it this way then feel free08:59
Laneybrb08:59
TrevinhoLaney: I meant what you can't propose (new branches), you can pull them if want09:03
LaneyI mean it's only easier in Launchpad because we happen to have an ubuntu/master branch, if this package was in sync it would be the same problem09:04
andyrockseb128: btw to check upstream/latest just do: git checkout upstream/latest; git diff 3.29.109:04
andyrockor git diff upstream/latest 3.29.109:05
seb128does that diff the disk content?09:05
seb128or would it tell you if a commit message was amended between the branches?09:05
TrevinhoAndyrock with .. I think09:05
Trevinhoseb128: you can also diff the history09:06
andyrockTrevinho: works the same here09:06
andyrockseb128: it would tell if I modified a file09:06
seb128Trevinho, that doesn't answer my question09:06
TrevinhoIn the mean time...09:06
Trevinhohttps://csorianognome.wordpress.com/2018/07/27/nautilus-3-30/09:06
seb128so diff is on disk content09:06
seb128not logs?09:06
andyrockseb128: to check if the history was properly imported just check the sha09:06
seb128k09:06
Trevinhoseb128: git diff yes... Checks rey content. Also the unstated one though09:07
TrevinhoUnstaged09:07
andyrocke.g. https://git.launchpad.net/~azzar1/ubuntu/+source/file-roller/commit/?id=12384cf98b64ef7a9f6970c454181fd34afedfa7 and https://gitlab.gnome.org/GNOME/file-roller/commit/12384cf98b64ef7a9f6970c454181fd34afedfa709:07
seb128I miss the good old bzr times where reviewing was only checking the debian dir was correctly changed :p09:07
seb128oh well, let's not start with those comments it's not helpful09:07
andyrockseb128: as you can see the sha is the same09:08
seb128andyrock, thx09:08
seb128I wonder if it would be easier to ask packaging debdiffs in that context09:08
seb128and let the sponsor do the gbp orig-import09:08
seb128and then copy the updated debian content over09:08
Laneynot in any way would that be easier for me09:08
seb128it's probably less work for everyone than having to deal with reviewing and merging 3 branches09:09
Laneyyou have to check you didn't mess up your own work anyway09:09
seb128well, if I do gbp import-orig myself I know I didn't meddle with the upstream content or history09:10
seb128so it's less intensive work than having to check a contributor upstream line09:10
seb128the one "interesting" part in the new version is the debian/ dir work, that's what is not automatically handled by the tools09:11
seb128e.g patches refreshes and maybe (build-)depends09:11
Laneyif you import a debdiff you get no proper commits09:11
Laneyyou get problems with transmitting data that diffs can't handle09:12
Laneythe contributor *and* the reviewer have to transport a patch out of git and back into it09:12
seb128k, either way it's work I guess :/09:13
andyrockit would be good btw do define standard ways to review this kind of stuff09:14
seb128yeah09:14
andyrockwhen I had to review Trevinho's work for nautilus I had to spend hours to understand that Marco was wrong :D09:14
seb128at this point I'm also unsure it makes sense to ask contributors to help on easy updates, it feels like it's more work for the sponsor to review/merge the stuff than to do the update (that's not true if there are patches to rebase though)09:14
seb128Trevinho, well done on getting mentioned on the nautilus .30 blog post :)09:15
LaneyI'm feeling a bit burned out by this whole exercise to be honest09:17
LaneyIt's really draining for me to have to advocate / fight for it all the time09:18
seb128yeah, sorry, I think we all feel burned in some way at this point09:19
seb128I'm not sure how to fix the situation though09:20
seb128the new workflow is powerful but that has a cost09:20
LaneyIt feels like you are fighting against it constantly, and I have to counter that09:20
seb128the cost is not that high for people who know the workflow/have commit access/see the advantage and are wanting to pay the cost09:20
seb128but it's high for others09:21
LaneyLike in this case, checking the branch is not high cost really, but we're jumping around questioning the whole thing09:21
LaneyIf the question stayed on "what is the best way to do this?"09:21
LaneyI could have showed you a screenshot of gitg09:21
seb128k, point taken09:22
seb128I'm going to step out of those discussions for the rest of the cycle09:24
seb128and see how the team goes with it, taking notes on the way09:24
seb128and we can do de-brief in Brussels09:24
seb128one fact is that doing an update is an higher number of commands and more error prone that the old workflow at this point (need to set the origns, don't forget to push 3 branches, tags, etc), maybe it's worth it though09:26
seb128maybe getting used to the system and building wrapper is going to help though09:26
seb128so let's people try to get used to it09:26
seb128k, on that note stepping out for early lunch, bbl09:29
LaneyThere's some learning / skill acquisition, yes. I'm not sure it's possible to avoid that but we should and have been trying to document it and make it as smooth as we can.09:31
seb128right, it's still a complex system and it shows09:32
seb128e.g see andyrock's feedback as a "new contributor"09:32
Laneywhat is the path out of this?09:32
seb128maybe the solution is to write some tooling/scripts to help with part of the repetitive work though09:32
seb128well, ether ^09:33
LaneyIt's a new thing for the team. We can't avoid people having to learn how to do some new things.09:33
LaneyHmm.09:33
seb128or maybe the conclusion is that the workflow with the 3 branches to maintain is too complex and not worth it, at least for "easy packages" and that ubuntu-git might be a better solution09:33
LaneyThis is getting me a bit irritated, I'm sorry, it's best I step out09:33
seb128k, sorry it's irritating to you09:34
seb128you see more the benefits from the system, where other see the complexity, we have probably work to do to see things from the others' perspective09:35
seb128and yes we can learn, but it's still error prone and a step learning curve for new contributors which mean be fine but we should still ask ourself the question if it's not overengineered09:36
seb128anyway, I'm stepping out of the discussion at this point09:36
seb128sorry it's frustating to you09:36
LaneyNo worries, maybe I really am blind to how crap it is.09:37
LaneyIf that's the case then it should definitely *not* be me arguing.09:37
andyrockmy point was not to irritate or to blame anyone. From a developer point of view the all process it's easy, but not from a reviewer09:40
andyrock*reviewer point of view09:40
andyrockgitg helps but I still feel we can have something better09:41
TrevinhoYes... And agree.09:46
TrevinhoHowever the tool itself from my side is waaaaay better. And doing things like all the backports from nautilus 3.28 to 3.26 without all the git tooling would have been just impossible (not easy anyway, but still possible and you've real tooling).09:47
TrevinhoI don't see why any request related how to do things or how to improve the work flow has to bring to irritation.09:49
TrevinhoIt's normal to discuss how to do things or what would en the best behavior. Without having to stop after 2 minutes saying that this is just too hard or too bad.09:49
TrevinhoThe time might invest now, might look like is lost, but I'm sure it will pay back in the future.09:50
TrevinhoSo.... Let's discuss, strongly on tech aspects if the case, but no need to complaining because a new system brings to that. It's quite normal.09:51
juliankHmm, in case we have two apps with the same name, one a snap and one a deb, would it make sense to show some badge or something on the icon to differentiate those?13:06
ograhmm, i suppose apt hasnt been tested much on single-core single-thread CPUs, has it ?13:09
ogra$ sudo apt update && sudo apt upgrade13:09
ogra...13:09
ograAll packages are up to date.13:09
ograE: Could not get lock /var/lib/dpkg/lock - open (11: Resource temporarily unavailable)13:09
ograE: Unable to lock the administration directory (/var/lib/dpkg/), is another process using it?13:09
ogra...13:09
ogra$13:10
ograi'm seeign this from time to time on an imx6ull (extremely low end ARM CPU)13:10
didrocksogra: I guess a question for mvo?13:10
jibelogra, it's usually the setup in test VMs13:10
ograif i call both commands separately it doesnt happen13:10
ogra(it also happens rarely (like 1 out of 100 calls or so) ... but i never see it anywhere else)13:11
didrocksmaybe apt doesn't call flush() when terminating?13:12
didrocksand so removal of the lock file isn't proceeded when the second command starts?13:12
didrocks(pure theory)13:12
ograjibel, well, thats x86 multi-threaded ... still like 100x faster than that ARM thing :)13:12
ogra(even when in single core mode)13:12
ogradidrocks, my theory was that apt simply operates async now ... the CPU easily blocks on IO so the second command might kick in while the backend is still busy with the first part13:14
didrocksogra: yeah, that's also plausible13:15
ograerr13:16
ogrageez ...13:16
ograwhy am i in -desktop !13:16
* ogra thought he was in -devel ... damned heat !!!13:17
willcookejuliank, it's on the backlog for consideration in future cycles13:23
willcookejuliank, i.e. yes, I think it would make a lot of sense13:23
juliankNice, thanks willcooke13:23
willcookeyw!13:23
jbichaogra: that issue also happens if you try to use apt shortly after booting because of a systemd timer checking for apt updates. If you wait several minutes, the background process finishes and the apt/dpkg locks are removed16:27
ograaha !16:28
ograjibel, thans for clearifying ... that makes sense16:28
ogra*thanks even16:28
jbichait looks like the timer also runs twice per day, but I notice it mostly when I boot an Ubuntu instance I had turned off16:30
jbichaseb128: hi, I saw LP: #1775329 on the sponsor queue. Do you think it's a good idea?16:38
ubot5Launchpad bug 1775329 in gnome-terminal (Ubuntu) "Feature request: Add a handler for CVE URLs" [Undecided,New] https://launchpad.net/bugs/177532916:38
willcookegnight all17:01
seb128jbicha, hey, no opinion on that, seems a wishlist/nice to have but maybe not worth a SRU18:10
seb128would be fine to include in a stable update with other changes though18:10

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