Thursday, 2022-07-14

*** dviroel|rover is now known as dviroel|out00:14
*** ysandeep|out is now known as ysandeep02:46
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server stable/xena: Bump mariadb version.  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84743506:12
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-galera_server stable/xena: Bump mariadb version.  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84743506:13
*** ysandeep is now known as ysandeep|afk06:46
opendevreviewMerged openstack/openstack-ansible-galera_server stable/wallaby: Bump mariadb version.  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84778208:32
jrossermorning08:49
jrosserlooks like nearly all the wallaby stuff is merged08:49
damiandabrowskihi, thanks for reviews on my tempest changes!08:55
noonedeadpunkjrosser: yeas, I already launched bump script08:55
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/xena: Switch ARA report type from html to database for openstack-ansible jobs  https://review.opendev.org/c/openstack/openstack-ansible/+/84979708:57
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/wallaby: Switch ARA report type from html to database for openstack-ansible jobs  https://review.opendev.org/c/openstack/openstack-ansible/+/84981208:58
anskiynoonedeadpunk: it is too late to add one to that change, now that it is merged? Should I create another one with release note only?09:03
jrosseranskiy: you can add a release note in another patch if you like09:04
opendevreviewJonathan Rosser proposed openstack/openstack-ansible stable/victoria: Switch ARA report type from html to database for openstack-ansible jobs  https://review.opendev.org/c/openstack/openstack-ansible/+/84979809:07
noonedeadpunkanskiy: given that we're to release it now, probably doesn't make sense09:09
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/wallaby: Bump OpenStack-Ansible for Wallaby  https://review.opendev.org/c/openstack/openstack-ansible/+/84979909:10
anskiynoonedeadpunk: next time then :(09:17
noonedeadpunkanskiy: oh, wait, you was taling about https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/849716 ? For it it makes sense to land reno)09:26
anskiynoonedeadpunk: yeah. I'll do it then. 09:28
jrosserurgh https://zuul.opendev.org/t/openstack/build/43494561c06d4fd082a20637d3a1a0b310:49
opendevreviewDanila Balagansky proposed openstack/openstack-ansible-os_mistral master: Add release notes for mistral_api_use_uwsgi  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84980410:49
*** ysandeep|afk is now known as ysandeep11:05
*** dviroel__ is now known as dviroel|rover11:15
noonedeadpunkwhaaaaat11:52
noonedeadpunkI have no idea what is that11:54
noonedeadpunkError in some release note?11:54
jrosserit could be12:00
mgariepyhttps://review.opendev.org/c/openstack/openstack-ansible/+/849799/1/releasenotes/notes/mariadb_security_upgrade-24e89cb878cc1389.yaml maybe ?12:04
mgariepyhttps://github.com/openstack/openstack-ansible/blob/master/releasenotes/notes/smart-sources-59cd0811dcf1ae49.yaml12:05
mgariepyidentation error.. <reno.sphinxext stable/wallaby>:62:Unexpected indentation.12:06
damiandabrowskimgariepy: You're right, was able to reproduce this issue locally, started to work fine after fixing this note12:17
damiandabrowskihttps://paste.openstack.org/show/bQ7SNVbXVAFgawAc2c4u/12:17
mgariepythe ci needs fix then. it should have been caught on https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84778212:23
damiandabrowskiyeah...12:24
damiandabrowskiPS. this form also works fine: https://paste.openstack.org/show/bkTO7ZPqAgUwTcccGiaD/12:24
noonedeadpunkyes, we need to push patch to fix original release not first12:28
noonedeadpunkcan somebody do that?:)12:29
noonedeadpunkbtw CI question is good12:29
noonedeadpunkI wonder why it was not caught12:29
mgariepylooks like the check doesn't really build the current re.12:30
mgariepylooks like the check doesn't really build the current release.12:30
mgariepyhttps://541e2403478ac154d5eb-056bfb946e355d1a1a86dc411a70c5ec.ssl.cf2.rackcdn.com/847782/4/gate/build-openstack-releasenotes/481939c/docs/unreleased.html12:31
noonedeadpunkugh12:37
*** ysandeep is now known as ysandeep|break12:38
mgariepyreally not sure where to look to debug that tho 12:49
mgariepymaybe we need to poke infra as they are the one providing the jobs? maybe we can go check other projects if they have the same issue.12:49
noonedeadpunkI think our issue is that we don't publish renos for roles nowadays. Maybe we should return them back... But what I wanted is to shut down independant reno pages for roles12:52
noonedeadpunkLikely that idea was a mistkae12:52
noonedeadpunk*mistake12:52
noonedeadpunkI will try look into that sometime soon12:53
noonedeadpunkwould be great to fix reno at least now....12:54
*** ysandeep|break is now known as ysandeep12:56
*** ysandeep is now known as ysandeep|afk13:01
opendevreviewDamian DÄ…browski proposed openstack/openstack-ansible-galera_server stable/wallaby: Fix mariadb release note  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84985513:04
opendevreviewDamian DÄ…browski proposed openstack/openstack-ansible stable/wallaby: Bump OpenStack-Ansible for Wallaby  https://review.opendev.org/c/openstack/openstack-ansible/+/84979913:06
damiandabrowskihmm, not sure what to do with 849799. Should i edit broken release note manually there? 13:08
damiandabrowskii think another way is to abandon it, wait until 849855 is merged and create bump again13:08
noonedeadpunkwell, depends on now is waste of CI.... I was thinking to update bump once 849855 merges13:11
noonedeadpunkno need to abandon though13:11
damiandabrowskiahh sure13:11
noonedeadpunkmanual update technically will work as well13:11
noonedeadpunkbut file must be identical, so better to bump probably13:11
jrosserdo we need to run that release notes formatting job in some other places too?13:21
jrosserto catch this before we merge stuff13:22
mgariepyif the job would have built the relnotes for wallaby it would have been caught13:24
jrosserwe probably don't want to publish them13:27
jrosserjust run the generation step13:27
mgariepyisn't what build-openstack-releasenotes supposed to do ?13:28
mgariepyhttps://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/847782/13:28
mgariepyfrom the logs it seems that it build current - 1 but not current. (which is wallaby in that patch)13:29
jrosserah well i think whats going on there is the releasenote was committed to one of our roles and merged13:44
jrosserthen we copy it over to the openstack-ansible repo for when making a release13:44
jrosseronly at that point did it break, rather than on the original patch13:45
opendevreviewchandan kumar proposed openstack/openstack-ansible-os_tempest stable/train: Remove tripleo jobs from stable/train branch  https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/84986113:47
opendevreviewDanila Balagansky proposed openstack/openstack-ansible-os_mistral master: Add release notes for mistral_api_use_uwsgi  https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/84980413:52
opendevreviewMerged openstack/openstack-ansible-galera_server stable/wallaby: Fix mariadb release note  https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/84985513:59
spatelDoes OSA has any documentation related integrate external Ceph storage ?14:28
jrosserspatel: there is some stuff here https://docs.openstack.org/openstack-ansible-ceph_client/latest/configure-ceph.html14:37
jrosseras usual everything is very configurable and what happens depends on if you have ssh access to the ceph cluster, or not14:38
spatelPerfect!!! This is great14:44
spatelI am creating blog to cover storage so need that example to show 14:45
spatelusing cephadm based deployment of ceph storage14:46
jrosserif OSA can ssh to the mon hosts then most of it is populating ceph_mons list with the IP14:47
jrosserno idea what that means for cephadm based stuff though, as it expects to find ceph.conf on the host14:47
*** ysandeep|afk is now known as ysandeep16:04
*** ysandeep is now known as ysandeep|out16:30
*** dviroel|rover is now known as dviroel|rover|brb16:35
*** dviroel|rover|brb is now known as dviroel|rover17:35
*** dviroel|rover is now known as dviroel|rover|afk21:59

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