adoroszlai commented on PR #7880: URL: https://github.com/apache/ozone/pull/7880#issuecomment-2659664460
Thanks @chiacyu for the patch. > It can test by CI. It is not tested by default, only `ha` is enabled: https://github.com/apache/ozone/blob/f2d9cc3bb40251e797ef527180b0ad74bfb112fb/hadoop-ozone/dist/src/main/compose/upgrade/test.sh#L36-L43 We'd need to add: ``` run_test non-ha non-rolling-upgrade 1.4.1 "$OZONE_CURRENT_VERSION" ``` I'd add it before the `ha` test, to fail the test earlier, since the test is currently failing (see below). However, as I mentioned in HDDS-12327, this is currently failing near the first steps (`ozone admin om finalizationstatus`). The problem is that the config does not have OM service ID set (since it used to be a non-HA cluster), but the command expects either the config or command-line arguments to define the OM service ID. We need to solve this problem first (and may also encounter further errors later in the test). -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
