fab-city-software-kit/CONTRIBUTING.md

Contributing

First of all, thank you for contributing to FabCityOS! The goal of this document is to provide everything you need to know in order to contribute to FabCityOS and its different integrations.

• Assumptions
• How to Contribute
• Development Workflow
• Git Guidelines
• Release Process (for internal team only)

Assumptions

1. You're familiar with Gitlab and the Merge Request(PR) workflow.
2. You've read the FabCityOS documentation and the README.
3. You know about the FabCityOS community. Please use this for help.

How to Contribute

1. Make sure that the contribution you want to make is explained or detailed in a GitHub issue! Find an existing issue or open a new one.
2. Once done, fork the FabCityOS Core Chart repository in your Gitlab account. Ask a maintainer if you want your issue to be checked before making a PR.
3. Create a new Git branch.
4. Review the Development Workflow section that describes the steps to maintain the repository.
5. Make the changes on your branch.
6. Submit the branch as a PR pointing to the main branch of the main fabcity-os-core-chart repository. A maintainer should comment and/or review your Pull Request within a few days. Although depending on the circumstances, it may take longer.
   We do not enforce a naming convention for the PRs, but please use something descriptive of your changes, having in mind that the title of your PR will be automatically added to the next release changelog.

Development Workflow

Setup

In order to use the different tools on this repository, you will first need to:

• Install Kubectl on your machine. This will allow you to run commands against your Kubernetes cluster with the kubectl command.

• Install Helm. This will allow you to run the helm command, enabling the installation and manipulation of Helm charts.

• A Kubernetes cluster up and running. Any Kubernetes cluster can be used, on the cloud or locally installed on your machine. This can be easily achieved by installing k3d, and creating a local cluster by running:

• Optional: Use the Nix Package Manager. This will allow you to make use of the nixpkgs, automatically managing all required dependencies from the nix shell env. In addition you also need the nixpkgs: direnv for automatic environment selection.

Setup the development Cluster

k3d cluster create --config test/k3d-fcos-dev.yaml
k3d cluster list
k3d node list
kubectl get --output wide nodes
kubectl get --all-namespaces pods

Deploy FabCityOS to the the development Cluster

kubectl kustomize overlays/dev --enable-helm | kubectl apply -f -

Watch all resources

watch -n 1 kubectl get all --all-namespaces

Remove the development Cluster

k3d cluster delete --all

Tests and Linter

Each PR should pass the linter to be accepted.

# Linter
helm lint .

Each PR should also check if the generated file manifests/fcos.yaml is updated with the new modifications. You can generated the manifest with the command line:

helm template fcos . | grep -v 'helm.sh/chart:\|app.kubernetes.io/managed-by:' > manifests/FabCityOS.yaml

Or just by comment the PR:

@FabCityOS sync-manifest

Additionally, the CI will run a test to check if there are changes introduced to the charts. If changes were introduced, it will require you to update the Chart version.

Git Guidelines

Git Branches

All changes must be made in a branch and submitted as PR. We do not enforce any branch naming style, but please use something descriptive of your changes.

Git Commits

As minimal requirements, your commit message should:

• be capitalized
• not finish by a dot or any other punctuation character (!,?)
• start with a verb so that we can read your commit message this way: "This commit will ...", where "..." is the commit message. e.g.: "Fix the home page button" or "Add more tests for create_index method"

We don't follow any other convention, but if you want to use one, we recommend this one.

Pull Requests

Some notes on PRs:

• Convert your PR as a draft if your changes are a work in progress: no one will review it until you pass your PR as ready for review.
  The draft PR can be very useful if you want to show that you are working on something and make your work visible.
• The branch related to the PR must be up-to-date with main before merging. Fortunately, this project integrates a bot to automatically enforce this requirement without the PR author having to do it manually.
• All PRs must be reviewed and approved by at least one maintainer.
• The PR title should be accurate and descriptive of the changes. The title of the PR will be indeed automatically added to the next release changelogs.

Release Process (for internal team only)

FabCityOS tools follow the Semantic Versioning Convention.

Automation to Rebase and Merge the PRs

This project integrates a bot that helps us manage pull requests merging.
Read more about this.

How to Publish the Release

⚠️ Before doing anything, make sure you got through the guide about Releasing an Integration.

⚠️ Every PR that is merged to main introducing changes to the Helm Chart needs to modify the file charts/fcos/Chart.yaml, by increasing the version of the chart accordingly.

Every PR that is merged to main triggers the automated release process, as specified at ``. A GitHub Action will be triggered and publish a new release on the GitHub repository releases. This will enable users to start using the new version of the chart immediately after publishing.

---

Thank you again for reading this through, we can not wait to begin to work with you if you made your way through this contributing guide ❤️