[GitHub] [flink-kubernetes-operator] mbalassi commented on pull request #266: [FLINK-27960] Make the apt-get updating optional

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

mbalassi commented on PR #266:
URL: https://github.com/apache/flink-kubernetes-operator/pull/266#issuecomment-1153879405

   @Aitozi apologies if I was unclear, for the sake of clarity of explanation let us introduce the `SKIP_OS_UPDATE` flag (you can choose to call this `FAST` or name the "shared" control flag `FAST` if you think having that makes sense).
   
   1. By default / for dev builds:
   ```
   SKIP_TESTS: true
   SKIP_OS_UPDATE: true
   ```
   So that if you choose to use `FAST` to control both it can be set to `true` by default.
   
   2. For the CI / automated builds:
   ```
   SKIP_TESTS: false
   SKIP_OS_UPDATE: false
   ```
   So that if you choose to use `FAST` to control both then the CI could set this flag to `false`.
   
   If you choose to not introduce a shared flag, then I slightly prefer calling the flag you introduce here `SKIP_OS_UPDATE`.


-- 
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@flink.apache.org

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