15:00:08 <noonedeadpunk> #startmeeting openstack_ansible_meeting
15:00:08 <opendevmeet> Meeting started Tue Apr  2 15:00:08 2024 UTC and is due to finish in 60 minutes.  The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:08 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:08 <opendevmeet> The meeting name has been set to 'openstack_ansible_meeting'
15:00:14 <noonedeadpunk> ok, now in time )
15:00:23 <noonedeadpunk> #topic roll call
15:00:42 <jrosser> o/ hello
15:02:54 <noonedeadpunk> o/
15:03:29 <noonedeadpunk> #topic office hours
15:03:46 <noonedeadpunk> we had couple of "bugs" during the weekend, though they were more "features"...
15:04:04 <noonedeadpunk> speaking of which was horizon deployment for aio scenario on metal
15:04:34 <noonedeadpunk> currently we have inconsistency between lxc/metal with horizon
15:04:56 <jrosser> thats not surprising, it was very difficult to get the horizon install to work on a metal deploy
15:05:10 <noonedeadpunk> yeah, but I think it's solved for couple of years now?
15:05:16 <noonedeadpunk> and that's just legacy?
15:05:23 <jrosser> it should be solved, yes
15:05:23 <NeilHanlon> o/
15:05:37 <noonedeadpunk> but then - do we want to spend time on horizon at all...
15:05:52 <jrosser> but i don't know how many actually use it on metal, vs it being there as a CI fixture
15:06:24 <noonedeadpunk> we do have some horizon tempest tests when horizon role runs fwiw
15:06:38 <jrosser> yeah
15:06:54 <jrosser> i was interested in your patch that added skyline with horizon at /horizon
15:07:04 <jrosser> and also to know if the opposite was possible
15:07:16 <noonedeadpunk> I wasn't able to make /skyline
15:07:26 <noonedeadpunk> BUT, it can be possible once we build with YARN
15:07:41 <noonedeadpunk> I just lacked enough time/interest I guess as it was way less trivial
15:07:57 <jrosser> i was considering the case of existing deployments where moving the horizon url might be tricky
15:08:25 <NeilHanlon> yeah i think it's definitely possible but the skyline code makes some (IMO) bad assumptions about how it will be deployed
15:08:48 <noonedeadpunk> well, potentially there should be different regexp I guess and do like `skyline.region.one` instead...
15:08:55 <noonedeadpunk> in haproxy mapping?
15:09:29 <noonedeadpunk> NeilHanlon: in yarn build config I found the way to make static content generated with some /path
15:09:34 <jrosser> i think also trying to undertstand the nginx setup better would be good
15:09:36 <noonedeadpunk> but it didn't help end result easily
15:09:42 <jrosser> oh and also this https://review.opendev.org/c/openstack/governance/+/914248
15:10:00 <noonedeadpunk> huh
15:10:09 <NeilHanlon> hm
15:10:42 <noonedeadpunk> I also raised question in ML without reply obviously
15:11:47 <noonedeadpunk> but in fact I don't think I'll be able to invest many more time into getting skyline work under /skyline
15:12:30 <jrosser> my interest is kind of depending on OIDC support really
15:12:35 <noonedeadpunk> but the variable is here: https://opendev.org/openstack/skyline-console/src/branch/master/config/webpack.prod.js#L46
15:13:06 <NeilHanlon> yeah, i think the 'real' answer is that skyline should use root-relative URLs instead of absolute.
15:13:08 <noonedeadpunk> (we're using an in-house UI anyway)
15:13:23 <noonedeadpunk> yeah, true...
15:13:35 * NeilHanlon will see if he can find some time, or find someone at $work to find an hour
15:13:45 <noonedeadpunk> and then also how to make nginx config respect that is another question
15:14:46 <noonedeadpunk> but other then that, I guess skyline is in a pretty much usable shape then it ever was :)
15:15:00 <NeilHanlon> that's really great news :)
15:15:17 <NeilHanlon> A year ago it was definitely not in anywhere close to such a shape
15:15:20 <jrosser> maybe this is things we can discuss with frickler depending how the skyline PTL stuff plays out
15:15:21 <noonedeadpunk> but I understand the question, if we should even suggest replacing horizon with skyline
15:15:53 <noonedeadpunk> according to the code it's already a second term
15:16:12 <noonedeadpunk> or well, third, sorry
15:17:08 <noonedeadpunk> ugh
15:17:18 * noonedeadpunk needs to dig for some docs
15:17:18 <NeilHanlon> ideally, i guess we would have some flag to decide which UI should be "default"
15:17:55 <noonedeadpunk> yeah, ideally I was thinking to place /horizon and /skyline and the flag to define redirect
15:18:14 <jrosser> ^ this would be ideal, like i say the url can be hard to change
15:18:30 <noonedeadpunk> yup...
15:18:39 <jrosser> knitted into SSO provider setup blah blah
15:19:34 <noonedeadpunk> ok, so what's the suggestion kinda?
15:19:48 <noonedeadpunk> like I don't know a way how to put /skyline today
15:20:11 <jrosser> that can wait i guess
15:20:18 <noonedeadpunk> and that would require yarn build 100%
15:20:27 <jrosser> but when we design the logic / vars that control it, the option should be open for *
15:20:32 <noonedeadpunk> which currently is not a default behaviour
15:21:07 <noonedeadpunk> (and that's why it feels kolla also don't have option for /skyline)
15:21:18 <noonedeadpunk> (as I'd assume they just install from pypi)
15:24:05 <jrosser> so long as we leave the variables sufficiently open, even if they implementation is missing then we will be OK for the future
15:24:33 <noonedeadpunk> I guess.... we have?
15:24:35 <noonedeadpunk> not sure
15:24:52 <noonedeadpunk> oh, or you mean to add logic to select default now?
15:25:29 <jrosser> NeilHanlon: you might want to join #openstack-qa
15:25:58 <noonedeadpunk> ++
15:26:15 <gmann> ++ rocky job failing on stable/2024.1 but passing on master
15:28:00 <noonedeadpunk> SO, basically we need to add variable `skyline_webroot` here https://review.opendev.org/c/openstack/openstack-ansible/+/859446/21/inventory/group_vars/skyline_all/haproxy_service.yml#34
15:28:10 <NeilHanlon> ack thanks jrosser
15:29:07 <noonedeadpunk> or make a follow-up?
15:29:28 <jrosser> noonedeadpunk: yes so i mean even if we can only deploy skyline to / today, having a var that "sets" that gives us a route for the future if it becomes configurable
15:30:02 <noonedeadpunk> ok, gotcha, can add it, sure
15:30:03 <jrosser> rather than bake in an assumption that it can only be at / forever
15:31:42 <noonedeadpunk> about oslo_concurency for glance - this seems to almost work: https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/914752
15:32:08 <noonedeadpunk> but I've spotted quite some weird failures in related patches
15:32:15 <noonedeadpunk> So I'm not sure 100% now
15:32:32 <noonedeadpunk> And also spawning an env right now to check for default volume type issues
15:32:58 <noonedeadpunk> as feels like iscsi doesn't really work now according to output of https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/901189
15:33:48 <noonedeadpunk> or well - https://zuul.opendev.org/t/openstack/build/757f078df8b6402797b7318c23d087cf/log/logs/host/cinder-volume.service.journal-12-45-13.log.txt#7706-7730
15:34:22 <noonedeadpunk> so trying to check on that now. And we have simmilar report about non-functional volumes from aio deployment
15:38:40 <noonedeadpunk> and didn't work on proper octavia ovn test yet
15:40:29 <noonedeadpunk> regarding PTG - will book a slot appropriate for UTC/EST timezones and will send a ML later today as well
15:41:33 <NeilHanlon> great!
15:43:39 <opendevreview> Dmitriy Rabotyagov proposed openstack/openstack-ansible master: [Feature] Add skyline deployment capability  https://review.opendev.org/c/openstack/openstack-ansible/+/859446
15:43:57 <noonedeadpunk> jrosser: you though about smth like that? https://review.opendev.org/c/openstack/openstack-ansible/+/859446/21..22/inventory/group_vars/skyline_all/haproxy_service.yml
15:48:04 <jrosser> noonedeadpunk: yeah like that
15:48:32 <noonedeadpunk> ok, will try to add smth to skyline role with a follow-up a bit later
15:48:42 <noonedeadpunk> who knows, maybe I will get it working :D
15:48:56 <jrosser> hah :)
15:49:06 <noonedeadpunk> from top of your head, would 302 redirect to /horizon work for SSO?
15:49:27 <noonedeadpunk> or it must be / strictly?
15:49:28 <jrosser> i don't know tbh
15:49:38 <jrosser> though this is something we can pretty easily check in an AIO
15:50:00 <noonedeadpunk> yeah, ok, let's save that for a bit later probably...
15:50:02 <jrosser> for horizon part of this is all handled in apache mod_oidc
15:50:25 <jrosser> and thats where i'm all a bit clueless about how that might be a thing for skyline
15:50:34 <jrosser> anyway
15:50:38 <noonedeadpunk> despite we're about to run that, I have quite vague understanding
15:51:21 <noonedeadpunk> well. Potentially, Apache can replace nginx in these overrides once we understand them. Or ask chatgpt to covert :D
15:51:30 <NeilHanlon> :D
15:51:53 <NeilHanlon> i am pretty familiar with nginx and apache, so.. i can help double check the AI :P
15:53:37 <NeilHanlon> tangent: Freezer meeting is in 7 minutes or so,right?
15:53:39 <noonedeadpunk> I pretty much forgot since was not dealing with them much for last... 7 years?
15:53:43 <noonedeadpunk> yeah
15:53:57 <noonedeadpunk> so I will wrap this up now
15:53:59 <NeilHanlon> what channel?
15:54:12 <NeilHanlon> ack. thanks for running as always!
15:54:21 <noonedeadpunk> ugh, that was video one as one folk asked for it
15:54:42 <NeilHanlon> ah I see. I will consult the mailing list then. thanks again
15:54:51 <NeilHanlon> I forgot to put details in my calendar lol
15:55:07 <noonedeadpunk> I'll post in #openstack-freezer
15:56:13 <NeilHanlon> ty. thought I'd joined that too but apparently I'm losing more of my mind than I knew
15:57:54 <noonedeadpunk> #endmeeting