Fix & un-@Ignore AclServiceTest
It turns out that c/45261 re. "@Override init() instead @PostConstruct
start()" broke AclServiceTest (c/42109); somehow Mycila Guice
Jsr250Module interprets @PostConstruct in parent class, or protected VS
public, differently than Apache Aries Blueprint impl.
This fixes it by putting @PostConstruct back, for now. I'll separately
investigate a cleaner fix (probably going either in Mycila Guice, Genius
or infrautils; not netvirt).
This problem was not seen pre-merge, and thus broke master, because we
are in the unhealthy habit of merging a bunch of changes without
re-re-rebasing EVERY change after EVERY merge. While that certainly
adds some overhead for commiters and build infra, doing that would lead
to more stability in such cases, as well as avoid Guitar Hero git log
<https://twitter.com/henryhoffman/status/
694184106440200192>, which in
turn would make it easier to locally try git revert and git bisect to
find out which commit cause what.
Change-Id: Ife74e3b920cca1fdf43fdf3aedc6401723050c46
Signed-off-by: Michael Vorburger <vorburger@redhat.com>