Thomas Vachuska | 4f1a60c | 2014-10-28 13:39:07 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2014 Open Networking Laboratory |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 16 | package org.onosproject.net.host.impl; |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 17 | |
| 18 | import org.apache.felix.scr.annotations.Activate; |
| 19 | import org.apache.felix.scr.annotations.Component; |
| 20 | import org.apache.felix.scr.annotations.Deactivate; |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 21 | import org.apache.felix.scr.annotations.Reference; |
| 22 | import org.apache.felix.scr.annotations.ReferenceCardinality; |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 23 | import org.apache.felix.scr.annotations.Service; |
Simon Hunt | ff66374 | 2015-05-14 13:33:05 -0700 | [diff] [blame] | 24 | import org.onlab.packet.IpAddress; |
| 25 | import org.onlab.packet.MacAddress; |
| 26 | import org.onlab.packet.VlanId; |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 27 | import org.onosproject.net.provider.AbstractListenerProviderRegistry; |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 28 | import org.onosproject.core.Permission; |
Ray Milkey | a412236 | 2015-08-18 15:19:08 -0700 | [diff] [blame] | 29 | import org.onosproject.net.config.NetworkConfigEvent; |
| 30 | import org.onosproject.net.config.NetworkConfigListener; |
| 31 | import org.onosproject.net.config.NetworkConfigService; |
Thomas Vachuska | 4998caa | 2015-08-26 13:28:38 -0700 | [diff] [blame^] | 32 | import org.onosproject.net.config.basics.BasicHostConfig; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 33 | import org.onosproject.net.ConnectPoint; |
| 34 | import org.onosproject.net.DeviceId; |
| 35 | import org.onosproject.net.Host; |
| 36 | import org.onosproject.net.HostId; |
| 37 | import org.onosproject.net.device.DeviceService; |
| 38 | import org.onosproject.net.host.HostAdminService; |
| 39 | import org.onosproject.net.host.HostDescription; |
| 40 | import org.onosproject.net.host.HostEvent; |
| 41 | import org.onosproject.net.host.HostListener; |
| 42 | import org.onosproject.net.host.HostProvider; |
| 43 | import org.onosproject.net.host.HostProviderRegistry; |
| 44 | import org.onosproject.net.host.HostProviderService; |
| 45 | import org.onosproject.net.host.HostService; |
| 46 | import org.onosproject.net.host.HostStore; |
| 47 | import org.onosproject.net.host.HostStoreDelegate; |
| 48 | import org.onosproject.net.host.PortAddresses; |
| 49 | import org.onosproject.net.packet.PacketService; |
Brian O'Connor | abafb50 | 2014-12-02 22:26:20 -0800 | [diff] [blame] | 50 | import org.onosproject.net.provider.AbstractProviderService; |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 51 | import org.slf4j.Logger; |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 52 | |
Simon Hunt | ff66374 | 2015-05-14 13:33:05 -0700 | [diff] [blame] | 53 | import java.util.Set; |
| 54 | |
| 55 | import static com.google.common.base.Preconditions.checkNotNull; |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 56 | import static com.google.common.base.Preconditions.checkState; |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 57 | import static org.onosproject.security.AppGuard.checkPermission; |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 58 | import static org.slf4j.LoggerFactory.getLogger; |
Simon Hunt | ff66374 | 2015-05-14 13:33:05 -0700 | [diff] [blame] | 59 | |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 60 | /** |
| 61 | * Provides basic implementation of the host SB & NB APIs. |
| 62 | */ |
| 63 | @Component(immediate = true) |
| 64 | @Service |
tom | 202175a | 2014-09-19 19:00:11 -0700 | [diff] [blame] | 65 | public class HostManager |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 66 | extends AbstractListenerProviderRegistry<HostEvent, HostListener, HostProvider, HostProviderService> |
tom | 89b63c5 | 2014-09-16 09:19:51 -0700 | [diff] [blame] | 67 | implements HostService, HostAdminService, HostProviderRegistry { |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 68 | |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 69 | private final Logger log = getLogger(getClass()); |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 70 | |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 71 | public static final String HOST_ID_NULL = "Host ID cannot be null"; |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 72 | |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 73 | private final NetworkConfigListener networkConfigListener = new InternalNetworkConfigListener(); |
| 74 | |
tom | c78acee | 2014-09-24 15:16:55 -0700 | [diff] [blame] | 75 | private HostStoreDelegate delegate = new InternalStoreDelegate(); |
| 76 | |
tom | 5bcc946 | 2014-09-19 10:11:31 -0700 | [diff] [blame] | 77 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 78 | protected HostStore store; |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 79 | |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 80 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
Jonathan Hart | 70da512 | 2014-10-01 16:37:42 -0700 | [diff] [blame] | 81 | protected DeviceService deviceService; |
| 82 | |
| 83 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 84 | protected PacketService packetService; |
| 85 | |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 86 | @Reference(cardinality = ReferenceCardinality.MANDATORY_UNARY) |
| 87 | protected NetworkConfigService networkConfigService; |
| 88 | |
Jonathan Hart | 70da512 | 2014-10-01 16:37:42 -0700 | [diff] [blame] | 89 | private HostMonitor monitor; |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 90 | |
| 91 | @Activate |
| 92 | public void activate() { |
tom | c78acee | 2014-09-24 15:16:55 -0700 | [diff] [blame] | 93 | store.setDelegate(delegate); |
tom | 96dfcab | 2014-08-28 09:26:03 -0700 | [diff] [blame] | 94 | eventDispatcher.addSink(HostEvent.class, listenerRegistry); |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 95 | networkConfigService.addListener(networkConfigListener); |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 96 | monitor = new HostMonitor(deviceService, packetService, this); |
Jonathan Hart | 8f6f1ea | 2014-10-03 16:05:19 -0700 | [diff] [blame] | 97 | monitor.start(); |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 98 | log.info("Started"); |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 99 | } |
| 100 | |
| 101 | @Deactivate |
| 102 | public void deactivate() { |
tom | c78acee | 2014-09-24 15:16:55 -0700 | [diff] [blame] | 103 | store.unsetDelegate(delegate); |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 104 | eventDispatcher.removeSink(HostEvent.class); |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 105 | networkConfigService.removeListener(networkConfigListener); |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 106 | log.info("Stopped"); |
| 107 | } |
| 108 | |
| 109 | @Override |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 110 | protected HostProviderService createProviderService(HostProvider provider) { |
Jonathan Hart | 70da512 | 2014-10-01 16:37:42 -0700 | [diff] [blame] | 111 | monitor.registerHostProvider(provider); |
tom | 5f38b3a | 2014-08-27 23:50:54 -0700 | [diff] [blame] | 112 | return new InternalHostProviderService(provider); |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 113 | } |
| 114 | |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 115 | @Override |
| 116 | public int getHostCount() { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 117 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 118 | return store.getHostCount(); |
| 119 | } |
| 120 | |
| 121 | @Override |
| 122 | public Iterable<Host> getHosts() { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 123 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 124 | return store.getHosts(); |
| 125 | } |
| 126 | |
| 127 | @Override |
| 128 | public Host getHost(HostId hostId) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 129 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 130 | checkNotNull(hostId, HOST_ID_NULL); |
| 131 | return store.getHost(hostId); |
| 132 | } |
| 133 | |
| 134 | @Override |
Ayaka Koshibe | a9c199f | 2014-09-16 16:21:40 -0700 | [diff] [blame] | 135 | public Set<Host> getHostsByVlan(VlanId vlanId) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 136 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 137 | return store.getHosts(vlanId); |
| 138 | } |
| 139 | |
| 140 | @Override |
Ayaka Koshibe | a9c199f | 2014-09-16 16:21:40 -0700 | [diff] [blame] | 141 | public Set<Host> getHostsByMac(MacAddress mac) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 142 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 143 | checkNotNull(mac, "MAC address cannot be null"); |
| 144 | return store.getHosts(mac); |
| 145 | } |
| 146 | |
| 147 | @Override |
Pavlin Radoslavov | 33f228a | 2014-10-27 19:33:16 -0700 | [diff] [blame] | 148 | public Set<Host> getHostsByIp(IpAddress ip) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 149 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 150 | checkNotNull(ip, "IP address cannot be null"); |
| 151 | return store.getHosts(ip); |
| 152 | } |
| 153 | |
| 154 | @Override |
| 155 | public Set<Host> getConnectedHosts(ConnectPoint connectPoint) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 156 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 157 | checkNotNull(connectPoint, "Connection point cannot be null"); |
| 158 | return store.getConnectedHosts(connectPoint); |
| 159 | } |
| 160 | |
| 161 | @Override |
| 162 | public Set<Host> getConnectedHosts(DeviceId deviceId) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 163 | checkPermission(Permission.HOST_READ); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 164 | checkNotNull(deviceId, "Device ID cannot be null"); |
| 165 | return store.getConnectedHosts(deviceId); |
| 166 | } |
| 167 | |
| 168 | @Override |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 169 | public void startMonitoringIp(IpAddress ip) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 170 | checkPermission(Permission.HOST_EVENT); |
Jonathan Hart | 70da512 | 2014-10-01 16:37:42 -0700 | [diff] [blame] | 171 | monitor.addMonitoringFor(ip); |
Jonathan Hart | fca736c | 2014-09-19 17:26:59 -0700 | [diff] [blame] | 172 | } |
| 173 | |
| 174 | @Override |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 175 | public void stopMonitoringIp(IpAddress ip) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 176 | checkPermission(Permission.HOST_EVENT); |
Jonathan Hart | 70da512 | 2014-10-01 16:37:42 -0700 | [diff] [blame] | 177 | monitor.stopMonitoring(ip); |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 178 | } |
| 179 | |
| 180 | @Override |
| 181 | public void requestMac(IpAddress ip) { |
Thomas Vachuska | 42e8cce | 2015-07-29 19:25:18 -0700 | [diff] [blame] | 182 | // FIXME!!!! Auto-generated method stub |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 183 | } |
| 184 | |
tom | 89b63c5 | 2014-09-16 09:19:51 -0700 | [diff] [blame] | 185 | @Override |
| 186 | public void removeHost(HostId hostId) { |
| 187 | checkNotNull(hostId, HOST_ID_NULL); |
| 188 | HostEvent event = store.removeHost(hostId); |
| 189 | if (event != null) { |
tom | 89b63c5 | 2014-09-16 09:19:51 -0700 | [diff] [blame] | 190 | post(event); |
| 191 | } |
| 192 | } |
| 193 | |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 194 | @Override |
Jonathan Hart | 09585c6 | 2014-09-23 16:58:04 -0700 | [diff] [blame] | 195 | public void bindAddressesToPort(PortAddresses addresses) { |
| 196 | store.updateAddressBindings(addresses); |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 197 | } |
| 198 | |
| 199 | @Override |
Jonathan Hart | 09585c6 | 2014-09-23 16:58:04 -0700 | [diff] [blame] | 200 | public void unbindAddressesFromPort(PortAddresses portAddresses) { |
| 201 | store.removeAddressBindings(portAddresses); |
| 202 | } |
| 203 | |
| 204 | @Override |
| 205 | public void clearAddresses(ConnectPoint connectPoint) { |
| 206 | store.clearAddressBindings(connectPoint); |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 207 | } |
| 208 | |
| 209 | @Override |
| 210 | public Set<PortAddresses> getAddressBindings() { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 211 | checkPermission(Permission.HOST_READ); |
Jonathan Hart | 43c182c | 2014-09-23 11:13:42 -0700 | [diff] [blame] | 212 | return store.getAddressBindings(); |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 213 | } |
| 214 | |
| 215 | @Override |
Jonathan Hart | a887ba8 | 2014-11-03 15:20:52 -0800 | [diff] [blame] | 216 | public Set<PortAddresses> getAddressBindingsForPort(ConnectPoint connectPoint) { |
Changhoon Yoon | 541ef71 | 2015-05-23 17:18:34 +0900 | [diff] [blame] | 217 | checkPermission(Permission.HOST_READ); |
Jonathan Hart | 43c182c | 2014-09-23 11:13:42 -0700 | [diff] [blame] | 218 | return store.getAddressBindingsForPort(connectPoint); |
Jonathan Hart | ac60c08 | 2014-09-23 08:55:17 -0700 | [diff] [blame] | 219 | } |
| 220 | |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 221 | // Personalized host provider service issued to the supplied provider. |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 222 | private class InternalHostProviderService |
| 223 | extends AbstractProviderService<HostProvider> |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 224 | implements HostProviderService { |
tom | cfde062 | 2014-09-09 11:02:42 -0700 | [diff] [blame] | 225 | InternalHostProviderService(HostProvider provider) { |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 226 | super(provider); |
| 227 | } |
| 228 | |
| 229 | @Override |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 230 | public void hostDetected(HostId hostId, HostDescription hostDescription) { |
| 231 | checkNotNull(hostId, HOST_ID_NULL); |
| 232 | checkValidity(); |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 233 | hostDescription = validateHost(hostDescription, hostId); |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 234 | HostEvent event = store.createOrUpdateHost(provider().id(), hostId, |
| 235 | hostDescription); |
| 236 | if (event != null) { |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 237 | post(event); |
| 238 | } |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 239 | } |
| 240 | |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 241 | // returns a HostDescription made from the union of the BasicHostConfig |
| 242 | // annotations if it exists |
| 243 | private HostDescription validateHost(HostDescription hostDescription, HostId hostId) { |
| 244 | BasicHostConfig cfg = networkConfigService.getConfig(hostId, BasicHostConfig.class); |
| 245 | checkState(cfg == null || cfg.isAllowed(), "Host {} is not allowed", hostId); |
Ayaka Koshibe | 5373e76 | 2015-08-06 12:31:44 -0700 | [diff] [blame] | 246 | |
| 247 | return BasicHostOperator.combine(cfg, hostDescription); |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 248 | } |
| 249 | |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 250 | @Override |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 251 | public void hostVanished(HostId hostId) { |
| 252 | checkNotNull(hostId, HOST_ID_NULL); |
| 253 | checkValidity(); |
| 254 | HostEvent event = store.removeHost(hostId); |
| 255 | if (event != null) { |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 256 | post(event); |
| 257 | } |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 258 | } |
| 259 | } |
tom | 7869ad9 | 2014-09-09 14:32:08 -0700 | [diff] [blame] | 260 | |
tom | c78acee | 2014-09-24 15:16:55 -0700 | [diff] [blame] | 261 | // Store delegate to re-post events emitted from the store. |
| 262 | private class InternalStoreDelegate implements HostStoreDelegate { |
| 263 | @Override |
| 264 | public void notify(HostEvent event) { |
| 265 | post(event); |
| 266 | } |
| 267 | } |
Sahil Lele | 3a0cdd5 | 2015-07-21 14:16:31 -0700 | [diff] [blame] | 268 | |
| 269 | // listens for NetworkConfigEvents of type BasicHostConfig and removes |
| 270 | // links that the config does not allow |
| 271 | private class InternalNetworkConfigListener implements NetworkConfigListener { |
| 272 | @Override |
| 273 | public void event(NetworkConfigEvent event) { |
| 274 | if ((event.type() == NetworkConfigEvent.Type.CONFIG_ADDED || |
| 275 | event.type() == NetworkConfigEvent.Type.CONFIG_UPDATED) && |
| 276 | event.configClass().equals(BasicHostConfig.class)) { |
| 277 | log.info("Detected Host network config event {}", event.type()); |
| 278 | kickOutBadHost(((HostId) event.subject())); |
| 279 | } |
| 280 | } |
| 281 | } |
| 282 | |
| 283 | // checks if the specified host is allowed by the BasicHostConfig |
| 284 | // and if not, removes it |
| 285 | private void kickOutBadHost(HostId hostId) { |
| 286 | BasicHostConfig cfg = networkConfigService.getConfig(hostId, BasicHostConfig.class); |
| 287 | if (cfg != null && !cfg.isAllowed()) { |
| 288 | Host badHost = getHost(hostId); |
| 289 | if (badHost != null) { |
| 290 | removeHost(hostId); |
| 291 | } else { |
| 292 | log.info("Failed removal: Host {} does not exist", hostId); |
| 293 | } |
| 294 | } |
| 295 | } |
tom | db0d03f | 2014-08-27 16:34:15 -0700 | [diff] [blame] | 296 | } |