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

Include container image short hash in actor record #32

Open
mwvaughn opened this issue Apr 19, 2018 · 0 comments
Open

Include container image short hash in actor record #32

mwvaughn opened this issue Apr 19, 2018 · 0 comments

Comments

@mwvaughn
Copy link

mwvaughn commented Apr 19, 2018

I propose including the short hash for the deployed container in the /actors response to help validate that the correct image is deployed. In theory the org/image:tag|hash should be good enough but a common use finds container with semantic versioning schemes and so may be rebuilt several times while iterating on functionality.

Current

{   "id": "w7LMK0k7JGZZQ",
    "image": "sd2e/agave-test:dev",
    "lastUpdateTime": "2018-04-19 16:32:00.506656" }

Proposed

{    "id": "w7LMK0k7JGZZQ",
    "image": "sd2e/agave-test:dev",
    "image_id": "8e481d5e3679",
    "lastUpdateTime": "2018-04-19 16:32:00.506656"  }

where image_id is the value from docker images sd2e/agave-test:dev:

docker images sd2e/agave-test:dev
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
sd2e/agave-test     dev                 8e481d5e3679        11 minutes ago      629MB
@mwvaughn mwvaughn changed the title Include image "hash" in actor record Include container image short hash in actor record Nov 16, 2018
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

2 participants