Age | Commit message (Collapse) | Author |
|
|
|
|
|
Instead of from the configuration file
|
|
Add external link for online artifacts
Closes #34102
See merge request gitlab-org/gitlab-ce!14399
|
|
|
|
|
|
|
|
|
|
Removed the max_size and timeout options from the git config in
gitlab.yml.
|
|
This reverts merge request !14148
|
|
'35012-navigation-add-option-to-change-navigation-color-palette' into 'master'"
This reverts merge request !13619
|
|
|
|
|
|
|
|
|
|
|
|
There are peaks of IO-wait time, most of them stay below 30 seconds.
|
|
|
|
`failure_count_threshold`: We should never need this, but we don't
want to block access in tests because of this.
`failure_wait_time`: Setting it to 0 now allows each storage attempt
`storage_timeout`: Try a bit longer to access storage on CI in case
the slow machines take a bit longer to spin up the process to perfom
the check
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Added authentiq provider to tests and updated documentation, gem and config file.
See merge request !12952
|
|
file.
|
|
Remove the need to use health check token by adding ability to whitelist hosts
Closes #33949
See merge request !12612
|
|
Test logging in via the Authentiq OAuth provider in a feature spec
Closes #34737
See merge request !12694
|
|
Remove option to disable Gitaly in Rails
See merge request !12677
|
|
|
|
- The `migration:path-pg` build was previously failing when the Authentiq
feature spec was enabled by placing Authentiq configuration in the `test`
section of `gitlab.yml`
- The `migration:path-pg` task checks out an old revision of the
codebase (`v8.14.10`) and runs a `schema:load`. It then checks out the commit
under test, and runs `db:migrate`, to verify that migrations run without
errors.
- The problem here is that `v8.14.10` does not have the Authentiq module
installed, but is run with the `gitlab.yml` for `master`, which would contain
the `Authentiq` configuration in the `test` section.
- The solution was to use the `v8.14.10` `gitlab.yml` for the `schema:load`,
rather than the `gitlab.yml` from master.
|
|
Fix `migration:path` build for "Honor the 'Remember me' parameter for OAuth-based login"
Closes #34713
See merge request !12668
|
|
- This is causing autoload-related errors in the `migration:path` builds. We
need to find a better way of testing this provider.
|
|
- Don't use `request.env['omniauth.params']` if it isn't present.
- Remove the `saml` section from the `gitlab.yml` test section. Some tests
depend on this section not being initially present, so it can be overridden
in the test. This MR doesn't add any tests for SAML, so we didn't really need
this in the first place anyway.
- Clean up the test -> omniauth section of `gitlab.yml`
|
|
- I tried to get this to work by stubbing out portions of the config within the
test. This didn't work as expected because Devise/Omniauth loaded before the
stub could run, and the stubbed config was ignored.
- I attempted to fix this by reloading Devise/Omniauth after stubbing the
config. This successfully got Devise to load the stubbed providers, but failed
while trying to access a route such as `user_gitlab_omniauth_authorize_path`.
- I spent a while trying to figure this out (even trying
`Rails.application.reload_routes!`), but nothing seemed to work.
- I settled for adding this config directly to `gitlab.yml` rather than go down
this path any further.
|
|
This reverts merge request !11963
|
|
Honor the "Remember me" parameter for OAuth-based login
Closes #18000
See merge request !11963
|
|
|
|
as deprecated
|
|
in favor of whitelist that will be used to
control the access to monitoring resources
|
|
|
|
- Don't use `request.env['omniauth.params']` if it isn't present.
- Remove the `saml` section from the `gitlab.yml` test section. Some tests
depend on this section not being initially present, so it can be overridden
in the test. This MR doesn't add any tests for SAML, so we didn't really need
this in the first place anyway.
- Clean up the test -> omniauth section of `gitlab.yml`
|
|
- I tried to get this to work by stubbing out portions of the config within the
test. This didn't work as expected because Devise/Omniauth loaded before the
stub could run, and the stubbed config was ignored.
- I attempted to fix this by reloading Devise/Omniauth after stubbing the
config. This successfully got Devise to load the stubbed providers, but failed
while trying to access a route such as `user_gitlab_omniauth_authorize_path`.
- I spent a while trying to figure this out (even trying
`Rails.application.reload_routes!`), but nothing seemed to work.
- I settled for adding this config directly to `gitlab.yml` rather than go down
this path any further.
|
|
|
|
|
|
|
|
|
|
Add username parameter to gravatar URL
Closes #24472
See merge request !11741
|