{"action":"opened","issue":{"url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/434","repository_url":"https://api.github.com/repos/projectatomic/atomic-reactor","labels_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/434/labels{/name}","comments_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/434/comments","events_url":"https://api.github.com/repos/projectatomic/atomic-reactor/issues/434/events","html_url":"https://github.com/projectatomic/atomic-reactor/issues/434","id":141566688,"number":434,"title":"plugins: use ContainerConfig to get Labels from docker inspect","user":{"login":"mmilata","id":85857,"avatar_url":"https://avatars.githubusercontent.com/u/85857?v=3","gravatar_id":"","url":"https://api.github.com/users/mmilata","html_url":"https://github.com/mmilata","followers_url":"https://api.github.com/users/mmilata/followers","following_url":"https://api.github.com/users/mmilata/following{/other_user}","gists_url":"https://api.github.com/users/mmilata/gists{/gist_id}","starred_url":"https://api.github.com/users/mmilata/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmilata/subscriptions","organizations_url":"https://api.github.com/users/mmilata/orgs","repos_url":"https://api.github.com/users/mmilata/repos","events_url":"https://api.github.com/users/mmilata/events{/privacy}","received_events_url":"https://api.github.com/users/mmilata/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-03-17T12:18:50Z","updated_at":"2016-03-17T12:18:50Z","closed_at":null,"body":"New base images are going to store labels in `inspect_result['ContainerConfig']['Labels']`. We need to look there instead of `inspect_result['Config']['Labels']`. The `pre_add_labels_in_df` plugin is mainly affected.\r\n\r\nNotes:\r\n* Until now, base images had the labels in `inspect_result['Config']['Labels']` and have `inspect_result['ContainerConfig']['Labels']` empty.\r\n* The result of `docker build` seems to always have both `Config` and `ContainerConfig` populated identically.\r\n* https://github.com/docker/docker/blob/9842c7a741ee480c71bfd8e71e1f22893d010411/image/image.go#L33\r\n\r\nCC @eliskasl"}}