| @jreed:matrix.org | brunomuniz daniel.caires - FYSA - I am submitting the initial review to create the repo. It's using a public github repo that I have to go populate afterward. | 15:48 |
|---|---|---|
| https://review.opendev.org/c/openstack/project-config/+/899487 | ||
| @jreed:matrix.org | * brunomuniz daniel.caires - FYSA - I am submitting the initial review to create the standalone app gen tool repo. It's using a public github repo that I have to go populate afterward. | 15:48 |
| https://review.opendev.org/c/openstack/project-config/+/899487 | ||
| @jreed:matrix.org | You already saw it fungi & scott.little :) | 15:49 |
| @fungicide:matrix.org | yep! | 15:55 |
| @fungicide:matrix.org | lgtm, other than the branch concern i noted in my last comment | 15:55 |
| @jreed:matrix.org | I saw it. scott.little made the place holder repo for me, so I'm waiting on him to loop back on it. | 16:18 |
| @fungicide:matrix.org | once he has a chance to either clean that up or assert that it was intentional, i'll go ahead and approve the change. after that, the automation that creates and links the project in all the right places will be triggered by it merging, and only takes a few minutes to finish | 18:04 |
| @jreed:matrix.org | Down to one branch now. | 19:00 |
| @fungicide:matrix.org | the project should exist now, and scott.little is the initial member of the new core review group for it | 19:54 |
| @brunomuniz:matrix.org | I guess creating a `.gitreview` is the next step? :) | 20:10 |
| ``` | ||
| >> git review -s | ||
| /usr/lib/python3/dist-packages/requests/__init__.py:87: RequestsDependencyWarning: urllib3 (1.26.5) or chardet (5.2.0) doesn't match a supported version! | ||
| warnings.warn("urllib3 ({}) or chardet ({}) doesn't match a supported " | ||
| No '.gitreview' file found in this repository. We don't know where | ||
| your gerrit is. | ||
| Please manually create a remote named "gerrit" or rename the default | ||
| one and try again. | ||
| ``` | ||
| @brunomuniz:matrix.org | I can suggest a quick change, I think... | 20:11 |
| @brunomuniz:matrix.org | https://review.opendev.org/c/starlingx/app-gen-tool/+/899511 | 20:33 |
| @jreed:matrix.org | I think initial core reviewers would be myself, igorpsoares , brunomuniz , and daniel.caires ? Any others? | 21:00 |
| @jreed:matrix.org | Now that its added I've got to get the manifest file updated so it clones with the rest of StarlingX, and we should probably add Zuul in there too. | 21:01 |
| @jreed:matrix.org | I also need to make a review to remove the old app gen from starlingx/tools | 21:02 |
| @jreed:matrix.org | Plan to do some of that next week. | 21:02 |
| @jreed:matrix.org | And thanks brunomuniz for adding the review file. | 21:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!