{"action":"created","issue":{"url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/51","repository_url":"https://api.github.com/repos/openshift/cluster-version-operator","labels_url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/51/labels{/name}","comments_url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/51/comments","events_url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/51/events","html_url":"https://github.com/openshift/cluster-version-operator/pull/51","id":380474988,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMwNjY0ODU1","number":51,"title":"WIP: start: Resolve the release image to an exact value instead of using tag","user":{"login":"smarterclayton","id":1163175,"node_id":"MDQ6VXNlcjExNjMxNzU=","avatar_url":"https://avatars0.githubusercontent.com/u/1163175?v=4","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"labels":[{"id":1005843873,"node_id":"MDU6TGFiZWwxMDA1ODQzODcz","url":"https://api.github.com/repos/openshift/cluster-version-operator/labels/approved","name":"approved","color":"0ffa16","default":false},{"id":1005846184,"node_id":"MDU6TGFiZWwxMDA1ODQ2MTg0","url":"https://api.github.com/repos/openshift/cluster-version-operator/labels/do-not-merge/work-in-progress","name":"do-not-merge/work-in-progress","color":"e11d21","default":false},{"id":1005849106,"node_id":"MDU6TGFiZWwxMDA1ODQ5MTA2","url":"https://api.github.com/repos/openshift/cluster-version-operator/labels/size/M","name":"size/M","color":"eebb00","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-13T23:49:17Z","updated_at":"2018-11-13T23:56:01Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/openshift/cluster-version-operator/pulls/51","html_url":"https://github.com/openshift/cluster-version-operator/pull/51","diff_url":"https://github.com/openshift/cluster-version-operator/pull/51.diff","patch_url":"https://github.com/openshift/cluster-version-operator/pull/51.patch"},"body":"If the CVO is started with a tag, it's possible the tag can drift from the\r\ninitial value and the cluster would start updating at a random time. Instead,\r\nduring startup the CVO attempts to identify the precise pull spec as\r\nKubernetes reports it and use that as the release version. When the reconciler\r\nruns we will update ourself to that version.\r\n\r\nThis is one way to approach the problem (there are others) that relies on\r\nthe values Kubernetes knows about itself to be correct. If the imageID\r\nvalue reported by the container runtime is unstable, we would loop, but\r\nimageID being unstable is really a bug:\r\n\r\n    quay.io/openshift/ocp-release:4.0.X ->\r\n    quay.io/openshift/ocp-release@sha256:XXXX ->\r\n    secure.quay.io/openshift/ocp-release@sha256:XXXX ->\r\n    quay.io/openshift/ocp-release@sha256:XXXX ->\r\n    ... // forever\r\n\r\nOtherwise, this has the nice property of being self converging even if\r\nthe user gives us wierd output.\r\n\r\nTODO:\r\n\r\n* [ ] do this when we download the next payload and resolve the tag id"},"comment":{"url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/comments/438484287","html_url":"https://github.com/openshift/cluster-version-operator/pull/51#issuecomment-438484287","issue_url":"https://api.github.com/repos/openshift/cluster-version-operator/issues/51","id":438484287,"node_id":"MDEyOklzc3VlQ29tbWVudDQzODQ4NDI4Nw==","user":{"login":"openshift-ci-robot","id":30189103,"node_id":"MDQ6VXNlcjMwMTg5MTAz","avatar_url":"https://avatars1.githubusercontent.com/u/30189103?v=4","gravatar_id":"","url":"https://api.github.com/users/openshift-ci-robot","html_url":"https://github.com/openshift-ci-robot","followers_url":"https://api.github.com/users/openshift-ci-robot/followers","following_url":"https://api.github.com/users/openshift-ci-robot/following{/other_user}","gists_url":"https://api.github.com/users/openshift-ci-robot/gists{/gist_id}","starred_url":"https://api.github.com/users/openshift-ci-robot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/openshift-ci-robot/subscriptions","organizations_url":"https://api.github.com/users/openshift-ci-robot/orgs","repos_url":"https://api.github.com/users/openshift-ci-robot/repos","events_url":"https://api.github.com/users/openshift-ci-robot/events{/privacy}","received_events_url":"https://api.github.com/users/openshift-ci-robot/received_events","type":"User","site_admin":false},"created_at":"2018-11-13T23:56:01Z","updated_at":"2018-11-13T23:56:01Z","author_association":"NONE","body":"@smarterclayton: The following tests **failed**, say `/retest` to rerun them all:\n\nTest name | Commit | Details | Rerun command\n--- | --- | --- | ---\nci/prow/e2e-aws | f0c9f612f95a5e76ed3860315f41fd9c7566e44f | [link](https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_cluster-version-operator/51/pull-ci-openshift-cluster-version-operator-master-e2e-aws/98) | `/test e2e-aws`\nci/prow/images | f0c9f612f95a5e76ed3860315f41fd9c7566e44f | [link](https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_cluster-version-operator/51/pull-ci-openshift-cluster-version-operator-master-images/103) | `/test images`\n\n[Full PR test history](https://openshift-gce-devel.appspot.com/pr/openshift_cluster-version-operator/51). [Your PR dashboard](https://openshift-gce-devel.appspot.com/pr/smarterclayton). Please help us cut down on flakes by [linking to](https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests) an [open issue](https://github.com/openshift/cluster-version-operator/issues?q=is:issue+is:open) when you hit one in your PR.\n\n<details>\n\nInstructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md).  If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).\n</details>\n<!-- test report -->"}}
{"actor":{"display_login":"openshift-ci-robot"}}