Thanks ahasenack!

> the apt-news hook does not log to "apt-news.service.txt"

You are correct, but `pro collect-logs` does collect the status of apt-
news.service and puts it in a file called "apt-news.service.txt". That
is what the test is testing.

> include in the test plan a logrotation scenario

Good idea! I'll add it

> would be best if you could make apt-esm-json-hook output something to
stderr on apt-get upgrade

Unfortunately, that would require `apt` to do something different in the
apt-hook jsonrpc protocol, and i can't think of a reasonable way to
insert that kind of error. So I think we'll stick with the mock that you
have in your steps if that is okay.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2074211

Title:
  Consider hiding error messages in apt-news.service to avoid being
  percived as "pro is broken"

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-advantage-tools/+bug/2074211/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to