X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/420d88e40bc5c2d12fe53a5f9f25ec19d80de797..88b0204247e448ba5f715cb01adba4597bb6aae7:/services/nodemanager/tests/test_computenode_dispatch.py diff --git a/services/nodemanager/tests/test_computenode_dispatch.py b/services/nodemanager/tests/test_computenode_dispatch.py index a1dfde30e1..96a70c6c96 100644 --- a/services/nodemanager/tests/test_computenode_dispatch.py +++ b/services/nodemanager/tests/test_computenode_dispatch.py @@ -79,14 +79,16 @@ class ComputeNodeSetupActorTestCase(testutil.ActorTestMixin, unittest.TestCase): self.make_mocks( arverror.ApiError(httplib2.Response({'status': '500'}), "")) self.make_actor() - self.setup_actor.stop_if_no_cloud_node() + self.assertTrue( + self.setup_actor.stop_if_no_cloud_node().get(self.TIMEOUT)) self.assertTrue( self.setup_actor.actor_ref.actor_stopped.wait(self.TIMEOUT)) def test_no_stop_when_cloud_node(self): self.make_actor() self.wait_for_assignment(self.setup_actor, 'cloud_node') - self.setup_actor.stop_if_no_cloud_node().get(self.TIMEOUT) + self.assertFalse( + self.setup_actor.stop_if_no_cloud_node().get(self.TIMEOUT)) self.assertTrue(self.stop_proxy(self.setup_actor), "actor was stopped by stop_if_no_cloud_node") @@ -129,8 +131,9 @@ class ComputeNodeShutdownActorMixin(testutil.ActorTestMixin): if not hasattr(self, 'timer'): self.make_mocks() monitor_actor = dispatch.ComputeNodeMonitorActor.start( - self.cloud_node, time.time(), self.shutdowns, self.timer, - self.updates, self.arvados_node) + self.cloud_node, time.time(), self.shutdowns, + testutil.cloud_node_fqdn, self.timer, self.updates, + self.arvados_node) self.shutdown_actor = self.ACTOR_CLASS.start( self.timer, self.cloud_client, monitor_actor, cancellable).proxy() self.monitor_actor = monitor_actor.proxy() @@ -218,8 +221,9 @@ class ComputeNodeMonitorActorTestCase(testutil.ActorTestMixin, if start_time is None: start_time = time.time() self.node_actor = dispatch.ComputeNodeMonitorActor.start( - self.cloud_mock, start_time, self.shutdowns, self.timer, - self.updates, arv_node).proxy() + self.cloud_mock, start_time, self.shutdowns, + testutil.cloud_node_fqdn, self.timer, self.updates, + arv_node).proxy() self.node_actor.subscribe(self.subscriber).get(self.TIMEOUT) def node_state(self, *states): @@ -227,28 +231,29 @@ class ComputeNodeMonitorActorTestCase(testutil.ActorTestMixin, def test_in_state_when_unpaired(self): self.make_actor() - self.assertIsNone(self.node_state('idle', 'alloc')) + self.assertIsNone(self.node_state('idle', 'busy')) def test_in_state_when_pairing_stale(self): self.make_actor(arv_node=testutil.arvados_node_mock( job_uuid=None, age=90000)) - self.assertIsNone(self.node_state('idle', 'alloc')) + self.assertIsNone(self.node_state('idle', 'busy')) def test_in_state_when_no_state_available(self): - self.make_actor(arv_node=testutil.arvados_node_mock(info={})) - self.assertIsNone(self.node_state('idle', 'alloc')) + self.make_actor(arv_node=testutil.arvados_node_mock( + crunch_worker_state=None)) + self.assertIsNone(self.node_state('idle', 'busy')) def test_in_idle_state(self): self.make_actor(2, arv_node=testutil.arvados_node_mock(job_uuid=None)) self.assertTrue(self.node_state('idle')) - self.assertFalse(self.node_state('alloc')) - self.assertTrue(self.node_state('idle', 'alloc')) + self.assertFalse(self.node_state('busy')) + self.assertTrue(self.node_state('idle', 'busy')) - def test_in_alloc_state(self): + def test_in_busy_state(self): self.make_actor(3, arv_node=testutil.arvados_node_mock(job_uuid=True)) self.assertFalse(self.node_state('idle')) - self.assertTrue(self.node_state('alloc')) - self.assertTrue(self.node_state('idle', 'alloc')) + self.assertTrue(self.node_state('busy')) + self.assertTrue(self.node_state('idle', 'busy')) def test_init_shutdown_scheduling(self): self.make_actor() @@ -293,7 +298,8 @@ class ComputeNodeMonitorActorTestCase(testutil.ActorTestMixin, self.assertFalse(self.node_actor.shutdown_eligible().get(self.TIMEOUT)) def test_no_shutdown_when_node_state_unknown(self): - self.make_actor(5, testutil.arvados_node_mock(5, info={})) + self.make_actor(5, testutil.arvados_node_mock( + 5, crunch_worker_state=None)) self.shutdowns._set_state(True, 600) self.assertFalse(self.node_actor.shutdown_eligible().get(self.TIMEOUT)) @@ -317,6 +323,13 @@ class ComputeNodeMonitorActorTestCase(testutil.ActorTestMixin, self.assertIsNone( self.node_actor.offer_arvados_pair(arv_node).get(self.TIMEOUT)) + def test_arvados_node_mismatch_first_ping_too_early(self): + self.make_actor(4) + arv_node = testutil.arvados_node_mock( + 4, first_ping_at='1971-03-02T14:15:16.1717282Z') + self.assertIsNone( + self.node_actor.offer_arvados_pair(arv_node).get(self.TIMEOUT)) + def test_update_cloud_node(self): self.make_actor(1) self.make_mocks(2) @@ -347,3 +360,20 @@ class ComputeNodeMonitorActorTestCase(testutil.ActorTestMixin, current_arvados = self.node_actor.arvados_node.get(self.TIMEOUT) self.assertEqual(testutil.ip_address_mock(4), current_arvados['ip_address']) + + def test_update_arvados_node_syncs_when_fqdn_mismatch(self): + self.make_mocks(5) + self.cloud_mock.extra['testname'] = 'cloudfqdn.zzzzz.arvadosapi.com' + self.make_actor() + arv_node = testutil.arvados_node_mock(5) + self.node_actor.update_arvados_node(arv_node).get(self.TIMEOUT) + self.assertEqual(1, self.updates.sync_node.call_count) + + def test_update_arvados_node_skips_sync_when_fqdn_match(self): + self.make_mocks(6) + arv_node = testutil.arvados_node_mock(6) + self.cloud_mock.extra['testname'] ='{n[hostname]}.{n[domain]}'.format( + n=arv_node) + self.make_actor() + self.node_actor.update_arvados_node(arv_node).get(self.TIMEOUT) + self.assertEqual(0, self.updates.sync_node.call_count)