Monday, 2020-07-13

*** hamalq has joined #openstack-oslo05:21
*** hamalq has quit IRC05:25
*** Luzi has joined #openstack-oslo05:37
*** rcernin has quit IRC06:02
*** rcernin has joined #openstack-oslo06:11
*** tosky has joined #openstack-oslo06:39
*** ralonsoh has joined #openstack-oslo07:19
*** rcernin has quit IRC08:06
*** hamalq has joined #openstack-oslo08:17
*** hamalq_ has joined #openstack-oslo08:18
*** hamalq has quit IRC08:21
*** hamalq_ has quit IRC08:23
*** dtantsur|afk is now known as dtantsur08:34
*** moguimar has joined #openstack-oslo09:08
*** rcernin has joined #openstack-oslo09:53
*** rcernin has quit IRC10:03
*** rcernin has joined #openstack-oslo10:07
*** rcernin has quit IRC10:40
*** finucannot is now known as stephenfin10:57
*** giblet is now known as gibi11:17
*** hamalq has joined #openstack-oslo11:19
*** hamalq has quit IRC11:24
*** hamalq has joined #openstack-oslo11:35
*** tkajinam has quit IRC11:37
moguimarstephenfin, thanks for the reviews =D11:40
*** hamalq has quit IRC11:40
*** raildo has joined #openstack-oslo11:56
*** adriant has quit IRC11:58
*** adriant has joined #openstack-oslo11:59
*** hberaud has quit IRC12:01
*** hberaud has joined #openstack-oslo12:03
*** dave-mccowan has joined #openstack-oslo12:10
*** dave-mccowan has quit IRC12:15
*** ralonsoh has quit IRC12:43
*** ralonsoh has joined #openstack-oslo12:44
*** kgiusti has joined #openstack-oslo12:49
*** lbragstad_ is now known as lbragstad12:51
*** zombieJulia is now known as TheJulia13:06
*** sean-k-mooney has quit IRC13:27
*** irclogbot_1 has quit IRC13:27
*** gibi has quit IRC13:27
*** sean-k-mooney has joined #openstack-oslo13:28
*** irclogbot_1 has joined #openstack-oslo13:28
*** gibi has joined #openstack-oslo13:28
*** Luzi has quit IRC14:00
openstackgerritMerged openstack/stevedore master: sphinxext: fix warning message for detailed list  https://review.opendev.org/74059614:20
dansmithbeekneemech: can I do an *or* type operation in a policy rule?14:31
dansmithbeekneemech: also, is there a more appropriate policy guru I should be asking?14:31
*** beekneemech is now known as bnemec14:38
bnemecdansmith: Yes. I believe the boolean ops are all just their literal name: and, or, not.14:39
dansmithoh really?14:39
dansmithdid I just assume obscurity? :)14:39
bnemecI can't imagine why. ;-)14:39
bnemecAlso, the keystone folks are more clued in about writing policy rules. They're the ones who had to figure out how to write all of these fancy policy rules in the first place.14:40
dansmithah okay14:41
*** cognifloyd has quit IRC14:51
*** cognifloyd has joined #openstack-oslo14:55
bnemec#startmeeting oslo15:00
bnemecCourtesy ping for bnemec, smcginnis, moguimar, johnsom, stephenfin, bcafarel, kgiusti, jungleboyj15:00
bnemec#link https://wiki.openstack.org/wiki/Meetings/Oslo#Agenda_for_Next_Meeting15:00
openstackMeeting started Mon Jul 13 15:00:47 2020 UTC and is due to finish in 60 minutes.  The chair is bnemec. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: oslo)"15:00
openstackThe meeting name has been set to 'oslo'15:00
johnsomo/15:01
kgiustio/15:01
toskyhi from your friendly zuulv3 community goal volunteer15:01
moguimaro/15:02
bnemeco/ tosky!15:02
bnemec#topic Red flags for/from liaisons15:04
*** openstack changes topic to "Red flags for/from liaisons (Meeting topic: oslo)"15:04
johnsomNothing from the Octavia team15:04
bnemecWe had a major release of stevedore last week that broke something.15:04
bnemecThere's a fix slated for release today.15:04
moguimarall fine in Barbican land15:05
smcginniso/15:05
bnemecOkay, I think that's all of the liaisons, so we can move on.15:06
bnemec#topic Releases15:06
*** openstack changes topic to "Releases (Meeting topic: oslo)"15:06
bnemechberaud has been handling most of these lately, so I'm a little out of the loop.15:07
bnemecGenerally speaking, I think our releases should be pretty current though.15:07
smcginnisI think that is accurate.15:09
bnemecHmm, no action items for one-off topics.15:09
bnemecAnd my VM that hosts reviewstats seems to be down, so weekly wayward review is problematic.15:10
moguimarI have some15:10
bnemec#topic Open discussion15:10
*** openstack changes topic to "Open discussion (Meeting topic: oslo)"15:10
toskyif I may :)15:10
moguimargo on tosky15:11
bnemecmoguimar: A review or a topic?15:11
moguimarI'll bring mine after15:11
moguimarI need to colect some links15:11
bnemecSounds good.15:11
toskyI'm here for the Zuul v3 porting goal; the overall status of oslo is good, but there are still a few jobs that needs some porting15:12
bnemecCool. Do you have a list?15:12
toskyI haven't sent a proper report yet (drafting it right now) but I can share this as a special preview: https://etherpad.opendev.org/p/goal-victoria-native-zuulv3-migration15:12
toskyoslo ones are around line 19715:13
bnemecWe can just kill that zmq job. We don't support that driver anymore and haven't for a while.15:14
toskyif I've got all of them, there are 10 legacy jobs, and 4 of them are grenade (which got their native zuul v3 version only recently)15:14
bnemecI assume there's a known migration path for grenade?15:14
toskyoh, kill the job or close the repository, whatever works for you; I'm fine as long as the job is not around :)15:14
toskyyes, grenade job received a native job just before branching ussuri thanks to, uhm15:15
*** hamalq has joined #openstack-oslo15:15
toskythis one https://review.opendev.org/#/c/548936/15:15
toskyif you look for merged patches with topic:native-zuulv3-migration, you will see a few grenade ones, even pretty complex (see ironic)15:16
*** hamalq has quit IRC15:17
bnemecOkay, I doubt we're doing anything crazy with it, so hopefully those won't be too hard.15:17
toskyjust a few random grenade jobs: https://review.opendev.org/#/c/726790/ https://review.opendev.org/#/c/729860/ https://review.opendev.org/#/c/703098/15:17
*** hamalq has joined #openstack-oslo15:17
toskyI haven't checked what these jobs do, but I think they won't be complicated to port as well15:17
toskyif you have any question please feel free to ping me (and add me as reviewer)15:17
bnemecLooking at the list, the ones that may require some Oslo-specific work are the messaging and pbr ones.15:17
kgiustibnemec: I'll take a crack at the messaging ones.15:18
bnemeckgiusti: Awesome, thanks!15:18
toskythe messaging ones may be easy, if the only things they do is to use oslo.messaging from source15:18
bnemec#action kgiusti to look at migrating messaging jobs to zuulv3.15:18
toskyjust add oslo.messaging to required-projects15:18
toskybut again I haven't checked the job in details :)15:18
kgiustitosky: I'll take a look and get back to you with questions15:19
toskysure!15:19
kgiustitosky: thanks :D15:19
*** hamalq_ has joined #openstack-oslo15:20
toskyyou probably know that already, but the migration to zuulv3 provides most of the work for the other community goal, the port from bionic to focal15:20
bnemecOkay, I've added this topic to the agenda for next week too so hopefully we won't lose track of it like we did with the docs goal...15:20
bnemecWe'll probably find some obscure deps for one of our drivers that doesn't exist on focal. :-)15:21
toskyonce the base jobs switch the ubuntu version, all native zuul v3 job are switched as well (which may bring some breakages in tests, but at least the general part is done)15:21
toskyeh15:21
toskythat's the project-specific part15:21
bnemecYep. We'll whack those moles when they pop their heads up.15:22
moguimaryou're done tosky ?15:23
toskyI think that's it from my side for now, yes15:23
toskythanks!15:23
moguimarack15:23
bnemecThanks tosky!15:23
moguimarI was working with herve last week on https://review.opendev.org/#/c/720771/15:23
*** hamalq has quit IRC15:23
moguimarpre-commit is very good as a tool, but I have my doubts that it will be as effective as a git hook15:25
moguimarso while anyone can install the hooks locally, I think we should also enforce pre-commit at gate15:26
moguimarusually when you run pre-commit, it will only consider modified/staged files15:26
moguimarso at the gate we can either run on all source or in the last commit15:26
bnemecIf we don't have it in the gate it will most likely get broken over time.15:26
moguimarand I think it should go in the pep8 check15:27
bnemecLast commit should be fine in the gate, at least once we get the whole project compliant in the first place.15:27
bnemecWould you propose adding it to the pep8 tox env then?15:28
moguimarin a following patch15:28
moguimarreplacing the flake8 like15:28
moguimaras flake8 is also included in pre-commit checks15:29
moguimarhttps://review.opendev.org/#/c/720771/6/tox.ini15:29
moguimarbasically replace line 27 with line 35 in a followup patch15:29
moguimarwith an intermediate patch making everything compliant15:29
moguimarso 1st, I need some one else to review this patch, 2nd, get to an agreement for gating15:30
moguimar2nd point shouds positive so far15:30
bnemecI don't have any strong objections. It looks like the stuff pre-commit is checking is all sane.15:30
moguimaryup, I'd +2 it but then I made some changes on it15:31
moguimarwith herve's consent15:31
bnemecstephenfin: ^might be relevant to your interests since I believe you were advocating for pre-commit too15:31
moguimar3rd point on this is a raise dead from the discuss list15:31
moguimarI'd like to, in yet another patch, introduce black through pre-commit15:32
bnemecThat I object to.15:32
moguimarthe impact on oslo.cache is around 18 files at first15:32
bnemecWe've discussed black in the past and didn't want to deal with the churn it would cause.15:33
moguimarthat was for big projects like openstack sdk15:33
moguimarI was looking at the options15:33
moguimarwe cound have 80 lines and ignore "" vs ''15:33
smcginnisPlease no black.15:33
bnemecI mean, we have 40 projects. Collectively I think that qualifies as "big".15:33
moguimarok, so lets stick to points 1 and 2 then15:34
* stephenfin is in another meeting but agrees with pre-commit integration (reviewed it this morning). Integrating it into the gate seems sound though we didn't bother doing it for nova since pep8 was the only one we really cared about and that was already validated15:34
moguimarso do we have a tribute to review https://review.opendev.org/#/c/720771 and I will work on the gating patch?15:36
bnemecI've approved that patch.15:37
moguimarach, thanks Ben15:37
bnemecWe can discuss how to handle the gating on the followup.15:37
moguimarI think we can watch pre-commit during this cycle and then decide on applying it to other projects next cycle15:38
bnemecI can't remember if there were any objections to gating on it when it was discussed previously.15:38
moguimarlike a topic in the retrospective or so15:38
bnemecI would even make it a standalone topic for the next PTG.15:39
moguimarok15:39
bnemecIt'll be easier to have that discussion not in the context of the overall retrospective.15:40
bnemec#action moguimar to propose patch integrating pre-commit to the gate for oslo.cache15:40
bnemecAnything else on this topic for now?15:41
moguimarnope15:41
moguimarand I don't have anything else15:41
bnemecOkay, anything else this week?15:42
bnemecLooks like "no". :-)15:44
bnemecThanks for joining, everyone!15:44
bnemec#endmeeting15:44
*** openstack changes topic to "OpenStack Common Libraries | https://wiki.openstack.org/wiki/Oslo"15:44
openstackMeeting ended Mon Jul 13 15:44:24 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:44
openstackMinutes:        http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-07-13-15.00.html15:44
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-07-13-15.00.txt15:44
openstackLog:            http://eavesdrop.openstack.org/meetings/oslo/2020/oslo.2020-07-13-15.00.log.html15:44
smcginnisbnemec: https://review.opendev.org/#/c/740704/ was updated based on Doug's feedback.15:44
bnemecThat's what I get for not actually looking at the patch. ;-)15:45
smcginnisI didn't really think it would be considered a new feature, but probably safer that way.15:45
bnemecYeah, a feature bump isn't a big deal.15:46
openstackgerritMerged openstack/oslo.messaging stable/queens: Simplify tools/test-setup.sh  https://review.opendev.org/73992216:14
*** sean-k-mooney has quit IRC17:01
openstackgerritMerged openstack/oslo.cache master: Introducing usage of pre-commit  https://review.opendev.org/72077117:10
*** dtantsur is now known as dtantsur|afk17:15
*** ralonsoh has quit IRC17:23
zanebkgiusti: it appears we are having an issue in Heat where we continue to send messages to a queue that only one engine is listening on, so that when it gets shut down we lose messages: http://eavesdrop.openstack.org/irclogs/%23heat/%23heat.2020-07-13.log.html#t2020-07-13T19:04:4219:53
zanebis there any documentation anywhere on how you're supposed to set queues up for these various purposes?19:54
kgiustizaneb: ping - sorry in a meeting.  IIRC the 'amqp_auto_delete' configuration setting in [oslo_messaging_rabbit] controls what gets passed into kombu when a queue is created: https://docs.openstack.org/oslo.messaging/latest/configuration/opts.html#oslo-messaging-rabbit20:26
*** raildo has quit IRC21:10
*** raildo has joined #openstack-oslo21:11
*** raildo has quit IRC21:17
*** hamalq_ has quit IRC22:02
*** hamalq has joined #openstack-oslo22:03
*** tosky has quit IRC22:42
*** tkajinam has joined #openstack-oslo22:54
*** rcernin has joined #openstack-oslo23:05
*** rcernin has quit IRC23:05
*** rcernin has joined #openstack-oslo23:08
*** hoonetorg has quit IRC23:24
*** adriant has quit IRC23:32
*** adriant has joined #openstack-oslo23:33
*** hoonetorg has joined #openstack-oslo23:39
*** hamalq has quit IRC23:50
*** moguimar has quit IRC23:55

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