Skip to content

Latest commit

 

History

History
95 lines (76 loc) · 4.32 KB

CONTRIBUTING.md

File metadata and controls

95 lines (76 loc) · 4.32 KB
Raw

Contributions Welcome

Firecracker is running serverless workloads at scale within AWS, but it's still day 1 on the journey guided by our mission. There's a lot more to build and we welcome all contributions.

There's a lot to contribute to in Firecracker. We've opened issues for all the features we want to build and improvements we want to make. Good first issues are labeled accordingly. We're also keen to hearing about your use cases and how we can support them, your ideas, and your feedback for what's already here.

If you're just looking for quick feedback for an idea or proposal, open an issue or chat with us on the Firecracker Slack workgroup.

Follow the contribution workflow for submitting your changes to the Firecracker codebase. If you want to receive high-level but still commit-based feedback for a contribution, follow the request for comments steps instead.

Contribution Workflow

Firecracker uses the “fork-and-pull” development model. Follow these steps if you want to merge your changes to Firecracker:

  1. Within your fork of Firecracker, create a branch for your contribution. Use a meaningful name.
  2. Create your contribution, meeting all contribution quality standards
  3. Create a pull request against the main branch of the Firecracker repository.
  4. Add two reviewers to your pull request (a maintainer will do that for you if you're new). Work with your reviewers to address any comments and obtain a minimum of 2 approvals, at least one of which must be provided by a maintainer. To update your pull request amend existing commits whenever applicable and then push the new changes to your pull request branch.
  5. Once the pull request is approved, one of the maintainers will merge it.

Request for Comments

If you just want to receive feedback for a contribution proposal, open an “RFC” (“Request for Comments”) pull request:

  1. On your fork of Firecracker, create a branch for the contribution you want feedback on. Use a meaningful name.
  2. Create your proposal based on the existing codebase.
  3. Create a pull request against the main branch of the Firecracker repository. Prefix your pull request name with [RFC].
  4. Discuss your proposal with the community on the pull request page (or on any other channel). Add the conclusion(s) of this discussion to the pull request page.

Contribution Quality Standards

Most quality and style standards are enforced automatically during integration testing. For ease of use you can setup a git pre-commit hook by running the following in the Firecracker root directory:

cargo install rusty-hook
rusty-hook init

Your contribution needs to meet the following standards:

  • Separate each logical change into its own commit.

  • Each commit must pass all unit & code style tests, and the full pull request must pass all integration tests. See tests/README.md for information on how to run tests.

  • Unit test coverage must increase the overall project code coverage.

  • Include integration tests for any new functionality in your pull request.

  • Document all your public functions.

  • Add a descriptive message for each commit. Follow commit message best practices.

  • A good commit message may look like

    A descriptive title of 50 characters or fewer

A concise description where each line is 72 characters or fewer.

Signed-off-by: <A full name> <A email>
Co-authored-by: <B full name> <B email>

  • Document your pull requests. Include the reasoning behind each change, and the testing done.

  • Acknowledge Firecracker's Apache 2.0 license and certify that no part of your contribution contravenes this license by signing off on all your commits with git -s. Ensure that every file in your pull request has a header referring to the repository license file.