New issue
Advanced search Search tips

Issue 869700 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

run lxd with syslog

Project Member Reported by smbar...@chromium.org, Aug 1

Issue description

LXD logs should be surfaced via syslog to be available in CrOS's syslog. 

Add support for this and send upstream.
 
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 2

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/d35f39e006fad23abc92721034e63461c15b646d

commit d35f39e006fad23abc92721034e63461c15b646d
Author: Stephen Barber <smbarber@chromium.org>
Date: Thu Aug 02 20:53:33 2018

vm_tools: maitred: invoke lxd with syslog option

BUG= chromium:869700 
TEST=start termina VM; check for lxd logs
CQ-DEPEND=CL:1157879

Change-Id: Ibdb9a0a90cca003ae8c0baa36c075b1434c10307
Reviewed-on: https://chromium-review.googlesource.com/1157880
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[modify] https://crrev.com/d35f39e006fad23abc92721034e63461c15b646d/vm_tools/maitred/service_impl.cc

Status: Assigned (was: Untriaged)
Labels: Merge-Request-69
Status: Fixed (was: Assigned)
Verified on 10944.0.0. Requesting merge to M69; this may help track down failures starting LXD.
Project Member

Comment 5 by sheriffbot@chromium.org, Aug 6

Labels: -Merge-Request-69 Merge-Review-69 Hotlist-Merge-Review
This bug requires manual review: M69 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-69 Merge-Approved-69
Merge approved, M69.
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 9

Labels: merge-merged-release-R69-10895.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/a3bc743d610797bdf127087744d4097a35d18499

commit a3bc743d610797bdf127087744d4097a35d18499
Author: Stephen Barber <smbarber@chromium.org>
Date: Thu Aug 09 21:10:43 2018

app-emulation/lxd: add syslog patch

BUG= chromium:869700 
TEST=start termina VM

Change-Id: I661d24f342c8a9e27d6d2535b0cc173a745a1a43
Reviewed-on: https://chromium-review.googlesource.com/1157879
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>
(cherry picked from commit f6ff76e343ee2257e7342aec04d192870fc4f52b)
Reviewed-on: https://chromium-review.googlesource.com/1169981
Reviewed-by: Stephen Barber <smbarber@chromium.org>
Commit-Queue: Stephen Barber <smbarber@chromium.org>

[rename] https://crrev.com/a3bc743d610797bdf127087744d4097a35d18499/app-emulation/lxd/lxd-3.0.0-r2.ebuild
[add] https://crrev.com/a3bc743d610797bdf127087744d4097a35d18499/app-emulation/lxd/files/3.0.0-lxd-expose-syslog-flag-via-CLI.patch
[modify] https://crrev.com/a3bc743d610797bdf127087744d4097a35d18499/app-emulation/lxd/lxd-3.0.0.ebuild

Project Member

Comment 8 by bugdroid1@chromium.org, Aug 9

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/3537fe7b7e359547cfe02b81ebe421c1f8f3b911

commit 3537fe7b7e359547cfe02b81ebe421c1f8f3b911
Author: Stephen Barber <smbarber@chromium.org>
Date: Thu Aug 09 21:24:25 2018

vm_tools: maitred: invoke lxd with syslog option

BUG= chromium:869700 
TEST=start termina VM; check for lxd logs
CQ-DEPEND=CL:1157879

Change-Id: Ibdb9a0a90cca003ae8c0baa36c075b1434c10307
Reviewed-on: https://chromium-review.googlesource.com/1157880
Commit-Ready: Stephen Barber <smbarber@chromium.org>
Tested-by: Stephen Barber <smbarber@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>
(cherry picked from commit d35f39e006fad23abc92721034e63461c15b646d)
Reviewed-on: https://chromium-review.googlesource.com/1170042
Reviewed-by: Stephen Barber <smbarber@chromium.org>
Commit-Queue: Stephen Barber <smbarber@chromium.org>

[modify] https://crrev.com/3537fe7b7e359547cfe02b81ebe421c1f8f3b911/vm_tools/maitred/service_impl.cc

Labels: -Merge-Approved-69

Sign in to add a comment