New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 740761 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

ap-daemons ebuild likely missing a dependency

Project Member Reported by akes...@chromium.org, Jul 10 2017

Issue description

ap-daemons failed to build on first attempt, and was retried, in https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/no_vmtest_pre_cq/72483

ap-daemons-0.0.3-r3008: /build/whirlwind/usr/share/proto: warning: directory does not exist.
ap-daemons-0.0.3-r3008: 
ap-daemons-0.0.3-r3008: [11/288] ACTION Generating DBus proxies C++ headers from ../ap/wireless/wifiblaster/dbus_bindings/org.chromium.ap.Wifiblaster.xml
ap-daemons-0.0.3-r3008: [12/288] ACTION Generating DBus proxies C++ headers from ../ap/wireless/wifiblaster/dbus_bindings/org.chromium.ap.Wifiblaster.xml
ap-daemons-0.0.3-r3008: [12/288] CXX platform/ap-daemons/diagnostics/resource-monitor.resource_monitor.o
ap-daemons-0.0.3-r3008: [13/288] CXX platform/ap-daemons/controller/tools/psk-generator.psk_generator.o
ap-daemons-0.0.3-r3008: [13/288] AR platform/ap-daemons/libpsk-generator.a
ap-daemons-0.0.3-r3008: [14/288] AR platform/ap-daemons/libpsk-generator.a
ap-daemons-0.0.3-r3008: [14/288] RULE Generating C++ code from fcm/mcs.proto
ap-daemons-0.0.3-r3008: [15/288] RULE Generating C++ code from fcm/mcs.proto
ap-daemons-0.0.3-r3008: /build/whirlwind/usr/share/proto: warning: directory does not exist.
ap-daemons-0.0.3-r3008: 
ap-daemons-0.0.3-r3008: [15/288] RULE Generating C++ code from fcm/android_checkin.proto
ap-daemons-0.0.3-r3008: [16/288] RULE Generating C++ code from fcm/android_checkin.proto
ap-daemons-0.0.3-r3008: /build/whirlwind/usr/share/proto: warning: directory does not exist.
ap-daemons-0.0.3-r3008: 
ap-daemons-0.0.3-r3008: [16/288] STAMP platform/ap-daemons/mcs_proto.actions_rules_copies.stamp
ap-daemons-0.0.3-r3008: [17/288] STAMP platform/ap-daemons/mcs_proto.actions_rules_copies.stamp
ap-daemons-0.0.3-r3008: [17/288] CXX platform/ap-daemons/gen/ap-daemons/fcm/mcs_proto.android_checkin.pb.o
ap-daemons-0.0.3-r3008: [18/288] CXX platform/ap-daemons/common/web-server-proxy.register_callback_interface.o
ap-daemons-0.0.3-r3008: [18/288] CXX platform/ap-daemons/gen/ap-daemons/fcm/mcs_proto.checkin.pb.o
ap-daemons-0.0.3-r3008: [19/288] CXX platform/ap-daemons/common/mdns-parser.mdns_parser.o
ap-daemons-0.0.3-r3008: [19/288] AR platform/ap-daemons/libmdns-parser.a
ap-daemons-0.0.3-r3008: [20/288] AR platform/ap-daemons/libmdns-parser.a
ap-daemons-0.0.3-r3008: [20/288] CXX platform/ap-daemons/gen/ap-daemons/fcm/mcs_proto.mcs.pb.o
ap-daemons-0.0.3-r3008: [21/288] CXX platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o
ap-daemons-0.0.3-r3008: FAILED: platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o 
ap-daemons-0.0.3-r3008: armv7a-cros-linux-gnueabi-clang++ -MMD -MF platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o.d -DUSE_METRICS_UPLOADER=1 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -Iplatform/ap-daemons/libipv6mtumonitor.gen/include -Igen/include -I/build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform2 -I/build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform -I/build/whirlwind/usr/include -Wall -Wno-psabi -Wunused -Wno-unused-parameter -ggdb3 -fstack-protector-strong -Wformat=2 -fvisibility=internal -Wa,--noexecstack -Werror --sysroot=/build/whirlwind -pthread -std=c++11 -pthread -DUSE_RTTI_FOR_TYPE_TAGS -Wno-c++11-extensions -Wno-unused-local-typedefs -DBASE_VER=395517 -pthread -I/build/whirlwind/usr/include/shill-client -I/build/whirlwind/usr/include/power_manager-client -I/build/whirlwind/usr/include/base-395517 -I/build/whirlwind/usr/include/glib-2.0 -I/build/whirlwind/usr/lib/glib-2.0/include -I/build/whirlwind/usr/include/nss -I/build/whirlwind/usr/include/nspr -I/build/whirlwind/usr/include/dbus-1.0 -I/build/whirlwind/usr/lib/dbus-1.0/include -I/build/whirlwind/usr/include/chromeos -fPIE -std=gnu++11 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -O2 -O2 -O2 -pipe -march=armv7-a -mfpu=vfpv3-d16 -mfloat-abi=hard -g -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables -clang-syntax -clang-syntax -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables  -c ../../../../../../../tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.cc -o platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o
ap-daemons-0.0.3-r3008: In file included from ../../../../../../../tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.cc:5:
ap-daemons-0.0.3-r3008: In file included from /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.h:19:
ap-daemons-0.0.3-r3008: In file included from /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform/ap-daemons/common/wan_configuration.h:8:
ap-daemons-0.0.3-r3008: /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3008/work/ap-daemons-0.0.3/platform/ap/common/configuration_monitor.h:13:10: fatal error: 'ap-daemons/controller/dbus-proxies.h' file not found
ap-daemons-0.0.3-r3008: #include "ap-daemons/controller/dbus-proxies.h"
ap-daemons-0.0.3-r3008:          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ap-daemons-0.0.3-r3008: 1 error generated.
ap-daemons-0.0.3-r3008: 
[snip]
WARNING: The following packages failed once or more,
but succeeded upon retry. This might indicate incorrect
dependencies.
  chromeos-base/ap-daemons-0.0.3-r3008



