| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Idabd8994651451183472eb3b777064e891e8a765
(cherry picked from commit 136ecbda21ac18f0e3bdfd332f6f90a695fe5baa)
|
|\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.2:
Bazel: Disable worker multiplexer to avoid sporadic build failures
Change-Id: Ide5463a529ad665c263422f892b239df32f5268e
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Bazel 4.0.0 has a known worker multiplexer bug, that causes Bazel to
crash spordically.
The bug was fixed: [1], but the fix wasn't released yet. See this
issue: [2] for more details.
[1] https://github.com/bazelbuild/bazel/commit/a607d9dc70ac67f1aa2c32ca954177f9c77860be
[2] https://github.com/bazelbuild/bazel/issues/13333
Change-Id: I2265c4ed7128a4b6ed259f44c5594ab717de58b0
|
|\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
* stable-3.2:
Remove unused _handleDropdownTap
Change-Id: I33cec95fe44ffb6ce875ddeb4f6dbd4a92e42674
|
| |\ \ \ \ |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Change-Id: Ie4a8effd6d814985cafa6dea9239903f503dae33
(cherry picked from commit 0762766561fdab34bb38b6037ab8040fe65a8bda)
|
|\ \ \ \ \ \ |
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: Ie4a8effd6d814985cafa6dea9239903f503dae33
(cherry picked from commit 0762766561fdab34bb38b6037ab8040fe65a8bda)
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Currently fataling because in construct we call
_getProjectBranchesSuggestions without params and
thus is undefined/false.
We remove the check as it is not needed.
Change-Id: I8d4ead0bf3a8c30d0ecefdc190e4ba4ea7ede29d
(cherry picked from commit 7db9fdcdd7e78140fee2ee3b662d94bddcbbe6a9)
|
|\ \ \ \ \ \ \
| | |/ / / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* stable-3.2:
Fix binding of DELETE REST calls from plugins
Change-Id: I22a4aba0cfef5e7252a41ad4644baa7f6eee48f2
|
| |\ \ \ \ \ \
| | | |_|/ / /
| | |/| | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* stable-3.1:
Fix binding of DELETE REST calls from plugins
Change-Id: I455ee56213355cbf2c555233af22322cb40d0aab
|
| | |\ \ \ \ \ |
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
As reported in the referenced issue the deletion of projects using:
DELETE /projects/foo
didn't work, while the other option, using POST request worked:
POST /projects/foo/delete-project~delete
The processing of the DELETE request actually never reached the
delete-project plugin due to bug in the RestApiServlet which wrongly
delegated the request to a child collection in Gerrit core. Limit that
"fallback" to GET requests only.
Bug: Issue 14127
Change-Id: I9b9632e8f719937e5f7c61466996be79e6f29c14
(cherry picked from commit 899698797ca1952c9adf3f5d098ee22dbbe22481)
|
|\| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.2:
OpenID: Remove support for Yahoo! OpenID provider
e2e-tests: Add a force option to delete project scenario
Set version to 3.2.9-SNAPSHOT
Set version to 3.2.8
Set version to 3.1.14-SNAPSHOT
Set version to 3.1.13
Change-Id: I25c15df92ce7ca2bc7c0e9de23794ffd0664dd9f
|
| |\| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.1:
OpenID: Remove support for Yahoo! OpenID provider
e2e-tests: Add a force option to delete project scenario
Set version to 3.1.14-SNAPSHOT
Set version to 3.1.13
Change-Id: I866e4843d95aca8a638f7bac84b81656a5631140
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Yahoo! discontinued support for OpenID provider: [1]
"OpenID2 will be EOLed on 6/30/20"
[1] https://developer.yahoo.com/oauth2/guide/OpenID2
Bug: Issue 13140
Bug: Issue 14330
Change-Id: I49a3adf5a331f119af8393d2403e98a7b3dcd654
|
| | |/ / / / /
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Before this change, DeleteProject step only performed project deletion
if there were no open changes within the test project. So, if one of
the test fails, leaving behind an open change. As a result the test
project will not be deleted in the DeleteProject step. Next run of the
same test will also keep failing until the test project is manually
removed.
In a scenario of automotive tests, a CI will keep producing erroneous
test reports, until the user fixes the problem by manually deleting the
test project.
With this change project deletion can be done with force option. Force
deletion is useful because it allows tests to run successively even if
one of the test failed and left an open change in a test project.
To enable forced deletion, an environment property needs to be
configured:
-Dcom.google.gerrit.scenarios.force_project_deletion=true
By default force_project_deletion is set to false.
Change-Id: I2b84419b3665b85465a441a214bf23c420fae0c5
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: Id7b374ff292d557d6730f83e87e6aa20247461e9
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: I5786d56e82059464a98080fd3f627ebbdeb1e7b5
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: I23404a1934fc2bd2f9562cc249b50a62a66a3353
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: Ic122a9e5bae5237f766680975cbb99a64579bc39
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: I4e1f4b2f8d5865d3e2a8102d2e5d22bde6bf5f6a
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Change-Id: I49dc6fd7ae7f3eb71edc5c2dafaa7634ede680d6
|
|\| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* stable-3.2:
PG: Remove legacy permission 'publishDrafts'
Add searching duplicate "external" externalIds only differing in case
find-duplicate-usernames.sh: add example output of git grep
Fix empty TopMenu drop down list
Script to find duplicate usernames only differing in case
AccountPatchReview mariadb: fix key length
RepositorySizeQuotaIT: change try/catch to assertThrows
Enforce repository size on pack rather than on object
Change-Id: I1cca72c0c6d47e7ed085d8a26af115f1a903c576
|
| |\| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
* stable-3.1:
PG: Remove legacy permission 'publishDrafts'
Add searching duplicate "external" externalIds only differing in case
find-duplicate-usernames.sh: add example output of git grep
Fix empty TopMenu drop down list
Script to find duplicate usernames only differing in case
AccountPatchReview mariadb: fix key length
RepositorySizeQuotaIT: change try/catch to assertThrows
Enforce repository size on pack rather than on object
Change-Id: Ida990f451a4b2bcaf4da33536771aa880fa2eb8d
|
| | |\ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.0:
RepositorySizeQuotaIT: change try/catch to assertThrows
Enforce repository size on pack rather than on object
Change-Id: I0936e512047c8c0a947a4470fd204b0ea4d32a27
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
It offers the same functionality with less boilerplate code.
Change-Id: If5e4876e8bb21a40da28fe977ea0227022bbf32e
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Issue:
Originally max repository size was enforced by the quota plugin.
However, that functionality was a perfect candidate for core quota
implementation and as such was implemented in [1].
As a result, Gerrit core is responsible for making sure that the limit
is enforced and the plugin is responsible for providing the limit.
Unfortunately, Gerrit core implementation developed a fault as it
enforced the limit on the object size instead of the pack size leading
to the situation where multiple smaller objects would fall under the
limit where the pack they belong to should already exhaust it.
Solution:
Enforce available repository size tokens on pack size (as it was
done this way originally in plugin).
[1] https://gerrit-review.googlesource.com/c/gerrit/+/224728
Bug: issue 14118
Change-Id: I37e573971a76abe53a3e6456498940a4bb803d0c
|
| | |\ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Setting up the db on mariadb currently fails with
com.google.gerrit.exceptions.StorageException: create failure on ACCOUNT_PATCH_REVIEWS
at com.google.gerrit.server.schema.JdbcAccountPatchReviewStore.createTableIfNotExists(JdbcAccountPatchReviewStore.java:188)
...
Caused by: java.sql.SQLSyntaxErrorException: (conn=13) Specified key was too long; max key length is 3072 bytes
If35aff6c8d dealt with this for mysql, this just ports it to mariadb.
Out of interest, I did a query on the file_name size on our production
database.
+---------------+---------------+---------------+----------------+
| pct_length_10 | pct_length_20 | pct_length_50 | pct_length_100 |
+---------------+---------------+---------------+----------------+
| 2.14 | 15.96 | 77.80 | 99.48 |
+---------------+---------------+---------------+----------------+
So 100 characters gets 99.48% of file names. Our longest filename is
249 characters apparently. We could probably optimize this with
partial indexes if required.
Change-Id: Ide9a8c647565d657d691a73be76129f237bf6ebb
|
| | |\ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
case" into stable-3.1
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Change-Id: I6474f63ff219b992efa30949c8faf060f33a592d
|
| | |\| | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
stable-3.1
|
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Change-Id: Idf843d61577f9c01695f6b5996a1a935b0dedd41
|
| | |/ / / / / / /
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
This was removed in 2.15 but was for some reason left in the FE.
Permission was removed with:
6ffb7d9358 Discontinue draft workflow
Change-Id: I21241f603bebe8b71ab0f9c4b98890d9d5d00bbc
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Before this change, external plugin can add new Menu List without proper
menu link. Empty menu list generates "TypeError: this.$.cursor.target is
undefined" onClick.
Issue appears on any plugin extending TopMenu with url containing
${projectName} or with empty MenuItem list.
Bug: Issue 14257
Change-Id: I5d09a4f0bbd24fcff1dec0b65d9366bdb05249f3
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Usage:
cd /path/to/All-Users.git
find-duplicate-usernames.sh [username|gerrit]
This script finds duplicate usernames only differing in case in the
given account schema ("username" or "gerrit") and their respective
accountIds.
Change-Id: I16713471ef026736274695f815428bf68c768e0c
|
|\| | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
* stable-3.2:
Apply a second filter after receiving the initial repo list
Change-Id: I5b082a816abc62b8a65dbd54b3dd36ecc5e59a69
|
| |\| | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
* stable-3.1:
Apply a second filter after receiving the initial repo list
Change-Id: I089efbe31c722e1753dad4942070adcab1cfbf05
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Before this change, searching a repository where the prefix is similar
to the name was problematic. It sent a query having two similar
conditions which resolved to one condition. For example, searching for a
project named f/f would send a query inname:f AND inname:f. This is
problematic when listing all the related projects to the filter, which
may not correspond to the user entry. Additional examples are in [1].
Before this change, if the user entered a filter containing a special
character, the query sent to the backend was modified to correspond to
the query language. The reason behind it being that the query language
uses some special characters as operators. Therefore, the result of the
output method was inaccurate and could contain extra data that are not
needed by the user.
This change introduces a second method that compares the filter with the
output list of the fetch method. This second filter takes the original
filter and compares it with each element of the resulting array from the
fetch method. This ensures that the output received by the user matches
their entry.
With this change, users should get the same output as the old UI.
[1] https://groups.google.com/u/0/g/repo-discuss/c/pfxB4_75EL0/m/dfT-nnnMAwAJ
Feature: Issue 12970
Change-Id: I5078237a9648871eb59628d4669ffa0b5b42cc08
|
|\| | | | | | | |
| |_|_|_|_|_|/ /
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.2:
GerritSimulation: Add method to get property directly from options
error-commit-already-exists.txt: fix error message
e2e-tests: Refactor duplicated "number" key name
PolyGerrit: Remove duplicate key from language map
e2e-tests: add RestoreChange scenario
Documentation: dev-bazel: Fix indent level
Ensure InternalUser can parse groups
PolyGerrit: Highlight tex files
e2e-tests: add AbandonChange scenario
Stop SPAMming Gerrit logs with 'Logging context is not empty'
Update git submodules
Update git submodules
Update git submodules
Update git submodules
GitRepositoryManager should be responsible to enable/disable the GC
e2e-tests: Fix regex to adapt to JSON_COMPACT
Change-Id: I806371a8c4564838e51f217e7ec95e79b84d1052
|
| |\| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.1:
GerritSimulation: Add method to get property directly from options
error-commit-already-exists.txt: fix error message
e2e-tests: Refactor duplicated "number" key name
Change-Id: I91959c25334a94981ea6b192a049222ba80ddeb2
|
| | |\ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
into stable-3.1
|
| | | |/ / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Before this change, in order to get a property, the replaceOverride
method was overriden to get the input of the user and then pass this
input as a parameter to the replaceProperty method. This had a
restriction that the method can only be overridden once by the plugin
test. However, some plugins, like gc-conductor, need to use it through
different variables that need to be interpreted as another type than a
string.
This change introduces a method that fetches automatically the input of
a property if it exists. The return of this method is a string but it
doesn't have to be overridden.
Change-Id: I311795614ae0d2e86b5824025a2a50f130136c4f
|
| | |\ \ \ \ \ \ |
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
CreateChange scenario now saves the "_number" taken from http response
as "number" instead, which is numberKey.
Change-Id: I41b250ed2ca0ff1bebe39b8a00d3f690332589b1
|
| | | |/ / / / /
| | |/| | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
The error message was changed in:
9775399 Fix %base option and "Create a new change for..." setting
But the message was never changed in the error message description.
Change-Id: Ifab82f7393d078a175e5a1fc9c01338ea04036fb
|
| |\| | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
* stable-3.1:
PolyGerrit: Remove duplicate key from language map
e2e-tests: add RestoreChange scenario
Documentation: dev-bazel: Fix indent level
PolyGerrit: Highlight tex files
e2e-tests: add AbandonChange scenario
Stop SPAMming Gerrit logs with 'Logging context is not empty'
Update git submodules
Update git submodules
Update git submodules
GitRepositoryManager should be responsible to enable/disable the GC
e2e-tests: Fix regex to adapt to JSON_COMPACT
Change-Id: I20d16ee00010b96c60b7724ac46a303400dbc0fd
|
| | |\ \ \ \ \ \ |
|
| | | |/ / / / /
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
In I41f855703f0 duplicate key for LANGUAGE_MAP was added. For one,
JavaScript disallows duplicated key, for another latex mode was already
fixed with the new release of highlight.js:
"69:3 error Duplicate key 'text/x-latex' no-dupe-keys"
Bug: Issue 14091
Change-Id: I55d70b429d4874ba419dd36b90162e69ab7ec3f0
|