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 multiple refs should be separate #2295

Open
burner1024 opened this issue Dec 14, 2024 · 0 comments
Open

EnvFrom multiple refs should be separate #2295

burner1024 opened this issue Dec 14, 2024 · 0 comments
Labels
bug Something isn't working needs-triage Priority and effort undetermined yet

Comments

@burner1024
Copy link

Description of the bug:

const deploymentEnv = new kplus.EnvFrom(depConfigMap, undefined, depSecret);

Use it in deployment.containers.envFrom. Resulting yaml is

- envFrom:
    - configMapRef:
      name: cmName
      secretRef:
      name: secretName

That won't work:
Deployment.apps "dep" is invalid: spec.template.spec.containers[0].envFrom: Invalid value: "": may not have more than one field specified at a time

Expected output:

- envFrom:
    - configMapRef:
      name: cmName
    - secretRef:
      name: secretName

Reproduction Steps:

See above

Error Log:

Environment:

  • Framework Version: cdk8s-plus-30
  • OS:

Other:


This is 🐛 Bug Report

@burner1024 burner1024 added bug Something isn't working needs-triage Priority and effort undetermined yet labels Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-triage Priority and effort undetermined yet
Projects
None yet
Development

No branches or pull requests

1 participant