[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-04-16 Thread Launchpad Bug Tracker
This bug was fixed in the package open-vm-tools -
2:10.3.5-7~ubuntu0.18.10.1

---
open-vm-tools (2:10.3.5-7~ubuntu0.18.10.1) cosmic; urgency=medium

  * Backport recent open-vm-tools (LP: #1813944)
- also adresses handling of quiesced snapshot failures (LP: #1814832)
- also adresses issues with resolutionKMS plugins sometimes fails to
  load at boot (LP: #1818473)

open-vm-tools (2:10.3.5-7) unstable; urgency=medium

  [ 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 

open-vm-tools (2:10.3.5-6) unstable; urgency=medium

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

open-vm-tools (2:10.3.5-5) unstable; urgency=medium

  * [54cce3e] Start vmtoolsd after apparmor.service.
Github issue #17

open-vm-tools (2:10.3.5-4) unstable; urgency=medium

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

open-vm-tools (2:10.3.5-3) unstable; urgency=medium

  [ 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)

open-vm-tools (2:10.3.5-2) unstable; urgency=medium

  [ 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 
  * [13d22e5] Breaks and Replaces for moving vmware-user.
Signed-off-by: Christian Ehrhardt 
  * [d56826a] Bump breaks and replaces to next version to be released.
Signed-off-by: Christian Ehrhardt 

  [ Bernd Zeimetz ]
  * [e4697c7] Fix race condition between open-vm-tools and 
systemd-tmpfiles-setup.
Thanks to Jean-Louis Dupond (Closes: #914910)

open-vm-tools (2:10.3.5-1) unstable; urgency=medium

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

 -- 

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-04-16 Thread Launchpad Bug Tracker
This bug was fixed in the package open-vm-tools -
2:10.3.5-7~ubuntu0.18.04.1

---
open-vm-tools (2:10.3.5-7~ubuntu0.18.04.1) bionic; urgency=medium

  * Backport recent open-vm-tools (LP: #1813944)
- also adresses handling of quiesced snapshot failures (LP: #1814832)
- also adresses issues with resolutionKMS plugins sometimes fails to
  load at boot (LP: #1818473)

open-vm-tools (2:10.3.5-7) unstable; urgency=medium

  [ 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 

open-vm-tools (2:10.3.5-6) unstable; urgency=medium

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

open-vm-tools (2:10.3.5-5) unstable; urgency=medium

  * [54cce3e] Start vmtoolsd after apparmor.service.
Github issue #17

open-vm-tools (2:10.3.5-4) unstable; urgency=medium

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

open-vm-tools (2:10.3.5-3) unstable; urgency=medium

  [ 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)

open-vm-tools (2:10.3.5-2) unstable; urgency=medium

  [ 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 
  * [13d22e5] Breaks and Replaces for moving vmware-user.
Signed-off-by: Christian Ehrhardt 
  * [d56826a] Bump breaks and replaces to next version to be released.
Signed-off-by: Christian Ehrhardt 

  [ Bernd Zeimetz ]
  * [e4697c7] Fix race condition between open-vm-tools and 
systemd-tmpfiles-setup.
Thanks to Jean-Louis Dupond (Closes: #914910)

open-vm-tools (2:10.3.5-1) unstable; urgency=medium

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

 -- 

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-26 Thread Christian Ehrhardt 
I got this from Oliver (per Mail) after asking for a verification:
"the tests for vgauth (related to SRU) have passed. It's good to go."

Marking verified

** Tags removed: verification-needed verification-needed-bionic 
verification-needed-cosmic
** Tags added: verification-done verification-done-bionic 
verification-done-cosmic

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-11 Thread Oliver Kurth via ubuntu-bugs
@Bernd, just the fixes in the stable-10.3.5-quiesced-snapshot branch.
The other branch from 10.3.5 , stable-10.3.5-appmonitorlib was created
on a request by another OS vendor, and is not needed in other
distributions.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-11 Thread Bernd Zeimetz
@Oliver are there any other fixes that we should add? There are some
more branches in the github repository for 10.3.5 - is there anything
that needs to go into Debian buster?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-11 Thread Oliver Kurth via ubuntu-bugs
Thanks all for the update!

I asked our QE team to verify the SRM fix, I will let you know. I just
did a sanity check for the quiesced snapshot fix, it looks good.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-11 Thread Christian Ehrhardt 
Thanks Steve for accepting.

@Oliver - verifying the SRM case on Bionic and Cosmic is up to you since
you have the setup to do so.

** Changed in: open-vm-tools (Ubuntu Bionic)
 Assignee: (unassigned) => Oliver Kurth (okurth-1)

** Changed in: open-vm-tools (Ubuntu Cosmic)
 Assignee: (unassigned) => Oliver Kurth (okurth-1)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-08 Thread Steve Langasek
Hello Oliver, or anyone else affected,

Accepted open-vm-tools into bionic-proposed. The package will build now
and be available at https://launchpad.net/ubuntu/+source/open-vm-
tools/2:10.3.5-7~ubuntu0.18.04.1 in a few hours, and then in the
-proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested and change the tag from
verification-needed-bionic to verification-done-bionic. If it does not
fix the bug for you, please add a comment stating that, and change the
tag to verification-failed-bionic. In either case, without details of
your testing we will not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Changed in: open-vm-tools (Ubuntu Bionic)
   Status: Triaged => Fix Committed

** Tags added: verification-needed-bionic

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-08 Thread Steve Langasek
Hello Oliver, or anyone else affected,

Accepted open-vm-tools into cosmic-proposed. The package will build now
and be available at https://launchpad.net/ubuntu/+source/open-vm-
tools/2:10.3.5-7~ubuntu0.18.10.1 in a few hours, and then in the
-proposed repository.

Please help us by testing this new package.  See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how
to enable and use -proposed.  Your feedback will aid us getting this
update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug,
mentioning the version of the package you tested and change the tag from
verification-needed-cosmic to verification-done-cosmic. If it does not
fix the bug for you, please add a comment stating that, and change the
tag to verification-failed-cosmic. In either case, without details of
your testing we will not be able to proceed.

Further information regarding the verification process can be found at
https://wiki.ubuntu.com/QATeam/PerformingSRUVerification .  Thank you in
advance for helping!

N.B. The updated package will be released to -updates after the bug(s)
fixed by this package have been verified and the package has been in
-proposed for a minimum of 7 days.

** Changed in: open-vm-tools (Ubuntu Cosmic)
   Status: Triaged => Fix Committed

** Tags added: verification-needed verification-needed-cosmic

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-05 Thread Christian Ehrhardt 
After improving on what we have found triggered by the first SRU review this is 
ready again.
Changes:
- Despite being an MRE in general all associated individual bugs have full SRU 
templates now.
- maintainers correctly updated
- The change to the vgauth Dependencies is now safer

That said, this is uploaded to Bionic/Cosmic unapproved again and
waiting for SRU Team re-review.


** Changed in: open-vm-tools (Ubuntu Bionic)
   Status: Incomplete => Triaged

** Changed in: open-vm-tools (Ubuntu Cosmic)
   Status: Incomplete => Triaged

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-05 Thread Christian Ehrhardt 
Re-fixed with the sync of 10.3.5-7, rebasing the SRU onto that ...

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Bernd Zeimetz
Uploaded with 10.3.5-7

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
Accepted in Debian, I'll make it part of Disco and afterwards also part
of the SRU respin.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
Subscribing Bernd here as well to be aware of the re-started discussion on 
better dependencies.
I have filed a PR to the Debian packaging at [1] and will continue once the 
discussion on that concluded.

[1]: https://github.com/bzed/pkg-open-vm-tools/pull/18

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
Actually we made it "PartOf" maybe it inherits all we need already ...

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
No it does not, so we need the change that I envisioned - prepping that
as mentioned ...

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
** Description changed:

+ [Impact]
+ 
+  * Failover cases with Vmware SRM lead to an issue that vgauth is required 
+but started far too late to get the helpers that run on failover 
+working correctly.
+ 
+  * As requested by upstream we will make vgauth (providing auth) to run 
+before vmtoolsd (triggers and runs the code on failover)
+ 
+ [Test Case]
+ 
+  * Run a case of SRM driven failover - that is easier said than done, 
+therefore Oliver Kurth of VMWare and owner of the upstream
+open-vm-tools who has a setup with that will run (or have coworkers 
+run) the test on the PPA as well as the -proposed verification.
+ 
+ [Regression Potential]
+ 
+  * We had some concerns about being so early that it would fail (comments 
+#3 to #12) but got confirmed that would be ok. Never the less after our 
+first SRU discussion we went into a deeper analysis what we could add 
+to make the early DefaultDependencies=no start not too early without 
+thwarting the purpose of the change. Since those extra dependencies are 
+added the concerns about potential issues is rather low - the vgauth 
+really doesn't need much to be around and there is no known pain to 
+have it available early.
+ 
+ [Other Info]
+  
+  * Re-rolled and improved based on first SRU review - I have to thank 
+Steve for making me improve our first iteration on this!
+ 
+ 
+ 
+ 
  In Ubuntu 18.04, when the VM is booting with disconnected NICs after SRM
  (Site Recovery Manager) failover, the VGAuthService is initialized by
  vmtoold far later than deployPkg script execution + the hardcoded 100
  seconds timeout for deployPkg POSIX processes.
  
  This causes the SRM guest enrollment script to fail since it is not able
  to find running VGAuthService in its execution time duration.
  
  The issue is that vgauth starts much later than vmtoolsd. The systemd
  service should have a dependency on vmtoolsd. Adding this line:
  
  After=vgauth.service
  
  to the file /lib/systemd/system/open-vm-tools.service fixes the issue.
  This is also done in the package for Fedora (and hence RHEL and CentOS),
  see https://src.fedoraproject.org/rpms/open-vm-
  tools/blob/master/f/vmtoolsd.service .
  
  This is a critical issue for us. The issue probably exists in older
  Ubuntu releases as well. I will also file a bug with Debian.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-04 Thread Christian Ehrhardt 
Disambiguation of services in comments discussion on this bug:
vgauth - open-vm-tools.vgauth.service
vmtoolsd - open-vm-tools.service
fs - systemd-remount-fs.service
tmp - systemd-tmpfiles-setup.service

Summarizing the dependencies (and wishes for them) once more:
- open-vm-tools.vgauth.service is supposed to be before before 
open-vm-tools.service (for the issue outlined here)
- open-vm-tools.service needs to be before cloud-init-local.service (to get 
datasources)

As discussed on the bug between comment #3 and #18 we expressed our
concerns about DefaultDependencies=no but were somewhat convinced by
testing that it might work, but still felt unsure.

After all even cloud-init-local has at least
  After=systemd-remount-fs.service


But then almost exactly as we were afraid of in general issues were found for 
open-vm-tools.service and this was added:
  After=systemd-remount-fs.service systemd-tmpfiles-setup.service
(one for issues we had with /tmp just as we assumed above in e.g. comment #12, 
and one for systemd-remount-fs just as cloud-init-local)
That was for Debian bug #914910 and is in 
https://github.com/bzed/pkg-open-vm-tools/commit/e4697c7


Now IMHO this same safety net dependencies should be added to 
open-vm-tools.vgauth.service as well to not find an issue there.

Eventually that should be no issue as - summarizing again with the above in 
mind:
- open-vm-tools.vgauth.service is supposed to be before before 
open-vm-tools.service (for the issue outlined here)
- open-vm-tools.service needs to be before cloud-init-local.service (to get 
datasources)
- cloud-init-local.service has DefaultDependencies=no but 
After=systemd-remount-fs.service
- open-vm-tools.service got this against races
  After=systemd-remount-fs.service systemd-tmpfiles-setup.service

That already makes it:
fs / tmp -> vmtoolsd -> cloud-init
Now throwing in the requirement to have vgauth to be before vmtoolsd means we 
can add fs / tmp dependencies there to be on the safe side without changing the 
current dependency chain.

It would then be
fs / tmp -> vgauth -> vmtoolsd -> cloud-init

I agree that this will make it feel more correct and as outlined will
not thwart the intention of the "get vmtools before vmtoolsd" change in
the consideration of cloud-init-local needing to be early but also after
open-vm-tools.

Let me suggest that to Bernd to make it right in Buster as well ...

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-03 Thread Christian Ehrhardt 
Hi Steve, thanks for the review - let me try to clarify:
IMHO it is mentioned in the changelog:
  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.
But I agree that it isn't properly SRU described in this bug here was it was a 
change that was pulled in by the "-sa -v" and from further down 
the changelog - that was my fault, sorry - prepping it better for a re-upload.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-01 Thread Steve Langasek
This is a packaging change in an SRU, and therefore requires a separate
SRU template with test case, independent of the exception granted to the
new upstream version of this package.

** Changed in: open-vm-tools (Ubuntu Cosmic)
   Status: Triaged => Incomplete

** Changed in: open-vm-tools (Ubuntu Bionic)
   Status: Triaged => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-03-01 Thread Steve Langasek
Also the vgauth service is now setting DefaultDependencies=no which is
evidently related to this change but not described by this bug.
DefaultDependencies=no requires a great deal of care and at first glance
this change looks INCORRECT because there are no manual dependencies
added in place of the default dependencies - it's almost certainly wrong
to say that vgauth has NO dependencies (at minimum: local-fs.target?)

More due diligence is definitely required here to demonstrate the
correctness of the changes.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-01-30 Thread Christian Ehrhardt 
FYI: The backport of this will be part of bug 1813944

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-01-30 Thread Launchpad Bug Tracker
** Merge proposal linked:
   
https://code.launchpad.net/~paelzer/ubuntu/+source/open-vm-tools/+git/open-vm-tools/+merge/362461

** Merge proposal linked:
   
https://code.launchpad.net/~paelzer/ubuntu/+source/open-vm-tools/+git/open-vm-tools/+merge/362462

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2019-01-30 Thread Christian Ehrhardt 
** Also affects: open-vm-tools (Ubuntu Cosmic)
   Importance: Undecided
   Status: New

** Also affects: open-vm-tools (Ubuntu Bionic)
   Importance: Undecided
   Status: New

** Changed in: open-vm-tools (Ubuntu Bionic)
   Status: New => Triaged

** Changed in: open-vm-tools (Ubuntu Cosmic)
   Status: New => Triaged

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-16 Thread Christian Ehrhardt 
Thanks to Bernd my MP was accepted and is in latest Debian and synced to 
Ubuntu-Disco now.
While I'm sort of confident on the change, I'd say we'll keep the new 10.3.5 in 
Disco until after the Christmas break and then take a look at the backports to 
18.04 and 18.10.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-14 Thread Launchpad Bug Tracker
This bug was fixed in the package open-vm-tools - 2:10.3.5-3

---
open-vm-tools (2:10.3.5-3) unstable; urgency=medium

  [ 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)

 -- Bernd Zeimetz   Fri, 14 Dec 2018 09:55:04 +0100

** Changed in: open-vm-tools (Ubuntu)
   Status: Triaged => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-13 Thread Christian Ehrhardt 
To help Debian just as much and since we are currently in sync (Thanks
Bzed for taking all my changes) I submitted that - after our discussion
and verification here - to Debian at: https://github.com/bzed/pkg-open-
vm-tools/pull/15

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-13 Thread Christian Ehrhardt 
There was no Debian bug (that I'd know of) filed yet, but the
description says "I will also file a bug with Debian". If there is any
let me know to link the PR with it and this bug.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-13 Thread Christian Ehrhardt 
** Changed in: open-vm-tools (Ubuntu)
   Status: Incomplete => Triaged

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-12 Thread Oliver Kurth
Okay, the packages have been tested by the SRM team. It works as
expected now.

Thanks for fixing this!

Also, updates on Scott's questions:
a) vgauth isn't always required, but to avoid issues when it is it's better to 
have it running always. However, it is already enabled by default. What if a 
user wants to disable it because it's not needed? With a "Requires" it would be 
started anyway.

b) /tmp isn't needed for vgauth, so the line "RequiresMountsFor=/tmp"
can be removed. I guess it has been added to vgauth.service when it was
added to open-vm-tools.service. It's needed for the latter, but not the
former.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-10 Thread Oliver Kurth
I pinged the team again to test these packages. Overall, the packages seem 
fine, as tested by 
Yuhua Zou on our email thread, quoted below:

It works well when do sanity check for open-vm-tools 10.3.5 from ppa:
ci-train-ppa-service/3530

1. Check in the following VMs

disco-server-cloudimg-amd64.ova daily build
ubuntu-18.10-Desktop-amd64 
ubuntu-18.10-server-amd64 
ubuntu-18.10-live-server-amd64
ubuntu-18.04-Desktop-amd64 
ubuntu-18.04-server-amd64 
ubuntu-18.04-live-server-amd64
 

2. Check items:

install / upgrade open-vm-tools from ppa: ci-train-ppa-service/3530
tools service and VGAuth service are running when install / upgrade 
open-vm-tools
tools service and VGAuth service are running after install / upgrade and reboot 
guestOS
uninstall open-vm-tools 10.3.5
 

3. Note:

1) Not check with daily build disco-server-amd64.iso because I
can’t install guestOS with this ISO successfully.

 It can’t pass the stage “Configure the package manager” during the
installing process.

2) I didn’t check the boot sequence between vmtoold and VGauth
services. So the issue https://bugs.launchpad.net/ubuntu/+source/open-
vm-tools/+bug/1804287

still need Oliver’s verification.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-10 Thread Christian Ehrhardt 
Yeah - I went a bit back and forth on it, but let us split this
discussion from the current merge for Ubuntu 19.04. There can always be
a follow on upload once this discussion is fully complete and verified.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-10 Thread Christian Ehrhardt 
Any update on deep-testing and verifying this one?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-10 Thread Christian Ehrhardt 
Alternative - complete the merge of 10.3.5 without this change for now
to un-couple it - opinions?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-07 Thread Oliver Kurth
I forwarded those questions to the developer for vgauth.

For a) my take is that vmtoolsd runs fine without vgauth, only when
special authentication is required is it needed. So I am not sure if
Requires=vgauth.service' is a good idea.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-12-07 Thread Scott Moser
@Oliver,

I have 2 questions about adding 'After=vgauth.service' to open-vm-tools.service
and adding 'DefaultDependencies=no' to vgauth.service.

a.) if open-vm-tools.service requires vgauth.service, then shouldn't we also
add a 'Requires=vgauth.service' ?
b.) Your MP adds RequiresMountsFor=/tmp to vgauth.service.  That would seem to 
imply that vgauth.service will *use* /tmp.  As we found in bug 1707222,
use of /tmp during boot is not safe. some systemd jobs might delete the
files while you're using them.  So.. the question is then how is
vgauth.service using /tmp and/or how is it avoiding a race condition.


** Changed in: open-vm-tools (Ubuntu)
   Status: Triaged => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-29 Thread  Christian Ehrhardt 
2:10.2.0-3~ubuntu0.16.04.1 in Xenial might be affected as well, Trusty is just 
very different in terms of startup constraints. But lets focus if it is even 
working fine before going further.
I'm still afraid making vgauth so early might case rare but severe issues - 
really try to triple check that on the verification.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-29 Thread Oliver Kurth
Thanks Christian!

Can you tell what versions of the package are affected? I'd expect all
older versions have this issue, but I am not sure since it hasn't been
reported before.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-29 Thread  Christian Ehrhardt 
Well, due to this being a bit unclear yet I didn't prep the PPA for all 
versions.
But if it helps in depth testing I can certainly do so.

I built the same open-vm-tools code for 18.04/18.10 in the same PPA that was 
referenced.
So any version >18.04 can be tested against the proposed changes including 
those Dependency changes.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-28 Thread Oliver Kurth
Thanks for addressing this issue.

The packages are built for Ubuntu/disco (19.04) - any chance these can
be built with bionic?

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-27 Thread  Christian Ehrhardt 
Subscribed John, Oliver and Yuhua who usually work with us to coordinate
and verify open-vm-tools

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-27 Thread  Christian Ehrhardt 
After discussing with smoser it seems we agree that we expect some potential 
issues.
Eventually it comes down to almost "chaning-any-dependency-anywhere-in-systemd 
has issues" :-)

In this particular case we are concerned that vgauth.service might have
issues as it has to start much earlier than usual.

That is different to the "Fedora has the same" argument, as they don't
have open-vm-tools.service not that early for cloud-init. Due to that
"pulling vgauth.service" before "open-vm-tools.service" is a bigger
change for Ubuntu one should be aware of.

As I said in comment #5 I have combined the requested change in PPA [1]
and it might come down to VMWare to verify that not only it fixes this
bug here, but also has no unwanted side effects.

Call for action:
- please verify the proposed package in [1] as part of our agreement to verify 
new versions
- while doing so please focus additionally on the services changed here 
open-vm-tools.service and vgauth.service to be correct
- please also verify that it really fixes the issue you opened this bug for

In addition I'll send a mail about that.

[1]: https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/3530

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-22 Thread  Christian Ehrhardt 
I'm waiting for comments on my concerns for the dependencies.
If anybody wants to take a more practical approach there are test builds in [1].

[1]: https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/3530

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-22 Thread Launchpad Bug Tracker
** Merge proposal linked:
   
https://code.launchpad.net/~paelzer/ubuntu/+source/open-vm-tools/+git/open-vm-tools/+merge/359184

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-22 Thread  Christian Ehrhardt 
Also let me know when you have a Debian bug, I want to outline my concerns 
there as well.
If you don't mind, copy my thoughts here to the Debian bug when opening it.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-22 Thread  Christian Ehrhardt 
I'm especially concerned about currently having intentionally (to be
super early)

open-vm-tools.service:
  DefaultDependencies=no
  Before=cloud-init-local.service

Which means as early as possible (no other dependencies) and in front of cloud 
init.
Which due to the suggested:
  After=vgauth.service
would effectively make open-vm-tools.service to be after DefaultDependencies 
again.

@Rharper / @Smoser - will that be a problem?
If so any suggestions how to resolve while still getting the request filed here 
considered?

Not sure if vgauth.service can work with "DefaultDependencies=no" if
that is the outcome to be needed.

Oliver - please share what you think as well for this discussion.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-22 Thread  Christian Ehrhardt 
Hi Oliver,
I can see the case that you are outlining.

I would consider this getting a part of my work on 10.3.5 which I start
now and eventually will be backported back to the latest LTS as you
know.

But we've had enough issues with these dependencies in the past, so once I have 
a test version I'd need:
- vmware to check the solution for the fix here
- vmware to do a full regression check (as usual, I'll send the mails)
- our cloud-init people to review and ack and test this as well as it will in 
turn push cloud-init-local.service to be much later and I want to be sure this 
is no problem.

** Changed in: open-vm-tools (Ubuntu)
   Status: Incomplete => Triaged

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 1804287] Re: vgauth needs to start before vmtoolsd

2018-11-21 Thread Karl Stenerud
Hi Oliver,

Thank for reporting this bug. Can you put together a repro case for this
issue so that we can verify any fixes? In a VM or container is best so
that we can be sure of a clean environment.

** Changed in: open-vm-tools (Ubuntu)
   Status: New => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1804287

Title:
  vgauth needs to start before vmtoolsd

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/open-vm-tools/+bug/1804287/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs