Skip to content

Latest commit

 

History

History
270 lines (200 loc) · 11 KB

File metadata and controls

270 lines (200 loc) · 11 KB

How to Contribute to Jaeger

We'd love your help!

Jaeger is Apache 2.0 licensed and accepts contributions via GitHub pull requests. This document outlines some of the conventions on development workflow, commit message formatting, contact points and other resources to make it easier to get your contribution accepted.

We gratefully welcome improvements to documentation as well as to code.

Table of Contents:

Making a Change

Open an issue first

Before making any significant changes, please open an issue. Each issue should describe the following:

  • Requirement - what kind of business use case are you trying to solve?
  • Problem - what in Jaeger blocks you from solving the requirement?
  • Proposal - what changes do you propose to solve the problem or improve the existing situation?
  • Any open questions to address

Discussing your proposed changes ahead of time will make the contribution process smooth for everyone. Once the approach is agreed upon, make your changes and open a pull request (PR).

Assigning Issues

We do not assign issues to contributors. It is almost never the case that multiple people jump on the same issue, and practice showed that occasionally people who ask for an issue to be assigned to them later have a change in priorities and are unable to find time to finish it, which leaves the issue in limbo. So if you have a desire to work on an issue, feel free to mention it in the comment and just submit a PR.

Creating a pull request

If you are new to GitHub's contribution workflow, we recommend the following setup:

  • Go to the respective Jaeger repo on GitHub and create a fork using the button at the top. Select a destination org where you have write permissions (usually it is your personal "org").
  • Clone the fork into your workspace.
  • (Recommended): register upstream repo as remote
    • After you clone your forked repo, running below command
      git remote -v
      will show origin, e.g. origin git@github.com:{username}/jaeger.git
    • Add upstream remote:
      git remote add upstream git@github.com:jaegertracing/jaeger.git
    • Fetch it:
      git fetch upstream main
    • Repoint your main branch:
      git branch --set-upstream-to=upstream/main main
    • With this setup, you will not need to keep your main branch in the fork in sync with the upstream repo.

Once you're ready to make changes:

  • Create a new local branch (DO NOT make changes to main, it will cause CI errors).
  • Commit your changes, making sure each commit is signed (see below):
    git commit -s -m "Your commit message"
  • You do not need to squash the commits, it will happen once the PR is merged into the official repo (but each individual commit must be signed).
  • When satisfied, push the changes. Git will likely ask for upstream destination, so you push commits like this:
    git push --set-upstream origin {branch-name}
  • After you push, look for the output, it usually contains a URL to create a pull request.
  • After raising a PR, please refrain from repeatedly merging the upstream main branch into your feature branch unless you're specifically resolving merge conflicts or updating for critical changes. Each merge triggers a reset of CI checks, requiring maintainers to re-approve your PR, which adds unnecessary overhead.

Each PR should have:

  • A descriptive title, known as "commit message". In summary:
    • Limit the title to 50 characters
    • Capitalize the title
    • Do not end the title with a period
    • Use the imperative mood in the title
  • A description of the problem it is solving. It could be simply a reference to the corresponding issue, e.g. Resolves #123.
  • A summary of changes made to solve the problem. Explain what and why instead of how.

AI Usage Policy

Goals

This policy exists to:

  • Keep the effort balanced – Before AI, contributors did most of the work (writing, testing, understanding). We want to keep it that way. AI should help you, not replace your effort.
  • Protect maintainer time – Large, low-quality AI-generated PRs shift the burden to reviewers. We want to avoid that.
  • Ensure understanding – Contributors should understand and be able to explain every change they submit.
  • Keep conversations human – Code review is a discussion between people, not bots.

Good use of AI

  • Using AI to help you understand the code.
  • Using AI to write drafts of code, tests, or docs.
  • Using AI to explore ideas or try different approaches.
  • Saying "AI helped me write this" in your PR description.

Disallowed use of AI

  • Copy-pasting AI output without reading or understanding it.
  • Submitting AI-generated code without testing.
  • Using AI to reply to review comments – reviewers want to talk to you, not a bot.

Your responsibility

  • You own everything you submit, even if AI wrote it.
  • You must understand your code well enough to explain it.
  • You must run tests locally before opening or updating a PR.
  • If AI wrote a big part of your PR, mention that in the PR description.

Enforcement

  • PRs that look like low-effort AI slop will be closed.
  • Repeated violators may be banned from the project.

