raboof commented on PR #7912:
URL: https://github.com/apache/geode/pull/7912#issuecomment-3226156357
If you click the failed integrationTest GitHub Actions job, you can see that
AnalyzeCoreSerializablesIntegrationTest is still failing:
```
AnalyzeCoreSerializablesIntegrationTest > testSerializables FAILED
java.lang.AssertionError: New or moved
classes----------------------------------------
org/apache/geode/management/internal/json/AbstractJSONFormatter$PreventReserializationModule$1,false,this$0:org/apache/geode/management/internal/json/AbstractJSONFormatter$PreventReserializationModule
org/apache/geode/management/internal/json/QueryResultFormatter$TypeSerializationEnforcerModule$1,false,this$0:org/apache/geode/management/internal/json/QueryResultFormatter$TypeSerializationEnforcerModule
If the class is not persisted or sent over the wire, add it to the file
/home/runner/work/geode/geode/geode-core/src/integrationTest/resources/org/apache/geode/codeAnalysis/excludedClasses.txt
Otherwise, if this doesn't break backward compatibility, copy the file
/home/runner/work/geode/geode/geode-core/build/integrationTest/test-worker-001022/actualSerializables.dat
to
/home/runner/work/geode/geode/geode-core/src/main/resources/org/apache/geode/internal/sanctioned-geode-core-serializables.txt
If this potentially breaks backward compatibility, follow the
instructions in
geode-serialization/ANALYZE_SERIALIZABLES.md
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@geode.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
