Startseite Erkunden Hilfe
Anmelden
akillibulut
/
cost-model
Mirror von https://github.com/kubecost/cost-model
2
0
Fork 0
Dateien Issues 0 Wiki
Branch: develop
Branches Tags
cost-model
/
CONTRIBUTING.md

CONTRIBUTING.md 9.8 KB
Permalink Verlauf Originalformat

Contributing to our project

Thanks for your help improving the OpenCost project! There are many ways to contribute to the project, including the following:

Getting Help

If you have a question about OpenCost or have encountered problems using it, you can start by asking a question on CNCF Slack in the #opencost channel or attend the biweekly OpenCost Working Group community meeting from the Community Calendar to discuss OpenCost development.

Workflow

This repository's contribution workflow follows a typical open-source model:

Building OpenCost

Follow these steps to build the OpenCost cost-model and UI from source and deploy. The provided build tooling is natively multi-architecture (built images will run on both AMD64 and ARM64 clusters).

Dependencies:

  1. Docker (with buildx)
  2. just (if you don't want to install it , Just read the justfile and run the commands manually)
  3. Multi-arch buildx builders set up via https://github.com/tonistiigi/binfmt
  4. manifest-tool via https://github.com/estesp/manifest-tool
  5. npm (if you want to build the UI)

Build the backend

  1. just build "<repo>/opencost:<tag>"
  2. Edit the pulled image in the kubernetes/opencost.yaml to <repo>/opencost:<tag>
  3. Set this environment variable to the address of your Prometheus server

Build the frontend

  1. cd ui && just build "<repo>/opencost-ui:<tag>"
  2. Edit the pulled image in the kubernetes/opencost.yaml to <repo>/opencost-ui:<tag>

Deploy to a cluster

  1. kubectl create namespace opencost
  2. kubectl apply -f kubernetes/opencost --namespace opencost
  3. kubectl -n opencost port-forward service/opencost 9090 9003

To test, build the OpenCost containers and then push them to a Kubernetes cluster with a running Prometheus.

To confirm that the server and UI are running, you can hit http://localhost:9090 to access the OpenCost UI. You can test the server API with curl http://localhost:9003/allocation/compute -d window=60m -G.

Running locally

To run locally cd into cmd/costmodel and go run main.go

OpenCost requires a connection to Prometheus in order to operate so setting the environment variable PROMETHEUS_SERVER_ENDPOINT is required. In order to expose Prometheus to OpenCost it may be required to port-forward using kubectl to your Prometheus endpoint.

For example:

kubectl port-forward svc/prometheus-server 9080:80

This would expose Prometheus on port 9080 and allow setting the environment variable as so:

PROMETHEUS_SERVER_ENDPOINT="http://127.0.0.1:9080"

If you want to run with a specific kubeconfig the environment variable KUBECONFIG can be used. OpenCost will attempt to connect to your Kubernetes cluster in a similar fashion as kubectl so the env is not required. The order of precedence is KUBECONFIG > default kubeconfig file location ($HOME/.kube/config) > in cluster config

Example:

export KUBECONFIG=~/.kube/config

An example of the full command:

PROMETHEUS_SERVER_ENDPOINT="http://127.0.0.1:9090" go run main.go

Code Formatting

Before submitting a pull request, ensure your code is properly formatted:

# Format all Go code
go fmt ./...

# Or use just command
just fmt

To check if your code is formatted without making changes:

just fmt-check

The CI pipeline will automatically check code formatting on pull requests.

Testing code

Testing is provided by the just test command which runs

Running the integration tests

To run these tests:

  • Make sure you have a kubeconfig that can point to your cluster, and have permissions to create/modify a namespace called "test"
  • Connect to your the Prometheus OpenCost emits to on localhost:9003: kubectl port-forward --namespace opencost service/prometheus-server 9003:80
  • Temporary workaround: Copy the default.json file in this project at cloud/default.json to /models/default.json on the machine your test is running on. TODO: fix this and inject the cloud/default.json path into provider.go.
  • Navigate to cost-model/test
  • Run go test -timeout 700s from the testing directory. The tests right now take about 10 minutes (600s) to run because they bring up and down pods and wait for Prometheus to scrape data about them.

Code Review Standards

All pull requests must be reviewed before merging. The review process ensures:

What reviewers check:

  • Correctness: Does the code do what it claims?
  • Tests: Are new features and bug fixes covered by tests?
  • Style: Does the code follow Go conventions (gofmt, go vet)?
  • Security: Are inputs validated? Are credentials handled safely?
  • Performance: Are there obvious performance issues (unbounded allocations, N+1 queries)?

Review requirements:

  • At least one approval from a Committer or Maintainer is required
  • The reviewer must be a different person than the PR author
  • For security-sensitive changes, review by a Maintainer is required
  • Emergency fixes may bypass review with post-merge review required within 48 hours (per GOVERNANCE.md)

Regression Tests

When fixing a bug, contributors SHOULD add a test that reproduces the bug before applying the fix. This ensures the bug does not recur. As a project-wide goal, at least 50% of bugs fixed in any six-month window should have corresponding regression tests. This is tracked by maintainers using issues labeled bug and measured during release reviews; it is an aspirational target for the project as a whole, not a requirement applied to individual contributors.

Finding Issues to Work On

Look for issues labeled good first issue or help wanted for a curated list of tasks suitable for new contributors.

Certificate of Origin

By contributing to this project, you certify that your contribution was created in whole or in part by you and that you have the right to submit it under the open source license indicated in the project. In other words, please confirm that you, as a contributor, have the legal right to make the contribution. This is enforced on Pull Requests and requires Signed-off-by with the email address for the author in the commit message.

Committing

Please write a commit message with Fixes Issue # if there is an outstanding issue that is fixed. It's okay to submit a PR without a corresponding issue; just please try to be detailed in the description of the problem you're addressing.

Code Formatting: All code must be formatted with go fmt ./... (or just fmt) before submitting. The CI pipeline will reject PRs with unformatted code. You can run just fmt-check to verify formatting locally.

Code Quality: While lint warnings are acceptable in some cases (e.g., comments on exported functions are nice but not strictly required), please address any critical issues reported by go vet.

Please reach us on CNCF Slack in the #opencost channel or attend the biweekly OpenCost Working Group community meeting from the Community Calendar to discuss OpenCost development.

Third Party Attributions

The THIRD_PARTY_LICENSES.txt file should contain the up-to-date license, copyright, and notice information for dependencies used by Opencost. When adding, updating, or removing dependencies, please update the associate section(s) of the THIRD_PARTY_LICENSES.txt file.

For example, the github.com/opencost/opencost/core dependency contains the following third party attributions. The license associated with the SPDX identifier should also be included in the attributions file, if it is not already present.

== Dependency
github.com/opencost/opencost/core

== License Type
SPDX:Apache-2.0

== Copyright
Copyright 2019 - 2022 Stackwatch Incorporated. All Rights Reserved.
Copyright 2022 - 2024 Cloud Native Computing Foundation

== Notices
OpenCost
Copyright 2022 - 2024 Cloud Native Computing Foundation

This product includes software developed at
The Cloud Native Computing Foundation (http://www.cncf.io).

The Initial Developer of some parts of the specification and project is
Kubecost (http://www.kubecost.com).
Copyright 2019 - 2022 Stackwatch Incorporated. All Rights Reserved.