| opendevreview | Merged openstack/openstack-ansible-os_horizon stable/xena: Disable barbican-ui dashboard https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/829175 | 02:29 |
|---|---|---|
| jrosser | morning | 07:29 |
| jrosser | whats going on here https://zuul.opendev.org/t/openstack/build/7d02dff0501f44bea9a9ca7125fc7187/log/job-output.txt#4717 | 07:30 |
| jrosser | oh no | 07:32 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/xena: Fix git SHA for openstack_hosts role https://review.opendev.org/c/openstack/openstack-ansible/+/829654 | 07:36 |
| jrosser | how is that even possible to merge a wrong SHA :/ | 07:36 |
| noonedeadpunk | mornings | 08:12 |
| noonedeadpunk | yeah, saw https://bugs.launchpad.net/openstack-ansible/+bug/1961106 | 08:12 |
| noonedeadpunk | I think in CI we don't really test a-r-r | 08:13 |
| noonedeadpunk | it 's back to - we don't test what we tag | 08:14 |
| noonedeadpunk | as it was that we either broke depedns-on with arr or we don't use arr durig bootstrap | 08:15 |
| noonedeadpunk | I think now we likely can workaround that somehow with filtering. but the problem always was iirc is how to understand for what repos we have depedns-on prepared by zuul | 08:16 |
| noonedeadpunk | maybe today there's var for that... | 08:16 |
| noonedeadpunk | likely we need to make just some simple test for verifying SHAs to cover that | 08:18 |
| jrosser | i was digging around how to understand what the depends-on are and we do have this from zuul https://zuul.opendev.org/t/openstack/build/44ed80927767440f8db82a36855535a4/log/zuul-info/inventory.yaml#54-84 | 08:45 |
| jrosser | but getting at that from the embedded ansible was tricky | 08:45 |
| jrosser | and i was hacking away on this to fix that https://review.opendev.org/c/openstack/openstack-ansible/+/821210 | 08:46 |
| jrosser | and that might be enough to get at the depends-on patches as you can refer to them as things like `refs/changes/13/825113/16` | 08:51 |
| noonedeadpunk | yes, but I'd say that logic how we should find them is tricky, as zuul basically just places ready-to-use repos | 08:55 |
| noonedeadpunk | or we need to somehow provide it shas we want which I doubt is possible tbh | 08:56 |
| jrosser | it's like we need to selectively overwrite the SHA from a-r-r with some of the info from zuul | 08:56 |
| jrosser | and you are right, resolving the SHA is not trivial | 08:57 |
| jrosser | we can maybe stack some more patches on top of 821210 if we have ideas on what to do | 08:58 |
| noonedeadpunk | I'd vote just to verify validity of a-r-r and services tbh with some simple job that would be triggered only when they change | 09:01 |
| jrosser | yeah, agree | 09:01 |
| noonedeadpunk | I will write smth up today for this | 09:01 |
| jrosser | perhaps we already have most of whats needed in the scripts directory | 09:01 |
| noonedeadpunk | yeah | 09:01 |
| jrosser | like theres already a script to update the sha | 09:01 |
| jrosser | to making that validate as well should not be impossible | 09:02 |
| noonedeadpunk | um, are you talking about this? https://github.com/evrardjp/osa_cli_releases/blob/master/osa_cli_releases/releasing.py | 09:02 |
| noonedeadpunk | but yes, anyway it should be pretty simple) | 09:03 |
| jrosser | oh well | 09:05 |
| jrosser | i was looking at this https://github.com/openstack/openstack-ansible/blob/master/scripts/sources-branch-updater.sh | 09:06 |
| jrosser | but i think i didnt know about the osa_cli_releases thing | 09:06 |
| noonedeadpunk | It's actually what I use for bumps https://docs.openstack.org/openstack-ansible/latest/contributor/periodic-work.html#releasing | 09:08 |
| noonedeadpunk | anyway | 09:08 |
| noonedeadpunk | I guess we can actually drop these from scripts... | 09:09 |
| opendevreview | Merged openstack/openstack-ansible-os_horizon stable/wallaby: Disable barbican-ui dashboard https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/829176 | 09:36 |
| evrardjp[m] | Yes, at some point both were used | 10:20 |
| evrardjp[m] | I think it’s worth refactoring the release tooling and put it in an official project | 10:21 |
| evrardjp[m] | I just quickly hacked this, it’s pretty darn ugly | 10:21 |
| evrardjp[m] | And for a-r-r see also our convo with etherpad about simplification of testing ;) | 10:22 |
| *** dviroel|out is now known as dviroel | 11:00 | |
| rohit02 | team,while deploying OSA wallaby on centos8 Stream setup openstack failed at https://paste.opendev.org/show/blITqhsE5irMhhBQpREG/ | 11:52 |
| rohit02 | TASK [python_venv_build : Fail if wheels are not built for multiple hosts] ***** | 11:57 |
| rohit02 | ansiblehost: fatal: [controller1_utility_container-89db1c9e]: FAILED! => {"changed": false, "msg": "You are not building wheels while running role against multiple hosts. This might result in DOS-ing OpenDev infrustructure servers. In order to proceed, please ensure that you have repo servers for selected OS version and architecture. If you want to avoid building wheel on purpose, ensure that you run playbook in serial manner. In case of | 11:57 |
| rohit02 | causing unreasonable load on the opendev.org git servers, your access may be blocked to protect other users and the OpenDev CI infrastructure which are reliant on this service. Found venv_build_targets: {'8': {'x86_64': 'controller1_repo_container-f02375da'}}"} | 11:57 |
| jrosser | rohit02: thats a known issue with trying to deploy rally | 12:00 |
| jrosser | which you probably don't need | 12:00 |
| jrosser | noonedeadpunk: did you see this ^ ? | 12:05 |
| noonedeadpunk | I read yestarday other person facing same | 12:14 |
| noonedeadpunk | But never looked into the reasons | 12:15 |
| rohit02 | jrosser noonedeadpunk: any workaround | 12:23 |
| opendevreview | Merged openstack/openstack-ansible stable/xena: Fix git SHA for openstack_hosts role https://review.opendev.org/c/openstack/openstack-ansible/+/829654 | 12:24 |
| rohit02 | jrosser: this known issue its only for multicontroller setups? | 12:26 |
| jrosser | rohit02: yes its only happening when you have 3 controllers | 12:35 |
| rohit02 | jrosser: thanx..is there any fix available which i can apply manually | 12:36 |
| jrosser | are you using rally? | 12:36 |
| jrosser | if you are not, then comment this line out https://github.com/openstack/openstack-ansible/blob/master/playbooks/setup-openstack.yml#L50 | 12:37 |
| jrosser | noonedeadpunk: we disable wheel builds for installing rally | 12:39 |
| jrosser | so it is always going to trigger the venv build warning/error when 3 controllers | 12:40 |
| noonedeadpunk | I wonder why we install it not like tempest only on 1 container but everywhere | 12:41 |
| noonedeadpunk | I doubt there's any usecase for that? | 12:41 |
| noonedeadpunk | ie https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/os-tempest-install.yml#L23 | 12:41 |
| jrosser | also i wonder why we install it unconditionally | 12:42 |
| jrosser | seems like could add env.d setup for it so it's only installed if the group is defined | 12:42 |
| noonedeadpunk | oh yes | 12:43 |
| jrosser | we could make it always do that in CI but default not to for regular deployments | 12:43 |
| noonedeadpunk | yeas, I fully agree here | 12:59 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible master: Do not install extra repos with the zuul configure-mirrors role https://review.opendev.org/c/openstack/openstack-ansible/+/829111 | 13:09 |
| jrosser | ^ this failed centos jobs as expected \o/ now i add the depends on to fix it...... zuul++ :) | 13:10 |
| NeilHanlon | morning folks o/ | 13:12 |
| NeilHanlon | i think the rockylinux-8 nodepool should be available today, 🤞 | 13:13 |
| jrosser | that would be great | 13:15 |
| NeilHanlon | just need https://review.opendev.org/c/828435 and https://review.opendev.org/c/828436, i think | 13:16 |
| jrosser | this is also related on the OSA side https://review.opendev.org/c/openstack/openstack-ansible/+/829111 | 13:17 |
| jrosser | seem to be making progress with that too now | 13:17 |
| NeilHanlon | nice | 13:19 |
| spotz__ | Hey I've never seen backport candidate before. I've +2'd but holding off on +w just in case - https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/828449/3..4 | 14:11 |
| noonedeadpunk | Yeah, I've added it for the dashboard but never documented how to use ;( | 14:12 |
| noonedeadpunk | So can be considered as WIP as for now | 14:12 |
| spotz__ | noonedeadpunk: Ok | 14:17 |
| opendevreview | Andrew Bonney proposed openstack/openstack-ansible-plugins master: Add reference_group support to provider_networks module https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/829741 | 16:14 |
| admin1 | has anyone in haproxy_extra_services been able to add a different port .. for example, for ceph object storage, i usually add port 8080 in front end and point to the backends.. and they are on the public url 8080 port .. how I want to add it is as s3.domain.com . on 443 itself and have haproxy do SNI .. the wildcard used is *.domain.com so it | 16:16 |
| admin1 | should work | 16:16 |
| admin1 | replace add a differnt port => add a new url/endpoint | 16:17 |
| jrosser | admin1: you would probably need to use an acl for that | 16:22 |
| jrosser | a variation on how the letsencrypt stuff gets directed to the right backend but based on the name rather than path | 16:22 |
| admin1 | i see one option called extra_lb_vip_addresses .. does that expose all to also this new endpoint ? | 16:24 |
| admin1 | its just an ip | 16:25 |
| admin1 | ok | 16:25 |
| opendevreview | Merged openstack/ansible-role-python_venv_build master: Move TripleO zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/829164 | 16:25 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: Move zuul jobs layout to centos9 only for master branch https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/828449 | 16:44 |
| prometheanfire | have others found the need to adjust their heartbeat_timeout_threshold to avoid missed heartbeats? | 16:59 |
| opendevreview | Merged openstack/openstack-ansible-openstack_hosts master: Fix condtional for powertools/crb repo installation https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/829019 | 17:28 |
| noonedeadpunk | I don't think we have changed that | 17:35 |
| opendevreview | Merged openstack/openstack-ansible-os_zun master: Restore CI jobs https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/824457 | 18:05 |
| *** sshnaidm is now known as sshnaidm|afk | 18:17 | |
| *** dviroel is now known as dviroel|out | 20:56 | |
| *** krumelmonster_ is now known as krumelmonster | 23:47 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!