Thursday, 2018-11-08

openstackgerritDamian Cikowski (dcdamien) proposed openstack/openstack-ansible master: Add extra volume types to AIO  https://review.openstack.org/56126600:00
*** ansmith has joined #openstack-ansible00:01
*** faizy98 has joined #openstack-ansible00:04
*** faizy_ has quit IRC00:06
*** dhellmann has joined #openstack-ansible00:07
*** cshen has joined #openstack-ansible00:12
*** cshen has quit IRC00:16
*** gyee has quit IRC00:55
*** nurdie has quit IRC01:02
*** nurdie has joined #openstack-ansible01:03
*** cmart has quit IRC01:04
*** nurdie has quit IRC01:07
*** faizy_ has joined #openstack-ansible01:28
*** faizy_ has quit IRC01:30
*** faizy_ has joined #openstack-ansible01:30
*** faizy98 has quit IRC01:32
*** faizy_ has quit IRC01:33
*** faizy98 has joined #openstack-ansible01:34
*** faizy98 has quit IRC01:34
*** faizy98 has joined #openstack-ansible01:35
*** faizy_ has joined #openstack-ansible01:38
*** dcdamien has quit IRC01:38
*** faizy98 has quit IRC01:40
*** mmercer has quit IRC01:45
openstackgerritMerged openstack/openstack-ansible stable/rocky: Ensure pip/setuptools/wheel are up to date in the utility venv  https://review.openstack.org/61525801:50
openstackgerritMerged openstack/openstack-ansible master: Imported Translations from Zanata  https://review.openstack.org/61577001:50
*** markvoelker has joined #openstack-ansible02:45
*** markvoelker has quit IRC02:45
*** markvoelker has joined #openstack-ansible02:45
*** faizy98 has joined #openstack-ansible02:49
*** markvoelker has quit IRC02:50
*** faizy_ has quit IRC02:51
*** markvoelker has joined #openstack-ansible02:55
*** mrhillsman has joined #openstack-ansible03:19
*** cmart has joined #openstack-ansible03:42
openstackgerritLogan V proposed openstack/openstack-ansible-os_neutron stable/queens: [WIP] Update Calico Felix to v3  https://review.openstack.org/61640703:57
openstackgerritLogan V proposed openstack/openstack-ansible stable/queens: Update etcd role to v2.0  https://review.openstack.org/61640804:03
*** ravirjn has quit IRC04:36
*** radeks_ has joined #openstack-ansible04:39
*** cmart has quit IRC04:48
*** cmart has joined #openstack-ansible05:01
*** cmart has quit IRC05:15
*** cshen has joined #openstack-ansible05:56
*** cshen has quit IRC06:11
*** cshen has joined #openstack-ansible06:24
*** speedmann has quit IRC06:28
*** cshen has quit IRC06:28
*** vnogin has joined #openstack-ansible07:01
*** vnogin has quit IRC07:06
*** cshen has joined #openstack-ansible07:30
*** pcaruana has joined #openstack-ansible07:34
*** fghaas has joined #openstack-ansible08:03
fghaasodyssey4me, jrosser: Following up on our discussion from yesterday. I respun my test environment from scratch, just to be sure, and confirmed that there's something seriously fishy with the ceph-rgw playbook. I still don't have a clue about the root cause, but maybe one of you could spare a minute to look into https://bugs.launchpad.net/bugs/1802195 and let me know if any of that rings a bell.08:14
openstackLaunchpad bug 1802195 in openstack-ansible "ceph-rgw installation/configuration lacks idempotency, configuration changes break the setup" [Undecided,New]08:14
jrosserfghaas: interesting. which OSA version did you use?08:38
fghaasstable/rocky from yesterday08:38
fghaasby the way, just to be clear: it could well be that the fishiness is not in the playbook proper, but in the role it applies.08:40
jrosserwell indeed - all of this stuff consumes ceph-ansible so it could easily be something there08:41
fghaasWhat I really fail to grok is how exactly the content of ceph.conf is being assembled here.08:42
fghaasAnd that empty-hostnamed [client.rgw.] section just leaves me completely baffled.08:43
fghaasAs does the non-idempotency.08:43
jrosserfunny things can happen with too many levels of inclusion in ansible - that may be a factor08:45
fghaasjrosser: also, the issue of using a non-stable branch for ceph-ansible could be biting us here08:46
*** rgogunskiy has joined #openstack-ansible08:59
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Selectively install tempest plugin distro packages  https://review.openstack.org/61586909:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Selectively install tempest plugin distro packages  https://review.openstack.org/61586909:08
fghaasjrosser: I just changed my local ceph-ansible role checkout to the stable-3.1 branch and voilĂ .09:09
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova stable/rocky: Clean up role tests  https://review.openstack.org/60187009:11
*** radeks has joined #openstack-ansible09:11
*** radeks_ has quit IRC09:12
*** DanyC has joined #openstack-ansible09:18
*** jcosmao has joined #openstack-ansible09:20
*** shardy has joined #openstack-ansible09:22
fghaasodyssey4me: OK so what's the correct way to fix this? Do I send a patch with the current tip of the stable-3.1 branch for ceph-ansible in ansible-role-requirements.yml for stable/rocky, and one with the stable-3.1 branch for master?09:26
odyssey4mefghaas yep, sounds about right09:30
odyssey4memake sure they both use the same change id as they're related09:31
odyssey4medoes anyone have any ideas whether we need to port https://review.openstack.org/616218 back to queens or earlier?09:33
odyssey4meyep, looks like queens is failing: http://logs.openstack.org/79/616379/2/check/openstack-ansible-deploy-aio_basekit-centos-7/ec28f55/job-output.txt.gz#_2018-11-08_00_47_00_07175509:34
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server stable/queens: Use vendored RPM GPG keys for RabbitMQ  https://review.openstack.org/61647509:40
jrosserfghaas: did you try stable-3.2?09:40
odyssey4mehmm, yeah - for rocky we'll want the closest branch to the current sha pin09:40
fghaasjrosser: no, and I see no use for including that as I'm about to explain in the commit message09:40
odyssey4meand for master we'll want the most recent stable branch09:40
fghaasstable-3.2 is at rc1. We don't want that yet.09:41
odyssey4meah ok - that sounds fine for master then?09:41
fghaasdoesn't ansible-role-requirements get updated from master at every stable release? Having master pointed at the wrong branch got us into this mess, and I'd like to avoid that next time09:42
jrosserooooh confusing - the branch names make it look like 3.2 is released09:42
odyssey4mesimilar to what we've done in https://github.com/openstack/openstack-ansible/blob/master/playbooks/defaults/repo_packages/openstack_services.yml#L35 we should probably implement something in https://github.com/openstack/openstack-ansible/blob/master/ansible-role-requirements.yml to specify the tracking branch, assuming that doesn't break ansible-galaxy09:42
odyssey4meI wonder whether evrardjp has given any thought to that for the release tooling.09:43
odyssey4mehwoarang any chance you could give https://review.openstack.org/615577 a quick review?09:44
hwoarangall done09:46
openstackgerritFlorian Haas proposed openstack/openstack-ansible master: Track a stable branch, not master, for ceph-ansible  https://review.openstack.org/61647909:47
openstackgerritFlorian Haas proposed openstack/openstack-ansible stable/rocky: Reset ceph-ansible to the current HEAD of stable-3.1  https://review.openstack.org/61648109:47
fghaasodyssey4me, jrosser: here you go. Naturally the two commit messages are rather quite different and only one of those patches actually *fixes* https://bugs.launchpad.net/bugs/1802195, the other one is merely related.09:48
openstackLaunchpad bug 1802195 in openstack-ansible "ceph-rgw installation/configuration lacks idempotency, configuration changes break the setup" [Undecided,New]09:48
jrossertinwood: are you around? i have progress on nova-lxd09:48
tinwoodjrosser, hi09:48
jrossertinwood: this is almost working https://review.openstack.org/#/c/616302/09:49
jrosseri wondered if you had this working with a dir backend, because it's currently failing like this http://paste.openstack.org/show/734403/09:50
*** vnogin has joined #openstack-ansible09:57
*** faizy_ has joined #openstack-ansible10:05
tinwoodjrosser, back; had a trades-person here -- now looking.10:06
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Correct conditional in l3_agent.ini template  https://review.openstack.org/61487810:07
*** shardy has quit IRC10:07
odyssey4mejrosser could you backport https://review.openstack.org/614878 as far as is required?10:08
*** shardy has joined #openstack-ansible10:09
*** faizy98 has quit IRC10:09
*** electrofelix has joined #openstack-ansible10:10
*** faizy_ has quit IRC10:12
*** shardy has quit IRC10:14
tinwoodjrosser, argh, this is to do with quotas and dir backend; i.e. the instance is speccing quotas for the disk, but the backend doesn't support it (i.e, lxd dir storage) -- this is essentially a bug in nova-lxd as complete handling of lxd storage hasn't been implemented (we never actually ship - in production - a dir backed lxd system - the performance would be awful!)10:14
tinwoodthis didn't happen before with lxd 2 as lxd 2 didn't use 'storage' in the same way -- i.e. it just ignored the quotas.10:15
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Selectively install tempest plugin distro packages  https://review.openstack.org/61586910:16
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible master: [DNM] - Testing OBS master  https://review.openstack.org/50745110:18
odyssey4mecores - when you have a moment, we have plenty of patches looking for review - these are priority: http://bit.ly/2NVPFCg10:18
*** shardy has joined #openstack-ansible10:19
odyssey4mejrosser did you see https://review.openstack.org/#/c/614591/1/scripts/upgrade-utilities/playbooks/galera-cluster-rolling-restart.yml ?10:25
odyssey4meI mean, using the openstack_service_setup_host is certainly an option, but using the physical_host would also work and take away a bunch of other deps in the chain of comms10:26
*** Emine has joined #openstack-ansible10:32
evrardjpodyssey4me: I have thought of that and it indeed breaks10:37
evrardjpI haven't tried in latest ansible though10:38
*** shardy has quit IRC10:38
evrardjpbut the idea was that we always set a hash, and we can follow the current tracking branch of the code, which is good enough for me10:38
*** shardy has joined #openstack-ansible10:39
odyssey4meevrardjp yeah, but in the case of external things it's not that simple10:39
odyssey4mefor example, for ceph-ansible we don't necessarily want to follow master - but instead the latest stable branch10:39
evrardjpyeah I agree but all our externals are tracking master and are not branched10:40
odyssey4meexcept that they're not - for example, ceph-ansible10:40
evrardjpI discussed with logan- about how he tags things, and the idea was to use sem-ver like behaviour to know when to bump10:40
evrardjpif a "major" element was bumped, this would require manual intervention10:40
odyssey4meso basically in the tooling that does these updates we're going to have to hard-code the branch10:41
evrardjpodyssey4me: ceph-ansible is the only one in that case10:41
evrardjpodyssey4me: I haven't done that10:42
evrardjpbut it is thinkable10:42
evrardjpI like the idea of re-trying to add a similar element to know what to track10:43
evrardjpit's very explicit10:43
evrardjpit could allow re-use of code10:43
evrardjpbut it sadly would require a 'semver like' behaviour for other roles10:44
*** shardy has quit IRC10:44
odyssey4meI wonder if a comment in a-r-r would break ansible-galaxy?10:44
evrardjpodyssey4me: that should work10:44
odyssey4meIt's not ideal, because we'd have to read the file as a text file and process that way, but it's an option.10:45
evrardjpI mean seperate comments. I don't know about line comments10:45
evrardjpif line comments are added, it's very doable using ruamel.yaml10:45
evrardjpas you can know on which line the comment matches, and therefore adapt. But it's easy to mess up10:45
evrardjpThis requirement for ceph-ansible automatic bump is new, and you're right, I didn't take care of that.10:46
odyssey4meevrardjp well, in the past we've just left it to be bumped manually by whoever wants a new feature or whatever10:48
odyssey4methe issue here is that when we did the pin for stable/rocky, we didn't pick the latest stable branch... and we hadn't been testing against the latest stable branch anyway, so if we did it might have broken10:49
fghaasevrardjp: ummm, who asked for a bump to a beta1 tag of a pending release?10:50
odyssey4meso ideally we should discover and use the latest stable branch in master, and make sure that's the case whenever we bump master SHA's10:50
fghaasnot even beta1 actually, but 130 random commits _after_ beta110:50
odyssey4mefghaas I don't think it was asked for - that was just the current head of master at the time10:50
fghaaswell, yeah, but the ceph-ansible master branch has _always_ been dubbed experimental10:51
odyssey4methat's why I'm suggesting we change strategy for how we pin ceph-ansible for our master10:51
odyssey4mefghaas just like our master ;)10:51
fghaasyes but now OSA has a stable branch that tracks an experimental Ceph10:52
evrardjpI remember a "I need this version now" but I can't remember when or in which branch that happened.10:52
odyssey4mefghaas yes, and that was a mistake - and we're discussing a way to prevent that happening again10:52
evrardjpIMO it should not be shipped within OSA10:52
odyssey4mewell, yes - I feel that way too10:53
evrardjpthis was a design mistake.10:53
odyssey4meI'd rather see the combined install go away, and just leave the integration parts - and have a user story for how you do them.10:53
evrardjpodyssey4me: well we can offer plays like that in ops, right10:53
odyssey4mewe don't need to, because ceph-ansible has its own plays10:54
evrardjpwhat I'd love to see, is that there are images in openstack CI that exercise this building of ceph cluster.10:54
odyssey4mebut yes, we can offer whatever is needed as user config to make it happen10:54
evrardjpand across deployment projects, we could just reuse said images10:54
evrardjp(I mean a zuul nodeset basically)10:54
evrardjpfghaas: could you enlighten me a little: how hard would it be now to move to a stable branch for people having deployed stable/rocky?10:55
evrardjpwe can issue a .1.0 release10:55
evrardjpit sounds a big deal enough10:56
*** faizy98 has joined #openstack-ansible10:56
fghaasevrardjp: well. People who deployed Ceph with stable/rocky deployed Ceph Luminous with ceph-ansible 3.2.0beta1-130. And by my reckoning the moment they rerun it, their radosgw Swift API endpoints would break. You'd hear about if that happened, and since you didn't until I filed my bug last night, I suppose that that particular breakage isn't encountered by many people.10:58
fghaasBut to be honest I have no bandwidth right now to test *all* the implications of downgrading to stable-3.1. What I've tested is that the downgrade doesn't break *my* setup, and was hoping for the gate to flag any other issues.11:00
fghaasI think for OSA stable/rocky you only have the option of downgrading now, and hoping not to affect too many users, or wait until stable-3.2 is actually stable, and hoping not to affect too many users during the wait.11:01
fghaass/option/options/11:01
odyssey4mefghaas evrardjp I've suggested that we bump to 18.1.0 and add a release note in https://review.openstack.org/#/c/616481/111:01
odyssey4mefghaas those who have been early adopters can decide what to do about it - we need to warn off any more people asap11:02
odyssey4meif anyone has a good solution, they can contribute it as a release note or whatever... it's not all up to me or you to work out the best solution11:02
odyssey4meour only responsible requirement is to raise the flag asap11:02
odyssey4meand to resolve the issue for any greenfield deploys from here on11:03
odyssey4meI happen to be doing some ceph testing in the near future, so perhaps I can work out the user story and publish that - then we can deprecate the combined deployment.11:04
jrosserevrardjp: i had trouble with the way the gnocchi external git repo sha is specified btw - feels very much related to the ceph-ansible discussion11:13
jrossertinwood: what would you recommend we do about the dir backend?11:13
noonedeadpunkmorning everyone11:14
noonedeadpunkevrardjp: probably you may take a look a this patch, once you'll have time to spare? https://review.openstack.org/#/c/584629/11:15
odyssey4mejrosser what sort of trouble? did it struggle to checkout the sha? we've seen that for some things and it seems to be something related to the newer releases of ansible11:15
jrosseryes exactly that11:15
odyssey4mewe've had reports that wiping the existing git checkout and doing it again from scratch worked11:16
odyssey4meso the suspicion right now is something to do with how the git module does the update11:16
noonedeadpunkYep, confirm, that worked for me11:16
jrosseri think if you git clone the gnocchi repo locally and try to checkout the hash specified it doesnt work11:16
odyssey4meor perhaps the options we're feeding it are causing the trouble11:16
odyssey4meoh, well that's a different issue11:16
jrosserlets me just try that now11:17
tinwoodjrosser, I think I need to try to patch nova-lxd to *not* do quotas if the storage backend is a dir ... I've got a much more major "storage" patch going on that changes some of the fundamentals of nova-lxd and lxd in terms of storage, but a quick patch to just "not do quotas on dir" is probably useful right now.11:18
jrossertinwood: so for our current OSA role we should probably switch up the test to use a different backend?11:19
tinwoodjrosser, well I know it works with ZFS as a backend; the only gotcha is that the zfs and nova-lxd pool names *have* to be the same.11:22
tinwoodjrosser, however, I'm going to patch nova-lxd anyway -- preferably today.11:24
odyssey4metinwood thanks - appreciate that11:25
jrossertinwood: thats great - we have some tests that use zfs already so it shouldnt be too hard to lift/shift some of that into the nova-lxd role11:26
jrosser*role test11:27
tinwoodjrosser, ok; do the zfs pools use a block dev or a dir?  I just don't know the interaction between lxd 3.0, dir storage pools, and if a root device on an instance has quotas!11:28
jrosseri think in our role tests it trunactes a sparse file and created the pool from that11:28
tinwoodjrosser, right a pseudo-dev should be okay like that.11:29
jrosserbut so long as you have a valid/working zfs pool what's underneath it shouldnt matter11:29
*** cshen has quit IRC11:29
jrosserodyssey4me: with gnocchi git checkout i was getting "fatal: reference is not a tree" when doing my Q->R upgrade with hash 1e506094949d68b452d044941c0d6f85ef997b9e11:31
jrosserand i adjusted the hash to be a bit earlier in the history dce5db9b9ff87bafd65b3c675c57b611cf29a6ff and it everything came good11:31
odyssey4mejrosser evrardjp I wonder if we shouldn't switch to using released versions of gnocchi, rather than from git.11:31
*** faizy_ has joined #openstack-ansible11:32
jrosseri'm convinced that i reproduced that locally, but i can't make it do that on a local checkout right now11:32
odyssey4mePerhaps the issue was a github data issue or something.11:32
jrosserodyssey4me: in fact here it is out of my log http://paste.openstack.org/11:33
jrosseroops11:33
jrosserhttp://paste.openstack.org/show/734412/11:34
*** faizy98 has quit IRC11:35
*** dcdamien has joined #openstack-ansible11:35
*** faizy98 has joined #openstack-ansible11:36
*** dcdamien has quit IRC11:36
*** dcdamien has joined #openstack-ansible11:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Selectively install tempest plugin distro packages  https://review.openstack.org/61586911:36
openstackgerritmelissaml proposed openstack/openstack-ansible-tests master: Trivial: Fix the pep8 warning  https://review.openstack.org/61651711:38
*** dcdamien has quit IRC11:38
*** dcdamien has joined #openstack-ansible11:38
*** faizy_ has quit IRC11:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Selectively install tempest plugin distro packages  https://review.openstack.org/61586911:38
*** dcdamien has quit IRC11:38
*** dcdamien has joined #openstack-ansible11:39
openstackgerritmelissaml proposed openstack/openstack-ansible-rsyslog_server master: Trivial: Fix the pep8 warning  https://review.openstack.org/61651911:44
openstackgerritMerged openstack/openstack-ansible master: Remove Dragonflow ML2 driver  https://review.openstack.org/61557711:48
openstackgerritmelissaml proposed openstack/openstack-ansible-rabbitmq_server master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652011:50
openstackgerritmelissaml proposed openstack/openstack-ansible-os_sahara master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652111:52
fghaasodyssey4me: for what it's worth I don't object to the combined Ceph cluster / OpenStack cluster install going away, but I'd strongly urge you to retain the ceph-rgw-install playbook for people who want a Swift endpoint that points to rgw.11:54
openstackgerritmelissaml proposed openstack/openstack-ansible-os_panko master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652211:54
odyssey4mefghaas sure - although a good case could be made for putting that into ceph-ansible instead11:56
openstackgerritmelissaml proposed openstack/openstack-ansible-os_monasca-ui master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652311:56
openstackgerritmelissaml proposed openstack/openstack-ansible-os_gnocchi master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652411:57
fghaasodyssey4me: It already is in ceph-ansible and OSA should be using the ceph-ansible rgw role, but you don't want to offload the rgw and Keystone endpoint config to the Ceph cluster admin.11:58
openstackgerritmelissaml proposed openstack/openstack-ansible-os_almanach master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652511:59
openstackgerritmelissaml proposed openstack/openstack-ansible-nspawn_hosts master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652612:01
openstackgerritmelissaml proposed openstack/openstack-ansible-nspawn_container_create master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652712:02
odyssey4mefghaas fair point12:02
*** cshen has joined #openstack-ansible12:02
openstackgerritmelissaml proposed openstack/openstack-ansible-lxc_container_create master: Trivial: Fix the pep8 warning  https://review.openstack.org/61652912:04
openstackgerritmelissaml proposed openstack/openstack-ansible-galera_server master: Trivial: Fix the pep8 warning  https://review.openstack.org/61653012:05
*** faizy98 has quit IRC12:05
*** faizy98 has joined #openstack-ansible12:06
*** cshen has quit IRC12:07
*** emine__ has joined #openstack-ansible12:08
*** Emine has quit IRC12:11
openstackgerritMerged openstack/openstack-ansible-os_nova stable/rocky: Clean up role tests  https://review.openstack.org/60187012:12
*** cshen has joined #openstack-ansible12:23
openstackgerritFlorian Haas proposed openstack/openstack-ansible master: Include Swift AUTH_%(tenant_id)s suffix in rgw Keystone endpoint  https://review.openstack.org/61419412:28
*** vnogin has quit IRC12:31
*** ansmith has quit IRC12:31
openstackgerritFlorian Haas proposed openstack/openstack-ansible master: Track a stable branch, not master, for ceph-ansible  https://review.openstack.org/61647912:37
*** asettle has joined #openstack-ansible12:54
fghaasodyssey4me: I can't do any more work on the Ceph issues today because I'm fighting fires. I'll get back to them when I can.12:56
*** asettle has quit IRC12:57
*** emine__ has quit IRC13:00
dcdamienfghaas: I've imagined this https://goo.gl/XBRNzo :D13:00
*** emine__ has joined #openstack-ansible13:00
fghaasdcdamien: I meant I'll get back to the issues, not the fires, when I can (just to be clear)13:01
hwoarangodyssey4me: qq. looking at  https://github.com/openstack/openstack-ansible-ceph_client/blob/master/tasks/ceph_install_python_libs.yml#L26-L42, is it always true that virtualenv is python2?13:05
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_zun master: fix tox python3 overrides  https://review.openstack.org/60679613:25
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_tempest master: fix tox python3 overrides  https://review.openstack.org/60681613:32
openstackgerritMerged openstack/openstack-ansible stable/queens: Update ansible to 2.4.6  https://review.openstack.org/61430813:32
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-plugins master: fix tox python3 overrides  https://review.openstack.org/60676913:34
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-nspawn_container_create master: fix tox python3 overrides  https://review.openstack.org/60679213:35
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-rsyslog_client master: fix tox python3 overrides  https://review.openstack.org/60676113:38
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_barbican master: fix tox python3 overrides  https://review.openstack.org/60682513:39
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_cinder master: fix tox python3 overrides  https://review.openstack.org/60682813:41
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_neutron master: fix tox python3 overrides  https://review.openstack.org/60681213:42
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_aodh master: fix tox python3 overrides  https://review.openstack.org/60682413:44
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-rsyslog_server master: fix tox python3 overrides  https://review.openstack.org/60675913:45
*** strattao has joined #openstack-ansible13:45
openstackgerritAndriy Shevchenko proposed openstack/openstack-ansible-os_swift master: fix tox python3 overrides  https://review.openstack.org/60682113:46
*** smoke87 has quit IRC13:47
*** ansmith has joined #openstack-ansible13:51
*** faizy98 has quit IRC14:02
*** dcdamien has quit IRC14:02
*** jcosmao has quit IRC14:02
*** rgogunskiy has quit IRC14:02
*** kukacz has quit IRC14:02
*** gokhan has quit IRC14:02
*** FrankZhang has quit IRC14:02
*** hwoarang has quit IRC14:02
*** mcarden has quit IRC14:02
*** nsmeds has quit IRC14:02
*** mrda has quit IRC14:02
*** mattoliverau has quit IRC14:02
*** hwoarang has joined #openstack-ansible14:02
*** cshen has quit IRC14:03
*** kukacz has joined #openstack-ansible14:04
*** faizy98 has joined #openstack-ansible14:04
*** dcdamien has joined #openstack-ansible14:08
*** fghaas has left #openstack-ansible14:29
openstackgerritmelissaml proposed openstack/openstack-ansible-galera_client master: Trivial: Fix the pep8 warning  https://review.openstack.org/61655314:35
*** nurdie has joined #openstack-ansible14:35
openstackgerritmelissaml proposed openstack/ansible-config_template master: Trivial: Fix the pep8 warning  https://review.openstack.org/61655714:36
openstackgerritmelissaml proposed openstack/ansible-role-python_venv_build master: Trivial: Fix the pep8 warning  https://review.openstack.org/61655814:38
odyssey4mehwoarang jrosser could I get some eyes on https://review.openstack.org/615869 which I think solves the current issue for distro installs with tempest and gives us a base to work with14:47
hwoarangodyssey4me: will look it to it. but do you have an answer to my previous question?14:47
hwoarang[13:05:11] <hwoarang> odyssey4me: qq. looking at  https://github.com/openstack/openstack-ansible-ceph_client/blob/master/tasks/ceph_install_python_libs.yml#L26-L42, is it always true that virtualenv is python2?14:47
odyssey4mehwoarang about that ceph-client bit, nova is doing https://github.com/openstack/openstack-ansible-os_nova/blob/master/tasks/drivers/kvm/nova_compute_kvm_install_source.yml which is a bit more generic, handling py2/py314:48
hwoarangi am asking because the ceph packages on leap 15 are only python314:48
hwoarangok so virtualenv is not python2-only14:48
hwoarangi wonder how bionic deals with ceph_client then...14:48
odyssey4mealso, masakari's learned from that and implemented something similar: https://review.openstack.org/#/c/584629/47/tasks/masakari_install.yml14:49
odyssey4meI've been thinking that it may make sense to include something to handle that in the python_venv_build role - then all we do is give it the packages to link into the venv and it will do it for us14:49
jrosser\o/ 503 https://gitlab.freedesktop.org/spice/spice-html5.git14:50
jrossergrrrr14:50
odyssey4mejrosser right - the question I find myself asking is why we're still installing novnc/spice from git when there are packages available these days... we put the git sourcing in back when the packages were badly out of date14:51
hwoarangyeah that would be ncie14:51
jrosseraaand it's back14:52
jrosserout long enough to bork my build14:52
jrosserodyssey4me: agreed if there are good packages it makes little sense to grab from source14:53
*** emine__ has quit IRC14:53
*** Emine has joined #openstack-ansible15:02
odyssey4memnaser guilhermesp stable/queens is broken for rabbitmq as well due to the gpg key change, so https://review.openstack.org/61647515:03
odyssey4me*gpg key URL change15:03
*** nurdie has quit IRC15:06
*** nurdie has joined #openstack-ansible15:07
*** shardy has joined #openstack-ansible15:08
*** shardy has quit IRC15:09
*** shardy has joined #openstack-ansible15:09
*** shardy has quit IRC15:09
*** shardy has joined #openstack-ansible15:11
*** nurdie has quit IRC15:11
*** irclogbot_0 has quit IRC15:17
cloudnullmornings all15:17
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Ensure LXD pool name is always specified in nova config  https://review.openstack.org/61630215:21
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Ensure LXD pool name is always specified in nova config  https://review.openstack.org/61630215:23
odyssey4mejrosser could we just merge https://review.openstack.org/#/c/616301/1 and https://review.openstack.org/#/c/616302/ together, and leave the test voting - that way it's easier to see whether the patch is working correctly and we move from a working xenial test to a working bionic test15:24
odyssey4meI guess the only reason to keep them split is to keep https://review.openstack.org/#/c/616302/ clean for backporting, but it seems that a modified backport will likely be needed for Rocky anyway.15:25
jrosserodyssey4me: of course, i can do that15:25
odyssey4meI'll also abandon https://review.openstack.org/#/c/612959/2 and https://review.openstack.org/#/c/612694/1 given they're doing the same things.15:26
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_nova master: remove loop control when running include_role  https://review.openstack.org/60781315:26
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-rabbitmq_server stable/queens: Update the version of erlang to 20  https://review.openstack.org/60700715:26
*** irclogbot_0 has joined #openstack-ansible15:32
*** irclogbot_0 has quit IRC15:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron stable/rocky: Correct conditional in l3_agent.ini template  https://review.openstack.org/61658715:34
*** irclogbot_0 has joined #openstack-ansible15:34
*** irclogbot_0 has joined #openstack-ansible15:36
*** irclogbot_0 has quit IRC15:36
*** hamzy has quit IRC15:36
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Fix nova-lxd tests for bionic.  https://review.openstack.org/61630215:37
*** irclogbot_0 has joined #openstack-ansible15:38
*** irclogbot_0 has quit IRC15:38
*** irclogbot_0 has joined #openstack-ansible15:40
*** irclogbot_0 has quit IRC15:40
*** shardy has quit IRC15:40
*** nurdie has joined #openstack-ansible15:44
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Fix ansible path for elk_metrics_6x tests  https://review.openstack.org/61421015:45
cloudnullvollman ^ I noticed a couple other nits and pop'd a patch for them instead of kicking it back. Thanks for that btw !15:46
cloudnullassuming I've not broken the gate we should have that in later today15:47
*** nurdie_ has joined #openstack-ansible15:48
*** nurdie has quit IRC15:48
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Metricbeat role check for a ceph api to monitor  https://review.openstack.org/61402715:50
openstackgerritMerged openstack/openstack-ansible-tests stable/queens: Update ansible to 2.4.6  https://review.openstack.org/61360015:50
*** irclogbot_0 has joined #openstack-ansible15:51
*** irclogbot_0 has quit IRC15:51
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Fix nova-lxd tests for bionic.  https://review.openstack.org/61630215:52
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible master: Increase CentOS test coverage  https://review.openstack.org/61031115:53
cloudnullvollman here's the diff https://pasted.tech/pastes/b512c7d3cbfe833122c646180dc2f24b1be54ac3 - which was nothing wrong with your patch just more cleanup that was missed from earlier PRs.15:53
*** devx has joined #openstack-ansible15:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/rocky: Use loop_var name in when clause  https://review.openstack.org/61659515:55
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/queens: Use loop_var name in when clause  https://review.openstack.org/61659615:55
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/pike: Use loop_var name in when clause  https://review.openstack.org/61659715:55
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible stable/ocata: Use loop_var name in when clause  https://review.openstack.org/61659815:56
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-ceph_client stable/rocky: Ensure ceph_client specific apt pin config  https://review.openstack.org/61660115:59
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Fix Cinder backup to use full class path  https://review.openstack.org/60584616:00
*** irclogbot_0 has joined #openstack-ansible16:02
odyssey4mehwoarang have you seen the feedback in https://review.openstack.org/614748 ?16:05
hwoarangodyssey4me: yeah just did not have time to make the changes just yet16:06
*** nurdie has joined #openstack-ansible16:06
*** nurdie_ has quit IRC16:09
*** irclogbot_0 has quit IRC16:09
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Fix nova-lxd tests for bionic.  https://review.openstack.org/61630216:10
*** irclogbot_0 has joined #openstack-ansible16:11
*** pcaruana has quit IRC16:14
*** irclogbot_0 has quit IRC16:15
*** gyee has joined #openstack-ansible16:18
*** hamzy has joined #openstack-ansible16:22
*** irclogbot_0 has joined #openstack-ansible16:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server stable/queens: Update the version of erlang to 20  https://review.openstack.org/60700716:26
odyssey4mecores - when you have a moment, we have plenty of patches looking for review - these are priority: http://bit.ly/2NVPFCg16:29
*** irclogbot_0 has quit IRC16:30
*** irclogbot_0 has joined #openstack-ansible16:32
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-ceph_client master: tasks: python: Add support for python3 environments  https://review.openstack.org/61660816:33
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible master: [DNM] - Testing OBS master  https://review.openstack.org/50745116:33
*** irclogbot_0 has quit IRC16:36
*** irclogbot_0 has joined #openstack-ansible16:38
*** irclogbot_0 has quit IRC16:42
*** cmart has joined #openstack-ansible16:42
*** irclogbot_0 has joined #openstack-ansible16:44
*** weezS has joined #openstack-ansible16:45
spotzchecking odyssey4me - still packing though and I leave this afternoon16:46
*** irclogbot_0 has quit IRC16:48
*** irclogbot_0 has joined #openstack-ansible16:50
weezSsafe trip16:51
*** irclogbot_0 has quit IRC16:54
hwoarangmbuil: should this be +2 now https://review.openstack.org/#/c/614282/ ?16:55
*** irclogbot_0 has joined #openstack-ansible16:58
openstackgerritMarkos Chandras (hwoarang) proposed openstack/openstack-ansible-tests master: test-log-collect.sh: Verify journal consinstency before gathering logs  https://review.openstack.org/61474817:00
*** ivve has joined #openstack-ansible17:02
*** irclogbot_0 has quit IRC17:03
*** nsmeds has joined #openstack-ansible17:08
openstackgerritMerged openstack/openstack-ansible-ceph_client stable/rocky: Ensure ceph_client specific apt pin config  https://review.openstack.org/61660117:08
*** nicolasbock_ has joined #openstack-ansible17:09
*** irclogbot_0 has joined #openstack-ansible17:10
*** irclogbot_0 has quit IRC17:11
openstackgerritMerged openstack/openstack-ansible-os_masakari master: Basic implementation of masakari-monitors  https://review.openstack.org/58462917:12
*** irclogbot_0 has joined #openstack-ansible17:13
openstackgerritMerged openstack/openstack-ansible-plugins stable/queens: Modify provider_network plugin to compare group_binds to group_names  https://review.openstack.org/61638017:14
openstackgerritMerged openstack/openstack-ansible-plugins stable/rocky: Modify provider_network plugin to compare group_binds to group_names  https://review.openstack.org/61637717:14
openstackgerritMerged openstack/openstack-ansible master: Track a stable branch, not master, for ceph-ansible  https://review.openstack.org/61647917:21
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_masakari stable/rocky: Add missing docs/releasenotes tests  https://review.openstack.org/61663517:23
*** Leo_m has joined #openstack-ansible17:27
openstackgerritMerged openstack/openstack-ansible-rabbitmq_server stable/queens: Use vendored RPM GPG keys for RabbitMQ  https://review.openstack.org/61647517:42
*** faizy98 has quit IRC17:44
*** DanyC has quit IRC17:45
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Update legacy neutron L3HA tool support  https://review.openstack.org/61617917:50
*** DanyC has joined #openstack-ansible17:50
*** DanyC has quit IRC17:51
noonedeadpunkAlso reviewing these 2 patches would be nice https://review.openstack.org/#/q/topic:fix/missing_services+(status:open+OR+status:merged)17:51
openstackgerritMerged openstack/openstack-ansible-os_nova master: remove loop control when running include_role  https://review.openstack.org/60781318:04
openstackgerritMerged openstack/openstack-ansible-os_neutron stable/rocky: Correct conditional in l3_agent.ini template  https://review.openstack.org/61658718:05
*** pcaruana has joined #openstack-ansible18:10
*** dcdamien has quit IRC18:12
openstackgerritMerged openstack/openstack-ansible stable/rocky: Adds option to enable ARA plugin install  https://review.openstack.org/61600418:30
*** _nyloc_ has joined #openstack-ansible18:34
*** nyloc has quit IRC18:34
openstackgerritMerged openstack/openstack-ansible stable/rocky: Restart cinder-scheduler instead of reloading  https://review.openstack.org/61564518:37
openstackgerritMerged openstack/openstack-ansible stable/queens: Restart cinder-scheduler instead of reloading  https://review.openstack.org/61564718:38
openstackgerritMerged openstack/openstack-ansible stable/rocky: Ensure ceph server apt pinning is well defined  https://review.openstack.org/61477718:38
*** Emine has quit IRC18:40
openstackgerritVadim Kuznetsov proposed openstack/openstack-ansible-os_octavia stable/queens: octavia container network address pool  https://review.openstack.org/61666718:41
openstackgerritAndy Smith proposed openstack/openstack-ansible master: Add qdrouterd role for oslo.messaging backend deployment  https://review.openstack.org/57026118:49
guilhermesphey thanks for this odyssey4me https://review.openstack.org/#/c/616475/18:51
openstackgerritMerged openstack/openstack-ansible master: Delegate check of db connectivity during upgrade  https://review.openstack.org/61459118:54
*** electrofelix has quit IRC18:55
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_gnocchi master: coordination_url should be placed in DEFAULT section  https://review.openstack.org/61667219:01
*** Emine has joined #openstack-ansible19:04
*** vnogin has joined #openstack-ansible19:08
*** Emine has quit IRC19:18
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/openstack-ansible-os_gnocchi master: coordination_url should be placed in DEFAULT section  https://review.openstack.org/61667219:21
noonedeadpunkI don't know, how this has happened (and why it appered just now), but it seems, that testing for gnocchi role is currently broken19:24
noonedeadpunkI've placed fix with ^ patch. If you would like to divide it into test fix and config change, please let me know19:33
*** fghaas has joined #openstack-ansible19:38
*** ThiagoCMC has joined #openstack-ansible19:44
ThiagoCMCGuys, does OSA supports DRBD?19:44
ThiagoCMCI'm looking for a way to use the Compute Nodes HDD as DRBD/Cinder volumes but, locally.19:45
ThiagoCMCPage 25: https://osec.pl/uploads/2016/12/drbd9anddrbdmanagearp2016.pdf  <- that's what I want to do with OSA!19:47
ThiagoCMCNo iSCSI, no Ceph...19:48
ThiagoCMCThey are calling it something like "DRBD SDS for OpenStack". ^_^19:51
*** dave-mccowan has joined #openstack-ansible19:56
*** vnogin has quit IRC20:04
*** fghaas has quit IRC20:12
*** fghaas has joined #openstack-ansible20:23
*** cshen has joined #openstack-ansible20:28
*** radeks_ has joined #openstack-ansible20:31
*** dave-mccowan has quit IRC20:33
*** radeks has quit IRC20:34
*** cshen has quit IRC20:36
*** radeks__ has joined #openstack-ansible20:36
openstackgerritJimmy McCrory proposed openstack/openstack-ansible stable/rocky: Provides group_names for use with provider_network plugin  https://review.openstack.org/61637820:37
*** radeks_ has quit IRC20:39
openstackgerritJimmy McCrory proposed openstack/openstack-ansible stable/queens: Provides group_names for use with provider_network plugin  https://review.openstack.org/61637920:40
*** ThiagoCMC has quit IRC20:42
*** cshen has joined #openstack-ansible20:45
*** cshen has quit IRC20:54
*** cshen has joined #openstack-ansible20:57
*** mftech has joined #openstack-ansible20:58
*** aedc has joined #openstack-ansible20:59
*** radeks__ has quit IRC21:06
*** fghaas has quit IRC21:10
*** ThiagoCMC has joined #openstack-ansible21:10
*** dcdamien has joined #openstack-ansible21:20
dcdamienmgariepy: would you like to restore W+1 which I accidentilly removed? ;) https://review.openstack.org/#/c/561266/21:22
*** DanyC has joined #openstack-ansible21:23
openstackgerritMerged openstack/openstack-ansible-os_neutron master: Plug port on OVS agents instead of L3 agents only  https://review.openstack.org/61428221:24
*** DanyC has quit IRC21:26
*** DanyC_ has joined #openstack-ansible21:26
*** dcdamien has quit IRC21:28
*** dcdamien has joined #openstack-ansible21:32
*** nicolasbock_ has quit IRC21:32
*** ThiagoCMC has quit IRC21:33
*** Emine has joined #openstack-ansible21:36
*** ansmith has quit IRC21:37
*** ivve has quit IRC21:38
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Fix nova-lxd tests for bionic.  https://review.openstack.org/61630221:39
*** hamzy has quit IRC21:43
*** strattao has quit IRC21:44
*** dcdamien has quit IRC21:46
*** dcdamien has joined #openstack-ansible21:46
openstackgerritDamian Cikowski (dcdamien) proposed openstack/openstack-ansible-os_cinder master: tasks: cinder_qos: Ommit nonexistent volume types  https://review.openstack.org/61637622:01
openstackgerritDamian Cikowski (dcdamien) proposed openstack/openstack-ansible-os_cinder master: Make open-iscsi as requirement just for volume and backup services  https://review.openstack.org/61455622:06
*** mattoliverau has joined #openstack-ansible22:06
*** mftech has quit IRC22:11
*** mftech has joined #openstack-ansible22:12
*** Emine has quit IRC22:14
*** nurdie has quit IRC22:20
*** cshen has quit IRC22:29
*** Emine has joined #openstack-ansible22:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Fix nova-lxd tests for bionic.  https://review.openstack.org/61630222:56
openstackgerritGuilherme  Steinmuller Pimentel proposed openstack/openstack-ansible-os_nova stable/queens: Add support to kvm nested virt  https://review.openstack.org/61671122:57
*** DanyC has joined #openstack-ansible23:00
*** DanyC_ has quit IRC23:03
*** Emine has quit IRC23:06
*** nurdie has joined #openstack-ansible23:11
*** mftech has quit IRC23:12
*** mftech has joined #openstack-ansible23:12
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_tempest master: Ignore list_all_implied_roles on Ubuntu distro installs  https://review.openstack.org/61671323:13
*** mftech has quit IRC23:13
*** Leo_m has quit IRC23:14
*** ThiagoCMC has joined #openstack-ansible23:15
*** nurdie has quit IRC23:15
*** DanyC has quit IRC23:24
*** med_ has joined #openstack-ansible23:26
*** cshen has joined #openstack-ansible23:33
*** ThiagoCMC has quit IRC23:34
*** cshen has quit IRC23:38
*** cmart has quit IRC23:38
openstackgerritKimMinsik proposed openstack/openstack-ansible-os_cinder master: Add resource_filter field in cinder.conf  https://review.openstack.org/61611323:51

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