rsyslog/CONTRIBUTING.md
Rainer Gerhards 97c7c68f5c
docs: streamline contrib templates and commit guidance
This modernizes contributor experience by simplifying PR templates and
removing the lengthy GDPR disclaimer that often discouraged or confused
new contributors. The update encourages more participation and shows how
responsible AI can be used to improve open source workflows.

Impact: none on runtime behavior; contributor workflow improved.

Before: PR template included long GDPR block; commit message rules were
scattered and partly implicit.
After: PR template is concise, GDPR text removed, and commit assistant
usage is documented across README, CONTRIBUTING, and AGENTS.md.

Technical changes include:
- PR template: drop GDPR notice, add commit-assistant references.
- CONTRIBUTING.md: add explicit commit rules and workflow guidance.
- AGENTS.md: require canonical base prompt and commit-first workflow.
- README.md: point to assistant and updated guidance.
- base_prompt.txt: enforce "Findings:" colon format.
- Minor formatting corrections in comments.
2025-08-18 13:59:53 +02:00

5.1 KiB
Raw Blame History

Rsyslog Contribution Guide

Rsyslog is a real open source project, welcoming contributions of all kinds—from code to documentation to community engagement. Whether you're an experienced developer or a first-time contributor, there are many ways to get involved.


Ways to Contribute

  • Be an ambassador: Spread the word about rsyslog and how to best use it.
  • Offer support:
  • Improve documentation:
    • Contributions to documentation should be made to the doc/ directory of this repository.
    • Help maintain rsyslog.com/doc
  • Maintain infrastructure: Help with project websites, CI, or packaging.
  • Develop code: Implement new features, improve existing ones, or fix bugs.
  • AI-assisted contributions should follow AGENTS.md for setup and workflow guidance.

Pull Requests

Drafts and Experiments

  • You may submit early PRs for CI testing. Mark them clearly as WiP:.
  • For experiments, use draft pull requests. Close them after testing.

Target Branch

  • All PRs must target the main branch.

AI-Based Code Review (Experimental)

We are currently testing AI-based code review for pull requests. At this time, we use Google Gemini to automatically analyze code and provide comments on new PRs.

  • These reviews are informational only.
  • Every contribution is still manually reviewed by human experts.
  • The goal is to evaluate how AI can support contributor feedback and code quality assurance.

Please report any issues, false positives, or suggestions about the AI review process.


Commit Guidelines for AI Agents

If you use an AI agent (e.g. GitHub Copilot, ChatGPT, Codex), include a commit footer tag:

AI-Agent: Codex 2025-06

This helps us track and evaluate contributions and agent capabilities.


Commit Messages & Commit Assistant

To speed up reviews and keep history consistent, please follow these rules for commit messages. You can use our assistant to generate compliant messages.

Rules (must):

  • ASCII only.
  • Title ≤ 62 characters; body lines ≤ 72 characters.
  • Title format: <component>: <concise action> Examples: omhttp: migrate to OMODTX, docs: clarify imfile wildcards.
  • Lead with a brief non-technical “why” (modernization, maintainability, performance, security, Docker/CI readiness, user value).
  • Include Impact: one line when tests or user-visible behavior change.
  • Add a one-line Before/After behavior summary.
  • Provide a technical overview (conceptual, not line-by-line) in 412 lines.
  • Footer with full-URL reference(s):
    • Fixes: https://github.com/rsyslog/rsyslog/issues/<id> (only if conclusively fixed)
    • otherwise Refs: https://github.com/rsyslog/rsyslog/issues/<id>

Do not include secrets/tokens or paste sensitive data from diffs.


Patch Requirements

All submissions must:

  • Use RainerScript (no legacy $... config statements).
  • Compile cleanly with no warnings on both gcc and clang.
  • Pass clang static analysis with no findings.
  • Pass all CI test jobs (see below).
  • Include:
    • Tests (for new features or bugfixes)
    • Documentation updates (for new features)
    • Squashed commits (why it matters)

Testbench Tips

  • Write small, focused tests.
  • Use similar existing tests as templates.
  • Test driver is make check, backed by tests/diag.sh.
  • Test concurrency is limited due to resource load.

Compiler Warnings

False positives must be resolved, not ignored. Only in extreme cases use:

#pragma diagnostic push
#pragma GCC diagnostic ignored "..."
/* ... function ... */
#pragma diagnostic pop

Apply to single functions only.

See: Why static analysis matters


Continuous Integration (CI)

All PRs are tested via:

  • GitHub Actions workflows (including multi-platform docker-based tests)
  • Additional off-GitHub runners (Solaris, exotic distros)

Build durations: 1040 min per matrix job. Some known flaky tests exist; we re-run these manually if needed.


GDPR and Privacy

By contributing, your name, email, commit hash, and timestamp become part of the git history. This is fundamental to git and public open source contribution. If you have privacy concerns, you may commit anonymously:

git commit --author "anonymous <gdpr@example.com>"

Avoid --gpg-sign in that case.

If you use your real identity, note:

  • We cannot delete commits retroactively without damaging project history.
  • Identity data is used only to maintain copyright tracking and audit trails.
  • Public commits may be copied by third parties and redistributed without control.