Skip to content

New hook type to send log links to Datadog

What does this MR do and why?

Based on @Andysoiron helping with !73178 (diffs)

I added:

  • A checkbox to enable the events to the integration form.
  • Additional data to the hook that we need to correlate the logs with the job.

Related to #326797 (closed)

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Example below:

  1. In rails console, enable the feature flag
    Feature.enable(:datadog_integration_logs_collection)

Database migration outputs

Up

== 20211026124336 AddArchiveTraceEventsToIntegrations: migrating ==============
-- add_column(:integrations, :archive_trace_events, :boolean, {:null=>false, :default=>false})
   -> 0.0087s
== 20211026124336 AddArchiveTraceEventsToIntegrations: migrated (0.0088s) =====

Down

rake db:migrate:down VERSION=20211026124336
== 20211026124336 AddArchiveTraceEventsToIntegrations: reverting ==============
-- remove_column(:integrations, :archive_trace_events, :boolean, {:null=>false, :default=>false})
   -> 0.0137s
== 20211026124336 AddArchiveTraceEventsToIntegrations: reverted (0.0179s) =====

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Andy Schoenen

Merge request reports