[GitHub] [ozone] adoroszlai commented on a change in pull request #1984: HDDS-4888. Merge basic CI checks

[GitBox <gi...@apache.org>]

adoroszlai commented on a change in pull request #1984:
URL: https://github.com/apache/ozone/pull/1984#discussion_r591726837



##########
File path: hadoop-ozone/dev-support/checks/install/_install.sh
##########
@@ -0,0 +1,34 @@
+#!/usr/bin/env bash

Review comment:
       > Tricky name, as this script is responsible both for install and executing the scripts.
   
   This is just a wrapper to invoke a specific install script.  It does not run the check.  The only reason for its existence: the workflow file can blindly invoke it without checking for the existence of the specific install script (I'm not even sure it would be possible).




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org