*** dviroel|out is now known as dviroel | 01:56 | |
*** dviroel is now known as dviroel|out | 02:18 | |
*** ysandeep|holiday is now known as ysandeep | 04:14 | |
noonedeadpunk | damiandabrowski: btw, 849420 and 846954 may be quite independant and not in relation chain | 06:13 |
---|---|---|
noonedeadpunk | And I posted a comment to https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/848642/3/releasenotes/notes/tempest_plugins_format-e37eaf74f608487f.yaml | 06:13 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/victoria: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848718 | 06:17 |
opendevreview | Dmitriy 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/+/849225 | 06:27 |
opendevreview | Dmitriy 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/+/849225 | 06:31 |
opendevreview | Dmitriy 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/+/849225 | 06:31 |
*** ysandeep is now known as ysandeep|afk | 06:31 | |
*** dmsimard0 is now known as dmsimard | 06:44 | |
opendevreview | Merged openstack/openstack-ansible-os_rally stable/wallaby: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848721 | 07:46 |
opendevreview | Merged openstack/openstack-ansible-haproxy_server master: Allow customisation of stick-tables for each service. https://review.opendev.org/c/openstack/openstack-ansible-haproxy_server/+/848657 | 07:52 |
jrosser | morning | 07:54 |
*** ysandeep|afk is now known as ysandeep | 07:59 | |
damiandabrowski | hi! | 08:11 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849227 | 08:32 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849227 | 08:33 |
noonedeadpunk | jrosser: I think if we merge wallaby, we can avoid disabling and the recovering upgrade jobs | 08:34 |
jrosser | yeah i was just thinking about that | 08:34 |
jrosser | oh i see we have backported releasenotes here too - is that bad? | 08:35 |
opendevreview | Jonathan Rosser proposed openstack/openstack-ansible-os_rally stable/yoga: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849227 | 08:37 |
opendevreview | Damian 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/+/848642 | 08:37 |
noonedeadpunk | jrosser: I find reno involves a bit of magic:) But from what I recall it;s bad to backport renos for integrated repo only | 08:37 |
noonedeadpunk | as 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 commit | 08:39 |
opendevreview | Damian 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/+/848643 | 08:39 |
noonedeadpunk | so backporting these should be kind of fine | 08:39 |
opendevreview | Damian 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/+/846954 | 08:39 |
opendevreview | Damian 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/+/849420 | 08:39 |
noonedeadpunk | that would involve minor release though, as new feature is being implemented | 08:41 |
jrosser | so for wallaby we should be able to merge this next? https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/846122 | 08:41 |
opendevreview | Merged openstack/openstack-ansible-os_rally master: Return upgrade jobs to voting https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848778 | 09:54 |
opendevreview | Merged openstack/openstack-ansible-os_rally stable/victoria: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/848718 | 09:58 |
opendevreview | Merged openstack/ansible-config_template master: Publish openstack.config_template on galaxy https://review.opendev.org/c/openstack/ansible-config_template/+/846391 | 10:08 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/victoria: Re-enable bionic jobs https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849493 | 10:42 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/wallaby: Re-enable disabled zuul jobs https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849494 | 10:43 |
noonedeadpunk | yup, I believe it should land now | 10:45 |
opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_rally stable/xena: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849229 | 10:45 |
noonedeadpunk | and now we can also fix Xena as upgrade jobs should pass there now | 10:46 |
jrosser | hmm how far back do we have to go fixing upgrade jobs i wonder | 11:13 |
*** dviroel|out is now known as dviroel|rover | 11:29 | |
*** ysandeep is now known as ysandeep|break | 11:59 | |
jrosser | uh-oh - POST FAILURES all over the place | 12:11 |
noonedeadpunk | Victoria 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 |
jrosser | is there a SHA bump needed for the os_rally role? | 12:20 |
jrosser | on victoria? | 12:20 |
noonedeadpunk | I don't think so... | 12:29 |
noonedeadpunk | It should be on stable/victoria... | 12:29 |
noonedeadpunk | Unless I forgot to push smth | 12:29 |
noonedeadpunk | https://opendev.org/openstack/openstack-ansible/src/branch/stable/victoria/ansible-role-requirements.yml#L188 | 12:30 |
jrosser | ah yes | 12:43 |
*** ysandeep|break is now known as ysandeep | 12:46 | |
anskiy | looks like mistral cron triggers are broken, when api runs within uwsgi | 14:15 |
mgariepy | what error do you see ? | 14:18 |
mgariepy | anskiy, and which release / os are you on ? | 14:18 |
anskiy | no 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 Oo | 14:20 |
anskiy | periodic 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 |
mgariepy | mistral has splitted the openstack actions into another repo at some point. | 14:27 |
mgariepy | https://opendev.org/openstack/mistral-extra/src/branch/stable/ussuri/mistral_extra | 14:27 |
mgariepy | seems like it was for U | 14:27 |
mgariepy | we do have a patch for master. but i don't think we backported it. https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/844049 | 14:28 |
noonedeadpunk | mgariepy: should we try backporting this then? https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/844049 | 14:28 |
noonedeadpunk | :D | 14:29 |
mgariepy | haha :D | 14:29 |
mgariepy | probably worth backporting.. | 14:30 |
mgariepy | since it's pretty much useless without it.. unless you want to do no-op and fetch a url... | 14:30 |
noonedeadpunk | yeah, likely worth doing it then | 14:30 |
anskiy | mgariepy: I already install them via "mistral_user_pip_packages", and I'm testing this thing with "std.noop" | 14:37 |
mgariepy | noonedeadpunk, does this needs to be backported https://review.opendev.org/c/openstack/openstack-ansible/+/846135 ? | 14:38 |
anskiy | if that was addressed to me... | 14:38 |
noonedeadpunk | yup. I guess it was | 14:38 |
noonedeadpunk | mgariepy: I think yes? No idea tbh | 14:39 |
mgariepy | lol ok | 14:39 |
noonedeadpunk | won't hurt and can be squashed with https://review.opendev.org/c/openstack/openstack-ansible/+/844048 | 14:39 |
noonedeadpunk | anskiy: so once you stop uwsgi and start as a service it starts working? | 14:43 |
noonedeadpunk | then we can implement compatability and provide variable like we do for glance. Should be quite easy to do | 14:44 |
anskiy | noonedeadpunk: yeah. Should whole cron_trigger->enabled be dependent on it? It's set to True by default. | 14:45 |
noonedeadpunk | I have no idea about mistral to have that said | 14:46 |
noonedeadpunk | never used it anywhere | 14:46 |
noonedeadpunk | anskiy: so using uwsgi or not should depend on cron_trigger value, right? | 14:47 |
mgariepy | if some part of mistral does require it not to be in uwsgh. i guess we should just configure it without. | 14:48 |
anskiy | noonedeadpunk: 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 defaults | 14:52 |
anskiy | mgariepy: 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 uwsgi | 14:52 |
*** ysandeep is now known as ysandeep|out | 15:05 | |
noonedeadpunk | #startmeeting openstack_ansible_meeting | 15:09 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:09 |
opendevmeet | The meeting name has been set to 'openstack_ansible_meeting' | 15:09 |
noonedeadpunk | #topic rollcall | 15:09 |
jrosser | o/ hello | 15:09 |
NeilHanlon | o/ | 15:09 |
noonedeadpunk | sorry for being late again. Lost my alarm clock for starting meeting :( | 15:09 |
damiandabrowski | hi! | 15:09 |
mgariepy | hey | 15:10 |
* noonedeadpunk just got notified that lsyncd available in centos 9s :D | 15:10 | |
noonedeadpunk | #topic office hours | 15:11 |
noonedeadpunk | talking about centos, are there any news from Rocky ?:) | 15:11 |
NeilHanlon | We'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 it | 15:13 |
noonedeadpunk | nice! | 15:13 |
jrosser | hopefully it should be relatively painless as the centos-9 support is done | 15:14 |
noonedeadpunk | We just unblocked our gates after branching | 15:14 |
jrosser | i was just about to start going through taking out any centos/rocky-8 special cases on master | 15:14 |
noonedeadpunk | maybe we should wait then until r9 just in case? | 15:15 |
noonedeadpunk | but not sure | 15:15 |
noonedeadpunk | not sure in terms of value for such special cases | 15:15 |
jrosser | was just looking for PTG notes to see what we want to prioritise working on | 15:19 |
noonedeadpunk | I think main pain point for now are broken stable ranches because of rally | 15:19 |
noonedeadpunk | I think these would be internal ssl? | 15:20 |
noonedeadpunk | I'd love to have that for sure | 15:20 |
jrosser | that is sure a big job | 15:21 |
noonedeadpunk | and quite good profit | 15:21 |
noonedeadpunk | iirc we haven't merged bueprint yet | 15:22 |
noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible-specs/+/822850 | 15:22 |
noonedeadpunk | it looked pretty fair as well as poc. I have a bit vague memories of that topic, but it was super close | 15:23 |
jrosser | i think the basic idea is figured out with glance as an example | 15:24 |
jrosser | and the dual-mode haproxy setup | 15:24 |
noonedeadpunk | Btw regarding PTG | 15:26 |
noonedeadpunk | Until August 12th (basically a month) I need to signup team for it | 15:27 |
noonedeadpunk | As you might now it's in person even in US. | 15:27 |
noonedeadpunk | So I was wondering if anybody was thinking about going there | 15:27 |
mgariepy | what is the date ? | 15:27 |
mgariepy | October 17-20, 2022 | 15:29 |
noonedeadpunk | Yup | 15:29 |
noonedeadpunk | Columbus, OH | 15:29 |
noonedeadpunk | As I was thinking of going there, but not sure how to handle that. | 15:33 |
NeilHanlon | don't go there, tis a silly place | 15:33 |
noonedeadpunk | that = tema registration | 15:33 |
noonedeadpunk | *team | 15:33 |
noonedeadpunk | NeilHanlon: oh, why? | 15:34 |
NeilHanlon | joking, mostly. it's about as dangerous as most American cities, which isn't saying much lol | 15:34 |
NeilHanlon | I would offer to drive/fly out there, not too far for me in Massachusetts, but the timing wouldn't line up unfortunately | 15:35 |
*** dviroel|rover is now known as dviroel|rover|biab | 15:36 | |
noonedeadpunk | ah, gotcha | 15:37 |
noonedeadpunk | I will try to think about how to make it in a mixed way actually. So everybody would be able to participate | 15:38 |
jrosser | i am really not having much desire to travel tbh | 15:39 |
noonedeadpunk | As eventually hearing talks during summit I think in-person PTG is good thing now | 15:39 |
mgariepy | i probably wouldn't go either. | 15:39 |
mgariepy | but i would do online :) | 15:39 |
* noonedeadpunk wonder if other teams also prefer online | 15:40 | |
noonedeadpunk | I think another good/big thing is support of tick/tock releases and upgrades through releases | 15:44 |
noonedeadpunk | Should be quite easy to implement but already I feel pain from circular dependencies | 15:44 |
noonedeadpunk | (that's what we discussed on previous PTG) | 15:44 |
jrosser | does that really end up with running N-2 upgrade jobs as well as N-1 | 15:47 |
* jrosser guesses it's more subtle than that | 15:47 | |
jrosser | and maybe we have to keep one extra older branch alive | 15:48 |
opendevreview | Merged openstack/openstack-ansible-rabbitmq_server stable/wallaby: Use cloudsmith repo for rabbit and erlang https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/846122 | 15:54 |
noonedeadpunk | regarding keeping older branches alive, I think it should be taken into account for release cadence anyway | 15:55 |
noonedeadpunk | but yes | 15:56 |
noonedeadpunk | my biggest fear is how subtle CI would be. | 15:57 |
noonedeadpunk | But we have quite good stats regarding upgrade jobs, and looking back to V - they're still passing if not erlang and rally... | 15:57 |
noonedeadpunk | more problematic would be to keep distro support | 15:57 |
noonedeadpunk | but 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 releases | 15:59 |
*** frickler is now known as frickler_pto | 16:00 | |
noonedeadpunk | #endmeeting | 16:04 |
opendevmeet | Meeting ended Tue Jul 12 16:04:59 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:04 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.html | 16:05 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.txt | 16:05 |
opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_ansible_meeting/2022/openstack_ansible_meeting.2022-07-12-15.09.log.html | 16:05 |
damiandabrowski | btw. I would really appreciate reviews on my latest tempest changes ;) https://review.opendev.org/q/topic:tempest-damian-2022-07 | 16:17 |
*** dviroel|rover|biab is now known as dviroel|rover | 16:42 | |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/xena: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849574 | 18:34 |
opendevreview | Marc 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/+/849575 | 18:36 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/wallaby: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849518 | 18:37 |
opendevreview | Marc 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/+/849519 | 18:37 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/wallaby: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849518 | 18:39 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/victoria: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849520 | 18:41 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/victoria: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849520 | 18:42 |
mgariepy | jrosser, are you around ? | 18:46 |
jrosser | yeah | 18:46 |
mgariepy | wondering if you should backport : https://github.com/openstack/openstack-ansible-os_mistral/commit/16539214fbd0a49abe840f20abae63bbefb96cb5 | 18:46 |
mgariepy | to v. | 18:46 |
mgariepy | or i need to edit the os_mistral patch for v instead. | 18:47 |
jrosser | i 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 V | 18:47 | |
mgariepy | i'm not either | 18:48 |
mgariepy | mistral-extra stuff need to be backported to U. | 18:48 |
jrosser | those all landed about the same time i think https://review.opendev.org/q/topic:osa-new-pip | 18:48 |
jrosser | so it might be that we need to adjust the mistral backport for the 'old way' with pip | 18:49 |
mgariepy | ok | 18:49 |
mgariepy | i'll do that. | 18:49 |
mgariepy | if it breaks we can fix it later | 18:49 |
opendevreview | Marc 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/+/849578 | 18:52 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/ussuri: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849521 | 18:54 |
opendevreview | Marc Gariépy proposed openstack/openstack-ansible stable/ussuri: Add mistra-extra repo https://review.opendev.org/c/openstack/openstack-ansible/+/849521 | 18:57 |
opendevreview | Marc 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/+/849522 | 18:58 |
mgariepy | done.. let's wait on the CI now :D | 18:58 |
noonedeadpunk | btw some rally patches are ready to be reviewed https://review.opendev.org/q/project:openstack/openstack-ansible-os_rally+status:open | 20:27 |
opendevreview | Merged openstack/openstack-ansible master: Add default rate-limits for API endpoints and Horizon authentication https://review.opendev.org/c/openstack/openstack-ansible/+/848659 | 20:58 |
opendevreview | Merged openstack/openstack-ansible-os_rally stable/xena: Control rally-openstack installed version https://review.opendev.org/c/openstack/openstack-ansible-os_rally/+/849229 | 22:25 |
opendevreview | Merged openstack/openstack-ansible-rabbitmq_server stable/victoria: Use cloudsmith repo for rabbit and erlang https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/849225 | 22:50 |
*** ysandeep|out is now known as ysandeep | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!