Skip to content

Latest commit

 

History

History
226 lines (159 loc) · 9.52 KB

CONTRIBUTING.md

File metadata and controls

226 lines (159 loc) · 9.52 KB
Raw

Contributing to Elastic Common Schema (ECS)

All information related to ECS is versioned in the elastic/ecs repository. All changes to ECS happen through Pull Requests submitted through Git.

ECS is an open source project and we love to receive contributions from our community - you!

Table of Contents

How to Contribute

There are two primary ways in which you can contribute to ECS.

  1. The RFC process is used for significant additions or breaking changes to the schema itself.
  2. For bug fixes or incremental, non-controversial additions to ECS, changes can be made directly to the ECS project and submitted as pull request.

Dev Tools

You need these tools to contribute to the ECS repo:

Submitting Changes

  • Sign the Contributor License Agreement.
  • Set up your git environment.
    • Create your own fork of the ECS repo.
    • Clone your fork to your machine.
  • Create a local branch to hold your changes.
    • Run git checkout -b branch-name, where branch-name is the name you want to give your local branch
  • Do your work.
    • Schema changes will be done in the .yml files under the schemas directory. Review the Schema Files section below.
    • Generator scripts and other tooling reside in the scripts directory.
    • Generated artifacts fall under the generated directory.
    • Documentation files are located in the docs directory.
  • Run make to update generated files.
  • If necessary, make sure tests pass.
    • Run make test
    • Add tests for your changes, if necessary
  • Commit your changes locally.
    • Run git commit -a -m "your message"
  • Push your changes to your own github.com fork.
    • Run git push --set-upstream origin branch-name
    • In this command, origin is an alias that references your fork.
  • Request feedback about your changes.
    • Create a Pull Request against the ECS repo.
      • (Look for the Compare & pull request button on your branch in github.com.)
    • Add an entry to CHANGELOG.next.md.
    • Wait for reviews on your PR.
    • Incorporate review comments and push updates if needed.
  • Thank you for your contribution!

Important: Be sure to push changes only to your own fork. Changes must be approved before they are merged into the main repository.

Git and Github Guidelines

Forking

We follow the Github forking model for collaboration in the ECS repo. Typically contributors will add a remote repository called upstream to point to the Elastic ECS repo to add latest changes from the repo to their fork.

Commits and Merging

  • When submitting a PR for review, please perform and interactive rebase to clean up the commit history. A logical commit history is easier for reviewers to follow.
  • Use meaningful and helpful commit messages on your changes and an explanation of why you made those changes.
  • When merging, maintainers will squash your commits into a single commit.

Pull Requests

Please follow these guidelines when submitting PRs:

  • Include an explanation of your changes in the PR description.
  • Links to relevant issues, external resources, or related PRs are helpful and useful.
  • Update any tests or add new tests where appropriate.

Issues

Please follow these guidelines when submitting Issues:

  • Go to the ECS repo: https://github.com/elastic/ecs
  • Click Issues in the nav bar under the repo name.
  • Click New issue. Provide as many details as possible to help reviewers and other contributors understand your proposal.
  • Add your text, and click Submit new issue.

Branching

ECS follows this branching strategy:

  • The master is the next major version. It is where all new contributions are first merged. This includes new features and bug fixes, and it may also include breaking changes.
  • The <major>.x is the next minor version and gets backports of most non-breaking features and fixes.
  • The <major>.<minor> is the next release of a minor version, including patch releases.

Changelog

ECS maintains two changelog files:

Breaking changes intended for the next major version should be included underneath the Breaking changes sections in CHANGELOG.next.md.

Backports

ECS maintains multiple release branches in the repo. The master branch is where all new contributions should be submitted, and features and bug fixes will be backported into other branches when appropriate. Any backporting needs will be handled by the ECS team.

Tooling

Refer to the backport tool's repo for requirements and install guide. A project config is maintained in the root of this repo.

Run:

$ npx backport --pr 1234

Select target branch(es) to backport to:

? Select commit #1234 (cb79e8f5)
? Select branch (Press <space> to select, <a> to toggle all, <i> to invert selection)
❯◉ 1.x
 ◯ 1.6
 ◯ 1.5
 ◯ 1.4
 ◯ 1.3
 ◯ 1.2
 ◯ 1.1
 ◯ 1.0

New PR(s) will be opened against the targeted branch(es).

Documentation

ECS documentation is written in asciidoc format in the docs/ directory.

To build the docs and open them in your browser:

make docs

Generated Documentation Files

The following files are generated based on the current schema using Jinja templates:

File Template
fields.asciidoc fields_template.j2
fields-values.asciidoc field_values_template.j2
field-details.asciidoc field_details.j2

Running make will update these files using the scripts/generators/asciidoc_fields.py generator. These doc files should not be modified directly. Any changes as a result of a schema update and subsequent run of make should be committed.

Jinja Templates

Jinja templates allow for formatting or styling changes to templates without needing to modify the generator script directly. Some details about the Jinja template engine and our implementation are covered below as a primer; the full syntax and semantics of Jinja is covered in the project's documentation.

Delimiters

  • Statements: {% ... %}
  • Expressions: {{ ... }}
  • Comments: {{# ... #}}

Whitespace Control

Whitespace can be stripped by adding the minus sign (-) to the start or end of a block. Adding - to the start or end of a block will remove the whitespace before or after that block.

{% for i in numbers -%}
    {{i}}
{%- endfor %}

All elements would be rendered without any separating whitespace. If numbers is list of numbers from 0 to 9, the output would be 0123456789.

Variables

Templates variables are passed to the template by the application. Typically these will either be used in an expression or within a control structure statement (e.g. a for loop). In the below example, users is passed into the template and is iterated over with a for loop.

<ul>
{% for user in users %}
<li>{{ user }}</li>
{% endfor %}
</ul>

Implementation

The @templated('template_file_name') decorator is used to inject the additional functionality that renders and returns the template's content to the generator. Decorated functions should return a dict used to generate the template. When the decorated function returns, the dictionary is passed to the template renderer.

@templated('fields_template.j2')
def page_field_index(intermediate_nested, ecs_version):
    fieldsets = ecs_helpers.dict_sorted_by_keys(intermediate_nested, ['group', 'name'])
    return dict(ecs_version=ecs_version, fieldsets=fieldsets)

Schema Files

The schemas directory contains the files which define the Elastic Common Schema data model. The file structure is documented in schemas/README.md. Field additions and modifications will be made to the schemas/*.yml files.

Users consuming ECS to generate something for other use cases should use the generated/ecs/*.yml files. More detail can be found here.

Additional Resources