Skip to content

Conversation

@jemoreira
Copy link
Member

Bug: b/321795781

@jemoreira jemoreira enabled auto-merge December 31, 2025 03:31
@jemoreira jemoreira added the kokoro:force-run Trigger a presubmit build unconditionally. label Dec 31, 2025
@GoogleCuttlefishTesterBot GoogleCuttlefishTesterBot removed the kokoro:force-run Trigger a presubmit build unconditionally. label Dec 31, 2025
The main differences come from abseil not having a single logger object
and not providing access to the already configured log sinks.
This change mostly replaces the included headers and turns LOG(DEBUG)
and LOG(VERBOSE) into VLOG(0) and VLOG(1) respectively.
This variable was instructing subprocesses to log at verbose level by
default, but it was then overriden in most cases by setting the stderr
logger to info and the file logger to verbose.
It was logging at INFO level only to fetch.log.

Removes handling of the verbosity flag at the fetch command handler
level and leaves it at cvd level only.
@jemoreira jemoreira added this pull request to the merge queue Dec 31, 2025
Merged via the queue into google:main with commit 415f008 Dec 31, 2025
22 checks passed
@jemoreira jemoreira deleted the absllog branch December 31, 2025 20:28
Databean added a commit to Databean/android-cuttlefish that referenced this pull request Jan 2, 2026
This was missed in google#1955 .

Bug: b/321795781
Databean added a commit to Databean/android-cuttlefish that referenced this pull request Jan 3, 2026
This was missed in google#1955 .

Changing the log header additionally caused a new unrelated error, which
is also fixed:
```
In file included from allocd/alloc_utils.cpp:16:
allocd/alloc_utils.h:28:10: error: module //allocd:alloc_utils does not depend on a module exporting 'cuttlefish/common/libs/fs/shared_fd.h'
   28 | #include "cuttlefish/common/libs/fs/shared_fd.h"
      |          ^
1 error generated.
```

Bug: b/321795781
github-merge-queue bot pushed a commit that referenced this pull request Jan 3, 2026
This was missed in #1955 .

Changing the log header additionally caused a new unrelated error, which
is also fixed:
```
In file included from allocd/alloc_utils.cpp:16:
allocd/alloc_utils.h:28:10: error: module //allocd:alloc_utils does not depend on a module exporting 'cuttlefish/common/libs/fs/shared_fd.h'
   28 | #include "cuttlefish/common/libs/fs/shared_fd.h"
      |          ^
1 error generated.
```

Bug: b/321795781
Databean added a commit to Databean/android-cuttlefish that referenced this pull request Jan 6, 2026
github-merge-queue bot pushed a commit that referenced this pull request Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants