X2Go Bug report logs - #1068
x2goserver: Restart of x2goserver fails after upgrade because of misconfiguration in service-file

Toggle useless messages

Message #5 received at submit@bugs.x2go.org (full text, mbox, reply):

From: Carsten Kumke <ckumke@kumke.net>

To: submit@bugs.x2go.org

Subject: x2goserver: Restart of x2goserver fails after upgrade because of misconfiguration in service-file

Date: Mon, 08 Aug 2016 13:13:24 +0200

[Message part 1 (text/plain, inline)]

Package: x2goserver
Version: 4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
Severity: normal

Dear Maintainer,

I just upgraded x2goserver packages to 4.1.0.0-
0x2go1.2~git20160807.1349+9.heuler.1 on two systems. The installation
procedure ends up  on both machines  with a message thrown in 

systemctl status x2goserver

The message was: "x2goserver.service: PID file /run/x2goserver.pid not
readable (yet?) after start: No such file or directory"

   * What led up to the situation?

Simply apt-get dist-upgrade or (later) apt-get -f install

   * What exactly did you do (or not do) that was effective (or
     ineffective)?

Validating the fault I found that the
file /lib/systemd/system/x2goserver was wrong. 

Instead of line 7

PIDFile=/run/x2goserver.pid

it should read

PID-File=/run/x2goserver.pid

With this adjustment the server starts as expected. 

Yours sincerely

Carsten Kumke


-- System Information:
Debian Release: testing/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.6.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages x2goserver depends on:
ii  bash                   4.3-15
ii  bc                     1.06.95-9+b1
ii  debconf [debconf-2.0]  1.5.59
ii  libfile-basedir-perl   0.07-1
ii  libfile-which-perl     1.21-1
ii  libswitch-perl         2.17-2
ii  libtry-tiny-perl       0.24-1
ii  libx2go-server-perl    4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  lsof                   4.89+dfsg-0.1
ii  net-tools              1.60+git20150829.73cef8a-2
ii  openssh-client         1:7.2p2-8
ii  openssh-server         1:7.2p2-8
ii  psmisc                 22.21-2.1+b1
ii  pwgen                  2.07-1.1
ii  x2goagent              2:3.5.0.33-0x2go1~git20160710.753+9.heuler.1
ii  x2goserver-common      4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  xauth                  1:1.0.9-1
ii  xfonts-base            1:1.0.4+nmu1
ii  xkb-data               2.17-1

Versions of packages x2goserver recommends:
ii  fontconfig             2.11.0-6.4
ii  sshfs                  2.8-1
ii  x11-apps               7.7+6
ii  x11-session-utils      7.7+2
ii  x11-utils              7.7+3
ii  x11-xfs-utils          7.7+2
ii  x11-xkb-utils          7.7+3
ii  x11-xserver-utils      7.7+7
ii  x2goserver-extensions  4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  x2goserver-fmbindings  4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  x2goserver-printing    4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  x2goserver-xsession    4.1.0.0-0x2go1.2~git20160807.1349+9.heuler.1
ii  xinit                  1.3.4-3

Versions of packages x2goserver suggests:
ii  pulseaudio-utils  9.0-1.1
pn  rdesktop          <none>

-- Configuration Files:
/etc/sudoers.d/x2goserver [Errno 13] Keine Berechtigung:
u'/etc/sudoers.d/x2goserver'

-- debconf information:
  x2goserver/upgrade-4_1_0_0:
  x2goserver/postgresql-upgrade-3_1_0_0:

[Message part 2 (text/html, inline)]

Message #10 received at 1068@bugs.x2go.org (full text, mbox, reply):

From: Benjamin Shadwick <benshadwick@gmail.com>

To: 1068@bugs.x2go.org

Subject: x2goserver: Restart of x2goserver fails after upgrade because of misconfiguration in service-file

Date: Tue, 9 Aug 2016 21:07:26 -0700

PID-File doesn't look right to me. Several other services on my system
use PIDFile and none use PID-File.

