ddtrace 0.13.0 → 0.47.0

data/.circleci/images/primary/Dockerfile-jruby-9.2.0.0

@@ -0,0 +1,73 @@
+# Current version: https://github.com/cpuguy83/docker-jruby/blob/db08637839406420a254a9c825d94994cded5021/9000/jre/Dockerfile
+FROM jruby:9.2.0.0
+
+# Make apt non-interactive
+RUN echo 'APT::Get::Assume-Yes "true";' > /etc/apt/apt.conf.d/90circleci \
+  && echo 'DPkg::Options "--force-confnew";' >> /etc/apt/apt.conf.d/90circleci
+
+ENV DEBIAN_FRONTEND=noninteractive
+
+# Install required packages
+RUN set -ex; \
+        apt-get update; \
+        mkdir -p /usr/share/man/man1; \
+        apt-get install -y --no-install-recommends \
+            git mercurial xvfb \
+            locales sudo openssh-client ca-certificates tar gzip parallel \
+            net-tools netcat unzip zip bzip2 gnupg curl wget \
+            tzdata rsync vim \
+            build-essential; \
+        rm -rf /var/lib/apt/lists/*;
+
+# Set timezone to UTC by default
+RUN ln -sf /usr/share/zoneinfo/Etc/UTC /etc/localtime
+
+# Set language
+RUN locale-gen en_US.UTF-8
+ENV LANG en_US.UTF-8
+ENV LANGUAGE en_US:en
+
+# Install jq
+RUN JQ_URL="https://circle-downloads.s3.amazonaws.com/circleci-images/cache/linux-amd64/jq-latest" \
+  && curl --silent --show-error --location --fail --retry 3 --output /usr/bin/jq $JQ_URL \
+  && chmod +x /usr/bin/jq \
+  && jq --version
+
+# Install Docker
+RUN set -ex \
+  && export DOCKER_VERSION=$(curl --silent --fail --retry 3 https://download.docker.com/linux/static/stable/x86_64/ | grep -o -e 'docker-[.0-9]*-ce\.tgz' | sort -r | head -n 1) \
+  && DOCKER_URL="https://download.docker.com/linux/static/stable/x86_64/${DOCKER_VERSION}" \
+  && echo Docker URL: $DOCKER_URL \
+  && curl --silent --show-error --location --fail --retry 3 --output /tmp/docker.tgz "${DOCKER_URL}" \
+  && ls -lha /tmp/docker.tgz \
+  && tar -xz -C /tmp -f /tmp/docker.tgz \
+  && mv /tmp/docker/* /usr/bin \
+  && rm -rf /tmp/docker /tmp/docker.tgz \
+  && which docker \
+  && (docker version || true)
+
+# Install Docker Compose
+RUN COMPOSE_URL="https://circle-downloads.s3.amazonaws.com/circleci-images/cache/linux-amd64/docker-compose-latest" \
+  && curl --silent --show-error --location --fail --retry 3 --output /usr/bin/docker-compose $COMPOSE_URL \
+  && chmod +x /usr/bin/docker-compose \
+  && docker-compose version
+
+# Install Dockerize
+RUN DOCKERIZE_URL="https://circle-downloads.s3.amazonaws.com/circleci-images/cache/linux-amd64/dockerize-latest.tar.gz" \
+  && curl --silent --show-error --location --fail --retry 3 --output /tmp/dockerize-linux-amd64.tar.gz $DOCKERIZE_URL \
+  && tar -C /usr/local/bin -xzvf /tmp/dockerize-linux-amd64.tar.gz \
+  && rm -rf /tmp/dockerize-linux-amd64.tar.gz \
+  && dockerize --version
+
+# Install RubyGems
+RUN gem update --system
+RUN mkdir -p "$GEM_HOME" && chmod -R 777 "$GEM_HOME"
+ENV BUNDLE_SILENCE_ROOT_WARNING 1
+
+# Ensure JRuby is available when running "bash --login"
+RUN echo "export PATH=/opt/jruby/bin:$BUNDLE_BIN:\$PATH" >> ~/.profile
+
+RUN mkdir /app
+WORKDIR /app
+
+CMD ["bin", "bash", "--login"]

data/.circleci/images/primary/{Dockerfile-1.9.3 → Dockerfile-truffleruby-21.0.0}

@@ -1,5 +1,4 @@
-# Last version: https://github.com/docker-library/ruby/blob/90c4e3be58d565007c518d311d4bd05086a1638c/1.9/Dockerfile
-FROM ruby:1.9.3
+FROM flavorjones/truffleruby:21.0.0-buster
 
 # Make apt non-interactive
 RUN echo 'APT::Get::Assume-Yes "true";' > /etc/apt/apt.conf.d/90circleci \
@@ -22,6 +21,7 @@ RUN set -ex; \
 RUN ln -sf /usr/share/zoneinfo/Etc/UTC /etc/localtime
 
 # Set language
+RUN echo "en_US.UTF-8 UTF-8" > /etc/locale.gen
 RUN locale-gen en_US.UTF-8
 ENV LANG en_US.UTF-8
 ENV LANGUAGE en_US:en
@@ -60,6 +60,10 @@ RUN DOCKERIZE_URL="https://circle-downloads.s3.amazonaws.com/circleci-images/cac
 
 # Install RubyGems
 RUN gem update --system
+RUN mkdir -p "$GEM_HOME" && chmod -R 777 "$GEM_HOME"
+
+# Upgrade RubyGems and Bundler
+RUN gem update --system
 RUN gem install bundler
 ENV BUNDLE_SILENCE_ROOT_WARNING 1
 

data/.env

@@ -1,6 +1,7 @@
+DD_AGENT_HOST=localhost
+DD_METRIC_AGENT_PORT=8125
+DD_TRACE_AGENT_PORT=8126
 TEST_DDAGENT_API_KEY=invalid_key_but_this_is_fine
-TEST_DDAGENT_HOST=localhost
-TEST_DDAGENT_PORT=8126
 TEST_ELASTICSEARCH_HOST=127.0.0.1
 TEST_ELASTICSEARCH_NATIVE_PORT=9300
 TEST_ELASTICSEARCH_REST_PORT=9200
@@ -19,5 +20,7 @@ TEST_POSTGRES_HOST=127.0.0.1
 TEST_POSTGRES_PASSWORD=postgres
 TEST_POSTGRES_PORT=5432
 TEST_POSTGRES_USER=postgres
+TEST_PRESTO_HOST=localhost
+TEST_PRESTO_PORT=8080
 TEST_REDIS_HOST=127.0.0.1
 TEST_REDIS_PORT=6379

data/.github/CODEOWNERS

@@ -0,0 +1 @@
+* @DataDog/apm-ruby

data/.github/workflows/add-milestone-to-pull-requests.yml

@@ -0,0 +1,42 @@
+name: Add milestone to pull requests
+on:
+  pull_request_target:
+    types: [closed]
+    branches:
+      - master
+
+jobs:
+  add_milestone_to_merged:
+    if: github.event.pull_request.merged && github.event.pull_request.milestone == null
+    name: Add milestone to merged pull requests
+    runs-on: ubuntu-latest
+    steps:
+      - name: Get project milestones
+        id: milestones
+        uses: actions/github-script@0.9.0
+        with:
+          github-token: ${{secrets.GITHUB_TOKEN}}
+          script: |
+            const list = await github.issues.listMilestonesForRepo({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              state: 'open'
+            })
+            // Need to manually sort because "sort by number" isn't part of the api
+            // highest number first
+            const milestones = list.data.sort((a,b) => (b.number - a.number))
+
+            return milestones.length == 0 ? null : milestones[0].number
+      - name: Update Pull Request
+        if: steps.milestones.outputs.result != null
+        uses: actions/github-script@0.9.0
+        with:
+          github-token: ${{secrets.GITHUB_TOKEN}}
+          script: |
+            // Confusingly, the issues api is used because pull requests are issues
+            await github.issues.update({
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              issue_number: ${{ github.event.pull_request.number }},
+              milestone: ${{ steps.milestones.outputs.result }},
+            });

data/.github/workflows/create-next-milestone.yml

@@ -0,0 +1,20 @@
+name: Create next milestone
+on:
+  milestone:
+    types: [closed]
+
+jobs:
+  create_next_milestone:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Get next minor version
+        id: semvers
+        uses: WyriHaximus/github-action-next-semvers@b135abb108d66990a85e18623d906404f4350ce4
+        with:
+          version: ${{ github.event.milestone.title }}
+      - name: Create next milestone
+        uses: WyriHaximus/github-action-create-milestone@ab85332e3150ec018daf497a0f761fe69d52bc7d
+        with:
+          title: ${{ steps.semvers.outputs.minor }}
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

data/.gitignore

@@ -50,6 +50,8 @@ build-iPhoneSimulator/
 
 # unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
 .rvmrc
+# Ignore local variables
+.envrc
 
 # lock files
 Gemfile.lock

data/.gitlab-ci.yml

@@ -0,0 +1,27 @@
+stages:
+  - deploy
+
+variables:
+  INDEX_FILE: index.txt
+
+.common: &common
+  tags: [ "runner:main", "size:large" ]
+
+copy_to_s3:
+  <<: *common
+  stage: deploy
+  image: registry.gitlab.com/gitlab-org/cloud-deploy/aws-base:latest
+  rules:
+    - if: '$CI_COMMIT_BRANCH == "master"'
+      when: on_success
+    - when: manual
+      allow_failure: true
+  script:
+    - export ACCESS_KEY_ID=$(aws ssm get-parameter --region us-east-1 --name ci.dd-trace-rb.secret_key_id --with-decryption --query "Parameter.Value" --out text)
+    - export SECRET_ACCESS_KEY=$(aws ssm get-parameter --region us-east-1 --name ci.dd-trace-rb.secret_sec_key_id --with-decryption --query "Parameter.Value" --out text)
+    - export AWS_ACCESS_KEY_ID=$ACCESS_KEY_ID
+    - export AWS_SECRET_ACCESS_KEY=$SECRET_ACCESS_KEY
+    - echo $CI_COMMIT_REF_NAME >> $INDEX_FILE
+    - echo $CI_COMMIT_SHA >> $INDEX_FILE
+    - echo $GITLAB_USER_NAME >> $INDEX_FILE
+    - aws s3 cp $INDEX_FILE s3://datadog-reliability-env/ruby/$INDEX_FILE

data/.rubocop.yml

@@ -1,9 +1,28 @@
+inherit_from: .rubocop_todo.yml
+
+require:
+  - rubocop-performance
+  - rubocop-rspec
+
 AllCops:
-  TargetRubyVersion: 2.1
+  TargetRubyVersion: 2.4 # Lowest version supported currently
+  Include:
+    - 'lib/**/*.rb'
+    - 'test/**/*.rb'
+    - 'spec/**/*.rb'
+    - 'Gemfile'
+    - 'Rakefile'
+  Exclude:
+    - 'Appraisals'
+    - '*.gemspec'
+    - 'lib/ddtrace/**/vendor/**/*.rb'
+    - 'integration/apps/*/bin/*'
+    - 'integration/apps/*/Gemfile'
+  NewCops: disable # Don't allow new cops to be enabled implicitly.
 
 # 80 characters is a nice goal, but not worth currently changing in existing
 # code for the sake of changing it to conform to a length set in 1928 (IBM).
-Metrics/LineLength:
+Layout/LineLength:
   Max: 124
 
 # These exceptions are good goals to attain, and probably will over time,
@@ -17,14 +36,14 @@ Metrics/AbcSize:
 Metrics/MethodLength:
   Max: 36
 
+Performance/Casecmp:
+  Enabled: false
+
 # TODO: this is not compliant with the Ruby community style guide. We
 # should enable again this rule but it will change the public API because
 # we're using set_ methods. We should work on that because also Rails
 # honors this rule.
-Style/AccessorMethodName:
-  Enabled: false
-
-Style/MethodCallWithoutArgsParentheses:
+Naming/AccessorMethodName:
   Enabled: false
 
 Style/RescueModifier:
@@ -54,6 +73,13 @@ Metrics/PerceivedComplexity:
 Lint/UnusedMethodArgument:
   Enabled: false
 
+Style/RescueStandardError:
+  Enabled: false
+
+# alias and alias_method are not equivalent
+Style/Alias:
+  Enabled: false
+
 # Disabling advices that would lead to incompatible Ruby 1.9 code
 Style/SymbolArray:
   Enabled: false
@@ -62,3 +88,241 @@ Style/SymbolArray:
 # expressions.
 Style/GuardClause:
   Enabled: false
+
+# Case equality is not intrinsically problematic.
+Style/CaseEquality:
+  Enabled: false
+
+# New cops since Rubocop 1.0.
+# We have to explicitly opt-in for new cops to apply
+# before the next major release.
+Gemspec/DateAssignment: # (new in 1.10)
+  Enabled: true
+Layout/SpaceBeforeBrackets: # (new in 1.7)
+  Enabled: true
+Lint/AmbiguousAssignment: # (new in 1.7)
+  Enabled: true
+Lint/DeprecatedConstants: # (new in 1.8)
+  Enabled: true
+Lint/DuplicateBranch: # (new in 1.3)
+  Enabled: true
+Lint/DuplicateRegexpCharacterClassElement: # (new in 1.1)
+  Enabled: true
+Lint/EmptyClass: # (new in 1.3)
+  Enabled: true
+Lint/LambdaWithoutLiteralBlock: # (new in 1.8)
+  Enabled: true
+Lint/NoReturnInBeginEndBlocks: # (new in 1.2)
+  Enabled: true
+Lint/NumberedParameterAssignment: # (new in 1.9)
+  Enabled: true
+Lint/OrAssignmentToConstant: # (new in 1.9)
+  Enabled: true
+Lint/RedundantDirGlobSort: # (new in 1.8)
+  Enabled: true
+Lint/SymbolConversion: # (new in 1.9)
+  Enabled: true
+Lint/ToEnumArguments: # (new in 1.1)
+  Enabled: true
+Lint/TripleQuotes: # (new in 1.9)
+  Enabled: true
+Lint/UnexpectedBlockArity: # (new in 1.5)
+  Enabled: true
+Lint/UnmodifiedReduceAccumulator: # (new in 1.1)
+  Enabled: true
+Style/ArgumentsForwarding: # (new in 1.1)
+  Enabled: true
+Style/DocumentDynamicEvalDefinition: # (new in 1.1)
+  Enabled: true
+Style/EndlessMethod: # (new in 1.8)
+  Enabled: true
+Style/HashExcept: # (new in 1.7)
+  Enabled: true
+Style/IfWithBooleanLiteralBranches: # (new in 1.9)
+  Enabled: true
+Style/NilLambda: # (new in 1.3)
+  Enabled: true
+Style/RedundantArgument: # (new in 1.4)
+  Enabled: true
+Style/SwapValues: # (new in 1.1)
+  Enabled: true
+Performance/AncestorsInclude: # (new in 1.7)
+  Enabled: true
+Performance/BigDecimalWithNumericArgument: # (new in 1.7)
+  Enabled: true
+Performance/BlockGivenWithExplicitBlock: # (new in 1.9)
+  Enabled: true
+Performance/CollectionLiteralInLoop: # (new in 1.8)
+  Enabled: true
+Performance/ConstantRegexp: # (new in 1.9)
+  Enabled: true
+Performance/MethodObjectAsBlock: # (new in 1.9)
+  Enabled: true
+Performance/RedundantSortBlock: # (new in 1.7)
+  Enabled: true
+Performance/RedundantStringChars: # (new in 1.7)
+  Enabled: true
+Performance/ReverseFirst: # (new in 1.7)
+  Enabled: true
+Performance/SortReverse: # (new in 1.7)
+  Enabled: true
+Performance/Squeeze: # (new in 1.7)
+  Enabled: true
+Performance/StringInclude: # (new in 1.7)
+  Enabled: true
+Performance/Sum: # (new in 1.8)
+  Enabled: true
+
+# Requires Ruby 2.1
+Style/HashConversion:
+  Enabled: false
+
+# Requires Ruby 2.1
+Lint/SendWithMixinArgument:
+  Enabled: false
+
+# Requires Ruby 2.2
+Style/HashSyntax:
+  Enabled: false
+
+# Requires Ruby 2.3
+Style/SafeNavigation:
+  Enabled: false
+
+# Requires Ruby 2.3
+Performance/UnfreezeString:
+  Enabled: false
+
+# Requires Ruby 2.4
+Style/HashTransformValues:
+  Enabled: false
+
+# Requires Ruby 2.4
+Style/CollectionCompact:
+  Enabled: false
+
+# Enforces negative/positive branching order,
+# which can hurt readability.
+Style/NegatedIfElseCondition:
+  Enabled: false
+
+# Converts `to change{foo.bar}` to `to change(foo, :bar)`.
+# It doesn't seem to make it more readable, and doesn't work
+# when `foo` has to be re-evaluated.
+RSpec/ExpectChange:
+  Enabled: false
+
+# Readability is not meaningfully affected by enforcing
+# `not_to` vs `to_not`.
+RSpec/NotToNot:
+  Enabled: false
+
+# Enforces `subject` above all else, even `include_context`
+# declarations.
+RSpec/LeadingSubject:
+  Enabled: false
+
+# Enforces usage of `expect(subject)` instead of `is_expected`.
+RSpec/ImplicitSubject:
+  Enabled: false
+
+# TODO: Disabling until we categorize which file are safe to do so.
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
+# Replaces `x == 0` with `x.zero?` and similar.
+# These methods are ~2x slower than the simple math comparison.
+Style/NumericPredicate:
+  Enabled: false
+
+# Allow for empty blocks (without a comment) in tests.
+Lint/EmptyBlock:
+  Exclude:
+    - 'test/**/*.rb'
+    - 'spec/**/*.rb'
+
+# Enforces that context description must start with 'when', 'with', or 'without'.
+RSpec/ContextWording:
+  Enabled: false
+
+# Enforces that examples should only have a limited amount of assertions.
+RSpec/MultipleExpectations:
+  Enabled: false
+
+# Enforces minimum character length for parameters.
+Naming/MethodParameterName:
+  Enabled: false
+
+# Enforces no instance variable in specs.
+RSpec/InstanceVariable:
+  Enabled: false
+
+# Enforces a maximum nested example group level.
+RSpec/NestedGroups:
+  Enabled: false
+
+# Enforces `receive` vs `have_received` expectation pattern.
+RSpec/MessageSpies:
+  Enabled: false
+
+# Enforces example line count limit.
+RSpec/ExampleLength:
+  Enabled: false
+
+# Enforces maximum let and subject calls.
+RSpec/MultipleMemoizedHelpers:
+  Enabled: false
+
+# Enforces `allow` over `expect` when configuring a response.
+RSpec/StubbedMock:
+  Enabled: false
+
+# Enforces no use of `expect` in `before` hooks.
+RSpec/ExpectInHook:
+  Enabled: false
+
+# Enforces no stubbing using `allow_any_instance_of`.
+RSpec/AnyInstance:
+  Enabled: false
+
+# Enforces strict matching of example class with spec file name.
+# This does not work for our current namespacing as our gem is called
+# `ddtrace` while our namespace is `::Datadog`.
+# It would have to either be `ddtrace` + `::DDTrace` or
+# `datadog` + `::Datadog`.
+RSpec/FilePath:
+  Enabled: false
+
+# Enforces no stubbing of methods of the object under test.
+RSpec/SubjectStub:
+  Enabled: false
+
+# Enforces that the first argument to describe should
+# be the class or module being tested.
+RSpec/DescribeClass:
+  Enabled: false
+
+# Enforces to no use let! to setup objects not referenced in tests.
+# Fails to account for shared examples across files.
+RSpec/LetSetup:
+  Enabled: false
+
+# Identifies where `fetch(key) { value }` can be replaced by `fetch(key, value)`.
+# Can't differentiate between `Hash#fetch` and `RailsCache#fetch`,
+# passing in wrong parameters to these methods.
+Style/RedundantFetchBlock:
+  Enabled: false
+
+# Warns about usage of `before(:all)`
+RSpec/BeforeAfterAll:
+  Enabled: false
+
+# Enforces snake case file names.
+Naming/FileName:
+  Exclude:
+    - 'integration/**/Gemfile'
+    - 'integration/**/Rakefile'
+
+# Enforces boolean parameters with default to be keyword arguments.
+Style/OptionalBooleanParameter:
+  Enabled: false