Open Stack

 ---- On Wed, 14 Jul 2021 16:23:43 -0500 Sean Mooney <smooney at redhat.com> wrote ----
 > On Wed, 2021-07-14 at 14:15 -0700, Clark Boylan wrote:
 > > On Wed, Jul 14, 2021, at 1:47 PM, Ghanshyam Mann wrote:
 > > > Hello Everyone,
 > > > 
 > > > As you might have noticed the discussion in infra channel, log size are 
 > > > increasing a lot
 > > > and one of the main reasons is warnings (especially repeatedly 
 > > > warnings). Few of them are
 > > > from oslo policy on new policy and DeprecdatedRule. However, those are 
 > > > not the only warnings,
 > > > there are lot from other lib also.
 > > > 
 > > > Before we start facing more issues in logging system and indexing, we 
 > > > would request all the project
 > > > team to start looking at the logs and fix the warnings as much as 
 > > > possible.
 > > 
 > > I think it is important to note that it isn't just the CI system's log indexing that is affected by this. Operators in the real world are likely getting flooded by the warnings and those don't help them fix the code. It can also make it more difficult to determine the underlying causes of real issues if you have to sift through many warnings.
 > > 
 > > All that to say this is an improvement not just to the CI tooling but also likely to real deployments .
 > +1
 > i was going to say this would only affect real deployemnt after xena is released but
 > actully looking at stable wallaby i guess 3.7.0 was included for wallaby
 > https://github.com/openstack/requirements/blob/stable/wallaby/upper-constraints.txt#L472
 > so yes this is affecting all wallaby installs

Well, as DeprecatedRule warning fix need requirement bump so we will not be able to
backport that even u-c is 3.7.0. But yes for Xena onwards it will be helpful for operator.

I have not checked all other warnings but many should be backport-able.

-gmann

 > 
 > victoria and older should not be impacted
 > https://github.com/openstack/requirements/blob/stable/victoria/upper-constraints.txt#L466
 > 
 > so yes this is imporant not just for ci but for really installs.
 > > 
 > > > 
 > > > I started fixing few of the oslo policy warning on DeprecatedRule in 
 > > > applicable projects and once all patches
 > > > are merged, we can convert them to error from oslo side to avoid any 
 > > > future warnings- 
 > > > 
 > > > https://review.opendev.org/q/topic:%22fix-oslo-policy-warnings%22+(status:open%20OR%20status:merged)
 > > > 
 > > > [1] 
 > > > https://meetings.opendev.org/meetings/tc/2021/tc.2021-07-08-15.00.log.html#l-89
 > > > 
 > > > -gmann
 > > > 
 > > > 
 > > 
 > 
 > 
 > 
 >