Friday, 2020-08-07

*** tetsuro has joined #openstack-tc00:33
*** diablo_rojo has quit IRC00:48
*** markvoelker has joined #openstack-tc02:31
*** markvoelker has quit IRC02:35
*** markvoelker has joined #openstack-tc02:37
*** markvoelker has quit IRC02:47
*** markvoelker has joined #openstack-tc04:31
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-tc04:33
*** markvoelker has quit IRC04:36
*** lxkong has quit IRC04:42
*** tkajinam has quit IRC05:34
*** tkajinam has joined #openstack-tc05:35
*** tkajinam has quit IRC05:50
*** tkajinam has joined #openstack-tc05:51
*** markvoelker has joined #openstack-tc06:21
*** markvoelker has quit IRC06:26
*** redrobot has quit IRC06:37
*** slaweq has joined #openstack-tc07:00
*** slaweq has quit IRC07:05
*** e0ne has joined #openstack-tc07:11
*** tetsuro has quit IRC07:17
*** slaweq has joined #openstack-tc07:50
*** markvoelker has joined #openstack-tc07:51
*** ralonsoh has joined #openstack-tc07:54
*** markvoelker has quit IRC07:56
*** tosky has joined #openstack-tc08:04
*** dmellado has quit IRC08:09
*** dmellado has joined #openstack-tc08:14
*** tetsuro has joined #openstack-tc08:41
*** tetsuro has quit IRC08:41
*** markvoelker has joined #openstack-tc09:04
*** markvoelker has quit IRC09:09
*** tkajinam has quit IRC09:17
*** ricolin has quit IRC09:59
*** markvoelker has joined #openstack-tc11:05
*** markvoelker has quit IRC11:10
*** cloudnull has quit IRC11:42
*** cloudnull has joined #openstack-tc11:43
*** ijolliffe has joined #openstack-tc12:30
openstackgerritMerged openstack/ideas master: New communication tools proposal  https://review.opendev.org/71893213:02
njohnstono/13:06
gmanno/13:12
evrardjpo/13:29
*** dirk has quit IRC13:31
openstackgerritJean-Philippe Evrard proposed openstack/governance master: Resolution to define distributed leadership for projects  https://review.opendev.org/74499513:58
*** dklyle has quit IRC14:40
*** david-lyle has joined #openstack-tc14:40
*** david-lyle is now known as dklyle14:41
*** Guest7899 has joined #openstack-tc15:03
*** Guest7899 is now known as redrobot15:06
*** ralonsoh has quit IRC15:08
*** ralonsoh has joined #openstack-tc15:08
*** e0ne has quit IRC15:21
*** gibi is now known as gibi_pto15:24
*** gibi_pto is now known as gibi_pto_til_24t15:24
*** gibi_pto_til_24t is now known as gibi_pto_24th15:24
*** ralonsoh has quit IRC17:02
jungleboyjo/17:52
openstackgerritSean McGinnis proposed openstack/governance-sigs master: Propose new Hardware Vendor SIG  https://review.opendev.org/74518518:11
*** slaweq has quit IRC19:03
*** slaweq has joined #openstack-tc19:06
*** KeithMnemonic has quit IRC19:31
mnasergmann: neat, looks like only `opendev/puppet-apps_site`, `opendev/puppet-releasestatus` and `opendev/puppet-stackalytics` are the ones that aren't "clean" yet19:34
mnaseri guess we could have the script ignore anything that's not in the openstack/ namespace?19:34
mnaserleaving `openstack/python-dracclient`, `openstack/networking-l2gw`, `openstack/networking-l2gw-tempest-plugin` and `openstack/neutron-vpnaas` as the ones to figure out19:35
gmannmnaser: yeah, i did not push the cleanup on those as acl to tc are only for openstack/19:35
mnaseropenstack/neutron-vpnaas seems still alive..19:35
gmannyeah openstack/neutron-vpnaas is not retired19:35
mnaserso should we unretire that in governance i guess?19:35
gmannopenstack/networking-l2gw is in progress19:35
fungiright, i wouldn't bother enforcing it for anything which moved out of the openstack namespace before the mandatory retirement resolution took effect19:36
gmannis it in legacy file ?19:36
mnasergmann: `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 so19:36
mnaserlet me find the commit that added it19:36
mnaseroh wow, retired in `2016-11-01` supposidly..19:36
mnaserbut it's had releases for quite sometime too19:36
fungiretired from neutron, but not retired from being developed19:37
gmannyeah, we should remove it from https://opendev.org/openstack/governance/src/branch/master/reference/legacy.yaml#L51019:37
mnaserfungi: but it's being released under openstack19:37
mnaserlike an actual openstack deliverable exists19:37
gmannas per slaweq  it is active deliverable  now19:37
fungioh, was it unretired?19:37
smcginnisWe have governance checks. Is it listed as owned by an official team *and* also listed as legacy?19:38
gmanni think yes19:38
mnaserhttps://releases.openstack.org/ussuri/index.html#ussuri-neutron-vpnaas19:38
fungishould be removed from legacy.yaml if it's resurrected out of retirement19:38
mnasermaybe that's the problem :)19:38
gmannsmcginnis: yeah in both19:38
njohnstonyes, I think that is the issue19:38
smcginnisI seem to recall either that one or -fwaas going through a period where they were looking for maintainers.19:38
mnasertrying to trace down the original commit19:39
smcginnisMaybe it was started to be retired, then someone stepped up and it ended up back in neutron.19:39
mnaserso this was retirement https://github.com/openstack/governance/commit/20f95dd947d2f87519b4bb50fb188e6f71deae7c19:39
fungihttps://review.openstack.org/50601219:39
fungiSubmitted-at: Tue, 19 Dec 2017 20:54:33 +000019:39
mnaserhttps://github.com/openstack/governance/commit/feb90ea14b0ceacf670bf8fce9a1ea6fdb69ab8d19:40
fungiwas the resurrection19:40
mnaserhttps://review.opendev.org/#/c/527255/19:40
smcginnisProbably around the time the neutron stadium idea came up.19:40
mnaserso this missed removing it from legacy.yaml19:40
fungiso it's been an official deliverable again (added back to reference/projects.yaml) since late 201719:40
fungiyep, i concur19:40
gmannhttps://review.opendev.org/#/c/52725519:41
gmannyeah that one19:41
njohnstonin 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.html19:42
fungibtw, 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 cetera19:42
openstackgerritMohammed Naser proposed openstack/governance master: Drop neutron-vpnaas from legacy projects  https://review.opendev.org/74540119:42
fungimakes running down stuff from history much easier19:43
openstackgerritMohammed Naser proposed openstack/governance master: Add legacy repository validation  https://review.opendev.org/73755919:43
mnaserand that's a rebase19:43
mnaserlet me also make it ignore opendev19:43
fungioh, and set notesRef = refs/notes/review19:43
fungisince that's how gerrit stores them19:43
mnaserthat's beyond my git-fu :)19:44
gmannmnaser: and also blacklist these three repo for now ?  -openstack/python-dracclient`, `openstack/networking-l2gw`, `openstack/networking-l2gw-tempest-plugin19:44
gmannwith TODO19:45
smcginnis[insert shameless plug for reviews on https://review.opendev.org/745185]19:45
openstackgerritMohammed Naser proposed openstack/governance master: Add legacy repository validation  https://review.opendev.org/73755919:48
openstackgerritMohammed Naser proposed openstack/governance master: Drop all exceptions for legacy validation  https://review.opendev.org/74540319:48
gmannsmcginnis: overall lgtm, once comment on SIG status19:48
gmann'active' can be moved once SIG start meeting/ML updates etc - https://governance.openstack.org/sigs/reference/sig-status.html19:49
mnaseri believe the current legacy validation job will pass as of now19:49
*** slaweq has quit IRC19:50
smcginnisSure, 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
smcginnisgmann: Would you prefer I change it though?19:50
gmanntc-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
gmannsmcginnis:  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
smcginnisOK, fair enough. Update coming right up.19:53
gmannit may be just after week or few days when we approved19:53
openstackgerritSean McGinnis proposed openstack/governance-sigs master: Propose new Hardware Vendor SIG  https://review.opendev.org/74518519:53
gmannsmcginnis:  thanks , +1. i think only rico form TC has the +2 right on governance-sigs and spotz or ttx may be19:54
spotzgmann smcginnis - looking19:55
fungimnaser: 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
fungier, ~/.gitconfig i mean19:56
smcginnisI was wondering how gitignore played into this. ;)19:57
smcginnisGood tips fungi!19:57
*** slaweq has joined #openstack-tc19:58
fungihow it plays in is that i also have a global ~/.gitignore set in my ~/.gitconfig, i guess ;)19:58
smcginnisHaha19:58
smcginnisfungi: Is there a trick to update the local refs? I get a "warning: notes ref refs/notes/review is invalid" with that.20:01
smcginnisFresh clone works though.20:02
fungihuh, interesting. i'm not sure, you might be able to mkdir -p .git/refs/notes&&touch .git/refs/notes/review or something i guess20:04
fungidid you git remote update first?20:04
fungii would have normally expected that to do it automagically20:04
smcginnisYeah, git remote update, git fetch origin, git pull --rebase, none of them got it.20:05
smcginnisProbably a good time to clean out crud and get some fresh clones for some things anyway.20:05
fungii've fetched refs/notes/* for so long that all my clones probably postdate when i set it20:08
fungithough 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 fine20:09
fungiwhat command was emitting the warning?20:09
fungigit log?20:09
smcginnisgit show20:10
smcginnisNo error on git log, but also no notes data.20:11
fungiinteresting20:13
fungiand 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 dont20:14
*** slaweq has quit IRC20:15
*** ijolliffe has quit IRC21:20
*** tosky has quit IRC22:58

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!