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

Document -Duser.timezone=time_zone_code for a customer issue. #7320

Closed
chirp1 opened this issue May 10, 2024 · 4 comments · Fixed by #7338
Closed

Document -Duser.timezone=time_zone_code for a customer issue. #7320

chirp1 opened this issue May 10, 2024 · 4 comments · Fixed by #7338
Assignees
Labels
doc bug Something isn't working.
Milestone

Comments

@chirp1
Copy link
Contributor

chirp1 commented May 10, 2024

Please describe the problem you are having with the documentation. Is information missing, inaccurate, or unclear? Tell us about the context where you encountered the problem so we can understand how to address it.

To address a customer issue, make the following updates in https://openliberty.io/docs/latest/reference/config/server-configuration-overview.html#jvm-options

Add the line in bold:
Common uses of jvm.options files include:
Setting JVM memory limits
Enabling Java Agents that are provided by monitoring products
Setting Java System Properties
Setting time zones

Add the line in bold:
The jvm.options file format uses one line per JVM option, as shown in the following example:
-Xmx512m
-Dmy.system.prop=This is the value.
-Duser.timezone=time_zone_code


Add the following text to https://openliberty.io/docs/latest/log-trace-configuration.html

You can use the user.timezone property to set the time zone for your application environments. Set the user.timezone property in the jvm.properties file. The updated time zone information is displayed in the messages.log file and the trace.log file. The following example shows how to set the time zone.
-Duser.timezone=time_zone_code

@dmuelle dmuelle self-assigned this May 11, 2024
dmuelle added a commit that referenced this issue May 14, 2024
@dmuelle
Copy link
Member

dmuelle commented May 14, 2024

Hi @chirp1 - the draft is now updated with these changes

https://docs-draft-openlibertyio.mqj6zf7jocq.us-south.codeengine.appdomain.cloud/docs/latest/reference/config/server-configuration-overview.html#jvm-options

if any further changes are needed, just let me know. Thanks

@chirp1
Copy link
Contributor Author

chirp1 commented May 15, 2024

@dmuelle Hi David, Thanks! The update looks good to me.

@Channyboy
Copy link
Contributor

@dmuelle @chirp1 I think it would make sense to also update the log trace and configuration page? (i.e., https://openliberty.io/docs/latest/log-trace-configuration.html)

Similar to the work done for the KC update.

@dmuelle
Copy link
Member

dmuelle commented May 16, 2024

Hi @Channyboy - I added this example to the end of the first section, in the same place it appears in IBM Docs

https://docs-draft-openlibertyio.mqj6zf7jocq.us-south.codeengine.appdomain.cloud/docs/latest/log-trace-configuration.html

@dmuelle dmuelle added this to the 24.0.0.5 milestone May 16, 2024
@dmuelle dmuelle added the doc bug Something isn't working. label May 16, 2024
This was referenced May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc bug Something isn't working.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants