Wednesday, 2024-02-28

opendevreviewMerged openstack/openstack-ansible master: Treat dashes/underscores as interchangeable symbols for container names  https://review.opendev.org/c/openstack/openstack-ansible/+/90543200:04
opendevreviewMerged openstack/openstack-ansible stable/2023.1: Remove yoga->antelope slurp upgrade jobs  https://review.opendev.org/c/openstack/openstack-ansible/+/91022000:26
opendevreviewAleksandr Chudinov proposed openstack/openstack-ansible-os_glance master: Add property protection configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/90982007:09
jrossernoonedeadpunk: thanks for the advice with the public network / magnum, it's passing now https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90519908:30
noonedeadpunkjrosser: the only comment I have on that was https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/905199/28/zuul.d/jobs.yaml08:33
noonedeadpunkor well, there's also a debug output I'm not sure about...08:34
jrosserah you are right, will tidy up08:34
noonedeadpunkand https://review.opendev.org/c/openstack/openstack-ansible-ops/+/906363 not looking bad at all, imo...08:35
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90519908:36
noonedeadpunkjrosser: and uncomment pls generic jobs :) https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/905199/29/zuul.d/project.yaml08:37
noonedeadpunkI think it's ready to be landed now?08:37
jrosseroh doh :)08:37
jrosserof course!08:37
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Move insecure param to keystone_auth section  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90511008:37
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90519908:38
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90519908:39
noonedeadpunkjrosser:  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/905199/30/zuul.d/nodesets.yml -> nodesets.yaml 08:39
opendevreviewJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add job to test Vexxhost cluster API driver  https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/90519908:40
noonedeadpunkah08:40
noonedeadpunkok, yes, GB vs G08:40
noonedeadpunkseems it finally got scheduled....08:43
jrosserlet me guess we now find some other mess in the regular magnum jobs08:45
noonedeadpunkhaha08:46
noonedeadpunkWell, I just realized that I completely forgot about openstack_resources conversion patches for magnum and octavia...08:47
noonedeadpunkand they're not merged and not passing CI08:47
noonedeadpunkor well. they were at some point, but not latest run...08:47
noonedeadpunkand also slightly o_O on https://zuul.opendev.org/t/openstack/build/a83d9ea4154745d7983a0eb5ceed6cd608:48
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Do not use underscores in container names  https://review.opendev.org/c/openstack/openstack-ansible/+/90543308:50
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Allow env.d to contain underscores in physical_skel  https://review.opendev.org/c/openstack/openstack-ansible/+/90543808:50
opendevreviewJonathan Rosser proposed openstack/ansible-role-python_venv_build master: Workaround ansible unsafe text templating bug  https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/91047708:53
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Bump ansible version to 2.15.9  https://review.opendev.org/c/openstack/openstack-ansible/+/90561908:53
jrosser^ this is guesswork just now but the code path is very different if you tell the package: module which package manager to use instead of 'auto'08:54
noonedeadpunkI wonder if we should try jumping to 2.16.4 if it works...08:56
noonedeadpunkfrom one side it's late, but probably not too late...08:56
noonedeadpunkor there were some concerns on Python versions....08:57
noonedeadpunkah. btw, yesterday we've found that passlib in python is in quite bad shape and not working on latest python versions08:58
noonedeadpunkon which ansible relies heavily. not only ansible though08:58
opendevreviewAleksandr Chudinov proposed openstack/openstack-ansible-os_glance master: Add property protection configuration  https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/90982009:31
jrossernoonedeadpunk: yeah for 2.16.4 i think there might be some deprecations to deal with so i held off on that09:37
jrosserof course can stack another patch on top to do that09:37
noonedeadpunkI guess you're right and makes sense at least to get 2.15.9 working properly first...09:38
jrosserit think that adding `use: "{{ ansible_facts['pkg_mgr'] }}"` to the package: module will let us move the ansible version forward10:25
jrosserlooks like it is working10:25
noonedeadpunkgreat news10:53
noonedeadpunkyeah, I see things passing10:54
jrosseroh yes so the regular magnum jobs are not looking good13:30
noonedeadpunkok, so feels like we'd need openstack_resources merged first13:41
jrossernoonedeadpunk: something not quote right here https://opendev.org/openstack/openstack-ansible/commit/1d59c06a5daee14cfddfca3e85fc5a7ec8642d7713:41
jrosseroh wait yes13:41
noonedeadpunkbut upgrade jobs are still failing :(13:41
noonedeadpunkhttps://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/901185?tab=change-view-tab-header-zuul-results-summary13:41
noonedeadpunkor wait, status is just not reported back yet :(13:42
noonedeadpunkyeah, but it's the same in zuul13:42
noonedeadpunkupgrade jobs are passing until they are TLS ones13:42
noonedeadpunksomehow TLS only is getting broken...13:42
jrosserfailing still https://zuul.opendev.org/t/openstack/status#90118513:43
noonedeadpunkbut only TLS... how does that make any sense....13:45
jrosserDetails: [{'request_id': '', 'code': 'client', 'status': 400, 'title': 'Cluster type (vm, Unset, kubernetes) not supported', 'detail': 'Cluster type (vm, Unset, kubernetes) not supported.', 'links': []}]13:45
noonedeadpunkyeah, like no image13:46
jrossermaybe it can’t talk to glance with tls14:05
noonedeadpunkthat would be weird error then....14:06
jrossersadly we never made magnum/heat driver work so I don’t have a reference to compare with14:06
noonedeadpunkWe do have it working as of today14:06
noonedeadpunkand over tls...14:06
jrosserhmm14:06
noonedeadpunkand don't have anything sdpecial14:06
noonedeadpunk(anything special in terms of overrrides)14:08
noonedeadpunkexcept defining default volume type explicitly (otherwise it does random choice)14:09
noonedeadpunksand we don;t stor /etc on upgrade jobs quite unfortunatelly :(14:10
noonedeadpunkI can try to reproduce it one more time of now, when there're less dependencies14:10
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Update upstream SHAs  https://review.opendev.org/c/openstack/openstack-ansible/+/91038614:11
opendevreviewJonathan Rosser proposed openstack/openstack-ansible master: Bump ansible version to 2.16.4  https://review.opendev.org/c/openstack/openstack-ansible/+/91051514:33
spatelAnybody here using cinder NFS backend? any advice or config I should be aware? 15:11
spatelI have deployed and everything looks good but somehow service is showing down. 15:12
spatelI can see NFS mounted on cinder-volume and permissions are all good 15:12
noonedeadpunkfinally fully migrated out from NFS15:35
noonedeadpunkadvice - don't use NFS15:35
jrossernoonedeadpunk: ewwwwwww https://review.opendev.org/c/openstack/openstack-ansible/+/910515?tab=change-view-tab-header-zuul-results-summary17:31
jrosserpython / ansible really does move faster than the LTS OS releases now17:31
noonedeadpunkwell17:34
noonedeadpunkwe should drop bullseye on 2024.1 I assume17:34
noonedeadpunkbut then we'll need to wait for el10....17:34
jrosserthere is certainly some things we can fix there in the meantime17:35
noonedeadpunkjammy/bookworm kinda pass in the meanwhile17:37
jrosserpretty much yes17:39
jrosserthere is an `include` deprecation which breaks the ceph job from inside ceph-ansible17:40
jrossershould we abandon open changes on V / W / X branches?17:48
noonedeadpunkceph-ansible is topic on it's own17:50
noonedeadpunkWell, I'd love to try fixing CI for some of these branches, but realistically - no time for that right now17:51
jrosserare they also all about to become unmaintained/<x>17:52
jrosseror is that only for yoga17:52
noonedeadpunkit's only from yoga and later17:53
noonedeadpunkbtw17:53
noonedeadpunkwe have broken renos due to yoga as well17:53
noonedeadpunkwhich I can't reproduce locally somehow17:53
jrosserlots of fallout/breakage from this change it seems17:55
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible master: Add ovn-bgp-agent to source install requirements  https://review.opendev.org/c/openstack/openstack-ansible/+/90969417:55
* noonedeadpunk not fan of it17:55
jrosserand it breaks things with many years of finessing built into them so are not always just easy to fix17:55
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Add support for ovn-bgp-agent deployment  https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/90978017:56
noonedeadpunkhm, but I somehow still do have origin/stable/yoga locally...18:04
opendevreviewOpenStack Release Bot proposed openstack/openstack-ansible master: reno: Update master for unmaintained/yoga  https://review.opendev.org/c/openstack/openstack-ansible/+/90849918:27
fricklervwx ist also planned to go unmaintained https://review.opendev.org/c/openstack/releases/+/91041419:06
noonedeadpunkfrickler: um, am I reading TC resolution linked there in a wrong way?19:31
noonedeadpunkthe first sentence is a bit confusing then I guesss19:37
noonedeadpunk"The current Extended Maintenance branches are not SLURP releases and wouldn’t be eligible for Unmaintained branches with the guidelines above"19:37
noonedeadpunkAs indeed, if start reading bullet point below, this one ^ is not addressing any branches?19:37
noonedeadpunkAs "Until the first SLURP release ends its maintained phase, all current branches are eligible for Unmaintained" -> Zed/Yoga19:38
noonedeadpunkBut "The last 3 active Extended Maintenance branches are automatically transitioned to Unmaintained branches." -> V/W/X ?19:38
gmannnoonedeadpunk: because SLURP thing started after 2023.1 only, moving three latest EM branches v/w/x to unmaintained are best effort for smooth migration to 'unmaintained' model19:47
opendevreviewMerged openstack/openstack-ansible-plugins stable/2023.1: Add override for gluster host used for bootstrap operations  https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/90986819:48
gmannwhen we will be in 2023.1 moving from maintained phase it will be clear with 'only SLURP release goes to unmaintained and non-slurp to EOL directly'19:50
noonedeadpunkgmann: yeah. I just somehow thought that existing EM are left intact19:57
noonedeadpunkand only ones created after resolution are affected19:57
gmannI remember we discussed that and did not do it to avoid confusion and extra work to maintain both 'EM' and 'unmaintianed' model19:58
noonedeadpunkAs it seems that first part of transition is still addressign this discussion, whilke second bullet point overrules it19:59
noonedeadpunkbut ok20:00
noonedeadpunkthe most confusing part is still the unmaintained group - have no time to reply Brian on the ML though :(20:00
gmannyeah, 2nd bullet just cover the temporary EM branches for this transition20:00
noonedeadpunkgmann: I think a new reno release might be needed though, before proceeding with EOM tags20:05
noonedeadpunkwrt https://review.opendev.org/c/openstack/reno/+/91054720:05
noonedeadpunkor well, at least with branches removal20:05
gmannnoonedeadpunk: so is that not covered by update of release note source from stable/unmaintained - https://review.opendev.org/q/topic:%22reno-eom-yoga%2220:10
gmannbecause unmaintained branches will be tagged as EOL when removal. that is my understanding but we can check with frickler /release team20:11
jrosserthe saddest part is that we have even more work than before to do anything on the new branch names20:11
jrosseras there is a ton of assumed stable/x in all sorts of places20:12
gmannyeah, changing the namespace added a lot of work but that is at least will be one time thing and less overahead of maintaining them as EM for 18 months or so20:13
gmannhopefully we will automate bot for such changes in future. it is just a start and we find many places/scripts to update it20:13
jrosserwell no one except me or noonedeadpunk are on the hook for that in osa20:14
gmannsetting up devstck/tempest testing for unmaintained is also good amount of work20:14
jrosserand we have dozens of repos20:14
gmanni understand that. and I am also realizing that moving vwx also to unmaintained adding a lot of work. 20:16
jrosserwhile I’m on this - the deployment tools often stretch the support beyond what the other projects do20:17
jrosserthat now super difficult as we have to somehow deal with changing branch names underneath something we still consider supported from an end user POV20:17
opendevreviewMerged openstack/openstack-ansible-os_nova stable/2023.1: Evaluate my_ip address once  https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/90869921:58

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