I also see this in the log when I try PIDFile:

Aug 09 21:05:02 shodan systemd[1]:
[/lib/systemd/system/x2goserver.service:7] Unknown lvalue 'PID-File'
in section 'Service'

Message #15 received at 1068@bugs.x2go.org (full text, mbox, reply):

From: Mihai Moldovan <ionic@ionic.de>

To: 1068-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 1068@bugs.x2go.org

Subject: X2Go issue (in src:x2goserver) has been marked as pending for release

Date: Sun, 21 Aug 2016 13:26:24 +0200 (CEST)

tag #1068 pending
fixed #1068 4.0.1.20
thanks

Hello,

X2Go issue #1068 (src:x2goserver) reported by you has been
fixed in X2Go Git. You can see the changelog below, and you can
check the diff of the fix at:

    http://code.x2go.org/gitweb?p=x2goserver.git;a=commitdiff;h=4d7615e

The issue will most likely be fixed in src:x2goserver (4.0.1.20).

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
commit 4d7615e28ba22d456cdaa23e0e62235ae9dcc95b
Author: Mihai Moldovan <ionic@ionic.de>
Date:   Sun Aug 21 13:16:30 2016 +0200

    x2goserver/sbin/x2gocleansessions: move in-file POD to end of file, rather than "commenting" out all the remaining code. Fixes: #1068.

diff --git a/debian/changelog b/debian/changelog
index b37edae..0647cf9 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -94,6 +94,8 @@ x2goserver (4.0.1.20-0x2go1) UNRELEASED; urgency=low
       appending new data. Fixes: #923.
     - x2goserver-xsession/etc/Xsession: fix wrong variable usage.
       Fixes: #1073.
+    - x2goserver/sbin/x2gocleansessions: move in-file POD to end of file,
+      rather than "commenting" out all the remaining code. Fixes: #1068.
   * x2goserver.spec:
     - Add sudo and logcheck as BuildRequires and Requires. Don't own
       directories that are owned by sudo and logcheck. Logcheck is not

Message #27 received at 1068@bugs.x2go.org (full text, mbox, reply):

From: X2Go Release Manager <git-admin@x2go.org>

To: 1068-submitter@bugs.x2go.org

Cc: control@bugs.x2go.org, 1068@bugs.x2go.org

Subject: X2Go issue (in src:x2goserver) has been marked as closed

Date: Sun, 20 Nov 2016 12:43:00 +0100 (CET)

close #1068
thanks

Hello,

we are very hopeful that X2Go issue #1068 reported by you
has been resolved in the new release (4.0.1.20) of the
X2Go source project »src:x2goserver«.

You can view the complete changelog entry of src:x2goserver (4.0.1.20)
below, and you can use the following link to view all the code changes
between this and the last release of src:x2goserver.

    http://code.x2go.org/gitweb?p=x2goserver.git;a=commitdiff;h=71e1e82d82acc718da71301d2fbd67eb0a71ad44;hp=8a494e794402c1b124d861893d649726a9016eb7

If you feel that the issue has not been resolved satisfyingly, feel
free to reopen this bug report or submit a follow-up report with
further observations described based on the new released version
of src:x2goserver.

Thanks a lot for contributing to X2Go!!!

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
X2Go Component: src:x2goserver
Version: 4.0.1.20-0x2go1
Status: RELEASE
Date: Sun, 20 Nov 2016 12:39:36 +0100
Fixes: 799 812 888 889 903 922 923 985 1065 1068 1073
Changes: 
 x2goserver (4.0.1.20-0x2go1) RELEASED; urgency=low
 .
   [ Mike DePaulo ]
   * New upstream version (4.0.1.20):
     - TERMINAL Session: Add support for qterminal
       (Lightweight terminal emulator written in Qt)
     - TRINITY: Add support for Q4OS
 .
   [ Mihai Moldovan ]
   * New upstream version (4.0.1.20):
     - x2goserver/bin/x2gogetapps: respect NoDisplay and Hidden values, don't
       parse the full desktop file if there are non-Desktop Entries groups.
       Based on a patch submitted by Jason Alavaliant. Fixes: #812.
     - x2goserver/bin/x2gogetapps: do not print spurious newlines.
     - Change string "X2go" to "X2Go" where appropriate.
     - Add runcommand.d hooks to x2goserver-extensions Makefile.
     - x2goserver/bin/x2gosessionlimit: don't iterate over the whole groups
       file. Only check the primary and secondary groups as returned by
       getgid()/getgroups(). Fixes long delays in session startup on machines
       with a gazillion of groups.
     - x2goserver/sbin/x2gocleansessions: re-fetch the current sessions' status
       before handling it. Using the "cached" value from the initial
       x2golistsessions run may be desynchronized with the actual current
       value. Avoids a race condition logging users out after they attached to a
       session.
     - x2goserver/sbin/x2gocleansessions: rename %remembered_sessions_since to
       %remembered_sessions_status_since. That's what it really is. Add
       comments to further clarify what it's actually doing. Change it to
       include a timestamp instead of the "iteration count multiplied by two"
       value. Compare timestamps to see if killing hanging nxagent processes is
       necessary.
     - x2goserver/sbin/x2gocleansessions: rename
       %remembered_sessions_status_since to
       %remembered_sessions_status_since_time. Add
       %remembered_sessions_status_since_iterations that will come in handy for
       the next change.
     - x2goserver/sbin/x2gocleansessions: give suspended-but-running sessions a
       grace period of one iteration. Fixes: #888.
     - x2goserver/sbin/x2gocleansessions: fix ugly missing whitespace.
     - x2goserver/sbin/x2gocleansessions: move up session status sync a bit,
       don't update session state when session has been removed from the
       database.
     - x2goserver/sbin/x2gocleansessions: unify log messages. Move dangling
       line around.
     - x2goserver/sbin/x2gocleansessions: whitespace only.
     - x2goserver/sbin/x2gocleansessions: also delete entries in the other,
       overlooked hashes.
     - x2goserver/sbin/x2gocleansessions: move socket and lock file cleanup
       into time-based block for finished/failed sessions.
     - x2goserver/sbin/x2gocleansessions: add @remembered_finished_sessions
       array. Merge with session list as queried from the database (won't
       include finished/failed sessions) and remember F-state sessions via this
       array. Only delete elements once work on these has been finished.
     - x2goserver/sbin/x2gocleansessions: don't add failed/finished sessions to
       remembered list if that list already includes them.
     - x2goserver/sbin/x2gocleansessions: don't use gmtime (time ()) but merely
       time (). The former one returns a formatted date-time string, which
       can't be checked by forming a difference. The latter one returns an UNIX
       timestamp, which is what we really want... minus leap seconds. Let's
       hope this doesn't bite back some time or later...
     - x2goserver/sbin/x2gocleansessions: add debugging output to show the
       state change when updating the cached state result.
     - x2goserver/sbin/x2gocleansessions: only output status refreshing debug
       message if the old and new states actually differ. Prevents log spam.
     - x2goserver/bin/x2gostartagent: changes to Robert Nowotny's SSH_PORT
       patch. Fixes: #922. Use default outgoing interface to determine IP
       address. Use the whole range of ${RANDOM}'s pool. Seed it for good
       measure. Change the default method to randomization instead of IP-based
       initialization. If IP-based initialization was requested but the default
       outgoing IP address unavailable, fall back to randomization.
     - x2goserver/bin/x2goruncommand: whitespace and comment changes only.
     - x2goserver/bin/x2gocleansessions: replace system_capture_stdout_output()
       with actual implementation. We don't have that function on the release
       branch.
     - x2goserver-xsession/etc/Xsession: use /bin/bash explicitly on RedHat-,
       Gentoo- and SuSE-based systems. Users might change /bin/sh to point to a
       shell like mksh or dash, which do not support the -l or -c flags to
       exec. C.f., BGO #575022 (Gentoo.)
     - x2goserver/bin/x2gomountdirs: remove blowfish Cipher spec hardcoded for
       sshfs mounts. The speed improvement is probably mostly an illusion
       (unless talking about high-bandwidth links) and more importantly recent
       OpenSSH server versions seem to disable the Blowfish cipher by default,
       leading to errors like "Connection reset by peer."
     - x2goserver/bin/x2goruncommand: whitespace only.
     - x2goserver/sbin/x2gocleansessions: add Perl documentation and --help,
       -h, -? and --man options. Fixes: #1065. Also print a usage message on
       unknown options.
     - x2goserver/man/man8/x2gocleansessions.8: drop obsolete man page.
     - x2goserver-xsession/etc/Xsession: fix uname calls when (potentially)
       creating session log/error file and add uname call where necessary.
     - x2goserver-xsession/etc/Xsession: include hostname in
       xsession-x2go-errors file name. Part of #923.
     - x2goserver-xsession/etc/Xsession: rotate old log/error file before
       appending new data. Fixes: #923.
     - x2goserver-xsession/etc/Xsession: fix wrong variable usage.
       Fixes: #1073.
     - x2goserver/sbin/x2gocleansessions: move in-file POD to end of file,
       rather than "commenting" out all the remaining code. Fixes: #1068.
     - x2goserver/sbin/x2godbadmin: exit earlier when not finding the required
       "x2gouser" user.
   * x2goserver.spec:
     - Add sudo and logcheck as BuildRequires and Requires. Don't own
       directories that are owned by sudo and logcheck. Logcheck is not
       available on *SUSE, FC < 20 and RHEL, so for those distros still own the
       directories.
     - Typo fix in comment: Recommands -> Recommends.
     - Fix installation on OpenSUSE 12.3 and higher: has replaced pwdutils with
       shadow. Also affects SLE{S,D} 12.
     - Add workaround documentation for sudoers on *SUSE 11.
     - Add shadow/shadow-utils/pwdutils Requires to x2goserver-printing
       sub package.
     - Add new dependency on bash.
     - Add new dependency on xkeyboard-config to x2goserver. Fixes: #903.
     - Add new dependency on perl and the Cwd module to x2goserver-xsession.
       Part of #923.
   * debian/control:
     - Add new dependency on bash.
     - Add new dependency on xkb-data to x2goserver.
     - Maintainer change in package: X2Go Developers <x2go-dev@lists.x2go.org>.
     - Uploaders: add myself. Also, force a rebuild due to the changed
       versioning.
     - Add new dependency on perl-modules for Cwd. Part of #923.
 .
   [ Henning Heinold ]
   * New upstream version (4.0.1.20):
     - x2goserver/sbin/x2godbadmin: remove trailing whitespace.
     - x2goserver/sbin/x2godbadmin: revoke all PRIVILEGES before trying to
       delete a user on postgres. Fixes: #889.
 .
   [ Mike Gabriel ]
   * debian/x2goserver.preinst:
     + Use proper comment header explaining about preinst script argument
       calls (not postinst).
 .
   [ Robert Nowotny ]
   * New upstream version (4.0.1.20):
     - x2goserver/bin/x2gostartagent: original patch for SSH_PORT
       randomization/IP-based initialization. Submitted as #922.
 .
   [ Stefan Baur ]
   * New upstream version (4.0.1.20):
     - x2goserver/bin/x2goruncommand: add the other trinity launcher scripts in
       order from newest to oldest. Fixes: #985.
 .
   [ Tor Perkins ]
   * New upstream version (4.0.1.20):
     - x2goserver/bin/x2goruncommand: capture x2goruncommand debug output for
       x2gofeature X2GO_XSESSION.
 .
   [ Orion Poplawski ]
   * New upstream version (4.0.1.20):
     - x2goserver/bin/x2golistdesktops: drop -u parameter to ss. Fixes: #799.
       We would filter out UDP sockets later anyway and the combination -lxu is
       known to be broken with some versions of iproute2.

Send a report that this bug log contains spam.