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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
