WIP: Haproxy as part of HA EE packages
1 unresolved thread
1 unresolved thread
Merge request reports
Activity
@twk3 Can you please review and see if this MR makes sense before I continue on it?
- Resolved by Marin Jankovski
- Resolved by Marin Jankovski
- spec/chef/recipes/gitlab-ee/haproxy_spec.rb 0 → 100644
157 158 context 'with invalid user configuration' do 159 before do 160 stub_gitlab_rb( 161 haproxy: { 162 enable: true, 163 global: { home: '/tmp/user' }, 164 defaults: { timeout: ['connect 5000'] }, 165 listen: { stats_7331: { bind: ['0.0.0.0:7331'], stats: ["enable", "hide-version"] } } 166 } 167 ) 168 end 169 170 it 'raises an error' do 171 # TODO This is not testable because we are doing it wrong! 172 # We are raising an error from within the template which will give Chef::Mixin::Template::TemplateError - Resolved by Marin Jankovski
Reassigned to @marin
Problem that caused the discussion starting with https://gitlab.com/gitlab-org/omnibus-gitlab/merge_requests/1092#note_18166942 is really annoying me and I want this to be tackled better. Given that the release is around the corner with some other features more important to get merged I won't rush to get this in 8.14.
Milestone changed to %8.15
changed milestone to %Backlog
mentioned in merge request !1251 (closed)
mentioned in issue #1807 (closed)
Please register or sign in to reply