| #summary Executive summary of the list of major Gerrit topics under review and temporarily blocked |
| |
| =Overview= |
| |
| Following a [https://groups.google.com/forum/#!topic/repo-discuss/qKz7AtZDlC4 discussion on the mailing list] we decided to create a summary page containing the list of outstanding topics that are currently under review and deserve a particular attention because of their nature. |
| |
| The purpose of this page is to keep track of them and prevent the risk of them being forgotten in the Gerrit changes backlog. The topics listed here are either related to the Gerrit architecture or to some fixes to severe bugs that need particular attention and time for being reviewed and merged. |
| |
| Currently outstanding topics are: |
| # [OutstandingTopics#Pluggable_authentication_backend auth-backends-HttpAuthProtocol] |
| # [OutstandingTopics#On-Line_edit online-3] |
| # [OutstandingTopics#Secure_Store secure-store] |
| # [OutstandingTopics#Angular_Gerrit angular-gerrit-integration] |
| |
| ----- |
| |
| =Pluggable authentication backend= |
| |
| Replace the current Gerrit authentication infrastructure, mainly based on mega-switch/case with the list of protocols/methods supported, with a new plugin-based authentication back-end. |
| |
| These changes would allow Gerrit to be more extensible, avoiding further growth of the mega-switch/case all over the code and support the ability to load user plugins to support other authentication systems in the same way that Gerrit groups have been refactored years ago. |
| |
| * Gerrit changes: [https://gerrit-review.googlesource.com/#/q/status:open+project:gerrit+branch:master+topic:auth-backends-HttpAuthProtocol changes] |
| * Owner: [https://gerrit-review.googlesource.com/#/q/owner:%22Dariusz+%25C5%2581uksza%22+status:open Dariusz] |
| * Status: review started, partially merged |
| * Issues: after having merged part of it, the review is now stuck. Needs Shawn attention as the first attempt to merge it broke the Gerrit authentication. |
| |
| =On-Line edit= |
| |
| Allows to perform small edits on the files included in an existing Gerrit change in order to submit a new patch-set. Currently it is already possible on the commit message but not on the code. |
| |
| The overall user-experience is nice and is provides a very quick and easy shortcut for avoiding to go through the check-out / modify / commit --amend / push workflow for small changes. GitHub provides already this feature and is really nice! |
| |
| * Gerrit changes: [https://gerrit-review.googlesource.com/#/q/status:open+project:gerrit+branch:master+topic:inline-3 changes] |
| * Owner: [https://gerrit-review.googlesource.com/#/q/owner:%22David+Ostrovsky+%253CDavid.Ostrovsky%2540gmx.de%253E%22+status:open DavidO] |
| * Status: third version of the on-line edit. Demoed at the last Gerrit User Summit 2014 @Google. Review should be finalized |
| * Issues: Third major overhaul of the series (after migration from CS to CS2 and decoupling of revision edits from GET /changes/<id>/detail REST endpoint): Edits are now stored on /refs/users/UU/UUUU/edit-CCCC ref and max. one edit per change per user. Parent of edit commit is the SHA-1 of the patch set it is based on. The major parts of the series were reviewed and tested by Björn Pedersen, David P., Shawn and Martin. |
| |
| =Secure Store= |
| |
| Makes the Gerrit secure.config more secure by encrypting its values. This change aims to make the security system applied to the secure.config more generic and pluggable. Potentially the key-value pairs could be stored in an external security vault and not even persisted in the filesystem. |
| |
| The challenges are actually on where the secure.config is used: all over the place in Gerrit whenever a @GerritServerConfig annotation is present. Additionally when the config gets created the plugin infrastructure and loader are not available: the pluggability then needs to use another discovery system. |
| |
| * Gerrit changes: [https://gerrit-review.googlesource.com/#/q/status:open+topic:secure-store changes] |
| * Owner: [https://gerrit-review.googlesource.com/#/q/owner:%22Dariusz+%25C5%2581uksza%22+status:open Dariusz] |
| * Status: review started |
| * Issues: positive and negative feedbacks on the change. Needs rethinking / reworking possibly to a much simpler approach. |
| |
| =Angular Gerrit= |
| |
| Dariusz presented at the Gerrit User a prototype for leveraging the REST-API through an AngularJS UX. It has been published to GitHub at https://github.com/dluksza/angular-gerrit. |
| |
| In order to use the Gerrit-Angular integration a set of changes in the Plugin infrastructure are needed and have been uploaded for review. |
| |
| * Gerrit changes: [https://gerrit-review.googlesource.com/#/q/status:open+topic:angular-gerrit-integration changes] |
| * Owner: [https://gerrit-review.googlesource.com/#/q/owner:%22Dariusz+%25C5%2581uksza%22+status:open Dariusz] |
| * Status: changes submitted, topic created. |
| * Issues: None at the moment. |
