Age | Commit message (Collapse) | Author |
|
|
|
go: Update module github.com/pelletier/go-toml/v2 to v2.0.5
See merge request gitlab-org/gitaly!4856
|
|
Add #gitaly-alerts to team member onboarding template
See merge request gitlab-org/gitaly!4859
|
|
|
|
Log only mode for new repository calculation methods
Closes #4447
See merge request gitlab-org/gitaly!4848
|
|
Use the disk usage formatting directive to calculate the repository size
using cat-file. Also change the logging statement to log the bytes,
rather than kilobytes for the sake of clarity.
|
|
go: Update module google.golang.org/grpc to v1.49.0
See merge request gitlab-org/gitaly!4853
|
|
go: Update github.com/ProtonMail/go-crypto digest to 4b6e5c5
See merge request gitlab-org/gitaly!4847
|
|
blob: Standardize returned errors
See merge request gitlab-org/gitaly!4849
|
|
We should use %w for the error wrapping. That will
allow us to unwrap the error without loosing any context.
It is not use right now, but maybe in the future if we need
to change response based on the returned error in the chain
of interceptors.
And one simplification from fmt.Errorf() to errors.New().
|
|
In some cases error returned by RPC methods are prefixed with
its names. This is redundant as the caller knows exactly what
method is called and also the log entry always contains full
method name including service name. Instead, the prefixes for
errors in some cases were added to better describe what operation
failed. It should help to better navigate to the reason of the
error.
|
|
Replace different representation of the missing repository
argument error with the pre-defined ErrEmptyRepository.
Fix order of arguments validation with localrepo creation in GetBlob.
Replace status.Error(f) with helper.ErrXXX to have a common pattern
and omit possible double-wrapping of the gRPC Status.
|
|
The %(objectsize:disk) formatting directive is useful for calculating
the amount of disk space that an object takes up. Add this as an option.
|
|
The RepositorySize is a high risk API, since an inaccurate number
(especially if it's much higher) can cause projects to be locked due to
usage quotas. In order to safely roll out a new calculation, add a log
only mode such that a feature flag has to be enabled in order to send
the value back to the Rails client. Without this feature flag, we merely
log the new calculation and continue to send the value calculated with
du.
|
|
README.md: Update minimum Go version
See merge request gitlab-org/gitaly!4857
|
|
Our minimum Go version is now 1.17. Update the README to reflect this.
|
|
Remove snowman parameters from links to issues URL
Closes #4442
See merge request gitlab-org/gitaly!4830
|
|
|
|
[ci skip]
|
|
praefect: Add `track-repositories` command
See merge request gitlab-org/gitaly!4845
|
|
A somewhat common way to migrate to Gitaly Cluster it to copy the
contents of an existing storage onto one node of the cluster and
manually track each repository. This is convenient for instances where
the migration must happen as a single 'big bang' with some downtime, vs
the zero-downtime, but much slower method of migrating via the API.
Currently admins can script the `track-repository` to run in a loop when
ingesting large numbers of repositories, but this is inconvenient and
error-prone. To better support this use case, a way to import
repositories in-bulk into Praefect is needed.
This commit add a new `track-repositories` subcommand that takes a JSON
file as input and adds each entry in a single execution.
Changelog: added
|
|
In preparation for a new Praefect subcommand that will track many
repositories in a single request, convert `track-repository` to use a
`trackRepositoryRequest` struct internally.
This will allow us to re-use this code when looping over multiple
requests.
|
|
Currently most output from the `track-repository` subcommand is made via
the commands writer, but a subset is done via logger. This leads to an
inconsistent format where some output is plaintext, while other messages
are in JSON, which is difficult to read.
Let's make this consistent by making all non-debug output use the
writer, which will be simpler to parse for a one-off CLI command.
|
|
Correct spelling of 'executed' in help string for `track-repository`
subcommand.
|
|
Address client issues
Closes #3206
See merge request gitlab-org/gitaly!4846
|
|
|
|
|
|
ruby: Update dependency gitlab-labkit to '~> 0.24'
See merge request gitlab-org/gitaly!4664
|
|
Simplify response type of FindLocalBranches
See merge request gitlab-org/gitaly!4850
|
|
template: Update issue templates
See merge request gitlab-org/gitaly!4837
|
|
[ci skip]
|
|
[ci skip]
|
|
[ci skip]
|
|
README: Add Praefect walkthrough video to training section
See merge request gitlab-org/gitaly!4852
|
|
|
|
New Gitaly team members need access to both Zendesk and gitaly ruby gem.
This change adds these requests to the onboarding issue template.
|
|
The `#g_create_gitaly` Slack channel has been renamed to `#g_gitaly`.
This change updates the feature flag and onboarding issue templates
accordingly.
|
|
[ci skip]
|
|
Do not assume client users want the same tracing interceptors with the same configuration.
|
|
|
|
Using a feature flag 'SimplifyFindLocalBranchesResponse', let's add
'LocalBranches' to the 'FindLocalBranchesResponse' structure. After the
rollout we'll deprecate the usage of 'Branches' field and only use
'LocalBranches'.
Modify the tests to accommodate this new flag.
|
|
Add a new flag 'SimplifyFindLocalBranchesResponse' which will facilitate
the addition of the new response type to 'FindLocalBranchesResponse'.
The new field 'LocalBranches' is meant to deprecate the existing
'Branches' field. The new field is of the generic 'Branch' type.
|
|
In the previous commit (cd071fd027d4dddecf69db4477e2becb25becf59) we
renamed 'assertContainsBranch' to
'assertContainsAllBranchesResponseBranch'. In this commit, let's
introduce a new 'assertContainsBranch' function which checks if a
'gitalypb.Branch' is present in a list of the same type.
|
|
Rename the function 'assertContainsAllBranchesResponseBranch' and its
usecases because its name is not generic enough and we want to
introduce a more generic 'assertContainsBranch' commit in the following
commits.
|
|
go: Update module github.com/rubenv/sql-migrate to v1
See merge request gitlab-org/gitaly!4698
|
|
'master'
Convert UserCreateBranch to use structured errors
Closes #4227
See merge request gitlab-org/gitaly!4835
|
|
Fix flaky tests of the TestGitalyServerFactory test suite
Closes #3916
See merge request gitlab-org/gitaly!4841
|
|
docs: Public Grafana dashboards does not exist any more
See merge request gitlab-org/gitaly!4831
|
|
proto: return 'Branch' in 'FindLocalBranchesResponse'
See merge request gitlab-org/gitaly!4842
|
|
proto: Remove deprecated drift_threshold_millis
Closes #4417
See merge request gitlab-org/gitaly!4843
|