Skip to content

chore: create separate layer for techdocs dependencies and venv creation#4551

Merged
nickboldt merged 4 commits intoredhat-developer:mainfrom
Omar-AlJaljuli:RHIDP-11298
Apr 8, 2026
Merged

chore: create separate layer for techdocs dependencies and venv creation#4551
nickboldt merged 4 commits intoredhat-developer:mainfrom
Omar-AlJaljuli:RHIDP-11298

Conversation

@Omar-AlJaljuli
Copy link
Copy Markdown
Contributor

Description

I moved the techdocs dependencies installations from the final layer in the runner image to a new techdocs layer created. This is a cleaner implementation of brute forcing the deletion of the packages at the final layer, also slightly furthers decreasing the image size.

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

The container image build workflow finished with status: cancelled.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

Image was built and published successfully. It is available at:

Comment thread build/containerfiles/Containerfile Outdated
Co-authored-by: Nick Boldt <nboldt@redhat.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 8, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

Image was built and published successfully. It is available at:

Copy link
Copy Markdown
Member

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

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

/lgtm

@nickboldt
Copy link
Copy Markdown
Member

/override "ci/prow/e2e-ocp-helm"

unrelated test failure, no need to block

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 8, 2026

@nickboldt: Overrode contexts on behalf of nickboldt: ci/prow/e2e-ocp-helm

Details

In response to this:

/override "ci/prow/e2e-ocp-helm"

unrelated test failure, no need to block

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants