# Contributing
Issues, whether bugs, tasks, or feature requests are essential for keeping Polaris great. We believe it should be as easy as possible to contribute changes that get things working in your environment. There are a few guidelines that we need contributors to follow so that we can keep on top of things.
# Code of Conduct
This project adheres to a code of conduct. Please review this document before contributing to this project.
# Sign the CLA
Before you can contribute, you will need to sign the Contributor License Agreement (opens new window).
# Project Structure
Polaris is built on top of controller-runtime (opens new window). It can run in 3 different modes, a dashboard, a webhook, or a reporter that prints or exports validation results. All of these modes make use of the shared validator
and config
packages. Adding new validations is possible by only making additions to those packages.
# Getting Started
We label issues with the "good first issue" tag (opens new window) if we believe they'll be a good starting point for new contributors. If you're interested in working on an issue, please start a conversation on that issue, and we can help answer any questions as they come up.
# Setting Up Your Development Environment
# Prerequisites
- A properly configured Golang environment with Go 1.11 or higher
- If you want to see the local changes you make on a Polaris dashboard, you will need access to a Kubernetes cluster defined in
~/.kube/config
# Installation
- Install the project with
go get github.com/fairwindsops/polaris
- Change into the polaris directory which is installed at
$GOPATH/src/github.com/fairwindsops/polaris
- See the dashboard with
go run main.go dashboard
, then open http://localhost:8080/ - See the audit data
go run main.go audit
. This command shows the audit information on the command line.
# Running Tests
The following commands are all required to pass as part of Polaris testing:
go list ./... | grep -v vendor | xargs golint -set_exit_status
go list ./... | grep -v vendor | xargs go vet
go test ./pkg/... -v -coverprofile cover.out
# Creating a New Issue
If you've encountered an issue that is not already reported, please create a new issue (opens new window), choose Bug Report
, Feature Request
or Misc.
and follow the instructions in the template.
# Creating a Pull Request
Each new pull request should:
- Reference any related issues
- Add tests that show the issues have been solved
- Pass existing tests and linting
- Contain a clear indication of if they're ready for review or a work in progress
- Be up to date and/or rebased on the master branch
# Creating a new release
# Patch releases
Patch releases only need to change this repo. The Helm chart and deploy scripts will automatically pull in the latest changes.
If the release involves changes to anything in the deploy/
folder (e.g. new RBAC permissions),
it needs to be a minor or major release in order to prevent breaking the Helm chart.
- Create a PR for this repo
- Bump the version number in:
- main.go
- README.md
- Update CHANGELOG.md
- Merge your PR
- Bump the version number in:
- Tag the latest branch for this repo
- Pull the latest commit for the
master
branch (which you just merged in your PR) - Run
git tag $VERSION && git push --tags
- Make sure CircleCI runs successfully for the new tag - this will push images to quay.io and create a release in GitHub
- If CircleCI fails, check with Codeowners ASAP
- Pull the latest commit for the
# Minor/Major releases
Minor and major releases need to change both this repository and the Helm chart repo (opens new window).
The steps are:
- Modify the Helm chart (opens new window)
- Clone the helm charts repo
git clone https://github.com/FairwindsOps/charts
git checkout -b yourname/update-polaris
- Bump the version number in:
- stable/polaris/README.md
- stable/polaris/Chart.yaml
- stable/polaris/values.yaml
- Make any necessary changes to the chart to support the new version of Polaris (e.g. new RBAC permissions)
- Don't merge yet!
- Clone the helm charts repo
- Create a PR for this repo
- Create a new branch named
yourname/update-version
- Bump the version number in:
- main.go
- README.md
- Regenerate the deployment files. Assuming you've cloned the charts repo to
~/git/charts
:CHARTS_DIR=~/git/charts ./scripts/generate-deployment-files.sh
- Update CHANGELOG.md
- Merge your PR
- Create a new branch named
- Tag the latest branch for this repo
- Pull the latest for the
master
branch - Run
git tag $VERSION && git push --tags
- Make sure CircleCI runs successfully for the new tag - this will push images to quay.io and create a release in GitHub
- If CircleCI fails, check with Codeowners ASAP
- Pull the latest for the
- Create and merge a PR for your changes to the Helm chart