OTA-2480: Remove OPC-UA support

Signed-off-by: Mike Sul <ext-mykhaylo.sul@here.com>

.travis.yml

@@ -9,7 +9,7 @@ env:
 - DOCKERFILE=docker/Dockerfile.ubuntu.bionic SCRIPT=scripts/test.sh
   DARGS="-eTEST_CMAKE_BUILD_TYPE=Valgrind -eTEST_WITH_COVERAGE=1 -eTEST_WITH_P11=1 -eTEST_WITH_FAULT_INJECTION=1 -eTEST_TESTSUITE_EXCLUDE=credentials -eTEST_SOTA_PACKED_CREDENTIALS=dummy-credentials"
 - DOCKERFILE=docker/Dockerfile.debian.testing SCRIPT=scripts/test.sh
-  DARGS="-eTEST_CC=clang -eTEST_WITH_LOAD_TESTS=1 -eTEST_WITH_TESTSUITE=0 -eTEST_WITH_STATICTESTS=1 -eTEST_WITH_OPCUA=1"
+  DARGS="-eTEST_CC=clang -eTEST_WITH_LOAD_TESTS=1 -eTEST_WITH_TESTSUITE=0 -eTEST_WITH_STATICTESTS=1"
 - DEPLOY_PKGS=1 RELEASE_NAME=ubuntu_18.04 DOCKERFILE=docker/Dockerfile.ubuntu.bionic INSTALL_DOCKERFILE=docker/Dockerfile-test-install.ubuntu.bionic SCRIPT=scripts/build_ubuntu.sh DARGS="-eTEST_INSTALL_RELEASE_NAME=-ubuntu_18.04"
 - DEPLOY_PKGS=1 RELEASE_NAME=ubuntu_16.04 DOCKERFILE=docker/Dockerfile.ubuntu.xenial INSTALL_DOCKERFILE=docker/Dockerfile-test-install.ubuntu.xenial SCRIPT=scripts/build_ubuntu.sh DARGS="-eTEST_INSTALL_RELEASE_NAME=-ubuntu_16.04"
 services:

CMakeLists.txt

@@ -31,7 +31,6 @@ option(BUILD_OSTREE "Set to ON to compile with ostree support" OFF)
 option(BUILD_DEB "Set to ON to compile with debian packages support" OFF)
 option(BUILD_P11 "Support for key storage in a HSM via PKCS#11" OFF)
 option(BUILD_SOTA_TOOLS "Set to ON to build SOTA tools" OFF)
-option(BUILD_OPCUA "Set to ON to compile with OPC-UA protocol support" OFF)
 option(BUILD_PARTIAL "Set to ON to compile with partial secondaries support" OFF)
 option(BUILD_ISOTP "Set to ON to compile with ISO/TP protocol support" OFF)
 option(BUILD_SYSTEMD "Set to ON to compile with systemd additional support" ${BUILD_SYSTEMD_DEFAULT})
@@ -72,10 +71,6 @@ if(ANDROID)
     endif()
 endif(ANDROID)
 
-if(BUILD_OPCUA)
-    list(APPEND BOOST_COMPONENTS serialization iostreams)
-endif(BUILD_OPCUA)
-
 # Mac brew library install paths
 
 if(EXISTS /usr/local/opt/openssl)
@@ -143,13 +138,6 @@ if(BUILD_SOTA_TOOLS)
     find_program(STRACE NAMES strace)
 endif(BUILD_SOTA_TOOLS)
 
-if(BUILD_OPCUA)
-    add_definitions(-DOPCUA_SECONDARY_ENABLED)
-    if (NOT BUILD_OSTREE)
-        message(FATAL_ERROR "BUILD_OPCUA requires BUILD_OSTREE")
-    endif(NOT BUILD_OSTREE)
-endif(BUILD_OPCUA)
-
 if(BUILD_ISOTP)
     add_definitions(-DISOTP_SECONDARY_ENABLED)
 endif(BUILD_ISOTP)
