Skip to content
Snippets Groups Projects

Add alerts task

Merged Cody Messick requested to merge cody.messick/gwcelery:alerts_task into main
All threads resolved!

This peels off the work in !852 (merged) related to the alert task specifically (as opposed to plugging the alerts task into the orchestrator).

I've bumped the version of fastavro to be at least 1.6.1, because 1.6.0 adds support for a strict option in the validate function that we use in the validation unit test, and 1.6.1 fixes a bug in that strict option.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Cody Messick
  • Cody Messick added 4 commits

    added 4 commits

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    Compare with previous version

  • Leo Pound Singer
  • Leo Pound Singer
  • Leo Pound Singer
  • Leo Pound Singer
  • Leo Pound Singer
  • Leo Pound Singer
  • Leo Pound Singer
  • Cody Messick added 1 commit

    added 1 commit

    • d3edfcc0 - Rework download external event file so its done in a canvas

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    Compare with previous version

  • Deep Chatterjee
  • Cody Messick added 2 commits

    added 2 commits

    • 578c480e - Remove leftover debugging print statement
    • 089ca294 - Change kafka_topic_up from flag to method on KafkaWriter, remove periodic task

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • f44a886a - Update nagios and nagios unit test

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • c14d68df - Change send_after_downloading_skymap from task to function

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • 5a908d12 - Rename send_after_downloading_skymap to download_skymap_and_send_alert, rename

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • 995bbcca - Rename send_after_downloading_skymap to download_skymap_and_send_alert, rename

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • f49fb201 - Move download of external event into _add_external_coinc_to_alert

    Compare with previous version

  • Deep Chatterjee approved this merge request

    approved this merge request

  • Deep Chatterjee resolved all threads

    resolved all threads

  • Leo Pound Singer
  • Leo Pound Singer
  • Cody Messick added 1 commit

    added 1 commit

    • ca28187f - Change _create_base_alert_dict from task to function

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • 71cc7172 - Drop task decorator on _create_base_alert_dict

    Compare with previous version

  • Cody Messick added 1 commit

    added 1 commit

    • 328257b5 - Fix thinko in get_event call in _add_external_coinc_to_alert

    Compare with previous version

  • Cody Messick resolved all threads

    resolved all threads

  • Cody Messick added 1 commit

    added 1 commit

    • 87d0da4c - Drop identity task from send

    Compare with previous version

  • Cody Messick resolved all threads

    resolved all threads

  • Cody Messick added 1 commit

    added 1 commit

    Compare with previous version

  • Leo Pound Singer
  • Leo Pound Singer
  • Cody Messick resolved all threads

    resolved all threads

  • Cody Messick added 1 commit

    added 1 commit

    • 2a4dbc4e - Rework download_skymap_and_send_alert function to drop identity task. Change

    Compare with previous version

  • Leo Pound Singer approved this merge request

    approved this merge request

  • Cody Messick added 1 commit

    added 1 commit

    Compare with previous version

  • Cody Messick enabled an automatic merge when the pipeline for 864343ce succeeds

    enabled an automatic merge when the pipeline for 864343ce succeeds

  • merged

  • Cody Messick mentioned in merge request !852 (merged)

    mentioned in merge request !852 (merged)

  • Please register or sign in to reply
    Loading