tektoncd / results

Long term storage of execution results.
Apache License 2.0
77 stars 74 forks source link

Passing Storage(S3, AWS) keys via kubernetes secrets #469

Closed khrm closed 1 year ago

khrm commented 1 year ago

Feature request

At present, we are passing S3 secrets as config. We need to use secrets to pass these configurations.

jkandasa commented 1 year ago

It is possible to inject custom secret values as env into tekton-results-api deployment as follows, I think no additional changes required in results code base.

apiVersion: apps/v1
kind: Deployment
metadata:
  name: tekton-results-api
  namespace: tekton-pipelines
spec:
  replicas: 1
  selector:
    # omitted ...
    spec:
      containers:
        - env:
            # omitted ...
          image: gcr.io/tekton-releases/github.com/tektoncd/results/cmd/api:v0.6.0@sha256:e3760a8fb8e8c056e247595d560dfc9aab38cee4a9c512fe5c198c7b88a923bd
          name: api
          envFrom: # supply your custom secret keys as env variable
            - secretRef:
                name: my_custom_s3_secret
                optional: true

I have created a PR in operator to support custom secret: https://github.com/tektoncd/operator/pull/1547

gabemontero commented 1 year ago

I think this is more or less a duplicate of the https://github.com/tektoncd/results/issues/432 feature I opened a month earlier, is it not @khrm ?

khrm commented 1 year ago

Yes. We can close this. I didn't notice this.