neutron-service is throwing an excessive amount of warnings

Bug #1399760 reported by Joe Gordon
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Wishlist
Unassigned

Bug Description

http://logs.openstack.org/58/136158/9/check/check-tempest-dsvm-neutron-full/abea49f/logs/screen-q-svc.txt.gz?level=TRACE

Is full of a lot of repetitive warnings, and it is unclear if something may be going wrong or not.

Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

That's a normal case for tempest tests, especially API tests which perform operations too quickly.

I don't think we need to do anything special with it.

Changed in neutron:
status: New → Incomplete
Revision history for this message
Joe Gordon (jogo) wrote :

I disagree, neutron should not be logging warnings for things that are not possibly issues. And if something is going wrong so often that we get this many warnings it probably should be fixed.

Changed in neutron:
status: Incomplete → New
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

In each case it might be an issue, it might be not.
API tests are special case where client doesn't care about consistent state between the backend (agents) and the database.
That consistent state becomes broken and that's why it is logged at warning level.

I think in real life scenarios it makes sense to detect such cases at logging level that is visible by default (e.g. INFO+).
We can lower it to info, but amount of logs will be the same.
What do you think?

Changed in neutron:
status: New → Incomplete
Revision history for this message
Joe Gordon (jogo) wrote :

Lowering the log severity in a case by case basis sounds like a good approach to me. What do you mean the API tests don't care about consistent state between agents and the database? When is it ever ok for the backends to get out of sync with the databse?

Changed in neutron:
status: Incomplete → New
Revision history for this message
Eugene Nikanorov (enikanorov) wrote :

For instance, tempest API tests used to create and then immediately delete an object, or create-update-delete.
Backend (agent side) might not keep up with frontend state changes, and thus it logs warnings that it could not find objects that should be there.
I think that's ok to have those messages at warning level, because the case is not practical.
In real life scenarios you would care about backend state too, but if backend detects that frontend went out of sync - it deserves visible logging level IMO

Changed in neutron:
status: New → Incomplete
Revision history for this message
Joe Gordon (jogo) wrote :
Joe Gordon (jogo)
Changed in neutron:
status: Incomplete → New
Changed in neutron:
importance: Undecided → Wishlist
status: New → Confirmed
assignee: nobody → Eugene Nikanorov (enikanorov)
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

This bug is > 365 days without activity. We are unsetting assignee and milestone and setting status to Incomplete in order to allow its expiry in 60 days.

If the bug is still valid, then update the bug status.

Changed in neutron:
assignee: Eugene Nikanorov (enikanorov) → nobody
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.