Commit c182b6a
authored
Add logging for errors about loading dependencies on startup (#4858)
**Which issue(s) this PR fixes**:
None.
**What this PR does / why we need it**:
Adds error handling similar to worker process.
https://github.com/fluent/fluentd/blob/8df10863769da32459aba2a1408134d0b8e7d9f4/lib/fluent/supervisor.rb#L1149-L1164
`Fluent::Engine.run_configure` calls `Fluent::Registry#search`.
It can run `Kernel.require`.
It can raise `LoadError`, `ConflictError` and others.
Without this error handling, Fluentd cannot log those errors.
This makes a situation where Fluentd fails to start, but there is no
error log in the log file.
This solves that problem.
This appears to be an oversight of the following fix:
* #2651
**Docs Changes**:
Not needed.
**Release Note**:
The same as the title.
Signed-off-by: Daijiro Fukuda <[email protected]>1 parent 60e3693 commit c182b6a
2 files changed
+13
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
721 | 721 | | |
722 | 722 | | |
723 | 723 | | |
| 724 | + | |
| 725 | + | |
| 726 | + | |
| 727 | + | |
| 728 | + | |
| 729 | + | |
| 730 | + | |
| 731 | + | |
| 732 | + | |
| 733 | + | |
| 734 | + | |
| 735 | + | |
724 | 736 | | |
725 | 737 | | |
726 | 738 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
570 | 570 | | |
571 | 571 | | |
572 | 572 | | |
573 | | - | |
| 573 | + | |
574 | 574 | | |
575 | 575 | | |
576 | 576 | | |
| |||
0 commit comments