summaryrefslogtreecommitdiff
path: root/spec/services
diff options
context:
space:
mode:
authorDylan Griffith <dyl.griffith@gmail.com>2018-02-12 14:22:15 +1100
committerDylan Griffith <dyl.griffith@gmail.com>2018-02-15 17:01:11 +1100
commitc1828eaed56159998d1eaafdaa135f1b3480549b (patch)
tree8cd903c910454bfdbc2bfdb1b1e9709eaa5583f1 /spec/services
parent5ca692b0b04b4f349fb5a08b9dcc7d87c774934e (diff)
downloadgitlab-ce-c1828eaed56159998d1eaafdaa135f1b3480549b.tar.gz
Persist external IP of ingress controller created for GKE (#42643)
Diffstat (limited to 'spec/services')
-rw-r--r--spec/services/clusters/applications/check_ingress_ip_address_service_spec.rb81
1 files changed, 81 insertions, 0 deletions
diff --git a/spec/services/clusters/applications/check_ingress_ip_address_service_spec.rb b/spec/services/clusters/applications/check_ingress_ip_address_service_spec.rb
new file mode 100644
index 00000000000..5d99196fbe6
--- /dev/null
+++ b/spec/services/clusters/applications/check_ingress_ip_address_service_spec.rb
@@ -0,0 +1,81 @@
+require 'spec_helper'
+
+describe Clusters::Applications::CheckIngressIpAddressService do
+ let(:application) { create(:clusters_applications_ingress, :installed) }
+ let(:service) { described_class.new(application) }
+ let(:kube_service) do
+ ::Kubeclient::Resource.new(
+ {
+ status: {
+ loadBalancer: {
+ ingress: ingress
+ }
+ }
+ }
+ )
+ end
+ let(:kubeclient) { double(::Kubeclient::Client, get_service: kube_service) }
+ let(:ingress) { [{ ip: '111.222.111.222' }] }
+
+ before do
+ allow(application.cluster).to receive(:kubeclient).and_return(kubeclient)
+ end
+
+ describe '#execute' do
+ context 'when the ingress ip address is available' do
+ it 'updates the external_ip for the app and does not schedule another worker' do
+ expect(ClusterWaitForIngressIpAddressWorker).not_to receive(:perform_in)
+
+ service.execute(1)
+
+ expect(application.external_ip).to eq('111.222.111.222')
+ end
+ end
+
+ context 'when the ingress ip address is not available' do
+ let(:ingress) { nil }
+
+ it 'it schedules another worker with 1 less retry' do
+ expect(ClusterWaitForIngressIpAddressWorker)
+ .to receive(:perform_in)
+ .with(ClusterWaitForIngressIpAddressWorker::INTERVAL, 'ingress', application.id, 0)
+
+ service.execute(1)
+ end
+
+ context 'when no more retries remaining' do
+ it 'does not schedule another worker' do
+ expect(ClusterWaitForIngressIpAddressWorker).not_to receive(:perform_in)
+
+ service.execute(0)
+ end
+ end
+ end
+
+ context 'when there is already an external_ip' do
+ let(:application) { create(:clusters_applications_ingress, :installed, external_ip: '001.111.002.111') }
+
+ it 'does nothing' do
+ expect(kubeclient).not_to receive(:get_service)
+
+ service.execute(1)
+
+ expect(application.external_ip).to eq('001.111.002.111')
+ end
+ end
+
+ context 'when a kubernetes error occurs' do
+ before do
+ allow(kubeclient).to receive(:get_service).and_raise(KubeException.new(500, 'something blew up', nil))
+ end
+
+ it 'it schedules another worker with 1 less retry' do
+ expect(ClusterWaitForIngressIpAddressWorker)
+ .to receive(:perform_in)
+ .with(ClusterWaitForIngressIpAddressWorker::INTERVAL, 'ingress', application.id, 0)
+
+ service.execute(1)
+ end
+ end
+ end
+end