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

Wrong service selector after switch from Dockerfile to Docker Image #16

Open
Cryptophobia opened this issue Mar 20, 2018 · 0 comments
Open

Comments

@Cryptophobia
Copy link
Member

From @felixbuenemann on August 7, 2017 19:19

Switching an app from Dockerfile to Docker Image deploys can cause the app to become unroutable, because the kubernetes service uses the type=web selector instead of type=cms.

Looking at the api_app table in the deis database shows that the structure column of the affected app contains {"web":0,"cmd":1} which causes the controller to choose the wrong selector.

Manually updating the structure column to {"cmd":1} and re-pulling the image fixes the problem.

The problem was observed in a cluster running workflow v2.14.0 for both the initial Dockerfile deploy and all following releases of the app:

v7    2017-08-07T17:29:32Z    buenemann deployed registry.example.com/buenemann/shop-api-cache
v6    2017-08-07T17:25:01Z    buenemann deployed registry.example.com/buenemann/shop-api-cache
v5    2017-08-07T17:22:29Z    buenemann added IMAGE_PULL_POLICY
v4    2017-05-30T20:01:51Z    buenemann deployed registry.example.com/buenemann/shop-api-cache
v3    2017-05-30T20:01:16Z    buenemann added registry info password, username
v2    2017-05-30T20:00:52Z    buenemann added CACHE_SIZE, PURGE_NET, BACKEND_PORT, BACKEND_HOST, PORT
v1    2017-05-30T19:59:33Z    buenemann created initial release

The app was running fine on v6, but I likely fixed the kubernetes service selector using kubectl edit previously and forgot about it, so the problem came back to to the stale proctype config in the deis database.

This problem looks similar to what was reported in deis/workflow#658 and should have been fixed in workflow v2.12 according to the PR #1201.

So either this fix was not sufficient or there was a regression.

Copied from original issue: deis/controller#1316

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

No branches or pull requests

1 participant