Hi,

While not pretty, this should work. It would seem we have some sort of bug in deleteNode that it didn’t work as intended, thus this deletion is one-by-one instead of just removing the ServerRedundancy node.Also it would seem the base nodeset does contain all the Properties of ALL possible redundancy types, thus they are all deleted.

It is important that the nodes are made back with the same exact NodeIds, as they are fixed in the base information model and could be used by clients to read the info directly. Thus this required some extra steps.

Also, the UaServer.init must have been called before this code.

    NodeManagerRoot nm = server.getNodeManagerRoot();

    // Deleting the old nodes
    nm.deleteNode(Identifiers.Server_ServerRedundancy, true, false);
    nm.deleteNode(Identifiers.Server_ServerRedundancy_CurrentServerId, true, false);
    nm.deleteNode(Identifiers.Server_ServerRedundancy_RedundancySupport, true, false);
    nm.deleteNode(Identifiers.Server_ServerRedundancy_RedundantServerArray, true, false);
    nm.deleteNode(Identifiers.Server_ServerRedundancy_ServerNetworkGroups, true, false);
    nm.deleteNode(Identifiers.Server_ServerRedundancy_ServerUriArray, true, false);


    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy));
    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy_CurrentServerId));
    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy_RedundancySupport));
    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy_RedundantServerArray));
    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy_ServerNetworkGroups));
    assertNull(nm.findNode(Identifiers.Server_ServerRedundancy_ServerUriArray));

    // Creating replacement nodes
    UaNodeBuilderConfiguration delegateConf = nm.getNodeBuilderConfiguration();
    UaNodeBuilderConfiguration conf = new JoinedNodeBuilderConfiguration(parameters -> {
      String name = parameters.getInstanceDeclaration().getNode().getBrowseName().getName();
      if (name.equals("ServerUriArray")) {
        return new DefaultInstantiationStrategy() {
          @Override
          public UaNode createInstance(InstantiationParameters params) {
            return params.getNodeManager().getNodeFactory().createNode(NodeClass.Variable,
                Identifiers.Server_ServerRedundancy_ServerUriArray, params.getBrowseName(), params.getDisplayName(),
                Identifiers.PropertyType);
          }
        };
      } else if (name.equals("RedundancySupport")) {
        return new DefaultInstantiationStrategy() {
          @Override
          public UaNode createInstance(InstantiationParameters params) {
            return params.getNodeManager().getNodeFactory().createNode(NodeClass.Variable,
                Identifiers.Server_ServerRedundancy_RedundancySupport, params.getBrowseName(), params.getDisplayName(),
                Identifiers.PropertyType);
          }
        };
      }
      return null; // not handled in this conf
    }, delegateConf);

    NodeBuilder nb =
        nm.createNodeBuilder(NonTransparentRedundancyTypeNode.class, conf);
    nb.setName("ServerRedundancy");
    nb.setNodeId(Identifiers.Server_ServerRedundancy);

    NonTransparentRedundancyTypeNode redundancyNode = nb.build();
    ServerTypeNode serverNode = nm.getServerData();
    serverNode.addComponent(redundancyNode);

    assertEquals(Identifiers.Server_ServerRedundancy, redundancyNode.getNodeId());
    assertEquals(Identifiers.Server_ServerRedundancy_ServerUriArray,
        redundancyNode.getServerUriArrayNode().getNodeId());
    assertEquals(Identifiers.Server_ServerRedundancy_RedundancySupport,
        redundancyNode.getRedundancySupportNode().getNodeId());

Great!

Hmm … actually it is just that the newer nodeset we have in the SDK (versus Simulation Server’s release version I used to check the address space) has changed so that those extra Properties are no longer below that node (this is good in general), thus the delete didn’t work for them.

Regardless they (the extra properties) are still in the nodeset and also do have incorrect BrowseName to be used as is:

i.e. the BrowseName must match to the one defined in the TypeDefinition (i.e. the “Server_ServerRedundancy_” prefix is wrong), since they would be part of the larger type (OR I’m not 100% sure anymore are these the Ids to use for these nodes, or maybe this is just a mistake in the nodeset).

Doesn’t change that what I said will work, but mainly why it didn’t work with just “nm.deleteNode(Identifiers.Server_ServerRedundancy, true, true);” I tried first…