diff options
-rw-r--r-- | .gitlab-ci.yml | 62 | ||||
-rw-r--r-- | app/assets/stylesheets/generic/header.scss | 19 | ||||
-rw-r--r-- | app/assets/stylesheets/generic/mobile.scss | 3 | ||||
-rw-r--r-- | app/views/doorkeeper/applications/_form.html.haml | 30 | ||||
-rw-r--r-- | app/views/doorkeeper/applications/new.html.haml | 7 | ||||
-rw-r--r-- | app/views/layouts/header/_default.html.haml | 5 | ||||
-rw-r--r-- | app/views/layouts/header/_public.html.haml | 4 | ||||
-rw-r--r-- | app/views/layouts/nav/_profile.html.haml | 2 | ||||
-rw-r--r-- | config/initializers/7_omniauth.rb | 8 | ||||
-rw-r--r-- | doc/development/db_dump.md | 5 | ||||
-rw-r--r-- | doc/release/monthly.md | 4 | ||||
-rw-r--r-- | features/steps/profile/profile.rb | 2 |
12 files changed, 91 insertions, 60 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 1411a9194b5..8fba268be7d 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -1,3 +1,4 @@ +# This file is generated by GitLab CI before_script: - export PATH=$HOME/bin:/usr/local/bin:/usr/bin:/bin - ruby -v @@ -7,33 +8,44 @@ before_script: - echo $PATH - cp config/database.yml.mysql config/database.yml - cp config/gitlab.yml.example config/gitlab.yml - - ! 'sed "s/username\:.*$/username\: runner/" -i config/database.yml' - - ! 'sed "s/password\:.*$/password\: ''password''/" -i config/database.yml' + - 'sed "s/username\:.*$/username\: runner/" -i config/database.yml' + - 'sed "s/password\:.*$/password\: ''password''/" -i config/database.yml' - sed "s/gitlabhq_test/gitlabhq_test_$((RANDOM/5000))/" -i config/database.yml - touch log/application.log - touch log/test.log - bundle install --without postgres production --jobs $(nproc) - bundle exec rake db:create RAILS_ENV=test -jobs: -- script: - - RAILS_ENV=test SIMPLECOV=true bundle exec rake spec - name: Rspec - runner: ruby,mysql -- script: - - RAILS_ENV=test SIMPLECOV=true bundle exec rake spinach - name: Spinach - runner: ruby,mysql -- script: - - RAILS_ENV=test SIMPLECOV=true bundle exec rake jasmine:ci - name: Jasmine - runner: ruby,mysql -- script: - - bundle exec rubocop - name: Rubocop - runner: ruby,mysql -- script: - - bundle exec rake brakeman - name: Brakeman - runner: ruby,mysql -deploy_jobs: [] -skip_refs: '' +Rspec: + script: + - RAILS_ENV=test SIMPLECOV=true bundle exec rake spec + tags: + - ruby + - mysql + +Spinach: + script: + - RAILS_ENV=test SIMPLECOV=true bundle exec rake spinach + tags: + - ruby + - mysql + +Jasmine: + script: + - RAILS_ENV=test SIMPLECOV=true bundle exec rake jasmine:ci + tags: + - ruby + - mysql + +Rubocop: + script: + - bundle exec rubocop + tags: + - ruby + - mysql + +Brakeman: + script: + - bundle exec rake brakeman + tags: + - ruby + - mysql
\ No newline at end of file diff --git a/app/assets/stylesheets/generic/header.scss b/app/assets/stylesheets/generic/header.scss index 5e8701830e7..8f17232592e 100644 --- a/app/assets/stylesheets/generic/header.scss +++ b/app/assets/stylesheets/generic/header.scss @@ -98,16 +98,16 @@ header { height: $header-height; .title { - position: relative; - float: left; margin: 0; - margin-left: 35px; + padding: 0 15px 0 35px; + overflow: hidden; font-size: 18px; line-height: $header-height; font-weight: bold; color: #444; - - @include str-truncated(37%); + text-overflow: ellipsis; + vertical-align: top; + white-space: nowrap; a { color: #444; @@ -116,6 +116,10 @@ header { } } } + + .navbar-collapse { + float: right; + } } .search { @@ -167,10 +171,6 @@ header { } @media (max-width: $screen-md-max) { - header .container .title { - max-width: 43%; - } - .header-collapsed, .header-expanded { @include collapsed-header; } @@ -190,7 +190,6 @@ header { font-size: 18px; .title { - max-width: 70%; } .navbar-nav { diff --git a/app/assets/stylesheets/generic/mobile.scss b/app/assets/stylesheets/generic/mobile.scss index f04c8eef904..a49775daf8b 100644 --- a/app/assets/stylesheets/generic/mobile.scss +++ b/app/assets/stylesheets/generic/mobile.scss @@ -61,8 +61,7 @@ } .container .title { - margin-left: 15px !important; - max-width: 70% !important; + padding-left: 15px !important; } .issue-info, .merge-request-info { diff --git a/app/views/doorkeeper/applications/_form.html.haml b/app/views/doorkeeper/applications/_form.html.haml index a5fec2fabdb..98a61ab211b 100644 --- a/app/views/doorkeeper/applications/_form.html.haml +++ b/app/views/doorkeeper/applications/_form.html.haml @@ -1,17 +1,22 @@ = form_for application, url: doorkeeper_submit_path(application), html: {class: 'form-horizontal', role: 'form'} do |f| - if application.errors.any? - .alert.alert-danger{"data-alert" => ""} - %p Whoops! Check your form for possible errors - = content_tag :div, class: "form-group#{' has-error' if application.errors[:name].present?}" do - = f.label :name, class: 'col-sm-2 control-label' + .alert.alert-danger + %ul + - application.errors.full_messages.each do |msg| + %li= msg + + .form-group + = f.label :name, class: 'control-label' + .col-sm-10 - = f.text_field :name, class: 'form-control' - = doorkeeper_errors_for application, :name - = content_tag :div, class: "form-group#{' has-error' if application.errors[:redirect_uri].present?}" do - = f.label :redirect_uri, class: 'col-sm-2 control-label' + = f.text_field :name, class: 'form-control', required: true + + .form-group + = f.label :redirect_uri, class: 'control-label' + .col-sm-10 - = f.text_area :redirect_uri, class: 'form-control' - = doorkeeper_errors_for application, :redirect_uri + = f.text_area :redirect_uri, class: 'form-control', required: true + %span.help-block Use one line per URI - if Doorkeeper.configuration.native_redirect_uri @@ -19,6 +24,7 @@ Use %code= Doorkeeper.configuration.native_redirect_uri for local tests + .form-actions - = f.submit 'Submit', class: "btn btn-primary wide" - = link_to "Cancel", applications_profile_path, class: "btn btn-default" + = f.submit 'Submit', class: "btn btn-create" + = link_to "Cancel", applications_profile_path, class: "btn btn-cancel" diff --git a/app/views/doorkeeper/applications/new.html.haml b/app/views/doorkeeper/applications/new.html.haml index 655845e4af5..fd32a468b45 100644 --- a/app/views/doorkeeper/applications/new.html.haml +++ b/app/views/doorkeeper/applications/new.html.haml @@ -1,2 +1,7 @@ -%h3.page-title New application +- page_title "New Application" + +%h3.page-title New Application + +%hr + = render 'form', application: @application
\ No newline at end of file diff --git a/app/views/layouts/header/_default.html.haml b/app/views/layouts/header/_default.html.haml index 8b4510d6516..4ec50f35890 100644 --- a/app/views/layouts/header/_default.html.haml +++ b/app/views/layouts/header/_default.html.haml @@ -5,9 +5,6 @@ = brand_header_logo %h3 GitLab .header-content - %h1.title - = title - %button.navbar-toggle %span.sr-only Toggle navigation = icon('bars') @@ -43,4 +40,6 @@ = link_to destroy_user_session_path, class: 'logout', method: :delete, title: 'Sign out', data: {toggle: 'tooltip', placement: 'bottom'} do = icon('sign-out') + %h1.title= title + = render 'shared/outdated_browser' diff --git a/app/views/layouts/header/_public.html.haml b/app/views/layouts/header/_public.html.haml index 6a031722aaa..2c5884a5b6d 100644 --- a/app/views/layouts/header/_public.html.haml +++ b/app/views/layouts/header/_public.html.haml @@ -5,10 +5,10 @@ = brand_header_logo %h3 GitLab .header-content - %h1.title= title - - unless current_controller?('sessions') .pull-right = link_to "Sign in", new_session_path(:user, redirect_to_referer: 'yes'), class: 'btn btn-sign-in btn-success btn-sm' + %h1.title= title + = render 'shared/outdated_browser' diff --git a/app/views/layouts/nav/_profile.html.haml b/app/views/layouts/nav/_profile.html.haml index 121665bd536..914e1b83d1f 100644 --- a/app/views/layouts/nav/_profile.html.haml +++ b/app/views/layouts/nav/_profile.html.haml @@ -9,7 +9,7 @@ = icon('gear fw') %span Account - = nav_link(path: ['profiles#applications', 'applications#edit', 'applications#show', 'applications#new']) do + = nav_link(path: ['profiles#applications', 'applications#edit', 'applications#show', 'applications#new', 'applications#create']) do = link_to applications_profile_path, title: 'Applications', data: {placement: 'right'} do = icon('cloud fw') %span diff --git a/config/initializers/7_omniauth.rb b/config/initializers/7_omniauth.rb index 6f1f267bf97..df73ec1304a 100644 --- a/config/initializers/7_omniauth.rb +++ b/config/initializers/7_omniauth.rb @@ -17,3 +17,11 @@ OmniAuth.config.allowed_request_methods << :get if Gitlab.config.omniauth.auto_s OmniAuth.config.before_request_phase do |env| OmniAuth::RequestForgeryProtection.new(env).call end + +if Gitlab.config.omniauth.enabled + Gitlab.config.omniauth.providers.each do |provider| + if provider['name'] == 'kerberos' + require 'omniauth-kerberos' + end + end +end diff --git a/doc/development/db_dump.md b/doc/development/db_dump.md index 4ad3bd534e0..21f1b3edecd 100644 --- a/doc/development/db_dump.md +++ b/doc/development/db_dump.md @@ -4,6 +4,9 @@ Sometimes it is useful to import the database from a production environment into a staging environment for testing. The procedure below assumes you have SSH+sudo access to both the production environment and the staging VM. +**Destroy your staging VM** when you are done with it. It is important to avoid +data leaks. + On the staging VM, add the following line to `/etc/gitlab/gitlab.rb` to speed up large database imports. @@ -12,6 +15,8 @@ large database imports. echo "postgresql['checkpoint_segments'] = 64" | sudo tee -a /etc/gitlab/gitlab.rb sudo touch /etc/gitlab/skip-auto-migrations sudo gitlab-ctl reconfigure +sudo gitlab-ctl stop unicorn +sudo gitlab-ctl stop sidekiq ``` Next, we let the production environment stream a compressed SQL dump to our diff --git a/doc/release/monthly.md b/doc/release/monthly.md index 8d475296a22..d50258f2796 100644 --- a/doc/release/monthly.md +++ b/doc/release/monthly.md @@ -30,9 +30,6 @@ All steps from issue template are explained below ``` Xth: (7 working days before the 22nd) -- [ ] Update the CE changelog (#LINK) -- [ ] Update the EE changelog (#LINK) -- [ ] Update the CI changelog (#LINK) - [ ] Triage the omnibus-gitlab milestone Xth: (6 working days before the 22nd) @@ -41,6 +38,7 @@ Xth: (6 working days before the 22nd) - [ ] Determine QA person and notify this person - [ ] Check the tasks in [how to rc1 guide](https://dev.gitlab.org/gitlab/gitlabhq/blob/master/doc/release/howto_rc1.md) and delegate tasks if necessary - [ ] Create CE, EE, CI RC1 versions (#LINK) +- [ ] Build RC1 packages (EE first) (#LINK) Xth: (5 working days before the 22nd) diff --git a/features/steps/profile/profile.rb b/features/steps/profile/profile.rb index 649aea8e3f4..10bd307320e 100644 --- a/features/steps/profile/profile.rb +++ b/features/steps/profile/profile.rb @@ -176,7 +176,7 @@ class Spinach::Features::Profile < Spinach::FeatureSteps end step 'I should see application form' do - expect(page).to have_content "New application" + expect(page).to have_content "New Application" end step 'I fill application form out and submit' do |