@@ -435,12 +423,6 @@ add_subdirectory("docs")
 file(GLOB_RECURSE ALL_SOURCE_FILES RELATIVE ${CMAKE_SOURCE_DIR}
     src/*.cc src/*.c src/*.h)
 foreach(FILE ${ALL_SOURCE_FILES})
-    # ignore opcuabridge
-    string (FIND ${FILE} "/opcuabridge/" EXCLUDE_DIR_FOUND)
-    if (NOT ${EXCLUDE_DIR_FOUND} EQUAL -1)
-      continue()
-    endif()
-
     string (FIND ${FILE} "/isotp_conn/" EXCLUDE_DIR_FOUND)
     if (NOT ${EXCLUDE_DIR_FOUND} EQUAL -1)
       continue()

Jenkinsfile

@@ -115,7 +115,6 @@ pipeline {
             TEST_CMAKE_BUILD_TYPE = 'Debug'
             TEST_TESTSUITE_ONLY = 'crypto'
             TEST_WITH_STATICTESTS = '1'
-            TEST_WITH_OPCUA = '1'
             TEST_WITH_LOAD_TESTS = '1'  // build only
           }
           steps {

README.adoc

@@ -65,7 +65,6 @@ Some features also require additional packages:
 * For PKCS#11 support, you will need `libp11-2 libp11-dev`.
 * For systemd support for secondaries, you will need `libsystemd-dev`.
 * For fault injection, you will need `fiu-utils libfiu-dev`.
-* For OPCUA support, you will need `libboost-iostreams-dev libboost-serialization-dev`.
 * For Debian packagin support, you will need `libdpkg-dev`.
 
 ==== Mac support

actions.md

@@ -269,7 +269,6 @@ These are internal requirements that are relatively opaque to the user and/or co
   - [x] Support virtual partial verification secondaries for testing
     - [x] Partial verification secondaries generate and store public keys (uptane_secondary_test.cc)
     - [x] Partial verification secondaries can verify Uptane metadata (uptane_secondary_test.cc)
-  - [x] Support OPC-UA secondaries (opcuabridge_messaging_test.cc, opcuabridge_secondary_update_test.cc, run_opcuabridge_ostree_repo_sync_test.sh)
 
 ### Expected action sequences
 

ci/gitlab/.gitlab-ci.yml

@@ -61,7 +61,6 @@ Debian build and test:
     TEST_CMAKE_BUILD_TYPE: 'Debug'
     TEST_TESTSUITE_ONLY: 'crypto'
     TEST_WITH_STATICTESTS: '1'
-    TEST_WITH_OPCUA: '1'
     TEST_WITH_LOAD_TESTS: '1'
   image: "$DEBIAN_TESTING_PR_IMAGE"
   stage: Build and Test

docker/Dockerfile-test-install.ubuntu.bionic

@@ -6,7 +6,6 @@ ENV DEBIAN_FRONTEND noninteractive
 RUN apt-get update && apt-get -y install debian-archive-keyring
 RUN apt-get update && apt-get install -y \
   libarchive13 \
-  libboost-iostreams1.65.1 \
   libboost-log1.65.1 \
   libboost-program-options1.65.1 \
   libboost-system1.65.1 \

docker/Dockerfile-test-install.ubuntu.xenial

@@ -6,7 +6,6 @@ ENV DEBIAN_FRONTEND noninteractive
 RUN apt-get update && apt-get -y install debian-archive-keyring
 RUN apt-get update && apt-get install -y \
   libarchive13 \
-  libboost-iostreams1.58.0 \
   libboost-log1.58.0 \
   libboost-program-options1.58.0 \
   libboost-system1.58.0 \

docker/Dockerfile.debian.testing

@@ -25,7 +25,6 @@ RUN apt-get update && apt-get -y install \
   lcov \
   libarchive-dev \
   libboost-dev \
-  libboost-iostreams-dev \
   libboost-log-dev \
   libboost-program-options-dev \
   libboost-system-dev \

docker/Dockerfile.ubuntu.bionic

@@ -22,7 +22,6 @@ RUN apt-get update && apt-get -y install \
   lcov \
   libarchive-dev \
   libboost-dev \
-  libboost-iostreams-dev \
   libboost-log-dev \
   libboost-program-options-dev \
   libboost-system-dev \

docker/Dockerfile.ubuntu.xenial

@@ -23,7 +23,6 @@ RUN apt-get update && apt-get -y install \
   lcov \
   libarchive-dev \
   libboost-dev \
-  libboost-iostreams-dev \
   libboost-log-dev \
   libboost-program-options-dev \
   libboost-system-dev \

docs/Doxyfile.in

@@ -805,7 +805,6 @@ INPUT                  = @CMAKE_SOURCE_DIR@/CONTRIBUTING.md \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/crypto \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/http \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/logging \
-                         @CMAKE_SOURCE_DIR@/src/libaktualizr/opcuabridge \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/package_manager \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/primary \
                          @CMAKE_SOURCE_DIR@/src/libaktualizr/socket_activation \

docs/README.adoc

@@ -22,8 +22,6 @@ link:./integrating-libaktualizr.adoc[integrating-libaktualizr.adoc] - How to use
 
 link:./linux-secondaries.adoc[linux-secondaries.adoc] - A quick how-to demonstrating aktualizr on a secondary ECU, using two QEMU devices.
 
-link:./opcua-bridge.adoc[opcua-bridge.adoc] - Some basic documentation on getting OPC-UA working. OPC-UA is a protocol for in-vehicle inter-ECU communication.
-
 link:./rollback.adoc[rollback.adoc] - Developer documentation on how aktualizr, OSTree, and u-boot can be used to implement automatic rollback of a failed update.
 
 link:./schema-migrations.adoc[schema-migrations.adoc] - aktualizr uses a SQLite database for storing some config information and keys. This describes the steps needed for migrating the DB schema. Only useful for aktualizr developers.

scripts/test.sh

@@ -20,7 +20,6 @@ TEST_WITH_OSTREE=${TEST_WITH_OSTREE:-1}
 TEST_WITH_DEB=${TEST_WITH_DEB:-1}
 TEST_WITH_ISOTP=${TEST_WITH_ISOTP:-1}
 TEST_WITH_PARTIAL=${TEST_WITH_PARTIAL:-1}
-TEST_WITH_OPCUA=${TEST_WITH_OPCUA:-0}
 TEST_WITH_FAULT_INJECTION=${TEST_WITH_FAULT_INJECTION:-0}
 TEST_WITH_LOAD_TESTS=${TEST_WITH_LOAD_TESTS:-0}
 
@@ -51,7 +50,6 @@ if [[ $TEST_WITH_OSTREE = 1 ]]; then CMAKE_ARGS+=("-DBUILD_OSTREE=ON"); fi
 if [[ $TEST_WITH_DEB = 1 ]]; then CMAKE_ARGS+=("-DBUILD_DEB=ON"); fi
 if [[ $TEST_WITH_ISOTP = 1 ]]; then CMAKE_ARGS+=("-DBUILD_ISOTP=ON"); fi
 if [[ $TEST_WITH_PARTIAL = 1 ]]; then CMAKE_ARGS+=("-DBUILD_PARTIAL=ON"); fi
-if [[ $TEST_WITH_OPCUA = 1 ]]; then CMAKE_ARGS+=("-DBUILD_OPCUA=ON"); fi
 if [[ $TEST_WITH_FAULT_INJECTION = 1 ]]; then CMAKE_ARGS+=("-DFAULT_INJECTION=ON"); fi
 if [[ $TEST_WITH_LOAD_TESTS = 1 ]]; then CMAKE_ARGS+=("-DBUILD_LOAD_TESTS=ON"); fi
 if [[ -n $TEST_SOTA_PACKED_CREDENTIALS ]]; then

src/aktualizr_secondary/CMakeLists.txt

@@ -22,20 +22,6 @@ add_library(aktualizr_secondary_static_lib STATIC
 
 target_link_libraries(aktualizr_secondary_static_lib aktualizr-posix)
 
-set(OPCUA_SECONDARY_SRC
-    opcuaserver_secondary_delegate.cc
-    aktualizr_secondary_opcua.cc
-    )
-
-if (BUILD_OPCUA)
-    target_sources(aktualizr_secondary_static_lib PRIVATE
-        ${OPCUA_SECONDARY_SRC} $<TARGET_OBJECTS:opcua_bridge>)
-    set_source_files_properties(${OPCUA_SECONDARY_SRC} ${AKTUALIZR_SECONDARY_SRC}
-        PROPERTIES COMPILE_FLAGS "-Wno-unused-parameter -Wno-float-equal -Wno-logical-op -Wno-unknown-warning-option")
-    target_include_directories(aktualizr_secondary_static_lib PUBLIC ${PROJECT_SOURCE_DIR}/src/libaktualizr/opcuabridge
-                                                                     ${PROJECT_SOURCE_DIR}/third_party/open62541)
-endif (BUILD_OPCUA)
-
 target_include_directories(aktualizr_secondary_static_lib PUBLIC
     $<TARGET_PROPERTY:socket_activation,INCLUDE_DIRECTORIES>
     $<TARGET_PROPERTY:asn1_lib,INCLUDE_DIRECTORIES>
@@ -58,8 +44,6 @@ set(ALL_AKTUALIZR_SECONDARY_HEADERS
     aktualizr_secondary_config.h
     aktualizr_secondary_common.h
     aktualizr_secondary_discovery.h
-    aktualizr_secondary_opcua.h
-    opcuaserver_secondary_delegate.h
     socket_server.h
     )
 
@@ -154,8 +138,7 @@ set_tests_properties(aktualizr_secondary_help_with_other_options
 
 aktualizr_source_file_checks(${AKTUALIZR_SECONDARY_SRC}
                              ${AKTUALIZR_SECONDARY_LIB_SRC}
-                             ${ALL_AKTUALIZR_SECONDARY_HEADERS}
-                             ${OPCUA_SECONDARY_SRC}
+                             ${ALL_AKTUALIZR_SECONDARY_HEADERS}                             
                              ${TEST_SOURCES})
 
 # vim: set tabstop=4 shiftwidth=4 expandtab:

src/aktualizr_secondary/main.cc

@@ -9,9 +9,6 @@
 #include "aktualizr_secondary.h"
 #include "aktualizr_secondary_config.h"
 #include "aktualizr_secondary_discovery.h"
-#ifdef OPCUA_SECONDARY_ENABLED
-#include "aktualizr_secondary_opcua.h"
-#endif
 #include "utilities/utils.h"
 
 #include "logging/logging.h"
@@ -44,10 +41,6 @@ class AktualizrSecondaryWithDiscovery : public AktualizrSecondaryInterface {
   std::thread discovery_thread_;
 };
 
-#ifdef OPCUA_SECONDARY_ENABLED
-typedef AktualizrSecondaryOpcua AktualizrSecondaryOpcuaWithDiscovery;
-#endif
-
 void check_secondary_options(const bpo::options_description &description, const bpo::variables_map &vm) {
   if (vm.count("help") != 0) {
     std::cout << description << '\n';
@@ -70,8 +63,7 @@ bpo::variables_map parse_options(int argc, char *argv[]) {
       ("server-port,p", bpo::value<int>(), "command server listening port")
       ("discovery-port,d", bpo::value<int>(), "discovery service listening port (0 to disable)")
       ("ecu-serial", bpo::value<std::string>(), "serial number of secondary ecu")
-      ("ecu-hardware-id", bpo::value<std::string>(), "hardware ID of secondary ecu")
-      ("opcua", "use OPC-UA protocol");
+      ("ecu-hardware-id", bpo::value<std::string>(), "hardware ID of secondary ecu");
   // clang-format on
 
   bpo::variables_map vm;
@@ -127,20 +119,8 @@ int main(int argc, char *argv[]) {
 
     // storage (share class with primary)
     std::shared_ptr<INvStorage> storage = INvStorage::newStorage(config.storage);
-
     std::unique_ptr<AktualizrSecondaryInterface> secondary;
-
-    if (commandline_map.count("opcua") != 0) {
-#ifdef OPCUA_SECONDARY_ENABLED
-      secondary = std_::make_unique<AktualizrSecondaryOpcuaWithDiscovery>(config, storage);
-#else
-      LOG_ERROR << "Built without OPC-UA support!";
-      return EXIT_FAILURE;
-#endif
-    } else {
-      secondary = std_::make_unique<AktualizrSecondaryWithDiscovery>(config, storage);
-    }
-
+    secondary = std_::make_unique<AktualizrSecondaryWithDiscovery>(config, storage);
     secondary->run();
 
   } catch (std::runtime_error &exc) {