mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-27 16:50:23 +00:00
136 lines
4.7 KiB
Markdown
136 lines
4.7 KiB
Markdown
|
# Contributing to opentelemetry-go-contrib
|
||
|
|
|
||
|
|
The Go special interest group (SIG) meets regularly. See the
|
||
|
|
OpenTelemetry
|
||
|
|
[community](https://github.com/open-telemetry/community#golang-sdk)
|
||
|
|
repo for information on this and other language SIGs.
|
||
|
|
|
||
|
|
See the [public meeting
|
||
|
|
notes](https://docs.google.com/document/d/1A63zSWX0x2CyCK_LoNhmQC4rqhLpYXJzXbEPDUQ2n6w/edit#heading=h.9tngw7jdwd6b)
|
||
|
|
for a summary description of past meetings. To request edit access,
|
||
|
|
join the meeting or get in touch on
|
||
|
|
[Slack](https://cloud-native.slack.com/archives/C01NPAXACKT).
|
||
|
|
|
||
|
|
## Development
|
||
|
|
|
||
|
|
There are some generated files checked into the repo. To make sure
|
||
|
|
that the generated files are up-to-date, run `make` (or `make
|
||
|
|
precommit` - the `precommit` target is the default).
|
||
|
|
|
||
|
|
The `precommit` target also fixes the formatting of the code and
|
||
|
|
checks the status of the go module files.
|
||
|
|
|
||
|
|
If after running `make precommit` the output of `git status` contains
|
||
|
|
`nothing to commit, working tree clean` then it means that everything
|
||
|
|
is up-to-date and properly formatted.
|
||
|
|
|
||
|
|
## Pull Requests
|
||
|
|
|
||
|
|
### How to Send Pull Requests
|
||
|
|
|
||
|
|
Everyone is welcome to contribute code to `opentelemetry-go-contrib` via
|
||
|
|
GitHub pull requests (PRs).
|
||
|
|
|
||
|
|
To create a new PR, fork the project in GitHub and clone the upstream
|
||
|
|
repo:
|
||
|
|
|
||
|
|
```sh
|
||
|
|
$ git clone https://github.com/open-telemetry/opentelemetry-go-contrib
|
||
|
|
```
|
||
|
|
This would put the project in the `opentelemetry-go-contrib` directory in
|
||
|
|
current working directory.
|
||
|
|
|
||
|
|
Enter the newly created directory and add your fork as a new remote:
|
||
|
|
|
||
|
|
```sh
|
||
|
|
$ git remote add <YOUR_FORK> git@github.com:<YOUR_GITHUB_USERNAME>/opentelemetry-go
|
||
|
|
```
|
||
|
|
|
||
|
|
Check out a new branch, make modifications, run linters and tests, update
|
||
|
|
`CHANGELOG.md` and push the branch to your fork:
|
||
|
|
|
||
|
|
```sh
|
||
|
|
$ git checkout -b <YOUR_BRANCH_NAME>
|
||
|
|
# edit files
|
||
|
|
# update changelog
|
||
|
|
$ make precommit
|
||
|
|
$ git add -p
|
||
|
|
$ git commit
|
||
|
|
$ git push <YOUR_FORK> <YOUR_BRANCH_NAME>
|
||
|
|
```
|
||
|
|
|
||
|
|
Open a pull request against the main `opentelemetry-go-contrib` repo. Be sure to add the pull
|
||
|
|
request ID to the entry you added to `CHANGELOG.md`.
|
||
|
|
|
||
|
|
### How to Receive Comments
|
||
|
|
|
||
|
|
* If the PR is not ready for review, please put `[WIP]` in the title,
|
||
|
|
tag it as `work-in-progress`, or mark it as
|
||
|
|
[`draft`](https://github.blog/2019-02-14-introducing-draft-pull-requests/).
|
||
|
|
* Make sure CLA is signed and CI is clear.
|
||
|
|
|
||
|
|
### How to Get PRs Merged
|
||
|
|
|
||
|
|
A PR is considered to be **ready to merge** when:
|
||
|
|
|
||
|
|
* It has received two approvals from Approvers/Maintainers (at
|
||
|
|
different companies).
|
||
|
|
* Feedback has been addressed.
|
||
|
|
* Any substantive changes to your PR will require that you clear any prior
|
||
|
|
Approval reviews, this includes changes resulting from other feedback. Unless
|
||
|
|
the approver explicitly stated that their approval will persist across
|
||
|
|
changes it should be assumed that the PR needs their review again. Other
|
||
|
|
project members (e.g. approvers, maintainers) can help with this if there are
|
||
|
|
any questions or if you forget to clear reviews.
|
||
|
|
* It has been open for review for at least one working day. This gives
|
||
|
|
people reasonable time to review.
|
||
|
|
* Trivial change (typo, cosmetic, doc, etc.) doesn't have to wait for
|
||
|
|
one day.
|
||
|
|
* `CHANGELOG.md` has been updated to reflect what has been
|
||
|
|
added, changed, removed, or fixed.
|
||
|
|
* Urgent fix can take exception as long as it has been actively
|
||
|
|
communicated.
|
||
|
|
|
||
|
|
Any Maintainer can merge the PR once it is **ready to merge**.
|
||
|
|
|
||
|
|
## Style Guide
|
||
|
|
|
||
|
|
* Make sure to run `make precommit` - this will find and fix the code
|
||
|
|
formatting.
|
||
|
|
* Check [opentelemetry-go Style Guide](https://github.com/open-telemetry/opentelemetry-go/blob/main/CONTRIBUTING.md#style-guide)
|
||
|
|
|
||
|
|
## Adding a new Contrib package
|
||
|
|
|
||
|
|
To add a new contrib package follow an existing one. An empty Sample instrumentation
|
||
|
|
provides base structure with an example and a test. Each contrib package
|
||
|
|
should be its own module. A contrib package may contain more than one go package.
|
||
|
|
|
||
|
|
### Folder Structure
|
||
|
|
- instrumentation/\<instrumentation-package> (**Common**)
|
||
|
|
- instrumentation/\<instrumentation-package>/trace (**specific to trace**)
|
||
|
|
- instrumentation/\<instrumentation-package>/metrics (**specific to metrics**)
|
||
|
|
|
||
|
|
#### Example
|
||
|
|
- instrumentation/gorm/trace
|
||
|
|
- instrumentation/kafka/metrics
|
||
|
|
|
||
|
|
## Approvers and Maintainers
|
||
|
|
|
||
|
|
Approvers:
|
||
|
|
|
||
|
|
- [Evan Torrie](https://github.com/evantorrie), Verizon Media
|
||
|
|
- [Josh MacDonald](https://github.com/jmacd), LightStep
|
||
|
|
- [Sam Xie](https://github.com/XSAM)
|
||
|
|
- [David Ashpole](https://github.com/dashpole), Google
|
||
|
|
- [Gustavo Silva Paiva](https://github.com/paivagustavo), LightStep
|
||
|
|
|
||
|
|
Maintainers:
|
||
|
|
|
||
|
|
- [Anthony Mirabella](https://github.com/Aneurysm9), Centene
|
||
|
|
- [Tyler Yahn](https://github.com/MrAlias), New Relic
|
||
|
|
|
||
|
|
### Become an Approver or a Maintainer
|
||
|
|
|
||
|
|
See the [community membership document in OpenTelemetry community
|
||
|
|
repo](https://github.com/open-telemetry/community/blob/main/community-membership.md).
|