Mark unimported relations as WARN
While doing an import on behalf of a customer, we didn't realize it was imported with errors because the warning is marked as INFO
. This should be marked as WARN
instead.
From relevant Slack conversation (internal):
The rake task did not emit any ERROR level log messages and concluded with "Done!", which taken together I assumed meant it succeeded. I didn't know that the INFO-level log line
INFO -- : Total number of not imported relations: 202
meant that an error had occurred. If the rake task labeled this as an ERROR, I wouldn't have assumed it was normal behavior.I would lean towards having the rake task treat this outcome as a failure and give some indication as to how to follow up, such as:
- Explicitly indicate that the project was imported with errors.
- List the failed relations and identifying them as db tables.
Edited by Cynthia "Arty" Ng