| yusufgungor | Hi @frickler I have seen that on https://docs.openstack.org/releasenotes/oslo.messaging/en_GB/xena.html#relnotes-12-9-4-1-stable-xena | 05:44 |
|---|---|---|
| mnasiadka | frickler: ah, right ;) | 06:07 |
| mnasiadka | frickler: soon to be unmaintained/xena, so maybe the wording will be more understandable for users ;) | 06:12 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: ansible: bump min version to 2.14 and max to 2.15 https://review.opendev.org/c/openstack/kolla-ansible/+/887227 | 06:14 |
| mnasiadka | frickler, bbezak: let's merge https://review.opendev.org/c/openstack/kolla-ansible/+/886412 and https://review.opendev.org/c/openstack/kolla-ansible/+/886412 - ansible bump (earlier this time) | 07:30 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: cephadm: Move osds to spec https://review.opendev.org/c/openstack/kolla-ansible/+/889921 | 07:42 |
| guesswhat[m] | mmalchuk: how can I make https://review.opendev.org/c/openstack/kolla-ansible/+/888494 and https://review.opendev.org/c/openstack/kolla-ansible/+/888496/6 independend? There is a circular dependency, can You help me solve this? Thanks | 07:46 |
| mmalchuk | why circular ? | 07:46 |
| mmalchuk | there are two way: 1. add changes as relation chain, and always rebase second if first modified. 2. add to the second change "Depends-On" header to the commit message | 07:48 |
| mnasiadka | rather use Depends-On if it's in a different repo/project | 07:49 |
| frickler | mnasiadka: what about the cephadm failure, is that unrelated? | 07:52 |
| mmalchuk | guesswhat[m] as for now you have Indirect ancestor in the change. rebase one above second to fix the issue. | 07:52 |
| guesswhat[m] | is possible to rebase it via gerrit ui ? | 07:53 |
| mmalchuk | yes | 07:54 |
| mnasiadka | frickler: seems unrelated, at least the ubuntu one, rocky seems to work fine, I'll recheck once the results come in | 07:54 |
| mmalchuk | guesswhat[m] when you press Rebase button you have the field to enter change number, ref or commit hash | 07:56 |
| opendevreview | howardlee proposed openstack/kolla-ansible master: A commit test https://review.opendev.org/c/openstack/kolla-ansible/+/889924 | 07:57 |
| mmalchuk | guesswhat[m] but if you face the conflict during rebase its need to be fixed in your favorite code editor | 07:57 |
| guesswhat[m] | I would like to get this done, its trivial fix, but somehow it dont know how to do it in Gerrit :X | 07:58 |
| guesswhat[m] | should I rebase ( Rebase on parent change ) https://review.opendev.org/c/openstack/kolla-ansible/+/888496/6 ? | 07:59 |
| mmalchuk | if you unsure the change rebase to you always can use commit hash, its easy to copy it to the clipboard pressing the sign near the hash | 08:05 |
| kevko | hi | 08:08 |
| kevko | regarding yesterdday oslo messaging bug ... wouldn't be better to merge my patches about patching the codde inside docker images ? | 08:09 |
| mmalchuk | kevko hi) | 08:09 |
| kevko | or install everything from git | 08:10 |
| mmalchuk | everything from git much better | 08:10 |
| mmalchuk | imho | 08:10 |
| kevko | or both | 08:11 |
| kevko | because sometimes some patch is on review for month even if it is critical bug ...check oslo messaging review history | 08:11 |
| opendevreview | Lukas M proposed openstack/kolla-ansible master: Adds SSO support for Skyline. https://review.opendev.org/c/openstack/kolla-ansible/+/888496 | 08:14 |
| guesswhat[m] | mmalchuk: PR1 => https://review.opendev.org/c/openstack/kolla-ansible/+/888494/6/ansible/roles/keystone/defaults/main.yml PR2 => https://review.opendev.org/c/openstack/kolla-ansible/+/888496/7/ansible/roles/keystone/defaults/main.yml | 08:17 |
| guesswhat[m] | Here is a merge conflict, You especially wanted to have these as two PRs, so I would like to kindly ask how should I solve this dependency / merge conflict. | 08:17 |
| mmalchuk | fix it using the git commands and your favorite code editor | 08:19 |
| opendevreview | howardlee proposed openstack/kolla-ansible master: A test commit https://review.opendev.org/c/openstack/kolla-ansible/+/889925 | 08:19 |
| mmalchuk | 'gist status' show the files in the conflict | 08:19 |
| mmalchuk | open it it the editor, edit, save and commit | 08:20 |
| kevko | i like meld :) | 08:20 |
| mmalchuk | guesswhat[m] https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-using-the-command-line | 08:21 |
| mmalchuk | kevko use it as difftool? or UI only? | 08:23 |
| guesswhat[m] | how can I pull the review branch ? its master, not sure how gerrit does it | 08:24 |
| kevko | difftool and mergetool | 08:24 |
| kevko | you can download ...three dots under your login on gerrit | 08:25 |
| mmalchuk | kevko cool, I use ViM and vimdiff))) | 08:25 |
| kevko | i also using vim | 08:25 |
| kevko | but not vimddiff : | 08:25 |
| kevko | :D | 08:25 |
| mmalchuk | guesswhat[m] git review -d 123456 | 08:26 |
| mmalchuk | will dowload latest patchset from the 123456 change to the current repo | 08:26 |
| guesswhat[m] | have different pc.., I guess gerrit does not handle git over http authentication | 08:27 |
| mmalchuk | and will create the new branch based on the name/tag/etc | 08:27 |
| mmalchuk | guesswhat[m] it support well... get the password from the settings | 08:28 |
| mmalchuk | kevko "Meld is not yet supported on OS X. Third-party binaries are available, and you can install from Homebrew, MacPorts or Fink." :( | 08:28 |
| mmalchuk | guesswhat[m] https://review.opendev.org/settings/#HTTPCredentials | 08:29 |
| mmalchuk | guesswhat[m] but the recomendation is to use ssh keys | 08:29 |
| kevko | mmalchuk: well, you should use real operating system ...for example linux : | 08:33 |
| kevko | :D | 08:33 |
| guesswhat[m] | hm, thanks, but still dont know what to do :D | 08:34 |
| guesswhat[m] | literally | 08:34 |
| guesswhat[m] | problably lack of git skills and sure how merge conflic solving is working | 08:35 |
| mmalchuk | kevko lol | 08:35 |
| kevko | :D | 08:35 |
| mmalchuk | guesswhat[m] https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-using-the-command-line | 08:36 |
| mmalchuk | guesswhat[m] https://www.youtube.com/results?search_query=Resolving+a+merge+conflict+using+the+command+line | 08:36 |
| guesswhat[m] | Holy moly, what is Closes-Bug header? :D Is it also Gerrit specific thingy? | 08:38 |
| kevko | yes, | 08:38 |
| kevko | it's autoclosing launchpad bugs | 08:39 |
| kevko | if merged | 08:39 |
| guesswhat[m] | I remember I had to work with Gerrit 10 years ago and it was a pain in arse, seems nothing changed :D | 08:39 |
| guesswhat[m] | Since then Github or Gitlab are lightyears away | 08:40 |
| kevko | i have no problems with gerrit :) | 08:40 |
| guesswhat[m] | I have to give up on this, its been weeks since I created initial PR, but there is not progress | 08:43 |
| mmalchuk | guesswhat[m] https://wiki.openstack.org/wiki/GitCommitMessages | 08:43 |
| mmalchuk | guesswhat[m] about bugs: https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references | 08:44 |
| guesswhat[m] | mmalchuk: https://review.opendev.org/c/openstack/kolla-ansible/+/888494/1 can be merged, right? Im gonna abandon PR for skyline, maybe someone from skyline would fix this | 08:45 |
| mmalchuk | guesswhat[m] use git in commandline and forget about Gerrit/GitHub UIs ;) | 08:45 |
| mmalchuk | guesswhat[m] you always can give up and abandon gerrit changes but create/leave bug-report, may be someone fix it | 08:48 |
| guesswhat[m] | I reported that bug with Keystone. | 08:48 |
| guesswhat[m] | Both PRs are now abandoned | 08:49 |
| mmalchuk | link? | 08:49 |
| guesswhat[m] | https://bugs.launchpad.net/kolla-ansible/+bug/2028921 | 08:50 |
| guesswhat[m] | But SSO support feature request for Skyline is not created | 08:51 |
| guesswhat[m] | I guess folks from skyline team should solve it | 08:51 |
| mmalchuk | so sso feature in skyline didn't work now? | 08:52 |
| guesswhat[m] | No | 08:52 |
| guesswhat[m] | Its not implemented | 08:53 |
| mmalchuk | bugreport in skyline? | 08:53 |
| guesswhat[m] | Its not a bug :D | 08:53 |
| mmalchuk | if it is not implemented, so changes in kolla-ansible didn't help either | 08:53 |
| guesswhat[m] | No, wrong | 08:53 |
| * mmalchuk confused | 08:54 | |
| guesswhat[m] | 1. skyline.yaml is not possible to override via confg | 08:54 |
| guesswhat[m] | 2. is enough to add two lines of yaml code into skyline.yaml to enable sso | 08:54 |
| guesswhat[m] | did You even check my PRs ? :D | 08:54 |
| mmalchuk | I'm asking about skayline code. there sso support exists? | 08:55 |
| mmalchuk | not kolla-ansible code | 08:55 |
| guesswhat[m] | of course | 08:55 |
| mmalchuk | oh. ok. | 08:55 |
| guesswhat[m] | it working ok | 08:55 |
| guesswhat[m] | but kolla-ansible does not care, but its enough to add two lines of config and also as I said, its not possible to override skyline.yaml like other components | 08:56 |
| guesswhat[m] | also Horizon SSO does not work | 08:56 |
| guesswhat[m] | so SSO does not work in kolla-ansible at all | 08:56 |
| guesswhat[m] | those PRs I created are fixing both Horizon and Skyline | 08:57 |
| guesswhat[m] | but the fact You required to have these spplitted into to PRs | 08:57 |
| guesswhat[m] | forced me to abandon both PRs, cuz I dont really know how to do it in separate PRs | 08:58 |
| guesswhat[m] | clear? :D | 08:58 |
| Vii | Hi. Is it possible to run VPNaaS on Neutron+OVN? | 08:59 |
| mmalchuk | thanks anyway. someone, maybe I, will fix it later. I've updated bugreport | 08:59 |
| guesswhat[m] | mmalchuk: and since sso is broken in kolla I requested backport to stable branches.... | 09:00 |
| guesswhat[m] | kevko: promised to look at these PRs also, but dint have a time | 09:00 |
| opendevreview | Merged openstack/kolla-ansible master: Fix loop label syntax error with ansible-core 2.15 https://review.opendev.org/c/openstack/kolla-ansible/+/886412 | 09:02 |
| guesswhat[m] | this one https://bugs.launchpad.net/kolla-ansible/+bug/2028921/comments/2 is also related | 09:02 |
| guesswhat[m] | mmalchuk: I updated the issue, now it should cover also Skyline and I added link to another fix which has to be merged | 09:05 |
| mmalchuk | guesswhat[m] thank a lot | 09:09 |
| kevko | mmalchuk: did you have a time for letsencrypt review ? | 09:25 |
| mmalchuk | yep. rebased? ci passes? | 09:25 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Fix L3 agent docs typo https://review.opendev.org/c/openstack/kolla-ansible/+/889938 | 09:45 |
| opendevreview | Jake Hutchinson proposed openstack/kolla-ansible master: Fix L3 agent docs typo https://review.opendev.org/c/openstack/kolla-ansible/+/889938 | 09:47 |
| guesswhat[m] | what uid/gid would use nova, cinder, glance when nfs backend is enabled? 0 or 1000 ? | 09:48 |
| Vii | why openstack antelope dont support 2 ceph backends? "cinder_ceph_backends" | 10:27 |
| guesswhat[m] | Vii: You can still use custom config, seems no other backend type is supports more than one backend | 10:34 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 10:35 |
| Vii | https://docs.openstack.org/kolla-ansible/latest/reference/storage/external-ceph-guide.html#cinder | 10:36 |
| Vii | cinder_ceph_backends remove fron ansible-playbook :( | 10:36 |
| guesswhat[m] | iterating over nested maps in ansible in not easy probably :D | 10:41 |
| *** hrww is now known as hrw | 10:50 | |
| mmalchuk | kevko I dit it) 23 comments... it was hard | 11:16 |
| guesswhat[m] | Is possible to use custom docker volume for cinder? Didnt find any option to se it | 11:28 |
| guesswhat[m] | *set it | 11:29 |
| guesswhat[m] | would be great for supporting https://docs.openstack.org/cinder/rocky/configuration/block-storage/backup/posix-backup-driver.html | 11:31 |
| mmalchuk | guesswhat[m] volumes can be configured | 11:38 |
| mmalchuk | there are cinder_api_default_volumes and cinder_api_extra_volumes variables for example for cinder-api | 11:39 |
| mmalchuk | yo always can use *_extra_volumes | 11:40 |
| mmalchuk | for any container | 11:40 |
| guesswhat[m] | oh, okay, thanks, but it wont work as nova_instance_datadir_volume or glance_file_datadir_volume, these are deleted with cleanup role | 11:42 |
| guesswhat[m] | I would create PR similar to this https://github.com/search?q=repo%3Aopenstack%2Fkolla-ansible+nova_instance_datadir_volume&type=code with POSIX backup driver for cinder-volume, but I am afraid that I wont work for You folks | 11:45 |
| mmalchuk | why? yo can share volumes between containers | 11:45 |
| guesswhat[m] | I mean the bureaucracy.. | 11:45 |
| guesswhat[m] | :D | 11:45 |
| guesswhat[m] | To easy AIO deployment | 11:45 |
| guesswhat[m] | ease | 11:46 |
| mmalchuk | not PR, but Gerrit review?) don't give up | 11:47 |
| guesswhat[m] | POSIX + LVM or POSIX + NFS, both are valid AIO lightweight implementations | 11:49 |
| guesswhat[m] | But NFS also does not work for multiple driver, cuz nfs_shares file from config directory is hardcoded https://github.com/openstack/kolla-ansible/blob/acf9375edc8f21439db5b9ae4cce10a92a791bdc/ansible/roles/cinder/templates/cinder-volume.json.j2#L18 | 11:49 |
| guesswhat[m] | it wont copy additional nfs_shares files for other drivers | 11:50 |
| guesswhat[m] | so both solutions are not working for more 1 drivers | 11:50 |
| guesswhat[m] | *drives, not drivers, but it drives me crazy ;) | 11:51 |
| opendevreview | Maksim Malchuk proposed openstack/kayobe master: Add forgotten Multipathd extra configuration. https://review.opendev.org/c/openstack/kayobe/+/889956 | 12:10 |
| Vii | when can we expect code linking to the latest version of "antelope" https://review.opendev.org/c/openstack/kolla-ansible/+/877413 ? | 12:20 |
| mnasiadka | Vii: this is in master, so bobcat only material. | 12:23 |
| Vii | yes i know it's master. But I'm testing 2023.1 and I'm using branch "2023.1" and I'm wondering when the code will be released in branch "2023.1" | 12:30 |
| Vii | I don't know what the release process looks like | 12:31 |
| mnasiadka | No, it will not be backported, it's a feature. | 12:35 |
| mnasiadka | You can backport that downstream (i.e. in your local repository fork) | 12:35 |
| opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: cephadm: log to file https://review.opendev.org/c/openstack/kolla-ansible/+/889968 | 12:58 |
| guesswhat[m] | How can I use own cinder backend config, without using default one cinder_backend_ceph? This https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova-cell/tasks/external_ceph.yml#L17C6-L17C6 step is skipped, so its not possible to configure keyrings if cinder_backend_ceph is not enabled | 13:32 |
| mmalchuk | read doc/source/reference/storage/external-ceph-guide.rst | 13:37 |
| guesswhat[m] | mmalchuk: ??? | 13:38 |
| opendevreview | Will Szumski proposed openstack/kayobe master: Adds initial support for vGPUs https://review.opendev.org/c/openstack/kayobe/+/887200 | 13:40 |
| guesswhat[m] | which sentence does reflect my use case? | 13:40 |
| guesswhat[m] | this step https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/nova-cell/tasks/external_ceph.yml#L17C6-L17C6 is skipped, becuase cinder_backend_ceph is false | 13:41 |
| guesswhat[m] | but also cinder_backend_ceph enabling default backend, which I dont want to be enabled | 13:41 |
| guesswhat[m] | so ? | 13:41 |
| guesswhat[m] | mmalchuk: ^ ? | 13:48 |
| mmalchuk | you should enable cinder_backend_ceph and use overrides for templates | 13:57 |
| guesswhat[m] | where https://github.com/openstack/kolla-ansible/blob/master/doc/source/reference/storage/external-ceph-guide.rst ? | 13:58 |
| mmalchuk | if you want use kolla-ansible and configure ceph backend please use external-ceph-guide.rst as source of knowledge | 13:58 |
| guesswhat[m] | what is "overrides for templates" ? | 13:59 |
| guesswhat[m] | You are misguiding me, same as with the PR for SSO :) | 14:01 |
| guesswhat[m] | https://github.com/search?q=repo%3Aopenstack%2Fkolla-ansible%20overrides%20for%20templates&type=code | 14:01 |
| guesswhat[m] | Nothing, so terms are probably not correct, maybe You did not read my use case correctly | 14:02 |
| mmalchuk | kolla-ansible uses merge_configs module for almost all configs, the ceph also uses | 14:02 |
| guesswhat[m] | Of course... | 14:03 |
| guesswhat[m] | But I asked without creating default Ceph backend... | 14:03 |
| mmalchuk | for example ceph.conf used from the several locations: | 14:03 |
| mmalchuk | "{{ node_custom_config }}/cinder/{{ cluster }}.conf" | 14:03 |
| mmalchuk | "{{ node_custom_config }}/cinder/{{ service_name }}/{{ cluster }}.conf" | 14:04 |
| mmalchuk | you can provide your own config | 14:04 |
| mmalchuk | and it would be merged with defaults from ansible | 14:04 |
| mmalchuk | the same with cinder.conf | 14:06 |
| mmalchuk | create your own file with backends you need | 14:06 |
| mmalchuk | and it will be merged | 14:06 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 14:54 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 15:15 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 15:17 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 15:19 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 16:26 |
| opendevreview | Bartosz Bezak proposed openstack/kayobe master: Add STP option for bridge interfaces https://review.opendev.org/c/openstack/kayobe/+/889859 | 16:29 |
| simondodsley | Can I get some reviews on 2 patches that have been hanging around since April this year? https://review.opendev.org/c/openstack/kolla-ansible/+/879844 and https://review.opendev.org/c/openstack/kolla-ansible/+/879846 | 18:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!