)]}'
{"id":"openstack%2Fopenstack-helm-infra~742312","triplet_id":"openstack%2Fopenstack-helm-infra~master~I6758cefbb17e369d3a39f812d63bd487a08f47f0","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"I6758cefbb17e369d3a39f812d63bd487a08f47f0","subject":"Allow for dynamic ks-user-job name","status":"ABANDONED","created":"2020-07-21 23:17:12.000000000","updated":"2022-05-16 20:44:23.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"abf5b305177f253a6b80f89c1bdb2d299c795aad","_number":742312,"virtual_id_number":742312,"owner":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},{"tag":"autogenerated:zuul:check","value":-1,"date":"2021-08-20 16:42:52.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"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":-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":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},{"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":-1,"max":1},"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"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":{"all":[{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},{"_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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-07-22 02:33:23.000000000","updated_by":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"reviewer":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"state":"REVIEWER"},{"updated":"2020-07-22 06:30:54.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":"2020-08-04 14:49:13.000000000","updated_by":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"reviewer":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"state":"REVIEWER"},{"updated":"2020-08-05 18:36:09.000000000","updated_by":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"reviewer":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"state":"REVIEWER"},{"updated":"2020-08-05 19:37:53.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":"2020-08-12 20:16:06.000000000","updated_by":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"reviewer":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"state":"REVIEWER"},{"updated":"2020-08-12 22:26:00.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":"2021-02-17 22:26:21.000000000","updated_by":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"reviewer":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"state":"REMOVED"}],"messages":[{"id":"ff219d40d67e45e561203bd74165d1a889efb798","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-07-21 23:17:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6a9b1a889b51d627851a4bf64b25583ecb710857","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2020-07-21 23:17:22.000000000","message":"Patch Set 1:\n\nCongratulations, you\u0027ve proposed your first change in OpenDev.\n\nYour submission will now be tested automatically by Zuul, our gatekeeper,\nand reviewed by other friendly developers. They will give you feedback and\nmay require you to refine it.\n\nPeople seldom get their patch approved on the first try, so don\u0027t be\nconcerned if requested to make corrections. Feel free to modify your patch\nand resubmit a new change-set.\n\nPatches often take days (and sometimes weeks) to get reviewed, so be\npatient. Don\u0027t hesitate to ask for help, and answer questions about your\nwork promptly if you can. The more you get to know reviewers and get to be\nknown by them, the smoother the review and approval process will become. The\nfastest way to accomplish this is by reviewing other proposed changes\nyourself: anybody can do it, and it\u0027s a great way to learn the code base.\n\nThanks again for participating in OpenDev, we look forward to seeing you\naround.\n\nWorkflow Guide: https://docs.openstack.org/infra/manual/developers.html","accounts_in_message":[],"_revision_number":1},{"id":"fd14f5984bdd4fab0b914de3557d76f2c08ad73e","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-07-21 23:26:05.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"896dd4ae7d7982a07d445ae13abb3a0b5113ac7d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-22 00:29:04.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/542ea3843cf847bcb3c9fc2dd318373c : SUCCESS in 3m 40s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/21f8d3390a6048ceadbd8c30b26c37fd : SUCCESS in 2m 54s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/b2e1613a0e5f4e13824a887efd38ca91 : SUCCESS in 39m 50s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/bd1163a0a1b7495d8fa688aca030b567 : FAILURE in 49m 53s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/b20e80ddd6354098b0512c7bb446873e : SUCCESS in 56m 45s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/43b0dd9f41ac454c9770f98a5ea8c42e : SUCCESS in 11m 43s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/1c602e44de3f4e1784ceb72e8a4757e2 : SUCCESS in 19m 50s","accounts_in_message":[],"_revision_number":2},{"id":"c0f9d83de32300223174ed8214f53d9130f933df","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-07-22 00:33:52.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"1b81c8413fbe7aede5d6f3a488198a8dff663480","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-22 01:36:22.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/abb19f6d8cc24e1ea6b81454ce7ea4ce : SUCCESS in 3m 58s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/8556165d5a264eb09372ba17ca7a6526 : SUCCESS in 2m 54s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/52d8e72d2c914c8ea6c72830a4cc6099 : SUCCESS in 44m 47s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/f05de490569242db9570756fdacf86d2 : SUCCESS in 44m 35s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/2c258bc7a9e9404da2553cd775a5577f : SUCCESS in 1h 00m 20s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/c76352a60f8b46afa66bb55e6034cd7f : SUCCESS in 13m 52s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/41043420582a4f568e4ae3df89599bab : SUCCESS in 19m 18s","accounts_in_message":[],"_revision_number":2},{"id":"45303ce06664930cbcf4a9656aa893574260dda6","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2020-07-22 02:33:23.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"caba3ecaf0818f2831e5395214b135a4c20a4d5c","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-07-22 06:30:54.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nI am not sure what this gets us other than just specifying another key when the original one works off the serviceName. Nothing added changes the serviceUserPretty which is what feeds into the resource name if unique name is desired.","accounts_in_message":[],"_revision_number":2},{"id":"abc75b07acf691cb8983e23a3c561ceeb02665d8","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-07-22 14:03:36.000000000","message":"Patch Set 2:\n\n\u003e (1 comment)\n \u003e \n \u003e I am not sure what this gets us other than just specifying another\n \u003e key when the original one works off the serviceName. Nothing added\n \u003e changes the serviceUserPretty which is what feeds into the resource\n \u003e name if unique name is desired.\n\n\nKs-user-job gets values from the values.yaml like:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\nSo if I want to use dynamic names, I would need to override those values in every armada chart and it will still fail helm lint because helm lint creates a unique release name so I wouldn’t\nKnow what to put in the values.yaml. I think in situations where the same helm chart will be deployed multiple times in the same namespace, we need to be able to have a unique name while keeping values.yaml and pass linting.","accounts_in_message":[],"_revision_number":2},{"id":"7128deea4175264df515baa4998a1a5a701f836f","author":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"date":"2020-08-04 14:49:13.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"360b16fe35db1db128b2e2dd9fd64bdb2bdcce07","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 16:01:55.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"9a244814c2295f60b7359ec5585f6a26f87e5c37","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-04 17:11:45.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a5d50eb9410243558f609504e4746426 : SUCCESS in 4m 08s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/bf528c7dad9d42c8b79627696b1787ce : SUCCESS in 3m 33s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/a84dabf8308240d2b2ff8fc7ea151b89 : SUCCESS in 43m 55s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/c81d024cd99d4cacaa7dfe84588dd0ed : FAILURE in 37m 53s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/257553048f634f77b220c54b68eb6eb3 : SUCCESS in 1h 00m 27s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/13879e37a2ae44bcb619218eafa190e9 : SUCCESS in 15m 41s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/3fb47839fc8f4edcb3ad52e04935ffee : SUCCESS in 18m 15s","accounts_in_message":[],"_revision_number":2},{"id":"f0523d27f859b5cae80ffd5466dee4663a295fd9","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 17:28:00.000000000","message":"Patch Set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"9a3d68fecc774565d0633b3d25b5b7ec94f2d7d0","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-04 18:23:11.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ee56f1722fe94f10ac6d36058e658fe3 : SUCCESS in 3m 25s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/edb37e49c9e04a25a5c9872e5aec7f59 : SUCCESS in 3m 12s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/2bbb6186920a442f843583f8a4962ce0 : SUCCESS in 39m 28s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/ad401083dcbb4645ad51f44e40dc9e64 : SUCCESS in 38m 59s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/d0f843f21e5c477a8f6d6057e069ffbc : SUCCESS in 54m 07s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/ba77954d900847e4b99acf2597104347 : SUCCESS in 14m 55s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/84a64b76283f47c0969baa66dd594fb3 : SUCCESS in 18m 11s","accounts_in_message":[],"_revision_number":3},{"id":"66fdd402140830b040f3fffb2a9276b4b8d487f4","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 19:50:51.000000000","message":"Patch Set 4: Published edit on patch set 3.","accounts_in_message":[],"_revision_number":4},{"id":"9d6cec62a4e5a269dceefb6babc441f80ab030c7","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 20:01:32.000000000","message":"Patch Set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"4f749f2286088203969b0ee6980ebf33d21c99c4","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-04 20:19:17.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/159d962f3aae4881890c9dd9e280ec05 : SUCCESS in 2m 43s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/d8eb3fca8f58465b9ce7bf2ac69913b6 : FAILURE in 2m 11s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/aacf41abd2b14aeeaece04241a491896 : FAILURE in 12m 15s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/2f63151ac06847bd96117ba4f12ca1c2 : FAILURE in 11m 55s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/ddeb9330709f4309bc9f9d95d4955244 : FAILURE in 11m 29s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/36c3aed2553e475bba56a1223c5591e7 : FAILURE in 10m 07s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/708c96efeafb4c39bc0ce9677f8eaa74 : FAILURE in 11m 26s","accounts_in_message":[],"_revision_number":5},{"id":"944289e78e2774254d9fcaf36016fd6ba4dd0cf1","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 20:40:53.000000000","message":"Patch Set 6: Published edit on patch set 5.","accounts_in_message":[],"_revision_number":6},{"id":"f6c240e802598bb36bd5553fce3b44ffc433e174","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-04 21:59:02.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6ce5b0a44e8640afbf130d166ac9c7bf : SUCCESS in 3m 25s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/1f53016f5549475a88ed8f3a748ff9ff : SUCCESS in 2m 59s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/c00d52b381db4b8698a30b0aba6c4d28 : SUCCESS in 50m 58s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/786818732eb54fd99d0f3be5dafc56cd : SUCCESS in 59m 17s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/6fa801ce85394da795783317df5e38d3 : FAILURE in 1h 16m 43s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/98eb74de5fa9441e9e77fae433d81d36 : SUCCESS in 15m 22s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/5c8726be86474e78899270b25057772b : SUCCESS in 16m 23s","accounts_in_message":[],"_revision_number":6},{"id":"baa3b8c36877d4b034ca25e47076e32885f8b2b2","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-04 22:13:44.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"95fb726e5a076c0434cf7be7084dd4f6082255e6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-04 23:27:08.000000000","message":"Patch Set 6:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ace2ae9aaeb14031b252a045ca59ec18 : SUCCESS in 4m 13s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/e519c52618dc46b3a19c2effe75a5c09 : SUCCESS in 3m 02s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/16e2e29375f1404eb1d9cb1183b59775 : SUCCESS in 35m 05s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/e15327f1e9b04c9d86fcc7ca222dc65b : SUCCESS in 33m 03s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/6053002177f54e10b347674b13a9d6bd : FAILURE in 1h 07m 01s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/a517e893e0114c3790a7b6d33b43652d : SUCCESS in 12m 09s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/a9cc103b200f478a98871af9a47fdf39 : SUCCESS in 13m 49s","accounts_in_message":[],"_revision_number":6},{"id":"2d558cea3ed856f0f60410f37c7637647483321e","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-05 14:40:38.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"bdcfee76c0518d9975fbb80fef4e94493bb593d6","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-05 15:58:49.000000000","message":"Patch Set 6:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8d556ba6ce56487e8de1d68ee53855fe : SUCCESS in 5m 09s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/63cf2e80f7d84c779309749457075cdc : SUCCESS in 2m 36s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/c8ca280e765240ba82a332e3a0ceeafb : SUCCESS in 41m 31s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/a7f7dd11054242698865cac17a495569 : SUCCESS in 38m 12s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/6dd5fb2a5b654c058cd8d61f95b4f103 : FAILURE in 1h 10m 37s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/dfdb13c49999401e8580421c69bbbdcc : SUCCESS in 10m 52s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/cba896aed843462380527530decc6cef : SUCCESS in 15m 53s","accounts_in_message":[],"_revision_number":6},{"id":"e5e75df933e40b84a297e948999502f9a11985c7","author":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"date":"2020-08-05 18:28:00.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"67900c68f203e71dd1628c99cbf7733d98a87766","author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"date":"2020-08-05 18:36:09.000000000","message":"Patch Set 7: Code-Review+1","accounts_in_message":[],"_revision_number":7},{"id":"ce1711bb038f2a2d27b23fc4409d7304ada607b8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-08-05 19:37:53.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/5b9faa57590647219a5723b92054f4ab : SUCCESS in 4m 53s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/f886117de03b4834917b245600e6621d : SUCCESS in 2m 53s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/84650254459543ebbc97ef089af8880a : SUCCESS in 38m 34s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/3cbc9a2958b54c0d984232d9a3926e26 : SUCCESS in 35m 06s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/b95e2116b96043c0b1cc486b9dda471c : SUCCESS in 1h 05m 07s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/7a163231c9ec425fabbef595bab27dba : SUCCESS in 14m 24s (non-voting)\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/924d95ee50ba4e64bd0119f58875c937 : SUCCESS in 17m 15s","accounts_in_message":[],"_revision_number":7},{"id":"1d6a960fd1fdb93a6d3f237a5bab1e1d4f868519","tag":"autogenerated:gerrit:deleteReviewer","author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"date":"2021-02-17 22:26:21.000000000","message":"Removed reviewer Dustin Specker with the following votes:\n\n* Code-Review+1 by Dustin Specker \u003cdustin.specker@att.com\u003e\n","accounts_in_message":[],"_revision_number":7},{"id":"402e83fc0516e9366329015687a1d7c798f47d42","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"date":"2021-08-20 15:20:44.000000000","message":"Patch Set 8: Patch Set 7 was rebased","accounts_in_message":[],"_revision_number":8},{"id":"c5f555b80223a630eef0b9159e893aec7990cd7a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-20 16:42:52.000000000","message":"Patch Set 8: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/63d356ff2e2544c0a69ad2dd7e3cc371 : SUCCESS in 4m 38s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/f274be2514b64150b53ca4af7a041059 : FAILURE in 4m 26s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/095a3b9fe1ff45c5aa2d69e9fa445e38 : SUCCESS in 39m 37s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/cee17e55d0ba4aaaa2a57cb29740f63b : SUCCESS in 44m 33s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/c67c6a0c6f044a648d4e701e82a1056f : FAILURE in 1h 09m 19s\n- openstack-helm-infra-openstack-support-ssl https://zuul.opendev.org/t/openstack/build/02a1fdab54074e3696ab6988dcc4d1a1 : SUCCESS in 49m 30s\n- openstack-helm-infra-metacontroller https://zuul.opendev.org/t/openstack/build/7f3e69572dc04c00a3a158b9926efea9 : SUCCESS in 13m 39s","accounts_in_message":[],"_revision_number":8},{"id":"abf5b305177f253a6b80f89c1bdb2d299c795aad","tag":"autogenerated:gerrit:abandon","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2022-05-16 20:44:23.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 or create a new change.","accounts_in_message":[],"_revision_number":8}],"current_revision_number":8,"current_revision":"816790f7e49e3bb26a822591cd912536f9b4eb49","revisions":{"0357c2eb03ec52dd92454abd773327004defe58b":{"kind":"REWORK","_number":1,"created":"2020-07-21 23:17:12.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/1"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 23:16:27.000000000","tz":0},"subject":"Allow for dyanmic ks-user-job name","message":"Allow for dyanmic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada charts,\nthis means that we will have overlapping Kubernetes resources in the same namespace.\nTo avoid issues and keep the resources attached to their specific armada chart (release),\nwe need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml like so:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve this AND\nit will fail helm lint, because who knows when you do a helm lint what you need to\nput in the values.yaml when it generates a unique release name??\n\nWe want more flexibility in the ks-user-job, to not basically use one parameter for everything, and\nallow simple use by adding another parameter so we don\u0027t have to override value in values.yaml\nand can pass helm linting (side effect of needing specific values in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/0357c2eb03ec52dd92454abd773327004defe58b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/0357c2eb03ec52dd92454abd773327004defe58b"}]},"branch":"refs/heads/master"},"0f6ecf5c0046ec2985966ccf5b6e07d34c0566df":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2020-07-21 23:26:05.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/2"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Kelly Carter","email":"KC755D@att.com","date":"2020-07-21 23:26:01.000000000","tz":0},"subject":"Allow for dyanmic ks-user-job name","message":"Allow for dyanmic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe want more flexibility in the ks-user-job, to allow for the name to be\nunique (based of release) while not having to override values in\nvalues.yaml and also pass helm linting (side effect of needing specific\nvalues in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/0f6ecf5c0046ec2985966ccf5b6e07d34c0566df"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/0f6ecf5c0046ec2985966ccf5b6e07d34c0566df"}]},"branch":"refs/heads/master"},"63fd039067e25c03a30b54adf947a98df7029c72":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2020-08-04 17:28:00.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/3"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Kelly Carter","email":"KC755D@att.com","date":"2020-08-04 17:27:57.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe want more flexibility in the ks-user-job, to allow for the name to be\nunique (based of release) while not having to override values in\nvalues.yaml and also pass helm linting (side effect of needing specific\nvalues in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/63fd039067e25c03a30b54adf947a98df7029c72"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/63fd039067e25c03a30b54adf947a98df7029c72"}]},"branch":"refs/heads/master"},"3b06d1f5291fa0e2bc875cbfd4d37cce626a0482":{"kind":"REWORK","_number":4,"created":"2020-08-04 19:50:51.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/4"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Kelly Carter","email":"KC755D@att.com","date":"2020-08-04 19:50:49.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe want more flexibility in the ks-user-job, to allow for the name to be\nunique (based of release) while not having to override values in\nvalues.yaml and also pass helm linting (side effect of needing specific\nvalues in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/3b06d1f5291fa0e2bc875cbfd4d37cce626a0482"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/3b06d1f5291fa0e2bc875cbfd4d37cce626a0482"}]},"branch":"refs/heads/master"},"d51cb9cb1651e6c122610a858796487fcb849199":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2020-08-04 20:01:32.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/5"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Kelly Carter","email":"KC755D@att.com","date":"2020-08-04 20:01:29.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe can send it as $serviceUser OR just attach release.name where it sets\nit. Going for option 2 because with option 1 we hit following issues:\nwhen ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe considered adding another argument to use for grabbing values from\nvalues.yaml, however didn\u0027t want to add another arg.\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/d51cb9cb1651e6c122610a858796487fcb849199"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/d51cb9cb1651e6c122610a858796487fcb849199"}]},"branch":"refs/heads/master"},"13a9aa36289e6296133fe4e3b443e0763955d12c":{"kind":"REWORK","_number":6,"created":"2020-08-04 20:40:53.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/6"}}},"commit":{"parents":[{"commit":"aaf52acc278fb179b9fd88bb14d2188192eeb70b","subject":"[ceph-client] Add back a new version of reweight_osds()","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/aaf52acc278fb179b9fd88bb14d2188192eeb70b"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Kelly Carter","email":"KC755D@att.com","date":"2020-08-04 20:40:50.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe can send it as $serviceUser OR just attach release.name where it sets\nit. Going for option 2 because with option 1 we hit following issues:\nwhen ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe considered adding another argument to use for grabbing values from\nvalues.yaml, however didn\u0027t want to add another arg.\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/13a9aa36289e6296133fe4e3b443e0763955d12c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/13a9aa36289e6296133fe4e3b443e0763955d12c"}]},"branch":"refs/heads/master"},"32d739ccc65eb670cf282ec77fba16454a8ccc7d":{"kind":"REWORK","_number":7,"created":"2020-08-05 18:28:00.000000000","uploader":{"_account_id":30745,"name":"Kelly Carter","email":"KC755D@att.com","username":"kc755d"},"ref":"refs/changes/12/742312/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/7"}}},"commit":{"parents":[{"commit":"8cd4e26a426e2ae128a56773b2391b82fcbe0e01","subject":"Merge \"Fluentd: Allow for Multiple Config Files\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/8cd4e26a426e2ae128a56773b2391b82fcbe0e01"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-08-05 18:25:43.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe want more flexibility in the ks-user-job, to allow for the name to be\nunique (based of release) while not having to override values in\nvalues.yaml and also pass helm linting (side effect of needing specific\nvalues in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/32d739ccc65eb670cf282ec77fba16454a8ccc7d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/32d739ccc65eb670cf282ec77fba16454a8ccc7d"}]},"branch":"refs/heads/master"},"816790f7e49e3bb26a822591cd912536f9b4eb49":{"kind":"TRIVIAL_REBASE","_number":8,"created":"2021-08-20 15:20:44.000000000","uploader":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"ref":"refs/changes/12/742312/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/12/742312/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/12/742312/8"}}},"commit":{"parents":[{"commit":"fa174c00db0d4469d779b7266c5a4f0d2060ebbf","subject":"Fix ceph-provisioner rbd-healer error","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/fa174c00db0d4469d779b7266c5a4f0d2060ebbf"}]}],"author":{"name":"Carter, Kelly (kc755d)","email":"kc755d@att.com","date":"2020-07-21 22:56:20.000000000","tz":0},"committer":{"name":"Chris Wedgwood","email":"cw@f00f.org","date":"2021-08-20 15:20:44.000000000","tz":0},"subject":"Allow for dynamic ks-user-job name","message":"Allow for dynamic ks-user-job name\n\nIn the scenario of a helm chart which will be used by multiple armada\ncharts, this means that we will have overlapping Kubernetes resources\nin the same namespace.\nTo avoid issues and keep the resources attached to their specific armada\nchart (release), we need the names of each resource to be:\n    {{ Release.Name }}-{{ Name }}.\nNow, in using the ks-user job, the name is set to:\n    name: {{ printf \"%s-%s\" $serviceUserPretty \"ks-user\" | quote }}\n^ $serviceUserPretty is just $serviceUser replacing “_”:\n    {{- $serviceUserPretty :\u003d $serviceUser | replace \"_\" \"-\" -}}\n\nSo if we want the name to be unique ({{ Release.Name }}-{{ Name }})\nwe have to send this in as $serviceUser …\nThe issue comes up when ks-user-job grabs values from values.yaml:\nenvAll.Values.secrets.identity $serviceUser\nenvAll.Values.endpoints.identity.auth $serviceUser\n\nNow our values.yaml HAS to be overridden in every armada chart to solve\nthis AND it will fail helm lint, because who knows when you do a helm\nlint what you need to put in the values.yaml when it generates a unique\nrelease name.\n\nWe want more flexibility in the ks-user-job, to allow for the name to be\nunique (based of release) while not having to override values in\nvalues.yaml and also pass helm linting (side effect of needing specific\nvalues in values.yaml).\n\nChange-Id: I6758cefbb17e369d3a39f812d63bd487a08f47f0\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/816790f7e49e3bb26a822591cd912536f9b4eb49"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/816790f7e49e3bb26a822591cd912536f9b4eb49"}]},"branch":"refs/heads/master","description":"Rebase"}},"requirements":[],"submit_records":[],"submit_requirements":[]}