On Mon, Nov 28, 2016 at 10:55:56AM -0500, Ian Cordasco wrote: > So, I'm not entirely certain we actually want use_stderr on for > everything by default. If you look at that failing test, it's > asserting there's stderr output from running one of Glance's > administrative commands (in this case glance-replicator, but it could > apply to any of the other ones). I think, what we *probably* want is > just to have stderr default to True for commands in glance.cmd > (probably excluding glance-api and glance-registry) so that operators > running those commands continue to see output. > > I've been looking at this a little, and I think what we want is to do > > CONF.set_override(name='use_stderr', default=True, enforce_type=True) > > Does this sound reasonable to everyone else? If so, I already made > https://review.openstack.org/403775 which does this and adds an > upgrade release note to document the change.
Thanks Ian. Great suggestion and I appreciate you running with this. Yours Tony.
signature.asc
Description: PGP signature
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
