Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Supress warning of inclusion of extra files #272

Closed
glarrain opened this issue Aug 3, 2013 · 2 comments · Fixed by #347
Closed

Supress warning of inclusion of extra files #272

glarrain opened this issue Aug 3, 2013 · 2 comments · Fixed by #347

Comments

@glarrain
Copy link

glarrain commented Aug 3, 2013

When extra files are included because they match option files in section include, supervisor adds a parsing warning. Why? The log level is wrong, it should be info, since nothing "wrong" is happening that we should tell the user/dev about.

I noticed this problem because these warnings ("Included extra file xyz during parsing") pollute my logs a lot: I have 6 files for inclusion thus I get 6 of these not useful lines in my log file each time I reload.

https://github.com/Supervisor/supervisor/blob/master/supervisor/options.py#L546
@blueyed blueyed mentioned this issue Jan 20, 2014
Closed
blueyed added a commit to blueyed/supervisor that referenced this issue Jan 20, 2014
@blueyed
Add parse_infos to options, use it for "parsed file" message
bf699a7 
This adds a new parse_infos besides parse_warnings, and uses it for the
'Included extra file "%s" during parsing' info.

Fixes Supervisor#272
@blueyed
Copy link
Contributor

blueyed commented Jan 20, 2014

@glarrain
I came across this (because of issue #345) and added a fix (see my pull request).

@blueyed blueyed mentioned this issue Jan 20, 2014
Merged
@analytically
Copy link

+1

@kormat kormat mentioned this issue Aug 24, 2015
Merged
5 tasks
mnaberez pushed a commit that referenced this issue May 10, 2016
@blueyed @mnaberez
Add parse_infos to options, use it for "parsed file" message
63a5685 
This adds a new parse_infos besides parse_warnings, and uses it for the
'Included extra file "%s" during parsing' info.

Fixes #272
alexsilva pushed a commit to alexsilva/supervisor that referenced this issue Jun 22, 2019
@blueyed @alexsilva
Add parse_infos to options, use it for "parsed file" message
d6a9d2d 
This adds a new parse_infos besides parse_warnings, and uses it for the
'Included extra file "%s" during parsing' info.

Fixes Supervisor#272

(cherry picked from commit bf699a7)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add parse_infos to options, use it for "parsed file" message blueyed/supervisor
3 participants
@blueyed @analytically @glarrain