[X2Go-Dev] GitLab Install LXC - Update (issue)
Hi all, I tried to omnibus GitLab install on the VM. Failed on: Expected process to exit with [0], but received '255' Begin output of sysctl -e -p /opt/gitlab/embedded/etc/90-omnibus-gitlab-kernel.shmmax.conf STDOUT: STDERR: sysctl: setting key "kernel.shmmax": Read-only file system End output of sysctl -e -p /opt/gitlab/embedded/etc/90-omnibus-gitlab-kernel.shmmax.conf Ran sysctl -e -p /opt/gitlab/embedded/etc/90-omnibus-gitlab-kernel.shmmax.conf returned 255 Due to: command "sysctl -e -p /opt/gitlab/embedded/etc/90-omnibus-gitlab-kernel.shmmax.conf" causing the error: sysctl: setting key "kernel.shmmax": Read-only file system Read-only of the /sys/ directory is typical the case with LCX containers... But the advice value of the shmax is: cat /opt/gitlab/embedded/etc/90-omnibus-gitlab-kernel.shmmax.conf kernel.shmmax = 17179869184 I can complete the setup if you want, by using a workaround by commenting-out the related section in: /opt/gitlab/embedded/cookbooks/cache/cookbooks/package/resources/gitlab_sysctl.rb. And run: gitlab-ctl reconfigure. But I rather like to get this working in a correct manner. Maybe somebody add at this shmmax statement above to the host /etc/sysctl.conf file? Let me know if somebody can help me with this or which direction you want to go. Ps. where are not the only one on the internet: http://letmegooglethat.com/?q=STDERR%3A+sysctl%3A+setting+key+%22kernel.shmmax%22%3A+Read-only+file+system Thx! Kind regards, Melroy van den Berg___ x2go-dev mailing list x2go-dev@lists.x2go.org https://lists.x2go.org/listinfo/x2go-dev
[X2Go-Dev] Bug#1476: Bug#1476: x2goclient interface code needs refactoring to make it more accessible and make improvements easier to implement and test
Cheers Mike. No problem. On Mon, 8 Jun 2020, 08:54 Mike Gabriel, wrote: > Hi Dave, > > On So 07 Jun 2020 17:49:21 CEST, Dave Chamberlin-Kidd wrote: > > > Package: x2goclient > > Version: 4.1.2.3 > > Severity: wishlist > > Tag: patch > > > > There are a number of minor issues, oddities and limitations with the > > current x2goclient gui which would be good to fix, resolve and improve > on. > > Like all projects however the core team have more important issues to > > resolve. The code in its current state is difficult to understand and > amend > > and as it has no tests it is risky for developers to make functional > > changes unless they spend a lot of time getting to know the code and > doing > > extensive manual testing. > > > > Attached are 10 small refactorings which start extracting out some of the > > code into smaller functions. As the code is separated out it becomes > easier > > to understand and change. > > > > There are no functional changes in these patches, nor do they attempt or > > claim to fix any known issues. On their own the benefit of these patches > is > > limited however the hope is that over time I will be able to submit more > > refactorings as I learn more and then start fixing issues and improving > the > > UI. Extracting out functions is just one of many techniques that can my > experience it's a good one to start > > with. It should also be noted that sometimes code is extracted "as is" > even > > when it is clear further refactoring can be done to improve things, this > is > > by design in order to keep risk and size of patches to a minimum. > > > > Cheers > > Dave > > Thanks for your patches. As the GitLab server for X2Go is in reach, > I'd like to postpone patch review until its there. I'd be happy if you > could file a merge request then and we dive into your proposals. > > Thanks+Greets, > Mike > -- > > DAS-NETZWERKTEAM > c\o Technik- und Ökologiezentrum Eckernförde > Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde > mobile: +49 (1520) 1976 148 > landline: +49 (4351) 850 8940 > > GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22 0782 9AF4 6B30 2577 1B31 > mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de > > ___ x2go-dev mailing list x2go-dev@lists.x2go.org https://lists.x2go.org/listinfo/x2go-dev
[X2Go-Dev] Bug#1476: Bug#1476: x2goclient interface code needs refactoring to make it more accessible and make improvements easier to implement and test
Hi Dave, On So 07 Jun 2020 17:49:21 CEST, Dave Chamberlin-Kidd wrote: Package: x2goclient Version: 4.1.2.3 Severity: wishlist Tag: patch There are a number of minor issues, oddities and limitations with the current x2goclient gui which would be good to fix, resolve and improve on. Like all projects however the core team have more important issues to resolve. The code in its current state is difficult to understand and amend and as it has no tests it is risky for developers to make functional changes unless they spend a lot of time getting to know the code and doing extensive manual testing. Attached are 10 small refactorings which start extracting out some of the code into smaller functions. As the code is separated out it becomes easier to understand and change. There are no functional changes in these patches, nor do they attempt or claim to fix any known issues. On their own the benefit of these patches is limited however the hope is that over time I will be able to submit more refactorings as I learn more and then start fixing issues and improving the UI. Extracting out functions is just one of many techniques that can be used to make code cleaner but in my experience it's a good one to start with. It should also be noted that sometimes code is extracted "as is" even when it is clear further refactoring can be done to improve things, this is by design in order to keep risk and size of patches to a minimum. Cheers Dave Thanks for your patches. As the GitLab server for X2Go is in reach, I'd like to postpone patch review until its there. I'd be happy if you could file a merge request then and we dive into your proposals. Thanks+Greets, Mike -- DAS-NETZWERKTEAM c\o Technik- und Ökologiezentrum Eckernförde Mike Gabriel, Marienthaler Str. 17, 24340 Eckernförde mobile: +49 (1520) 1976 148 landline: +49 (4351) 850 8940 GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22 0782 9AF4 6B30 2577 1B31 mail: mike.gabr...@das-netzwerkteam.de, http://das-netzwerkteam.de pgpHeGPMZNSbZ.pgp Description: Digitale PGP-Signatur ___ x2go-dev mailing list x2go-dev@lists.x2go.org https://lists.x2go.org/listinfo/x2go-dev