<div dir="auto"><div>Hi Julia,<div dir="auto"><br></div><div dir="auto">Thanks for bringing this up.</div><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 13, 2018, 2:41 PM Julia Kreger, <<a href="mailto:juliaashleykreger@gmail.com">juliaashleykreger@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Greetings fellow ironicans!<br>
<br>
As many of you might know an openstack/ironic-staging-drivers[1]<br>
repository exists. What most might not know is that it was<br>
intentionally created outside of ironic's governance[2].<br>
<br>
At the time it was created ironic was moving towards removing drivers<br>
that did not meet our third-party CI requirement[3] to be in-tree. The<br>
repository was an attempt to give a home to what some might find<br>
useful or where third party CI is impractical or cost-prohibitive and<br>
thus could not be officially part of Ironic the service. There was<br>
hope that drivers could land in ironic-staging-drivers and possibly<br>
graduate to being moved in-tree with third-party CI. As our community<br>
has evolved we've not stopped and revisited the questions.<br>
<br>
With our most recent release over, I believe we need to ask ourselves<br>
if we should consider moving ironic-staging-drivers into our<br>
governance.<br>
<br>
Over the last couple of releases several contributors have found<br>
themselves trying to seek out two available reviewers to merge even<br>
trivial fixes[4]. Due to the team being so small this was no easy<br>
task. As a result, I'm wondering why not move the repository into<br>
governance, grant ironic-core review privileges upon the repository,<br>
and maintain the purpose and meaning of the repository. This would<br>
also result in the repository's release becoming managed via the<br>
release management process which is a plus.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">If I understand, it seems like the main issue is lack of reviewers. As mentioned by others, I would not be opposed to adding existing ironic cores to this repo.</div><div dir="auto">Whether folks review is a different question.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We could then propose an actual graduation process and help alleviate<br>
some of the issues where driver code is iterated upon for long periods<br>
of time before landing. At the same time I can see at least one issue<br>
which is if we were to do that, then we would also need to manage<br>
removal through the same path.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I am not sure I see any advantages to this. The ansible driver was in the staging repo for awhile before it went into ironic so we know that is do-able :)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I know there are concerns over responsibility in terms of code<br>
ownership and quality, but I feel like we already hit such issues[5],<br>
like those encountered when Dmitry removed classic drivers[6] from the<br>
repository and also encountered issues just prior to the latest<br>
release[7][8].<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I don't mind making changes or reviewing changes to this repo, especially if there are unit tests. However, that is the most responsibility I am comfortable having with this repo.</div><div dir="auto"><br></div><div dir="auto">Right now, I don't see any good reasons for putting it under the ironic governance. I am, of course, open to being convinced otherwise!</div><div dir="auto"><br></div><div dir="auto">--ruby</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This topic has come up in passing at PTGs and most recently on IRC[9],<br>
and I think we ought to discuss it during our next weekly meeting[10].<br>
I've gone ahead and added an item to the agenda, but we can also<br>
discuss via email.</blockquote></div></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-Julia<br>
<br>
[1]: <a href="http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/projects.yaml#n4571" rel="noreferrer noreferrer" target="_blank">http://git.openstack.org/cgit/openstack-infra/project-config/tree/gerrit/projects.yaml#n4571</a><br>
[2]: <a href="http://git.openstack.org/cgit/openstack/ironic-staging-drivers/tree/README.rst#n16" rel="noreferrer noreferrer" target="_blank">http://git.openstack.org/cgit/openstack/ironic-staging-drivers/tree/README.rst#n16</a><br>
[3]: <a href="https://specs.openstack.org/openstack/ironic-specs/specs/approved/third-party-ci.html" rel="noreferrer noreferrer" target="_blank">https://specs.openstack.org/openstack/ironic-specs/specs/approved/third-party-ci.html</a><br>
[4]: <a href="https://review.openstack.org/#/c/548943/" rel="noreferrer noreferrer" target="_blank">https://review.openstack.org/#/c/548943/</a><br>
[5]: <a href="https://review.openstack.org/#/c/541916/" rel="noreferrer noreferrer" target="_blank">https://review.openstack.org/#/c/541916/</a><br>
[6]: <a href="https://review.openstack.org/567902" rel="noreferrer noreferrer" target="_blank">https://review.openstack.org/567902</a><br>
[7]: <a href="https://review.openstack.org/590352" rel="noreferrer noreferrer" target="_blank">https://review.openstack.org/590352</a><br>
[8]: <a href="https://review.openstack.org/590401" rel="noreferrer noreferrer" target="_blank">https://review.openstack.org/590401</a><br>
[9]: <a href="http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2018-08-09.log.html#t2018-08-09T11:55:27" rel="noreferrer noreferrer" target="_blank">http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2018-08-09.log.html#t2018-08-09T11:55:27</a><br>
[10]: <a href="https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting" rel="noreferrer noreferrer" target="_blank">https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting</a><br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div></div>