Glance follows the same Review guidelines outlined by the OpenStack community. This page provides additional information that is helpful for reviewers of patches to Glance.
Glance uses the Gerrit tool to review proposed code changes. The review site is https://review.opendev.org
Gerrit is a complete replacement for Github pull requests. All Github pull requests to the Cinder repository will be ignored.
See Quick Reference for information on quick reference for developers. See Getting Started for information on how to get started using Gerrit. See Development Workflow for more detailed information on how to work with Gerrit.
With the demise of Python 2.7 in January 2020, beginning with the Ussuri development cycle, Glance only needs to support Python 3 runtimes (in particular, 3.6 and 3.7). Thus we can begin to incorporate Python 3 language features and remove Python 2 compatibility code. At the same time, however, we are still supporting stable branches that must support Python 2. Our biggest interaction with the stable branches is backporting bugfixes, where in the ideal case, we’re just doing a simple cherry-pick of a commit from master to the stable branches. You can see that there’s some tension here.
With that in mind, here are some guidelines for reviewers and developers that the Glance community has agreed on during this phase where we want to write pure Python 3 but still must support Python 2 code.
Glance requires unit tests with all patches that introduce a new branch or function in the code. Changes that do not come with a unit test change should be considered closely and usually returned to the submitter with a request for the addition of unit test.
Except where otherwise noted, this document is licensed under Creative Commons Attribution 3.0 License. See all OpenStack Legal Documents.