Pull Request Limits for New Contributors

To ensure high-quality code reviews and long-term codebase stability, we limit the number of simultaneous open PRs for new contributors.

The Policy

Your limit of simultaneous open PRs is based on your history with this project:

Merged PRs in this project Max Simultaneous Open PRs
0 (First-time contributor) 1
1 merged PR 2
2 merged PRs 3
3+ merged PRs Unlimited

Why We Do This

AI tools have dramatically reduced the cost of creating pull requests, but the burden on maintainers for reviewing them remains the same. Large-scale or complex refactors require significant effort to review, and a high volume of PRs from new contributors often leads to:

  • Review Bottlenecks: Quality reviews take time. A flood of PRs prevents us from giving any single PR the attention it deserves.
  • Context Fragmentation: Refactoring legacy code requires deep understanding. We prefer to work with you on one area of the code at a time to ensure the architectural direction is correct.
  • Reduced Noise: This policy helps us distinguish between intentional improvements and automated "bulk" refactors that may introduce subtle regressions.

How to Proceed

If you reach your limit, please focus on addressing feedback and merging your existing PR(s) before opening new ones. PRs opened in excess of these limits may be labeled as on-hold or closed to keep our backlog manageable.

Tips for Success

  • Verify AI Output: If you use AI tools to assist in refactoring, you are responsible for manually verifying every line. We expect contributors to be able to explain the "why" behind every change.
  • Small over Large: Smaller, atomic PRs are much more likely to be merged quickly than large refactors.

License

By contributing your code, you agree to license your contribution under the terms of the Apache License.

Copyright Header

If you are adding a new file it should have a header like below. In some languages, e.g. Python, you may need to change the comments to start with #. The easiest way is to copy the header from one of the existing source files and make sure the year is current and the copyright says "The Jaeger Authors".

// Copyright (c) 2026 The Jaeger Authors.
// SPDX-License-Identifier: Apache-2.0

Never remove existing copyright headers.

Some files may have other copyright headers, such as:

// Copyright (c) 2017 Uber Technologies, Inc.

If you are modifying such a file you may add Jaeger copyright on top:

// Copyright (c) 2026 The Jaeger Authors.
// Copyright (c) 2017 Uber Technologies, Inc.

Certificate of Origin - Sign your work

By contributing to this project you agree to the Developer Certificate of Origin (or simply DCO). This document was created by the Linux Kernel community and is a simple statement that you, as a contributor, have the legal right to make the contribution.

The sign-off is a simple line at the end of the explanation for the patch, which certifies that you wrote it or otherwise have the right to pass it on as an open-source patch. The rules are pretty simple: if you can certify the conditions in the DCO, then just add a line to every git commit message:

Signed-off-by: Bender Bending Rodriguez <bender.is.great@gmail.com>

using your real name (sorry, no pseudonyms or anonymous contributions.) You can add the sign off when creating the git commit via git commit -s.

Missing sign-offs

Note that every commit in the pull request must be signed. Jaeger repositories are configured with a DCO-bot that will check sign-offs on every commit and block the PR from being merged if some commits are missing sign-offs. If you only have one commit or the latest commit in the PR is missing a sign-off, the simplest way to fix this is to run:

git commit --amend -s

which will prompt you to edit the commit message while adding a signature. Simply accept the text as is, and push the branch:

git push --force

If some commit in the middle of your commit history is missing the sign-off, the simplest solution is to squash the commits into one and sign it. For example, suppose that your branch history looks like this:

fe43631 - Fix HotROD Docker command
933efb3 - Add files for ingester
214c133 - Rename gas to gosec
0a40309 - Update Makefile build_ui target to lerna structure
7919cd9 - Add support for Cassandra reconnect interval
a0dc40e - Fix deploy step
77a0573 - (tag: v1.6.0) Prepare release 1.6.0

Let's assume that the first commit 77a0573 was the commit before you started work on your PR, and commits from a0dc40e to fe43631 are your changes that you want to squash. You can run the soft reset command:

git reset --soft 77a0573

It will undo all changes after commit 77a0573 and stage them. You can commit them all at once while adding the signature:

git commit -s -m 'your commit message, e.g. the PR title'

Then push the branch:

git push --force

Branches

Before submitting a PR make sure to create a named branch in your forked repository. Our CI will fail if you submit a PR from the main branch. If that happens, just create a new branch and re-submit the PR from that branch.