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

Create buildah-proxy-oci-ta. #1853

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matejonnet
Copy link

@matejonnet matejonnet commented Jan 23, 2025

Middleware (PNC) needs a way to run builds with controlled network access. Instead of blocking all network access (hermetic build) this PR add a Domain Proxy and Indy Sidecar to the buildah-oci task.

The Domain proxy provides an option to get controlled network access from the unshare namespace. Domain proxy allows all network traffic but forces it to go through Indy proxy and Indy maven repository.

Indy sidecar is used to forward all network traffic to Indy http proxy, it's required because Indy http proxy running in OpenShift endpoint cannot be exposed as an OpenShift Route (Ingress) providing proxy functionality.

This PR is replacing the
generic proxy: #1815
domain proxy: #1822

image

@matejonnet matejonnet requested a review from a team as a code owner January 23, 2025 13:57
app.kubernetes.io/version: 0.2.1
build.appstudio.redhat.com/build_type: docker
spec:
description: |-
Copy link
Collaborator

Choose a reason for hiding this comment

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

There is nothing about the proxy or its purpose in the task description.

apiVersion: tekton.dev/v1
kind: Task
metadata:
name: buildah-proxy-oci-ta
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it supposed to be just a buildah task with a generic proxy?

Copy link
Collaborator

@mmorhun mmorhun left a comment

Choose a reason for hiding this comment

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

Could you update the CODEOWNERS file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants