-
Notifications
You must be signed in to change notification settings - Fork 9
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.
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
Add spidermon. #23
base: main
Are you sure you want to change the base?
Add spidermon. #23
Conversation
"Product/name": 1.0, | ||
"Product/metadata": 1.0, | ||
} | ||
SPIDERMON_ITEM_COUNT_INCREASE = 1 # At least 1 item every 5 minutes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feels redundant with the recommended CLOSESPIDER_TIMEOUT_NO_ITEM value of 600 in the setup instructions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@VMRuiz what do you think?
The project installs `spidermon <https://spidermon.readthedocs.io/en/latest/>`_ | ||
and configures some useful monitors, but it doesn't enable them by default. To | ||
do that you should add ``SPIDERMON_ENABLED = True`` to your project settings | ||
(either ``settings.py`` or Scrapy Cloud settings). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disabled by default per the today's discussion.
I thought about adding SPIDERMON_ENABLED = False
to settings.py
but I don't think it's useful,
No description provided.