Thursday, 2020-01-30

*** kprice789 has quit IRC00:14
*** luksky has quit IRC01:04
*** gyee has quit IRC01:13
*** scarlisle has quit IRC01:22
*** goldyfruit_ has quit IRC01:43
*** tinwood has quit IRC02:10
*** tinwood has joined #openstack-ansible02:12
*** udesale has joined #openstack-ansible04:40
*** raukadah is now known as chkumar|rover05:57
*** southquist has joined #openstack-ansible06:00
*** elenalindq has joined #openstack-ansible06:36
*** udesale_ has joined #openstack-ansible06:46
*** udesale has quit IRC06:47
*** luksky has joined #openstack-ansible07:14
*** jawad_axd has joined #openstack-ansible07:26
*** yolanda has quit IRC07:32
*** ebbex has quit IRC07:55
*** udesale_ has quit IRC07:58
*** udesale_ has joined #openstack-ansible07:58
*** cshen has joined #openstack-ansible08:08
*** sshnaidm|afk is now known as sshnaidm08:12
*** rpittau|afk is now known as rpittau08:14
*** udesale_ has quit IRC08:28
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493508:29
*** dmsimard5 has joined #openstack-ansible08:31
*** dmsimard has quit IRC08:33
*** dmsimard5 is now known as dmsimard08:33
*** tosky has joined #openstack-ansible08:46
openstackgerritMohammed Naser proposed openstack/openstack-ansible master: manila: refactor playbooks  https://review.opendev.org/70494208:50
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493508:55
*** luksky has quit IRC08:58
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:02
*** DanyC has joined #openstack-ansible09:02
mnaserso looks like we're installing uwsgi everywhere09:02
mnaserwhich means it is living on compute nodes and in my case manila in that wip patch09:03
mnaserhttps://github.com/openstack/ansible-role-uwsgi/blob/master/tasks/uwsgi_install.yml09:03
mnaserthere's no conditionals on that09:03
mnasernor is there any here: https://github.com/openstack/openstack-ansible-os_nova/blob/master/tasks/main.yml#L184-L19209:03
*** DanyC has quit IRC09:04
*** DanyC has joined #openstack-ansible09:04
mnasereven tho uwsgi is empty, we still create a user and create a python_venv09:05
mnaserwe dont deploy any uwsgi files or create systemd units but..09:05
noonedeadpunkoh....... I yes, that's true....09:06
noonedeadpunkI think it's easier to patch uwsgi role09:07
noonedeadpunklet me do that09:07
mnaserthat's the whole reason we invented it :D09:07
mnaserone change and we fix it all09:07
*** DanyC has quit IRC09:09
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/ansible-role-uwsgi master: Run uwsgi tasks only when uwsgi_services defined  https://review.opendev.org/70494409:12
noonedeadpunkmnaser: I think this is exactly what I missed ^09:13
mnasernoonedeadpunk: is "when: <empty-list>" eval to false? i always forget lol09:15
mnaserif thats the case, then that seems good09:15
*** DanyC has joined #openstack-ansible09:15
mnasernoonedeadpunk: we can probably even skip the include_vars too honestly09:15
noonedeadpunkin this empty dict, but yes - I'v tested this as I always forget as well lol)09:15
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:15
mnasernoonedeadpunk: if you have a compelling reason not to skip the include_vars im up for it09:16
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/ansible-role-uwsgi master: Run uwsgi tasks only when uwsgi_services defined  https://review.opendev.org/70494409:17
noonedeadpunkyeah, good call09:18
*** ivve has joined #openstack-ansible09:19
mnaseri'm going to +1 my patches when ive tested them locally as we dont have much manila ci right now09:19
noonedeadpunkactually we could probably test things with https://review.opendev.org/#/c/675934/16 since it passes metal jobs09:21
noonedeadpunkbtw, don't you face issues with undefined things like manila_default_store here https://opendev.org/openstack/openstack-ansible/src/branch/master/playbooks/os-manila-install.yml#L39-L5209:23
noonedeadpunkAs I saw you left these bind mounts while for me playbook was just failing09:25
mnasernoonedeadpunk: yeah i had to manually define it to "generic" or something09:25
mnaseri need to get back to that later and figure out what/how09:25
mnaseror what it really meant09:25
noonedeadpunkthe thing is it's not used anywhere http://codesearch.openstack.org/?q=manila_default_store&i=nope&files=&repos=09:25
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:25
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: refactor: service group names  https://review.opendev.org/70494509:25
noonedeadpunkSo I just dropped this out, but if you'll figure out that they are really required for smth...09:26
mnasernoonedeadpunk: yes i actually noticed they werent used for anything too so i was confused, so i might be in favour of dropping them too09:27
noonedeadpunkwhat I'm thinking is maybe we will merge at least some CI before refactoring?09:31
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: refactor: service group names  https://review.opendev.org/70494509:31
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:31
mnaserif you have something passing im up for it ya09:31
noonedeadpunkat least centos and ubuntu metal jobs with ganesha pass09:33
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Add manila tempest testing  https://review.opendev.org/67465309:34
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add manila CI  https://review.opendev.org/67593409:34
mnasernoonedeadpunk: added a few things to the review09:35
*** udesale has joined #openstack-ansible09:35
*** luksky has joined #openstack-ansible09:35
noonedeadpunkyeah09:36
noonedeadpunkfixing09:36
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: refactor: service group names  https://review.opendev.org/70494509:36
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:36
jrossero/ morning09:38
mnasero/09:39
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493509:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Revert "Mark ubuntu distro install as non voting"  https://review.opendev.org/70494609:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Revert "Mark ubuntu distro install as non voting"  https://review.opendev.org/70494609:41
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add manila CI  https://review.opendev.org/67593409:42
jrossernoonedeadpunk: mnaser: this stack of 3 patches fix up how we install tempest on master in light of the dropping of py2, they're passing now so ready for reviews09:43
jrosserhttps://review.opendev.org/#/c/704567/909:43
mnaserjrosser: reviewing09:43
jrosseri propose to backport 704567 to at least train (and possibly stein) to make the ubuntu distro jobs more sensible and resilient to future changes in tempest09:44
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add manila CI  https://review.opendev.org/67593409:50
mnaserjrosser: reviewed -- and most of https://review.opendev.org/#/q/topic:osa/manila-generic+(status:open+OR+status:merged) is ok right now09:57
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: cleanup: config values  https://review.opendev.org/70495110:06
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Always use virtualenv and pip to install tempest on Ubuntu  https://review.opendev.org/70456710:06
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Always use virtualenv and pip to install tempest on Ubuntu  https://review.opendev.org/70456710:16
openstackgerritMagnus Bergman proposed openstack/openstack-ansible-haproxy_server master: Enable ip_nonlocal_bind for IPv6 in addition to IPv4  https://review.opendev.org/70495510:20
*** spatel has joined #openstack-ansible10:29
openstackgerritMohammed Naser proposed openstack/openstack-ansible master: Add manila_service_region  https://review.opendev.org/70495710:32
*** spatel has quit IRC10:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Always use virtualenv and pip to install tempest on Ubuntu  https://review.opendev.org/70456710:35
*** pcaruana has quit IRC10:46
*** spatel has joined #openstack-ansible10:47
jawad_axd Hi all ! Question is, from instances, download/upload speed is around 10MB/s while from underlying host is 80MB/s. How can I increase/tune bandwidth parameters? I am using Stein,linux bridge,vxlans setup.Any pointers?10:50
*** zbr has quit IRC10:52
*** udesale has quit IRC10:53
jrosserjawad_axd: https://www.youtube.com/watch?v=uItivBAzz8s11:12
*** rpittau is now known as rpittau|bbl11:14
*** cshen has quit IRC11:16
jawad_axd@jrosser Thanks11:18
*** cshen has joined #openstack-ansible11:23
*** zbr has joined #openstack-ansible11:23
*** csmart has quit IRC11:30
jrosserjawad_axd: seeing what you say in the nova channel i assumed you were talking about simply network throughput11:32
jrossertry to test things that independently test your network and storage if you are concerned about the performance of "wget this big file onto my ceph backed disk"11:34
*** zbr has quit IRC11:35
noonedeadpunkmnaser: looking at https://review.opendev.org/#/c/704575/1/tasks/magnum_post_install.yml one more time I have a question - we will run only one time for the play with change you've offered. But we will also upload image every time when role is launched?11:35
*** zbr has joined #openstack-ansible11:36
jawad_axd@jrosser Yes. I guess I need to test on different levels..between instances, on same network, same hosts, different hosts etc..then disk. I am going to do it now. Thanks11:36
*** cshen has quit IRC11:40
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: [WIP] Add image download for manila  https://review.opendev.org/70497211:42
*** alti_17 has joined #openstack-ansible11:43
*** cshen has joined #openstack-ansible11:47
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Mark ubuntu distro install as non voting  https://review.opendev.org/70475011:47
*** pcaruana has joined #openstack-ansible11:48
openstackgerritMerged openstack/openstack-ansible-haproxy_server master: Enable ip_nonlocal_bind for IPv6 in addition to IPv4  https://review.opendev.org/70495511:52
openstackgerritMerged openstack/openstack-ansible-os_tempest master: Use contraints for tempest plugins  https://review.opendev.org/70397912:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest stable/train: Mark ubuntu distro install as non voting  https://review.opendev.org/70497812:03
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest stable/train: Use contraints for tempest plugins  https://review.opendev.org/70498012:03
mnasernoonedeadpunk: no because os_image is smart enough to do a sha of the file locally and compare it to the remote one12:19
*** pcaruana has quit IRC12:21
*** DanyC has quit IRC12:24
*** DanyC has joined #openstack-ansible12:25
*** DanyC has quit IRC12:29
*** DanyC has joined #openstack-ansible12:30
*** DanyC has quit IRC12:31
noonedeadpunkmnaser: than we probably should simplify octavia image upload process12:32
mnasernoonedeadpunk: i tihnk the reason we do that is because os_image doesnt support tags :(12:32
noonedeadpunkI mean maybe we can delegate image upload to os_image and just afterwards set a tag, instead of getting old id, dropping it, and uploading with command12:36
*** DanyC has joined #openstack-ansible12:36
noonedeadpunk(or maybe we can patch os_image one day:))12:36
mnasernoonedeadpunk: i like that idea of delegate image upload to os_image12:43
mnaserbecause we can actually use register:12:43
mnaserand just run the tag12:43
mnaseri mean openstack modules live inside opendev now :p12:44
noonedeadpunkyeah, so offering change to os_image maybe also the way easier, but not sure about backporting....12:45
noonedeadpunkit's new feature anyway12:45
*** ansmith has quit IRC12:45
noonedeadpunkmnaser: btw, CI passed https://review.opendev.org/#/c/675934/ so we can use it in some way to check refactoring12:46
mnasernoonedeadpunk: +1d12:49
noonedeadpunkcan you also take a look at https://review.opendev.org/#/c/674653/ as it's the last unmerged dependency12:51
mnasernoonedeadpunk: done12:56
*** rh-jelabarre has joined #openstack-ansible12:56
*** goldyfruit_ has joined #openstack-ansible12:59
mnasernoonedeadpunk: i will rebase all my stuff ontop of yours13:01
noonedeadpunkmnaser: can I rebase your patches on top of 675934 so that they were tested with CI as well?13:01
mnaser.... lol =P13:01
noonedeadpunkyeah, ok)13:02
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Add manila CI  https://review.opendev.org/67593413:07
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: refactor: service group names  https://review.opendev.org/70494513:07
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493513:07
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: cleanup: config values  https://review.opendev.org/70495113:07
mnasernoonedeadpunk: there13:07
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_keystone stable/train: Federated openid support using auth_mod_openidc  https://review.opendev.org/70500113:08
noonedeadpunkgshippey: why ^ is not for master?13:09
mnasernoonedeadpunk: can i rebase your image download wip stuff (i can test it off locally first of all and also it has some things that break with teh key changes i made)13:10
noonedeadpunkyes, sure13:10
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_keystone master: Federated openid support using auth_mod_openidc  https://review.opendev.org/70500213:10
noonedeadpunkI also see a bug in it...13:11
mnasernoonedeadpunk: what is it, i can fix that too?13:11
noonedeadpunkos_distro: "{{ item.distro }}"13:11
noonedeadpunkhttps://review.opendev.org/#/c/704972/1/tasks/manila_image.yml13:11
gshippey@noonedeadpunk fixed!13:11
noonedeadpunklike properties are probably not needed here13:11
mnasernoonedeadpunk: ah yea13:12
noonedeadpunk(anyway key is not defined)13:12
noonedeadpunkas well as item.public13:12
noonedeadpunkSo probably I should rebase it?13:12
mnasernoonedeadpunk: i can fix those13:12
noonedeadpunkand fix stuff13:13
mnaseralso we probably need to upload it to the service project13:13
mnaservs looks like it uploads to admin one right now13:13
noonedeadpunkyeah, it does upload to admin13:13
noonedeadpunkI was kinda in progress of writing when realizied that it's time reallocate from home, and everything was done on home laptop)13:14
noonedeadpunkso just pushed not to loose progress lol13:14
mnaseroh lol13:14
mnaseryeah i can pick this up as i have an env im going back and forth on right now13:14
noonedeadpunkok, than I'll focus on fixing lxc deployments with ganesha13:16
mnaser++13:17
noonedeadpunkitem.file is undefined as well((13:18
jrosseris there any doc on the architecture you are aiming at here?13:18
jrosseri've been reading the manila docs to try to figure out what you're doing13:19
jrosserbut they are a bit "all choices are available"13:19
noonedeadpunkI think we're doing a bit different things like the docs said :p I'm trying to do in CI smth like https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/13/html-single/cephfs_via_nfs_back_end_guide_for_the_shared_file_system_service/index13:20
noonedeadpunkCephFS via NFS13:20
mnaserjrosser: yeah its very "do as you wish" :13:20
mnaser:(13:20
jrosserwhats the manila image doing?13:21
jrosserthe share-server right?13:21
mnaserjrosser: thats when you want to use the 'generic' driver13:21
mnaser(which im working to implement in this case)13:21
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Add image download for manila  https://review.opendev.org/70497213:22
mnaserkinda like octavia service-model thing13:22
jrosserright yes proper multitenant13:22
mnaserthe cephfs via nfs is pretty multitenant too as it uses ganesha and that has some sort of proper setup too13:22
mnaserafaik,13:22
jrosserhave had some folk ask for shared nfs recently so this is interesting13:22
jrosseryou end up sharing a network though?13:23
jrosserso the different project instances can see each other <- thats maybe my misunderstanding of the ganesha setup13:23
mnaserwith generic manila plugs the vm into the tenant straight13:23
noonedeadpunkmnaser: I think it's used for tempest as well for ganesha deployment13:23
noonedeadpunkout of what I see in tempest logs...13:24
noonedeadpunkor maybe it's jsut my setup....13:25
noonedeadpunkmnaser: seems all patches since https://review.opendev.org/#/c/704935/9 are in conflict because of its dependency13:27
mnasernoonedeadpunk: i think they arent necesarily in conflict but bc that change failed13:28
noonedeadpunkso they depend on https://review.opendev.org/#/c/704942/1 and https://review.opendev.org/#/c/674653/ which are conflicting13:29
*** cshen has quit IRC13:29
*** rpittau|bbl is now known as rpittau13:29
mnasernoonedeadpunk: thats a different repo tho :p13:29
noonedeadpunkyes, but manila patches depends on these two13:30
noonedeadpunkand zuul probably can't use these dependecies as they're conflicting13:31
mnaseryep13:31
mnaseri rechedk the first one which failed13:31
mnaser2nd one seems ok13:31
noonedeadpunkI mean that probably 704942 should be rebased on top of the 67465313:31
*** rh-jelabarre has quit IRC13:36
*** ansmith has joined #openstack-ansible13:38
*** cshen has joined #openstack-ansible13:38
djhankbHello again, I'm still working on my S->T Upgrade and am running into a problem with Swift on the venv build-  if anyone would be able to possibly be able to point me in the right direction: http://paste.openstack.org/show/788973/ it would be greatly appreciated.13:42
jrosserdjhankb: what operating system is that you are using?13:51
djhankbDebian 913:52
jrosserand would this be the swift service virtualenv, on a controller node, with LXC?13:56
djhankbthat is correct, I am running "setup-openstack.yml" on TASK [python_venv_build : Install python packages into the venv]13:57
djhankbit also fails on the swift node itself13:57
jrossercan you try something13:58
djhankbof course13:58
jrosserthe ansible role is in /etc/ansible/roles/os_swift13:58
jrosseri am thinking there is something missing here https://github.com/openstack/openstack-ansible-os_swift/blob/master/vars/debian.yml#L24-L2813:59
jrossercould you add "libsystemd-dev" as an extra item in that list13:59
djhankband this would be on the deployment node correct?14:00
jrosserthats right14:01
jrosseri'm not sure on debian which package provides libsystemd-journal.pc14:02
*** pcaruana has joined #openstack-ansible14:03
djhankbI'll check that out here14:04
jrosserwhat is probably happening here is that the packages which supply those missing .pc files in your log are provided by another ansible role in the CI tests14:05
jrosserwhich run on bare metal without containers, so you can get "pollution" between what one role installs and the next one requires14:05
jrosserafaik we don't actually have a CI job which tests swift in an LXC setup14:05
jrosserjust as reference, here is what happens in os_glance https://github.com/openstack/openstack-ansible-os_glance/blob/master/vars/debian.yml#L27-L2914:07
*** sshnaidm is now known as sshnaidm|afk14:07
*** bjolo has joined #openstack-ansible14:08
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501914:14
jrosserdjhankb: libsystemd-journal.pc may not actually be required..... https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=79514714:17
openstackDebian bug 795147 in libsystemd-journal-dev "libsystemd-journal-dev is missing in Debian Stretch" [Normal,Open]14:17
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501914:20
*** heikkine has joined #openstack-ansible14:22
*** sshnaidm|afk is now known as sshnaidm14:32
*** pcaruana has quit IRC14:34
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501914:38
*** rh-jelabarre has joined #openstack-ansible14:49
*** jawad_axd has quit IRC14:50
ionii had an issue with octavia not sure if because i have some sort of frankenstein with magnum/octavia/heat from train and the rest from stein14:50
ionioctavia-lib was too old in venv for octavia14:50
*** jawad_axd has joined #openstack-ansible14:50
ioniit was octavia-lib 1.1.1 and octavia 5.0.2devsomething14:50
ionii had to manually update octavia-lib to latest14:51
ioniit was an upgrade from stein to train for octavia14:51
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest stable/train: Use contraints for tempest plugins  https://review.opendev.org/70498014:53
*** jawad_ax_ has joined #openstack-ansible14:54
*** jawad_a__ has joined #openstack-ansible14:55
*** jawad_axd has quit IRC14:55
jrosserioni: did you make sure the constraints were pointing to the right train for the components you updated?14:56
jrosserright train / right branch14:56
ionijrosser, yes14:57
jrosserinteresting14:57
ionii did not test to see if is working without any venv, like starting from scratch14:58
ionii just fixed it manually and get over it14:58
*** udesale has joined #openstack-ansible14:58
*** jawad_ax_ has quit IRC14:58
noonedeadpunkmnaser: https://review.opendev.org/#/c/705019/3 is exactly what I was testing localy before pushing14:58
noonedeadpunkbut also public key distibution to nfs hosts15:00
*** jawad_a__ has quit IRC15:00
*** pcaruana has joined #openstack-ansible15:02
*** trident has quit IRC15:06
ioninoonedeadpunk, nice to see that you guys work on manilla15:07
ionii do want to use it in the near future15:08
ioniwith ceph15:08
jrosser^ same, i think i am most interested in the generic model with per project nfs servers, octavia style15:09
*** southquist has quit IRC15:09
jrosserthere is some networking fun to be had there for sure15:10
ionii don't know if used it wrong but i did activate legacy policy on octavia :)15:10
ionido you guys want to drop that?15:11
jrosserlikewise15:11
jrosseri'd like to understand what to do instead becasue unless you do that, out of the box it's not usuable by end users15:12
ionii mostly look at manilla for shared volume for kubernets15:12
*** trident has joined #openstack-ansible15:17
openstackgerritMerged openstack/openstack-ansible stable/train: Use nova_placement_service_password for placement_service_password  https://review.opendev.org/70484215:18
*** chkumar|rover is now known as raukadah15:24
djhankbjrosser: I think your fix got me past where I was however I am now seeing this one which I thought was interesting: ERROR: Error [Errno 2] No such file or directory: 'git' while executing command git clone -q https://opendev.org/openstack/swift /tmp/pip-install-wj3geysk/swift15:39
jrosserdjhankb: i would expect that to come in from here15:43
*** spatel has quit IRC15:43
openstackgerritGeorgina Shippey proposed openstack/openstack-ansible-os_keystone master: Federated openid support using auth_mod_openidc  https://review.opendev.org/70500215:44
jrosserdjhankb: https://github.com/openstack/openstack-ansible-os_swift/blob/master/vars/debian.yml#L2515:46
*** cshen has quit IRC15:52
*** spatel has joined #openstack-ansible15:53
*** gyee has joined #openstack-ansible16:04
djhankbjrosser: is that /etc/ansible/roles/os_swift/vars/debian.yml used to build the LXC containers as well? The git-core package was not installed automatically for some reason16:05
*** ivve has quit IRC16:10
jrosserthat should be on the venv build host (repo container) if i remember correctly16:11
jrosseri.e part of the python_venv_build role tasks is to install those _devel_ packages onto the venv build host16:12
djhankbgotcha, I don't completely understand the underpinnings of what all is going on here, so I'm trying to educate myself as much as possible as I've been building out my labs.16:14
jrosserrun just the os_swift_install playbook16:15
djhankbroger that16:15
jrossertry setting '-e venv_rebuild=yes' on the end of the command16:16
jrossersetting this https://github.com/openstack/ansible-role-python_venv_build/blob/master/defaults/main.yml#L10716:16
jrosserand watch what/where it does the python_venv_build role tasks from within that16:16
*** jawad_axd has joined #openstack-ansible16:23
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: manila: refactor playbooks  https://review.opendev.org/70494216:24
*** alti_17 has quit IRC16:26
*** jawad_axd has quit IRC16:27
*** admin0 has quit IRC16:28
*** admin0 has joined #openstack-ansible16:30
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: Add manila_service_region  https://review.opendev.org/70495716:31
*** udesale has quit IRC16:43
*** jawad_axd has joined #openstack-ansible16:43
*** jawad_axd has quit IRC16:48
*** jawad_axd has joined #openstack-ansible17:04
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: refactor: service group names  https://review.opendev.org/70494517:06
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Start using uWSGI role  https://review.opendev.org/70493517:07
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: cleanup: config values  https://review.opendev.org/70495117:07
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add image download for manila  https://review.opendev.org/70497217:07
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501917:08
*** jawad_axd has quit IRC17:08
spatelFolks, I am trying setup LDAP auth with keystone and doing  keystone_ldap: stanza and running ansible playbook "openstack-ansible os-keystone-install.yml --tags keystone-config" but its not touching any config17:15
spateldoes "--tags keystone-config" is correct way to handle this ?17:16
noonedeadpunkon stein tags do not work17:18
noonedeadpunkon train they probably should....17:18
spatelnoonedeadpunk: really???? i am running stein17:18
spatelnoonedeadpunk: you are saying i have to run full playbook?17:19
spateljust worried if i break something :O17:19
noonedeadpunkyeah, or replace in main.yml mainly include_tasks with import_tasks where it's applicable17:20
noonedeadpunkI think this is commit https://opendev.org/openstack/openstack-ansible-os_keystone/commit/a2783e4e3f84c2681093c90d5fd2daa0f91dac7717:21
spatelnoonedeadpunk: oh! so i can just make change in line 160 instead of everyone right?17:24
spatelinclude_tasks: keystone_ldap_setup.yml17:24
spatelto17:24
spatelimport_tasks: keystone_ldap_setup.yml17:24
*** jawad_axd has joined #openstack-ansible17:25
noonedeadpunkif you need to run only these tasks than yes17:25
spatelYes just single task at present.. do i need to do same on keystone_uwsgi.yml  file or should i leave it alone17:25
noonedeadpunkThere might be a ceveat if some variable which was set during runtime (like with set_fact) is used inside ldap stuff17:26
noonedeadpunkhum, wondering how this works... https://opendev.org/openstack/openstack-ansible-os_keystone/src/branch/master/tasks/main.yml#L13517:27
noonedeadpunklike import with variable in imported filename shouldn't work....17:28
spatelhmm... i think i should run full playbook in that case...17:29
*** jawad_axd has quit IRC17:29
spatelthis is too much editing...17:29
noonedeadpunkyou can try at least17:29
noonedeadpunkchange L160 as you said17:30
spatellets try with L160 and see if i am lucky17:30
spatelwish openstack-ansible has dry run option before commit change17:31
*** evrardjp has joined #openstack-ansible17:33
spatelnoonedeadpunk: that works...17:34
*** jawad_axd has joined #openstack-ansible17:45
*** jawad_axd has quit IRC17:50
*** DanyC has quit IRC17:51
spatelnoonedeadpunk: Thank you!!! so much...17:52
redkriegIs it possible to run two different ceph clusters under openstack-ansible?  The goal is to use one for glance and one for cinder.17:53
*** errr has quit IRC17:56
*** errr has joined #openstack-ansible17:56
*** rpittau is now known as rpittau|afk18:02
*** jawad_axd has joined #openstack-ansible18:06
*** jawad_axd has quit IRC18:11
redkriegor a tiered system using crush maps?18:16
*** sshnaidm is now known as sshnaidm|afk18:20
*** luksky has quit IRC18:25
*** jawad_axd has joined #openstack-ansible18:27
jrosserredkrieg: is there a reason to use multiple clusters? glance and cinder currently use different pools anyway so you can put those on different device classes if you wish18:29
jrosserso you can do hdd for glance and ssd for cinder, as an example18:30
jrosserbut I think you need to think very very carefully, because what I described there might be a bad idea performance wise18:30
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add image download for manila  https://review.opendev.org/70497218:31
jrosserCeph is smart enough to do a no-op snapshot of a glance image and use that in cinder18:31
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501918:31
*** jawad_axd has quit IRC18:31
jrosserso you might be surprised in a bad way about th performance you get with “slower” storage for glance18:32
redkriegjrosser: thanks for the advice.  are device classes the only way to differentiate?  apparently the reasoning behind keeping the services on different hardware is primarily for capacity planning18:34
jrosserone Ceph cluster could have hosts full of ssd / nvme / hdd on the most appropriate hardware for each18:35
jrosseryou can define your own device classes, in inbuilt hdd/ssd just get you going for free18:36
jrosserthen you can choose which pools are placed on which device classes18:36
jrosserall of this can happen within one cluster18:36
redkriegokay, sounds like we'll be using ssd for cinder and hdd for glance.  you're saying that reads for the base image will go to the hdd osds and the reads for everything else on the volume will be on ssd?18:37
jrosserI believe so but you should double check18:38
jrosseryou can also have multiple Ceph back ends in cinder18:38
jrosserI make hdd and ssd volume types available too18:39
redkriegthanks, that helps a lot.  is openstack-ansible able to handle assigning device classes to the cinder/glance pools or is that something I'll have to configure manually afterward?18:39
jrosserup to you really18:39
jrosserosa is a thin wrapper around Ceph-ansible18:39
jrosserso it’s a choice of you using Ceph-a single directly and deploying your own storage cluster, or putting the same data in osa and having everything there18:40
jrosserargh phone typos18:41
redkriegawesome, I'll have to look in to that more.  thanks again for the pointers, I know what direction to look now.18:41
jrosserreally depends on how you want to manage the ceph18:41
jrosserseparate concern, or integrated18:41
jrosserpersonally I separate18:41
redkriegintegrated is always my preference only because I like to have configuration management in a state where I can simply run it again on fresh installs and get the same setup as my production cluster18:42
jrosserthere is some value in decoupling the version of Ceph deployed by a particular osa release and what Ceph-ansible can do natively - otherwise you might be backed in a corner down the line18:44
jrosserlook at the requirements for Ceph release vs. Ceph-ansible version. vs required ansible version18:45
redkriegno direct upgrade path in ceph?18:45
jrosserthere’s a not terribly nice mapping with the osa release version vs ansible version, for example18:45
jrosserrequiring the same a single version for both severely limits your options18:46
jrosser*ansible18:46
*** ivve has joined #openstack-ansible18:47
*** jawad_axd has joined #openstack-ansible18:47
*** jawad_axd has quit IRC18:52
*** sshnaidm|afk is now known as sshnaidm|off18:53
spotznoonedeadpunk: You mind if I clean up some wording in https://review.opendev.org/#/c/705019/5/defaults/main.yml?19:00
noonedeadpunkyeah, sure feel free:) Anyway it's mnaser's patch :p19:01
spotzhehehe19:02
jrosserspotz: o/ thanks for the heads up on the London ops meeting a while back - gshippey and I went a long and it was great19:04
spotzjrosser: great news! Wish I'd been there19:04
openstackgerritAmy Marrich (spotz) proposed openstack/openstack-ansible-os_manila master: Add Manila key generation and distribution  https://review.opendev.org/70501919:05
*** DanyC has joined #openstack-ansible19:09
*** luksky has joined #openstack-ansible19:14
*** DanyC has quit IRC19:39
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible master: [WIP] Adjustments for LXC CI  https://review.opendev.org/70510719:58
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/openstack-ansible-os_manila master: Add LXC CI job  https://review.opendev.org/70510820:00
djhankbjrosser: Just a heads-up, adding 'libsystemd-dev' to 'swift_devel_distro_packages' in /etc/ansible/roles/os_swift/vars/debian.yml and then running the full os-swift-install.yml with '-e venv_rebuild=yes' completed successfully20:08
jrosserdjhankb: that’s great20:10
jrosserwould you like to make a patch?20:10
djhankbI would love to, however I'm not sure of the exact process20:10
jrosserif you are interested in learning that then this is a very small change which is a great way to start out20:11
jrosserotherwise I can do it for you20:11
*** hamzy_ has joined #openstack-ansible20:11
djhankbof course I would be, if you would be willing to point me in the right direction.20:11
jrosserhave a read of this https://docs.openstack.org/infra/manual/developers.html20:12
jrosserfirst patch you submit involves setting up a bunch of stuff but once that is done it’s much easier for 2nd/3rd.....20:13
*** hamzy has quit IRC20:14
jrosserdjhankb: there is a sandbox repo that you can test this all out in https://docs.openstack.org/infra/manual/sandbox.html20:16
djhankbawesome, I already have most of that set up- UbuntuOne, Git SSH Keys, etc. That sandbox looks helpful too. I've used Git a lot on my own personal stuff but not really at all when it comes to shared repos and pull requests and stuff.20:18
jrosserok, so really the new thing here is the gerrit workflow and git-review which is a little different to what you do on github20:19
jrosserit’s getting a little late here, spotz are you around to help out getting to grips with making patches if needed ^ ?20:21
djhankbOk cool, I'll give this a go. Thank you so much for your time and patience with helping me out today.20:22
spotzYou mean how to submit, etc? That I can do:)20:23
djhankbThanks spotz20:24
spotzdjhankb: If I don't respond right away please bear with me I'm on a conference call20:25
jrosserawesome! thanks for taking the time to contribute djhankb20:25
djhankbI'm a 20+ year sysadmin/network engineer - learning the workflows around coding have been challenging but fun and rewarding.20:25
djhankbspotz: not a problem, I have another thing I'm going to need to wrap up before I can pivot over to the patch20:27
*** jawad_axd has joined #openstack-ansible20:31
*** andymccr has quit IRC20:35
*** jawad_axd has quit IRC20:35
spotzdjhankb: Ok off the call, if you haven't yet checkout https://docs.openstack.org/contributors/code-and-documentation/index.html I'm going to get you a link to the PDF I used to use for Git and Gerrit Lunch and Learn in case you still need to get accounts20:45
djhankboh that would be awesome20:46
spotzdjhankb: https://github.com/spotz/ghc/blob/master/gitghcpdf.pdf20:46
spotzSetting people up to contribute is my jam:)20:46
djhankbIt's nice to help those help themselves!20:47
spotzI love handson workshops:)20:51
*** jawad_axd has joined #openstack-ansible20:51
*** jawad_axd has quit IRC20:56
cjloaderspotz is great!21:04
spotzThanks cjloader:)21:04
*** elenalindq has quit IRC21:11
*** jawad_axd has joined #openstack-ansible21:12
*** jawad_axd has quit IRC21:16
*** ansmith has quit IRC21:24
djhankbok spotz - you still there?21:28
*** jawad_axd has joined #openstack-ansible21:32
spotzdjhankb: yeep21:33
*** jawad_axd has quit IRC21:37
*** rh-jelabarre has quit IRC21:38
djhankbCool - So I've got my account all in order, ssh keys added, git configured, and I'm at the point of creating a branch (I think) and am curious if I need to be referencing a specific Bug# or if we just give a good description of the issue21:40
djhankbThere wouldn't be a bug# because I just hit it this morning21:40
djhankb(The way your documentation reads is a bit intimidating for the novice!)  ;-)21:41
*** spatel has quit IRC21:46
*** jawad_axd has joined #openstack-ansible21:53
*** jawad_axd has quit IRC21:57
djhankbOk, so once I have the branch created, I simply make my change, commit and then run "git review" ?22:04
*** ansmith has joined #openstack-ansible22:07
openstackgerritHenry Bonath proposed openstack/openstack-ansible-os_swift master: add libsystemd-dev to swift_devel_distro_packages  https://review.opendev.org/70512522:13
djhankbooh there it goes! (sorry I am excited)22:13
djhankbspotz: Now that it's been added to Gerrit - is that all I need to do at this point?22:17
spotzdjhankb: this yours https://review.opendev.org/#/c/705125/ ?22:18
spotzdjhankb: Keep in mind the PDF is for a workshiop with helpers in the room. Sorry had to do real work22:19
djhankbspotz: yes that was me22:20
*** jawad_axd has joined #openstack-ansible22:21
spotzdjhankb: Ok so the code itself looks good and some people would review it at this point. I tend to watch to see if it passes tests and gets verified before reviewing. Now if it fails you need to check the logs, sometimes it's not your patch but might need a recheck22:22
spotzNow depending on the load verification can be quick or take a while22:24
*** jawad_axd has quit IRC22:25
djhankbok right on, I just want to make sure I'm following the proper procedure. This was a simple thing as you could see.22:27
spotzThose are my favorites:)22:33
*** jawad_axd has joined #openstack-ansible22:42
*** jawad_axd has quit IRC22:47
*** ivve has quit IRC23:07
*** errr has quit IRC23:10
*** errr has joined #openstack-ansible23:10
*** luksky has quit IRC23:21
*** jawad_axd has joined #openstack-ansible23:23
*** jawad_axd has quit IRC23:28
*** jawad_axd has joined #openstack-ansible23:44
*** jawad_axd has quit IRC23:52

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!