Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Eirini regression on app environment variable handling #76

Closed
michaelgrifalconi opened this issue May 9, 2019 · 3 comments
Closed

Eirini regression on app environment variable handling #76

michaelgrifalconi opened this issue May 9, 2019 · 3 comments

Comments

@michaelgrifalconi
Copy link

Description

Applications deployed with eirini with a special environment variable with a ":" in the name will never end up in a pod.

Example:

env:
  "TE:ST": "bla" 

CF will wait until the health check timeout but no pod gets created and we were not able to find a detailed error other than the timeout.

Steps to reproduce

You can use this application as example
cf push eirini-regression-app

What was expected to happen

The app should be deployed without errors like it happens on CF using diego and the variable should be present

 cf env eirini-regression-app
Getting env variables for app eirini-regression-app in org MYORG / space MYSPACE as MYUSER...
OK

System-Provided:
[...]

User-Provided:
TE:ST: bla

What actually happened

0 of 1 instances running, 1 down
0 of 1 instances running, 1 down
0 of 1 instances running, 1 down
0 of 1 instances running, 1 down
0 of 1 instances running, 1 down
0 of 1 instances running, 1 down
FAILED
Error restarting application: Start app timeout

Additional information

One use case of this unusual variable name would be the following:

"npm_config_@sap:registry":"https://npm.sap.com"

This is to setup npm_config options as described here where the configuration is @sap:registry https://npm.sap.com as described here

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/165910423

The labels on this github issue will be updated when the story is started.

@alex-slynko
Copy link
Contributor

@michaelgrifalconi Unfortunately, this is a bug in Kubernetes. kubernetes/kubernetes#69415

It worth mentioning your use-case in that bug, so it will get more attention. Unfortunately, it won't land in Kubernetes soon.

We don't plan to implement a workaround for this in Eirini in the near future.

@michaelgrifalconi
Copy link
Author

Thank you @alex-slynko for the clarification!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants