Skip to content

Commit af50ffb

Browse files
Merge branch 'support-tls-websocket-omnibus' into 'master'
Remove the outdated warning message for websocket TLS termination Closes gitlab-org/cluster-integration/gitlab-agent#220 See merge request https://gitlab.com/gitlab-org/omnibus-gitlab/-/merge_requests/7181 Merged-by: Balasankar 'Balu' C <[email protected]> Approved-by: Balasankar 'Balu' C <[email protected]> Approved-by: Dustin Collins <[email protected]> Approved-by: Mikhail Mazurskiy <[email protected]> Approved-by: Hordur Freyr Yngvason <[email protected]> Co-authored-by: Harshit Prasad <[email protected]>
2 parents 0779e2d + 7427706 commit af50ffb

File tree

2 files changed

+0
-29
lines changed

2 files changed

+0
-29
lines changed

files/gitlab-cookbooks/gitlab-kas/recipes/enable.rb

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -65,17 +65,6 @@
6565
recursive true
6666
end
6767

68-
ruby_block 'websocket TLS termination' do
69-
block do
70-
message = [
71-
"Enabling gitlab-kas API TLS termination and websocket tunnelling at the same time is not supported.",
72-
"See <https://gitlab.com/gitlab-org/cluster-integration/gitlab-agent/-/issues/217>"
73-
]
74-
LoggingHelper.warning(message.join("\n\n"))
75-
end
76-
only_if { node['gitlab_kas']['listen_websocket'] && node['gitlab_kas']['certificate_file'] && node['gitlab_kas']['key_file'] }
77-
end
78-
7968
version_file 'Create version file for Gitlab KAS' do
8069
version_file_path File.join(working_dir, 'VERSION')
8170
version_check_cmd '/opt/gitlab/embedded/bin/gitlab-kas --version'

spec/chef/cookbooks/gitlab-kas/recipes/gitlab-kas_spec.rb

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -612,24 +612,6 @@
612612
end
613613
end
614614

615-
context 'when the certificate/key bundle is defined and websocket tunneling is enabled' do
616-
before do
617-
stub_gitlab_rb(
618-
external_url: 'https://gitlab.example.com',
619-
gitlab_kas: {
620-
enable: true,
621-
listen_websocket: true,
622-
certificate_file: '/path/to/cert.pem',
623-
key_file: '/path/to/key.pem',
624-
}
625-
)
626-
end
627-
628-
it 'logs a warning' do
629-
expect(chef_run).to run_ruby_block('websocket TLS termination')
630-
end
631-
end
632-
633615
context 'log directory and runit group' do
634616
context 'default values' do
635617
it_behaves_like 'enabled logged service', 'gitlab-kas', true, { log_directory_owner: 'git' }

0 commit comments

Comments
 (0)