| *** rlandy_ is now known as rlandy|out | 01:08 | |
| *** soniya29 is now known as soniya29|ruck | 04:44 | |
| *** raukadah is now known as chkumar|rover | 04:45 | |
| *** soniya is now known as soniya|ruck | 05:32 | |
| *** ysandeep|out is now known as ysandeep | 05:46 | |
| *** gibi_summit is now known as gibi | 06:43 | |
| *** ysandeep is now known as ysandeep|lunch | 07:30 | |
| *** soniya|ruck is now known as soniya|ruck|lunch | 08:58 | |
| *** jpena|off is now known as jpena | 09:38 | |
| *** soniya|ruck|lunch is now known as soniya|ruck | 09:51 | |
| *** ysandeep|lunch is now known as ysandeep | 10:14 | |
| *** ysandeep is now known as ysandeep|sick | 10:16 | |
| *** rlandy__ is now known as rlandy | 10:27 | |
| *** soniya29 is now known as soniya29|ruck | 11:49 | |
| opendevreview | Dr. Jens Harbott proposed openstack/diskimage-builder master: [wip] test f36 builds https://review.opendev.org/c/openstack/diskimage-builder/+/844912 | 12:11 |
|---|---|---|
| *** dasm|off is now known as dasm | 13:39 | |
| opendevreview | Blaise R Pabon proposed opendev/infra-manual master: Update the sandbox instructions Clarify the instructions, reduce ambiguity, increase consistency. Rather than change the legacy OpenStack URLs, I changed the text to acknowledge that we refer to OpenStack. https://review.opendev.org/c/opendev/infra-manual/+/845593 | 14:15 |
| opendevreview | Blaise R Pabon proposed opendev/infra-manual master: Update the sandbox instructions https://review.opendev.org/c/opendev/infra-manual/+/845593 | 14:16 |
| BlaisePabon[m]1 | My first real Gerrit change. | 14:20 |
| fungi | thanks for that! | 14:21 |
| opendevreview | Lajos Katona proposed openstack/project-config master: Add py38/py39 jobs to Grafana https://review.opendev.org/c/openstack/project-config/+/845596 | 14:28 |
| fungi | BlaisePabon[m]1: looks great, i left a few potential improvements as inline review comments. also if you want to see a preview build with your proposed change, zuul provided one at https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0ef/845593/2/check/opendev-tox-docs/0ef31d1/docs/sandbox.html | 14:37 |
| fungi | that url will change at each iteration, but you can find it by following the opendev-tox-docs link in each new zuul comment to its build result dashboard, and then under the artifacts tab there's a docs preview site link | 14:38 |
| fungi | of course, you should also be able to `tox -e docs` locally and then point your browser at the resulting doc/build/html tree | 14:39 |
| *** dviroel is now known as dviroel|lunch | 14:54 | |
| clarkb | BlaisePabon[m]1: thank you for the change. Looks like fungi left a review though so I don't need to pile on :) thanks again | 15:47 |
| *** soniya29|ruck is now known as soniya29|ruck|dinner | 16:02 | |
| *** dviroel|lunch is now known as dviroel | 16:06 | |
| chris_dohmen | Hey folks! I want to make sure I am not missing anything with this review / process. I have a review for PBR and it currently states its awaiting action from me. However I believe at this point its waiting for the workflow approval. I just want to make sure this is correct and its not being held up by me. Any insight would be appreciated! Thanks! | 16:07 |
| chris_dohmen | https://review.opendev.org/c/openstack/pbr/+/844586 | 16:07 |
| clarkb | chris_dohmen: it needs a second reviewer to look it over and if they are happy with it they can approve it. fungi may have time for that at some point | 16:11 |
| *** rlandy is now known as rlandy|biab | 16:11 | |
| opendevreview | Clark Boylan proposed opendev/system-config master: Set gerrit change.conflictsPredicateEnabled to true https://review.opendev.org/c/opendev/system-config/+/845605 | 16:25 |
| *** soniya29|ruck|dinner is now known as soniya29|ruck | 16:32 | |
| fungi | chris_dohmen: i'll take a look. many of us have been in a focus tunnel leading up to a conference all last week, so things have been moving a bit more slowly in places like code review | 16:40 |
| fungi | also the openstack/pbr project is officially overseen by folks in #openstack-oslo, for future reference (but some of us in here also review changes for it) | 16:41 |
| chris_dohmen | @fungi, no rush! I just wanted to make sure it was not waiting on me. | 16:41 |
| fungi | chris_dohmen: normally you'd also add a release note with fixes in pbr, but for me that one's on the edge of maybe not needing one | 16:43 |
| chris_dohmen | Ah ok. I will create another review adding that info to contributing.rst to explicitly call that one out as I was not aware of it | 16:56 |
| *** jpena is now known as jpena|off | 17:06 | |
| fungi | chris_dohmen: ooh, thanks that's a great idea | 17:16 |
| fungi | chris_dohmen: for the record, the release notes are managed with this tool: https://docs.openstack.org/reno/latest/user/usage.html | 17:20 |
| *** rlandy|biab is now known as rlandy | 18:01 | |
| chris_dohmen | @fungi: is reno used throughout all of openstack repos or only specific ones? I am trying to figure out if I should update pbr's documentation or if https://docs.opendev.org/opendev/infra-manual/latest/developers.html#development-workflow should be updated instead. | 19:34 |
| clarkb | chris_dohmen: its used in a lot of repos but not necessarily all of them. It is probably too specific for infra-manual other than maybe making a note to check if a release note is appropriate regardless of what system is used for that | 19:36 |
| chris_dohmen | clarkb: perfect, ill make the update to pbr then! thanks all for the help | 19:37 |
| fungi | yeah, some of the opendev repos also use reno, for example opendev/bindep and opendev/git-review | 20:15 |
| fungi | it's definitely used beyond openstack, but it's also used by projects entirely outside the opendev collaboratory | 20:15 |
| fungi | and certainly a project-specific choice | 20:16 |
| fungi | so the contributing.rst is a sensible place to mention it | 20:16 |
| *** dviroel is now known as dviroel|out | 21:30 | |
| opendevreview | Blaise R Pabon proposed opendev/infra-manual master: Make requested changes and other suggestions. https://review.opendev.org/c/opendev/infra-manual/+/845642 | 21:37 |
| fungi | BlaisePabon[m]1: did you mean to commit --append that? ^ | 21:56 |
| fungi | https://docs.opendev.org/opendev/infra-manual/latest/developers.html#updating-a-change | 21:56 |
| *** dasm is now known as dasm|off | 22:18 | |
| *** rlandy is now known as rlandy|bbl | 22:28 | |
| clarkb | about to send out the infra meeting agenda. I'ev made some quick edits to it. Anything else to add? | 23:08 |
| fungi | i got nothing | 23:22 |
| clarkb | ok sent | 23:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!