| noonedeadpunk | about 921028 - it';s clean cherry-pick from stable/zed where there was a CI | 07:12 |
|---|---|---|
| noonedeadpunk | so - whatever Iguess | 07:12 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2024.1: Run Notify part when RPC is disabled https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921490 | 07:13 |
| noonedeadpunk | and we should totally update that example I guess... | 07:14 |
| noonedeadpunk | jamesden-: if you tested heat and it works nicely - your vote on https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921443 would be pretty much aprpeciated | 07:15 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-tests master: Bump Ansible version for tests repo https://review.opendev.org/c/openstack/openstack-ansible-tests/+/921434 | 07:31 |
| noonedeadpunk | (meaning https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921444 patch) | 07:32 |
| jrosser | noonedeadpunk: it never was very clear why the examples always put the cinder backend config in o-u-c | 08:14 |
| jrosser | almost like it dates from before having group_varsity and never got adjusted to match | 08:14 |
| jrosser | *group_vars | 08:14 |
| jrosser | as what it does a use the dynamic inventory to put those vars from o-u-c only into certain groups | 08:16 |
| noonedeadpunk | yeah, which might be fine only if you have multiple backends and don't want to do extra groups... | 08:26 |
| noonedeadpunk | but yeah, totally makes sense to do that through groups | 08:26 |
| noonedeadpunk | in our latest deployment we have literally 1 variable in user_*.yml | 08:27 |
| noonedeadpunk | which is sshd_settings_rhel7 - as it need to have precedence to override | 08:27 |
| noonedeadpunk | really updating docs is quite outdue thing which I fail to find time on :( | 08:31 |
| noonedeadpunk | *overdue | 08:31 |
| frickler | friendly reminder with my release team hat on that the official deadline for the 2024.1 release was yesterday, what's your current status? | 08:41 |
| noonedeadpunk | we've found 3 nasty bugs yesterday, which we're backporting now | 08:43 |
| noonedeadpunk | at very least waiting for https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921490 | 08:45 |
| noonedeadpunk | I do believe that we will be able to have an SHA for the final release today | 08:46 |
| frickler | ok, thx | 08:48 |
| opendevreview | Merged openstack/openstack-ansible-plugins master: Allow to remove role assignment from users https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921443 | 10:30 |
| opendevreview | Merged openstack/openstack-ansible master: Start 2024.2 (Dalmatian) development https://review.opendev.org/c/openstack/openstack-ansible/+/921367 | 10:50 |
| opendevreview | Merged openstack/openstack-ansible-os_skyline stable/2024.1: Fix skyline user creation https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/921462 | 10:51 |
| opendevreview | Merged openstack/openstack-ansible master: Drop support for Debian 11 (bullseye) https://review.opendev.org/c/openstack/openstack-ansible/+/921370 | 10:53 |
| opendevreview | Merged openstack/openstack-ansible master: Switch services SHAs to track master https://review.opendev.org/c/openstack/openstack-ansible/+/921368 | 10:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2024.1: Allow to remove role assignment from users https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921512 | 10:56 |
| opendevreview | Merged openstack/openstack-ansible-os_heat master: Handle removal of wrong role assignment https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921444 | 12:42 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat stable/2024.1: Handle removal of wrong role assignment https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921519 | 12:57 |
| lowercase | https://docs.openstack.org/zun-ui/latest/configuration/index.html#image-for-cloud-shell , I would like to make this modification. "edit CLOUD_SHELL_IMAGE variable in file _0330_cloud_shell_settings.py.sample, and copy it to horizon/openstack_dashboard/local/local_settings.d/_0330_cloud_shell_settings.py" I was looking through the horizon documentation and didn't see a path forward. I tried writing a policy and a config override but i | 13:15 |
| noonedeadpunk | lowercase: I don't think there's a way of doing that, except either proposing patch to the plugin, or forking it and using your fork for deployment | 13:17 |
| noonedeadpunk | or well, dunno, let me check | 13:18 |
| noonedeadpunk | ok, maybe you can try doing that | 13:19 |
| noonedeadpunk | lowercase: have you tried that? https://paste.openstack.org/show/bzrP6jsbwcgIqytHduR9/ | 13:20 |
| opendevreview | Merged openstack/openstack-ansible stable/2024.1: [doc] Properly promote deploy guide https://review.opendev.org/c/openstack/openstack-ansible/+/921447 | 13:22 |
| lowercase | Sorry, I got pulled into a sev. I am not able to look at this right now. | 13:27 |
| lowercase | Will circle back with you later. Sorry and thank you. | 13:27 |
| jrosser | do we install sshd into the containers in the functional tests? i think we removed needing it from lxc_hosts https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921490?tab=change-view-tab-header-zuul-results-summary | 14:01 |
| noonedeadpunk | I think only to keystone as of today | 14:07 |
| noonedeadpunk | though functional jobs likely do install | 14:07 |
| noonedeadpunk | also weirdly - that failed only on gate | 14:08 |
| noonedeadpunk | I'm trying to fight that here https://review.opendev.org/c/openstack/openstack-ansible-tests/+/921434 | 14:08 |
| lowercase | noonedeadpunk, I did try that exact thing earlier. I tried it again and it didn't make the expected modification. | 14:51 |
| noonedeadpunk | huh | 14:52 |
| noonedeadpunk | weird... | 14:52 |
| lowercase | It did update, /etc/horizon/local_settings.py , with HORIZON_CONFIG.update({'CLOUD_SHELL_IMAGE': 'openstackhelm/openstack-client:2023.1'}) | 14:53 |
| lowercase | https://github.com/openstack/zun-ui/blob/master/zun_ui/content/cloud_shell/views.py | 14:57 |
| noonedeadpunk | I'd expect local_settings to be kinda source of truth in a way | 14:58 |
| noonedeadpunk | the latest thing that get's applied | 14:58 |
| noonedeadpunk | have you tried to debug if `settings.CLOUD_SHELL_IMAGE` is defined with this approach? | 14:58 |
| opendevreview | Merged openstack/openstack-ansible-plugins stable/2024.1: Allow to remove role assignment from users https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921512 | 15:01 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-plugins stable/2024.1: Run Notify part when RPC is disabled https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/921490 | 15:07 |
| lowercase | noonedeadpunk instead of it being in the horizn_config.update section. It just needs to be in the local_settings.py file as just `key` = `value` | 15:15 |
| lowercase | In other words, I modified /etc/horizon/local_settings.py, `CLOUD_SHELL_IMAGE = openstackhelm/openstack-client:2023.1` and it works now | 15:15 |
| lowercase | excuse me, CLOUD_SHELL_IMAGE wasn't present. I added it to the config file. | 15:16 |
| noonedeadpunk | huh | 15:18 |
| noonedeadpunk | yeah, in fact I don't see how to do that today | 15:20 |
| noonedeadpunk | but I believe it can be done with simple key/value pair | 15:20 |
| lowercase | echo 'CLOUD_SHELL_IMAGE = openstackhelm/openstack-client:2023.1' >> /etc/horizon/local_settings.py | 15:20 |
| lowercase | sorry, mt | 15:20 |
| noonedeadpunk | this should be quite trivial to implement eventually | 15:21 |
| lowercase | Zun gives the ability to run a cloud shell within the browser. Thought it would be a fun feature to have. I'm not sure why it does nothing but for right now, its nice to know its not because the baked image is 6 years old. | 15:26 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_horizon master: Implement ability to define arbitrary options in local_settings.py https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/921538 | 15:45 |
| noonedeadpunk | lowercase: I assume this is smth you'd need then ? ^ | 15:45 |
| lowercase | Yes, it would be something that I could use. | 15:49 |
| lowercase | I'm on antelope rn | 15:49 |
| noonedeadpunk | well, not sure we'll backport that to antelope... but might be to 2024.1 after final release | 15:50 |
| noonedeadpunk | you can fork/cherry-pick role (or just use one from 2024.1) | 15:51 |
| lowercase | okay np | 15:57 |
| -opendevstatus- NOTICE: Gerrit will be restarted at around 17:45 UTC to pick up some small image updates | 16:44 | |
| hamburgler | does anyone use aodh for monitoring instances across multiple projects - or is that more geared towards the project admins? or something else? | 16:49 |
| opendevreview | Kevin Carter proposed openstack/openstack-ansible-os_skyline master: Update the skyline proxy config https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/921545 | 17:00 |
| noonedeadpunk | cloudnull: are you sure that `flushpackets=on` is applicable in that context? | 17:48 |
| noonedeadpunk | or well, makes any difference | 17:48 |
| jamesden- | noonedeadpunk also discovered that the default maxrewrite for haproxy was too small (1280?) - needed to bump higher, otherwise the skyline UI would return 500 errors seemingly randomly. Skyline itself was returning 200 but haproxy was returning the 500 | 18:25 |
| jamesden- | so we bumped to 8096 | 18:26 |
| noonedeadpunk | ah | 18:46 |
| noonedeadpunk | I haven't catched that though | 18:48 |
| noonedeadpunk | but I never tried skyline outside of poc | 18:49 |
| noonedeadpunk | so you set `haproxy_tuning_params: tune.maxrewrite`, right? | 18:49 |
| noonedeadpunk | aha, so we have that default: https://opendev.org/openstack/openstack-ansible-haproxy_server/src/branch/master/vars/main.yml#L96-L97 | 18:51 |
| noonedeadpunk | it's kinda weird that it's not enough given it's absolutely fine for CSP even... | 18:52 |
| noonedeadpunk | but feel free to propose patch to cover that! | 18:52 |
| noonedeadpunk | jamesden-: if around - vote on https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921519 backport would be nice:) | 18:58 |
| noonedeadpunk | jamesden-: also - you get 500 with apache vhost improvments you did or without them as well?:) | 18:59 |
| jamesden- | i think the 500s were more related to the buffer | 19:06 |
| jamesden- | I overrode tune.maxrewrite: 1280 with 8096 which has seemingly corrected the 500s | 19:07 |
| jamesden- | 8096 might be big - i didn't test smaller stuff | 19:07 |
| noonedeadpunk | ProxyPreserveHost can actually contribute to that a bit... but yeah, likely you're right | 19:07 |
| noonedeadpunk | but yeah, just propose something reasonable there | 19:08 |
| noonedeadpunk | I can't wait right now with release anymore though :( | 19:09 |
| jamesden- | i'll try and play around with it | 19:09 |
| jamesden- | yes of course | 19:09 |
| noonedeadpunk | but that's smth that is overridable, so it's fine | 19:09 |
| jamesden- | yep | 19:09 |
| noonedeadpunk | are you upgrading some of old prods? or just playing with sandbox? | 19:10 |
| jamesden- | a little of both? | 19:11 |
| noonedeadpunk | aha | 19:12 |
| noonedeadpunk | just was wondering how much upgrade screwed things :D | 19:12 |
| jamesden- | oh actually not much at all | 19:13 |
| jamesden- | i'd say that was a very successful A->C upgrade | 19:13 |
| jamesden- | thanks to your quick patches should be painless for the next person :) | 19:13 |
| noonedeadpunk | well, keep us updated if you feel any difference with quorum queues now :D | 19:14 |
| jamesden- | definitely will. so far so good | 19:14 |
| noonedeadpunk | I played in dev env only with them, as decided to wait for 2024.1 for the prod one | 19:14 |
| jamesden- | Are you running A in prod currently? | 19:15 |
| noonedeadpunk | due to quite high amount of changes to oslo.messages that OVH landed | 19:15 |
| noonedeadpunk | yup | 19:15 |
| opendevreview | Merged openstack/openstack-ansible-os_heat stable/2024.1: Handle removal of wrong role assignment https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921519 | 20:53 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_heat master: Revert "Handle removal of wrong role assignment" https://review.opendev.org/c/openstack/openstack-ansible-os_heat/+/921581 | 21:22 |
| opendevreview | Merged openstack/openstack-ansible-os_skyline master: Update the skyline proxy config https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/921545 | 22:02 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_skyline stable/2024.1: Update the skyline proxy config https://review.opendev.org/c/openstack/openstack-ansible-os_skyline/+/921583 | 22:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!