)]}'
{"id":"openstack%2Fopenstack-helm~559208","triplet_id":"openstack%2Fopenstack-helm~master~I8211874fb32f58e23578f0cc0dbb2e496cc23cea","project":"openstack/openstack-helm","branch":"master","topic":"rfc-dynamic-file-overlay","hashtags":[],"change_id":"I8211874fb32f58e23578f0cc0dbb2e496cc23cea","subject":"[RFC] generic file-overlay mechanism","status":"ABANDONED","created":"2018-04-06 01:24:56.000000000","updated":"2020-07-15 19:28:55.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"51ab13185bb8972aab5fd2ba169a30dfbb7db996","_number":559208,"virtual_id_number":559208,"owner":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},{"value":-1,"date":"2018-04-06 02:59:06.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"rejected":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"all":[{"value":-1,"date":"2018-04-06 01:27:28.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"date":"2018-04-06 01:26:47.000000000","_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"date":"2018-04-06 01:26:33.000000000","_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"value":0,"date":"2018-04-06 01:26:43.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"date":"2018-04-06 04:20:10.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2018-04-06 01:26:29.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2018-04-06 01:26:29.000000000","updated_by":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"reviewer":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"state":"REVIEWER"},{"updated":"2018-04-06 01:26:33.000000000","updated_by":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"reviewer":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"state":"REVIEWER"},{"updated":"2018-04-06 01:26:43.000000000","updated_by":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"reviewer":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"state":"REVIEWER"},{"updated":"2018-04-06 01:26:47.000000000","updated_by":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"reviewer":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"state":"REVIEWER"},{"updated":"2018-04-06 02:59:06.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2018-04-06 04:20:10.000000000","updated_by":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"reviewer":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"state":"REVIEWER"}],"messages":[{"id":"c9f947be506c73ad66587c3b7db33c375f76b633","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2018-04-06 01:24:56.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"49ca2f54d6891386dfa9aadeb3f9b0f822e60e3c","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2018-04-06 01:27:28.000000000","message":"Patch Set 1: Workflow-1","accounts_in_message":[],"_revision_number":1},{"id":"e60bae700b73f4fb092f2e80fac244514167915c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-04-06 02:59:06.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- build-openstack-sphinx-docs http://logs.openstack.org/08/559208/1/check/build-openstack-sphinx-docs/1248fbb/html/ : SUCCESS in 3m 46s\n- openstack-helm-linter http://logs.openstack.org/08/559208/1/check/openstack-helm-linter/5b1bfba/ : SUCCESS in 1m 09s\n- openstack-helm-dev-deploy-ceph-radosgw-ovs http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-ceph-radosgw-ovs/8011b16/ : POST_FAILURE in 14m 16s\n- openstack-helm-dev-deploy-ceph-rbd-ovs http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-ceph-rbd-ovs/a180e2f/ : FAILURE in 45m 47s\n- openstack-helm-dev-deploy-nfs-ovs http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-nfs-ovs/a4a291a/ : FAILURE in 44m 51s\n- openstack-helm-dev-deploy-ceph-lb http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-ceph-lb/1036649/ : FAILURE in 51m 15s (non-voting)\n- openstack-helm-dev-deploy-nfs-lb http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-nfs-lb/adb9f4a/ : FAILURE in 39m 22s (non-voting)\n- openstack-helm-dev-deploy-nfs-ldap http://logs.openstack.org/08/559208/1/check/openstack-helm-dev-deploy-nfs-ldap/1967a7d/ : FAILURE in 39m 54s (non-voting)\n- openstack-helm-multinode-ubuntu http://logs.openstack.org/08/559208/1/check/openstack-helm-multinode-ubuntu/816971d/ : FAILURE in 58m 35s (non-voting)\n- openstack-helm-ironic-ubuntu http://logs.openstack.org/08/559208/1/check/openstack-helm-ironic-ubuntu/a313541/ : FAILURE in 58m 49s (non-voting)\n- openstack-helm-armada-fullstack-deploy http://logs.openstack.org/08/559208/1/check/openstack-helm-armada-fullstack-deploy/64a80ec/ : FAILURE in 1h 28m 23s (non-voting)\n- openstack-helm-kolla-deploy-ceph-ovs http://logs.openstack.org/08/559208/1/check/openstack-helm-kolla-deploy-ceph-ovs/f9c88d7/ : FAILURE in 47m 05s\n- openstack-helm-ocata-loci-deploy-ceph http://logs.openstack.org/08/559208/1/check/openstack-helm-ocata-loci-deploy-ceph/35fc171/ : FAILURE in 47m 26s (non-voting)\n- openstack-helm-ocata-kolla-deploy-ceph http://logs.openstack.org/08/559208/1/check/openstack-helm-ocata-kolla-deploy-ceph/0bc213b/ : FAILURE in 48m 18s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"99da15e5c2938eccd0b6785beb5f6b3be99688f9","author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"date":"2018-04-06 04:18:03.000000000","message":"Patch Set 1:\n\nThere is a pattern now for processing gotpl within values.  Look at some of the heat script bootstrap examples floating around. This may be what you are looking for.","accounts_in_message":[],"_revision_number":1},{"id":"8785039dc0531d988a000b7aec4cb1b3be4b6518","author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"date":"2018-04-06 04:20:10.000000000","message":"Patch Set 1:\n\nAnd I like this as a generic mechanism to provide operators the ability to ‘hot patch’ containers.  I’m not sure that was the full original intention but it strikes me as quite useful in that space.","accounts_in_message":[],"_revision_number":1},{"id":"38d486438773e205aa91963f572e65f61cbbdb21","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2020-07-15 19:28:55.000000000","message":"Abandoned\n\nAbandoning since there hasn\u0027t been any recent activity, if anyone wants to continue this work, please feel free to restore this.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"177eaee474e2a53c33d1468373bc87041bf16ce0","revisions":{"177eaee474e2a53c33d1468373bc87041bf16ce0":{"kind":"REWORK","_number":1,"created":"2018-04-06 01:24:56.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/08/559208/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/08/559208/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/08/559208/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/08/559208/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/08/559208/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/08/559208/1"}}},"commit":{"parents":[{"commit":"44841fbffcf75321243cd2352025b5f33d84feca","subject":"Storage: increase robustness of storage clean jobs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/44841fbffcf75321243cd2352025b5f33d84feca"}]}],"author":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2018-04-06 01:16:20.000000000","tz":0},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2018-04-06 01:16:20.000000000","tz":0},"subject":"[RFC] generic file-overlay mechanism","message":"[RFC] generic file-overlay mechanism\n\nThis is a very rough *non-working* PoC for illustration purposes and\nfeedback on the general direction.\n\nThe suggestion here is we defined (in values) fileoverlay { \"key\": {\n\"path\": ..., \"readOnly\": ..., \"contents\": ... } }, for example:\n\nfileoverlay:\n  passwd:\n    path: /etc/passwd\n    readOnly: true\n    contents: |\n      root:x:0:0:root:/root:/bin/bash\n      daemon:*:1:1:daemon:/usr/sbin:/usr/sbin/nologin\n\nthen using an iterator in configmap-fileoverlay.yaml and another in\nthe pod definition(s) create \u0027volumes\u0027 to overlay each of those files.\n\nRight now this patch doesn\u0027t work, I suspect it\u0027s because the yaml\nparser in helm doesn\u0027t accept this construct:\n\nfoo: |\n  # line 1\n  # line 2 {{ $gotpl_stuff }}\n\nin a way that line 2 is processed as expected.\n\nIf this is fixable and this direction tolerable then the goal would be\nto move this login in a generic way to the helm-toolkit.\n\nChange-Id: I8211874fb32f58e23578f0cc0dbb2e496cc23cea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/177eaee474e2a53c33d1468373bc87041bf16ce0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/177eaee474e2a53c33d1468373bc87041bf16ce0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}