cdk8s is licensed under Apache 2.0 and accepts contributions via GitHub pull requests. This document outlines some of the conventions on commit message formatting, contact points for developers, and other resources to help get contributions into cdk8s.
You can contribute to cdk8s in many ways. Contributions of all shapes and sizes are welcome and celebrated:
We follows the CNCF Community Code of Conduct
If any part of the project has bugs or documentation mistakes, please let us know by raising an issue. We treat bugs and mistakes very seriously and believe no issue is too small. Before creating a bug report, please check that an issue reporting the same problem does not already exist.
An issue can either be a bug report or a suggestion. If you wish to ask a question or seek guidance, please consider one of the other support channels.
To make the bug report accurate and easy to understand, please try to create bug reports that are:
- Specific: Include as much details as possible: which version, what environment, what configuration, etc.
- Reproducible: Include the steps to reproduce the problem. We understand some issues might be hard to reproduce, please includes the steps that might lead to the problem. If possible, please provide a minimal code snippet that reproduces the bug.
- Isolated: Please try to isolate and reproduce the bug with minimum dependencies. It would significantly slow down the speed to fix a bug if too many dependencies are involved in a bug report.
- Unique: Do not duplicate existing bug report.
- Scoped: One bug per report. Do not follow up with another bug inside one report.
We might ask for further information to locate a bug. A duplicated bug report will be closed.
Submit a bug report here.
We also accept suggestions for new features or missing capabilities as GitHub issues. The most important aspect of a suggestion issue is to provide as many details as possible about your use case and less focus on the solution. It is usually possible to support different use cases in many different ways, and we need to understand the motivation before we dive into a solution.
If you wish to suggest a major change to the project, please consider to submit an RFC instead of a simple issue. An RFC also starts with a GitHub issue.
Submit a suggestion here
The general workflow for code contributions:
- Submit/find an issue
- Clone this repo
- Make your code change
- Write tests & update docs
- Build & test locally
- Submit a pull request
- (Iterate)
- Your PR will be approved and merged
All pull requests should be tracked with a GitHub issue.
You should search for an existing issue or raise a new bug or suggestion.
Add a comment indicating you are willing to pick it up in order to ensure no one else is currently working on it.
If this is a major contribution, consider submitting an RFC to obtain feedback from the community and maintainers.
Prerequisites:
Prepare your environment:
- Fork this repo and obtain a local clone.
- Install all dependencies:
yarn install - Run
yarn buildto build all modules.
The script ./tools/link-all.sh can be used to generate symlinks to all modules in this repository under some node_module
directory. This can be used to develop against this repo as a local dependency.
One can use the postinstall script to symlink this repo:
{
"scripts": {
"postinstall": "../cdk8s/tools/link-all.sh"
}
}This assumes this repo is a sibling of the target repo and will install the CDK as a linked dependency during yarn install.
Unit tests are located under the test/ directory within each module and use the jest framework.
To run unit tests, execute yarn test either from the root of the repo (to unit test all modules) or from individual module directories:
yarn test
Out tests utilize jest snapshot testing. In case a snapshot needs to be updated, just run:
yarn test -u
Integration tests are executed after we bundle the release. This means that in order to execute integration tests you'll need to create a bundle by running the following command from the root of the repo:
yarn run package
This will result in ./dist that contains all the ready-to-publish artifacts.
Now, you can run integration tests via:
yarn integ
Our integration tests also utilize snapshot testing. To update integration test snapshots, run:
yarn integ:update
The test directory contains integration tests for the cdk8s project.
Each subdirectory represents a single test, with an entrypoint of test.sh.
Tests are written as simple shell scripts and can simulate user activity.
You can either run individual tests by executing their entrypoint directly (e.g.
test-python-app/test.sh) or run all tests by executing the script
./test-all.sh.
Tests assume the cdk8s CLI is installed and in the PATH, and will use the same
version of the cdk8s module (this is the behavior of cdk8s init).
You can also execute a test (or all of them) against the dist build artifact:
$ yarn install
$ yarn build
$ yarn run package # creates "dist/"
$ cd test
$ ./run-against-dist ./test-all.sh
# or
$ ./run-against-dist ./test-typescript-app/test.sh- Create a new subdirectory with a
test-prefix. - Create a file named
test.sh, make it executable.
Test Environment:
- The script
test.shis executed within a temporary working directory under /tmp/xxxx/test (where xxxx is some random tmp file). - See existing tests as examples on how to bring in auxiliary files to the test.
- Test MUST NOT install any dependencies or the
cdk8sCLI. They can expect it to be available in the environment. - To install dependencies from package managers, use
yarn,npm,pipenv,mvnandnuget. Those programs will be shimmed to allow consuming local dependencies.
Some integration tests utilize a simple snapshot testing mechanism.
To update snapshots, run tests with:
UPDATE_SNAPSHOTS=1
Or, run this from the root of the repo:
yarn integ:update
Due to the polyglot nature of the jsii tools used by cdk8s,
the toolchain requirements are somewhat more complicated than for most projects. To
help with this, you can use the jsii/superchain docker image that includes all the required tools.
In order to get an interactive shell within a superchain container you can use the following command.
docker run --rm --net=host -it -v $PWD:$PWD -w $PWD jsii/superchainThen once in the docker shell, you can package and execute the tests as normal.
$ pip install pipenv # Currently not included in jsii docker image
$ yarn build
$ yarn run package
$ yarn integ:updateNote: this may leave some files owned as the docker root user id. These will need to be cleaned up manually.
We use the PR title when we automatically generate the change log for each release. Therefore please following these guidelines to the letter:
- PR title:
- Must adhere to conventional commits.
- All lowercase with no period at the end of the title
- If this is a
fix(bug) the title should describe the bug - If this is a
feat(feature) the title should describe the feature
- PR description:
- Describe how did you fix the bug or what changes you had to make in order to implement the feature
- Indicate
fixes #NNNorresolves #NNNwith the tracking issue number. - If you had to test your change manually, describe how you tested it and paste the test results.
- If this is a breaking change, the last paragraph should describe the
breaking change with the prefix
BREAKING CHANGE: xxxxxx.
Documentation is rendered from markdown using
mkdocs-material and sourced from
the docs directory.
API documentation for cdk8s and all cdk8s-plus-* packages is auto-generated from inline
docstrings during build.
To test locally, install python3 deps:
$ pip3 install -r docs/requirements.txtAnd then:
$ mkdocs serveThis will serve a local web server with the website.
A good reference for syntax and capabilities is the mkdocs-material website.
Examples are stored under
examples and
organized according to programming language.
Every example also has an entry under
docs/examples/xxx
which describes the example and includes links to the source code (on the main
branch).
An RFC (request for comments) is a document that proposes and details a change or addition to cdk8s. It also is a process for reviewing and discussing the proposal and tracking its implementation. "Request for comments" means a request for discussion and oversight about the future of cdk8s from contributors and users. It is an open forum for suggestions, questions, and feedback.
To create an RFC follow this process:
- Create an issue which will be the tracking issue for this RFC.
- Title should represent the title of the RFC.
- Description should provide the motivation for the RFC.
- Create a markdown file based off of
rfc/0000-template.md under the
rfc/<nnnn>-<title-of-rfc>where<nnnn>is the tracking issue number and<title-of-rfc>is a symbolic name for the title. For example:rfc/0030-construct-operators.md. - File a pull request with this markdown file. The title of the PR should
indicate
rfc: <nnnn> <same as issue title>. - The RFC will be reviewed as a pull request and once merged it means it is ready for implementation.
Join us for the CDK8s community meeting which takes place the 2nd Monday of the month at 9:00am Pacific Time.
- Meeting link: https://chime.aws/7929414778
- Agenda
This section includes information that is relevant for the maintainers of the project.
The current version of the project is mastered in the root lerna.json file. All other
package.json files use 0.0.0. This allows bumping a new version without
needing to modify multiple files and avoid merge conflicts in post-release rebases.
To release a new version of cdk8s following these steps:
$ yarn bumpThis will create a new CHANGELOG entry (from conventional commits), bump the version in
package.json and create a bump commit.
Now, push to master (in the future we will release from a release branch, but in the meantime we release directly from master):
$ git push origin masterThis will trigger the release workflow which will release to all package managers and will also create a GitHub release with a tag.
Portions of this guide were adopted from the contribution guides of the AWS CDK and etcd.