OCA / runbot-addons

Odoo Runbot Custom Features to be used by community.
https://odoo-community.org/project/community-maintainers-46
GNU Affero General Public License v3.0
28 stars 95 forks source link

[RFC][runbot_exclude_warns] Ability to exclude certain warnings in runbot #172

Closed JordiBForgeFlow closed 6 years ago

JordiBForgeFlow commented 6 years ago

Sometimes a runbot execution produces warnings while installing python modules using pip that are not really harmful and could be ignored. See for example https://github.com/OCA/ddmrp/pull/17#issuecomment-409593086

The runbot log shows a warning that is not really harmful: image

The PR appears as if Runbot is broken: image

We propose a module that would allow you to exclude warnings that contain a certain string, like: "numpy.dtype size changed".

This behaviour would be added here: https://github.com/odoo/runbot/blob/master/runbot/models/build.py#L834

And the strings to exclude could be specific to a repo.

Perhaps it's a crazy idea ¿??

cc @lreficent

pedrobaeza commented 6 years ago

You should fix the warning, as it shouldn't happen.

JordiBForgeFlow commented 6 years ago

Was fixed in numpy package 2 days ago https://github.com/numpy/numpy/pull/11634, but seems that an older version of numpy is being installed....

pedrobaeza commented 6 years ago

Add the new one to the requirements.txt file in root

moylop260 commented 6 years ago

FYI I just used ssh runbot connection to check numpy issue...

Please, check my comment here:

IMHO this warning is a valid one Closing since that currently we don't have a real warning to ignore.