[PATCH] trace-cruncher: Enable Github development flow

[PATCH] trace-cruncher: Enable Github development flow

[Tzvetomir Stoyanov (VMware)]

Github development flow is more popular among python developers. The
CONTRIBUTING file is updated to enable both mailing list based (current
one) and PR based (github) development flows. The main motivation for
this change in to attract more contributors.

Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
---
 CONTRIBUTING.md | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index 865fcc8..c22780b 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -5,21 +5,25 @@
 The trace-cruncher project team welcomes contributions from the community. All contributions to this repository must be
 signed. Your signature certifies that you wrote the patch or have the right to pass it on as an open-source patch.
 
-The development process of trace-cruncher is strongly coupled to the development of the Linux kernel tracing libraries [libtraceevent](https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git), [libtracefs](https://git.kernel.org/pub/scm/libs/libtrace/libtracefs.git/) and [KernelShark](https://git.kernel.org/pub/scm/utils/trace-cmd/kernel-shark.git/). Because of this reason we adopt and follow the development workflow established by those projects.
+### Submitting patches
 
-For contributions to development, please send patches to: linux-trace-devel@vger.kernel.org
+Patches can be submitted by either:
+ 1. Using the regular [Github Flow](https://docs.github.com/en/get-started/quickstart/github-flow):
+    - The main branch is **tracecruncher**.
+    - Any changes must be on a feature branch or on a fork.
+    - Tests must pass before merging, and the pull request must be [reviewed](https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests) and approved.
+    - Break the complex Pull Requests into small self-contained patches.
 
-[Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
+ 2. Using the [Linux kernel](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html) development workflow:
+    - Add prefix "trace-cruncher:" to the patch subject.
+    - Break the changes into small self-contained patches and group them in a patch set.
+    - Send patches to linux-trace-devel@vger.kernel.org
+    - [Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
 
 ### Code Style
 
 The preferred coding style for the project is the [Linux kernel coding style](https://www.kernel.org/doc/html/v4.10/process/coding-style.html#linux-kernel-coding-style)
 
-### Formatting Commit Messages
-
-The project follows the conventions for [submitting patches](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html)
-as described by the Linux kernel.
-
 ### Tests
 
 Make sure that all your changes are covered by the tests. Before submitting your patch, check if everything works at 100% by running the tests in **tracecruncher/tests**. Compile your changes and install trace-cruncher (`sudo make install`), to make sure that your code is used in the tests. As trace-cruncher interacts with the Linux kernel tracing infrastructure, the tests must be run with root privileges:
@@ -30,6 +34,6 @@ sudo python3 -m unittest discover .
 ```
 
 ## Reporting Bugs and Creating Issues
-For bug reports and issues, please file it [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---)
-
-When opening a new issue, try to roughly follow the commit message format conventions above.
+Bug reports and issues can be submitted by either:
+1. Create a [Github issue](https://docs.github.com/es/issues/tracking-your-work-with-issues/creating-an-issue) in [trace-cruncher](https://github.com/vmware/trace-cruncher).
+2. Report them in [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---), **Tools and utilities** category, **Trace-cmd and kernelshark** component.
-- 
2.35.1

Re: [PATCH] trace-cruncher: Enable Github development flow

[John 'Warthog9' Hawley]

On 6/15/2022 8:15 AM, Tzvetomir Stoyanov (VMware) wrote:
> Github development flow is more popular among python developers. The
> CONTRIBUTING file is updated to enable both mailing list based (current
> one) and PR based (github) development flows. The main motivation for
> this change in to attract more contributors.
> 
> Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>

This looks fine to me, and will be interesting to see how this ends up 
working out.

- John 'Warthog9' Hawley

> ---
>   CONTRIBUTING.md | 26 +++++++++++++++-----------
>   1 file changed, 15 insertions(+), 11 deletions(-)
> 
> diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
> index 865fcc8..c22780b 100644
> --- a/CONTRIBUTING.md
> +++ b/CONTRIBUTING.md
> @@ -5,21 +5,25 @@
>   The trace-cruncher project team welcomes contributions from the community. All contributions to this repository must be
>   signed. Your signature certifies that you wrote the patch or have the right to pass it on as an open-source patch.
>   
> -The development process of trace-cruncher is strongly coupled to the development of the Linux kernel tracing libraries [libtraceevent](https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git), [libtracefs](https://git.kernel.org/pub/scm/libs/libtrace/libtracefs.git/) and [KernelShark](https://git.kernel.org/pub/scm/utils/trace-cmd/kernel-shark.git/). Because of this reason we adopt and follow the development workflow established by those projects.
> +### Submitting patches
>   
> -For contributions to development, please send patches to: linux-trace-devel@vger.kernel.org
> +Patches can be submitted by either:
> + 1. Using the regular [Github Flow](https://docs.github.com/en/get-started/quickstart/github-flow):
> +    - The main branch is **tracecruncher**.
> +    - Any changes must be on a feature branch or on a fork.
> +    - Tests must pass before merging, and the pull request must be [reviewed](https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests) and approved.
> +    - Break the complex Pull Requests into small self-contained patches.
>   
> -[Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
> + 2. Using the [Linux kernel](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html) development workflow:
> +    - Add prefix "trace-cruncher:" to the patch subject.
> +    - Break the changes into small self-contained patches and group them in a patch set.
> +    - Send patches to linux-trace-devel@vger.kernel.org
> +    - [Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
>   
>   ### Code Style
>   
>   The preferred coding style for the project is the [Linux kernel coding style](https://www.kernel.org/doc/html/v4.10/process/coding-style.html#linux-kernel-coding-style)
>   
> -### Formatting Commit Messages
> -
> -The project follows the conventions for [submitting patches](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html)
> -as described by the Linux kernel.
> -
>   ### Tests
>   
>   Make sure that all your changes are covered by the tests. Before submitting your patch, check if everything works at 100% by running the tests in **tracecruncher/tests**. Compile your changes and install trace-cruncher (`sudo make install`), to make sure that your code is used in the tests. As trace-cruncher interacts with the Linux kernel tracing infrastructure, the tests must be run with root privileges:
> @@ -30,6 +34,6 @@ sudo python3 -m unittest discover .
>   ```
>   
>   ## Reporting Bugs and Creating Issues
> -For bug reports and issues, please file it [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---)
> -
> -When opening a new issue, try to roughly follow the commit message format conventions above.
> +Bug reports and issues can be submitted by either:
> +1. Create a [Github issue](https://docs.github.com/es/issues/tracking-your-work-with-issues/creating-an-issue) in [trace-cruncher](https://github.com/vmware/trace-cruncher).
> +2. Report them in [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---), **Tools and utilities** category, **Trace-cmd and kernelshark** component.

Re: [PATCH] trace-cruncher: Enable Github development flow

[Yordan Karadzhov]

Looks good to me.

Acked-by: Yordan Karadzhov <y.karadz@gmail.com>

On 6/15/22 18:15, Tzvetomir Stoyanov (VMware) wrote:
> Github development flow is more popular among python developers. The
> CONTRIBUTING file is updated to enable both mailing list based (current
> one) and PR based (github) development flows. The main motivation for
> this change in to attract more contributors.
>
> Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@gmail.com>
> ---
>   CONTRIBUTING.md | 26 +++++++++++++++-----------
>   1 file changed, 15 insertions(+), 11 deletions(-)
>
> diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
> index 865fcc8..c22780b 100644
> --- a/CONTRIBUTING.md
> +++ b/CONTRIBUTING.md
> @@ -5,21 +5,25 @@
>   The trace-cruncher project team welcomes contributions from the community. All contributions to this repository must be
>   signed. Your signature certifies that you wrote the patch or have the right to pass it on as an open-source patch.
>   
> -The development process of trace-cruncher is strongly coupled to the development of the Linux kernel tracing libraries [libtraceevent](https://git.kernel.org/pub/scm/libs/libtrace/libtraceevent.git), [libtracefs](https://git.kernel.org/pub/scm/libs/libtrace/libtracefs.git/) and [KernelShark](https://git.kernel.org/pub/scm/utils/trace-cmd/kernel-shark.git/). Because of this reason we adopt and follow the development workflow established by those projects.
> +### Submitting patches
>   
> -For contributions to development, please send patches to: linux-trace-devel@vger.kernel.org
> +Patches can be submitted by either:
> + 1. Using the regular [Github Flow](https://docs.github.com/en/get-started/quickstart/github-flow):
> +    - The main branch is **tracecruncher**.
> +    - Any changes must be on a feature branch or on a fork.
> +    - Tests must pass before merging, and the pull request must be [reviewed](https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests) and approved.
> +    - Break the complex Pull Requests into small self-contained patches.
>   
> -[Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
> + 2. Using the [Linux kernel](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html) development workflow:
> +    - Add prefix "trace-cruncher:" to the patch subject.
> +    - Break the changes into small self-contained patches and group them in a patch set.
> +    - Send patches to linux-trace-devel@vger.kernel.org
> +    - [Subscribe](http://vger.kernel.org/vger-lists.html#linux-trace-devel) / [Archives](https://lore.kernel.org/linux-trace-devel/)
>   
>   ### Code Style
>   
>   The preferred coding style for the project is the [Linux kernel coding style](https://www.kernel.org/doc/html/v4.10/process/coding-style.html#linux-kernel-coding-style)
>   
> -### Formatting Commit Messages
> -
> -The project follows the conventions for [submitting patches](https://www.kernel.org/doc/html/v5.4/process/submitting-patches.html)
> -as described by the Linux kernel.
> -
>   ### Tests
>   
>   Make sure that all your changes are covered by the tests. Before submitting your patch, check if everything works at 100% by running the tests in **tracecruncher/tests**. Compile your changes and install trace-cruncher (`sudo make install`), to make sure that your code is used in the tests. As trace-cruncher interacts with the Linux kernel tracing infrastructure, the tests must be run with root privileges:
> @@ -30,6 +34,6 @@ sudo python3 -m unittest discover .
>   ```
>   
>   ## Reporting Bugs and Creating Issues
> -For bug reports and issues, please file it [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---)
> -
> -When opening a new issue, try to roughly follow the commit message format conventions above.
> +Bug reports and issues can be submitted by either:
> +1. Create a [Github issue](https://docs.github.com/es/issues/tracking-your-work-with-issues/creating-an-issue) in [trace-cruncher](https://github.com/vmware/trace-cruncher).
> +2. Report them in [bugzilla](https://bugzilla.kernel.org/buglist.cgi?component=Trace-cmd%2FKernelshark&product=Tools&resolution=---), **Tools and utilities** category, **Trace-cmd and kernelshark** component.