| Age | Commit message (Collapse) | Author |
|---|
|
|
|
The hooks documentation does not document the reference-transaction
hook. Add this missing information.
|
|
The hook documentation we have is hard to understand at times, and most
importantly it is out-of-date in many places. Rewrite it to reflect the
current state.
|
|
|
|
While writing docs for gitaly-backup it became clear that using the name
`locator` did not make sense.
This would be a breaking change except that the flag is part of an
experimental feature.
Changelog: changed
|
|
|
|
|
|
Better define 'node' in the design document
See merge request gitlab-org/gitaly!3773
|
|
|
|
Changelog: other
|
|
Changelog: removed
|
|
Because Postgres database is required for tests to pass
we explicitly says about it in the documentation and provide
a simple instruction on how to get one.
The change also updates documentation of the glsql package
to align it with the new approach. The old info that is not
actual anymore is removed.
|
|
Add documentation for pack-object cache
See merge request gitlab-org/gitaly!3636
|
|
Changelog: other
|
|
Changelog: other
|
|
GITALY_SOCKET_PATH, GITALY_LISTEN_ADDR and GITALY_PROMETHEUS_LISTEN_ADDR
were marked as deprecated in eb5b9b3af (Document GITALY_DEBUG, 2017-05-10).
It is time to remove them from the code.
To exclude situation when some configuration values could be overridden
by the env vars by mistake the change removes envconfig.Process call
on the gitaly configuration struct.
|
|
Reference transactions have now been baking for several releases and is
performing well. With backchannel voting having landed in release 13.12,
the last blocker which has kept customers from deploying these has been
removed. As a result, we can now finally drop the feature flag
altogether and always enable reference transactions.
Changelog: changed
|
|
The path re-writer is the go script to re-write imports
in the go source code files, proto files and go.mod file.
The script accepts path to the project dir where go.mod file
locates, current module version and desired module version.
Upgrading a module requires re-generating the gRPC stubs
from proto file that is why the code of the path re-writer
script is imported in a new 'upgrade-module' task which covers
that need.
Part of: https://gitlab.com/gitlab-org/gitaly/-/issues/3177
|
|
This commit adds documentation about the reconciliation jobs the
reconciler schedules to fix various inconsistencies that might happen
in a virtual storage.
|
|
This updates the virtual storage documentation to cover repository
assignments.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Now that we have a single distribution of git across all of GitLab's
official distributions of Gitaly, we are in a nicer position to add
custom patches to git. In order to ensure high standards and assert that
we won't ever start to diverge from upstream, this commit documents the
high bar we want to set ourselves for adding custom patches.
|
|
|
|
|
|
It wasn't clear from the existing checklist item added in
2d9d7f9c9 (Add roll out issue template, 2019-06-11) what the removal
steps are, and what "remove feature flag" meant.
Split this up into steps where we discuss the removal from the
codebase, and what changelog entries we expect there, and then discuss
how to remove the feature via chatops.
|
|
|
|
Amend the docs I added in 86c8480e8 (Feature flag rollout doc: expand
on post-100% steps, 2021-01-12) to mention the example of
0ff3ee285 (Remove on-by-default go_user_delete_{branch,tag} feature
flags, 2021-01-21) instead of fbc9f83ab (Remove Ruby code for 100% on
user_delete_{branch,tag} in Go feature, 2021-01-15).
As noted in 0ff3ee285 what I did in fbc9f83ab was wrong &
dangerous. Let's not recommend it.
|
|
I screwed this up in [1] and [2] and as a result the feature didn't
get enabled 100% on January 14th like I thought, but on January 27th
when the changes [3][4] to remove the feature flag itself went
live ([4] being the relevant change).
1. https://gitlab.com/gitlab-org/gitaly/-/issues/3412#note_485304117
2. https://gitlab.com/gitlab-org/gitaly/-/issues/3413#note_485303898
3. https://gitlab.com/gitlab-org/gitaly/-/merge_requests/3035
4. https://gitlab.com/gitlab-org/gitaly/-/merge_requests/3033
|
|
Re-flow a paragraph I added in 40f953069 (Feature flag rollout doc:
rewrite & make better use of template, 2020-12-15).
|
|
doc: Add documentation on object pools
See merge request gitlab-org/gitaly!3060
|
|
After long rounds of refactoring we finally can remove
git.NewCommand function.
The change also includes removal of the TODO left because
of the NewCommand usage with the global config.Config variable.
And as a final step the doc is updated and now mentions usage
of the git.CommandFactory interface instead of git.NewCommand().
Part of: https://gitlab.com/gitlab-org/gitaly/-/issues/2699
|
|
|
|
We do not have any documentation of our object pools. This commit adds
an initial document which details some aspects of how they work and what
problem they aim to solve.
|
|
|
|
You land on a 404 once you sign in, this is confusing, document it a
bit.
|
|
I added this because of a question @pks-t had in
https://gitlab.com/gitlab-org/gitaly/-/merge_requests/2995#note_484390302
It's easy to miss that we auto-add the featureflag::disabled flag
initially, let's cover that and provide handy links to the tracker to
see the status of each label.
|
|
Add more steps reflecting what we need to do after we have the feature
at 100%. Includes a paraphrased version of what I noted in my
a96949aef (Enable feature flag go_user_delete_{branch,tag} by default,
2021-01-12) as part of
https://gitlab.com/gitlab-org/gitaly/-/merge_requests/2994
|
|
I only found out about this today, or I would have added it in my
40f953069 (Feature flag rollout doc: rewrite & make better use of
template, 2020-12-15).
|
|
Beginners guide doc: markdown quote localhost:3000
See merge request gitlab-org/gitaly!3001
|
|
Back when we introduced the Git DSL, we still had conflicting sets of
safe and unsafe functions. Because of this legacy, our safe set of
functions is still has the "Safe" prefix.
This commit now ends that chapter and renames `SafeCmd()`. In alginment
with the preceding renames, this is being renamed to `NewCommand()`.
|
|
The rest of the "localhost:3000" occurrences are quoted, let's quote
this one too.
|
|
Rather than duplicating things between the issue template and process
doc, let's cross link the two.
Also elaborate on a lot of things I encountered when enabling my first
features. This is the result of feedback on the initial version of
this [1] and a video chat with @zj-gitlab this morning.
1. https://gitlab.com/gitlab-org/gitaly/-/merge_requests/2910
|
|
|
|
|
