Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/#2755 signaling the data source that has a problem #2953

Merged

Conversation

Limraj
Copy link
Collaborator

@Limraj Limraj commented Jul 13, 2024

No description provided.

Limraj and others added 30 commits November 6, 2023 20:16
Created class: DataPointUnreliableUtils;

Signaling added in:
*  Meta Data Source;
*  SNMP;
*  HTTP Retriever;
*  BACnet I/P;
*  DNP3 (IP/Serial);

Signaling use DataPointUnreliableUtils in:
* Modbus IP/Serial;
* Viconics;
* Pachube;
* MQTT/AMQP
- fixed feature refresh point value
- DataPointUnreliableUtilsTest.java;
- use methods returnToNormal and raiseEvent from DataSourceRT;
- added methods: LoggingUtils.info, LoggingUtils.dataPointInfo, LoggingUtils.dataPointRtInfo, PollingDataSource.getDataPoints, EventDataSource.getDataPoints, DataSourceRT.getDataPoints;
- fixed Data Source use serial port - System Exception;
- corrected terminate method for: Alpha2DataSource, Dnp3DataSource, DrStorageHt5bDataSource, IEC101DataSource, OPCDataSource, RadiuinoEventDataSource, RadiuinoPollingDataSource;
- fixed reset unreliable HttpImageDataSourceRT when error retriever;
- refactor DataPointUnreliableUtils;
- fixed RuntimeManager.startDataPoint;
- added tests: InitializeDataSourceRtTest, InitializeWithErrorsDataSourceRtTest;
- corrected test: DataPointUnreliableUtilsTest;
- replacing class CommPortIdentifier from RXTXcomm.jar with a new class ScadaCommPortIdentifier: ASCIISerialDataSource, DrStorageHt5bDataSource, RadiuinoEventDataSource, RadiuinoPollingDataSource;
- corrected signaling errors in: BACnetIPDataSourceRT, HttpReceiverDataSourceRT, ModbusDataSource, NmeaDataSourceRT, PachubeDataSourceRT, PersistentDataSourceRT, SnmpDataSourceRT, ViconicsDataSourceRT;
- try catch Throwable;
- change logging level from WARN to ERROR for dataSourceRtLoggingLevel, dataPointRtLoggingLevel (log4j2.xml);
- refactoring DataPointUnreliableUtils;
- added methods: DataPointRT.isUnreliable, DataPointRT.isSetUnreliable, DataSourceUtils.checkInitialized;
- corrected test: InitializeWithErrorsDataSourceRtTest;
- change type catch from IOException to Throwable;
- corrected test: InitializeWithErrorsDataSourceRtTest for VMStatDataSourceRT - update PrepareForTest annotation, if dataSourceRT instanceof VMStatDataSourceRT, for MessagingDataSourceRT with MQTT throw MessagingServiceException after invoke MessageService.open;
- fixed generate many events by ModbusDataSource;
- corrected test InitializeDataSourceRtTest;
- refactor InitializeDataSourceRtMockUtils;
- added class: RuntimeMockUtils
- fixed generate many events by: Pop3DataSourceRT, PachubeDataSourceRT, NmeaDataSourceRT, HttpRetrieverDataSourceRT, BACnetIPDataSourceRT, change to active events;
- change logging fatal to debug level in methods: MBusDataSourceRT.initialize(), MBusDataSourceRT.terminate();
- fixed DataPointUnreliableUtilsTest, InitializeDataSourceRtTest, InitializeWithErrorsDataSourceRtTest, mocking TimeoutTask;
- corrected terminate method for: Alpha2DataSource, Dnp3DataSource, DrStorageHt5bDataSource, IEC101DataSource, OPCDataSource, RadiuinoEventDataSource, RadiuinoPollingDataSource;
- fixed reset unreliable HttpImageDataSourceRT when error retriever;
- refactor DataPointUnreliableUtils;
- fixed RuntimeManager.startDataPoint;
- added tests: InitializeDataSourceRtTest, InitializeWithErrorsDataSourceRtTest;
- corrected test: DataPointUnreliableUtilsTest;
- replacing class CommPortIdentifier from RXTXcomm.jar with a new class ScadaCommPortIdentifier: ASCIISerialDataSource, DrStorageHt5bDataSource, RadiuinoEventDataSource, RadiuinoPollingDataSource;
- corrected signaling errors in: BACnetIPDataSourceRT, HttpReceiverDataSourceRT, ModbusDataSource, NmeaDataSourceRT, PachubeDataSourceRT, PersistentDataSourceRT, SnmpDataSourceRT, ViconicsDataSourceRT;
- try catch Throwable;
- change logging level from WARN to ERROR for dataSourceRtLoggingLevel, dataPointRtLoggingLevel (log4j2.xml);
- refactoring DataPointUnreliableUtils;
- added methods: DataPointRT.isUnreliable, DataPointRT.isSetUnreliable, DataSourceUtils.checkInitialized;
- corrected test: InitializeWithErrorsDataSourceRtTest;
- change type catch from IOException to Throwable;
- corrected test: InitializeWithErrorsDataSourceRtTest for VMStatDataSourceRT - update PrepareForTest annotation, if dataSourceRT instanceof VMStatDataSourceRT, for MessagingDataSourceRT with MQTT throw MessagingServiceException after invoke MessageService.open;
- fixed generate many events by ModbusDataSource;
- corrected test InitializeDataSourceRtTest;
- refactor InitializeDataSourceRtMockUtils;
- added class: RuntimeMockUtils
- fixed generate many events by: Pop3DataSourceRT, PachubeDataSourceRT, NmeaDataSourceRT, HttpRetrieverDataSourceRT, BACnetIPDataSourceRT, change to active events;
- change logging fatal to debug level in methods: MBusDataSourceRT.initialize(), MBusDataSourceRT.terminate();
- fixed DataPointUnreliableUtilsTest, InitializeDataSourceRtTest, InitializeWithErrorsDataSourceRtTest, mocking TimeoutTask;
- fixed generate many events by SqlDataSourceRT;
- fixed export DrStorageHt5bDataSourceVO by removed @JsonRemoteProperty from validate method;
Limraj and others added 3 commits May 16, 2024 16:58
- fixed updateAttemptsCounters;
- test.failFast, testLogging.exceptionFormat = full;
@Limraj Limraj added this to the 2.8.0 milestone Jul 13, 2024
@Limraj Limraj requested a review from SoftQ as a code owner July 13, 2024 21:55
Copy link

