*** tetsuro has joined #openstack-tc | 00:33 | |
*** diablo_rojo has quit IRC | 00:48 | |
*** markvoelker has joined #openstack-tc | 02:31 | |
*** markvoelker has quit IRC | 02:35 | |
*** markvoelker has joined #openstack-tc | 02:37 | |
*** markvoelker has quit IRC | 02:47 | |
*** markvoelker has joined #openstack-tc | 04:31 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-tc | 04:33 | |
*** markvoelker has quit IRC | 04:36 | |
*** lxkong has quit IRC | 04:42 | |
*** tkajinam has quit IRC | 05:34 | |
*** tkajinam has joined #openstack-tc | 05:35 | |
*** tkajinam has quit IRC | 05:50 | |
*** tkajinam has joined #openstack-tc | 05:51 | |
*** markvoelker has joined #openstack-tc | 06:21 | |
*** markvoelker has quit IRC | 06:26 | |
*** redrobot has quit IRC | 06:37 | |
*** slaweq has joined #openstack-tc | 07:00 | |
*** slaweq has quit IRC | 07:05 | |
*** e0ne has joined #openstack-tc | 07:11 | |
*** tetsuro has quit IRC | 07:17 | |
*** slaweq has joined #openstack-tc | 07:50 | |
*** markvoelker has joined #openstack-tc | 07:51 | |
*** ralonsoh has joined #openstack-tc | 07:54 | |
*** markvoelker has quit IRC | 07:56 | |
*** tosky has joined #openstack-tc | 08:04 | |
*** dmellado has quit IRC | 08:09 | |
*** dmellado has joined #openstack-tc | 08:14 | |
*** tetsuro has joined #openstack-tc | 08:41 | |
*** tetsuro has quit IRC | 08:41 | |
*** markvoelker has joined #openstack-tc | 09:04 | |
*** markvoelker has quit IRC | 09:09 | |
*** tkajinam has quit IRC | 09:17 | |
*** ricolin has quit IRC | 09:59 | |
*** markvoelker has joined #openstack-tc | 11:05 | |
*** markvoelker has quit IRC | 11:10 | |
*** cloudnull has quit IRC | 11:42 | |
*** cloudnull has joined #openstack-tc | 11:43 | |
*** ijolliffe has joined #openstack-tc | 12:30 | |
openstackgerrit | Merged openstack/ideas master: New communication tools proposal https://review.opendev.org/718932 | 13:02 |
---|---|---|
njohnston | o/ | 13:06 |
gmann | o/ | 13:12 |
evrardjp | o/ | 13:29 |
*** dirk has quit IRC | 13:31 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/governance master: Resolution to define distributed leadership for projects https://review.opendev.org/744995 | 13:58 |
*** dklyle has quit IRC | 14:40 | |
*** david-lyle has joined #openstack-tc | 14:40 | |
*** david-lyle is now known as dklyle | 14:41 | |
*** Guest7899 has joined #openstack-tc | 15:03 | |
*** Guest7899 is now known as redrobot | 15:06 | |
*** ralonsoh has quit IRC | 15:08 | |
*** ralonsoh has joined #openstack-tc | 15:08 | |
*** e0ne has quit IRC | 15:21 | |
*** gibi is now known as gibi_pto | 15:24 | |
*** gibi_pto is now known as gibi_pto_til_24t | 15:24 | |
*** gibi_pto_til_24t is now known as gibi_pto_24th | 15:24 | |
*** ralonsoh has quit IRC | 17:02 | |
jungleboyj | o/ | 17:52 |
openstackgerrit | Sean McGinnis proposed openstack/governance-sigs master: Propose new Hardware Vendor SIG https://review.opendev.org/745185 | 18:11 |
*** slaweq has quit IRC | 19:03 | |
*** slaweq has joined #openstack-tc | 19:06 | |
*** KeithMnemonic has quit IRC | 19:31 | |
mnaser | gmann: neat, looks like only `opendev/puppet-apps_site`, `opendev/puppet-releasestatus` and `opendev/puppet-stackalytics` are the ones that aren't "clean" yet | 19:34 |
mnaser | i guess we could have the script ignore anything that's not in the openstack/ namespace? | 19:34 |
mnaser | leaving `openstack/python-dracclient`, `openstack/networking-l2gw`, `openstack/networking-l2gw-tempest-plugin` and `openstack/neutron-vpnaas` as the ones to figure out | 19:35 |
gmann | mnaser: yeah, i did not push the cleanup on those as acl to tc are only for openstack/ | 19:35 |
mnaser | openstack/neutron-vpnaas seems still alive.. | 19:35 |
gmann | yeah openstack/neutron-vpnaas is not retired | 19:35 |
mnaser | so should we unretire that in governance i guess? | 19:35 |
gmann | openstack/networking-l2gw is in progress | 19:35 |
fungi | right, i wouldn't bother enforcing it for anything which moved out of the openstack namespace before the mandatory retirement resolution took effect | 19:36 |
gmann | is it in legacy file ? | 19:36 |
mnaser | gmann: `openstack/neutron-vpnaas is not properly retired, files: ['.coveragerc', '.gitignore', '.gitreview', '.mailmap', '.pylintrc', '.stestr.conf', '.zuul.yaml', 'CONTRIBUTING.rst', 'HACKING.rst', 'LICENSE', 'README.rst', 'TESTING.rst', 'devstack', 'doc', 'etc', 'lower-constraints.txt', 'neutron_vpnaas', 'playbooks', 'rally-jobs', 'releasenotes', 'requirements.txt', 'roles', 'setup.cfg', 'setup.py', | 19:36 |
mnaser | 'test-requirements.txt', 'tools', 'tox.ini'].` it seems so | 19:36 |
mnaser | let me find the commit that added it | 19:36 |
mnaser | oh wow, retired in `2016-11-01` supposidly.. | 19:36 |
mnaser | but it's had releases for quite sometime too | 19:36 |
fungi | retired from neutron, but not retired from being developed | 19:37 |
gmann | yeah, we should remove it from https://opendev.org/openstack/governance/src/branch/master/reference/legacy.yaml#L510 | 19:37 |
mnaser | fungi: but it's being released under openstack | 19:37 |
mnaser | like an actual openstack deliverable exists | 19:37 |
gmann | as per slaweq it is active deliverable now | 19:37 |
fungi | oh, was it unretired? | 19:37 |
smcginnis | We have governance checks. Is it listed as owned by an official team *and* also listed as legacy? | 19:38 |
gmann | i think yes | 19:38 |
mnaser | https://releases.openstack.org/ussuri/index.html#ussuri-neutron-vpnaas | 19:38 |
fungi | should be removed from legacy.yaml if it's resurrected out of retirement | 19:38 |
mnaser | maybe that's the problem :) | 19:38 |
gmann | smcginnis: yeah in both | 19:38 |
njohnston | yes, I think that is the issue | 19:38 |
smcginnis | I seem to recall either that one or -fwaas going through a period where they were looking for maintainers. | 19:38 |
mnaser | trying to trace down the original commit | 19:39 |
smcginnis | Maybe it was started to be retired, then someone stepped up and it ended up back in neutron. | 19:39 |
mnaser | so this was retirement https://github.com/openstack/governance/commit/20f95dd947d2f87519b4bb50fb188e6f71deae7c | 19:39 |
fungi | https://review.openstack.org/506012 | 19:39 |
fungi | Submitted-at: Tue, 19 Dec 2017 20:54:33 +0000 | 19:39 |
mnaser | https://github.com/openstack/governance/commit/feb90ea14b0ceacf670bf8fce9a1ea6fdb69ab8d | 19:40 |
fungi | was the resurrection | 19:40 |
mnaser | https://review.opendev.org/#/c/527255/ | 19:40 |
smcginnis | Probably around the time the neutron stadium idea came up. | 19:40 |
mnaser | so this missed removing it from legacy.yaml | 19:40 |
fungi | so it's been an official deliverable again (added back to reference/projects.yaml) since late 2017 | 19:40 |
fungi | yep, i concur | 19:40 |
gmann | https://review.opendev.org/#/c/527255 | 19:41 |
gmann | yeah that one | 19:41 |
njohnston | in January of this year it was reconfirmed as part of the neutron stadium when new maintainers arose to manage it: http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011918.html | 19:42 |
fungi | btw, if you set git up to fetch git notes from +refs/notes/*:refs/notes/* then git log will show you the review url, when the commit actually merged, who approved it, et cetera | 19:42 |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Drop neutron-vpnaas from legacy projects https://review.opendev.org/745401 | 19:42 |
fungi | makes running down stuff from history much easier | 19:43 |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Add legacy repository validation https://review.opendev.org/737559 | 19:43 |
mnaser | and that's a rebase | 19:43 |
mnaser | let me also make it ignore opendev | 19:43 |
fungi | oh, and set notesRef = refs/notes/review | 19:43 |
fungi | since that's how gerrit stores them | 19:43 |
mnaser | that's beyond my git-fu :) | 19:44 |
gmann | mnaser: and also blacklist these three repo for now ? -openstack/python-dracclient`, `openstack/networking-l2gw`, `openstack/networking-l2gw-tempest-plugin | 19:44 |
gmann | with TODO | 19:45 |
smcginnis | [insert shameless plug for reviews on https://review.opendev.org/745185] | 19:45 |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Add legacy repository validation https://review.opendev.org/737559 | 19:48 |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Drop all exceptions for legacy validation https://review.opendev.org/745403 | 19:48 |
gmann | smcginnis: overall lgtm, once comment on SIG status | 19:48 |
gmann | 'active' can be moved once SIG start meeting/ML updates etc - https://governance.openstack.org/sigs/reference/sig-status.html | 19:49 |
mnaser | i believe the current legacy validation job will pass as of now | 19:49 |
*** slaweq has quit IRC | 19:50 | |
smcginnis | Sure, we can say forming. There are folks ready to get going though with the openstack/python-dracclient as soon as this gets approved, so we could also just go right to active. | 19:50 |
smcginnis | gmann: Would you prefer I change it though? | 19:50 |
gmann | tc-members: as TC and UC are same thing now, should we add all TC members into openstack/governance-sigs core list or follow the TC house rules like formal-vote rule etc? | 19:51 |
gmann | smcginnis: as per status definition i think we should wait for some progress to mark it active. or at least first meeting/ML update happen ? | 19:53 |
smcginnis | OK, fair enough. Update coming right up. | 19:53 |
gmann | it may be just after week or few days when we approved | 19:53 |
openstackgerrit | Sean McGinnis proposed openstack/governance-sigs master: Propose new Hardware Vendor SIG https://review.opendev.org/745185 | 19:53 |
gmann | smcginnis: thanks , +1. i think only rico form TC has the +2 right on governance-sigs and spotz or ttx may be | 19:54 |
spotz | gmann smcginnis - looking | 19:55 |
fungi | mnaser: lines 5-9 here are how i'm currently setting my ~/.gitignore to fetch git notes and to show the "review" notes gerrit adds (without disabling "commits" notes which git would normally show by default if those also exist) http://paste.openstack.org/show/796660/ | 19:55 |
fungi | er, ~/.gitconfig i mean | 19:56 |
smcginnis | I was wondering how gitignore played into this. ;) | 19:57 |
smcginnis | Good tips fungi! | 19:57 |
*** slaweq has joined #openstack-tc | 19:58 | |
fungi | how it plays in is that i also have a global ~/.gitignore set in my ~/.gitconfig, i guess ;) | 19:58 |
smcginnis | Haha | 19:58 |
smcginnis | fungi: Is there a trick to update the local refs? I get a "warning: notes ref refs/notes/review is invalid" with that. | 20:01 |
smcginnis | Fresh clone works though. | 20:02 |
fungi | huh, interesting. i'm not sure, you might be able to mkdir -p .git/refs/notes&&touch .git/refs/notes/review or something i guess | 20:04 |
fungi | did you git remote update first? | 20:04 |
fungi | i would have normally expected that to do it automagically | 20:04 |
smcginnis | Yeah, git remote update, git fetch origin, git pull --rebase, none of them got it. | 20:05 |
smcginnis | Probably a good time to clean out crud and get some fresh clones for some things anyway. | 20:05 |
fungi | i've fetched refs/notes/* for so long that all my clones probably postdate when i set it | 20:08 |
fungi | though also git should gracefully ignore missing notes files. i have that in my global ~/.gitconfig but still use repos like github.com/python/cpython just fine | 20:09 |
fungi | what command was emitting the warning? | 20:09 |
fungi | git log? | 20:09 |
smcginnis | git show | 20:10 |
smcginnis | No error on git log, but also no notes data. | 20:11 |
fungi | interesting | 20:13 |
fungi | and yeah, for me git show and git log are working warning-free on opendev repo commits which have attached notes but also cpython commits from github which dont | 20:14 |
*** slaweq has quit IRC | 20:15 | |
*** ijolliffe has quit IRC | 21:20 | |
*** tosky has quit IRC | 22:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!