wu-sheng commented on a change in pull request #8481:
URL: https://github.com/apache/skywalking/pull/8481#discussion_r793188099
##########
File path: .github/workflows/e2e.cluster.yaml
##########
@@ -38,9 +38,6 @@ jobs:
matrix:
config-file:
- cluster/zk/es/e2e.yaml
- - cluster/zk/mysql/e2e.yaml
- - cluster/zk/influxdb/e2e.yaml
- - cluster/zk/iotdb/e2e.yaml
Review comment:
All previous cluster tests are not working, as the coordinator is never
set correctly.
Now, based on discussions between Kai and me, for the cluster test cases,
the key is to make sure the coordinator works, rather than the storage option.
So, if we want to add more test cases, other coordinators should be tested
in similar ways. But all existing cases are not relevant.
This is the reason it is removed here.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]