X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=restconf%2Frestconf-nb-rfc8040%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Frestconf%2Fnb%2Frfc8040%2Frests%2Fservices%2Fimpl%2FRestconfDataServiceImpl.java;h=1dd7ac400bbfd866b6408cc3b20e2bfab3fccbd3;hb=acd7e4834a74b40ed622411ee7fb2283eb305f14;hp=389643b5e77f7c42d7d0c677abb88f3069e96d6f;hpb=442aeca4229ff2c66889c9c44cf86874988afa5c;p=netconf.git diff --git a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImpl.java b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImpl.java index 389643b5e7..1dd7ac400b 100644 --- a/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImpl.java +++ b/restconf/restconf-nb-rfc8040/src/main/java/org/opendaylight/restconf/nb/rfc8040/rests/services/impl/RestconfDataServiceImpl.java @@ -11,25 +11,30 @@ import static java.util.Objects.requireNonNull; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfDataServiceConstant.PostPutQueryParameters.INSERT; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfDataServiceConstant.PostPutQueryParameters.POINT; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.NOTIFICATION_STREAM; +import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.STREAMS_PATH; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.STREAM_ACCESS_PATH_PART; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.STREAM_LOCATION_PATH_PART; import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.STREAM_PATH; +import static org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfStreamsConstants.STREAM_PATH_PART; +import com.google.common.collect.ImmutableList; +import java.net.URI; import java.time.Clock; import java.time.LocalDateTime; import java.time.format.DateTimeFormatter; import java.util.List; import java.util.Map.Entry; import java.util.Optional; +import java.util.concurrent.ExecutionException; import javax.ws.rs.Path; import javax.ws.rs.WebApplicationException; import javax.ws.rs.core.Response; import javax.ws.rs.core.UriInfo; -import org.eclipse.jdt.annotation.NonNull; import org.eclipse.jdt.annotation.Nullable; +import org.opendaylight.mdsal.common.api.LogicalDatastoreType; import org.opendaylight.mdsal.dom.api.DOMActionResult; -import org.opendaylight.mdsal.dom.api.DOMDataBroker; import org.opendaylight.mdsal.dom.api.DOMMountPoint; +import org.opendaylight.mdsal.dom.api.DOMSchemaService; import org.opendaylight.restconf.common.context.InstanceIdentifierContext; import org.opendaylight.restconf.common.context.NormalizedNodeContext; import org.opendaylight.restconf.common.context.WriterParameters; @@ -39,14 +44,16 @@ import org.opendaylight.restconf.common.errors.RestconfError.ErrorTag; import org.opendaylight.restconf.common.errors.RestconfError.ErrorType; import org.opendaylight.restconf.common.patch.PatchContext; import org.opendaylight.restconf.common.patch.PatchStatusContext; +import org.opendaylight.restconf.nb.rfc8040.Rfc8040; import org.opendaylight.restconf.nb.rfc8040.handlers.ActionServiceHandler; import org.opendaylight.restconf.nb.rfc8040.handlers.DOMMountPointServiceHandler; import org.opendaylight.restconf.nb.rfc8040.handlers.SchemaContextHandler; import org.opendaylight.restconf.nb.rfc8040.handlers.TransactionChainHandler; -import org.opendaylight.restconf.nb.rfc8040.references.SchemaContextRef; import org.opendaylight.restconf.nb.rfc8040.rests.services.api.RestconfDataService; import org.opendaylight.restconf.nb.rfc8040.rests.services.api.RestconfStreamsSubscriptionService; -import org.opendaylight.restconf.nb.rfc8040.rests.transactions.TransactionVarsWrapper; +import org.opendaylight.restconf.nb.rfc8040.rests.transactions.MdsalRestconfStrategy; +import org.opendaylight.restconf.nb.rfc8040.rests.transactions.RestconfStrategy; +import org.opendaylight.restconf.nb.rfc8040.rests.transactions.RestconfTransaction; import org.opendaylight.restconf.nb.rfc8040.rests.utils.DeleteDataTransactionUtil; import org.opendaylight.restconf.nb.rfc8040.rests.utils.PatchDataTransactionUtil; import org.opendaylight.restconf.nb.rfc8040.rests.utils.PlainPatchDataTransactionUtil; @@ -54,18 +61,26 @@ import org.opendaylight.restconf.nb.rfc8040.rests.utils.PostDataTransactionUtil; import org.opendaylight.restconf.nb.rfc8040.rests.utils.PutDataTransactionUtil; import org.opendaylight.restconf.nb.rfc8040.rests.utils.ReadDataTransactionUtil; import org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfDataServiceConstant; +import org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfDataServiceConstant.PostPutQueryParameters.Insert; import org.opendaylight.restconf.nb.rfc8040.rests.utils.RestconfInvokeOperationsUtil; -import org.opendaylight.restconf.nb.rfc8040.utils.RestconfConstants; +import org.opendaylight.restconf.nb.rfc8040.streams.Configuration; +import org.opendaylight.restconf.nb.rfc8040.streams.listeners.NotificationListenerAdapter; +import org.opendaylight.restconf.nb.rfc8040.utils.mapping.RestconfMappingNodeUtil; +import org.opendaylight.restconf.nb.rfc8040.utils.parser.IdentifierCodec; import org.opendaylight.restconf.nb.rfc8040.utils.parser.ParserIdentifier; +import org.opendaylight.yang.gen.v1.urn.sal.restconf.event.subscription.rev140708.NotificationOutputTypeGrouping.NotificationOutputType; import org.opendaylight.yangtools.concepts.Immutable; import org.opendaylight.yangtools.yang.common.QName; import org.opendaylight.yangtools.yang.common.Revision; import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier; import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode; +import org.opendaylight.yangtools.yang.data.api.schema.MapEntryNode; import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode; import org.opendaylight.yangtools.yang.model.api.ActionDefinition; +import org.opendaylight.yangtools.yang.model.api.EffectiveModelContext; +import org.opendaylight.yangtools.yang.model.api.NotificationDefinition; import org.opendaylight.yangtools.yang.model.api.SchemaNode; -import org.opendaylight.yangtools.yang.model.api.SchemaPath; +import org.opendaylight.yangtools.yang.model.api.stmt.SchemaNodeIdentifier.Absolute; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -74,11 +89,12 @@ import org.slf4j.LoggerFactory; */ @Path("/") public class RestconfDataServiceImpl implements RestconfDataService { + // FIXME: we should be able to interpret 'point' and refactor this class into a behavior private static final class QueryParams implements Immutable { final @Nullable String point; - final @Nullable String insert; + final @Nullable Insert insert; - QueryParams(final @Nullable String insert, final @Nullable String point) { + QueryParams(final @Nullable Insert insert, final @Nullable String point) { this.insert = insert; this.point = point; } @@ -88,6 +104,7 @@ public class RestconfDataServiceImpl implements RestconfDataService { private static final DateTimeFormatter FORMATTER = DateTimeFormatter.ofPattern("yyyy-MMM-dd HH:mm:ss"); private final RestconfStreamsSubscriptionService delegRestconfSubscrService; + private final SubscribeToStreamUtil streamUtils; // FIXME: evaluate thread-safety of updates (synchronized) vs. access (mostly unsynchronized) here private SchemaContextHandler schemaContextHandler; @@ -99,12 +116,15 @@ public class RestconfDataServiceImpl implements RestconfDataService { final TransactionChainHandler transactionChainHandler, final DOMMountPointServiceHandler mountPointServiceHandler, final RestconfStreamsSubscriptionService delegRestconfSubscrService, - final ActionServiceHandler actionServiceHandler) { + final ActionServiceHandler actionServiceHandler, + final Configuration configuration) { this.actionServiceHandler = requireNonNull(actionServiceHandler); this.schemaContextHandler = requireNonNull(schemaContextHandler); this.transactionChainHandler = requireNonNull(transactionChainHandler); this.mountPointServiceHandler = requireNonNull(mountPointServiceHandler); this.delegRestconfSubscrService = requireNonNull(delegRestconfSubscrService); + streamUtils = configuration.isUseSSE() ? SubscribeToStreamUtil.serverSentEvents() + : SubscribeToStreamUtil.webSockets(); } @Override @@ -129,21 +149,25 @@ public class RestconfDataServiceImpl implements RestconfDataService { @Override public Response readData(final String identifier, final UriInfo uriInfo) { - final SchemaContextRef schemaContextRef = new SchemaContextRef(this.schemaContextHandler.get()); + final EffectiveModelContext schemaContextRef = this.schemaContextHandler.get(); final InstanceIdentifierContext instanceIdentifier = ParserIdentifier.toInstanceIdentifier( - identifier, schemaContextRef.get(), Optional.of(this.mountPointServiceHandler.get())); + identifier, schemaContextRef, Optional.of(this.mountPointServiceHandler.get())); final WriterParameters parameters = ReadDataTransactionUtil.parseUriParameters(instanceIdentifier, uriInfo); final DOMMountPoint mountPoint = instanceIdentifier.getMountPoint(); - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper( - instanceIdentifier, mountPoint, getTransactionChainHandler(mountPoint)); - final NormalizedNode node = ReadDataTransactionUtil.readData(identifier, parameters.getContent(), - transactionNode, parameters.getWithDefault(), schemaContextRef, uriInfo); + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); + final NormalizedNode node; + if (parameters.getFieldPaths() != null && !parameters.getFieldPaths().isEmpty()) { + node = ReadDataTransactionUtil.readData(parameters.getContent(), instanceIdentifier.getInstanceIdentifier(), + strategy, parameters.getWithDefault(), schemaContextRef, parameters.getFieldPaths()); + } else { + node = readData(identifier, parameters.getContent(), instanceIdentifier.getInstanceIdentifier(), strategy, + parameters.getWithDefault(), schemaContextRef, uriInfo); + } if (identifier != null && identifier.contains(STREAM_PATH) && identifier.contains(STREAM_ACCESS_PATH_PART) && identifier.contains(STREAM_LOCATION_PATH_PART)) { final String value = (String) node.getValue(); - final String streamName = value.substring( - value.indexOf(NOTIFICATION_STREAM + RestconfConstants.SLASH)); + final String streamName = value.substring(value.indexOf(NOTIFICATION_STREAM + '/')); this.delegRestconfSubscrService.subscribeToStream(streamName, uriInfo); } if (node == null) { @@ -159,7 +183,7 @@ public class RestconfDataServiceImpl implements RestconfDataService { return Response.status(200) .entity(new NormalizedNodeContext(instanceIdentifier, node, parameters)) .header("ETag", '"' + type.getModule().getRevision().map(Revision::toString).orElse(null) - + type.getLocalName() + '"') + + "-" + type.getLocalName() + '"') .header("Last-Modified", FORMATTER.format(LocalDateTime.now(Clock.systemUTC()))) .build(); } @@ -167,60 +191,111 @@ public class RestconfDataServiceImpl implements RestconfDataService { return Response.status(200).entity(new NormalizedNodeContext(instanceIdentifier, node, parameters)).build(); } + /** + * Read specific type of data from data store via transaction and if identifier read data from + * streams then put streams from actual schema context to datastore. + * + * @param identifier identifier of data to read + * @param content type of data to read (config, state, all) + * @param strategy {@link RestconfStrategy} - object that perform the actual DS operations + * @param withDefa value of with-defaults parameter + * @param schemaContext schema context + * @param uriInfo uri info + * @return {@link NormalizedNode} + */ + private NormalizedNode readData(final String identifier, final String content, + final YangInstanceIdentifier path, final RestconfStrategy strategy, final String withDefa, + final EffectiveModelContext schemaContext, final UriInfo uriInfo) { + if (identifier != null && identifier.contains(STREAMS_PATH) && !identifier.contains(STREAM_PATH_PART)) { + createAllYangNotificationStreams(strategy, schemaContext, uriInfo); + } + return ReadDataTransactionUtil.readData(content, path, strategy, withDefa, schemaContext); + } + + private void createAllYangNotificationStreams(final RestconfStrategy strategy, + final EffectiveModelContext schemaContext, final UriInfo uriInfo) { + final RestconfTransaction transaction = strategy.prepareWriteExecution(); + + for (final NotificationDefinition notificationDefinition : schemaContext.getNotifications()) { + final NotificationListenerAdapter notifiStreamXML = + CreateStreamUtil.createYangNotifiStream(notificationDefinition, schemaContext, + NotificationOutputType.XML); + final NotificationListenerAdapter notifiStreamJSON = + CreateStreamUtil.createYangNotifiStream(notificationDefinition, schemaContext, + NotificationOutputType.JSON); + writeNotificationStreamToDatastore(schemaContext, uriInfo, transaction, notifiStreamXML); + writeNotificationStreamToDatastore(schemaContext, uriInfo, transaction, notifiStreamJSON); + } + try { + transaction.commit().get(); + } catch (final InterruptedException | ExecutionException e) { + throw new RestconfDocumentedException("Problem while putting data to DS.", e); + } + } + + private void writeNotificationStreamToDatastore(final EffectiveModelContext schemaContext, + final UriInfo uriInfo, final RestconfTransaction transaction, final NotificationListenerAdapter listener) { + final URI uri = streamUtils.prepareUriByStreamName(uriInfo, listener.getStreamName()); + final MapEntryNode mapToStreams = RestconfMappingNodeUtil.mapYangNotificationStreamByIetfRestconfMonitoring( + listener.getSchemaPath().lastNodeIdentifier(), schemaContext.getNotifications(), null, + listener.getOutputType(), uri); + + final String name = listener.getSchemaPath().lastNodeIdentifier().getLocalName(); + transaction.merge(LogicalDatastoreType.OPERATIONAL, + IdentifierCodec.deserialize(Rfc8040.MonitoringModule.PATH_TO_STREAM_WITHOUT_KEY + name, schemaContext), + mapToStreams); + } + @Override public Response putData(final String identifier, final NormalizedNodeContext payload, final UriInfo uriInfo) { requireNonNull(payload); final QueryParams checkedParms = checkQueryParameters(uriInfo); - final InstanceIdentifierContext iid = payload - .getInstanceIdentifierContext(); + final InstanceIdentifierContext iid = payload.getInstanceIdentifierContext(); PutDataTransactionUtil.validInputData(iid.getSchemaNode(), payload); PutDataTransactionUtil.validTopLevelNodeName(iid.getInstanceIdentifier(), payload); PutDataTransactionUtil.validateListKeysEqualityInPayloadAndUri(payload); final DOMMountPoint mountPoint = payload.getInstanceIdentifierContext().getMountPoint(); - final TransactionChainHandler localTransactionChainHandler; - final SchemaContextRef ref; - if (mountPoint == null) { - localTransactionChainHandler = this.transactionChainHandler; - ref = new SchemaContextRef(this.schemaContextHandler.get()); - } else { - localTransactionChainHandler = transactionChainOfMountPoint(mountPoint); - ref = new SchemaContextRef(mountPoint.getEffectiveModelContext()); - } + final EffectiveModelContext ref = mountPoint == null + ? this.schemaContextHandler.get() : modelContext(mountPoint); - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper( - payload.getInstanceIdentifierContext(), mountPoint, localTransactionChainHandler); - return PutDataTransactionUtil.putData(payload, ref, transactionNode, checkedParms.insert, checkedParms.point); + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); + return PutDataTransactionUtil.putData(payload, ref, strategy, checkedParms.insert, checkedParms.point); } private static QueryParams checkQueryParameters(final UriInfo uriInfo) { boolean insertUsed = false; boolean pointUsed = false; - String insert = null; + Insert insert = null; String point = null; for (final Entry> entry : uriInfo.getQueryParameters().entrySet()) { switch (entry.getKey()) { case INSERT: - if (!insertUsed) { - insertUsed = true; - insert = entry.getValue().get(0); - } else { + if (insertUsed) { throw new RestconfDocumentedException("Insert parameter can be used only once.", - RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); + RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); + } + + insertUsed = true; + final String str = entry.getValue().get(0); + insert = Insert.forValue(str); + if (insert == null) { + throw new RestconfDocumentedException("Unrecognized insert parameter value '" + str + "'", + RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); } break; case POINT: - if (!pointUsed) { - pointUsed = true; - point = entry.getValue().get(0); - } else { + if (pointUsed) { throw new RestconfDocumentedException("Point parameter can be used only once.", - RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); + RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); } + + pointUsed = true; + point = entry.getValue().get(0); break; default: throw new RestconfDocumentedException("Bad parameter for post: " + entry.getKey(), @@ -232,15 +307,18 @@ public class RestconfDataServiceImpl implements RestconfDataService { return new QueryParams(insert, point); } - private static void checkQueryParams(final boolean insertUsed, final boolean pointUsed, final String insert) { - if (pointUsed && !insertUsed) { - throw new RestconfDocumentedException("Point parameter can't be used without Insert parameter.", + private static void checkQueryParams(final boolean insertUsed, final boolean pointUsed, final Insert insert) { + if (pointUsed) { + if (!insertUsed) { + throw new RestconfDocumentedException("Point parameter can't be used without Insert parameter.", RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); - } - if (pointUsed && (insert.equals("first") || insert.equals("last"))) { - throw new RestconfDocumentedException( + } + + if (insert != Insert.BEFORE && insert != Insert.AFTER) { + throw new RestconfDocumentedException( "Point parameter can be used only with 'after' or 'before' values of Insert parameter.", RestconfError.ErrorType.PROTOCOL, RestconfError.ErrorTag.BAD_ELEMENT); + } } } @@ -253,35 +331,24 @@ public class RestconfDataServiceImpl implements RestconfDataService { public Response postData(final NormalizedNodeContext payload, final UriInfo uriInfo) { requireNonNull(payload); if (payload.getInstanceIdentifierContext().getSchemaNode() instanceof ActionDefinition) { - return invokeAction(payload, uriInfo); + return invokeAction(payload); } final QueryParams checkedParms = checkQueryParameters(uriInfo); - final DOMMountPoint mountPoint = payload.getInstanceIdentifierContext().getMountPoint(); - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper( - payload.getInstanceIdentifierContext(), mountPoint, getTransactionChainHandler(mountPoint)); - return PostDataTransactionUtil.postData(uriInfo, payload, transactionNode, + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); + return PostDataTransactionUtil.postData(uriInfo, payload, strategy, getSchemaContext(mountPoint), checkedParms.insert, checkedParms.point); } @Override public Response deleteData(final String identifier) { - final SchemaContextRef schemaContextRef = new SchemaContextRef(this.schemaContextHandler.get()); final InstanceIdentifierContext instanceIdentifier = ParserIdentifier.toInstanceIdentifier( - identifier, schemaContextRef.get(), Optional.of(this.mountPointServiceHandler.get())); + identifier, this.schemaContextHandler.get(), Optional.of(this.mountPointServiceHandler.get())); final DOMMountPoint mountPoint = instanceIdentifier.getMountPoint(); - final TransactionChainHandler localTransactionChainHandler; - if (mountPoint == null) { - localTransactionChainHandler = this.transactionChainHandler; - } else { - localTransactionChainHandler = transactionChainOfMountPoint(mountPoint); - } - - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper(instanceIdentifier, mountPoint, - localTransactionChainHandler); - return DeleteDataTransactionUtil.deleteData(transactionNode); + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); + return DeleteDataTransactionUtil.deleteData(strategy, instanceIdentifier.getInstanceIdentifier()); } @Override @@ -292,9 +359,8 @@ public class RestconfDataServiceImpl implements RestconfDataService { @Override public PatchStatusContext patchData(final PatchContext context, final UriInfo uriInfo) { final DOMMountPoint mountPoint = requireNonNull(context).getInstanceIdentifierContext().getMountPoint(); - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper( - context.getInstanceIdentifierContext(), mountPoint, getTransactionChainHandler(mountPoint)); - return PatchDataTransactionUtil.patchData(context, transactionNode, getSchemaContext(mountPoint)); + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); + return PatchDataTransactionUtil.patchData(context, strategy, getSchemaContext(mountPoint)); } @Override @@ -309,61 +375,41 @@ public class RestconfDataServiceImpl implements RestconfDataService { PutDataTransactionUtil.validateListKeysEqualityInPayloadAndUri(payload); final DOMMountPoint mountPoint = payload.getInstanceIdentifierContext().getMountPoint(); - final TransactionChainHandler localTransactionChainHandler; - final SchemaContextRef ref; - if (mountPoint == null) { - localTransactionChainHandler = this.transactionChainHandler; - ref = new SchemaContextRef(this.schemaContextHandler.get()); - } else { - localTransactionChainHandler = transactionChainOfMountPoint(mountPoint); - ref = new SchemaContextRef(mountPoint.getEffectiveModelContext()); - } - - final TransactionVarsWrapper transactionNode = new TransactionVarsWrapper( - payload.getInstanceIdentifierContext(), mountPoint, localTransactionChainHandler); - - return PlainPatchDataTransactionUtil.patchData(payload, transactionNode, ref); - } + final EffectiveModelContext ref = mountPoint == null + ? this.schemaContextHandler.get() : modelContext(mountPoint); + final RestconfStrategy strategy = getRestconfStrategy(mountPoint); - private TransactionChainHandler getTransactionChainHandler(final DOMMountPoint mountPoint) { - return mountPoint == null ? transactionChainHandler : transactionChainOfMountPoint(mountPoint); + return PlainPatchDataTransactionUtil.patchData(payload, strategy, ref); } - private SchemaContextRef getSchemaContext(final DOMMountPoint mountPoint) { - return mountPoint == null ? new SchemaContextRef(schemaContextHandler.get()) - : new SchemaContextRef(mountPoint.getEffectiveModelContext()); + private EffectiveModelContext getSchemaContext(final DOMMountPoint mountPoint) { + return mountPoint == null ? schemaContextHandler.get() : modelContext(mountPoint); } - /** - * Prepare transaction chain to access data of mount point. - * @param mountPoint - * mount point reference - * @return {@link TransactionChainHandler} - */ - private static TransactionChainHandler transactionChainOfMountPoint(final @NonNull DOMMountPoint mountPoint) { - final Optional domDataBrokerService = mountPoint.getService(DOMDataBroker.class); - if (domDataBrokerService.isPresent()) { - return new TransactionChainHandler(domDataBrokerService.get()); + // FIXME: why is this synchronized? + public synchronized RestconfStrategy getRestconfStrategy(final DOMMountPoint mountPoint) { + if (mountPoint == null) { + return new MdsalRestconfStrategy(transactionChainHandler); } - final String errMsg = "DOM data broker service isn't available for mount point " + mountPoint.getIdentifier(); - LOG.warn(errMsg); - throw new RestconfDocumentedException(errMsg); + return RestconfStrategy.forMountPoint(mountPoint).orElseThrow(() -> { + LOG.warn("Mount point {} does not expose a suitable access interface", mountPoint.getIdentifier()); + return new RestconfDocumentedException("Could not find a supported access interface in mount point " + + mountPoint.getIdentifier()); + }); } /** * Invoke Action operation. * - * @param payload - * {@link NormalizedNodeContext} - the body of the operation - * @param uriInfo - * URI info + * @param payload {@link NormalizedNodeContext} - the body of the operation * @return {@link NormalizedNodeContext} wrapped in {@link Response} */ - public Response invokeAction(final NormalizedNodeContext payload, final UriInfo uriInfo) { + public Response invokeAction(final NormalizedNodeContext payload) { final InstanceIdentifierContext context = payload.getInstanceIdentifierContext(); final DOMMountPoint mountPoint = context.getMountPoint(); - final SchemaPath schemaPath = context.getSchemaNode().getPath(); + final Absolute schemaPath = Absolute.of(ImmutableList.copyOf(context.getSchemaNode().getPath() + .getPathFromRoot())); final YangInstanceIdentifier yangIIdContext = context.getInstanceIdentifier(); final NormalizedNode data = payload.getData(); @@ -373,15 +419,15 @@ public class RestconfDataServiceImpl implements RestconfDataService { } final DOMActionResult response; - final SchemaContextRef schemaContextRef; + final EffectiveModelContext schemaContextRef; if (mountPoint != null) { response = RestconfInvokeOperationsUtil.invokeActionViaMountPoint(mountPoint, (ContainerNode) data, schemaPath, yangIIdContext); - schemaContextRef = new SchemaContextRef(mountPoint.getEffectiveModelContext()); + schemaContextRef = modelContext(mountPoint); } else { response = RestconfInvokeOperationsUtil.invokeAction((ContainerNode) data, schemaPath, this.actionServiceHandler, yangIIdContext); - schemaContextRef = new SchemaContextRef(this.schemaContextHandler.get()); + schemaContextRef = this.schemaContextHandler.get(); } final DOMActionResult result = RestconfInvokeOperationsUtil.checkActionResponse(response); @@ -400,6 +446,12 @@ public class RestconfDataServiceImpl implements RestconfDataService { } return Response.status(200).entity(new NormalizedNodeContext(new InstanceIdentifierContext<>(yangIIdContext, - resultNodeSchema, mountPoint, schemaContextRef.get()), resultData)).build(); + resultNodeSchema, mountPoint, schemaContextRef), resultData)).build(); + } + + private static EffectiveModelContext modelContext(final DOMMountPoint mountPoint) { + return mountPoint.getService(DOMSchemaService.class) + .flatMap(svc -> Optional.ofNullable(svc.getGlobalContext())) + .orElse(null); } }