Skip to content

Latest commit

 

History

History
47 lines (41 loc) · 2.56 KB

CONTRIBUTING.md

File metadata and controls

47 lines (41 loc) · 2.56 KB
Raw

Contributing

When contributing not obvious change to Thanos repository, please first discuss the change you wish to make via issue or slack, or any other method with the owners of this repository before making a change.

Please follow the code of conduct in all your interactions with the project.

Pull Request Process

  1. Read getting started docs and prepare Thanos.
  2. Familarize yourself with Makefile commands like format, build, proto and test.
  3. Fork improbable-eng/thanos.git and start development from your own fork. Here are sample steps to setup your development environment:
$ mkdir -p $GOPATH/src/github.com/improbable-eng
$ cd $GOPATH/src/github.com/improbable-eng
$ git clone https://github.com/<your_github_id>/thanos.git
$ cd thanos
$ git add remote upstream https://github.com/improbable-eng/thanos.git
$ git remote update
$ git merge upstream/master
$ make build
$ ./thanos -h
  1. Keep PRs as small as possible. For each of your PR, you create one branch based on the latest master. Chain them if needed (base PR on other PRs). Here are sample steps you can follow. You can get more details about the workflow from here.
$ git checkout master
$ git remote update
$ git merge upstream/master
$ git checkout -b <your_branch_for_new_pr>
$ make build
$ <Iterate your development>
$ git push origin <your_branch_for_new_pr>
  1. If you don't have a live object store ready add these envvars to skip tests for these:
  • THANOS_SKIP_GCS_TESTS to skip GCS tests.
  • THANOS_SKIP_S3_AWS_TESTS to skip AWS tests.
  • THANOS_SKIP_AZURE_TESTS to skip Azure tests.
  • THANOS_SKIP_SWIFT_TESTS to skip SWIFT tests.

If you skip all of these, the store specific tests will be run against memory object storage only. CI runs GCS and inmem tests only for now. Not having these variables will produce auth errors against GCS, AWS or Azure tests.

  1. If your change affects users (adds or removes feature) consider adding the item to CHANGELOG
  2. You may merge the Pull Request in once you have the sign-off of at least one developers with write access, or if you do not have permission to do that, you may request the second reviewer to merge it for you.
  3. If you feel like your PR waits too long for a review, feel free to ping #thanos-dev channel on our slack for review!