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
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
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
@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,
@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.
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.
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:
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
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
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,
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
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:
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
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
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
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
** 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
+
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
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
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
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
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
** 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
** 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
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
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
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
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.
** 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
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
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
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
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:
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
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
@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
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
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.
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
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
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
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
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,
** 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
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.
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:
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
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
48 matches
Mail list logo
