Skip to content
This repository was archived by the owner on May 6, 2020. It is now read-only.

DEIS_DEPLOY_PROCFILE_MISSING_REMOVE broken #1320

Closed
Cryptophobia opened this issue Aug 10, 2017 · 4 comments
Closed

DEIS_DEPLOY_PROCFILE_MISSING_REMOVE broken #1320

Cryptophobia opened this issue Aug 10, 2017 · 4 comments

Comments

@Cryptophobia
Copy link
Contributor

Cryptophobia commented Aug 10, 2017

DEIS_DEPLOY_PROCFILE_MISSING_REMOVE was set to true in our controller (quay.io/deis/controller:v2.16.0) and it does not seem to clean up the Procfile processes from the deis api structure object.

After removing the process types from the Procfile and redeploying we still see them in the structure object:

...
    "structure": {
        "web2": 0,
        "worker": 2,
        "web": 1,
        "worker-two": 0
    },
...

EDIT: We tested it 4 times and deployed 4 times to make sure this was so.

@Cryptophobia
Copy link
Contributor Author

Is there another way to remove the procfile types?

@bacongobbler
Copy link
Member

bacongobbler commented Aug 11, 2017

That is intended behaviour. The behaviour is you should not see any pods running in the cluster.

It's merely a scale-down operation. We keep it around so users can scale up those processes if desired.

@Cryptophobia
Copy link
Contributor Author

I see. Thanks!

We were thinking the processes should disappear from the API. What is the best way to remove those zombie processes from the deis-controller API then? Let's say we will never use web2 ever again. Is there a way we can make it disappear from the deis app api structure completely?

@Cryptophobia
Copy link
Contributor Author

Cryptophobia commented Aug 14, 2017

So the problem is that, the structure (processes) are not removed from the app object API call.

This is a problem because if you try to use structure object to send a subsequent API call to /scale endpoint, you would not be able to do it...

{    "detail": "Container type web2 does not exist in application" }

In the above scenario, we are using an API output from one API endpoint to feed into another dependent endpoint and it does not work. Thus, the Procfile variable should remove the process_types from the structure object of app if possible.

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

2 participants