Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge tag 'pull-misc-for-7.2-151122-2' of https://gitlab.com/stsquad/…
…qemu into staging Testing and doc updates: - Only probe if docker or podman binaries in path - tweak avacado console to better find login prompts - reduce console noise for aspeed avocado tests - update documents on maintainer roles and process - raise timeout for ppc64 avocado tests - integrate coverage reports into gitlab # -----BEGIN PGP SIGNATURE----- # # iQEzBAABCgAdFiEEZoWumedRZ7yvyN81+9DbCVqeKkQFAmNzhJYACgkQ+9DbCVqe # KkRbGAf+M4SNBda/ORXV7qolIv1CkgqxK4gk/EbAf4bSSHJpR9UTtCibKmyLpM67 # Hdk/p1Z6kD2k13l1MVLI0F0wjDzdusjvfHOhrcKSUlOmrKwF+bvLUu8qHiEPKP62 # scMSDP/xY2a4ZjV5imLzoRekfcL+c7Xl6NR6zt7Ib0lBH7lbJrD6wYqrCgnJKus6 # SXrcvLbeH+wIWWvva1Q4nPvvyG76te6+aPCEu0b2D4x/uQ8Tv+JHTuyDs3PsFKlf # 3zdxIxoKTubX4ZwyIX5fYya46qfOsjJvXQcYfdp+Om/U+20ajhiAkD7SzBtEuMSX # UJXlmPAj/b9HmHhhn2WQO0bRHSD8ww== # =8sPk # -----END PGP SIGNATURE----- # gpg: Signature made Tue 15 Nov 2022 07:22:46 EST # gpg: using RSA key 6685AE99E75167BCAFC8DF35FBD0DB095A9E2A44 # gpg: Good signature from "Alex Bennée (Master Work Key) <alex.bennee@linaro.org>" [unknown] # gpg: WARNING: This key is not certified with a trusted signature! # gpg: There is no indication that the signature belongs to the owner. # Primary key fingerprint: 6685 AE99 E751 67BC AFC8 DF35 FBD0 DB09 5A9E 2A44 * tag 'pull-misc-for-7.2-151122-2' of https://gitlab.com/stsquad/qemu: gitlab: integrate coverage report tests/avocado: Raise timeout for boot_linux.py:BootLinuxPPC64.test_pseries_tcg docs/devel: try and improve the language around patch review docs/devel: simplify the minimal checklist docs/devel: make language a little less code centric docs/devel: add a maintainers section to development process tests/docker: allow user to override check target tests/avocado/machine_aspeed.py: Reduce noise on the console for SDK tests tests/avocado: improve behaviour waiting for login prompts Run docker probe only if docker or podman are available Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
- Loading branch information
Showing
12 changed files
with
297 additions
and
56 deletions.
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
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
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 |
---|---|---|
@@ -1,3 +1,5 @@ | ||
.. _code_of_conduct: | ||
|
||
Code of Conduct | ||
=============== | ||
|
||
|
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
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,106 @@ | ||
.. _maintainers: | ||
|
||
The Role of Maintainers | ||
======================= | ||
|
||
Maintainers are a critical part of the project's contributor ecosystem. | ||
They come from a wide range of backgrounds from unpaid hobbyists | ||
working in their spare time to employees who work on the project as | ||
part of their job. Maintainer activities include: | ||
|
||
- reviewing patches and suggesting changes | ||
- collecting patches and preparing pull requests | ||
- tending to the long term health of their area | ||
- participating in other project activities | ||
|
||
They are also human and subject to the same pressures as everyone else | ||
including overload and burnout. Like everyone else they are subject | ||
to project's :ref:`code_of_conduct` and should also be exemplars of | ||
excellent community collaborators. | ||
|
||
The MAINTAINERS file | ||
-------------------- | ||
|
||
The `MAINTAINERS | ||
<https://gitlab.com/qemu-project/qemu/-/blob/master/MAINTAINERS>`__ | ||
file contains the canonical list of who is a maintainer. The file | ||
is machine readable so an appropriately configured git (see | ||
:ref:`cc_the_relevant_maintainer`) can automatically Cc them on | ||
patches that touch their area of code. | ||
|
||
The file also describes the status of the area of code to give an idea | ||
of how actively that section is maintained. | ||
|
||
.. list-table:: Meaning of support status in MAINTAINERS | ||
:widths: 25 75 | ||
:header-rows: 1 | ||
|
||
* - Status | ||
- Meaning | ||
* - Supported | ||
- Someone is actually paid to look after this. | ||
* - Maintained | ||
- Someone actually looks after it. | ||
* - Odd Fixes | ||
- It has a maintainer but they don't have time to do | ||
much other than throw the odd patch in. | ||
* - Orphan | ||
- No current maintainer. | ||
* - Obsolete | ||
- Old obsolete code, should use something else. | ||
|
||
Please bear in mind that even if someone is paid to support something | ||
it does not mean they are paid to support you. This is open source and | ||
the code comes with no warranty and the project makes no guarantees | ||
about dealing with bugs or features requests. | ||
|
||
|
||
|
||
Becoming a reviewer | ||
------------------- | ||
|
||
Most maintainers start by becoming subsystem reviewers. While anyone | ||
is welcome to review code on the mailing list getting added to the | ||
MAINTAINERS file with a line like:: | ||
|
||
R: Random Hacker <rhacker@example.com> | ||
|
||
will ensure that patches touching a given subsystem will automatically | ||
be CC'd to you. | ||
|
||
Becoming a maintainer | ||
--------------------- | ||
|
||
Maintainers are volunteers who put themselves forward or have been | ||
asked by others to keep an eye on an area of code. They have generally | ||
demonstrated to the community, usually via contributions and code | ||
reviews, that they have a good understanding of the subsystem. They | ||
are also trusted to make a positive contribution to the project and | ||
work well with the other contributors. | ||
|
||
The process is simple - simply send a patch to the list that updates | ||
the ``MAINTAINERS`` file. Sometimes this is done as part of a larger | ||
series when a new sub-system is being added to the code base. This can | ||
also be done by a retiring maintainer who nominates their replacement | ||
after discussion with other contributors. | ||
|
||
Once the patch is reviewed and merged the only other step is to make | ||
sure your GPG key is signed. | ||
|
||
.. _maintainer_keys: | ||
|
||
Maintainer GPG Keys | ||
~~~~~~~~~~~~~~~~~~~ | ||
|
||
GPG is used to sign pull requests so they can be identified as really | ||
coming from the maintainer. If your key is not already signed by | ||
members of the QEMU community, you should make arrangements to attend | ||
a `KeySigningParty <https://wiki.qemu.org/KeySigningParty>`__ (for | ||
example at KVM Forum) or make alternative arrangements to have your | ||
key signed by an attendee. Key signing requires meeting another | ||
community member **in person** [#]_ so please make appropriate | ||
arrangements. | ||
|
||
.. [#] In recent pandemic times we have had to exercise some | ||
flexibility here. Maintainers still need to sign their pull | ||
requests though. |
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
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
Oops, something went wrong.