This likely indicates a missing package dependency, which is causing this package to need to get retried, wasting build time.

Assigning to non-pacific sheriff to track down and fix by adding necessary dependency.
 
Cc: dmitrygr@google.com
Please CC caiz and sduvvuri. sduvvuri is responsible for ap-daemons these
days.

cheers,
grant
Cc: caiz@chromium.org sduvvuri@chromium.org
(done)

Comment 4 by sduvvuri@google.com, Jul 11 2017

Cc: jintao@chromium.org
tao,
 will the changes you are working on to cleanup dependencies address this issue ?

Comment 5 by jintao@google.com, Jul 11 2017

Yes, I have built the whole image and all packages from a clean tree with the GYP cleanup branch, and everything builds successfully. 

I think this error was due to some missing dependencies or improperly set dependencies when this new ipv6 code was checked in.  This error probably didn't occur to developer b/c the ap-common was already built and installed, thus the error was hidden. 
Cc: sjg@chromium.org
Components: OS>Packages
Labels: -Pri-2 OS-Chrome Pri-1
Status: Assigned (was: Untriaged)
The problem seems to have gotten worse in some configurations.  It is now sometimes failing to build on both attempts:

https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/no_vmtest_pre_cq/41889
https://luci-milo.appspot.com/buildbot/chromiumos.tryserver/no_vmtest_pre_cq/82134

Retrying emerge of chromeos-base/ap-daemons-0.0.3-r3058.
Started chromeos-base/ap-daemons-0.0.3-r3058 (logged in /tmp/ap-daemons-0.0.3-r3058-gRqcTI)
=== Start output for job ap-daemons-0.0.3-r3058 (0m10.1s) ===
ap-daemons-0.0.3-r3058: >>> Emerging (1 of 1) chromeos-base/ap-daemons-0.0.3-r3058::jetstream-private for /build/whirlwind/
...
ap-daemons-0.0.3-r3058: [83/298] CXX platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o
ap-daemons-0.0.3-r3058: FAILED: platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o 
ap-daemons-0.0.3-r3058: armv7a-cros-linux-gnueabi-clang++ -MMD -MF platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o.d -DUSE_METRICS_UPLOADER=1 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -Iplatform/ap-daemons/libipv6mtumonitor.gen/include -Igen/include -I/build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform2 -I/build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform -I/build/whirlwind/usr/include -Wall -Wno-psabi -Wunused -Wno-unused-parameter -ggdb3 -fstack-protector-strong -Wformat=2 -fvisibility=internal -Wa,--noexecstack -Werror --sysroot=/build/whirlwind -pthread -std=c++11 -pthread -DUSE_RTTI_FOR_TYPE_TAGS -Wno-c++11-extensions -Wno-unused-local-typedefs -DBASE_VER=395517 -pthread -I/build/whirlwind/usr/include/shill-client -I/build/whirlwind/usr/include/power_manager-client -I/build/whirlwind/usr/include/base-395517 -I/build/whirlwind/usr/include/glib-2.0 -I/build/whirlwind/usr/lib/glib-2.0/include -I/build/whirlwind/usr/include/nss -I/build/whirlwind/usr/include/nspr -I/build/whirlwind/usr/include/dbus-1.0 -I/build/whirlwind/usr/lib/dbus-1.0/include -I/build/whirlwind/usr/include/chromeos -fPIE -std=gnu++11 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -O2 -O2 -O2 -pipe -march=armv7-a -mfpu=vfpv3-d16 -mfloat-abi=hard -g -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables -clang-syntax -clang-syntax -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables  -c ../../../../../../../tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.cc -o platform/ap-daemons/common/libipv6mtumonitor.ipv6_upstream_mtu_monitor.o
ap-daemons-0.0.3-r3058: In file included from ../../../../../../../tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.cc:5:
ap-daemons-0.0.3-r3058: In file included from /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform/ap-daemons/common/ipv6_upstream_mtu_monitor.h:19:
ap-daemons-0.0.3-r3058: In file included from /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform/ap-daemons/common/wan_configuration.h:8:
ap-daemons-0.0.3-r3058: /build/whirlwind/tmp/portage/chromeos-base/ap-daemons-0.0.3-r3058/work/ap-daemons-0.0.3/platform/ap/common/configuration_monitor.h:13:10: fatal error: 'ap-daemons/controller/dbus-proxies.h' file not found
ap-daemons-0.0.3-r3058: #include "ap-daemons/controller/dbus-proxies.h"
ap-daemons-0.0.3-r3058:          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ap-daemons-0.0.3-r3058: 1 error generated.

Comment 7 by sjg@google.com, Jul 24 2017

Cc: oka@chromium.org
Owner: sjg@chromium.org
I'm going to have a quick look at this. oka@ if you have started please let me know.

Comment 8 by sjg@google.com, Jul 24 2017

Status: Started (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, Jul 28 2017

Comment 11 by sjg@google.com, Jul 31 2017

Status: Fixed (was: Started)
I believe this is fixed. It's a bit hard to tell since I can't find a build that changes that package. I'm closing this bug. If it happens again, please reopen.

Comment 12 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment