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

Loading