| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Wiki attachments can be workhorse accelerated.
This commit is backward compatible with older workhorse
|
|
|
|
| |
This class encapsulates our use of the Danger gem.
|
|\
| |
| |
| |
| | |
Port CreateGithubPullRequestEvents migration from EE
See merge request gitlab-org/gitlab-ce!31802
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Detect if pipeline runs for a GitHub pull request
When using a mirror for CI/CD only we register a pull_request
webhook. When a pull_request webhook is received, if the
source branch SHA matches the actual head of the branch in the
repository we create immediately a new pipeline for the
external pull request. Otherwise we store the
pull request info for when the push webhook is received.
When using "only/except: external_pull_requests" we can detect
if the pipeline has a open pull request on GitHub and create or
not the job based on that.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Most of the project templates bundled with GitLab have an empty string
set for `external_authorization_classification_label` in their
`project.json`. When imported this overrides the default label set on
the instance, causing the label to appear empty.
This change sets empty labels to nil during import, allowing the
default label to override it and be applied to the project.
|
|\ \
| | |
| | |
| | |
| | | |
Revert "Merge branch '66596-allow-danger-to-be-run-locally' into 'master'"
See merge request gitlab-org/gitlab-ce!32741
|
| | |
| | |
| | | |
This reverts merge request !32196
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
'59729-estimate-quick-action-does-not-produce-correct-time-for-1mo' into 'master'
Resolve "estimate quick action does not produce correct time for 1mo"
See merge request gitlab-org/gitlab-ce!32165
|
| | |
| | |
| | |
| | |
| | | |
Patches ChronicDuration to use our custom conversions
when parsing months
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Break up Danger rules into local or CI only
See merge request gitlab-org/gitlab-ce!32196
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The benefit of running the danger_local Rake task
is that it displays some informational messages
for clarity and also exits with 1 if there's any
output.
|
| | | |
| | | |
| | | |
| | | | |
This class encapsulates our use of the Danger gem.
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
Support adding and removing labels w/ push opts
Closes #5942
See merge request gitlab-org/gitlab-ce!31831
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
MergeRequests::PushOptionsHandlerService has been updated to allow
adding and removing labels to a merge request using git push options.
To create a new merge request and add 2 labels to it:
git push -u origin -o merge_request.create \
-o merge_request.label="My label 1" \
-o merge_request.label="My label 2"
To update an existing merge request and remove a label while
adding a different label:
git push -u origin -o merge_request.label="My added label" \
-o merge_request.unlabel="My removed label"
Issue https://gitlab.com/gitlab-org/gitlab-ce/issues/64320
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Add Danger Roulette rules for pipeline files
See merge request gitlab-org/gitlab-ce!32616
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
into 'master'
Resolve "user-content.gitlab-static.net brings back 404 only"
See merge request gitlab-org/gitlab-ce!32703
|
| |/ / / / |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Since it is not possible to dynamically detect if a job is automatically
cancellable or not, a this new attribute is necessary. Moreover, it let
the maintainer of the repo to adjust the behaviour of the auto cancellation
feature to match exactly what he needs.
|
|\ \ \ \
| |_|/ /
|/| | |
| | | |
| | | | |
Extract /internal/allowed API Actor logic out
See merge request gitlab-org/gitlab-ce!31564
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Created new API::Support::GitAccessActor class to
encapsulate some of the more edge logic, making
the /internal/allowed route much cleaner.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Move code for creating a new column from old into a function so that
it can be reused.
- Also add comments above the methods.
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'66150-remove-dynamically-constructed-feature-flags-starting-with-prometheus_transaction_' into 'master'
Resolve "Remove dynamically constructed feature flags starting with `prometheus_transaction_`"
Closes #66150
See merge request gitlab-org/gitlab-ce!32395
|
| | | |
| | | |
| | | |
| | | | |
Those feature flags were always enabled so we can remove them safely.
|
|/ / / |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Avoid calling freeze on already frozen strings in lib/gitlab
See merge request gitlab-org/gitlab-ce!32637
|
| | | | |
|
| | | | |
|
| |/ /
|/| |
| | |
| | |
| | | |
Now, when the dns rebinging setting is disabled, we will
allow urls that are not resolvable.
|
| | |
| | |
| | |
| | |
| | |
| | | |
- `edge_nodes` needs to get called on the object
- added `include GlobalID::Identification` in a couple places
- renamed `object` to `item` in spec due to conflict
|
| | |
| | |
| | |
| | |
| | | |
When using /due quick action with an invalid date a meaninful error
message is shown.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- Due to https://github.com/exAspArk/batch-loader/pull/32,
we changed BatchLoader.for into BatchLoader::GraphQL.for
- since our results are wrapped in a BatchLoader::GraphQL,
calling `sync` during authorization is required to get real object
- `graphql` now has it's own authorization system. Our
`authorized?` method conflicted and required renaming
|
| | |
| | |
| | |
| | |
| | | |
The current state of group issue boards does not show the "Add issues"
button on the UI for users that are reporters of group child projects.
|
| | |
| | |
| | |
| | | |
(cherry picked from commit 0bf7fecdee5a200eeb4f27b4888a57ceee63bde9)
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Persist `needs:` validation as config error
See merge request gitlab-org/gitlab-ce!32648
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In case when `needs:` is missing, but when requested by service,
we would not save the pipeline with config_error.
This makes it explicit that we want to persist the error
as `config_error` failure reason.
|
| | | |
| | | |
| | | |
| | | | |
Improve diff lines count efficiency
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Extract Workhorse <-> GitLab JWT authentication to make it reusable
See merge request gitlab-org/gitlab-ce!32623
|
| | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | | |
Introduce JWTAutheticatable module that can be reused for ai=uthtication
between Pages and GitLab (the same way we use do now for Workhorse).
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/61927.
|
|/ / /
| | |
| | |
| | |
| | |
| | | |
so that we can use API::Internal namespace.
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/61927.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix performance bar thresholds
See merge request gitlab-org/gitlab-ce!32587
|
| | | |
| | | |
| | | |
| | | |
| | | | |
These were written in seconds but are supposed to be in
milliseconds. The total Gitaly time was wrong for the same reason.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Resolve "Typo in RELEASE_ENDPOINT_REQUIREMETS constant"
Closes #66464
See merge request gitlab-org/gitlab-ce!32468
|
| | |/ /
| |/| | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Improve GC_REPORT_BUCKETS granularity
See merge request gitlab-org/gitlab-ce!32586
|
| | | | | |
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Modified schema via migrations.
Added one-to-one relationship between the two models.
Added changelog file
|