| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
|
| |
Relates to
https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/7743#note_212379866
|
| |\
| |
| |
| |
| | |
Replace indexes for counting active users
See merge request gitlab-org/gitlab-ce!32538
|
| |\ \
| |/
| |
| |
| | |
Replace indexes for counting active users
See merge request gitlab-org/gitlab-ce!32538
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This adjusts the partial condition for an index. The index is intended
to be used when counting active users with `ghost IS NOT TRUE AND
bot_type IS NULL`.
With the current index, this wasn't working as the partial condition
didn't match the query: `ghost <> TRUE` is not semantically equivalent
to `ghost IS NOT TRUE` (null semantics).
The reason we add an index particularly intended for EE is that the EE
query is going to have the additional part `AND bot_type IS NULL`
whereas the CE query doesn't. Logically, it'd be enough to have an index
for `ghost IS NOT TRUE`. However, on GitLab.com, the query planner makes
poor choices when the additional `AND bot_type IS NULL` part is present:
It goes for the index on `bot_type` and doesn't use the partial index.
Note the existing index isn't being used at all according to GitLab.com
index statistics. Hence we can first remove it and don't have to worry
about the window of time without an index.
Relates to https://gitlab.com/gitlab-org/gitlab-ce/issues/66770
|
| |\ \
| | |
| | |
| | |
| | | |
Handle :9090 and 0.0.0.0:9090 listen_address formats
See merge request gitlab-org/gitlab-ce!32456
|
| |/ /
| |
| |
| |
| |
| | |
Prometheus listen_address can be in formats of :9090 and 0.0.0.0:9090.
But before these can be used to connect a project to Prometheus, they
have to converted into absolute URIs.
|
| | |
| |
| | |
[ci skip]
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Update screenshots for License Compliance
Closes gitlab-ee#8910
See merge request gitlab-org/gitlab-ce!32537
|
| | |/ |
|
| |\ \
| | |
| | |
| | |
| | | |
Support selective highlighting of lines
See merge request gitlab-org/gitlab-ce!32514
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Instead of highlighting all lines when not all of them
are needed, only highlight from the beginning up to
the specified limit.
The `BlobPresenter#highlight` method has been updated
to support `to` param. This param will be used to limit
the content to be highlighted.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Added example for passing Maven repo credentials
Closes gitlab-ee#6711
See merge request gitlab-org/gitlab-ce!32073
|
| |/ / / |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Docs james gill contribution
See merge request gitlab-org/gitlab-ce!32520
|
| |/ / / |
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
CE: Reduce amount of property definitions in ChatNotificationService
See merge request gitlab-org/gitlab-ce!32363
|
| |/ /
| |
| |
| |
| |
| |
| | |
Before this commit we redefined properties for supported events on each
new instance of ChatNotificationService during "runtime".
This commit defines the properties only once during "compile time".
|
| |\ \
| | |
| | |
| | |
| | | |
Fix permissions check in `RelativeLinkFilter`
See merge request gitlab-org/gitlab-ce!32448
|
| | | |
| | |
| | |
| | |
| | | |
Fixes RelativeLinkFilter for users that don't have access
to the project's repository
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Update views for testability and update page objects
See merge request gitlab-org/gitlab-ce!32451
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Before this change, by default the expand button would only be
clicked after 10 seconds, but now it will only wait for 1 sec.
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Add CI Pages template to display static OpenAPI specification files in SwaggerUI
See merge request gitlab-org/gitlab-ce!31183
|
| |/ / / /
| | | |
| | | |
| | | | |
as requested by the reviewer
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
'65389_wrong_format_on_ms_teams_integration_push_events_with_multi_line_commit_messages' into 'master'
Fixing #65389 - Wrong format on MS teams integration push events with multi line commit messages
Closes #65389
See merge request gitlab-org/gitlab-ce!32180
|
| |/ / / /
| | | |
| | | |
| | | | |
Wrong format on MS Teams integration push events with multi line commit messages
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
quick action label must be first in issue comment
Closes #60561
See merge request gitlab-org/gitlab-ce!32367
|
| |/ / / / |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Rename shared examples according to our guidelines
Closes gitlab-ee#14316
See merge request gitlab-org/gitlab-ce!32526
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \ \ \ \
| |/ / / /
|/| | | |
| | | | |
| | | | | |
Remove lint:javascript:report job
See merge request gitlab-org/gitlab-ce!32462
|
| |/ / / / |
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Fix Peek on Puma
Closes #66528
See merge request gitlab-org/gitlab-ce!32213
|
| | | | | | |
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Previously, we called the `peek_enabled?` method like so:
prepend_before_action :set_peek_request_id, if: :peek_enabled?
Now we don't have a `set_peek_request_id` method, so we don't need that
line. However, the `peek_enabled?` part had a side-effect: it would also
populate the request store cache for whether the performance bar was
enabled for the current request or not.
This commit makes that side-effect explicit, and replaces all uses of
`peek_enabled?` with the more explicit
`Gitlab::PerformanceBar.enabled_for_request?`. There is one spec that
still sets `SafeRequestStore[:peek_enabled]` directly, because it is
contrasting behaviour with and without a request store enabled.
The upshot is:
1. We still set the value in one place. We make it more explicit that
that's what we're doing.
2. Reading that value uses a consistent method so it's easier to find in
future.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Peek's `Peek.request_id` method doesn't work well with a multi-threaded
server and concurrent requests, because requests can 'steal' another
request's ID, or unset it before it was due.
The upstream change resolves this; the commit here is just to ensure
that GitLab works with that upstream change, mostly by not using
`Peek.request_id` any more (as the method doesn't exist).
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Add CSS & JS global flags to represent browser and platform on login pages
See merge request gitlab-org/gitlab-ce!32521
|
| | | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
- Adds CSS classes to represent browser & platform
- Adds JS flags to `gl` object to represent
browser & platform
|
| |\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Resolve "Improve pages load wait time experience"
Closes #65304
See merge request gitlab-org/gitlab-ce!32122
|
| | | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This commit adds a warning message that is always displayed that informs
users that their GitLab Pages sites will take a bit of time before they
can be accessed after being deployed for the first time.
|
| |\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Update Mermaid to v8.2.6
See merge request gitlab-org/gitlab-ce!32502
|
| | | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
This fixes a regression where an underscore in labels no
longer worked:
https://github.com/knsv/mermaid/releases
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Fix space in geo doc
See merge request gitlab-org/gitlab-ce!32522
|
| |/ / / / / / / / |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Clarify new note for GitLab Pages
See merge request gitlab-org/gitlab-ce!32443
|
| |/ / / / / / / / |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
'14634-broken-master-ee-spec-services-slash_commands-global_slack_handler_spec-rb-ce' into 'master'
Resolve "Broken master: ee/spec/services/slash_commands/global_slack_handler_spec.rb"
Closes #14634
See merge request gitlab-org/gitlab-ce!32518
|
| | | | | | | | | | |
|
| | | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
- Pass in a nil project to Gitlab::SlashCommands::ApplicationHelp.new
if the chatops command is help.
- Modify the Gitlab::SlashCommands::Presenters::Help message to skip
printing out project information.
|
