-
Notifications
You must be signed in to change notification settings - Fork 478
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
646a7cf
commit 3605816
Showing
2 changed files
with
224 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,221 @@ | ||
# Contributing Guide | ||
|
||
## Welcome to Percona Server for MySQL! | ||
|
||
We're excited to have you join the Percona community and participate in keeping open-source open. | ||
|
||
You can contribute in one of the following ways: | ||
|
||
1. Submit a bug report or a feature | ||
request in [ our public Jira ](https://jira.percona.com/projects/PS/issues). | ||
2. Submit a pull request (PR) with the code patch. | ||
3. Contribute to [the documentation](https://github.com/percona/psmysql-docs/blob/innovation-release/contributing.md) | ||
4. Reach us on our [ Forums ](https://forums.percona.com/c/mysql-mariadb) | ||
|
||
This document describes the workflow for submitting pull requests. | ||
By contributing, you agree to the [Percona Community code of conduct](https://percona.community/contribute/coc/). Thank | ||
you for helping us enhance Percona Server for MySQL. | ||
|
||
## Why Your Contribution Matters? | ||
|
||
Main driving force of open-source project is community. We, as developers team, highly appreciate any contributions. | ||
|
||
It may be advisable to initially submit bug fixes to the upstream project that will eventually become part of the next | ||
release of Percona. Alternatively, Percona provides expedited response times, transparent development processes, and | ||
streamlined contribution acceptance within a public repository. | ||
|
||
## Creating a Jira Issue | ||
|
||
The creation of a Jira issue is _mandatory_. A comprehensive description of the bug or feature is essential. Thorough | ||
testing of the solution, inclusion of pertinent information in the release notes, and other relevant details are highly | ||
recommended. Additionally, linking the pull request to the corresponding issue with a detailed description will | ||
facilitate the development process. | ||
|
||
To create Jira issue, please open the [ LINK ](https://perconadev.atlassian.net/jira/software/c/projects/PS/issues) and | ||
authenticate with your favorite social network account (Google, Facebook, etc.). Then follow the instructions and verify | ||
your e-mail. If Jira starts to redirect you back and forth (their bug, not ours), just open a new browser window with | ||
the link above. | ||
|
||
Now you're ready to an create issue. Find the big "Create" button in the top menu and press it. Choose issue type from | ||
the second drop-down, saying it is a Bug, an Improvement, or a New feature. Fill in all the required fields below. Don't | ||
spare the letters for the description field. Accurate and detailed description is a huge first step to the solution. The | ||
rest of the fields are mostly for internal use; the core team developers will fill it later. | ||
|
||
## Working with the Source Code | ||
|
||
First, you should have a GitHub account. You can not work with the Percona repository directly, you have | ||
to [ "Fork" ](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/fork-a-repo) | ||
our repository to your GitHub account. To do so, | ||
open [ Percona PS repository](https://github.com/percona/percona-server) in a browser. Then, right to the repository | ||
name you'll find the "Fork" button. Press it. | ||
Then return to your GitHub home page, find your "percona-server" repository and clone it locally. | ||
Note | ||
that [ creating SSH keys and putting it to the GitHub ](https://docs.github.com/en/authentication/connecting-to-github-with-ssh/adding-a-new-ssh-key-to-your-github-account) | ||
settings will save a lot of time for you. | ||
|
||
In the cloned local repository checkout the "trunk" branch you are going to work on. For 8.0.X releases is 8.0. For | ||
8.4.X series it is 8.4, and for 9.0.X it is "trunk". | ||
|
||
Now you are ready to create a branch for your contribution. Please name the branch according to the following pattern: | ||
|
||
``` | ||
PS-9876-8.0-bug_in_some_module | ||
^^^^^^^ | ||
Jira issue number | ||
^^^ | ||
Base version | ||
^^^^^^^^^^^^^^^^^^^ | ||
Very short definition of the issue | ||
``` | ||
|
||
Before compilation of the code, you need to fetch third-party code which is organized as Git submodules. Go to the | ||
project directory and type the following: | ||
|
||
``` | ||
git submodule update --init | ||
``` | ||
|
||
At this point Git will fetch required third-party modules into the source tree. | ||
|
||
Now we can try to build the project. Below is a quick and dirty shell script for that (works with GCC 14). It gives you | ||
hints how to do outside-the-tree build and set CMake flags. | ||
|
||
``` | ||
#!/bin/bash | ||
CMAKE_FLAGS="-DCMAKE_BUILD_TYPE=Debug \ | ||
-DDOWNLOAD_BOOST=1 \ | ||
-DWITH_BOOST=/home/${USER}/projects/boost\ | ||
-DWITH_DEBUG=ON\ | ||
-DWITH_ROCKSDB=ON\ | ||
-DWITH_COREDUMPER=OFF\ | ||
-DWITH_CURL=system\ | ||
-DWITH_FIDO=system\ | ||
-DWITH_ZLIB=bundled\ | ||
-DCMAKE_CXX_FLAGS=-Wno-error=template-id-cdtor -Wno-error=dangling-reference \ | ||
-Wno-error=deprecated-declarations | ||
SRC_DIR=$1 | ||
if [ -z ${SRC_DIR} ] ; then | ||
echo "Please provide source directory as 1st parameter!" | ||
exit 1 | ||
fi | ||
if [ -d $SRC_DIR ] ; then | ||
echo "Source dir is $SRC_DIR" | ||
else | ||
echo "Source dir $SRC_DIR does not exist" | ||
exit 1 | ||
fi | ||
ABS_SRC_DIR=`realpath $SRC_DIR` | ||
if [ -z $2 ] ; then | ||
BUILD_DIR="build-${SRC_DIR}" | ||
else | ||
BUILD_DIR=$2 | ||
fi | ||
echo "Build dir is $BUILD_DIR" | ||
if [ -d $BUILD_DIR ] ; then | ||
echo "Build dir exists. Cmake will re-use cached data." | ||
else | ||
mkdir $BUILD_DIR | ||
fi | ||
cd $BUILD_DIR | ||
cmake $CMAKE_FLAGS $ABS_SRC_DIR | ||
make -j 16 | ||
``` | ||
|
||
Don't expect it will work from the first run as a charm. MySQL has a lot of dependencies on different libraries. So, you | ||
may use "try and fail" approach installing the development packages, or use some hints | ||
from [ documentation ](https://docs.percona.com/percona-server/8.0/compile-percona-server.html) . | ||
|
||
When the source finally successfully compiles, you're ready to run tests. You have to run at least the start and | ||
shutdown test of mysqld. | ||
|
||
``` | ||
cd build-percona-server/mysql-test | ||
./mtr main.1st | ||
``` | ||
|
||
When the tests are OK, you're ready to set up your favorite IDE for the project development. The build script above will | ||
give you a clue what CMake parameters to add to the project in IDE. | ||
|
||
To be able to run the mysqld binary in your IDE, first run the following command in the "mysql-test" directory of the | ||
build tree: | ||
|
||
``` | ||
./mtr --manual-debug main.1st | ||
``` | ||
|
||
Then find the string stating with "args:". This is the command line arguments you should pass to the execution of " | ||
mysqld" in your IDE. This step allows to run/debug "mysqld" from the IDE. | ||
|
||
From this moment you're ready to code and debug! | ||
|
||
### Testing | ||
|
||
Prior to submitting your code, ensure it is thoroughly tested, both manually and with automated test suites. The | ||
maintenance of high-quality code is paramount to ensure the longevity and functionality of our project. | ||
|
||
Testing of MySQL is a whole big thing, so please go to | ||
the [ official documentation on MySQL testing ](https://dev.mysql.com/doc/dev/mysql-server/latest/PAGE_TESTING_TOOLS.html) | ||
first. | ||
|
||
If it is bug fix, make sure that corresponding tests from the test suite work fine. It's a good idea to patch existing | ||
tests to cover your specific case. For the introduction of new features, the development of a comprehensive test suite | ||
is strongly recommended. | ||
|
||
## Preparing PR | ||
|
||
After finishing the tasks outlined above, you must prepare your branch for a pull request. | ||
Please do not forget to run ``` git clang-format ``` on your code. | ||
|
||
Then, squash all commits to one or a couple of logically consistent commits. | ||
For example, the fix of bug should have just one commit. If you have a new feature that requires some changes to the | ||
existing code, then the first commit should contain required changes, and the second is a new feature and tests. It is | ||
not a dogma, on the other hand, we value a clean and consistent Git history. Please help us maintain this standard. | ||
|
||
Now you can push your local branch to your GitHub repository and make the pull request to the Percona repository. | ||
|
||
Please, write meaningful title for the PR and put the number of Jira issue at the beginning of the first line in the | ||
description. The second line is usually a link to the Jira issue. Then the short description follows. No need to write a | ||
long description because you wrote one in the Jira issue. | ||
|
||
Peer review of the code always goes before PR merge. So, you have to add some core development team members in the " | ||
Reviewers" field of PR. Best guess is to assign the PR to Yura Sorokin, our team lead. | ||
|
||
Our GitHub pipeline runs a lot of checks on each PR. Please make sure you fixed all comments provided by checks. Well, | ||
sometime not all of them :) , you understand... | ||
|
||
Following this step, the development team will conduct a thorough review of your code at their earliest convenience. Our | ||
primary objective is to ensure the highest quality standards. We encourage you to interpret our comments as | ||
opportunities for improvement and constructive feedback. | ||
|
||
When the review is finished and all the issues are addressed, your pull request will be merged into the main source | ||
tree. | ||
|
||
Congratulations! You are now a recognized contributor. | ||
|
||
## Your Code and MySQL Versions | ||
|
||
Percona officially supports 3 version lines of MySQL: | ||
|
||
1. 8.0.X - stable line. | ||
2. 8.4.X - next stable line, or LTS line. | ||
3. 9.0.X - innovation line, a test bed for everything new at the moment. | ||
|
||
Next LTS version will be 9.7, so until then 9.X are "innovation" versions. | ||
|
||
It is not easy to keep track of all the changes and understand how the new code will work with different versions. | ||
Therefore, your best bet is to work with the version line you're currently using and familiar with. Nevertheless, | ||
Percona developers team will take care of your code, and port or backport it to other version lines if it is applicable. | ||
|
||
## When will My Code be in the release? | ||
|
||
Percona releases follow upstream MySQL releases. We merge upstream code changes and our code from the "trunk" as soon as | ||
the next official upstream release becomes available. Then we build packages and test everything thoroughly. It takes | ||
time. | ||
|
||
We use a "train" approach. It means, that if some code is already in one of the "trunk" branches before the official | ||
upstream release, it will be released in Percona's distribution of new release. So, if we are already in the process of | ||
working on some fresh upstream release, PR will be merged to the "trunk" but not to the release branch. And then your | ||
code will be released after the current one. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters