Skip to content

Latest commit

 

History

History
154 lines (112 loc) · 5.63 KB

CONTRIBUTING.md

File metadata and controls

154 lines (112 loc) · 5.63 KB
Raw

Contributing

Flux is Apache 2.0 licensed and accepts contributions via GitHub pull requests. This document outlines some of the conventions on to make it easier to get your contribution accepted.

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

Certificate of Origin

By contributing to this project you agree to the Developer Certificate of Origin (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.

We require all commits to be signed. By signing off with your signature, you certify that you wrote the patch or otherwise have the right to contribute the material by the rules of the DCO:

Signed-off-by: Jane Doe <[email protected]>

The signature must contain your real name (sorry, no pseudonyms or anonymous contributions) If your user.name and user.email are configured in your Git config, you can sign your commit automatically with git commit -s.

Communications

For realtime communications we use Slack: To join the conversation, simply join the CNCF Slack workspace and use the #flux-contributors channel.

To discuss ideas and specifications we use Github Discussions.

For announcements we use a mailing list as well. Simply subscribe to flux-dev on cncf.io to join the conversation (there you can also add calendar invites to your Google calendar for our Flux meeting).

Understanding Flux and the GitOps Toolkit

If you are entirely new to Flux and the GitOps Toolkit, you might want to take a look at the introductory talk and demo.

This project is composed of:

Understanding the code

To get started with developing controllers, you might want to review our guide which walks you through writing a short and concise controller that watches out for source changes.

How to run the test suite

Prerequisites:

  • go >= 1.19
  • kubectl >= 1.20
  • kustomize >= 4.4
  • coreutils (on Mac OS)

Install the controller-runtime/envtest binaries with:

make install-envtest

Then you can run the unit tests with:

make test

After installing Kubernetes kind on your machine, create a cluster for testing with:

make setup-kind

Then you can run the end-to-end tests with:

make e2e

When the output of the Flux CLI changes, to automatically update the golden files used in the test, pass -update flag to the test as:

make e2e TEST_ARGS="-update"

Since not all packages use golden files for testing, -update argument must be passed only for the packages that use golden files. Use the variables TEST_PKG_PATH for unit tests and E2E_TEST_PKG_PATH for e2e tests, to set the path of the target test package:

# Unit test
make test TEST_PKG_PATH="./cmd/flux" TEST_ARGS="-update"
# e2e test
make e2e E2E_TEST_PKG_PATH="./cmd/flux" TEST_ARGS="-update"

Teardown the e2e environment with:

make cleanup-kind

Acceptance policy

These things will make a PR more likely to be accepted:

  • a well-described requirement
  • tests for new code
  • tests for old code!
  • new code and tests follow the conventions in old code and tests
  • a good commit message (see below)
  • all code must abide Go Code Review Comments
  • names should abide What's in a name
  • code must build on both Linux and Darwin, via plain go build
  • code should have appropriate test coverage and tests should be written to work with go test

In general, we will merge a PR once one maintainer has endorsed it. For substantial changes, more people may become involved, and you might get asked to resubmit the PR or divide the changes into more than one PR.

Format of the Commit Message

For the GitOps Toolkit controllers we prefer the following rules for good commit messages:

  • Limit the subject to 50 characters and write as the continuation of the sentence "If applied, this commit will ..."
  • Explain what and why in the body, if more than a trivial change; wrap it at 72 characters.

The following article has some more helpful advice on documenting your work.