Skip to content

Create log files for components#173

Merged
1000TurquoisePogs merged 8 commits intov3.x/stagingfrom
feature/v3/component-log-files
Apr 9, 2026
Merged

Create log files for components#173
1000TurquoisePogs merged 8 commits intov3.x/stagingfrom
feature/v3/component-log-files

Conversation

@1000TurquoisePogs
Copy link
Copy Markdown
Member

@1000TurquoisePogs 1000TurquoisePogs commented Nov 12, 2025

Currently, app-server & zss create log files.
Any other component or extension may or may not.
This causes people to think the log directory contains useful information, when it often doesnt.

This PR changes that by just having the launcher make log files for components.
The launcher already captures all output of components, so in this PR:

  1. each component startup, a log file name is determined, the N newest files of that pattern are retained but others deleted
  2. every message the launcher receives from a component is written to that file
  3. each component stop, the log file is closed

New property zowe.logging.rolloverCount is used to determine how many files to keep.

log files are named

<zowe.logDirectory>/<zowe.job.prefix>_<haInstance>_<component>_<timestamp>.log

As you can see from the screenshot, names are lengthy but unique.
This allows for 1 log folder to be used by multiple zowes.

image

I have not implemented, but setting rolloverCount to 0 or a similar option could be a way to disable logging entirely if desired???

…er HA instance

Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
1000TurquoisePogs and others added 3 commits November 12, 2025 15:28
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
@struga0258
Copy link
Copy Markdown
Contributor

I tested this pr and seems to be working as expected, the only issue is that we need to remove the old logs from being generated. We have 2 sets of zss and app-server logs.

Comment thread src/main.c
Comment thread src/main.c
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
JoeNemo
JoeNemo previously approved these changes Mar 24, 2026
Copy link
Copy Markdown
Contributor

@JoeNemo JoeNemo left a comment

Choose a reason for hiding this comment

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

Approved.

Comment thread src/main.c
Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
@1000TurquoisePogs 1000TurquoisePogs merged commit 636f6b6 into v3.x/staging Apr 9, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants