Skip to content

Conversation

hrishikesh-nalawade
Copy link
Member

Description

Currently, the API ML logs are generated with UTC timestamps. These changes will allow users to configure the timezone for log timestamps, giving them the option to set it to either their LOCAL or UTC. This configuration can be managed within the zowe.yaml file using the ZWE_zowe_logging_timezone variable.

Linked to #3921

Type of change

  • fix: Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • PR title conforms to commit message guideline ## Commit Message Structure Guideline
  • I have commented my code, particularly in hard-to-understand areas. In JS I did provide JSDoc
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The java tests in the area I was working on leverage @nested annotations
  • Any dependent changes have been merged and published in downstream modules

hrishikesh-nalawade and others added 5 commits July 7, 2025 17:30
Signed-off-by: hrishikesh-nalawade <[email protected]>
Signed-off-by: hrishikesh-nalawade <[email protected]>
Signed-off-by: hrishikesh-nalawade <[email protected]>
…gurable-log-timezone-v2' into hrishikesh-nalawade/GH3921/configurable-log-timezone-v2
Signed-off-by: hrishikesh-nalawade <[email protected]>
hrishikesh-nalawade and others added 3 commits July 16, 2025 15:03
Signed-off-by: hrishikesh-nalawade <[email protected]>
…gurable-log-timezone-v2' into hrishikesh-nalawade/GH3921/configurable-log-timezone-v2
Signed-off-by: hrishikesh-nalawade <[email protected]>
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
55.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

-Dibm.serversocket.recover=true \
-Dfile.encoding=UTF-8 \
-Dlogging.charset.console=${ZOWE_CONSOLE_LOG_CHARSET} \
-Dlogging.timezone=${ZWE_zowe_logging_timezone:UTC} \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

: is invalid syntax, need :-
this appears in each shell script.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pointing that out, updated it!

Copy link

sonarqubecloud bot commented Oct 1, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
55.2% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Signed-off-by: hrishikesh-nalawade <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

2 participants