{"action":"created","issue":{"url":"https://api.github.com/repos/helm/helm/issues/4595","repository_url":"https://api.github.com/repos/helm/helm","labels_url":"https://api.github.com/repos/helm/helm/issues/4595/labels{/name}","comments_url":"https://api.github.com/repos/helm/helm/issues/4595/comments","events_url":"https://api.github.com/repos/helm/helm/issues/4595/events","html_url":"https://github.com/helm/helm/issues/4595","id":357353343,"node_id":"MDU6SXNzdWUzNTczNTMzNDM=","number":4595,"title":"Helm upgrade with reuse-values, without --set gives YAML error","user":{"login":"timgclark","id":5014080,"node_id":"MDQ6VXNlcjUwMTQwODA=","avatar_url":"https://avatars1.githubusercontent.com/u/5014080?v=4","gravatar_id":"","url":"https://api.github.com/users/timgclark","html_url":"https://github.com/timgclark","followers_url":"https://api.github.com/users/timgclark/followers","following_url":"https://api.github.com/users/timgclark/following{/other_user}","gists_url":"https://api.github.com/users/timgclark/gists{/gist_id}","starred_url":"https://api.github.com/users/timgclark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timgclark/subscriptions","organizations_url":"https://api.github.com/users/timgclark/orgs","repos_url":"https://api.github.com/users/timgclark/repos","events_url":"https://api.github.com/users/timgclark/events{/privacy}","received_events_url":"https://api.github.com/users/timgclark/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-05T18:37:41Z","updated_at":"2018-09-05T18:45:44Z","closed_at":null,"author_association":"NONE","body":"Output of `helm version`:\r\n```\r\nClient: &version.Version{SemVer:\"v2.10.0\", GitCommit:\"9ad53aac42165a5fadc6c87be0dea6b115f93090\", GitTreeState:\"clean\"}\r\nServer: &version.Version{SemVer:\"v2.10.0\", GitCommit:\"9ad53aac42165a5fadc6c87be0dea6b115f93090\", GitTreeState:\"clean\"}\r\n```\r\n\r\nOutput of `kubectl version`:\r\n```\r\nClient Version: version.Info{Major:\"1\", Minor:\"10\", GitVersion:\"v1.10.1\", GitCommit:\"d4ab47518836c750f9949b9e0d387f20fb92260b\", GitTreeState:\"clean\", BuildDate:\"2018-04-12T14:26:04Z\", GoVersion:\"go1.9.3\", Compiler:\"gc\", Platform:\"linux/amd64\"}\r\nServer Version: version.Info{Major:\"1\", Minor:\"10\", GitVersion:\"v1.10.1\", GitCommit:\"d4ab47518836c750f9949b9e0d387f20fb92260b\", GitTreeState:\"clean\", BuildDate:\"2018-04-12T14:14:26Z\", GoVersion:\"go1.9.3\", Compiler:\"gc\", Platform:\"linux/amd64\"}\r\n```\r\n\r\nCloud Provider/Platform (AKS, GKE, Minikube etc.):\r\nInstalled via kubeadm\r\n\r\nAs previously reported (https://github.com/helm/helm/issues/4337#issuecomment-415603340) #4337 seems to still impact v2.10.0.\r\n\r\n`helm upgrade --reuse-values` always seems to fail unless `--set` is also passed to helm with literally *any* key-value:\r\n\r\n```\r\n$ helm upgrade percona ./percona-0.5.2.tgz --reuse-values\r\nError: UPGRADE FAILED: yaml: line 13: could not find expected ':'\r\n$ helm upgrade percona ./percona-0.5.2.tgz --reuse-values --set foo=bar\r\nRelease \"percona\" has been upgraded. Happy Helming!\r\nLAST DEPLOYED: Wed Sep  5 18:17:54 2018\r\n... \r\n```"},"comment":{"url":"https://api.github.com/repos/helm/helm/issues/comments/418838844","html_url":"https://github.com/helm/helm/issues/4595#issuecomment-418838844","issue_url":"https://api.github.com/repos/helm/helm/issues/4595","id":418838844,"node_id":"MDEyOklzc3VlQ29tbWVudDQxODgzODg0NA==","user":{"login":"bacongobbler","id":1360539,"node_id":"MDQ6VXNlcjEzNjA1Mzk=","avatar_url":"https://avatars2.githubusercontent.com/u/1360539?v=4","gravatar_id":"","url":"https://api.github.com/users/bacongobbler","html_url":"https://github.com/bacongobbler","followers_url":"https://api.github.com/users/bacongobbler/followers","following_url":"https://api.github.com/users/bacongobbler/following{/other_user}","gists_url":"https://api.github.com/users/bacongobbler/gists{/gist_id}","starred_url":"https://api.github.com/users/bacongobbler/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bacongobbler/subscriptions","organizations_url":"https://api.github.com/users/bacongobbler/orgs","repos_url":"https://api.github.com/users/bacongobbler/repos","events_url":"https://api.github.com/users/bacongobbler/events{/privacy}","received_events_url":"https://api.github.com/users/bacongobbler/received_events","type":"User","site_admin":false},"created_at":"2018-09-05T18:45:44Z","updated_at":"2018-09-05T18:45:44Z","author_association":"MEMBER","body":"interesting. I think we fixed this in #4364, but looking at the commit log, it looks like it got missed in the mix and isn't in the 2.10 release. Terribly sorry about that!\r\n\r\nCan you try the canary release and see if it's been fixed there? We are closing out on a 2.11 release so we should have a new release out soon. Sorry about that!\r\n\r\nhttps://docs.helm.sh/using_helm/#from-canary-builds"}}
{"actor":{"display_login":"bacongobbler"}}