Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Quarkus 1.6][iCal] TimeZone-less DTSTART and DTEND changed to GMT in native mode #838

Closed
ppalaga opened this issue Mar 3, 2020 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Mar 3, 2020

Probably caused by quarkusio/quarkus#5016

As a result, only zoned DTSTART and DTEND work properly.

@ppalaga ppalaga added the bug Something isn't working label Mar 3, 2020
@ppalaga ppalaga changed the title [iCal] TimeZone-less DTSTART and DTEND changed to UTF in native mode [iCal] TimeZone-less DTSTART and DTEND changed to GMT in native mode Mar 19, 2020
@ppalaga
Copy link
Contributor Author

ppalaga commented Jun 17, 2020

This was definitely caused by the fact that with pre-20.1.0 GraalVM versions, the native image inherited the default time zone from the build environment. The GMT time zone seemed to be set by the Quarkus docker container (currently using GraalVM 19.3.1).

When testing with GraalVM 20.1.0 the issue does not occur anymore. From 20.1.0 release notes:

Improved support for timezones: information about all timezones is now included in every image, and the default timezone is correctly initialized at image startup and no longer fixed at image generation time.

The Quarkus builder image was upgraded to 20.1.0 recently, so the quirks in https://github.com/apache/camel-quarkus/blob/master/integration-tests/dataformat/src/main/java/org/apache/camel/quarkus/component/dataformat/it/ICalUtils.java#L87-L90 can be removed after the upgrade to Quarkus 1.6

@ppalaga ppalaga changed the title [iCal] TimeZone-less DTSTART and DTEND changed to GMT in native mode [Quarkus 1.6][iCal] TimeZone-less DTSTART and DTEND changed to GMT in native mode Jun 17, 2020
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 17, 2020
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 26, 2020
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 30, 2020
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 30, 2020
@ppalaga ppalaga added this to the 1.0.0-CR3 milestone Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant