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.

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

Missing alerts table creation during init #1211

Merged
merged 3 commits into from
Mar 26, 2020

Conversation

chunyong-lin
Copy link
Contributor

to: @airbnb/streamalert-maintainers
related to: #1163
resolves:

Background

In #1163, we had additional check on if firehose module (for logs) enable or not before creating the alerts table. By default, the firehose module is disabled and alerts table won't be created during the init time. The alerts table is not depend on firehose module (for logs). The fix is easy, move the "firehose_data_bucket" check after "alerts" table creation.

Changes

  • Move the "firehose_data_bucket" check after "alerts" table creation.
  • Change logging severity to warning if firehose module is disabled in conf/global.json.
  • Correct a typo in the logging message.

Testing

Tested in staging and the alerts table is created as expected when firehose module remaining disable.

@chunyong-lin chunyong-lin added this to the 3.1.0 milestone Mar 26, 2020
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.424% when pulling 7b4411e on missing_alerts_table_creation_during_init into 530a154 on release-3-1-0.

@chunyong-lin chunyong-lin merged commit f7f69e2 into release-3-1-0 Mar 26, 2020
@chunyong-lin chunyong-lin deleted the missing_alerts_table_creation_during_init branch March 26, 2020 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants