Skip to content

Add installation instructions for OpenShift#335

Merged
tekton-robot merged 1 commit into
tektoncd:mainfrom
concaf:patch-1
Jan 19, 2022
Merged

Add installation instructions for OpenShift#335
tekton-robot merged 1 commit into
tektoncd:mainfrom
concaf:patch-1

Conversation

@concaf

@concaf concaf commented Jan 16, 2022

Copy link
Copy Markdown
Contributor

This commit documents the steps required to install Chains on OpenShift.
These steps are pretty similar to what's required for other components like Pipeline - https://github.com/tektoncd/pipeline/blob/main/docs/install.md#installing-tekton-pipelines-on-openshift

CC: @vdemeester @sbose78

@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 16, 2022
This commit documents the steps required to install Chains on OpenShift.
@cooktheryan

Copy link
Copy Markdown

thanks @concaf !!!

Comment thread README.md
#### Installation on OpenShift
Prerequisite: you'll need [Tekton Pipelines](https://github.com/tektoncd/pipeline/blob/main/docs/install.md#installing-tekton-pipelines-on-openshift) installed on your OpenShift cluster before you install Chains.

To install Tekton Chains on OpenShift, you must first apply the `anyuid` security

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it required for the chains controller to run as root ? or to run as a specific user ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was getting permission denied errors without doing this, so yes I guess 😅

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.

The recent version needs to run as root. I had to specify runAsUser: 0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah ok 👍🏼 We might want to see how to enhance this in the future (cc @priyawadhwa) but for now, let's document it this way 👼🏼

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@vdemeester isn't this the case with pipeline and triggers as well!? 🤔

@priyawadhwa

Copy link
Copy Markdown
Contributor

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 18, 2022
@tekton-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: priyawadhwa, vdemeester

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

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 19, 2022
@tekton-robot tekton-robot merged commit 7265681 into tektoncd:main Jan 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants