~paelzer/ubuntu/+source/open-vm-tools:lp-1822204-backport-10.3.10-cosmic

Last commit made on 2019-05-14
Get this branch:
git clone -b lp-1822204-backport-10.3.10-cosmic https://git.launchpad.net/~paelzer/ubuntu/+source/open-vm-tools
Only Christian Ehrhardt  can upload to this branch. If you are Christian Ehrhardt  please log in for upload directions.

Branch merges

Branch information

Name:
lp-1822204-backport-10.3.10-cosmic
Repository:
lp:~paelzer/ubuntu/+source/open-vm-tools

Recent commits

767ebfd... by Christian Ehrhardt 

changelog: Backport recent open-vm-tools (LP: #1822204)

Signed-off-by: Christian Ehrhardt <email address hidden>

ee802e1... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.10-1 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 1842b04d1e6e9c93fbf52341910bd5f8dd923a22

New changelog entries:
  * [122e511] Update upstream source from tag 'upstream/10.3.10'
    Update to upstream version '10.3.10'
    with Debian dir fb12c7cfc99a9497795475c29306e78d08cc3712
    - Closes: #925940
    - Bugfix release for the 10.3 series.
      - Correct and/or improve handling of certain quiesced
        snapshot failures (shipped as patch in 2:10.3.5-6).
      - Fix some bad derefs in primary NIC gather code
      - Fix possible security issue with the permissions of the
        intermediate staging directory and path.
        Closes: #925959
      - CONSTANT_EXPRESSION_RESULT in TimeUtil_StringToDate()
        Found by coverity.
      - Deploypkg log files of linux should not be world readable.
        They might contain sensitive data.
      - General code clean-up:
        - Treat local variables "len" consistently as "size_t"
          type in Posix_Getmntent_r()
        - Improve readability of error handling logic in
          ShrinkDoWipeAndShrink() and remove another line of dead code.
        - Setting "errno" to ENOENT when there is no passwd entry
          for the user.
      - Fix NULL pointer dereference and remove three lines of dead code.
    - Other changes/fixes, not related to Debian:
      - Update copyright years
      - Fix CentOS 7.6 detection
      - Include vmware/tools/log.h to define g_info (fix for SLES)
      - Special-case profile loading for StartProgram
        (Win32 only)
      - Changes to common source files not applicable to
        open-vm-tools. (Code used by other vmware tools, unrelated
        to open-vm-tools).
      - Bump up the SYSIMAGE_VERSION for VMware tools 10.3.10

  * [18de70f] Removing backported patches, shipped in 10.3.10.

1842b04... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-8 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 728914f5b3056c04635f185cd7ac9efe31c0ee67

New changelog entries:
  [ Jean-Baptiste Lallement ]
  * [0f35aee] Add modaliases to open-vm-tools-desktop.
    Added Modaliases to open-vm-tools-desktop to auto-discover and
    auto-install the driver on Ubuntu via ubuntu-drivers. The driver is then
    installed at installation time and available on first boot for an
    improved user experience (LP: #1819207)
  [ Bernd Zeimetz ]
  * [dc4e1ce] Load vmwgfx module before vmtoolsd starts.
    As discussed on github in vmware/open-vm-tools#214
    we need to load the vmwgfx module before starting vmtoolsd
    for desktop users. Otherwise it is not able to retrieve the KMS
    resolutions and resizing the VM desktop fails.
    Thanks to @thomashvmw @rhertzog (Closes: #924518)

728914f... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-7 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 0eba19e3921ebed9ccb65b5c3fc1e49547cf504d

New changelog entries:
  [ Christian Ehrhardt ]
  * [71b468f] make vgauth service execution more reliable.
    Since d3d47039 "Start vgauth before vmtoolsd" there is a potential race
    of starting vgauth so early that it might have issues. This was
    discussed back in the day in [1] to [2], but confirmed to be ok by
    VMWare.
    We were all somewhat convinced by this, but a bad feeling remained not
    only with me but also with Bernd [4].
    A recent SRU review denial made me rethink all of it and I think we can
    make it safer without thwarting the purpose of the original change.
    Note: Disambiguation of service names used below:
    vgauth - open-vm-tools.vgauth.service
    vmtoolsd - open-vm-tools.service
    fs - systemd-remount-fs.service
    tmp - systemd-tmpfiles-setup.service
    cloud-init - cloud-init-local.service
    Currently we have these dependency requirements:
    - vgauth should be before vmtoolsd
    - cloud init should be before vmtoolsd
    - cloud init has to be really early in general
      - therefore this is using DefaultDependencies=No
    That lead to this graph:
     fs / tmp -> vmtoolsd -> cloud-init
    And d3d47039 added it to be like:
     fs / tmp -> vmtoolsd -> cloud-init
                   ^
     vgauth --|
    But there is no need to have vgauth without any pre-dependencies at all.
    It is only needed to be "before" vmtoolsd, therefore we can make it:
     fs / tmp -> vgauth -> vmtoolsd -> cloud-init
    That will make execution of vgauth much less error-prone (even though I
    have no hard issue to report) while at the same time holding up all
    known required ordering constraints.
    [1]: https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/comments/3
    [2]: https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/comments/12
    [3]: https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/comments/25
    [4]: https://github.com/bzed/pkg-open-vm-tools/pull/15#issuecomment-447237910
    Signed-off-by: Christian Ehrhardt <email address hidden>

0eba19e... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-6 to debian/sid

Imported using git-ubuntu import.

Changelog parent: d7cb7b259a7885c65b899ec74de6361a22a561a5

New changelog entries:
  * [43ec618] Correct and/or improve handling of certain quiesced
    snapshot failures.
    Thanks to Oliver Kurth (Closes: #921470)

d7cb7b2... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-5 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 66144680d163219de8e44d0509fe8bfcccfc7702

New changelog entries:
  * [54cce3e] Start vmtoolsd after apparmor.service.
    Github issue #17

6614468... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-4 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 7c2caae65ab9ca9aeb252be4f89c09c4238552f2

New changelog entries:
  [ Alf Gaida ]
  * [e13792d] udevadm trigger should not fail (Closes: #917642)

7c2caae... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-3 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 2b53b12d8efa8fa40407cfca2260c16bb28f256c

New changelog entries:
  [ Christian Ehrhardt ]
  * [d3d4703] Start vgauth before vmtoolsd.
    VGAuthService needs to be ready when vmtoolsd runs. Certain cases - e.g.
    Site Recovery Manager failover - will need vgauth to be up.
    Therefore add an After=vgauth.service dependency to open-vm-tools.service
    To have vgauth be able start early - and not pull cloud-init back late - it
    is also required to drop default dependencies which according to VMware is
    fine to do so.
    (LP: #1804287)

2b53b12... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-2 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 07f838b4af2be42125d61e877f56eab4e11a663c

New changelog entries:
  [ Raphaël Hertzog ]
  * [db2a364] Ensure vmwgfx module is loaded before start of vmtoolsd.
    This avoids a failure to start the resolutionKMS plugin and it's
    achieved through a drop-in snippet extending open-vm-tools.service
    adding an ExecStartPre directive loading the module prior to
    the start of the service. (Closes: #915031)
  [ Christian Ehrhardt ]
  * [e6e0ab8] d/rules: fix dangling symlink of vmware-user.
    Back in 2:9.4.0-1280544-6 vmware-user* was moved to the
    open-vm-tools-desktop package and some follow on fixes moved bits that
    were forgotten like the man page.
    (LP: #1807441)
    There still is a symlink in /usr/bin/vmware-user that is forgotten in
    the base package and broken unless open-vm-tools-desktop is installed.
    Change d/rules to move the symlink as well.
    Signed-off-by: Christian Ehrhardt <email address hidden>
  * [13d22e5] Breaks and Replaces for moving vmware-user.
    Signed-off-by: Christian Ehrhardt <email address hidden>
  * [d56826a] Bump breaks and replaces to next version to be released.
    Signed-off-by: Christian Ehrhardt <email address hidden>
  [ Bernd Zeimetz ]
  * [e4697c7] Fix race condition between open-vm-tools and systemd-tmpfiles-setup.
    Thanks to Jean-Louis Dupond (Closes: #914910)

07f838b... by Bernd Zeimetz

Import patches-unapplied version 2:10.3.5-1 to debian/sid

Imported using git-ubuntu import.

Changelog parent: 2095b474e734433d18afa6e60372218294e91ce0

New changelog entries:
  * [7061cb7] Update upstream source from tag 'upstream/10.3.5'
    Update to upstream version '10.3.5'
    with Debian dir 9315f58cab8ba1356c1e4aa77d714257dc0651f2