c12s-kubespray/CONTRIBUTING.md
Cristian Calin e6976a54e1
add pre-commit hook to facilitate local testing (#9158)
* add pre-commit hook configuration

* add tmp.md to .gitignore

* describe the use of pre-commit hook in CONTRIBUTING.md

* fix docs/integration.md errors identified by markdownlint

* fix docs/<file>.md errors identified by markdownlint

* docs/azure-csi.md
* docs/azure.md
* docs/bootstrap-os.md
* docs/calico.md
* docs/debian.md
* docs/fcos.md
* docs/vagrant.md
* docs/gcp-lb.md
* docs/kubernetes-apps/registry.md
* docs/setting-up-your-first-cluster.md
* docs/vagrant.md
* docs/vars.md

* fix contrib/<file>.md errors identified by markdownlint
2022-08-24 06:54:03 -07:00

2.3 KiB

Contributing guidelines

How to become a contributor and submit your own code

Environment setup

It is recommended to use filter to manage the GitHub email notification, see examples for setting filters to Kubernetes Github notifications

To install development dependencies you can set up a python virtual env with the necessary dependencies:

virtualenv venv
source venv/bin/activate
pip install -r tests/requirements.txt

Linting

Kubespray uses pre-commit hook configuration to run several linters, please install this tool and use it to run validation tests before submitting a PR.

pre-commit install
pre-commit run -a  # To run pre-commit hook on all files in the repository, even if they were not modified

Molecule

molecule is designed to help the development and testing of Ansible roles. In Kubespray you can run it all for all roles with ./tests/scripts/molecule_run.sh or for a specific role (that you are working with) with molecule test from the role directory (cd roles/my-role).

When developing or debugging a role it can be useful to run molecule create and molecule converge separately. Then you can use molecule login to SSH into the test environment.

Vagrant

Vagrant with VirtualBox or libvirt driver helps you to quickly spin test clusters to test things end to end. See README.md#vagrant

Contributing A Patch

  1. Submit an issue describing your proposed change to the repo in question.
  2. The repo owners will respond to your issue promptly.
  3. Fork the desired repo, develop and test your code changes.
  4. Install pre-commit and install it in your development repo.
  5. Addess any pre-commit validation failures.
  6. Sign the CNCF CLA (https://git.k8s.io/community/CLA.md#the-contributor-license-agreement)
  7. Submit a pull request.
  8. Work with the reviewers on their suggestions.
  9. Ensure to rebase to the HEAD of your target branch and squash un-necessary commits (https://blog.carbonfive.com/always-squash-and-rebase-your-git-commits/) before final merger of your contribution.