Home

EGI Ansible Style Guide

Community guide to developing re-usable Ansible products for EGI infrastructure

Contributing to the EGI Ansible Style Guide

Contributing

Thank you for taking the time to contribute to this project. The maintainers greatly appreciate the interest of contributors and rely on continued engagement with the community to ensure that this project remains useful. We would like to take steps to put contributors in the best possible position to have their contributions accepted. Please take a few moments to read this short guide on how to contribute; bear in mind that contributions regarding how to best contribute are also welcome.

Feedback and Questions

If you wish to discuss anything related to the project, please open an issue or start a topic on the EGI Community Forum. The maintainers will sometimes move issues off of GitHub to the community forum if it is thought that longer, more open-ended discussion would be beneficial, including a wider community scope.

Kinds of contributions

The maintainers recognise that contributions can be made in many forms, depending on the skills, experience and perspectives of interested parties. The maintainers recognise the multilingual nature of the community and welcome translations of the documentation and project supporting files. We undertake to identify these contributions through consensus-building and recognise them as formal contributions to the project where applicable. Contributions may come in the form of:

  • Feature or documentation requests, where they describe a need or gap
  • Authoring or review of releases
  • Direct authorship of code or documentation
  • Identifying and fixing bugs

Contribution Process

Before proposing a contribution via pull request, please ensure that an issue is open describing the need for your contribution. You will need to refer to this issue number when you submit the pull request.

  • It is recommended to make pull requests against release candidate branches, whenever features are involved, instead of against the master branch. See Release Cycle below.
  • Pull requests to the master branch can be made in the case obvious fixes. See Obvious Fix Policy

We have a 3 step process for contributions.

  1. Fork the project if you have not, and commit changes to a git branch
  2. Create a GitHub Pull Request for your change, following the instructions in the pull request template.
  3. Perform a Code Review with the maintainers on the pull request.

Pull Request Requirements

  1. Explain your contribution in plain language. To assist the maintainers in understanding and appreciating your pull request, please use the template to explain why you are making this contribution, rather than just what the contribution entails.
  2. This style guide is built to last. We strive to ensure high quality and long-term applicability of the guide, ensuring that it stays up to date with the development of Ansible.
  3. Tests. We expect tests to pass before peer review will begin.

Code Review Process

Code review takes place in GitHub pull requests. See this article if you’re not familiar with GitHub Pull Requests.

Once you open a pull request, maintainers will review your code using the built-in code review process in Github PRs. The process at this point is as follows:

  1. A maintainer will review your code and merge it if no changes are necessary. Your change will be merged into the repository’s master branch and will be noted in the project’s CHANGELOG.md at the time of release.
  2. If want your contribution to motivate your inclusion in the authorship, please add a line to that effect in the pull request
  3. If a maintainer has feedback or questions on your changes they they will set request changes in the review and provide an explanation.

Obvious Fix Policy

Small contributions, such as fixing spelling errors, where the content is small enough to not be considered intellectual property can be made against the master branch

As a rule of thumb, changes are obvious fixes if they do not introduce any new functionality or creative thinking. Assuming the change does not affect functionality, some common obvious fix examples include the following:

  • Spelling / grammar fixes
  • Typo correction, white space and formatting changes
  • Comment clean up
  • Bug fixes that change default return values or error codes stored in constants
  • Adding logging messages or debugging output
  • Changes to ‘metadata’ files like Gemfile, .gitignore, build scripts, etc.
  • Moving source files from one directory or package to another

Whenever you invoke the “obvious fix” rule, please say so in your commit message:

------------------------------------------------------------------------
commit 370adb3f82d55d912b0cf9c1d1e99b132a8ed3b5
Author: Julia Child <devguru@egi.eu>
Date:   Wed Sep 18 11:44:40 2015 -0700

  Fix typo in the README.

  Obvious fix.

------------------------------------------------------------------------

Using git

For collaboration purposes, it is best if you create a GitHub account and fork the repository to your own account. Once you do this you will be able to push your changes to your GitHub repository for others to see and use, and it will be easier to send pull requests.

Branches and Commits

You should submit your patch as a git branch named after the Github issue, such as #3. This is called a topic branch and allows users to associate a branch of code with the ticket.

It is a best practice to have your commit message have a summary line that includes the ticket number, followed by an empty line and then a brief description of the commit. This also helps other contributors understand the purpose of changes to the code.

    #3 - platform_family and style

    * use platform_family for platform checking
    * update notifies syntax to "resource_type[resource_name]" instead of
      resources() lookup
    * GH-692 - delete config files dropped off by packages in conf.d
    * dropped debian 4 support because all other platforms have the same
      values, and it is older than "old stable" debian release

Release Cycle

We follow the Semantic Versioning as far as applicable. This pattern says that software versions should take an X.Y.Z pattern where:

  • X is a major release, which may not be fully compatible with prior major releases
  • Y is a minor release, which adds both new features and bug fixes
  • Z is a patch release, which adds just bug fixes

Releases are generally performed after any bugfix / feature enhancement pull request merge. You can watch the Github repository for updates. The latest release will always point to the master branch, whilst release candidates will be done in version-specific branches, such as v0.2.0-rc.

Publishing Releases

Major releases are published in Zenodo, using the GitHub integration. A codemeta.json must accompany each release accurately describing the research object, and properly recognising author and contributor metadata.

Contribution Do’s and Don’t’s

  1. Please do include tests for your contribution.
  2. If you need help, ask on the EGI Operations community
  3. Please do indicate new platform (families) or platform versions in the commit message, and update the relevant ticket.
  4. If a contribution adds new platforms or platform versions, indicate
  5. such in the body of the commit message(s), and update the relevant issues.
  6. When writing commit messages, it is helpful for others if you indicate the issue.

Community

EGI benefits from a strong community of developers and system administrators, and vice-versa. If you have any questions or if you would like to get involved in the wider EGI community you can check out:

Also here are some additional pointers to some Ansible documentation:

This file has been modified from the Chef Cookbook Contributing Guide.