Skip to content

[env_op_images] Reduce POST-RUN console output#4051

Open
sdatko wants to merge 1 commit into
openstack-k8s-operators:mainfrom
sdatko:reduce-logs
Open

[env_op_images] Reduce POST-RUN console output#4051
sdatko wants to merge 1 commit into
openstack-k8s-operators:mainfrom
sdatko:reduce-logs

Conversation

@sdatko

@sdatko sdatko commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

This commit suppress verbose console output from Kubernetes API
queries and CRI-O log collection tasks to reduce POST-RUN phase
output by ~80% in Zuul jobs.

The suppressed data that were previously visible in job-output.txt
were actually redundant and already collected in ci-framework-data/
artifacts anyway as:

  • operator_images.yaml
  • crio_logs/*

Keeping job-output.txt smaller reduces impact on the web browser
when opening Zuul build logs.

For debugging purposes logging can be re-enabled by setting
the cifmw_nolog: false variable.

Changes:

  • Add no_log to 4 tasks env_op_images role.
  • Reworked the CRI-O log collection tasks.

Impact:

  • 75% reduction in console output (~60k → ~15k lines)
  • 90% smaller file size (~20 MB → ~2 MB)
  • All data still preserved for subsequent tasks + dumped as artifacts

Assisted-by: Claude Sonnet 4.5

danpawlik
danpawlik previously approved these changes Jul 13, 2026
@danpawlik

Copy link
Copy Markdown
Contributor

nice improvement @sdatko !

evallesp
evallesp previously approved these changes Jul 13, 2026

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/approve

@openshift-ci

openshift-ci Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evallesp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sdatko

sdatko commented Jul 13, 2026

Copy link
Copy Markdown
Contributor Author

FINALLY READY FOR REVIEW

@nemarjan nemarjan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM overall -- the 80% reduction in job-output.txt is a big win, and I like that all the data is still preserved as artifacts. Just left a small suggestions inline, nothing blocking.

Comment thread roles/env_op_images/tasks/main.yml Outdated
This commit suppress verbose console output from Kubernetes API
queries and CRI-O log collection tasks to reduce POST-RUN phase
output by ~80% in Zuul jobs.

The suppressed data that were previously visible in `job-output.txt`
were actually redundant and already collected in ci-framework-data/
artifacts anyway as:
- `operator_images.yaml`
- `crio_logs/*`

Keeping `job-output.txt` smaller reduces impact on the web browser
when opening Zuul build logs.

For debugging purposes logging can be re-enabled by setting
the `cifmw_nolog: false` variable.

Changes:
- Add `no_log` to 4 tasks `env_op_images` role.
- Reworked the CRI-O log collection tasks.

Impact:
- 75% reduction in console output (~60k → ~15k lines)
- 90% smaller file size (~20 MB → ~2 MB)
- All data still preserved for subsequent tasks + dumped as artifacts

Assisted-by: Claude Sonnet 4.5
Signed-off-by: Szymon Datko <sdatko@redhat.com>
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/a8eede1ebcab48e0840ffbdaa179879c

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 25m 45s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 30m 04s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 49m 22s
cifmw-crc-podified-edpm-baremetal-minor-update FAILURE in 2h 12m 22s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 44s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 9m 53s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 00s
cifmw-molecule-env_op_images TIMED_OUT in 31m 20s

@danpawlik

Copy link
Copy Markdown
Contributor

recheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants