[GitHub] [arrow] jonkeane commented on a change in pull request #10703: ARROW-13215: [R] [CI] Add ENV TZ to docker files?

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

jonkeane commented on a change in pull request #10703:
URL: https://github.com/apache/arrow/pull/10703#discussion_r669007854



##########
File path: .github/workflows/r.yml
##########
@@ -89,7 +89,9 @@ jobs:
         run: |
           sudo sysctl -w kernel.core_pattern="core.%e.%p"
           ulimit -c unlimited
-          archery docker run ubuntu-r
+          # Setting a non-default and non-probable Marquesas French Polynesia time
+          # it has both with a .45 offset and very very few people who live there.
+          archery docker run -e TZ=MART ubuntu-r

Review comment:
       Yeah, you're right — this isn't totally parameterized the way we want. 
   
   The outcome _is_ similar to what we want: `TZ` is set to `"UTC"` (really any valid value that's not `""` will get rid of the nuisance warning) and then when during the docker run when the tests are run `TZ` is set to `"MART"` and so the tests are run in a non-UTC timezones (which is when we care about it being an improbable timezone). 
   
   It shouldn't matter what the TZ is set to during the build process, but I'm going to parameterize this fully so that this confusion doesn't bite us elsewhere.




-- 
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: github-unsubscribe@arrow.apache.org

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