github-actions bot commented Jul 13, 2024

Java Script Mocha Unit Test Results

268 tests   268 ✅  4s ⏱️
 70 suites    0 💤
  1 files      0 ❌

Results for commit a38938c.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 13, 2024

Java JUnit Test Results

2 357 tests   2 357 ✅  42s ⏱️
  113 suites      0 💤
  113 files        0 ❌

Results for commit a38938c.

♻️ This comment has been updated with latest results.

Limraj and others added 3 commits July 22, 2024 12:35
…gnaling_the_Data_Source_that_has_a_problem

# Conflicts:
#	webapp-resources/messages_de.properties
#	webapp-resources/messages_en.properties
#	webapp-resources/messages_es.properties
#	webapp-resources/messages_fi.properties
#	webapp-resources/messages_fr.properties
#	webapp-resources/messages_nl.properties
#	webapp-resources/messages_pt.properties
#	webapp-resources/messages_ru.properties
#	webapp-resources/messages_zh.properties
- Corrected config log4j2.xml for script: ScriptExecutor, ContextualizedScriptRT;
- Added new event type DataSourcePointEventType;
- Added migration V2_8_0_1__AddTypeRef3ColumnToEvents;
@Limraj Limraj requested a review from Patrykb0802 August 14, 2024 13:49
Limraj and others added 8 commits August 14, 2024 15:53
- fixed alarmAck.tag for new event type;
- fixed double actions for eventList.jsp;
- fixed url: scripting.jsp, dataPointEdit.jsp, systemSettings.jsp;
- corrected MetaPointLocatorRT - reduce generate events, DataSourceEditDwr, PointValueService, ContextualizedScriptRT - catch Exception;
- fixed ScriptExecutor, event per point;
- set IGNORE_SAME_MESSAGE for DataSourcePointEventType.duplicateHandling;
- fixed ASCIIFileDataSource, BACnetIPDataSourceRT;
- set ERROR logging level for dataSourceRtLoggingLevel
Copy link
Contributor

@Patrykb0802 Patrykb0802 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ad.1. After corrections meta data point is signalling error when context points are disabled.

Ad.2. Imported given cofigruation, after disabling virtual/sql data source you can see on watchlist that points might be unreliable as well as alarms which are visible on alarms view.

Ad.3. Case from issue to check, you can check it first on the old code; --->
I created 3 points, 1 virtual, 2 meta,
virtual was context point for one of the meta data point (mdp1) and second meta point (mdp2) had mdp1 as context. So hierarchy looked like this:
mdp2<-<
|
mdp1<-^
|
vdp1>--^

vdp1 is context of mdp1, mdp1 is context of mdp2

Old version:
All points are toggled on, let's turn off vdp1, only mdp1 is signalling issue with context.

New version:
All points are toggled on, let's turn off vdp1, both mdp1 and mdp2 are signaling issue. -> correct

Ad.4. Created such configuration, only one alarm is generated, works correctly

Ad.5. Generation of multiple events works correctly:

image

@Limraj Limraj merged commit 1d22805 into release/2.8.0 Sep 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants