Tuesday, 2022-07-12

*** dviroel|out is now known as dviroel01:56
*** dviroel is now known as dviroel|out02:18
*** ysandeep|holiday is now known as ysandeep04:14
noonedeadpunkdamiandabrowski: btw, 849420 and 846954 may be quite independant and not in relation chain06:13
noonedeadpunkAnd I posted a comment to https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/848642/3/releasenotes/notes/tempest_plugins_format-e37eaf74f608487f.yaml06:13
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/victoria: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84871806:17
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/victoria: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84922506:27
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/victoria: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84922506:31
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-rabbitmq_server stable/victoria: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84922506:31
*** ysandeep is now known as ysandeep|afk06:31
*** dmsimard0 is now known as dmsimard06:44
opendevreviewMerged openstack/openstack-ansible-os_rally stable/wallaby: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84872107:46
opendevreviewMerged openstack/openstack-ansible-haproxy_server master: Allow customisation of stick-tables for each service.  https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/84865707:52
jrossermorning07:54
*** ysandeep|afk is now known as ysandeep07:59
damiandabrowskihi!08:11
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84922708:32
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84922708:33
noonedeadpunkjrosser: I think if we merge wallaby, we can avoid disabling and the recovering upgrade jobs08:34
jrosseryeah i was just thinking about that08:34
jrosseroh i see we have backported releasenotes here too - is that bad?08:35
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84922708:37
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Make _tempest_plugins a list of dicts.  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84864208:37
noonedeadpunkjrosser: I find reno involves a bit of magic:) But from what I recall it;s bad to backport renos for integrated repo only08:37
noonedeadpunkas reno distingushes release with pbr and commit sha. For cherry-picks sha remains the same which confuises reno. But for everything except integrated, we collect release notes with bump script, and merge with independant commit08:39
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Allow to define extra tempest plugins  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84864308:39
noonedeadpunkso backporting these should be kind of fine08:39
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Fix the behavior of defining fixed_network_name  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84695408:39
opendevreviewDamian Dąbrowski proposed openstack/openstack-ansible-os_tempest master: Use network UUID for floating_network_name  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84942008:39
noonedeadpunkthat would involve minor release though, as new feature is being implemented08:41
jrosserso for wallaby we should be able to merge this next? https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84612208:41
opendevreviewMerged openstack/openstack-ansible-os_rally master: Return upgrade jobs to voting  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84877809:54
opendevreviewMerged openstack/openstack-ansible-os_rally stable/victoria: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84871809:58
opendevreviewMerged openstack/ansible-config_template master: Publish openstack.config_template on galaxy  https://review.opendev.org/c/openstack/ansible-config_template/+/84639110:08
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/victoria: Re-enable bionic jobs  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84949310:42
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/wallaby: Re-enable disabled zuul jobs  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84949410:43
noonedeadpunkyup, I believe it should land now10:45
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/xena: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84922910:45
noonedeadpunkand now we can also fix Xena as upgrade jobs should pass there now10:46
jrosserhmm how far back do we have to go fixing upgrade jobs i wonder11:13
*** dviroel|out is now known as dviroel|rover11:29
*** ysandeep is now known as ysandeep|break11:59
jrosseruh-oh - POST FAILURES all over the place12:11
noonedeadpunkVictoria was enough this time. But I wonder why we fail https://zuul.opendev.org/t/openstack/build/029f73e5b1ab4f45b7a58f745425456a/log/job-output.txt#17286 byt the time of re-check fix for victoria was already merged...12:17
jrosseris there a SHA bump needed for the os_rally role?12:20
jrosseron victoria?12:20
noonedeadpunkI don't think so...12:29
noonedeadpunkIt should be on stable/victoria...12:29
noonedeadpunkUnless I forgot to push smth12:29
noonedeadpunkhttps://opendev.org/openstack/openstack-ansible/src/branch/stable/victoria/ansible-role-requirements.yml#L18812:30
jrosserah yes12:43
*** ysandeep|break is now known as ysandeep12:46
anskiylooks like mistral cron triggers are broken, when api runs within uwsgi 14:15
mgariepywhat error do you see ?14:18
mgariepyanskiy, and which release / os are you on ?14:18
anskiyno errors, I just don't see any executions created. I'm on X. I think, it's because periodic tasks engine resides in API: https://opendev.org/openstack/mistral/src/branch/master/mistral/api/app.py#L57-L60 Oo14:20
anskiyperiodic tasks engine -- probably "service" would be more correct term: https://opendev.org/openstack/mistral/src/branch/master/mistral/services/periodic.py and it's threaded by itself too.14:24
mgariepymistral has splitted the openstack actions into another repo at some point. 14:27
mgariepyhttps://opendev.org/openstack/mistral-extra/src/branch/stable/ussuri/mistral_extra14:27
mgariepyseems like it was for U 14:27
mgariepywe do have a patch for master. but i don't think we backported it. https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84404914:28
noonedeadpunkmgariepy: should we try backporting this then? https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84404914:28
noonedeadpunk:D14:29
mgariepyhaha :D14:29
mgariepyprobably worth backporting.. 14:30
mgariepysince it's pretty much useless without it.. unless you want to do no-op and fetch a url...14:30
noonedeadpunkyeah, likely worth doing it then14:30
anskiymgariepy: I already install them via "mistral_user_pip_packages", and I'm testing this thing with "std.noop"14:37
mgariepynoonedeadpunk,  does this needs to be backported https://review.opendev.org/c/openstack/openstack-ansible/+/846135 ?14:38
anskiyif that was addressed to me...14:38
noonedeadpunkyup. I guess it was14:38
noonedeadpunkmgariepy: I think yes? No idea tbh14:39
mgariepylol ok 14:39
noonedeadpunkwon't hurt and can be squashed with https://review.opendev.org/c/openstack/openstack-ansible/+/84404814:39
noonedeadpunkanskiy: so once you stop uwsgi and start as a service it starts working?14:43
noonedeadpunkthen we can implement compatability and provide variable like we do for glance. Should be quite easy to do14:44
anskiynoonedeadpunk: yeah. Should whole cron_trigger->enabled be dependent on it? It's set to True by default.14:45
noonedeadpunkI have no idea about mistral to have that said14:46
noonedeadpunknever used it anywhere14:46
noonedeadpunkanskiy: so using uwsgi or not should depend on cron_trigger value, right?14:47
mgariepyif some part of mistral does require it not to be in uwsgh. i guess we should just configure it without.14:48
anskiynoonedeadpunk: it could be, but cron_trigger is implicitly enabled in mistral's source code, so cron_trigger->enabled should be set to False in role defaults14:52
anskiymgariepy: https://opendev.org/openstack/openstack-ansible-os_mistral/src/branch/master/defaults/main.yml#L55-L84 API is the only part that's run within uwsgi14:52
*** ysandeep is now known as ysandeep|out15:05
noonedeadpunk#startmeeting openstack_ansible_meeting15:09
opendevmeetMeeting started Tue Jul 12 15:09:23 2022 UTC and is due to finish in 60 minutes.  The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot.15:09
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:09
opendevmeetThe meeting name has been set to 'openstack_ansible_meeting'15:09
noonedeadpunk#topic rollcall15:09
jrossero/ hello15:09
NeilHanlono/15:09
noonedeadpunksorry for being late again. Lost my alarm clock for starting meeting :(15:09
damiandabrowskihi!15:09
mgariepyhey15:10
* noonedeadpunk just got notified that lsyncd available in centos 9s :D15:10
noonedeadpunk#topic office hours15:11
noonedeadpunktalking about centos, are there any news from Rocky ?:)15:11
NeilHanlonWe'll be releasing 9.0 this week :) I started a review for DIB to add in a build for 9, but i need to check in on it15:13
noonedeadpunknice!15:13
jrosserhopefully it should be relatively painless as the centos-9 support is done15:14
noonedeadpunkWe just unblocked our gates after branching15:14
jrosseri was just about to start going through taking out any centos/rocky-8 special cases on master15:14
noonedeadpunkmaybe we should wait then until r9 just in case?15:15
noonedeadpunkbut not sure15:15
noonedeadpunknot sure in terms of value for such special cases15:15
jrosserwas just looking for PTG notes to see what we want to prioritise working on15:19
noonedeadpunkI think main pain point for now are broken stable ranches because of rally15:19
noonedeadpunkI think these would be internal ssl?15:20
noonedeadpunkI'd love to have that for sure15:20
jrosserthat is sure a big job15:21
noonedeadpunkand quite good profit15:21
noonedeadpunkiirc we haven't merged bueprint yet15:22
noonedeadpunkhttps://review.opendev.org/c/openstack/openstack-ansible-specs/+/82285015:22
noonedeadpunkit looked pretty fair as well as poc. I have a bit vague memories of that topic, but it was super close15:23
jrosseri think the basic idea is figured out with glance as an example15:24
jrosserand the dual-mode haproxy setup15:24
noonedeadpunkBtw regarding PTG15:26
noonedeadpunkUntil August 12th (basically a month) I need to signup team for it15:27
noonedeadpunkAs you might now it's in person even in US.15:27
noonedeadpunkSo I was wondering if anybody was thinking about going there15:27
mgariepywhat is the date ?15:27
mgariepyOctober 17-20, 202215:29
noonedeadpunkYup15:29
noonedeadpunkColumbus, OH15:29
noonedeadpunkAs I was thinking of going there, but not sure how to handle that.15:33
NeilHanlondon't go there, tis a silly place15:33
noonedeadpunkthat = tema registration15:33
noonedeadpunk*team15:33
noonedeadpunkNeilHanlon: oh, why?15:34
NeilHanlonjoking, mostly. it's about as dangerous as most American cities, which isn't saying much lol15:34
NeilHanlonI would offer to drive/fly out there, not too far for me in Massachusetts, but the timing wouldn't line up unfortunately15:35
*** dviroel|rover is now known as dviroel|rover|biab15:36
noonedeadpunkah, gotcha 15:37
noonedeadpunkI will try to think about how to make it in a mixed way actually. So everybody would be able to participate15:38
jrosseri am really not having much desire to travel tbh15:39
noonedeadpunkAs eventually hearing talks during summit I think in-person PTG is good thing now15:39
mgariepyi probably wouldn't go either.15:39
mgariepybut i would do online :)15:39
* noonedeadpunk wonder if other teams also prefer online15:40
noonedeadpunkI think another good/big thing is support of tick/tock releases and upgrades through releases15:44
noonedeadpunkShould be quite easy to implement but already I feel pain from circular dependencies15:44
noonedeadpunk(that's what we discussed on previous PTG)15:44
jrosserdoes that really end up with running N-2 upgrade jobs as well as N-115:47
* jrosser guesses it's more subtle than that15:47
jrosserand maybe we have to keep one extra older branch alive15:48
opendevreviewMerged openstack/openstack-ansible-rabbitmq_server stable/wallaby: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84612215:54
noonedeadpunkregarding keeping older branches alive, I think it should be taken into account for release cadence anyway15:55
noonedeadpunkbut yes15:56
noonedeadpunkmy biggest fear is how subtle CI would be.15:57
noonedeadpunkBut we have quite good stats regarding upgrade jobs, and looking back to V - they're still passing if not erlang and rally...15:57
noonedeadpunkmore problematic would be to keep distro support15:57
noonedeadpunkbut again, given that upgrade path should be followed by services, they should not implement breaking changes in the meanwhile. But might be issues with upgrade releases15:59
*** frickler is now known as frickler_pto16:00
noonedeadpunk#endmeeting16:04
opendevmeetMeeting ended Tue Jul 12 16:04:59 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:04
opendevmeetMinutes:        https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.html16:05
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.txt16:05
opendevmeetLog:            https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.log.html16:05
damiandabrowskibtw. I would really appreciate reviews on my latest tempest changes ;) https://review.opendev.org/q/topic:tempest-damian-2022-0716:17
*** dviroel|rover|biab is now known as dviroel|rover16:42
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/xena: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84957418:34
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-os_mistral stable/xena: Add mistral-extra in the mistral venv  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84957518:36
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/wallaby: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84951818:37
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-os_mistral stable/wallaby: Add mistral-extra in the mistral venv  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84951918:37
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/wallaby: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84951818:39
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/victoria: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84952018:41
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/victoria: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84952018:42
mgariepyjrosser, are you around ?18:46
jrosseryeah18:46
mgariepywondering if you should backport : https://github.com/openstack/openstack-ansible-os_mistral/commit/16539214fbd0a49abe840f20abae63bbefb96cb518:46
mgariepyto v.18:46
mgariepyor i need to edit the os_mistral patch for v instead.18:47
jrosseri guess the backport is cleaner.... is that one we missed before?18:47
* jrosser not sure if that new pip resolver stuff was needed in V18:47
mgariepyi'm not either18:48
mgariepymistral-extra stuff need to be backported to U.18:48
jrosserthose all landed about the same time i think https://review.opendev.org/q/topic:osa-new-pip18:48
jrosserso it might be that we need to adjust the mistral backport for the 'old way' with pip18:49
mgariepyok18:49
mgariepyi'll do that.18:49
mgariepyif it breaks we can fix it later18:49
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-os_mistral stable/victoria: Add mistral-extra in the mistral venv  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84957818:52
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/ussuri: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84952118:54
opendevreviewMarc Gariépy proposed openstack/openstack-ansible stable/ussuri: Add mistra-extra repo  https://review.opendev.org/c/openstack/openstack-ansible/+/84952118:57
opendevreviewMarc Gariépy proposed openstack/openstack-ansible-os_mistral stable/ussuri: Add mistral-extra in the mistral venv  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84952218:58
mgariepydone.. let's wait on the CI now :D18:58
noonedeadpunkbtw some rally patches are ready to be reviewed https://review.opendev.org/q/project:openstack/openstack-ansible-os_rally+status:open20:27
opendevreviewMerged openstack/openstack-ansible master: Add default rate-limits for API endpoints and Horizon authentication  https://review.opendev.org/c/openstack/openstack-ansible/+/84865920:58
opendevreviewMerged openstack/openstack-ansible-os_rally stable/xena: Control rally-openstack installed version  https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/84922922:25
opendevreviewMerged openstack/openstack-ansible-rabbitmq_server stable/victoria: Use cloudsmith repo for rabbit and erlang  https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/84922522:50
*** ysandeep|out is now known as ysandeep23:58

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