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

GatewayParameters imagePullSecrets definition unclear #10209

Open
huzlak opened this issue Oct 15, 2024 · 0 comments
Open

GatewayParameters imagePullSecrets definition unclear #10209

huzlak opened this issue Oct 15, 2024 · 0 comments
Labels
Type: Bug Something isn't working

Comments

@huzlak
Copy link
Contributor

huzlak commented Oct 15, 2024

Gloo Edge Product

Enterprise

Gloo Edge Version

v1.17.3

Kubernetes Version

1.28

Describe the bug

When setting up gatewayparameters via helm chart, I can not specify imagePullSecrets, even though helm api reference states there is gloo.kubeGateway.gatewayParameters.glooGateway.envoyContainer.image.pullSecret this option seems to have no effect. In kubernetes this is defined per pod and based on gatewayparameters crd, there's no such configuration in the image section, but it should be part of podTemplate:

$ kubectl explain gatewayparameters.spec.kube.envoyContainer.image
GROUP:      gateway.gloo.solo.io
KIND:       GatewayParameters
VERSION:    v1alpha1

FIELD: image <Object>

DESCRIPTION:
    <empty>
FIELDS:
  digest	<string>
    <no description>

  pullPolicy	<string>
    <no description>

  registry	<string>
    <no description>

  repository	<string>
    <no description>

  tag	<string>
    <no description>
$ kubectl explain gatewayparameters.spec.kube.podTemplate
GROUP:      gateway.gloo.solo.io
KIND:       GatewayParameters
VERSION:    v1alpha1

FIELD: podTemplate <Object>

DESCRIPTION:
    <empty>
FIELDS:
  affinity	<Object>
    <no description>

  extraAnnotations	<map[string]string>
    <no description>

  extraLabels	<map[string]string>
    <no description>

  imagePullSecrets	<[]Object>
    <no description>

  nodeSelector	<map[string]string>
    <no description>

  securityContext	<Object>
    <no description>

  tolerations	<[]Object>
    <no description>

Our chart is missing that in the podTemplate so it's currently not possible to define pullsecret via helm

Expected Behavior

I'd like to have an option to set the image pull secret for the gateway pods via helm chart

Steps to reproduce the bug

  1. Install gloo gw with values:
  kubeGateway:
    enabled: true
    gatewayParameters:
      glooGateway:
        envoyContainer:
          image:
            pullSecret: secret
  1. See that no pull secret is set for the gateway pods
  2. Add below into podTemplate inside gatewayparameters manually:
imagePullSecrets:
- name: secret
  1. Pods have the imagePullSecret set now

Additional Environment Detail

No response

Additional Context

No response

@huzlak huzlak added the Type: Bug Something isn't working label Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant