opendevreview | Michal Nasiadka proposed openstack/kolla master: CI: Fix pip mirror multiline https://review.opendev.org/c/openstack/kolla/+/948803 | 06:07 |
---|---|---|
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add key type for letsencrypt certificate https://review.opendev.org/c/openstack/kolla-ansible/+/934683 | 06:08 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add key type for letsencrypt certificate https://review.opendev.org/c/openstack/kolla-ansible/+/934683 | 06:09 |
sahid | o/ | 06:50 |
sahid | other quick queetions :-) | 06:51 |
sahid | is there a command to pull a tag on all nodes in my inventary easily from kolla-ansible? | 06:51 |
sahid | what a fork is doing, i'm trying to understang whether increasing the number of work will help or just busy the pipeline | 06:52 |
opendevreview | Merged openstack/kolla master: Enabling support for Thales Luna user id and group id. https://review.opendev.org/c/openstack/kolla/+/948084 | 07:45 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: Add key type for letsencrypt certificate https://review.opendev.org/c/openstack/kolla-ansible/+/934683 | 07:53 |
mnasiadka | morning | 07:53 |
opendevreview | Merged openstack/kolla master: bifrost: fix creation of python3 lib symlink https://review.opendev.org/c/openstack/kolla/+/948723 | 07:54 |
opendevreview | Merged openstack/kolla master: Allow specification of a key type https://review.opendev.org/c/openstack/kolla/+/934681 | 07:54 |
sahid | o/ | 09:31 |
sahid | how increasing forks improve deploy performance? I'm asking that because I would like to ensure this will improve deployment and not busy pipline | 09:32 |
opendevreview | Michal Nasiadka proposed openstack/kolla master: [letsencrypt] Add ProxySQL support for certificate renewal https://review.opendev.org/c/openstack/kolla/+/936920 | 11:56 |
mnasiadka | frickler: thinking if we could use https://gerrit-review.googlesource.com/Documentation/access-control.html#category_remove_label for removing RP by other core-reviewers | 13:19 |
frickler | and then we start RP wars ;) but I'm not opposed to trying it, feel free to propose an ACL change | 13:24 |
mnasiadka | well, tried to find something similar somewhere, but failed :) | 13:26 |
mnasiadka | so I'll need to reinvent the wheel | 13:26 |
mnasiadka | frickler: I posted https://review.opendev.org/c/openstack/releases/+/948875 - if there's anything else we need now - let me know | 13:32 |
mnasiadka | ah, the LE key_type, let me have a look | 13:32 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Add key type for letsencrypt certificate https://review.opendev.org/c/openstack/kolla-ansible/+/934683 | 13:33 |
frickler | mnasiadka: hmm, we also wanted https://review.opendev.org/c/openstack/kolla-ansible/+/836941 but it needs a small update. or rather delay and possibly backport? | 13:39 |
WiniciusAllan[m] | sahid: about the pull images. yes, there is a way. you'll need to specify the image URI in globals, then run kolla-ansible pull | 13:57 |
WiniciusAllan[m] | generally, you can specify with <service>_image_full: | 13:58 |
sahid | WiniciusAllan[m]: ack thanks, it's a good workaround, I have build a ansible playbook to do the same without having to touch globals.yml | 14:09 |
WiniciusAllan[m] | sahid: can you share it? =) | 14:10 |
sahid | it's specific to our env, we build our own images, with the help of mistralai or chatgpt you should build it easily ;-) | 14:14 |
mnasiadka | frickler: I think if you're not going to fix it - then we need to delay it | 14:17 |
mnasiadka | As in we need to cut RC1 this week | 14:17 |
mnasiadka | :) | 14:17 |
frickler | *sigh* so I'm going to update it myself and hope that it was really just a typo and not something important that I don't understand ... | 14:48 |
frickler | hmm, checking a local docker install everything in /var/lib/docker/volumes really has mode 0x701. weird. but then so be it I guess | 14:52 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Use pause module to prompt for deprovision https://review.opendev.org/c/openstack/kayobe/+/943012 | 14:55 |
frickler | +2 and resolved all the open comments, feel free to disagree and complain | 14:58 |
mnasiadka | frickler: I'm Polish - I always complain :) | 15:29 |
frickler | ugh, now you scared me and I had to check what you did to that poor review ;) | 15:40 |
frickler | btw. any reason not to branch ansible-collection-kolla at the same time= | 15:40 |
mnasiadka | Ah right, forgot | 17:10 |
mnasiadka | Will update after all these changes merge | 17:10 |
mnasiadka | frickler: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/925083 this is deps to the container engine migration patch | 17:25 |
mnasiadka | I can tidy it up tomorrow | 17:25 |
mnasiadka | frickler: actually it looks good - it has my +2 if you're willing to have a look today to merge it | 17:32 |
opendevreview | Merged openstack/kolla-ansible master: Add key type for letsencrypt certificate https://review.opendev.org/c/openstack/kolla-ansible/+/934683 | 17:40 |
sean-k-mooney | hi folks just an fyi, as part of https://governance.openstack.org/tc/goals/proposed/migrate-from-wsgi-scripts-to-module-paths.html nova and placement may drop our wsgi_script entry point and the assocated wsgi binary this cyle i have more details in https://review.opendev.org/c/openstack/nova/+/902688/comments/52e1364f_7d8fa1cc but what that means for kolla is you will need | 18:44 |
sean-k-mooney | to update https://github.com/openstack/kolla-ansible/blob/354322c2361f8fa205b4c019163c07c73236f8d5/ansible/roles/nova/templates/nova-metadata-wsgi.conf.j2#L37 and https://github.com/openstack/kolla-ansible/blob/354322c2361f8fa205b4c019163c07c73236f8d5/ansible/roles/nova/templates/nova-api-wsgi.conf.j2#L37 to point at the python module instead i.e. | 18:44 |
sean-k-mooney | /usr/lib/python3/dist-packages/nova/wsgi/osapi_compute.py when using mod_wsgi | 18:44 |
sean-k-mooney | i dont know if kolla still supprot both binary and source installed conatiner but for the souce installled version that would be the relevent module relitive to the venv | 18:44 |
sean-k-mooney | uwsgi can use the file path or the module path as documented in our release ntoes form dalmation https://review.opendev.org/c/openstack/nova/+/902687/4/releasenotes/notes/add-nova-wsgi-module-3cc250a78fef7365.yaml#19 | 18:46 |
sean-k-mooney | for watcher i just added pyproject.toml supprot last week and supprot for this wsgi module | 18:46 |
sean-k-mooney | we backported that to 2025.1 also but we likely wont remove supprot for the wsgi_scripts entry point until 2026.2 | 18:47 |
sean-k-mooney | i have suggested that we send an eamil to the mailing list ot let all installer knwo before we do this and break all your gates | 18:48 |
sean-k-mooney | but just be aware that designate already drop there wsgi_script as part of addign pyproject.toml support | 18:48 |
sean-k-mooney | so that ship may have already sailed | 18:49 |
sean-k-mooney | https://review.opendev.org/c/openstack/designate/+/902846 | 18:49 |
frickler | sean-k-mooney: thanks for the note. kolla only supports source installs since some cycles. we will branch 2025.1 this week and then update for 2025.2 I'd think | 18:59 |
mnasiadka | Thanks sean-k-mooney - we’re sort of transitioning to uwsgi - and we don’t use wsgi_script there, but I didn’t think to do the same for mod_wsgi | 19:00 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!