tux_ | I have VLAN 31 for my customer how does neutron will provide IP to VLAN 31 if infra node doesn't have VLAN 31 ??? | 00:01 |
---|---|---|
*** yolanda_ has joined #openstack-ansible | 00:06 | |
*** yolanda__ has quit IRC | 00:09 | |
tux_ | I can see dhcp namespace got created on neutron_agents_container but question is how does that namespace will talk to my VLAN 31 instance ? | 00:09 |
*** chyka has joined #openstack-ansible | 00:10 | |
tux_ | I didn't create any router yet. Do i need to create router in-order to make it work? | 00:12 |
*** chyka has quit IRC | 00:15 | |
*** tux_ has quit IRC | 00:43 | |
openstackgerrit | Merged openstack/openstack-ansible-os_octavia master: Adds the issuer to the CAs https://review.openstack.org/576234 | 00:44 |
*** tux_ has joined #openstack-ansible | 00:45 | |
*** abelur_ has joined #openstack-ansible | 00:45 | |
*** gregoryo has joined #openstack-ansible | 00:48 | |
*** abelur_ has quit IRC | 00:48 | |
*** tux_ has quit IRC | 00:49 | |
*** dxiri has quit IRC | 00:56 | |
*** tux_ has joined #openstack-ansible | 00:57 | |
*** tux_ has quit IRC | 00:58 | |
*** vurtz has quit IRC | 01:04 | |
*** harryhuang_ has joined #openstack-ansible | 01:04 | |
*** tux_ has joined #openstack-ansible | 01:23 | |
*** cmart has joined #openstack-ansible | 01:28 | |
*** yolanda__ has joined #openstack-ansible | 03:15 | |
*** yolanda_ has quit IRC | 03:18 | |
*** yolanda__ has quit IRC | 03:23 | |
*** yolanda__ has joined #openstack-ansible | 03:29 | |
*** yolanda_ has joined #openstack-ansible | 03:34 | |
*** yolanda__ has quit IRC | 03:36 | |
*** chyka has joined #openstack-ansible | 03:36 | |
*** chyka has quit IRC | 03:41 | |
*** cshen has joined #openstack-ansible | 03:43 | |
*** yolanda__ has joined #openstack-ansible | 03:44 | |
*** yolanda_ has quit IRC | 03:45 | |
*** cshen has quit IRC | 03:47 | |
*** yolanda_ has joined #openstack-ansible | 04:03 | |
*** yolanda__ has quit IRC | 04:06 | |
*** yolanda has joined #openstack-ansible | 04:08 | |
*** poopcat has quit IRC | 04:08 | |
openstackgerrit | Merged openstack/openstack-ansible-os_designate master: Add packages required for osprofiler https://review.openstack.org/573502 | 04:09 |
*** yolanda_ has quit IRC | 04:09 | |
*** yolanda_ has joined #openstack-ansible | 04:10 | |
*** yolanda has quit IRC | 04:13 | |
*** hamzy has quit IRC | 04:17 | |
openstackgerrit | Merged openstack/openstack-ansible-os_aodh master: Add packages required for osprofiler https://review.openstack.org/573496 | 04:17 |
*** hamzy has joined #openstack-ansible | 04:18 | |
*** Bhujay has quit IRC | 04:20 | |
*** tux_ has quit IRC | 04:21 | |
*** hamzy has quit IRC | 04:22 | |
*** hamzy has joined #openstack-ansible | 04:36 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Set force=true when creating the rpc users https://review.openstack.org/577988 | 05:02 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Set force=true when creating the RPC users https://review.openstack.org/577988 | 05:02 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-os_glance master: Convert role to use a common systemd mount role https://review.openstack.org/552185 | 05:06 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Add kibana custom dashboard https://review.openstack.org/577942 | 05:10 |
*** chyka has joined #openstack-ansible | 05:14 | |
*** chyka has quit IRC | 05:19 | |
*** Bhujay has joined #openstack-ansible | 05:20 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Add kafka output plugin to logstash https://review.openstack.org/576949 | 05:21 |
openstackgerrit | Merged openstack/openstack-ansible-openstack_hosts master: Update when syntax and fix deprecation warnings https://review.openstack.org/574757 | 05:22 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Add kibana custom dashboard https://review.openstack.org/577942 | 05:31 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: Remove the duplicated word https://review.openstack.org/577456 | 05:33 |
*** udesale has joined #openstack-ansible | 05:44 | |
*** jmnk has joined #openstack-ansible | 05:49 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-os_glance master: Correct role rerun when using glance with NFS https://review.openstack.org/577993 | 05:57 |
cloudnull | Folks, this is a pretty important change should a deployment be using NFS with glance. Pretty simple change overall, just need to make sure we're not trying to recreate directories unnecessarily. | 05:59 |
*** openstackgerrit has quit IRC | 06:04 | |
*** mbuil has joined #openstack-ansible | 06:06 | |
cloudnull | https://review.openstack.org/#/q/starredby:cloudnull+status:open,n,z | 06:06 |
cloudnull | those are a couple important change we need to address, if folks get a change to review them it'd be greatly appreciated. | 06:07 |
*** biffi has joined #openstack-ansible | 06:15 | |
*** vnogin has joined #openstack-ansible | 06:20 | |
*** yolanda__ has joined #openstack-ansible | 06:21 | |
*** cshen has joined #openstack-ansible | 06:23 | |
*** yolanda_ has quit IRC | 06:24 | |
*** openstackgerrit has joined #openstack-ansible | 06:25 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Set force=true when creating the RPC users https://review.openstack.org/577988 | 06:25 |
*** lvdombrkr has joined #openstack-ansible | 06:26 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible-plugins master: Update physical host mapping for ansible 2.4+ https://review.openstack.org/572229 | 06:31 |
ivve | cloudnull: nearly up & running with new keepalived | 06:34 |
ivve | stay tuned! | 06:34 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Update Ansible to 2.5.5 https://review.openstack.org/578000 | 06:37 |
*** skyscraper has quit IRC | 06:37 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/openstack-ansible master: Set force=true when creating the RPC users https://review.openstack.org/577988 | 06:37 |
cloudnull | ivve nice! | 06:39 |
cloudnull | I'm off to sleep. let me know how it goes, im interested if that's the fix and something we might be able to backport where appropriate. | 06:39 |
*** nicolasbock has joined #openstack-ansible | 06:40 | |
ivve | will do | 06:44 |
ivve | so far so good | 06:44 |
*** maybebuggy has joined #openstack-ansible | 06:48 | |
*** gkadam has joined #openstack-ansible | 06:58 | |
*** shardy has joined #openstack-ansible | 07:10 | |
jrosser | odyssey4me: i tested this locally and it worked https://review.openstack.org/#/c/568637/ however it got rebased last night and now fails in new and interesting ways in ci | 07:13 |
*** pcaruana has joined #openstack-ansible | 07:20 | |
*** vnogin has quit IRC | 07:22 | |
*** epalper has joined #openstack-ansible | 07:22 | |
*** klippo has joined #openstack-ansible | 07:38 | |
*** gillesMo has joined #openstack-ansible | 07:39 | |
*** tosky has joined #openstack-ansible | 07:41 | |
*** gillesMo has quit IRC | 07:42 | |
*** gillesMo has joined #openstack-ansible | 07:42 | |
*** lvdombrkr has quit IRC | 07:47 | |
*** lvdombrkr has joined #openstack-ansible | 07:48 | |
*** pmannidi has quit IRC | 07:51 | |
*** lvdombrkr89 has joined #openstack-ansible | 07:56 | |
*** lvdombrkr has quit IRC | 07:58 | |
ivve | cloudnull: okay so the verdict, it seems to work fine so far. i will do some more testing but no faults from keepalived. some things to concider. the keepalived.conf needs some rebuilding. absolute paths to binaries and some default user definition that i haven't checked into yet. also i would concider the ping check to use something else, its currently sending icmp to k-root server(s) | 07:58 |
ivve | can't remember the variable but its the in the keepalived playbook | 07:59 |
ivve | 193.0.14.129, this is the address | 08:00 |
jrosser | ivve: i think you can overrride that in your user vars | 08:02 |
*** mbuil has quit IRC | 08:11 | |
*** gillesMo has quit IRC | 08:14 | |
*** koukou has joined #openstack-ansible | 08:15 | |
ivve | jrosser: ofc but it should be some kinda default in user_variables.yml so that it is very obvious. perhaps in the user_variables.yml.example | 08:24 |
ivve | i will submit something | 08:24 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Ensure that repo checks use the right values https://review.openstack.org/564422 | 08:27 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Ensure that the repo servers also use the pypi cache https://review.openstack.org/568502 | 08:27 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Remove default pip_links value https://review.openstack.org/568637 | 08:27 |
koukou | Hello, on https://review.openstack.org/#/c/577384/ error is "percona-xtrabackup-2.3.10-1.el7.x86_64 Requires: libev.so.4", only centos fails. Fix should be: "In order to sucessfully install |Percona XtraBackup| ``libev`` package will need to be installed first. ``libev`` package can be installed from the `EPEL <https://fedoraproject.org/wiki/EPEL>`_ repositories." | 08:29 |
*** electrofelix has joined #openstack-ansible | 08:35 | |
openstackgerrit | kourosh vivan proposed openstack/openstack-ansible-galera_server master: Fix xtrabackup install https://review.openstack.org/578028 | 08:37 |
*** rpittau has joined #openstack-ansible | 08:37 | |
*** shardy has quit IRC | 08:38 | |
*** gregoryo has quit IRC | 08:39 | |
*** shardy has joined #openstack-ansible | 08:43 | |
*** gkadam_ has joined #openstack-ansible | 08:44 | |
*** gkadam has quit IRC | 08:45 | |
*** gkadam_ has quit IRC | 08:45 | |
*** gkadam_ has joined #openstack-ansible | 08:45 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Correct README https://review.openstack.org/578030 | 08:47 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Update README to include using the file-based backing store https://review.openstack.org/578031 | 08:47 |
*** gkadam__ has joined #openstack-ansible | 08:48 | |
*** gkadam_ has quit IRC | 08:51 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Run each setup playbook individually https://review.openstack.org/578035 | 09:00 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Use package module properly https://review.openstack.org/578036 | 09:00 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Set tftp/pxe directory permissions using file module https://review.openstack.org/578037 | 09:00 |
*** tux_ has joined #openstack-ansible | 09:08 | |
*** tux_ has quit IRC | 09:09 | |
*** shardy has quit IRC | 09:10 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Ensure a consistent and readable style https://review.openstack.org/578043 | 09:13 |
*** shardy has joined #openstack-ansible | 09:13 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Ensure a consistent and readable style https://review.openstack.org/578043 | 09:14 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Enable saving and re-using file-based backing images https://review.openstack.org/578047 | 09:24 |
odyssey4me | jrosser cloudnull ^ that's a string of patches to the MNAIO tooling which I'd appreciate reviews for - if there're any more ideas to improve how it's used, I'm game for doing more | 09:25 |
jrosser | sure will look | 09:26 |
jrosser | we had excellent success with ramdisk backing for the VM | 09:26 |
jrosser | which i'm not sure ever got written up | 09:26 |
odyssey4me | yep, I've not seen that exposed anywhere | 09:29 |
*** vnogin has joined #openstack-ansible | 09:29 | |
odyssey4me | jrosser: I'm thinking of adding a thing that allows the secondary disk appropriation to be configured as whatever file system you like, instead of forced usage of lvm | 09:35 |
jrosser | odyssey4me: think is that /dev/shm is a basically a ramdisk | 09:36 |
jrosser | and anything that gets stuck in there goes faaaaast but is ephemeral | 09:36 |
odyssey4me | apparently so: https://www.cyberciti.biz/tips/what-is-devshm-and-its-practical-usage.html | 09:37 |
jrosser | odyssey4me: on a big server (128G ram or so) it works straight out of the box just locating all the VM on /dev/shm, there really isnt much config to do | 09:42 |
jrosser | odyssey4me: also an awful lot of time in the MNAIO is spent setting up many instances of basically the same VM | 09:44 |
jrosser | i always though that was a very good place for some optimisation to make it boot some4 kind of base image, or PXEboot one and reuse it | 09:44 |
jrosser | becasue it really really hurts IO wise when it's booting all of that up | 09:45 |
odyssey4me | yep, one of those patches implements the use of a base image for each vm - I'm looking now into extending that to base it on a single image downloaded - so there's a chain: base distro image -> server-specific image -> runtime image | 09:46 |
odyssey4me | the only trouble with doing that is that the pxeboot is doing some nice things which a downloaded image won't have | 09:46 |
*** yolanda_ has joined #openstack-ansible | 09:50 | |
odyssey4me | hmm, it does look somewhat impractical to do what I was thinking as it would require re-implementing a bunch of the network/disk setup in ansible | 09:50 |
odyssey4me | perhaps it's better to just use a file system which deduplicates | 09:50 |
odyssey4me | it won't kill the IO though | 09:50 |
openstackgerrit | Merged openstack/openstack-ansible-os_panko master: Add packages required for osprofiler https://review.openstack.org/573526 | 09:52 |
*** yolanda__ has quit IRC | 09:53 | |
jrosser | odyssey4me: regarding this https://review.openstack.org/#/c/568637/ is there a reason the behaviour seemed to change when it was rebased? | 09:54 |
odyssey4me | yeah, rebased without the others actually makes the pip_links work, but in a bad way - without https://review.openstack.org/564422 the pypi server being used for the build will always be upstream, instead of the repo server at the right time | 09:56 |
odyssey4me | and without https://review.openstack.org/568502 the onmetal build breaks | 09:56 |
*** dxiri has joined #openstack-ansible | 09:56 | |
jrosser | don't the patches always get applied to the head of the branch when tested? thats what i'm not quite following | 09:57 |
odyssey4me | ah no, this chain will be based on each other when implemented | 09:58 |
odyssey4me | but the whole chain will be rebased on top of the head of the branch | 09:58 |
*** dxiri has quit IRC | 10:01 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Correct Mac OS X instructions for virt-manager https://review.openstack.org/578053 | 10:02 |
jrosser | odyssey4me: that /dev/shm backing store is applicable to AIO as well, i've done that with zfs+dedupe (same would go for btrfs) entirely in RAM on a 16G laptop | 10:06 |
jrosser | key is a choice of filesystem/container tech that can combine both snapshots and dedupe | 10:07 |
*** mbuil has joined #openstack-ansible | 10:08 | |
*** cshen has quit IRC | 10:09 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Disable SSH key checks on host https://review.openstack.org/578059 | 10:23 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Disable SSH key checks on host https://review.openstack.org/578059 | 10:37 |
odyssey4me | jrosser: it looks to me like this could be a run_once task? https://github.com/openstack/openstack-ansible-ops/blob/master/multi-node-aio/playbooks/deploy-vms.yml#L87-L95 | 10:42 |
odyssey4me | it seems to me that it's checking for whether the host is ready for guest imports, rather than verifying anything to do with the guests | 10:43 |
odyssey4me | actually, it just doesn't need the with_items as far as I can see | 10:43 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Check capabilities only once https://review.openstack.org/578060 | 10:49 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Correct README https://review.openstack.org/578030 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Update README to include using the file-based backing store https://review.openstack.org/578031 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Run each setup playbook individually https://review.openstack.org/578035 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Use package module properly https://review.openstack.org/578036 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Set tftp/pxe directory permissions using file module https://review.openstack.org/578037 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Ensure a consistent and readable style https://review.openstack.org/578043 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Enable saving and re-using file-based backing images https://review.openstack.org/578047 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Correct Mac OS X instructions for virt-manager https://review.openstack.org/578053 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Disable SSH key checks on host https://review.openstack.org/578059 | 10:54 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Check capabilities only once https://review.openstack.org/578060 | 10:54 |
*** cshen has joined #openstack-ansible | 11:00 | |
openstackgerrit | Merged openstack/openstack-ansible-plugins master: Update physical host mapping for ansible 2.4+ https://review.openstack.org/572229 | 11:04 |
*** vnogin has quit IRC | 11:09 | |
*** vnogin has joined #openstack-ansible | 11:10 | |
*** cmart has quit IRC | 11:15 | |
*** cshen has quit IRC | 11:17 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Improve change/error handling for VM define/create https://review.openstack.org/578067 | 11:19 |
*** cshen has joined #openstack-ansible | 11:20 | |
openstackgerrit | Merged openstack/openstack-ansible-os_rally master: Switch to using project-templates https://review.openstack.org/577517 | 11:23 |
*** cmart has joined #openstack-ansible | 11:29 | |
openstackgerrit | Merged openstack/openstack-ansible-openstack_openrc master: Switch to using project-templates https://review.openstack.org/577495 | 11:32 |
*** udesale has quit IRC | 11:33 | |
*** ansmith has quit IRC | 11:48 | |
*** cshen_ has joined #openstack-ansible | 11:49 | |
*** cshen has quit IRC | 11:52 | |
*** gillesMo has joined #openstack-ansible | 11:55 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Switch lxc_container_backing_store to default "dir" https://review.openstack.org/578079 | 11:56 |
*** shardy_ has joined #openstack-ansible | 12:12 | |
*** shardy has quit IRC | 12:15 | |
*** shardy_ is now known as shardy | 12:15 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Revert "Add LXC and PIP mirrors" https://review.openstack.org/578086 | 12:22 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Revert "Add LXC and PIP mirrors" https://review.openstack.org/578086 | 12:23 |
odyssey4me | cloudnull mnaser logan- ^ please take a peek at that revert... it's necessary to keep everything working as it should be | 12:24 |
odyssey4me | mnaser: no worries, it's way more complicated than it should be :/ | 12:34 |
mnaser | odyssey4me: yeah I just kinda was like “oh those mirrors are missing from here” lol | 12:35 |
odyssey4me | this is why I want to converge to a single test system | 12:35 |
odyssey4me | there're just too many subtle differences | 12:35 |
odyssey4me | and it takes a lot of code walking to unwind it all | 12:35 |
*** maybebuggy has quit IRC | 12:42 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ops master: MNAIO: Resolve all Ansible 2.5-related deprecation warnings https://review.openstack.org/578091 | 12:44 |
jrosser | cloudnull: perhaps this would be cleaner than using elasticdump https://github.com/rocknsm/rock-dashboards/blob/master/load.sh | 12:47 |
jrosser | npm is causing me a pile of grief | 12:47 |
*** kysse has quit IRC | 13:03 | |
*** KeithMnemonic has joined #openstack-ansible | 13:06 | |
*** gillesMo has quit IRC | 13:08 | |
koukou | we have a lot of failed jobs on 'Connection' object has no attribute 'container_user', why ? | 13:09 |
*** ansmith has joined #openstack-ansible | 13:10 | |
*** jamesdenton has joined #openstack-ansible | 13:12 | |
mnaser | koukou: https://review.openstack.org/#/c/577884/ | 13:18 |
mnaser | odyssey4me: ^ fwiw the above is passing | 13:24 |
mnaser | hwoarang: as much as i'd like to update the commit message, that's another 2 hours or so of going through ci + queue time + potential timeouts because of issues at packethost.. and our gates are blocked, so it would be nice to land it today | 13:25 |
odyssey4me | fair enough - what's the deal with packethost? | 13:25 |
*** tux_ has joined #openstack-ansible | 13:26 | |
*** evrardjp has joined #openstack-ansible | 13:27 | |
mnaser | odyssey4me: for some reason our jobs seem to crash netwokring and the job times out | 13:28 |
mnaser | it also turns out that neutron is affected because of the smaller mtu | 13:28 |
mnaser | there's a stacktrace in the logs which seems to be related to gro/lro/so and it loses everything | 13:29 |
*** evrardjp_ has quit IRC | 13:30 | |
odyssey4me | mnaser: hmm, is this happening for role tests and integrated build tests? | 13:31 |
mnaser | odyssey4me: yes, everything | 13:31 |
*** udesale has joined #openstack-ansible | 13:31 | |
odyssey4me | ah ok, unfortunately that's a space I'm no good at :/ | 13:32 |
cloudnull | Cores if we can get this one through too https://review.openstack.org/#/c/578079 it should help stable/queens out | 13:32 |
mnaser | that's a good one | 13:33 |
odyssey4me | I wonder if jamesdenton or mbuil could perhaps help out? | 13:33 |
mnaser | odyssey4me: i suspect it's an infra issue at that point | 13:34 |
jamesdenton | you rang? | 13:34 |
mnaser | lol | 13:34 |
odyssey4me | in infra issue as in the cloud provider is doing something funky which we can't work around? | 13:34 |
mnaser | yes, i don't think it's something we can work around, i think it's a hypervisor issue from my experience but | 13:35 |
*** jwitko has joined #openstack-ansible | 13:35 | |
odyssey4me | ok, so I take it that devstack's also hitting issues | 13:35 |
mnaser | yep | 13:35 |
mnaser | http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2018-06-25.log.html#t2018-06-25T16:01:34 some discussion here | 13:35 |
odyssey4me | aha, ok | 13:35 |
* odyssey4me puts jamesdenton back in the box. | 13:35 | |
*** klamath has joined #openstack-ansible | 13:35 | |
mnaser | fun traceback at the time of crash http://logs.openstack.org/38/577838/1/check/openstack-ansible-functional-centos-7/089d056/logs/host/messages.txt.gz | 13:36 |
mbuil | odyssey4me: how could I help you? | 13:36 |
mnaser | and http://eavesdrop.openstack.org/irclogs/%23openstack-infra/latest.log.html#t2018-06-26T11:19:24 | 13:36 |
mnaser | cloudnull: https://review.openstack.org/#/c/577884/ ill trade you for that one | 13:37 |
mnaser | :P | 13:37 |
cloudnull | done | 13:39 |
mnaser | thank you | 13:39 |
mgariepy | quick and easy review https://review.openstack.org/#/c/577151/ | 13:40 |
odyssey4me | mbuil jamesdenton apologies, it appears that for now the issues at packethost are not something we can do anything about just yet, and mnaser has it covered for now... but thanks for responding! | 13:42 |
odyssey4me | cloudnull: all those ops role patches are piled on top of each other :) | 13:43 |
odyssey4me | I'm looking into making the vm storage back-end more pluggable instead of just having LVM or file. | 13:44 |
odyssey4me | I figure that zfs or btrfs might be very nice and fast. :) | 13:44 |
jamesdenton | no worries | 13:45 |
cloudnull | yup. I refrained from +w if it had something functional being changed. I hope that someone else using it might be able to take a look to get those in. | 13:45 |
cloudnull | I think a Btrfs OR zfs option would be fantastic | 13:46 |
cloudnull | small, fast, light | 13:46 |
odyssey4me | sounds good, d34dh0r53 has been using the tooling, so perhaps he can also take a peek | 13:46 |
cloudnull | ++ | 13:46 |
cloudnull | I have a build going so if he doesn't get to it I will circle back | 13:46 |
*** tux_ has quit IRC | 13:48 | |
cloudnull | here's another fun one. | 13:48 |
cloudnull | https://review.openstack.org/#/c/578000/ | 13:48 |
cloudnull | ansible 2.5.4 has a bug in it https://github.com/ansible/ansible-modules-extras/issues/1278 which is noted in the commit message | 13:49 |
cloudnull | changing rabbitmq user password, like we'll do when converting from $BLAH_rabbitmq_password to $BLAH_oslomsg_$BLAH_password will fail | 13:50 |
cloudnull | its fixed in 2.5.5 | 13:50 |
odyssey4me | well, that's always nice | 13:50 |
mbuil | odyssey4me: ok , good luck :) | 13:52 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Correct README https://review.openstack.org/578030 | 13:53 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Update README to include using the file-based backing store https://review.openstack.org/578031 | 13:53 |
*** kysse has joined #openstack-ansible | 13:58 | |
openstackgerrit | Maksim Malchuk proposed openstack/openstack-ansible master: Use standard env variables for the AIO bootstrap https://review.openstack.org/577652 | 14:04 |
*** cjloader has joined #openstack-ansible | 14:05 | |
openstackgerrit | Maksim Malchuk proposed openstack/openstack-ansible stable/queens: Use standard env variables for the AIO bootstrap https://review.openstack.org/577653 | 14:05 |
*** biffi has quit IRC | 14:16 | |
*** chhagarw has joined #openstack-ansible | 14:19 | |
*** tux_ has joined #openstack-ansible | 14:23 | |
*** tux__ has joined #openstack-ansible | 14:32 | |
tux__ | cloudnull: morning | 14:33 |
tux__ | i have question for you | 14:33 |
*** tux_ has quit IRC | 14:35 | |
*** DanyC has joined #openstack-ansible | 14:35 | |
tux__ | I have this setting in my physical_interface_mappings = vlan:br-vlan,flat:eth12 and i want to get rid of flat:eth12 | 14:36 |
tux__ | I think i need to remove container_interface: "eth12" & host_bind_override: "eth12" whole stanza from config.yml | 14:37 |
tux__ | anyone know how to do try test run on OSA ? | 14:37 |
d34dh0r53 | odyssey4me: which one? | 14:38 |
tux__ | I want to see what its changing ? | 14:38 |
cjloader | can I get another review on https://review.openstack.org/#/c/575504/? | 14:38 |
*** chhavi__ has joined #openstack-ansible | 14:41 | |
*** chhagarw has quit IRC | 14:41 | |
*** dxiri has joined #openstack-ansible | 14:43 | |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_octavia stable/queens: Adds the issuer to the CAs https://review.openstack.org/578118 | 14:44 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_glance master: Drop python-cinderclient usage https://review.openstack.org/578120 | 14:54 |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_tempest master: Drop python-cinderclient usage https://review.openstack.org/578122 | 14:55 |
d34dh0r53 | alright ansible gods, wtf am I doing wrong here http://paste.openstack.org/show/724320/ | 14:56 |
*** vnogin has quit IRC | 14:58 | |
d34dh0r53 | nevermind, I'm a moron | 14:58 |
d34dh0r53 | ^^ the answer to most of my questions | 14:58 |
*** cjloader_ has joined #openstack-ansible | 14:59 | |
*** cjloader has quit IRC | 14:59 | |
openstackgerrit | weizj proposed openstack/openstack-ansible-os_heat master: Drop python-cinderclient usage https://review.openstack.org/578123 | 15:00 |
*** vnogin has joined #openstack-ansible | 15:01 | |
mnaser | evrardjp, cloudnull: http://git.openstack.org/cgit/openstack/openstack-ansible/tree/tests/roles/bootstrap-host/defaults/main.yml#n206 do you know the reason behind that failure? | 15:01 |
mnaser | err, behind the introduction of checksums there | 15:02 |
openstackgerrit | weizj proposed openstack/openstack-ansible-repo_build master: Drop python-cinderclient usage https://review.openstack.org/578124 | 15:02 |
mnaser | i found it introduced here https://github.com/openstack/openstack-ansible/commit/51f4dec329d5b451450234990fcf8c6baa3fa80a | 15:02 |
*** FrankZhang has joined #openstack-ansible | 15:02 | |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass. https://review.openstack.org/571388 | 15:07 |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass. https://review.openstack.org/571388 | 15:08 |
*** goldenfri has joined #openstack-ansible | 15:12 | |
*** cshen_ has quit IRC | 15:12 | |
*** Bhujay has quit IRC | 15:12 | |
odyssey4me | d34dh0r53: all my patches against the ops repo (there are a few ;)) | 15:14 |
d34dh0r53 | odyssey4me: ok, cool | 15:14 |
d34dh0r53 | odyssey4me: I'll take a look | 15:15 |
odyssey4me | tyvm | 15:15 |
*** vnogin has quit IRC | 15:20 | |
mnaser | https://wiki.openstack.org/wiki/Meetings/openstack-ansible#Meeting_section_.22Last_week_highlights.22 | 15:22 |
mnaser | please add stuff if you have things to bring up :-) | 15:22 |
openstackgerrit | Merged openstack/openstack-ansible stable/queens: Switch lxc_container_backing_store to default "dir" https://review.openstack.org/578079 | 15:23 |
cjloader_ | odyssey4me: can you possibly look at https://review.openstack.org/#/c/575504/? | 15:26 |
cjloader_ | or cloudnull | 15:27 |
*** kysse has quit IRC | 15:36 | |
odyssey4me | cjloader_: that patch is doing exactly the opposite of what we've been trying to achieve which is to *reduce* group_vars | 15:40 |
cjloader_ | we need it though | 15:40 |
*** koukou has quit IRC | 15:40 | |
cjloader_ | for Ironic | 15:41 |
openstackgerrit | Taseer Ahmed proposed openstack/openstack-ansible-os_blazar master: This patch ensures that the functional tests for Blazar pass. https://review.openstack.org/571388 | 15:42 |
*** throwsb1 has joined #openstack-ansible | 15:42 | |
*** pcaruana has quit IRC | 15:42 | |
*** kysse has joined #openstack-ansible | 15:42 | |
odyssey4me | cjloader_ prometheanfire there's been a *massive* amount of feedback in https://review.openstack.org/#/c/561277/ which will improve the role and make it work better.. can we expect to see a set of patches based on that feedback some time soon? Some of the feedback actually will help to resolve this particular issue | 15:44 |
*** cjloader_ has quit IRC | 15:45 | |
odyssey4me | cjloader_ take a look at the work Tahvok has been doing and it should provide insight into the preferred way of dealing with the requirement you have: https://review.openstack.org/#/q/topic:cleanup_vars+(status:open+OR+status:merged) | 15:46 |
*** cjloader has joined #openstack-ansible | 15:46 | |
*** kysse has quit IRC | 15:47 | |
odyssey4me | adding more group_vars increases the memory usage by ansible and decreases the performance, and this cycle that was identified as something to work on reducing where possible | 15:48 |
prometheanfire | odyssey4me: I can probably look at it today, I just don't like having to basically rewrite the role being dumped on us (even if it really needs it) | 15:48 |
*** cjloader has quit IRC | 15:48 | |
*** cjloader has joined #openstack-ansible | 15:48 | |
*** cjloader has quit IRC | 15:48 | |
*** cjloader has joined #openstack-ansible | 15:49 | |
odyssey4me | prometheanfire if the role is not actively maintained, then it becomes a lot of work when a contributor wants something done in it - better to actively maintain it over time if it matters to your needs, then the technical debt is spread out and contributions can be more agile... | 15:49 |
*** gyee has joined #openstack-ansible | 15:50 | |
*** chyka has joined #openstack-ansible | 15:52 | |
prometheanfire | odyssey4me: agreed | 15:52 |
prometheanfire | I just hate to be that first person, feels like my first neutron review | 15:52 |
*** lvdombrkr89 has quit IRC | 15:52 | |
Tahvok | Do we have a meeting today? | 15:53 |
odyssey4me | prometheanfire in terms of 'dumping it on you' - I could have done it myself in considerably less time than it took to write all those comments and guidance... my hope is that you can learn from it and become a more regular contributor... and writing the patches myself doesn't help that happen | 15:53 |
*** mbuil has quit IRC | 15:53 | |
prometheanfire | odyssey4me: I'm not mad at you about the 'dump' I do see it as a necessary thing, just unexpected work is all | 15:54 |
prometheanfire | not happy at the situation, but now that I have more time it should be good | 15:54 |
evrardjp | is there a problem? | 15:54 |
prometheanfire | and as for becoming more active, my focus may be more along the lines of stable-maint | 15:54 |
odyssey4me | prometheanfire: oh sure, unfortunately that's what happens when a body of code is not actively maintained - and that's an issue when the community around it is smaller | 15:54 |
prometheanfire | evrardjp: no | 15:54 |
prometheanfire | odyssey4me: yarp | 15:55 |
odyssey4me | Tahvok: yes, I think mnaser is running it... ? | 15:55 |
mnaser | yes i am | 15:55 |
Tahvok | evrardjp not with us today? | 15:55 |
Tahvok | I have only a few minutes.. | 15:56 |
evrardjp | Tahvok: hello | 15:56 |
evrardjp | Tahvok: what can I do? | 15:56 |
Tahvok | mnaser: maybe you can answer me here even before the meeting? https://bugs.launchpad.net/openstack-ansible/+bug/1776606 | 15:56 |
openstack | Launchpad bug 1776606 in openstack-ansible "qemu-kvm missing on new compute hosts" [Undecided,Incomplete] | 15:56 |
Tahvok | evrardjp: just say hello :) | 15:56 |
evrardjp | Tahvok: hello : ) | 15:57 |
mnaser | evrardjp: can you please help me and run the generate bugs thing if you have time? | 15:57 |
evrardjp | it's been quite a busy day for me : ) | 15:57 |
evrardjp | mnaser: sure | 15:57 |
mnaser | i'm trying to run it and.. struggling with pyenv and all sorts of other stuff | 15:57 |
mnaser | thank you so much, its a bit my fault being last minute | 15:57 |
prometheanfire | odyssey4me: one thing I've thought of to simplify configuration is to just create a map/dictionary that's matched against for the table here https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html | 15:57 |
mnaser | Tahvok: that's interesting. we might not be installing qemu-kvm | 15:58 |
mnaser | and i guess dependencies might have changed | 15:58 |
prometheanfire | odyssey4me: each row of that table seems to be a supported config, you can't mix and match things | 15:58 |
logan- | mnaser: i think we'll need to revive https://review.openstack.org/#/c/557484/ in light of the packethost mtu stuff | 15:58 |
mnaser | Tahvok: could you push up a patch adding qemu-kvm to that list? | 15:58 |
mnaser | massive highlight incoming | 15:59 |
mnaser | cloudnull, DimGR, andymccr, d34dh0r53, hughsaunders, b3rnard0, palendae, odyssey4me, serverascode, rromans, erikmwilson, mancdaz, _shaps_, BjoernT, claco, echiu, dstanek, jwagner, ayoung, prometheanfire, evrardjp, arbrandes, scarlisle, luckyinva, ntt, javeriak, spotz, vdo, jmccrory, alextricity25, jasondotstar, admin0, michaelgugino, ametts, bgmccollum, darrenc, JRobinson__, colinmcnamara, thorst, adreznec, eil397, | 15:59 |
mnaser | qwang,nishpatwa_, cathrichardson, drifterza, hwoarang, cshen, ullbeking, Tahvok, mnaser, nicolasbock | 15:59 |
mnaser | osa meeting in t-1m | 15:59 |
spotz | woot | 15:59 |
Tahvok | mnaser: yes, I'll create a review. | 15:59 |
mnaser | Tahvok: wonderful, please add me to it and ping me about it :) | 15:59 |
mnaser | #startmeeting openstack_ansible_meeting | 16:00 |
openstack | Meeting started Tue Jun 26 16:00:05 2018 UTC and is due to finish in 60 minutes. The chair is mnaser. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
*** hamzy has quit IRC | 16:00 | |
mnaser | #topic rollcall | 16:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
*** openstack changes topic to " (Meeting topic: openstack_ansible_meeting)" | 16:00 | |
openstack | The meeting name has been set to 'openstack_ansible_meeting' | 16:00 |
*** openstack changes topic to "rollcall (Meeting topic: openstack_ansible_meeting)" | 16:00 | |
mnaser | o/ | 16:00 |
openstackgerrit | Logan V proposed openstack/openstack-ansible master: Bootstrap lxc_net mtu for gate https://review.openstack.org/557484 | 16:00 |
odyssey4me | prometheanfire apologies, but I'm way out of context to be able to grok what you're talking about | 16:00 |
d34dh0r53 | o/ | 16:00 |
Tahvok | o/ for 5 minutes | 16:00 |
logan- | o/ | 16:00 |
mnaser | Tahvok: anything in particular you want us to start with if you'er around for 5? :-) | 16:00 |
prometheanfire | o/ | 16:00 |
ansmith | o/ | 16:00 |
Tahvok | mnaser: no, you've already answered to my question | 16:01 |
mnaser | wonderful | 16:01 |
evrardjp | o/ | 16:01 |
Tahvok | Thanks a lot btw.. | 16:01 |
mgariepy | o/ for 10-15 min | 16:01 |
mnaser | thank YOU :-) | 16:01 |
mnaser | #topic last week highlights | 16:01 |
*** openstack changes topic to "last week highlights (Meeting topic: openstack_ansible_meeting)" | 16:01 | |
evrardjp | mnaser: sorry for the delay, my venvs are busted. | 16:01 |
mnaser | evrardjp: np, we have sometime | 16:01 |
mnaser | so, updates on my side | 16:01 |
jmccrory | o/ | 16:01 |
mnaser | the big breakages removing epel have mostly been resolved | 16:01 |
mnaser | transition to rdo is pretty much complete, we use it in our jobs now entirely (yay) | 16:01 |
*** sew has joined #openstack-ansible | 16:01 | |
odyssey4me | w00t! | 16:02 |
mnaser | rdo pushed up uwsgi so distro jobs are working | 16:02 |
mnaser | which should help hwoarang efforts in adding distro support | 16:02 |
mnaser | ci jobs clean up is still in progress, heba from my side has patches up for all projects to update/add openstack-ansible-role-jobs project-template to make managing those jobs easier and more centralized | 16:02 |
hwoarang | o/ for 10 minutes | 16:02 |
mnaser | you can follow it here https://review.openstack.org/#/q/owner:%22Heba+Naser+%253Cheba%2540vexxhost.com%253E%22+is:open (we have been looking at unbreaking roles that are breaking out of that) | 16:03 |
mnaser | we merged a whole bunch of mirrors so our jobs should be more reliable (percona being a bad one). mariadb is wip with infra (mirrors are almost up) and that will be a big unreliable one which will be fixed :) | 16:03 |
*** gyee has quit IRC | 16:03 | |
mnaser | i broke the world here - https://review.openstack.org/#/c/578086/ -- so thanks to odyssey4me for fixing my mistakes :) | 16:03 |
mnaser | cloudnull's been fixing nspawn stuff, which is cool because we can start landing fixes in that side (another yay) | 16:04 |
mnaser | and finally, packethost was having issues regarding mtu/checksumming/etc. logan- seems to be digging into that (i totally dragged you into that one, sorry) | 16:04 |
mnaser | packethost has been disabled, so timeouts should be gone, but hopefully i think logan- was talking about reviving that | 16:04 |
cloudnull | o/ | 16:04 |
mnaser | now that's all for my monologue, anyone else has anything exciting they're working on the past week? | 16:04 |
* cloudnull is late but here :) | 16:05 | |
odyssey4me | thanks muchly both mnaser and logan- for climbing into that one | 16:05 |
hwoarang | what is packethost? | 16:05 |
mnaser | mostly logan-, i just did the complaining | 16:05 |
hwoarang | the cloud provider or something else? | 16:05 |
mnaser | hwoarang: packethost is a baremetal company provider who we have a cloud deployed on that was recently added to infra | 16:05 |
hwoarang | aha ok | 16:05 |
mnaser | https://www.packet.net/ | 16:05 |
hwoarang | thanks! | 16:05 |
logan- | yep we should be able to get it working with https://review.openstack.org/#/c/557484/ | 16:06 |
*** Taseer has quit IRC | 16:06 | |
cjloader | o/ | 16:07 |
logan- | the uplink interface has the correct mtu from dhcp, but the bridges we build always assume 1500, so the tldr is just set the lxc_net bridge to whatever the uplink is | 16:07 |
openstackgerrit | Albert Mikaelyan proposed openstack/openstack-ansible-os_nova master: Add qemu-kvm to package list for ubuntu-16.04 https://review.openstack.org/578140 | 16:07 |
mnaser | yeah i figured that was a reason behind why it occured | 16:07 |
Tahvok | mnaser: done ^ | 16:07 |
mnaser | Tahvok: thank you | 16:08 |
Tahvok | I'm out guys. Will try to move my tennis session, so I could be full hour with you next time | 16:08 |
odyssey4me | yep, seems sensible to me - it'll also help people with AIO's in other environments where the MTU is slightly lower than usual (private openstack clouds, for example) | 16:08 |
*** Taseer has joined #openstack-ansible | 16:09 | |
mnaser | https://review.openstack.org/#/c/576884/3 | 16:09 |
mnaser | can i have eyes on that, it's the patch below setting upgrade jobs to nv | 16:09 |
mnaser | so that will help push it into the gate so we can unblock gates | 16:09 |
mnaser | (while we wait for the bugs list to go up :p) | 16:09 |
*** udesale has quit IRC | 16:09 | |
mnaser | https://review.openstack.org/#/c/577885/3 would also be cool to prepare things for mariadb mirrors work | 16:10 |
evrardjp | mnaser: the bug list is up | 16:10 |
mnaser | sweet, thanks evrardjp. i'll do my homework on time next time. | 16:10 |
*** kysse has joined #openstack-ansible | 16:10 | |
mnaser | i'll move to bug triage but please any cores have a look at the two changes above so we can unblock our gates | 16:10 |
mnaser | #topic triage | 16:11 |
*** openstack changes topic to "triage (Meeting topic: openstack_ansible_meeting)" | 16:11 | |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778663 | 16:11 |
openstack | Launchpad bug 1778663 in openstack-ansible "Pike→Queens upgrade fails on final step in run_upgrade.sh (running haproxy-install.yml)" [Undecided,New] | 16:11 |
evrardjp | mnaser: it's my fault, my code isn't UTF8 ready and ^ bug causes issues in python3. | 16:12 |
mnaser | https://github.com/openstack/openstack-ansible/blob/stable/queens/scripts/run-upgrade.sh mentioned file | 16:12 |
odyssey4me | I think there's a patch for that already - hang a sec | 16:12 |
mnaser | oh its missing ${UPGRADE_PLAYBOOKS} | 16:12 |
mnaser | https://github.com/openstack/openstack-ansible/blob/stable/queens/scripts/run-upgrade.sh#L203 | 16:12 |
odyssey4me | https://review.openstack.org/569857 | 16:13 |
mnaser | wait no, that's not it | 16:13 |
mnaser | (at my comment) | 16:13 |
odyssey4me | oh, unless I'm making a broad assumption without reading :o | 16:13 |
mnaser | ahhhh | 16:13 |
mnaser | no i think that might be it | 16:13 |
evrardjp | I am not sure we should use tags there | 16:14 |
mnaser | evrardjp: user seems to report that everythign worked well when they did | 16:14 |
odyssey4me | tags aren't essential for that, but yeah - that patch fixes it | 16:14 |
odyssey4me | I can modify the commit to add the bug ref if that'd be good for everyone? | 16:14 |
evrardjp | https://github.com/openstack/openstack-ansible/commit/21739027606df272f8caff0a4b36f5d2bd82681b | 16:14 |
evrardjp | odyssey4me: yeah | 16:15 |
mnaser | odyssey4me: do that quickly and we'll +A | 16:15 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Upgrade task missing equals on tag https://review.openstack.org/569857 | 16:15 |
evrardjp | I think that's when the upgrade started to fail. And I can definitely it's during my "experiment month" | 16:15 |
odyssey4me | ok, done - although as I did it there were very many votes added :p | 16:16 |
mnaser | can i get another core to +A that ^ | 16:16 |
* mnaser pokes spotz again | 16:16 | |
mnaser | or that :p | 16:16 |
mnaser | ok cool | 16:16 |
odyssey4me | done already | 16:16 |
spotz | oh sure.... | 16:16 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778586 | 16:16 |
openstack | Launchpad bug 1778586 in openstack-ansible "aio_lxc fils on Leap 42.3" [Undecided,New] | 16:16 |
evrardjp | reviewed. | 16:16 |
mnaser | hmm | 16:17 |
mnaser | hwoarang: any idea about the above? ^ | 16:17 |
nicolasbock | I vote for hwoarang :) | 16:17 |
mnaser | evrardjp: do we bug you about suse stuff or not yet | 16:17 |
mnaser | :P | 16:17 |
evrardjp | haha. good question. | 16:17 |
nicolasbock | I have the env still up and can provide more details if necessary | 16:18 |
evrardjp | ofc you can : ) | 16:18 |
nicolasbock | I can also try to debug this myself, but I need someone to hold my hand ;) | 16:18 |
odyssey4me | hmm, looks like a package conflict unless I'm reading it wrong? | 16:18 |
mnaser | odyssey4me: yeah that's what i see it too | 16:19 |
mnaser | but our gates are ok for opensuse right now | 16:19 |
nicolasbock | Yes that's how I am reading it odyssey4me | 16:19 |
mnaser | so im not sure how the state changed | 16:19 |
evrardjp | master hasn't move that much since that patch | 16:19 |
mnaser | are you using any special mirrors or something | 16:19 |
nicolasbock | No | 16:19 |
mnaser | no corporate reverse or transparent proxy | 16:19 |
nicolasbock | I created a kvm vm and run the normal AIO stuff | 16:19 |
nicolasbock | No, no procies | 16:19 |
nicolasbock | proxies | 16:20 |
mnaser | ok | 16:20 |
mnaser | i don't really know enough about zypper and friends :( | 16:20 |
hwoarang | nicolasbock: can you put some repo info on the bug? | 16:20 |
evrardjp | I will run a new leap42.3 with vbox when my host is setup | 16:20 |
evrardjp | I will take it | 16:20 |
nicolasbock | Yes hwoarang | 16:20 |
evrardjp | wonder why sysstat is required though | 16:21 |
odyssey4me | it's likely best for nicolasbock, evrardjp and hwoarang to partner up to figure it out to confirm and triage... perhaps we leave current bug state and move on | 16:21 |
mnaser | cool, shall i assign to you evrardjp ? and it's nice nicolasbock is here too so we can do some more interactive debugging after the meeting (or whenever) | 16:21 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_octavia stable/queens: Adds the issuer to the CAs https://review.openstack.org/578118 | 16:21 |
odyssey4me | ? | 16:21 |
hwoarang | sure | 16:21 |
evrardjp | yup | 16:21 |
mnaser | odyssey4me: i agree to that, cool | 16:21 |
nicolasbock | Yes, sounds good | 16:21 |
evrardjp | the dream team in action | 16:21 |
odyssey4me | evrardjp: did you see what I did there? ;) | 16:21 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778537 | 16:21 |
openstack | Launchpad bug 1778537 in openstack-ansible "LXC bridge issue with networkd on CentOS 7" [Undecided,New] | 16:21 |
nicolasbock | I attached the repo info hwoarang | 16:21 |
*** tux__ has quit IRC | 16:22 | |
evrardjp | odyssey4me: assign more work to ppl, great! | 16:22 |
mnaser | cloudnull: has been doing some networkd fixes | 16:22 |
nicolasbock | evrardjp, :+1: | 16:22 |
mnaser | cloudnull: do you think the above might already be fixed? | 16:22 |
* cloudnull reading | 16:22 | |
odyssey4me | well, in this case the person did the prep themselves - they're asking for docs of known issues, which I think is fair | 16:22 |
cloudnull | that was a new issue tux was running into | 16:22 |
odyssey4me | I think tux is the reporter | 16:23 |
*** epalper has quit IRC | 16:23 | |
evrardjp | maybe we should have this networkd behavior optional, and distro based | 16:23 |
cloudnull | I'm not sure what the issue was in the environment. from what I can tell the networkd setup was done by hand largely following mhayden's blog post. | 16:23 |
mnaser | can we say this is confirmed and assign to cloudnull or was this resolved with the recent set of patches? | 16:23 |
logan- | would be interesting to see iptables-save output from the host, ip addr / networkctl status output from the container, and see if lxc-net dnsmasq is running on the host | 16:23 |
mnaser | or maybe incomplete and add what logan- is asking for? | 16:24 |
logan- | im thinking either dnsmasq isnt running or the nat rules is missing | 16:24 |
cloudnull | that gate seems to run fine with networkd as it stands today . | 16:24 |
evrardjp | or started at a wrong point | 16:24 |
odyssey4me | perhaps we need a user story which discusses prepping the hosts with networkd config and shares the things that need knowing? | 16:24 |
evrardjp | cloudnull: well we don't test what all deployers do -- rebooting things | 16:24 |
cloudnull | so im not certain there's anything specifically wrong with networkd and centos | 16:25 |
mnaser | that's a good point odyssey4me | 16:25 |
evrardjp | cloudnull: that's a gap in our testing we could cover in the future | 16:25 |
cloudnull | in tux's case it wasn't working at all, nevermind a reboot. | 16:25 |
evrardjp | odyssey4me: I am fine with the user story | 16:25 |
mnaser | i dont think any other deployment tool tests rebooting things so we have other fun things to catch up on before that :P | 16:25 |
evrardjp | because that's not part of the "osa" deploy itself | 16:25 |
cloudnull | it was something off with lxcbr0 and NAT | 16:26 |
odyssey4me | cloudnull: would you be up for writing up at least the skeleton of a user story given the networkd experience - I'm happy to make it pretty and have spotz make it prettier | 16:26 |
logan- | yeah ive seen issues with dnsmasq not starting/restarting properly on eni->networkd conversions in lxc_hosts, but never issues on greenfield, so it seems like this will need some more diag | 16:26 |
cloudnull | odyssey4me sure. | 16:26 |
evrardjp | logan-: agreed with you | 16:26 |
spotz | hehe | 16:26 |
evrardjp | it's not the first one. | 16:26 |
cloudnull | I have some tools running in the lab with an all networkd setup | 16:26 |
odyssey4me | ok, so assign to cloudnull for now - I'd say low but confirmed | 16:27 |
cloudnull | i can doc that | 16:27 |
cloudnull | ++ | 16:27 |
mnaser | yay so given the resolution for this is 'soft notes', can i assign to cloudnull ? | 16:27 |
mnaser | cool | 16:27 |
evrardjp | yeah sounds nice | 16:27 |
mnaser | done! | 16:27 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778463 | 16:27 |
openstack | Launchpad bug 1778463 in openstack-ansible "[magnum][pike] Image upload in playbook does not work" [Undecided,New] | 16:27 |
mnaser | ahhhhhhhhh | 16:27 |
mnaser | i remember this | 16:27 |
mnaser | i pushed up a patch | 16:27 |
*** cshen has joined #openstack-ansible | 16:27 | |
evrardjp | ok great | 16:27 |
evrardjp | that sounds an easy fix | 16:28 |
mnaser | let me find it | 16:28 |
mnaser | https://review.openstack.org/#/c/543256/ | 16:28 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible stable/queens: Run openstack_openrc before Magnum installation https://review.openstack.org/578147 | 16:28 |
mnaser | backport to stable/queens | 16:28 |
odyssey4me | alrighty, so note the link to the change and assign to you? | 16:29 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible stable/queens: Run openstack_openrc before Magnum installation https://review.openstack.org/578147 | 16:29 |
mnaser | odyssey4me: if you don't mind i added a closes-bug | 16:29 |
mnaser | ill take the +2 back and yep assigned to me | 16:30 |
evrardjp | for the last part: https://github.com/openstack/openstack-ansible/blob/feef46a4b4af249fd3d2a48d1fad9f248d9b82e8/playbooks/os-keystone-install.yml#L22-L29 | 16:30 |
mnaser | juggling workflows lol | 16:30 |
evrardjp | odyssey4me: did you finish https://github.com/openstack/openstack-ansible/commit/d8fcd1ae5378ab623b4ca02d10037514dba97e03 ? | 16:30 |
evrardjp | applying to magnum too | 16:30 |
odyssey4me | sure, although I don't think jeepyb will do things to the ticker given that the review is targeted to stable/queens | 16:30 |
odyssey4me | it would do things to the bug if there was a series assignment, but OSA hasn't used those since newton IIRC | 16:31 |
spotz | odyssey4me: Ok that made no sense to me:) | 16:31 |
spotz | the one before newton about the ticker | 16:31 |
mnaser | hmm i see | 16:31 |
evrardjp | odyssey4me: nevermind it's defaulted | 16:31 |
mnaser | are we good on this one? | 16:32 |
evrardjp | with backports yes. | 16:32 |
odyssey4me | evrardjp: not just yet - need to figure out how to get https://review.openstack.org/568142 fixed up - then will do other roles | 16:32 |
mnaser | cools | 16:32 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778412 | 16:32 |
openstack | Launchpad bug 1778412 in openstack-ansible "Create OpenStack-Ansible requirement wheels" [Undecided,New] | 16:32 |
odyssey4me | spotz: will explain after | 16:32 |
spotz | odyssey4me: thanks | 16:33 |
mnaser | user says | 16:33 |
mnaser | "Please close this one, i think i don't have permission to close." | 16:33 |
evrardjp | will check if that's a real dupe | 16:33 |
odyssey4me | yay, free bug close points | 16:33 |
mnaser | looks like that was related to mismatching libvirt | 16:33 |
mnaser | which is something we've already resolved moving forward | 16:33 |
evrardjp | yay | 16:33 |
mnaser | what do we put as cancelled | 16:34 |
mnaser | invalid? | 16:34 |
prometheanfire | we've been looking at updating the requirement for libvirt-python on stable releases | 16:34 |
evrardjp | mnaser: it's a real duplicate | 16:34 |
evrardjp | mnaser: so clicking on top right | 16:34 |
mnaser | oh mark as dup | 16:34 |
evrardjp | then 1730314 | 16:34 |
prometheanfire | distros seem to update libvirt not like other packages (which they pin more or less) | 16:34 |
odyssey4me | hmm, I *think* we got this figured out | 16:34 |
evrardjp | then save | 16:34 |
*** kysse has quit IRC | 16:34 | |
odyssey4me | my memory is rusty | 16:34 |
evrardjp | odyssey4me: ? | 16:35 |
odyssey4me | IIRC this was a CentOS issue, and we resolved this by switching to symlinking the host libvirt-pyton into the venv, right? | 16:35 |
mnaser | prometheanfire: yeah afaik centos just ignores what the constraints say.. | 16:35 |
evrardjp | we are marking it as duplicate | 16:35 |
mnaser | their comment: "most of our team builds libvirt, we know what we're doing" *shrugs* | 16:35 |
odyssey4me | this was the whole issue around centos using a different libvirt to everyone else | 16:35 |
mnaser | Bug 1730314 is already a duplicate of bug 1636567. You can only mark a bug report as duplicate of one that isn't a duplicate itself. | 16:36 |
openstack | bug 1636567 in openstack-ansible "duplicate for #1730314 devstack mitaka installation fails with error "Running setup.py bdist_wheel for libvirt-python: finished with status 'error'" in Ubuntu 16.10" [High,Confirmed] https://launchpad.net/bugs/1636567 | 16:36 |
openstack | bug 1636567 in openstack-ansible "devstack mitaka installation fails with error "Running setup.py bdist_wheel for libvirt-python: finished with status 'error'" in Ubuntu 16.10" [High,Confirmed] https://launchpad.net/bugs/1636567 | 16:36 |
*** kysse has joined #openstack-ansible | 16:36 | |
odyssey4me | haha | 16:36 |
evrardjp | mnaser: yeah you might have to track : ) | 16:36 |
mnaser | dependency solving yay | 16:36 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778098 | 16:36 |
openstack | Launchpad bug 1778098 in openstack-ansible "os_horizon role fails if horizon_custom_themes is specified" [Undecided,New] | 16:36 |
prometheanfire | mnaser: yep, it was a cent issue | 16:36 |
mnaser | it looks like we somehow already have a fix for this | 16:37 |
mnaser | hmm this is weird | 16:37 |
mnaser | so if you configure a theme, it expects it to be uploaded | 16:38 |
evrardjp | that is documented | 16:38 |
mnaser | o | 16:38 |
evrardjp | just a sec | 16:38 |
mnaser | cool | 16:38 |
evrardjp | yeah the docs is bad | 16:39 |
evrardjp | https://docs.openstack.org/openstack-ansible-os_horizon/latest/ | 16:39 |
evrardjp | in default | 16:39 |
evrardjp | it should be a more explicit thing | 16:39 |
*** sew has quit IRC | 16:39 | |
evrardjp | spotz: could you deal with this? | 16:39 |
spotz | evrardjp: yeah | 16:39 |
prometheanfire | would updating the constraint in requrements help? | 16:40 |
evrardjp | basically writing a story about how to use a custom theme | 16:40 |
evrardjp | it should be based on https://docs.openstack.org/openstack-ansible-os_horizon/latest/ default's explanation, mixed with the user's input of the bug above | 16:40 |
evrardjp | explaining another variable is required | 16:40 |
evrardjp | it can be done in two steps: changing the defaults/main.yml to be more explicit, but also adding a complete user story | 16:40 |
evrardjp | cool thanks | 16:41 |
mnaser | spotz: are you cool with taking this on? :p | 16:42 |
odyssey4me | apologies - took a little while to track down the patches which solved https://launchpad.net/bugs/1636567 | 16:42 |
openstack | Launchpad bug 1636567 in openstack-ansible "devstack mitaka installation fails with error "Running setup.py bdist_wheel for libvirt-python: finished with status 'error'" in Ubuntu 16.10" [High,Fix released] - Assigned to Jesse Pretorius (jesse-pretorius) | 16:42 |
spotz | mnaser: Yeah, you know me I'll ask questions:) | 16:42 |
*** sew has joined #openstack-ansible | 16:42 | |
mnaser | ++ | 16:42 |
mnaser | spotz: marking as confirmed, low and assigning to you :) | 16:43 |
spotz | mnaser: okie:) | 16:43 |
mnaser | last one | 16:43 |
mnaser | #link https://bugs.launchpad.net/openstack-ansible/+bug/1778085 | 16:43 |
openstack | Launchpad bug 1778085 in openstack-ansible "Horizon health check fails when developer panels enabled" [Undecided,New] | 16:43 |
mnaser | ooou a regression perhaps | 16:44 |
evrardjp | hahah I KNEW it. | 16:44 |
evrardjp | I suggest we assign that to cloudnull who did the first patch with the renames :p | 16:45 |
mnaser | the workaround/fix seems relatively reasonable | 16:45 |
odyssey4me | lol | 16:45 |
mnaser | is that an ok fix? | 16:45 |
evrardjp | mnaser: yes | 16:45 |
mnaser | if it is, it's a matter of just pushing the patch | 16:45 |
odyssey4me | man, those were done in mitaka - I'm surprised it's taken this long to surface | 16:45 |
cloudnull | https://review.openstack.org/#/c/573318/ | 16:45 |
cloudnull | maybe that helps? | 16:45 |
evrardjp | damn you really want to install elasticsearch pip package everywhere | 16:46 |
jrosser | cloudnull: that exposes the bug | 16:46 |
mnaser | i wonder why that wasn't caught in ci | 16:47 |
evrardjp | mnaser: different config : ) | 16:47 |
odyssey4me | I must say, installing the elasticsearch package everywhere by default makes me uncomfortable - no matter how useful it is, especially because the version override's also being done | 16:47 |
jrosser | is it intalled by default? | 16:48 |
jrosser | *installed | 16:48 |
evrardjp | that way it will. | 16:48 |
odyssey4me | I'd really that was more of an opt-in, personally... but the operators should really be chiming in. | 16:48 |
mnaser | if it doesn't hurt, i can't imagine it being too much of an issue. the long term resolution would be adding some sort of opt-in thing | 16:48 |
evrardjp | mnaser: it can hurt for packagers | 16:48 |
mnaser | but i dont want to burden contributions that don't hurt things personally | 16:48 |
mnaser | sure, that's why it can be opt in and supported in certain types of deployment | 16:49 |
odyssey4me | https://review.openstack.org/#/q/topic:osprofiler+(status:open+OR+status:merged) looks to me like it went with adding by default, not opt-in | 16:49 |
evrardjp | mnaser: I agree there. | 16:49 |
evrardjp | odyssey4me: I asked for opt-in. | 16:49 |
cloudnull | evrardjp https://review.openstack.org/#/c/573318/10/defaults/main.yml - that package is required | 16:49 |
evrardjp | I didn't -2. | 16:49 |
mnaser | we don't have a huge team and we shouldn't start being super strict about these things that do "good" things at the end of teh day | 16:49 |
odyssey4me | sure | 16:49 |
cloudnull | the driver we added is for elk, | 16:49 |
evrardjp | cloudnull: ok I will -2 it then. | 16:50 |
cloudnull | without it it wont do anything | 16:50 |
odyssey4me | is it time that we have a generic 'I want these extra packages in every venv' variable that's used in all roles? | 16:50 |
evrardjp | I am sad we arrive to that place | 16:50 |
evrardjp | opt-in was the deal | 16:50 |
evrardjp | that's no opt-in | 16:50 |
cloudnull | it has to be there for horizon | 16:50 |
jrosser | err https://review.openstack.org/#/c/573526/3/defaults/main.yml what am i missing? | 16:50 |
mnaser | ok | 16:50 |
mnaser | i'll ask that we take a step back first | 16:50 |
jrosser | i thought those osprofiler patches all got changed to remove the elasticsearch pip install | 16:50 |
mnaser | can someone push `horizon_server_name` change first | 16:51 |
mnaser | which is for https://bugs.launchpad.net/openstack-ansible/+bug/1778085 | 16:51 |
openstack | Launchpad bug 1778085 in openstack-ansible "Horizon health check fails when developer panels enabled" [Undecided,New] | 16:51 |
evrardjp | jrosser: same. Which was okay for me | 16:51 |
odyssey4me | mnaser: well, the reporter asked if that was a suitable fix - we can simply respond affirmatively | 16:51 |
cloudnull | it did. all other roles are not adding the elasticsearch plugin. | 16:51 |
mnaser | odyssey4me: ill push it up quickly :) | 16:51 |
odyssey4me | perhaps the reporter will submit the patch, and I think we should provide that opportunity | 16:52 |
cloudnull | however the driver we built for horizon is elasticsearch specific | 16:52 |
stuartgr1 | I reported the bug, I can submit a patch | 16:52 |
evrardjp | cloudnull: I'd prefer if we write a user story about how this gets done. Because the implications are bigger than what you see. | 16:52 |
odyssey4me | let's leave it as a new bug (so it's discussed again next week), but respond to the reporter | 16:52 |
odyssey4me | ah, and there we have stuartgr1 :) | 16:52 |
cloudnull | evrardjp I'd love to know more about what those implications are? | 16:52 |
openstackgerrit | Mohammed Naser proposed openstack/openstack-ansible-os_horizon master: Switch to using ansible_fqdn in horizon_server_name https://review.openstack.org/578151 | 16:53 |
mnaser | odyssey4me: oops | 16:53 |
mnaser | sorry | 16:53 |
mnaser | sorry stuartgr1 :< | 16:53 |
odyssey4me | mnaser abandon :) let stuartgr1 go ahead | 16:53 |
evrardjp | odyssey4me: that's a good idea | 16:53 |
evrardjp | cloudnull: ofc | 16:53 |
mnaser | stuartgr1: it's all yours :) | 16:53 |
stuartgr1 | ok, thanks | 16:53 |
mnaser | please let me know if you need help submitting the patch | 16:53 |
odyssey4me | stuartgr1 thanks for asking in the bug, very often just pushing up a review with the bug reference would stimulate a faster response :) | 16:54 |
mnaser | #topic open discussion | 16:54 |
*** openstack changes topic to "open discussion (Meeting topic: openstack_ansible_meeting)" | 16:54 | |
mnaser | odyssey4me: ++ | 16:54 |
mnaser | do we want to discuss the issue regarding elasticsearch in os_horizon | 16:54 |
mnaser | it looks like cloudnull agreed to drop it in all venvs except for horizon where it seems to be a requirement | 16:54 |
prometheanfire | the ironic role needs to be overhauled | 16:54 |
odyssey4me | evrardjp cloudnull if we're ok with going with elasticsearch as the 'opinionated' way of doing profiling, then no opt-in is fine... adding opt-in sometimes adds more code for little benefit | 16:55 |
mnaser | i think cloudnull did a fairly ok compromise by dropping it but having it in one role isn't that huge of a deal. :x | 16:55 |
mnaser | prometheanfire: what type of work does it need right now? | 16:55 |
odyssey4me | well, fine IMO :) | 16:55 |
prometheanfire | it's mainly in the hardware type stuff https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html | 16:55 |
prometheanfire | my first pass was at https://review.openstack.org/#/c/561277/ | 16:55 |
mnaser | wow that's a big document | 16:56 |
mnaser | lol | 16:56 |
prometheanfire | my idea was to turn each classic driver into a key, with a list containing the rest of the values (hardware type, boot, deploy, inspect, management, power) as the value | 16:56 |
evrardjp | odyssey4me: I disagree with you on the elasticsearch part, even if opinionated. | 16:56 |
odyssey4me | much of the overhaul is provided in detail in that review and has nothing to do with the hardware/ironic changes - it's just role code style and simplification to make maint easier | 16:56 |
cjloader | +1 prometheanfire ironic role does need overhauled | 16:57 |
mnaser | odyssey4me: do we feel like that's an extra thing that prometheanfire has to deal with | 16:57 |
mnaser | as in: does it block his work? :x | 16:57 |
odyssey4me | the ironic role as it stands today is a mitaka implementation which was overly complex and has had no maint except that which was required | 16:57 |
prometheanfire | odyssey4me: sure, I was mainly seeking clarification about my plan to use that key/value(list) method for setting the keys | 16:57 |
prometheanfire | mnaser: I'm more or less the 'ironic guy' here :P | 16:58 |
mnaser | i'm all for improving things but it would be a bummer if prometheanfire gets stuck with having to clean it all up just to land their patch | 16:58 |
mnaser | (i know we'll be over time, but it seems like we got some stuff to clear out) | 16:58 |
odyssey4me | prometheanfire: yep, I think using a key:value mechanism was the suggested approach - following what's been done in nova/neutron/etc | 16:58 |
prometheanfire | mnaser: there is that, but I'm willing to do the work | 16:58 |
mnaser | oh cool, i'm happy to hear that | 16:58 |
prometheanfire | odyssey4me: cool, that's the main guidance I was looking for | 16:58 |
mnaser | i'm happy to help with reviews. | 16:58 |
odyssey4me | mnaser: unfortunately the role has reached a state where it can't pass tests, so it needs the overhaul to even get a pass | 16:59 |
mnaser | odyssey4me: ugh, that's a bit of a bummer | 16:59 |
mnaser | i can have a look and hack on a thing or two, but that would be on personal time (probably weekends) | 16:59 |
evrardjp | for the elasticsearch profiling, I think it's a very good user story, and we should explain how it's done anyway. We just give an extra override for horizon. Until it's made opt-in in the role and needs a simple boolean flip | 16:59 |
odyssey4me | overhauling the structure first, then implementing the deprecation changes would be a good approach - so that the overhauls can be back ported | 16:59 |
evrardjp | that would be nice | 17:00 |
evrardjp | but hard to do | 17:00 |
evrardjp | overhauling backport is something we need to accept as a community too | 17:00 |
mnaser | so hear me out here | 17:00 |
odyssey4me | prometheanfire: I'd even suggest disabling voting for the functional testing for the overhaul parts, then building up a chain of patches which does the overhaul in small bits and the updating last which includes turnig voting back on. | 17:00 |
mnaser | do we really need to backport the overhauls | 17:00 |
evrardjp | no | 17:00 |
mnaser | if it wasn't working in old releases, do we expect users to show up now and use it | 17:00 |
evrardjp | that's what I meant as "accept as a community too" | 17:01 |
mnaser | maybe that can reduce the workload on prometheanfire (at the expense of a new user who tries to deploy newton ironic and that not working) | 17:01 |
odyssey4me | mnaser in order to enable back porting other work to make ironic work in queens/pike/etc, yes - right now the role is unmaintainable | 17:01 |
evrardjp | I think we don't have to actively backport it. | 17:01 |
prometheanfire | odyssey4me: it's depricated in queens, not rocky | 17:01 |
prometheanfire | so the change over has to be backported | 17:01 |
mnaser | oh i see | 17:01 |
prometheanfire | at least that's what I've seen, I probably need to reconfirm that | 17:01 |
mnaser | this is a tough one. i think we just have to do the hard/tough job of working on fixing that | 17:02 |
prometheanfire | mnaser: yep | 17:02 |
evrardjp | yup it seems to | 17:02 |
odyssey4me | mnaser: specifically newton/pike/queens onwards matters to RAX... newton is fine as far as I know, pike+ is broken | 17:02 |
prometheanfire | the good part about it being unmaintianed is that it should be easier to backport :P | 17:02 |
prometheanfire | pike was fine | 17:02 |
evrardjp | odyssey4me: prometheanfire mnaser I think that's the important part | 17:02 |
prometheanfire | queens works with cjloader's patches | 17:02 |
evrardjp | if it's important for RAX, as member of the community, it can then bring the patches | 17:03 |
cjloader | I can help prometheanfire with the overhaul | 17:03 |
evrardjp | we'll vote for it, as any member of the community | 17:03 |
odyssey4me | prometheanfire: ok, that's good news | 17:03 |
mnaser | i know this might be unrelated but | 17:03 |
mnaser | i hate asking this but prometheanfire are you currently with rax? | 17:03 |
evrardjp | that's great to see ppl at rax contributing to ironic : ) | 17:03 |
prometheanfire | I'm done with the topic unless others have questions | 17:03 |
prometheanfire | mnaser: yarp | 17:03 |
evrardjp | or at least os_ironic : ) | 17:03 |
mnaser | ok cool! | 17:04 |
evrardjp | cjloader: great | 17:04 |
mnaser | so the backport best interest is in yours too | 17:04 |
mnaser | didnt want to have a load on you for something that you don't necessarily need :P | 17:04 |
mnaser | now | 17:04 |
mnaser | one more topic | 17:04 |
ansmith | good progress on qdrouterd role, looking to land base job if someone can take a look, https://review.openstack.org/#/c/575505/ | 17:04 |
mnaser | i see a happy xenial job, yay | 17:04 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Add repo_build pip cache https://review.openstack.org/576884 | 17:04 |
odyssey4me | prometheanfire thanks for taking it on, and apologies for the burden, but trying to review any patches to ironic is a minefield and the overhaul will resolve that | 17:05 |
mnaser | i'll have a look shortly. | 17:05 |
ansmith | mnaser: thanks! | 17:05 |
mnaser | elasticsearch and horizon. i think cloudnull made an appropriate comprimise in dropping it from all roles | 17:05 |
evrardjp | mnaser: odyssey4me we don't need a happy xenial job for this | 17:05 |
mnaser | except horizon which it does not seem to be possible | 17:05 |
evrardjp | ansmith: * | 17:05 |
prometheanfire | odyssey4me: si | 17:05 |
mnaser | i don't think it's something that warrants a -2 at this point | 17:05 |
evrardjp | mnaser: I see it as an obstacle for some deployers. We don't want to have obstacles :) | 17:06 |
mnaser | we're not in a state where we have full time contributors that we can ask for overhauls and adding opt-in systems for a contribution that is largely more of a part-time thing | 17:06 |
odyssey4me | prometheanfire feel free to ping me if you're stuck, I'm happy to partner up to get it to a better place | 17:06 |
mnaser | i don't see how it is an obstacle, an extra library in a venv | 17:06 |
mnaser | (with my deployer hat on, i don't even KNOW what libraries go in the venv) | 17:06 |
mnaser | if it works, it works | 17:06 |
cjloader | prometheanfire: same, I'd be happy to help | 17:06 |
cloudnull | evrardjp what obstacle? | 17:07 |
evrardjp | mnaser: it could be licensing. | 17:07 |
cloudnull | its already part of global requirements | 17:07 |
cloudnull | there is no licensing | 17:07 |
evrardjp | cloudnull: I have noted my comments on the bug | 17:07 |
prometheanfire | odyssey4me: sure :D | 17:07 |
mnaser | if its part of global requirements then it cant have license problems. it's also apache-2.0 licensed so it's okay | 17:07 |
cloudnull | ^ | 17:08 |
openstackgerrit | Merged openstack/openstack-ansible-tests master: Switch upgrade jobs to be non-voting https://review.openstack.org/577884 | 17:08 |
mnaser | a -2 of a core is kindof an effective veto of "i will refuse to merge this" | 17:08 |
mnaser | not "i disagree with it" | 17:08 |
cloudnull | if this is the stance we're going to take then we need to do a significant audit of the packages we install in the venvs. | 17:08 |
evrardjp | cloudnull: I think we should do it indeed. | 17:09 |
odyssey4me | my concern is more around us overriding the version - putting us out of sync with g-r/u-c... but if the project's contributors are happy to take the risk that come with that, it's fine to me | 17:09 |
cloudnull | we should also probably emove things like NFS from glance, and non-reference drivers from neutron. | 17:09 |
evrardjp | cloudnull: this work of cleanup was already started this cycle | 17:09 |
mnaser | well, doing that in theory sure sounds like adding more pain for operators.. | 17:09 |
evrardjp | we said we're gonna for new features next cycles, and clean this cycle. This doesn't seem aligned either | 17:09 |
mnaser | things happen. resources disappear. times disappears. | 17:09 |
mnaser | we've discussed this, we aren't all full time anymore so we have to understand that we need to operate in a different way | 17:10 |
mnaser | this can be handled differently if we all work full time on OSA, very easy to dedicate the time to clear it all out | 17:10 |
evrardjp | cloudnull: I will work on removing the non-reference drivers. That's sadly what I will have to do I think. | 17:10 |
mnaser | but given a lot of us are doing this 'on the side', we need to be a bit more flexible to help continue the project to progress | 17:10 |
evrardjp | mnaser: I think you're right on the flexibility | 17:10 |
mnaser | working the non reference driver makes me sad because we have users like OPNFV which uses ODL to test things | 17:10 |
odyssey4me | well, perhaps we need to be more opinionated | 17:10 |
mnaser | and i was going to start pushing up some work to support opencontrail as an optoin | 17:11 |
evrardjp | mnaser: but here, there is nothing that can't be done with a user story | 17:11 |
odyssey4me | we carry a very flexible framework, but more options means more work to maintain it | 17:11 |
mnaser | because of customer demand | 17:11 |
cloudnull | evrardjp I think removing the non-reference drivers would be a waist of time, on top of the fact it'd be detrimental for adoption. | 17:11 |
mnaser | odyssey4me: thing is, we don't have to test and support them all. we can still test the base scenario, but let's not take away the out-of-the-box experience | 17:11 |
mnaser | OPNFV folks *love* OSA and use it for all their deployments, we'd hurt them if we took away this stuff. | 17:11 |
odyssey4me | we ideally need to try and implement facilities instead of specific things, as that means you get the basics and can plug other things in | 17:11 |
cloudnull | what mnaser said, | 17:12 |
evrardjp | cloudnull: it has a dobule effect: For me it shows the aptitude of having 3rd party extensions that can be written by everyone | 17:12 |
evrardjp | mnaser: don't get me wrong, I don't want these to be removed | 17:12 |
mnaser | i just want us to scope back and look at the specific problem here which is: installing elasticsearch in the horizon venv. | 17:12 |
logan- | most if not all non reference drivers are installed optionally so its not an apples to apples comparison | 17:12 |
evrardjp | I want to make sure it's part of a way to behave with 3rd party ways if necessary | 17:12 |
odyssey4me | I mean for this specific case, wouldn't it be better to be able to set a flag to enable profiling, and have a var which sets which packages you want added to the venv? Instead of a hard-cded set of packages which you then have to remove if you don't want that thing? | 17:13 |
evrardjp | agreed with logan- too : p | 17:13 |
evrardjp | odyssey4me: agreed with you too. | 17:13 |
d34dh0r53 | odyssey4me: ++, that's what I was thinking | 17:13 |
mnaser | odyssey4me: i think evrardjp is ok with this, i'm just trying to not add extra workload on someone whos adding a productive useful change for 'optimizing' | 17:13 |
evrardjp | I think we should have a profiling variable that can be flipped in group vars if necessary | 17:13 |
mnaser | kinda like "Hey i showed up to help with this thing but instead ended up with a giant load of extra optimizations and requests" | 17:14 |
mnaser | which again, totally cool if we're full time, but we're not, and so by doing that, that feature might just never land. | 17:14 |
evrardjp | well do you prefer land a hurtful feature for some ppl, helping some others, OR not merging? | 17:14 |
odyssey4me | blast - I typed a bunch and pushed a bad key and lost it all | 17:15 |
evrardjp | I'd prefer the patch to be okay for everyone | 17:15 |
cloudnull | evrardjp what is this hurting ? | 17:15 |
mnaser | dont see how it's hurtful | 17:15 |
cloudnull | odyssey4me i hate it when i do that :) | 17:15 |
mnaser | it's even part of openstack requirements so packagers won't have too bad of a time | 17:15 |
mnaser | odyssey4me: so frustrating makes you even give up having to rewrite things sometimes lol | 17:15 |
odyssey4me | basically I suggest a more generic facility for each role - something like neutron_extra_optional_packages | 17:16 |
odyssey4me | that can then be re-used for *anything* a deployer wants without having to override the ddefault package list as they have to today | 17:16 |
evrardjp | that's exactly what I suggested odyssey4me | 17:16 |
mnaser | odyssey4me: i agree, but who's gonna do that work... | 17:16 |
mnaser | cloudnull: do you mind posting this on the ML so that we can have a discussion where we can kinda think about stuff and formulate answers | 17:17 |
*** tux_ has joined #openstack-ansible | 17:17 | |
logan- | it doesn't have to be implemented everywhere at once. just implement it in horizon for know so the pattern is established and can be implemented elsewhere as needed | 17:17 |
logan- | for now* | 17:17 |
mnaser | just to kinda say "hey operators, what do you think about having elasticsearch in here" | 17:17 |
odyssey4me | it'd be massively easier to do after moving to using the common venv build role, and I can implement something like that as a follow on - but there're no guarantees that'll make the Rocky release | 17:17 |
mnaser | and "hey cores, what are your thoughts about that?" | 17:17 |
d34dh0r53 | odyssey4me: I can help implementing it | 17:18 |
logan- | it basically already exists here: https://github.com/openstack/ansible-hardening/blob/1fd694a40c4d4013a2d407f043df99f3fc0e9e47/vars/redhat.yml#L63-L69 | 17:18 |
cloudnull | odyssey4me I added a way to inject any such package into the venv create and repo build role - https://review.openstack.org/#/c/574544/ https://review.openstack.org/#/c/574546/ | 17:18 |
odyssey4me | for now it'd be a bit of a slog, but an easy common pattern | 17:18 |
logan- | except for distro packages. but that pattern could be easily adopted for extra pip packages | 17:18 |
d34dh0r53 | I'm about to knock two things off my plate this week so I'll have some cycles | 17:18 |
mnaser | oh that's awesome | 17:19 |
odyssey4me | cloudnull: hmm, lemme look at those - that could be a good interim solution | 17:20 |
*** gyee has joined #openstack-ansible | 17:20 | |
odyssey4me | I'm not fond of taking intelligence out of the roles again, but if we can implement something that keeps it in the roles and uses another facility then that's cool | 17:20 |
odyssey4me | or at least is easy to transition later | 17:20 |
mnaser | ok so conclusion to this: cloudnull maybe post to ML to discuss this, d34dh0r53 to help add 'opt-in' pattern? | 17:20 |
cloudnull | however with horizon, because the config exists in python, if the lib does not exist and osprofiler is enabled, horizon breaks in terrible, cryptic ways. | 17:20 |
cloudnull | so I guess we could cause it to fail if the lib is not found in the venv and elasticsearch is present in the connection string? | 17:21 |
evrardjp | cloudnull: nothing a doc can't fix, as we need a doc anyway | 17:21 |
tux_ | cloudnull: hey you back!!! how do i run try ansible on openstack ? | 17:21 |
tux_ | i want to make changes and see if its going to right place using dry tun | 17:22 |
tux_ | run* | 17:22 |
odyssey4me | I'm happy to look at that horizon review more closely when I have a fresh head. Right now it's the end of the day. | 17:22 |
mnaser | tux_: we're in the middle of a meeting, should be able to talk soon about that :) | 17:22 |
tux_ | :) enjoy! sorry | 17:22 |
odyssey4me | I suspect we can find a happy medium | 17:22 |
mnaser | ++ | 17:22 |
odyssey4me | what's the review again? | 17:22 |
mnaser | https://review.openstack.org/#/c/573318/ | 17:22 |
mnaser | primarily https://review.openstack.org/#/c/573318/10/defaults/main.yml | 17:23 |
cloudnull | evrardjp a doc can't fix this issue. | 17:23 |
cloudnull | the lib has to be present at venv buld time. | 17:23 |
mnaser | fwiw all other projects got 'osprofiler' except this one is the one 'odd' case which i'm okay with making an exception | 17:23 |
cloudnull | we dont run pip install when the venv is available on teh repo server. | 17:23 |
odyssey4me | cloudnull at a glance, could we have something like an 'optional' set of packages (much like neutron does) which puts those packages there at runtime if that option is enabled? | 17:24 |
evrardjp | cloudnull: that can be in /etc/openstack_deploy like all the rest of the user_variables. | 17:24 |
cloudnull | evrardjp so the user has to override the entire package list / | 17:24 |
mnaser | can we please move this to a mailing list discussion, as i think it's more productive there? | 17:24 |
evrardjp | omg that could be fixed by using a temp variables in vars/ | 17:24 |
evrardjp | that's nothing | 17:24 |
mnaser | i'd like to wrap up the meeting | 17:25 |
evrardjp | convenience has an end | 17:25 |
cloudnull | odyssey4me yes. though that's a lot of extra tasks to accomplish something extreamly simple, with no known impact. | 17:25 |
prometheanfire | I'd like to eat | 17:25 |
mnaser | please? :) | 17:25 |
evrardjp | yeah | 17:25 |
evrardjp | thanks for chairing | 17:25 |
mnaser | if not, i'll post a message | 17:25 |
mnaser | quoting the meeting | 17:25 |
cloudnull | mnaser sure. | 17:25 |
mnaser | and people can read | 17:25 |
mnaser | yay | 17:25 |
mnaser | thanks | 17:25 |
mnaser | that's much better | 17:25 |
evrardjp | have a good day. | 17:25 |
mnaser | THANK YOU EVERYONE :) | 17:25 |
mnaser | #endmeeting | 17:26 |
*** openstack changes topic to "Launchpad: https://launchpad.net/openstack-ansible || Weekly Meetings: https://wiki.openstack.org/wiki/Meetings/openstack-ansible || Review Dashboard: https://goo.gl/tTmdgs" | 17:26 | |
openstack | Meeting ended Tue Jun 26 17:26:00 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:26 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-06-26-16.00.html | 17:26 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-06-26-16.00.txt | 17:26 |
openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_ansible_meeting/2018/openstack_ansible_meeting.2018-06-26-16.00.log.html | 17:26 |
mnaser | prometheanfire: enjoy your meal | 17:26 |
prometheanfire | :D | 17:26 |
*** vurtz has joined #openstack-ansible | 17:39 | |
*** hamzy has joined #openstack-ansible | 17:50 | |
openstackgerrit | David Wilde proposed openstack/openstack-ansible-ops master: Improve multi-node AIO robustness https://review.openstack.org/578162 | 17:52 |
*** electrofelix has quit IRC | 17:52 | |
*** cjloader_ has joined #openstack-ansible | 17:54 | |
*** cjloader has quit IRC | 17:54 | |
*** cjloader_ has quit IRC | 17:54 | |
*** cjloader has joined #openstack-ansible | 17:54 | |
*** cshen has quit IRC | 17:56 | |
*** cjloader has quit IRC | 18:01 | |
*** cjloader has joined #openstack-ansible | 18:05 | |
*** cjloader has quit IRC | 18:05 | |
*** cjloader has joined #openstack-ansible | 18:05 | |
*** poopcat has joined #openstack-ansible | 18:07 | |
*** chhavi__ has quit IRC | 18:37 | |
*** sew has quit IRC | 18:49 | |
*** sew has joined #openstack-ansible | 18:54 | |
*** cshen has joined #openstack-ansible | 18:59 | |
*** yolanda__ has joined #openstack-ansible | 19:04 | |
*** yolanda_ has quit IRC | 19:07 | |
*** yolanda_ has joined #openstack-ansible | 19:09 | |
*** sew has quit IRC | 19:09 | |
*** yolanda__ has quit IRC | 19:12 | |
*** vnogin has joined #openstack-ansible | 19:13 | |
d34dh0r53 | odyssey4me: I reviewed your patches, I -1'd one of them as I addressed that with my patch, but I'll need to rebase once everything merges | 19:13 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Run each setup playbook individually https://review.openstack.org/578035 | 19:14 |
d34dh0r53 | cloudnull: ^ fyi | 19:14 |
*** yolanda__ has joined #openstack-ansible | 19:15 | |
*** yolanda_ has quit IRC | 19:18 | |
*** dxiri has quit IRC | 19:22 | |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Use package module properly https://review.openstack.org/578036 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Set tftp/pxe directory permissions using file module https://review.openstack.org/578037 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Ensure a consistent and readable style https://review.openstack.org/578043 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Enable saving and re-using file-based backing images https://review.openstack.org/578047 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Correct Mac OS X instructions for virt-manager https://review.openstack.org/578053 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Disable SSH key checks on host https://review.openstack.org/578059 | 19:23 |
openstackgerrit | Merged openstack/openstack-ansible-ops master: MNAIO: Check capabilities only once https://review.openstack.org/578060 | 19:23 |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_octavia stable/queens: Adds the issuer to the CAs https://review.openstack.org/578118 | 19:35 |
*** vnogin has quit IRC | 19:35 | |
*** hachi has quit IRC | 19:39 | |
d34dh0r53 | damnit zuul | 19:54 |
*** zul has joined #openstack-ansible | 19:56 | |
*** sew has joined #openstack-ansible | 20:02 | |
*** dxiri has joined #openstack-ansible | 20:05 | |
openstackgerrit | Merged openstack/openstack-ansible stable/queens: Upgrade task missing equals on tag https://review.openstack.org/569857 | 20:17 |
*** tux_ has quit IRC | 20:39 | |
*** ansmith has quit IRC | 20:41 | |
*** hamzy has quit IRC | 20:47 | |
tomtom001 | Getting the following error when trying to run the octavia playbook for queens (http://paste.openstack.org/show/724340/) anyone had similar issue or know what it means? | 20:48 |
*** FrankZhang has quit IRC | 20:50 | |
*** tux_ has joined #openstack-ansible | 20:54 | |
ivve | cloudnull: you around? | 21:00 |
*** zul has quit IRC | 21:02 | |
cloudnull | ivve kinda , whats up ? | 21:09 |
cloudnull | tomtom001 -cc xgerman_ ? | 21:09 |
xgerman_ | what’s up? | 21:09 |
cloudnull | ivve did you get the keepalived things working? | 21:10 |
cloudnull | xgerman_ "Getting the following error when trying to run the octavia playbook for queens (http://paste.openstack.org/show/724340/) anyone had similar issue or know what it means?" | 21:10 |
xgerman_ | the system is trying to set up iptables rules — I think I configured it to ignore that error since I found it always erroring but still doing what I wanted | 21:11 |
ivve | cloudnull: aye some improvements | 21:11 |
ivve | cloudnull: keepalived doing its job properly now on hosts | 21:12 |
cloudnull | that's good! | 21:12 |
cloudnull | but? | 21:12 |
*** yolanda_ has joined #openstack-ansible | 21:12 | |
ivve | im not sure garps are being sent from ha routers properly | 21:12 |
ivve | there used to be a setting in l3_agent.ini | 21:13 |
xgerman_ | yeah, also which step was that. Scrolling at the end “No first item, sequence was empty.” wpuldn’t be the error I expect with iptables | 21:13 |
ivve | perhaps thats the issue at hand | 21:13 |
cloudnull | so its better with 3.2+ but still kinda broken. | 21:13 |
ivve | well its possible changes have to be done | 21:13 |
cloudnull | is there an option we could set ? | 21:13 |
ivve | i.e paths to binaries must be absolute | 21:14 |
ivve | so ping and kill is used | 21:14 |
cloudnull | ? | 21:14 |
*** yolanda__ has quit IRC | 21:14 | |
cloudnull | is something in neutron itself ? | 21:14 |
ivve | keepalived | 21:14 |
cloudnull | like an override to the ping / kill binary ? | 21:14 |
cloudnull | s/override/option/ | 21:15 |
ivve | it uses scripts | 21:15 |
xgerman_ | tomtom001: I need to know more. Octavia uses some heuristics to “guess” network settings - if they differ in your env. you will need to set them explicitly | 21:15 |
ivve | vrrp_script | 21:15 |
cloudnull | ok. is there an interface we can use to customize that to make it better for l3+ha? | 21:15 |
ivve | send_arp_for_ha | 21:18 |
cloudnull | it looks like neutron generates its own scripts https://github.com/openstack/neutron/blob/master/neutron/agent/linux/keepalived.py#L557-L566 | 21:18 |
ivve | this was used during 16.0.0 neutron | 21:18 |
ivve | yes for keepalived in neutron agents it configures its own | 21:19 |
ivve | those seem fine, apart from sending garps i think, possible due to this option. im going to test it out now | 21:19 |
ivve | https://docs.openstack.org/ocata/config-reference/networking/samples/l3_agent.ini.html | 21:19 |
cloudnull | https://docs.openstack.org/releasenotes/neutron/pike.html#upgrade-notes | 21:20 |
cloudnull | looks like that option was deprecated | 21:20 |
tomtom001 | xgerman_ ok I've setup in defaults/main.yml in octavia role the neutron defined network that octavia should use to contact the amphora instances. What specifics do you need? | 21:21 |
cloudnull | ivve looks like deprecated in ocata https://docs.openstack.org/releasenotes/neutron/ocata.html#deprecation-notes | 21:21 |
cloudnull | seems it arps whenever a new float is created ... | 21:21 |
*** FrankZhang has joined #openstack-ansible | 21:21 | |
tomtom001 | xgerman_ I also installed openstack and saved the octavia for a later step in installation because it was asking for a network in neutron that doesn't exist in pre-install or during install. | 21:22 |
*** cshen has quit IRC | 21:22 | |
xgerman_ | ok, here are all the network settings: https://github.com/openstack/openstack-ansible-os_octavia/blob/master/defaults/main.yml#L292-L319 | 21:22 |
*** jwitko_ has joined #openstack-ansible | 21:22 | |
xgerman_ | it will also create a neutron network by itself | 21:22 |
tomtom001 | xgerman_ oh I see, let me try and let it do that. | 21:23 |
xgerman_ | I think this setting goes sideways for you: https://github.com/openstack/openstack-ansible-os_octavia/blob/master/defaults/main.yml#L312 | 21:23 |
xgerman_ | in a lot of cases we just set that directly… | 21:24 |
ivve | cloudnull: i think i need more testing.. i had some issues with services possibly not restarted properly and in the right order. i restarted the entire environment recently | 21:25 |
*** jwitko has quit IRC | 21:26 | |
ivve | moving around FLIPs is no issue | 21:26 |
tomtom001 | Yeah so: "msg": "(400) Client Error for url: http://10.80.20.24:9696/v2.0/networks.json Invalid input for operation: physical_network 'Octavia-Provider' unknown for VLAN provider network." | 21:27 |
*** jwitko_ has quit IRC | 21:27 | |
tomtom001 | xgerman_ I was creating that in neutron first because I got the above error when trying to let octavia playbook do it. | 21:27 |
xgerman_ | yes, it will trying to guess the provider network by assuming you named the provider network (not the neutron one) lbaas-mgmt | 21:28 |
*** FrankZhang_ has joined #openstack-ansible | 21:29 | |
xgerman_ | and it then reads the list of provider networks which have been added to ovs/linux-bridge | 21:29 |
tomtom001 | For the octavia_provider_network do you set the Name as shown? http://paste.openstack.org/show/724341/ | 21:30 |
tomtom001 | xgerman_ so the provider network would be br-mgmt then? | 21:30 |
*** FrankZhang has quit IRC | 21:30 | |
xgerman_ | no, it;s the name you choose in /etc/openstack_deploy/user_config… | 21:31 |
xgerman_ | https://github.com/openstack/openstack-ansible-os_octavia/blob/master/tests/host_vars/octavia1.yml | 21:32 |
xgerman_ | or https://github.com/openstack/openstack-ansible-os_octavia/blob/master/tests/host_vars/octavia1.yml | 21:32 |
xgerman_ | err https://github.com/openstack/openstack-ansible-os_octavia/blob/master/tests/os_octavia-overrides.yml#L81-L85 | 21:32 |
*** ggnel has quit IRC | 21:34 | |
*** sew has quit IRC | 21:35 | |
tomtom001 | xgerman_ so network_flat_networks needs to specify the provider network that octavia will use? | 21:36 |
tomtom001 | xgerman_ also, those neutron settings should be in user_variables.yml? | 21:38 |
xgerman_ | Well, the provider nets are usually declared there... yes, if you need to overwrite some of Octavia’s guesses that should go there | 21:39 |
*** ggnel has joined #openstack-ansible | 21:43 | |
*** DanyC has quit IRC | 21:47 | |
*** DanyC has joined #openstack-ansible | 21:47 | |
*** DanyC has quit IRC | 21:52 | |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic master: Overhaul ironic role https://review.openstack.org/578219 | 21:56 |
prometheanfire | mnaser: odyssey4me https://review.openstack.org/578219 :D | 21:56 |
prometheanfire | I made it a new review because I didn't like where the old review was going, I think this way is simpler | 21:57 |
prometheanfire | passed lint at least | 21:58 |
*** tux_ has quit IRC | 22:06 | |
*** tux_ has joined #openstack-ansible | 22:06 | |
*** yolanda__ has joined #openstack-ansible | 22:07 | |
*** yolanda_ has quit IRC | 22:10 | |
*** ggnel has quit IRC | 22:11 | |
*** tux_ has quit IRC | 22:11 | |
*** cjloader has quit IRC | 22:13 | |
*** ggnel has joined #openstack-ansible | 22:14 | |
*** sew has joined #openstack-ansible | 22:16 | |
*** vurtz has quit IRC | 22:20 | |
*** yolanda_ has joined #openstack-ansible | 22:21 | |
*** lihi has quit IRC | 22:22 | |
*** yolanda__ has quit IRC | 22:24 | |
*** vurtz has joined #openstack-ansible | 22:24 | |
*** lihi has joined #openstack-ansible | 22:25 | |
d34dh0r53 | anyone know how to make zuul merge this https://review.openstack.org/#/c/578091/ ? | 22:30 |
d34dh0r53 | it's passed everything, I may have made it worse by removing and re-adding my workflow vote | 22:30 |
*** throwsb1 has quit IRC | 22:31 | |
prometheanfire | d34dh0r53: be very nice to fungi :P | 22:33 |
prometheanfire | iirc WF is checked at the start and end of a merge, not during | 22:33 |
*** ansmith has joined #openstack-ansible | 22:35 | |
d34dh0r53 | yeah | 22:37 |
*** threestrands has joined #openstack-ansible | 22:38 | |
*** threestrands has quit IRC | 22:38 | |
*** threestrands has joined #openstack-ansible | 22:38 | |
*** FrankZhang_ has quit IRC | 22:41 | |
*** FrankZhang has joined #openstack-ansible | 22:42 | |
openstackgerrit | David Wilde proposed openstack/openstack-ansible-ops master: Improve multi-node AIO robustness https://review.openstack.org/578162 | 22:44 |
*** sew has quit IRC | 22:47 | |
*** sew has joined #openstack-ansible | 22:52 | |
*** cmart has quit IRC | 22:59 | |
*** sew has quit IRC | 23:03 | |
*** nicolasbock has quit IRC | 23:06 | |
*** tosky has quit IRC | 23:08 | |
*** tux_ has joined #openstack-ansible | 23:17 | |
*** tux_ has quit IRC | 23:21 | |
*** chyka has quit IRC | 23:36 | |
*** ggnel has quit IRC | 23:37 | |
*** cjloader has joined #openstack-ansible | 23:37 | |
*** cjloader has quit IRC | 23:41 | |
openstackgerrit | Matthew Thode proposed openstack/openstack-ansible-os_ironic master: Overhaul ironic role https://review.openstack.org/578219 | 23:47 |
*** sew has joined #openstack-ansible | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!