| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
We added retry_count increment when job is recovered
|
| |
|
|
|
|
|
| |
In the admin user edit form, access level radio button labels didn't have
the correct 'for' value. Clicking on the label did not select the radio
button. This makes usability a bit nicer since the click area is increased
when the label is clickable.
|
| |
|
|
| |
This reverts commit c3751046d217008404a0bd371e59d6ffd6734923.
|
| |
|
|
| |
Add project path to sql query to build proper path
|
| |
|
|
|
|
|
| |
API endpoints for requesting container repositories
and container repositories with their tag information
are enabled for users that want to specify the group
containing the repository rather than the specific project.
|
| | |
|
| |\
| |
| |
| |
| |
| |
| |
| |
| | |
'63474-applying-a-quick-action-does-not-show-the-correct-message' into 'master'
Fixed bug from extra parenthesis
Closes #63474
See merge request gitlab-org/gitlab-ce!31479
|
| | | |
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
'11467-remove-feature-flag-for-visual-review-app-button-ce-backport' into 'master'
Remove visual review app feature flag
See merge request gitlab-org/gitlab-ce!31429
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is the CE backport for
https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14958
Now that we are confident with the visual review
app link being there, we are removing the
feature flag entirely.
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
'master'
Refactor passing props to jobs/index.js
See merge request gitlab-org/gitlab-ce!31472
|
| | | | |
|
| |/ /
| |
| |
| | |
- Also add requirements to the docs for custom metrics.
|
| |\ \
| | |
| | |
| | |
| | | |
Optimize relative re-positioning
See merge request gitlab-org/gitlab-ce!30938
|
| | | |
| | |
| | |
| | | |
Also removes unnecessary methods causing extra queries
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Moving of neighbour items was done recursively - this
was extremely expensive when multiple items had to be moved.
This change optimizes the code to find nearest possible gap where
items can be moved and moves all of them with single update query.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'master'
Resolve "Kubernetes > Applications > Uninstall Cert Manager"
See merge request gitlab-org/gitlab-ce!31166
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Our current version of Cert-Manager does not uninstall
cleanly, and we must manually remove custom resource
definitions.
|
| | |/ /
|/| |
| | |
| | | |
It also prepare for the EE merge request https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14786
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Extend PipelineProcessWorker to accept a list of builds
See merge request gitlab-org/gitlab-ce!31425
|
| | | |/
| |/|
| | |
| | |
| | |
| | |
| | | |
This changes used worker from `BuildProcessWorker`
to `PipelineProcessWorker` to make pipeline
processing much simpler. We process `pipeline_id`,
based on some triggers.
|
| | | | |
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Persistent user callouts now support deferred links, which are links
that can be used to dismiss the callout, and then proceed to follow
the link's original location.
This ensures that the callout dismissal is properly recorded
before the user leaves the page.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Add outbound setting for system hooks
See merge request gitlab-org/gitlab-ce!31177
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
`allow_local_requests_for_hooks_and_services` was renamed to
`allow_local_requests_for_web_hooks_and_services`.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Simplify SystemHookUrlValidator to inherit from PublicUrlValidator
Refactor specs to move out shared examples to be used in both
system hooks and public url validators.
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Updating security/webhooks.md to match new behaviour
as well as drying up few specs to extract shared
examples
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This MR adds new application setting to network section
`allow_local_requests_from_system_hooks`. Prior to this change
system hooks were allowed to do local network requests by default
and we are adding an ability for admins to control it.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Properly process all needs
Closes #65488
See merge request gitlab-org/gitlab-ce!31415
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Currently, some of the jobs with `needs:`
would be processed in stages, it means
that `when:` for such jobs would not be
respected.
This changes the behavior to have a separate
execution paths for jobs with `needs:`.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Group `parent not preloaded` errors in sentry
See merge request gitlab-org/gitlab-ce!31424
|
| | |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
I still haven't found a moment to look into the exact cases that
causes this error, but we're not loosing any information this way.
In fact, it will be easier to find the right information as we only
need to look to one entry in Sentry.
|
| |\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | | |
Backport: Removed pendo poc code
See merge request gitlab-org/gitlab-ce!31307
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Only display the details of the cluster page when the cluster
exists. If it is in "creating" state, show a message and a spinner
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
- Also, include pikaday styles through gitlab-ui
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Also creates specs
Only allow Helm to be uninstalled if it's the only app
- Remove Tiller leftovers after reser command
- Fixes specs and offenses
Adds changelog file
Fix reset_command specs
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Make `needs:` to require a strong reference
Closes #65512
See merge request gitlab-org/gitlab-ce!31419
|
| | | |/ /
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This changes `needs:` from weak reference
to have a strong reference.
This means that job will not be created
unless all needs are present as part of
a pipeline.
|
| |\ \ \ \
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | | |
'master'
Removes update_statistics_namespace feature flag
See merge request gitlab-org/gitlab-ce!31392
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
After measuring the impact of the namespace storage on
https://gitlab.com/gitlab-org/gitlab-ce/issues/64092. It was decided
that it's performant enough. So we can freely remove the feature flag
Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/64092
|
| |\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | | |
Respect needs on artifacts
Closes #65466
See merge request gitlab-org/gitlab-ce!31413
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
When `needs:` is defined, the value of it is
not respected when returning a list of artifacts
to the runner from the job.
|
| | |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
Currently we write out empty CSS classes (`class=""`) every time we
create a new tag. This adds 9 unnecessary bytes per span element. In a
recent trace, I have counted 11950 span elements. So we transported 105
unnecessary kilobytes!
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
Fix performance bar warnings when Rugged is disabled
Closes #65491
See merge request gitlab-org/gitlab-ce!31395
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`detailed_metric.vue` was attempting to read the `duration` field even
when the metric did not exist in the JSON payload. We now check for the
existence of the metrics before attempting to display the value.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65491
|
| | | | |
|
