<div><div>Hi there.</div><div> </div><div>Sorry, I totally missed that email, since we usually use tags to address specific teams, so please, use "[${PROJECT}]" in topic if you address a ML to specific group in future:)</div><div> </div><div>1. There bunch of issues with code proposed, actually, which have been commented: [1] and neither of them were reflected in any way since 10 December. Gerrit Code-Review [2] system is a point where proposed code is being reviewed by Core Reviewers. Which it has been done in quite timely manner if you reffer to timestaps in patch of topic.</div><div>Why I said about ansible module, because current proposed solution is not idempotent and is hard to maintain. As if you want to fix or change smth in script that manages vault tokens, you will need to edit it in every role that uses it, which is really hard to manage.On the contrary ansible module is being managed from single place, so you just call it from role and don't need to do duplicate code for each role.</div><div>Also, current solution would create a new vault secret each time role runs even when secret already has been stored which is not idempotent way. Not saying about other 8 comments and that patches were never passing CI.</div><div>So from my perspective solution requires some effort before it can be considered as ready one. And are we quite picky when it comes to code quality that we merge.</div><div> </div><div>2. According to OpenStack Releases guidelines [3], new features are not eligible for being backported. Also branches you;re mentioning are under Extended Maintenance which means only security patching is generally provided for them.</div><div>However, OpenStack-Ansible is flexible enough. So you should be able to deploy older OpenStack code with recent roles. We define SHA for services that are being deployed by OSA using SHAs [4], so technically it should be possible to use Yoga version of OpenStack-Ansible and override OpenStack version to Stein to get stein version of OpenStack services deployed.</div><div>It could be quite tricky in practice though, since we could drop some required variables that are now deprecated, but in most cases it can be fixed trivially. So what I'm saying that technically there's a way to use your code from master for older versions.</div><div> </div><div>As Jonathan mentioned, we're quite open for communication in #opnestack-ansible channel on IRC.</div><div> </div><div>[1] <a href="https://review.opendev.org/q/topic:bp%252Fprotecting-plaintext-configs" rel="noopener noreferrer" target="_blank">https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/814865</a></div><div><div>[2] https://review.opendev.org/</div></div><div>[3] https://docs.openstack.org/project-team-guide/stable-branches.html#maintained</div><div>[4] https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/defaults/repo_packages/openstack_services.yml</div></div><div> </div><div>04.04.2022, 17:33, "Kelsi Parenteau" <k.parenteau@connectria.com>:</div><blockquote><div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt">Good morning Openstack,</div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt">I hope this message finds you well. I wanted to follow up from Alex's last email below to help to clarify our questions here. <span style="background-color:#ffffff">We're reaching out to ask your reviewers for their feedback on what had changed on your side during our course of work.</span> <span style="font-family:'calibri' , sans-serif;font-size:11pt;line-height:107%"><a href="https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/814865" rel="noopener noreferrer">https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/814865</a></span></div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt">We had been working with your team over many months, and had been tracking to commit the code upstream. We were not sure why the Openstack reviewers had not brought up this potential concern for us earlier on in our discussions to be addressed. </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt">Can you please advise us why that particular comment regarding the requirement for this to be an ansible plugin stops us from being able to commit the code?</div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt">We look forward to your feedback here, and would be happy to schedule a call as well to talk this through. Please let us know if you have any questions.</div><div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div><div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"> </div><div> </div><div> </div><div> </div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"><p style="background-color:white;color:rgb( 34 , 34 , 34 );font-family:'arial' , 'helvetica' , sans-serif;font-size:small;margin:0px"><span style="color:black;font-family:'arial';font-size:14.67px">Thank you,</span></p><p style="background-color:white;color:rgb( 34 , 34 , 34 );font-family:'arial' , 'helvetica' , sans-serif;font-size:small;margin:0px"> </p><p style="background-color:white;color:rgb( 34 , 34 , 34 );font-family:'arial' , 'helvetica' , sans-serif;font-size:small;margin:0px"> </p><div style="background-color:white;font-family:'arial' , 'helvetica' , sans-serif;font-size:small;margin:0px"><table width="450" style="border-collapse:collapse;font-family:'lato' , 'arial' , 'san-serif';font-size:14px"><tbody><tr><td align="left" width="290" style="font-family:'roboto' , 'robotodraft' , 'helvetica' , 'arial' , sans-serif;margin:0px;padding:0px 0px 10px 0px"><div style="margin:0px"> </div></td></tr></tbody></table><table style="background-color:white;border-collapse:collapse;color:rgb( 32 , 31 , 30 );font-size:15px"><tbody><tr style="height:105pt"><td style="height:105pt;padding:0px 0px 8pt 0px"><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><strong><span style="color:black;font-family:'arial' , sans-serif;font-size:10.5pt;margin:0px">Kelsi Parenteau, PMP, PMI-ACP, CSM</span></strong></p><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><span style="color:black;font-family:'arial' , sans-serif;font-size:10.5pt;margin:0px">Senior Project Manager</span></p><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><span style="color:black;font-family:'arial' , sans-serif;font-size:10.5pt;margin:0px">d: 586.473.1230 I m: 313.404.3214</span></p><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><em><span style="color:black;font-family:'arial' , sans-serif;font-size:10.5pt;margin:0px"> </span></em></p><table style="background-color:white;border-collapse:collapse;width:337.5pt"><tbody><tr><td style="padding:0px 0px 7.5pt 0px;width:217.5pt"><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><span style="color:black;font-family:'helvetica' , sans-serif;font-size:10.5pt;margin:0px">  </span></p></td></tr><tr><td style="width:337.5pt"><p style="font-family:'calibri' , sans-serif;font-size:11pt;margin:0px"><span style="color:black;font-family:'helvetica' , sans-serif;font-size:10.5pt;margin:0px">  </span></p></td></tr></tbody></table></td></tr></tbody></table></div></div></div></div></div><div> </div><hr style="width:98%" /><div><font color="#000000" face="Calibri, sans-serif" style="font-size:11pt"><strong>From:</strong> Alexander Yeremko <<a href="mailto:a.yeremko@connectria.com" rel="noopener noreferrer">a.yeremko@connectria.com</a>><br /><strong>Sent:</strong> Tuesday, March 29, 2022 4:10 PM<br /><strong>To:</strong> <a href="mailto:openstack-discuss@lists.openstack.org" rel="noopener noreferrer">openstack-discuss@lists.openstack.org</a> <<a href="mailto:openstack-discuss@lists.openstack.org" rel="noopener noreferrer">openstack-discuss@lists.openstack.org</a>><br /><strong>Cc:</strong> Tina Wisbiski <<a href="mailto:t.wisbiski@connectria.com" rel="noopener noreferrer">t.wisbiski@connectria.com</a>>; Kelsi Parenteau <<a href="mailto:k.parenteau@connectria.com" rel="noopener noreferrer">k.parenteau@connectria.com</a>>; Yuliia Romanova <<a href="mailto:y.romanova@connectria.com" rel="noopener noreferrer">y.romanova@connectria.com</a>><br /><strong>Subject:</strong> plain text config parameters encryption feature</font><div> </div></div><div><div style="color:rgb( 0 , 0 , 0 );font-family:'calibri' , 'arial' , 'helvetica' , sans-serif;font-size:12pt"><span style="background-color:#ffffff">Dear OpenStack community,</span><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">we are developing <span style="margin:0px">plain</span> <span style="margin:0px">text</span> <span style="margin:0px">config</span> secrets <span style="margin:0px">encryption</span> <span style="margin:0px">feature</span> according to the next specification:</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"><a href="https://specs.openstack.org/openstack/openstack-ansible-specs/specs/xena/protecting" rel="noopener noreferrer">https://specs.openstack.org/openstack/openstack-ansible-specs/specs/xena/protecting</a>-<span style="margin:0px">plain</span>text-<span style="margin:0px">config</span>s.html</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">We started from Glance OS service and submitted two patchsets already:</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"><a href="https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/814865" rel="noopener noreferrer">https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/814865</a></div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">Now we have two questions that we need to clarify to proceed our work on that <span style="margin:0px">feature</span> and finish our development:</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">1. Is it correct that we need to develop more patchsets to rework some logic of <span style="margin:0px">encryption</span> mechanism according</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">to comment to 'files/encypt_secrets.py' script that arised at the second patchset (PatchSet 2) dated Nov/30/2021 ?</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">Comment is by Dmitry Rabotyagov: "We _really_ should make it as an ansible plugin and re-work logic"</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">2. We wish to have such <span style="margin:0px">feature</span> in previous releases also, not just in upcoming Yoga or Zed.</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">Stein, Train and Victoria - it would be excellent to have <span style="margin:0px">plain</span> <span style="margin:0px">text</span> secrets <span style="margin:0px">encryption</span> with these releases also.</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">So question is how is it possible to use our <span style="margin:0px">feature</span> in those releases also? Can we push some backports to those releases openstack-ansible repo?<br /><br />Could someone be so kind and give us answers?</div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px"> </div><div style="background-color:rgb( 255 , 255 , 255 );margin:0px">Best regards and wishes,</div> <span style="background-color:#ffffff">Alex Yeremko</span></div></div>This E-Mail (including any attachments) may contain privileged or confidential information. It is intended only for the addressee(s) indicated above. The sender does not waive any of its rights, privileges or other protections respecting this information. Any distribution, copying or other use of this E-Mail or the information it contains, by other than an intended recipient, is not sanctioned and is prohibited. If you received this E-Mail in error, please delete it and advise the sender (by return E-Mail or otherwise) immediately. Any calls held by you with Connectria may be recorded by an automated note taking system to ensure prompt follow up and for information collection purposes, and your attendance on any calls with Connectria confirms your consent to this. Any E-mail received by or sent from Connectria is subject to review by Connectria supervisory personnel.</div></blockquote><div> </div><div> </div><div>-- <br />Kind Regards,</div><div>Dmitriy Rabotyagov</div><div> </div>