[openstack-dev] [glance][oslo] oslo.log 3.17.0 use_stderr default change
harlowja at fastmail.com
Mon Nov 28 19:23:53 UTC 2016
>> Hello Glance team!
> Hello Tony!
>> About a month ago the oslo team released 3.17.0 of oslo.log which contains 
>> which switches the default for use_stderr from True to False. It hasn't made
>> it into upper-constraints.txt because glance is failing. There are 2 easy
>> 1) switch the glance test to look at stdout rather then stderr ; or
>> 2) update the glance config files to explictly set "use_stderr = true"
>> I feel like Option 2 is correct as clearly preserves the existing behaviour the
>> operators/deployers can opt out if they want.
> 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.
> I don't want it to be merged without appropriate discussion, so I've
> -1'd the Workflow.
Sounds reasonable to me, I'd like to get that merged since its the last
current bug with the oslo periodic jobs for glance:
(all of these)
More information about the OpenStack-dev