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

envFrom does not propagate to master #560

Closed
rosscdh opened this issue May 5, 2021 · 3 comments · Fixed by #564
Closed

envFrom does not propagate to master #560

rosscdh opened this issue May 5, 2021 · 3 comments · Fixed by #564
Labels
enhancement New feature or request
Milestone

Comments

@rosscdh
Copy link

rosscdh commented May 5, 2021

Describe the solution you'd like
envFrom should propagate to the master pod so that env vars can be used in JSasC setups

Describe alternatives you've considered
when JSasC runs it needs access to env vars that may contain secrets (and thus come from secrets of configmaps) envFrom is supported on the Jenkins scheme but do not propagate tot eh generated master pod.

ie.. instead of

            - gitLabApiTokenImpl:
                scope: GLOBAL
                id: gitlab-cloud-token
                apiToken: "xxxxXXXXXXXX"
                description: "Gitlab Cloud Token for api access"

which breaks good practice and enterprise governance rules

            - gitLabApiTokenImpl:
                scope: GLOBAL
                id: gitlab-cloud-token
                apiToken: "${GITLAB_TOKEN}"
                description: "Gitlab Cloud Token for api access"

which is then referenced on the Jenkins resource as

apiVersion: jenkins.io/v1alpha2
kind: Jenkins
metadata:
  name: jenkins
  namespace: jenkins
spec:
  master:
    containers:
      - name:  jenkins-master
        envFrom:
        - secretRef:
            name: jenkins-gitlab-token
@rosscdh rosscdh added the enhancement New feature or request label May 5, 2021
@SylwiaBrant
Copy link

Hello. You gave an example of declaring variables out in the open. Does this approach not work?
https://kubernetes.io/docs/tasks/configure-pod-container/configure-pod-configmap/#define-container-environment-variables-using-configmap-data

@prryb prryb added this to the 0.6.0 milestone May 11, 2021
prryb pushed a commit that referenced this issue May 11, 2021
* Fixed envFrom not propagate to master pod
@prryb prryb reopened this May 11, 2021
@rosscdh
Copy link
Author

rosscdh commented May 26, 2021

Great to see this being addressed. May i ask when 0.6.0 is expected?

@KorusMateusz
Copy link

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

Successfully merging a pull request may close this issue.

4 participants