Skip to content
This repository has been archived by the owner on Mar 22, 2023. It is now read-only.

Latest commit

 

History

History
170 lines (123 loc) · 6.42 KB

CONTRIBUTING.md

File metadata and controls

170 lines (123 loc) · 6.42 KB
Raw

Contributing to pmemstream

Opening new issues

Please log bugs or suggestions as GitHub issues. Details such as OS and PMDK version are always appreciated.

Code style

  • See .clang-format file in the repository for details
  • Indent with tabs (width: 8)
  • Max 120 chars per line
  • Space before '*' and '&' (rather than after)

If you want to check and format your source code properly you can use CMake's DEVELOPER_MODE and CHECK_CPP_STYLE options. When enabled additional checks are switched on (cppstyle, whitespaces and headers).

cmake .. -DDEVELOPER_MODE=ON -DCHECK_CPP_STYLE=ON

If you just want to format your code you can make adequate target:

make cppformat

NOTE: We're using specific clang-format - version exactly 11.1 is required.

Submitting Pull Requests

We take outside code contributions to PMEMSTREAM through GitHub pull requests.

If you add a new feature or fix a critical bug please append appropriate entry to ChangeLog under newest release.

NOTE: If you do decide to implement code changes and contribute them, please make sure you agree your contribution can be made available under the BSD-style License used for PMEMSTREAM.

NOTE: Submitting your changes also means that you certify the following:

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

In case of any doubt, the gatekeeper may ask you to certify the above in writing, i.e. via email or by including a Signed-off-by: line at the bottom of your commit comments.

To improve tracking of who is the author of the contribution, we kindly ask you to use your real name (not an alias) when committing your changes to PMEMSTREAM:

Author: Random J Developer <[email protected]>

Adding new dependency

Adding each new dependency (including new docker image and package) should be done in a separate commit. The commit message should be:

New dependency: dependency_name

license: SPDX license tag
origin: https://dependency_origin.com

Extending Public API

When adding a new public function, you have to make sure to update:

  • documentation,
  • manpage links,
  • map file with debug symbols,
  • appropriate examples (to show usage),
  • tests,
  • ChangeLog with new entry for next release.

Configuring GitHub fork

To build and submit documentation as an automatically generated pull request, the repository has to be properly configured.

  • Personal access token for GitHub account has to be generated.

    • Such personal access token has to be set in the repository's secrets as DOC_UPDATE_GITHUB_TOKEN variable.

  • DOC_UPDATE_BOT_NAME secret variable has to be set. In most cases it will be the same as GitHub account name.

  • DOC_REPO_OWNER secret variable has to be set. Name of the GitHub account, which will be target to make an automatic pull request with documentation. In most cases it will be the same as GitHub account name.

To enable automatic images pushing to GitHub Container Registry, following variables:

  • CONTAINER_REG existing environment variable (defined in workflow files, in .github/ directory) has to be updated to contain proper GitHub Container Registry address (to forking user's container registry),

  • GH_CR_USER secret variable has to be set up - an account (with proper permissions) to publish images to the Container Registry (tab Packages in your GH profile/organization).

  • GH_CR_PAT secret variable also has to be set up - Personal Access Token (with only read & write packages permissions), to be generated as described here for selected account (user defined in above variable).

Debugging

GDB

Test framework supports debugging under gdb.

  • For remote debugging set GDBSERVER environment variable. Its content will be passed to gdbserver application as first positional argument, so it should define how gdbserver will communicate with gdb. Example usage with a single test:
GDBSERVER=localhost:4444 ctest -R vector_comp_operators_0_none --output-on-failure
  • For local debugging in graphical environment using cgdb, set CGDB environment variable to gnome-terminal or konsole accordingly to your setup.

Tests helpers

When something unexpected appears in the pmemstream, e.g., entries' count does not match expectation or a trash spans are encountered - it is possible to easily print the content of stream's file. There's available a tests helper class - stream_span_helpers.hpp which provides a quick way to gather information of all spans (entries, regions, empty spaces) recognized within the stream. It's simply possible to print this content by running C++ code like this (from a gdb or example code):

auto spans = span_runtimes_from_stream(stream);
std::cout << spans << std::endl;