| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Move Murano/Senlin/Sahara to Inactive state https://review.opendev.org/c/openstack/openstack-ansible/+/916900 | 07:55 |
|---|---|---|
| jrosser_ | there is a very large amount of code review outstanding and i think we are also light on reviewers for a some time | 09:55 |
| jrosser_ | NeilHanlon: mgariepy if you are able to find time to take a look at the reviews regularly as we approach release it would be super helpful | 09:56 |
| jrosser_ | argh need to also go over cherry picking the backport candidates | 09:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_gnocchi master: Ensure Gnocchi is connected to MySQL coordination with TLS https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/918036 | 10:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_aodh master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/917947 | 10:51 |
| noonedeadpunk | jrosser_: I think I went through ones that had backport label | 10:51 |
| jrosser_ | i think we have the rmq ha stuff that merged recently | 11:16 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible/+/916043 - yeah | 11:20 |
| noonedeadpunk | https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916042 - this was proposed already | 11:21 |
| noonedeadpunk | I'm also quite eager to create new stable releases tbh | 11:21 |
| jrosser_ | yeah sure | 11:26 |
| jrosser_ | i'm also worried we have not made progress on the unmaintained branches | 11:27 |
| jrosser_ | there was talk in the tc about what might happen if we don't get the CI up and running again quickly | 11:27 |
| jrosser_ | i think for victoria i got to a point where some input on what to do next was needed | 11:28 |
| jrosser_ | as the umaintainted/<blah> branches are not necessarily present for all the things we use, and it's then a case to find the right git identifier to use | 11:28 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server stable/2023.2: Remove RabbitMQ restart when changing policy https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916993 | 11:29 |
| noonedeadpunk | I'm slightly /o\ with everything going on right now frankly speaking. And we got second bug report basically about openstack.cloud.identity_domain renaming Default to default | 11:38 |
| noonedeadpunk | https://bugs.launchpad.net/openstack-ansible/+bug/2064718 | 11:39 |
| noonedeadpunk | I wonder if we should somehow review what can potentially supply `default` to service_setup role as a domain, and convert that to Default | 11:40 |
| noonedeadpunk | but I do believe that module behaviour is just broken... | 11:40 |
| noonedeadpunk | though they're almost unmaintained, and it's a sig, which means it's not treated as normal project by tc... | 11:41 |
| noonedeadpunk | which is all /o\ | 11:41 |
| noonedeadpunk | about unmaintained - level of testing for them is not defined, it can be just docs and linters fwiw... | 11:41 |
| noonedeadpunk | while it's would be great to keep CI more then that - it's not required afaik to keep branches alive | 11:42 |
| noonedeadpunk | I also found couple of nits - gnocchi not using and trove image upload seems uncoverd by openstack_resources | 11:45 |
| noonedeadpunk | *gnocchi not using zookeeper when it's available | 11:45 |
| jrosser_ | so i think victoria is real close | 11:49 |
| jrosser_ | i spent nearly a week on it /o\ | 11:49 |
| jrosser_ | just a case of picking the right branch / tag names i think | 11:49 |
| jrosser_ | and then it also brings another thing, that perhaps at release (or some time) we should just pin all the tempest plugin version to something that is working for that release | 11:50 |
| noonedeadpunk | oh yes | 11:50 |
| noonedeadpunk | we're doing this tempest pinning from Xena I beleive | 11:50 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts stable/2023.2: Ensure codename is lowercased for cloudsig install https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/917736 | 11:51 |
| jrosser_ | as that is also a trivial but somehow awkward thing to do | 11:51 |
| noonedeadpunk | and now we don't in fact update temepst right after branching is done | 11:51 |
| jrosser_ | like looking through pypi at the release dates and guessing at version numbers | 11:51 |
| jrosser_ | ^ thats what i started for victoria but it was super tedious | 11:51 |
| noonedeadpunk | And backport from Wallaby didn't work? | 11:52 |
| jrosser_ | oh well you know i didnt try that :) | 11:52 |
| noonedeadpunk | oh, crap, it was Xena as said... | 11:52 |
| noonedeadpunk | https://opendev.org/openstack/openstack-ansible/src/branch/unmaintained/xena/playbooks/defaults/repo_packages/openstack_testing.yml | 11:52 |
| jrosser_ | its totally possible that those sha are better than master for victoria though | 11:56 |
| opendevreview | Merged openstack/openstack-ansible-rabbitmq_server stable/2023.2: Adjust legacy OpenStack HA policy to make reply queues HA https://review.opendev.org/c/openstack/openstack-ansible-rabbitmq_server/+/916995 | 12:04 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible unmaintained/victoria: Backport openstack_testing from Yoga https://review.opendev.org/c/openstack/openstack-ansible/+/918043 | 12:09 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/2023.2: Apply haproxy_service_configs to hosts idempotently https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/917885 | 12:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_gnocchi master: Ensure Gnocchi is connected to MySQL coordination with TLS https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/918036 | 12:19 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible unmaintained/victoria: DNM - testing unmaintained branch https://review.opendev.org/c/openstack/openstack-ansible/+/916175 | 12:41 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-ceph_client master: Switch default ceph version to reef https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/917015 | 12:59 |
| jrosser_ | oh https://zuul.opendev.org/t/openstack/build/94742cc197ca4fe7b2fd003768fe904a/log/job-output.txt#5794-5805 | 13:07 |
| jrosser_ | i guess for now we dont run the upgrade jobs | 13:08 |
| jrosser_ | we should let the job run to see how the others turn out | 13:08 |
| NeilHanlon | jrosser_: ack. I will add it to the morning routine. | 13:09 |
| noonedeadpunk | yeah, doesn't make sense to run upgrade jobs on unmaintained, imo.. | 13:11 |
| NeilHanlon | i guess the only reason i could see is if we need to ensure SLURP works from an unmaintained branch | 13:17 |
| noonedeadpunk | well. then upgrade job is added to current SLURP | 13:35 |
| noonedeadpunk | I guess question is - if we should test upgrade from previous slurp when it goes to unmaintained | 13:35 |
| NeilHanlon | I sorta feel like unmaintained is, well.. unmaintained | 13:40 |
| noonedeadpunk | yeah, so potentially there's should be no need to test upgrade from unmaintained anymore... | 13:44 |
| jrosser_ | this is all kind of surprise to operators who thought they had 3 branches of stable release to upgrade through | 13:46 |
| jrosser_ | well, 4 soon also | 13:47 |
| jrosser_ | but that is a decision not taken by us | 13:47 |
| mgariepy | are there ppl doing blind upgrade on their prod env ? | 13:47 |
| mgariepy | i do usually test at least once or twice in a vm with a similiar config as my env, then on a testbed, after that the production | 13:49 |
| NeilHanlon | YOLO | 13:49 |
| NeilHanlon | :P | 13:49 |
| mgariepy | it's never like prod but it gives me some confidence that i won't have to work on it all night ahhaha | 13:49 |
| mgariepy | haha NeilHanlon :) | 13:50 |
| mgariepy | yep and i don't want to work all night for an upgrade hahaha | 13:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_gnocchi master: Ensure Gnocchi is connected to MySQL coordination with TLS https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/918036 | 14:25 |
| spatel | does openstack-ansible support opensearch logging ? | 15:00 |
| spatel | I deployed for kolla and it looks great - https://satishdotpatel.github.io/openstack-central-logging-using-opensearch/ | 15:01 |
| spatel | I want to try with OSA | 15:01 |
| noonedeadpunk | no idea | 15:03 |
| noonedeadpunk | I think Jonathad runs just ELK, I do prefer graylog. | 15:04 |
| jrosser_ | opensearch basically == elasticsearch | 15:05 |
| jrosser_ | and i use the stuff we have in the ops repo, but it will certainly not be "just deploy" it like the kolla integration | 15:05 |
| jrosser_ | folks tend to have their own plan or existing thing for logging, so OSA has tried to just leave that open | 15:06 |
| noonedeadpunk | actually, opensearch seems to be a backend to graylog nowadays | 15:06 |
| noonedeadpunk | and then there's also stuff like loki from grafana | 15:07 |
| noonedeadpunk | which also looks kinda neat | 15:07 |
| jrosser_ | i would note that opensearch requires use of the OSS log collection agents from elasticsearch | 15:08 |
| jrosser_ | so it is kind of, messy, in some way | 15:08 |
| noonedeadpunk | ugh, that messy indeed | 15:08 |
| noonedeadpunk | but well | 15:08 |
| jrosser_ | if you want for example journalbeat which is my main thing for it | 15:08 |
| jrosser_ | and filebeat | 15:08 |
| noonedeadpunk | journald-remote is broken anyway kinda... so you still need smth that will send data from it | 15:09 |
| jrosser_ | journalbeat is smart enough to pull all the container logs off the lxc host | 15:09 |
| jrosser_ | it does not care that there are N journal files | 15:09 |
| noonedeadpunk | yeah, same is journal2gelf iirc | 15:10 |
| jrosser_ | right | 15:10 |
| noonedeadpunk | you don't need that in each container | 15:10 |
| jrosser_ | spatel: for me it is kind of like ceph if you are at good scale | 15:11 |
| jrosser_ | should be seperate | 15:11 |
| jrosser_ | but you know, opinions :) | 15:11 |
| noonedeadpunk | I _think_ right now we actually pull logs from hosts using vector.... | 15:12 |
| spatel | opensearch looks neat and clean and has lots of good stuff (I am sure we not going to use all of them) | 15:13 |
| spatel | why are we not using flunted ? | 15:13 |
| jrosser_ | you can use what you like :) | 15:13 |
| noonedeadpunk | so as graylog... and elk... and loki... | 15:13 |
| jrosser_ | like i say, out of the box OSA does not give you anything, you can choose | 15:13 |
| noonedeadpunk | actually, fluentd is another interesting thing | 15:13 |
| noonedeadpunk | I always lack time to make it work | 15:13 |
| noonedeadpunk | jrosser_: actually, I think we could provide a variable to have alternative to journald | 15:14 |
| noonedeadpunk | like fluent | 15:14 |
| noonedeadpunk | but again, time | 15:14 |
| jrosser_ | kind of randomly we were doing some stuff this week that led us to osquery | 15:14 |
| jrosser_ | which is a whole different type of tool | 15:14 |
| noonedeadpunk | yeah | 15:14 |
| noonedeadpunk | that would be neat to be able to enable as well... | 15:14 |
| noonedeadpunk | (have ability to enable) | 15:15 |
| jrosser_ | there is (rotted) stuff in the ops repo | 15:15 |
| noonedeadpunk | spatel: I think main answer - lack of time/priority | 15:15 |
| noonedeadpunk | jrosser_: oh yes, tons of. and not only there | 15:15 |
| spatel | agreed! I am just comparing some stuff from kolla | 15:15 |
| jrosser_ | but also to answer spatel, which to choose for put loads of effort into | 15:16 |
| noonedeadpunk | spatel: and I'd pick incus over any logging initiative easily, for instance | 15:16 |
| jrosser_ | take example of my integration of capi vs the one in kolla | 15:16 |
| spatel | I am running both deployment and gaining some experience about differences | 15:16 |
| noonedeadpunk | that's good one actually :D | 15:16 |
| mgariepy | incus ftw ! | 15:16 |
| jrosser_ | mine is super complicated and basically complete / battle ready | 15:16 |
| jrosser_ | kolla just install the driver and you DIY the rest | 15:17 |
| jrosser_ | so for all these things you have do make some call on how deep / excellent to make the integratio | 15:17 |
| jrosser_ | but what that does mean is whilst mine is complete, the learning curve is pretty bad, and kind of scary for newcomers | 15:18 |
| jrosser_ | so same goes for monitoring | 15:18 |
| jrosser_ | what we have in the ops repo for ELK is completely scalable H/A, all architecural modes of elasticsearch are possible | 15:18 |
| jrosser_ | rather than just "turn it on and pretty graphs" | 15:19 |
| jrosser_ | i find this totally hard to know what the right approach is | 15:19 |
| noonedeadpunk | and then due to changed license we here picked not ELK, which makes tricky to align efforts in a way :( | 15:19 |
| noonedeadpunk | but it's not opensearch again | 15:20 |
| jrosser_ | absolutely - hence everyone has opinion based on their situation and all working on the same thing gets tough | 15:20 |
| noonedeadpunk | there are just too many opinions | 15:20 |
| jrosser_ | spatel: ^ this pretty much sums things up i think :) | 15:20 |
| noonedeadpunk | but I guess if someone wants to bring some integration in to the ops repo - that's pretty much welcomed. but there're not much volunteers I'd say :D | 15:21 |
| jrosser_ | in a lot of ways elk vs opensearch would be just switching up some variables | 15:22 |
| noonedeadpunk | and yup, I totally get that kolla has quite great stuff as well | 15:22 |
| jrosser_ | if the divergence is manageable | 15:22 |
| noonedeadpunk | yeah, sure | 15:27 |
| jrosser_ | oh on monitoring, we are also just starting to use node_exporter | 15:29 |
| jrosser_ | we have memory issues on gpu hosts which are pretty hard to get a handle on so trying that | 15:29 |
| spatel | sorry I was in meeting.. | 15:33 |
| spatel | recently I am working on prometheus to monitor openstack infra | 15:33 |
| spatel | I have created dashboard - https://ibb.co/0C1p4Pj | 15:33 |
| noonedeadpunk | that looks quite neat | 15:35 |
| spatel | I will put that dashboard out on github soon | 15:35 |
| jrosser_ | we have looooads of work on this with also dashboards showing usage per project | 15:37 |
| spatel | There are more, I have created per project based dashboard about vms stats - https://ibb.co/nB5fQxW | 15:37 |
| noonedeadpunk | well. the only way I know how to do that with mapping to projects - is through ceilometer.... | 15:37 |
| spatel | prometheus is great tool :) | 15:37 |
| noonedeadpunk | (meh) | 15:38 |
| * noonedeadpunk likes zabbix | 15:38 | |
| * noonedeadpunk nobody around shares this though | 15:39 | |
| spatel | noonedeadpunk ? | 15:39 |
| spatel | prometheus can easily do per project vms status. I am building dash and will publish soon | 15:45 |
| noonedeadpunk | um. but how it maps instances to projects? | 15:47 |
| noonedeadpunk | screenshot actually shows per vm, not per project | 15:48 |
| spatel | noonedeadpunk thing like this - https://ibb.co/SsZbh2r | 15:49 |
| spatel | Just select project and it will pull all the vms inside that project and show you cpu/memory/disk/network etc... | 15:49 |
| noonedeadpunk | huh, but how it maps that... | 15:49 |
| spatel | prometheus magic :) | 15:49 |
| noonedeadpunk | as I don't think that it's present on libvirt side | 15:50 |
| spatel | https://github.com/kumina/libvirt_exporter | 15:50 |
| noonedeadpunk | and I can't recall openstack exporter caring about it | 15:50 |
| spatel | it has option to send metrics with project tag | 15:50 |
| noonedeadpunk | well, it's inactive as all other libvirt exporters... | 15:50 |
| spatel | it move to other github.. let send it | 15:51 |
| noonedeadpunk | oh, so there's `NovaOwner` tag in domxml... | 15:51 |
| spatel | I am using this repo to build my libvert exporter - https://github.com/zhangjianweibj/prometheus-libvirt-exporter | 15:51 |
| noonedeadpunk | crap, there's indeed <nova:owner> | 15:52 |
| noonedeadpunk | how I overlooked it... | 15:52 |
| spatel | haha | 15:52 |
| noonedeadpunk | lol, this one had last update 3yo as well :D | 15:52 |
| spatel | projectName="hk" | 15:52 |
| spatel | I understand they are old but again its go lang and you can do whatever you want.. | 15:53 |
| noonedeadpunk | nah, I meant if you do `visrh dumpxml instance-*****` you will get project and user uuid who spawned it | 15:53 |
| spatel | It works for me.. and I am happy its sending project tags | 15:53 |
| noonedeadpunk | so yeah, somehow I didn't know it's stored in xml | 15:53 |
| noonedeadpunk | or well in domain at all | 15:54 |
| spatel | In my environment I don't care about user but we do care about project. | 15:54 |
| noonedeadpunk | yeah, makes sense | 15:54 |
| spatel | I can edit go lang and pull whatever metrics I like. its not that hard for go expert. | 15:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_aodh master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/917948 | 15:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_aodh master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/917949 | 15:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_trove master: Manage trove images through openstack_resources role https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/918103 | 16:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ceilometer master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/918105 | 16:52 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ceilometer master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/918107 | 16:57 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/918108 | 17:16 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/918109 | 17:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_cloudkitty master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/918110 | 17:22 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/918111 | 17:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/918111 | 17:28 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/918112 | 17:29 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_designate master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_designate/+/918113 | 17:31 |
| noonedeadpunk | seems, I won't propose all needed changes this week ( | 17:32 |
| nixbuilder | So I have a reliable process to install Bearcat on one server... basically a AIO install. But now I am trying to re-install and this time adding a second infra plus two haproxy servers. But for some reason when the repo_server tries to install I get an error because it cannot launch nginx on the infra server because haproxy already loaded on infra and can't attach to port 8181. I have gone through my config files and | 17:37 |
| nixbuilder | understand why haproxy is installing on infra... I think I am missing something in my config. | 17:37 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/918114 | 17:38 |
| jrosser_ | nixbuilder: just fyi - two infra is a baaad idea long term, you are in almost certain split-brain situation | 17:40 |
| jrosser_ | nixbuilder: you are doing a metal deploy? | 17:42 |
| nixbuilder | jrosser_: Yep... metal on Rocky. So what is the best practice for having two infra nodes to do load balancing? | 17:43 |
| jrosser_ | get a third one | 17:43 |
| jrosser_ | specifically this is a disaster sitation for mariadb | 17:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/918115 | 17:44 |
| jrosser_ | but 2 / 3 servers is not to do with your repo server issue | 17:44 |
| nixbuilder | jrosser_: Ok... I have a third that to use... was just kind of starting slow and just adding things gradually. | 17:44 |
| jrosser_ | what IP did you assign to the infra hosts in openstack_user_config | 17:44 |
| jrosser_ | and then what did you choose for your internal/external VIP | 17:46 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_ironic master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_ironic/+/918116 | 17:46 |
| nixbuilder | jrosser_: https://paste.openstack.org/show/bVEYs3eWN0zx6jRDg6li/ | 17:50 |
| nixbuilder | jrosser_: I know... I probably did something dumb... :-( | 17:50 |
| jrosser_ | right, so, in your aio tests the ip of the 'aio' node is not the same as the ip of the internal vip | 17:51 |
| jrosser_ | and in a metal deploy, nginx will bind to the first one, and haproxy to the second one | 17:51 |
| jrosser_ | in a metal deploy everything is suuuper sensitive to getting the addressing scheme right | 17:52 |
| jrosser_ | so you need to put internal vip `10.255.61.29` somewhere else, or change the IP of the infra host | 17:53 |
| nixbuilder | jrosser_: Hmmm... OK... I think I *may* have it now... at least for the next 30 seconds :-) | 17:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/918117 | 17:55 |
| jrosser_ | no worries, in the network stack an ip:port combination is only available once for a server process like haproxy or nginx to attach/bind to | 17:56 |
| nixbuilder | jrosser_: But haproxy gets installed on all infra hosts AND both my haproxy hosts? | 17:56 |
| jrosser_ | for an OSA LXC deploy nginx would have its own unique ip inside the repo server container, so this basically never goes wrong | 17:56 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/918119 | 17:59 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_magnum master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/918120 | 18:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/918122 | 18:08 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/918123 | 18:10 |
| jrosser_ | nixbuilder: i can only guess that somehow the inventory has got messed up | 18:11 |
| jrosser_ | noonedeadpunk: repo_all is not in shared-infra_hosts is it? | 18:11 |
| jrosser_ | there is a tool in scripts/inventory-manage to examine the inventory | 18:11 |
| noonedeadpunk | should be | 18:12 |
| noonedeadpunk | oops, it's not | 18:13 |
| noonedeadpunk | it would be `shared-infra_containers` here https://opendev.org/openstack/openstack-ansible/src/branch/master/inventory/env.d/pkg_repo.yml#L25 | 18:14 |
| noonedeadpunk | I think reason for that is distro path that does not require repo | 18:14 |
| jrosser_ | oh blimey this is distro + metal? | 18:15 |
| jrosser_ | nixbuilder: ^ | 18:15 |
| noonedeadpunk | oh, I meant that reason why repo not in share-infra is because it's not needed for distro, which also might define shared-infra_hosts | 18:16 |
| jrosser_ | oooooh i see yes | 18:16 |
| * noonedeadpunk haven't read your conversation, so no idea about rest | 18:17 | |
| * noonedeadpunk trying to make as much patches as possible | 18:17 | |
| jrosser_ | ^ i will try to cover these with andrew next week | 18:17 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_masakari master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_masakari/+/918124 | 18:18 |
| nixbuilder | jrosser_: Metal plus source install. | 18:21 |
| jrosser_ | ok well your config looks ok i guess (missing the haproxy sectoin) so you'll need to use inventory-manage | 18:22 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_mistral master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/918125 | 18:23 |
| jrosser_ | nixbuilder: oh sorry - i am confusing the two things - haproxy and repo server | 18:26 |
| jrosser_ | sorry | 18:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_mistral master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/918127 | 18:26 |
| nixbuilder | jrosser_: :) that's ok | 18:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_mistral master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_mistral/+/918128 | 18:28 |
| noonedeadpunk | I have a feeling that manila will fail CI dramatically | 18:30 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/918129 | 18:32 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/918130 | 18:34 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/918129 | 18:38 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/918130 | 18:38 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_manila master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/918131 | 18:38 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Add tests of more modern distros https://review.opendev.org/c/openstack/ansible-role-frrouting/+/910373 | 18:40 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Update project location in .gitreview https://review.opendev.org/c/openstack/ansible-role-frrouting/+/916741 | 18:40 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Add EL repos to supported by the role https://review.opendev.org/c/openstack/ansible-role-frrouting/+/910381 | 18:40 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Define repository and packages variables in role defaults https://review.opendev.org/c/openstack/ansible-role-frrouting/+/910525 | 18:40 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Update role metadata to match other deliverables https://review.opendev.org/c/openstack/ansible-role-frrouting/+/916746 | 18:40 |
| opendevreview | Merged openstack/ansible-role-frrouting master: Drop native multinode test https://review.opendev.org/c/openstack/ansible-role-frrouting/+/916747 | 18:40 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/918132 | 18:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Add service policies defenition https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/918132 | 18:43 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Add variable to globally control notifications enablement https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/918133 | 18:48 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_zun master: Implement variables to address oslo.messaging improvements https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/918134 | 18:52 |
| noonedeadpunk | ok, so I left with patches for: neutron, nova, octavia, swift and tacker... | 18:52 |
| mgariepy | hmm : Change 916810 in project openstack/openstack-ansible does not share a change queue with 916647 in project openstack/openstack-ansible-os_magnum | 18:57 |
| mgariepy | wasn't the depends-on working correcly on ours git repos ? | 18:57 |
| noonedeadpunk | it was, but not in gates | 18:57 |
| mgariepy | i havne't been doing much lately.. how do we proceed then ? | 18:58 |
| mgariepy | merge one then the other ? | 18:58 |
| noonedeadpunk | yeah, smth like that | 19:19 |
| noonedeadpunk | I guess it could be as dependency failed to merge | 19:19 |
| noonedeadpunk | or well, it was voted latter | 19:19 |
| noonedeadpunk | but matter of recheck, so not that bad | 19:19 |
| mgariepy | have a nice weekend. | 19:23 |
| noonedeadpunk | thanks, you too! | 19:24 |
| opendevreview | Merged openstack/openstack-ansible-os_nova master: Include PKI role only once https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/915413 | 19:41 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Add support for ovn-bgp-agent deployment https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/909780 | 19:59 |
| opendevreview | Merged openstack/openstack-ansible-os_senlin master: Preserve actual production playbook in examples https://review.opendev.org/c/openstack/openstack-ansible-os_senlin/+/916892 | 20:18 |
| opendevreview | Merged openstack/openstack-ansible-os_trove master: Add independent set of variables for guestagent configuration https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/917047 | 20:19 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Set correct permissions for rootwrap.d https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/917785 | 20:25 |
| opendevreview | Merged openstack/openstack-ansible master: Better check if we're in CI during collection bootstrap https://review.opendev.org/c/openstack/openstack-ansible/+/916633 | 20:26 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Fix rootwrap.conf distribution for EL https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/917787 | 20:27 |
| opendevreview | Merged openstack/openstack-ansible master: Respect is_management_address for provider_networks https://review.opendev.org/c/openstack/openstack-ansible/+/915195 | 20:31 |
| opendevreview | Merged openstack/openstack-ansible master: Update global pins for 2024.1 https://review.opendev.org/c/openstack/openstack-ansible/+/916792 | 20:31 |
| opendevreview | Merged openstack/openstack-ansible master: Add Tempest test for OVN Octavia driver https://review.opendev.org/c/openstack/openstack-ansible/+/916872 | 20:48 |
| opendevreview | Merged openstack/openstack-ansible master: Reflect frrouting role new place https://review.opendev.org/c/openstack/openstack-ansible/+/916719 | 20:48 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Update OpenStack Release to Caracal https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/916917 | 20:59 |
| opendevreview | Merged openstack/openstack-ansible-galera_server master: Implement installation method selection for MariaDB role https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/914530 | 21:00 |
| opendevreview | Merged openstack/openstack-ansible-galera_server master: Ensure find is installed on MariaDB containers https://review.opendev.org/c/openstack/openstack-ansible-galera_server/+/917132 | 21:00 |
| opendevreview | Merged openstack/openstack-ansible master: Increase number of threads to 2 for glance in AIO https://review.opendev.org/c/openstack/openstack-ansible/+/916810 | 21:12 |
| opendevreview | Merged openstack/openstack-ansible master: Switch default Ceph deployment to the Reef release https://review.opendev.org/c/openstack/openstack-ansible/+/917014 | 21:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!