Re: [I] [Feature] build manager image in build-images.sh [incubator-hugegraph-computer]

["imbajin (via GitHub)" <gi...@apache.org>]

imbajin commented on issue #278:
URL: https://github.com/apache/incubator-hugegraph-computer/issues/278#issuecomment-1788364023

   > Hi @imbajin , get it, thanks.
   > 
   > However, may I ask why we comment the cron schedule in `publish_computer_image.yml`? If we comment these, we should let user know how to build the latest images by themselves, maybe have to maintain the latest script(with new image `manager`) in this repo, either, and declare in our doc. [hugegraph/actions@`4eb9db3`/.github/workflows/publish_computer_image.yml#L3-L5](https://github.com/hugegraph/actions/blob/4eb9db3d644408df8f47558ecf151b510a1ffd24/.github/workflows/publish_computer_image.yml#L3-L5)
   
   Good catch & thanks for the reminder (Context: computer has not been **updated for a period of time** before, and it is unreasonable to keep updating the image on a regular crontab. We should **use a hook mechanism** that updates when there is a new commit)
   
   @Radeity @aroundabout could u follow the PR (https://github.com/hugegraph/actions/pull/2) to update computer image? (when free)


-- 
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: issues-unsubscribe@hugegraph.apache.org

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


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