Thursday, 2018-08-02

*** dave-mccowan has quit IRC00:02
*** olivierb- has quit IRC00:05
*** DanyC has joined #openstack-ansible00:06
*** poopcat has quit IRC00:08
*** ivve has quit IRC00:08
*** ggnel has quit IRC00:09
*** DanyC_ has quit IRC00:09
*** poopcat has joined #openstack-ansible00:10
*** gyee has quit IRC00:11
*** dave-mccowan has joined #openstack-ansible00:22
*** vnogin has joined #openstack-ansible00:22
*** cshen has joined #openstack-ansible00:26
*** cshen has quit IRC00:30
*** poopcat has quit IRC00:41
*** gregoryo has joined #openstack-ansible00:42
*** poopcat has joined #openstack-ansible00:42
*** dave-mccowan has quit IRC00:55
*** EmilienM has quit IRC01:02
*** EmilienM has joined #openstack-ansible01:03
*** dave-mccowan has joined #openstack-ansible01:46
*** poopcat has quit IRC03:00
*** coreywright has quit IRC03:03
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_cinder master: Add scheduler filters when backends are defined  https://review.openstack.org/58812603:06
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_cinder master: Resolve cinder config deprecation warnings  https://review.openstack.org/58812703:10
cloudnullkaiokmo excellent !03:12
cloudnullI guess it just needed a repo update?03:12
*** coreywright has joined #openstack-ansible03:20
*** dave-mccowan has quit IRC03:33
*** udesale has joined #openstack-ansible03:46
*** dxiri has joined #openstack-ansible04:10
*** Bhujay has joined #openstack-ansible04:38
*** FrankZhang has quit IRC04:41
*** mattinator has quit IRC04:43
*** dxiri has quit IRC05:11
*** coreywright has quit IRC05:20
*** coreywright has joined #openstack-ansible05:24
*** shyamb has joined #openstack-ansible05:36
*** sm806 has quit IRC06:04
*** skyscraper has quit IRC06:04
*** udesale has quit IRC06:23
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible master: Use standard env variables for the AIO bootstrap  https://review.openstack.org/57765206:38
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Use standard env variables for the AIO bootstrap  https://review.openstack.org/57765306:40
*** cshen has joined #openstack-ansible06:45
*** shyamb has quit IRC06:45
*** shyamb has joined #openstack-ansible06:46
*** shyamb has quit IRC06:59
*** shyamb has joined #openstack-ansible06:59
*** vnogin has quit IRC07:00
*** vnogin has joined #openstack-ansible07:00
*** vnogin has quit IRC07:07
jrosserthis https://review.openstack.org/#/c/587932/ then https://review.openstack.org/#/c/587944/ if anyone has a moment07:11
jrosserthen we should be able to take a view on bionic role tests voting07:12
odyssey4meo/07:14
andymccrbgmccollum: sorry i missed your ping yesterday! Yeah tht looks good, i thought about it a bit more, another alternative would be to have a free field so bootstrap_opts is just straight vars (x=y z=a) etc - and a free field "Additional_opts" or something similar, to allow a manual entry of anything that is needed after - but your way works.07:19
andymccri'll put a review on that07:19
andymccr(if it hasnt already merged)07:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Reduce testing tech debt  https://review.openstack.org/58266807:22
odyssey4meandymccr: could you review those two jrosser just mentioned please? they're quick and simple07:23
*** lkoranda has quit IRC07:23
*** shyamb has quit IRC07:26
*** shyamb has joined #openstack-ansible07:26
*** lkoranda has joined #openstack-ansible07:31
odyssey4meor evrardjp when you're in07:31
*** tosky has joined #openstack-ansible07:33
*** pmannidi has joined #openstack-ansible07:36
*** canori01 has quit IRC07:38
*** mattt has quit IRC07:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/58232407:39
*** Tahvok has quit IRC07:45
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute image setup against octavia_service_setup_host  https://review.openstack.org/58816507:45
*** flaper87 has quit IRC07:46
*** juhak has quit IRC07:46
*** Tahvok has joined #openstack-ansible07:46
*** juhak has joined #openstack-ansible07:46
*** zerick has quit IRC07:46
*** flaper87 has joined #openstack-ansible07:46
*** juhak is now known as Guest7097107:47
*** zerick has joined #openstack-ansible07:47
*** shyamb has quit IRC07:48
evrardjphaha the commit message jrosser07:52
evrardjpoh it's odyssey4me07:52
evrardjpmy bad07:52
odyssey4me:)07:52
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/58232407:56
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute image setup against octavia_service_setup_host  https://review.openstack.org/58816507:56
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute flavor setup against octavia_service_setup_host  https://review.openstack.org/58816607:56
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/58232407:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute image setup against octavia_service_setup_host  https://review.openstack.org/58816507:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute flavor setup against octavia_service_setup_host  https://review.openstack.org/58816607:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/58232407:59
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute image setup against octavia_service_setup_host  https://review.openstack.org/58816507:59
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute flavor setup against octavia_service_setup_host  https://review.openstack.org/58816607:59
odyssey4mesorry about the spam07:59
eumel8rofl08:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute service setup against a delegated host using Ansible built-in modules  https://review.openstack.org/58232408:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute image setup against octavia_service_setup_host  https://review.openstack.org/58816508:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute flavor setup against octavia_service_setup_host  https://review.openstack.org/58816608:05
*** gregoryo has quit IRC08:06
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Revert "Use generic vars file for ubuntu"  https://review.openstack.org/58793208:11
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Add Rocky UCA for bionic  https://review.openstack.org/58794408:11
*** coreywright has quit IRC08:12
*** coreywright has joined #openstack-ansible08:16
*** gillesMo has joined #openstack-ansible08:20
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute network setup against octavia_service_setup_host  https://review.openstack.org/58817408:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Move role setup into service setup tasks  https://review.openstack.org/58817508:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute security group setup against octavia_service_setup_host  https://review.openstack.org/58817608:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Remove octavia_requires_pip_packages  https://review.openstack.org/58817708:22
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Remove octavia_requires_pip_packages  https://review.openstack.org/58817708:27
*** pmannidi has quit IRC08:28
*** vnogin has joined #openstack-ansible08:32
*** electrofelix has joined #openstack-ansible08:37
*** shyamb has joined #openstack-ansible08:38
*** vnogin has quit IRC08:43
*** vnogin has joined #openstack-ansible08:45
*** udesale has joined #openstack-ansible08:46
*** jbadiapa has quit IRC08:47
*** pcaruana has joined #openstack-ansible09:00
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: [TEST] bionic  https://review.openstack.org/58663509:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_aodh master: [TEST] bionic  https://review.openstack.org/58665409:01
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_barbican master: [TEST] bionic  https://review.openstack.org/58665509:02
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible master: Use standard env variables for the AIO bootstrap  https://review.openstack.org/57765209:05
openstackgerritMaksim Malchuk proposed openstack/openstack-ansible stable/queens: Use standard env variables for the AIO bootstrap  https://review.openstack.org/57765309:10
*** Bhujay has quit IRC09:11
*** shyamb has quit IRC09:19
odyssey4meevrardjp: well, I have bad news and good news09:24
*** shyamb has joined #openstack-ansible09:27
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819009:40
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-tests master: Prevent common MQ task set from synchronizing  https://review.openstack.org/58819109:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819009:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819009:44
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819009:45
*** vollman has joined #openstack-ansible09:46
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: [TEST] bionic  https://review.openstack.org/58665609:49
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819009:49
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_cinder master: [TEST] bionic  https://review.openstack.org/58665709:50
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_designate master: [TEST] bionic  https://review.openstack.org/58665809:50
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_glance master: [TEST] bionic  https://review.openstack.org/58665909:50
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: [TEST] bionic  https://review.openstack.org/58666009:51
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_heat master: [TEST] bionic  https://review.openstack.org/58666109:52
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_horizon master: [TEST] bionic  https://review.openstack.org/58666209:52
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: [TEST] bionic  https://review.openstack.org/58666309:53
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: [TEST] bionic  https://review.openstack.org/58666409:53
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_molteniron master: [TEST] bionic  https://review.openstack.org/58666509:53
*** vnogin has quit IRC09:53
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819609:54
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: [TEST] bionic  https://review.openstack.org/58666609:54
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: [TEST] bionic  https://review.openstack.org/58666709:54
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_octavia master: [TEST] bionic  https://review.openstack.org/58666809:55
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_rally master: [TEST] bionic  https://review.openstack.org/58666909:55
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_sahara master: [TEST] bionic  https://review.openstack.org/58667009:55
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_swift master: [TEST] bionic  https://review.openstack.org/58667109:56
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tacker master: [TEST] bionic  https://review.openstack.org/58667209:56
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: [TEST] bionic  https://review.openstack.org/58667309:57
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_trove master: [TEST] bionic  https://review.openstack.org/58667409:57
*** vnogin has joined #openstack-ansible10:04
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: [TEST] bionic  https://review.openstack.org/58663510:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_aodh master: [TEST] bionic  https://review.openstack.org/58665410:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_barbican master: [TEST] bionic  https://review.openstack.org/58665510:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: [TEST] bionic  https://review.openstack.org/58665610:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_cinder master: [TEST] bionic  https://review.openstack.org/58665710:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_designate master: [TEST] bionic  https://review.openstack.org/58665810:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_glance master: [TEST] bionic  https://review.openstack.org/58665910:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: [TEST] bionic  https://review.openstack.org/58666010:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_heat master: [TEST] bionic  https://review.openstack.org/58666110:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_horizon master: [TEST] bionic  https://review.openstack.org/58666210:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: [TEST] bionic  https://review.openstack.org/58666310:16
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: [TEST] bionic  https://review.openstack.org/58666410:17
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_molteniron master: [TEST] bionic  https://review.openstack.org/58666510:17
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: [TEST] bionic  https://review.openstack.org/58666610:17
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: [TEST] bionic  https://review.openstack.org/58666710:17
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_rally master: [TEST] bionic  https://review.openstack.org/58666910:18
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_sahara master: [TEST] bionic  https://review.openstack.org/58667010:18
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_swift master: [TEST] bionic  https://review.openstack.org/58667110:18
*** shyamb has quit IRC10:18
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tacker master: [TEST] bionic  https://review.openstack.org/58667210:18
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: [TEST] bionic  https://review.openstack.org/58667310:18
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_trove master: [TEST] bionic  https://review.openstack.org/58667410:18
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Execute security group setup against octavia_service_setup_host  https://review.openstack.org/58817610:34
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_octavia master: Remove octavia_requires_pip_packages  https://review.openstack.org/58817710:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819610:41
*** jamespage has quit IRC11:04
*** fmccarthy has quit IRC11:04
*** neith has quit IRC11:04
*** viks_ has quit IRC11:04
*** cshen has quit IRC11:08
*** ansmith has quit IRC11:11
*** priteau has joined #openstack-ansible11:18
*** shyamb has joined #openstack-ansible11:23
*** dave-mccowan has joined #openstack-ansible11:26
*** udesale has quit IRC11:27
*** canori01 has joined #openstack-ansible11:35
*** cshen has joined #openstack-ansible11:46
*** gillesMo has quit IRC12:08
jrossera bunch of the bionic role tests have failed lxc cache prep all the same like this http://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4/logs/host/lxc-cache-prep-commands.log.txt.gz12:08
odyssey4mejrosser: looks like the keys copied alright: http://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4/logs/ara-report/result/0a0a8ad5-a83a-4b7c-b071-30e3bb855d8d/12:11
evrardjpwhat's the bad news?12:11
evrardjpodyssey4me: ^12:11
odyssey4meit generated fine: http://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4/logs/ara-report/#12:11
odyssey4meand wrote to a file just fine: http://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4/logs/ara-report/result/4efc6710-5165-47c1-a4a7-823b95c765c2/12:11
jrosseroh hrrm i'm reading this a bit wrong then12:12
odyssey4meevrardjp: the bad news - the mq setup tasks can't be used with tags, which is a pretty important use-case12:12
odyssey4methe good news: I have a solution12:12
odyssey4methe bad news: it's going to require patches to every role again12:12
odyssey4mebusy testing with https://review.openstack.org/588196 and https://review.openstack.org/58819012:13
jrosserodyssey4me: this is where i started digging http://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4/job-output.txt.gz#_2018-08-02_10_32_08_02798212:13
odyssey4mejrosser: yeah, maybe the output format is wrong with bionic?12:14
*** EmilienM has left #openstack-ansible12:21
mgariepyhttp://logs.openstack.org/54/586654/4/check/openstack-ansible-functional-ubuntu-bionic/510e8e4//logs/host/lxc-cache-prep-commands.log.txt.gz << gpg: can't connect to the agent..12:23
*** shyamb has quit IRC12:26
evrardjpodyssey4me: I am reviewing this, with an extra idea, just a sec12:28
*** shyamb has joined #openstack-ansible12:29
*** shyamb has quit IRC12:29
*** shyamb has joined #openstack-ansible12:29
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server master: Revert "Use generic vars file for ubuntu"  https://review.openstack.org/58823012:31
odyssey4mejrosser evrardjp given that most roles have bionic working now, should we change the role test to voting - then we can disable that per role where it's not working12:32
jrosserthis'll do it of folks think its time https://review.openstack.org/#/c/587478/12:32
odyssey4mejrosser: perhaps make that one depend on https://review.openstack.org/58823012:33
odyssey4mebut yeah, I think it's time - otherwise I'll do more dumb patches which break it :p12:33
evrardjpodyssey4me: don't forget that the plan is to sync mq_setup12:33
jrosserodyssey4me: i'll add that12:33
odyssey4meevrardjp: yes, but if we sync those changes now it'll break every role12:33
odyssey4methat's why I did https://review.openstack.org/58819112:34
odyssey4methe roles all work as-is now, and I can change them all up to fix them to work with tags, then fix that sync file and rename it back12:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-tests master: Change bionic to be voting  https://review.openstack.org/58747812:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-tests master: Change bionic to be voting  https://review.openstack.org/58747812:34
evrardjpodyssey4me: well, I am not sure to understand that last patch12:35
odyssey4meevrardjp: looks like https://review.openstack.org/588196 worked - so if you're happy with the way that's done then I can release it for review and update the cinder one too12:35
evrardjphttps://review.openstack.org/#/c/588191/112:35
evrardjpodyssey4me: because it syncs when a change is done in the tests repo12:35
odyssey4meevrardjp: so if we rename that file, then the sync process will not find that file and do nothing anywhere12:35
*** ansmith has joined #openstack-ansible12:36
evrardjpagreed12:36
evrardjpbut the sync triggers on file change12:36
evrardjpso if no change, no problem12:36
odyssey4methe sync process checks for the existance of the file, then only syncs if it's present in the target repo12:36
evrardjp:p12:36
evrardjpagreed12:36
odyssey4meso, rather than blocking all changes to the tests repo - I figured we rename the file instead to prevent any further changes to tests proposing changes to the os_* roles12:37
evrardjpbut I was talking about the trigger condition12:37
evrardjpbut why not work in tests, and see how it goes?12:37
odyssey4mea rename of the file will not trigger a proposal to rename all the existing files of that name12:37
evrardjpI agree12:37
evrardjpI am just confused about why NOT working directly in tests repo12:37
evrardjpit's meant for that12:37
odyssey4mebecause a sync of that set of changes will break every single role12:38
odyssey4meevery role uses with_items, the new mechanism doesn't use with_items12:38
evrardjpoh yeah the includes will change12:38
evrardjpneed*12:38
evrardjp< dummy12:38
odyssey4meso yeah, every role needs to be updated first, then we can change the tests repo12:38
evrardjpor we break stuff12:39
evrardjpit's fine12:39
odyssey4meyep, and we broke enough stuff lately :p12:39
evrardjpit will be faster12:39
odyssey4meit's the price of progress ;)12:39
evrardjpthat's a lot of things12:39
evrardjpfor something that can just break and can be emergently fixed12:39
odyssey4meyep, that's fine - I'll likely figure out how to script it - given all the roles now have a uniform structure it should be reasonably ok12:39
evrardjpkeep in mind it will break only when it's merged12:39
odyssey4mewell, yes - another approach could be to push patches to all repositories and hold the patches back, then merge them all in one go12:40
evrardjpodyssey4me: yeah12:40
evrardjpthat's what I thought12:40
odyssey4meI thought that'd just create review list noise.12:41
odyssey4meEither approach is fine, although holding back patches also creates the possibility of merge conflicts while we wait for them all to be ready.12:41
odyssey4meI'd prefer to merge as we go, because that also allows more testing to happen - but I'll take whatever your preference is.12:42
*** shyamb has quit IRC12:42
*** shyamb has joined #openstack-ansible12:42
evrardjpnah it's fine I wanted to streamline the thing, and make it faster, not take too much mental energy for this12:45
evrardjpbut it's your patches :)12:45
odyssey4meevrardjp: if you have suggestions to make it faster, please suggest - either patch on top or review comment12:49
odyssey4meif it's just in the tests file, that'll be easy to modify later - but if it's not, then rather suggest now12:50
*** udesale has joined #openstack-ansible12:51
evrardjpI reviewed one of the two patches swift I think12:51
evrardjpI had an idea but it doesn't work12:52
odyssey4meok, will get back to that in a bit - heading to a meeting now12:58
*** shyamb has quit IRC13:03
*** vnogin has quit IRC13:10
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_keystone master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58447913:17
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_cinder master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58263213:19
*** vnogin has joined #openstack-ansible13:24
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_neutron master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58302113:40
*** savvas` has joined #openstack-ansible13:50
savvas`cloudnull: http://paste.openstack.org/show/727142/ we spoke about this yesterday (savvas). Same issue on fresh install, all nodes now13:51
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_nova master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58445013:52
*** DanyC has quit IRC13:59
*** DanyC has joined #openstack-ansible14:00
*** xteknoid has joined #openstack-ansible14:00
xteknoidhi. has anyone had an issue with the aio installation (queens) where instances dns lookups timeout? i have a full working openstack-ansible aio, bidirectional network access. Only problme is the ability to resolve dns in instances which just seems to hang / timeout14:02
*** DanyC has quit IRC14:04
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_heat master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58451314:07
*** spatel_ has joined #openstack-ansible14:10
openstackgerritJonathan Rosser proposed openstack/ansible-hardening master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825414:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-apt_package_pinning stable/queens: Add bionic to galaxy_info metadata  https://review.openstack.org/58825514:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-ceph_client master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825614:13
jrosserarrgh :(14:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-memcached_server master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825714:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-openstack_hosts master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825814:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_aodh master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825914:13
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_barbican master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826014:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ceilometer master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826114:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_cinder master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826214:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_designate master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826314:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_glance master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826414:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_gnocchi master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826514:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_heat master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826614:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_horizon master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826714:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826814:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_keystone master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826914:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827014:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_molteniron master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827114:14
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_neutron master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827214:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_nova master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827314:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_octavia master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827414:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_rally master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827514:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_sahara master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827614:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_swift master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827714:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tacker master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827814:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_tempest master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827914:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_trove master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828014:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-rsyslog_client master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828114:18
*** cshen has quit IRC14:19
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-rsyslog_server master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828314:19
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-openstack_openrc master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828414:19
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828514:20
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_container_create master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828714:20
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-galera_server master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828914:20
odyssey4mego jrosser go!14:23
jrossergross bash script time :)14:24
evrardjpjrosser: haha14:25
evrardjpjrosser: so they are all ready for voting?14:25
jrosserevrardjp: will i'd have said they were14:26
jrosserhowever theres a bunch now failing lxc cache prep which i've not been able to investigate yet14:26
evrardjplet's make them independant of lxc then! :p14:27
*** biffi has quit IRC14:28
odyssey4meevrardjp: asked a question in https://review.openstack.org/#/c/588196/2 - if you can answer, then I'll revise14:28
odyssey4meperfect, thanks evrardjp14:32
evrardjpodyssey4me: please read my last comment14:34
evrardjpfor completeness :)14:34
evrardjpnot that it matters, you got things right in the first commit IMO.14:34
evrardjpthe rest is just bonus14:34
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-apt_package_pinning master: Add bionic to galaxy_info metadata  https://review.openstack.org/58829714:35
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: correct typo  https://review.openstack.org/58830114:43
odyssey4meevrardjp: yep, will get to it a bit later again14:43
odyssey4mecloudnull: FYI, you were right - tags don't work, so I've worked up a proposed fix which unfortunately will require patches to all repositories again... but it's scriptable14:44
cloudnullbuu...14:44
cloudnullthats a bummer.14:44
cloudnullbut glad that its scriptable :)14:44
odyssey4memeh, better to have caught it now than later14:44
cloudnullhows it btw ?14:44
cloudnullRocky clouds are working BTW. my lab is now running the head of master. so progress :)14:45
savvas`cloudnull: http://paste.openstack.org/show/727142/ we spoke about this yesterday (savvas). Same issue on fresh install, all nodes now14:46
savvas`just wanted to provide you the additional feedback I owed you, what was it again that you wanted me to check / execute with blkid?14:48
cloudnullcan you run blkid -t PTTYPE=gpt /dev/loop014:49
cloudnullwhat does that return ?14:49
cloudnullthat command is failing, id be curious if the it has any useful return information14:49
savvas`no output14:51
savvas`on none of them, all 3 nodes14:51
mgariepyanyone has a playbook to shutdown the cluster ?14:52
*** tomtom002 has quit IRC14:56
mnasermorning everyone14:58
savvas`morning14:59
*** vnogin has quit IRC15:02
kaiokmomorning15:04
cloudnullmgariepy like a graceful power down  ?15:07
*** FrankZhang has joined #openstack-ansible15:07
cloudnullI guess you could run `lxc-system-down stop-containers` then just poweroff the host ?15:08
cloudnullsavvas` what does `losetup -a` show ?15:08
savvas`./dev/loop0: [2049]:6816686 (/var/lib/machines.raw)15:09
savvas`I think it is actually failing on the parted command15:09
savvas`rather than the blkid15:09
savvas`parted -s /dev/loop0 mklabel gpt15:09
savvas`Error: Partition(s) on /dev/loop0 are being used.15:09
*** spatel_ has quit IRC15:09
savvas`when I unmount that, it'll go back to the fsyncing error we had last night15:10
savvas`@cloudnull I am not sure how the loop device is otherwise used but I am almost certain that if I were to execute ceph-install.yml before running setup-infrastructure that I would probably not have this error, thoughts?15:12
openstackgerritMerged openstack/openstack-ansible-openstack_openrc master: Add bionic to galaxy_info metadata  https://review.openstack.org/58828415:13
openstackgerritMerged openstack/openstack-ansible-os_molteniron master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827115:13
bgmccollumxteknoid did you set `neutron_dnsmasq_dns_servers` in your user_variables.yml? -- https://github.com/openstack/openstack-ansible-os_neutron/blob/stable/queens/defaults/main.yml#L33715:13
cloudnullsavvas` that's the loop device for the containers, is there a loop device for the devices used by ceph?15:13
*** tomtom001 has joined #openstack-ansible15:14
savvas`well there are plenty available but the playbook seems to want to use loop015:14
cloudnullyea thats odd.15:15
savvas`and since it is executed after the tasks for setting up the containers, it crashes there15:15
openstackgerritMerged openstack/openstack-ansible-openstack_hosts master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825815:15
*** udesale has quit IRC15:15
cloudnullsavvas` looks like that task is using the devices variable15:16
cloudnullhttp://paste.openstack.org/show/727155/15:16
cloudnullfrom /etc/ansible/roles/ceph-ansible/roles/ceph-osd/tasks/check_gpt.yml15:17
mgariepycloudnull, yep gracefull15:17
cloudnullis that lo device defined somewhere in your user_variables ?15:17
mgariepycloudnull, powered off the cluster, but galera didn't liked it.15:17
cloudnullmgariepy maybe run something like `ansible -m shell -a 'systemctl stop neutron* nova* cinder* swift*; lxc-system-manager containers-stop; shutdown -P now' hosts` ?15:18
cloudnullthat would stop all of the process on the hosts.15:19
cloudnullhault the containers gracefully15:19
cloudnullthen power down15:19
mgariepyi did  a shutdown on all the containers15:19
*** cshen has joined #openstack-ansible15:19
mgariepybut anyway.15:19
savvas`don't think so cloudnull, the relevant bits pasted here: http://paste.openstack.org/show/727158/15:21
openstackgerritMerged openstack/ansible-hardening master: Add bionic to galaxy_info metadata  https://review.openstack.org/58825415:22
cloudnullsavvas` I see devices is commented out. should that be defined ?15:22
cloudnull with the actual devices your using for ceph ?15:23
savvas`cloudnull: I had that before but on one of the test nodes the drives were misplaced so my ceph drive there is different, which lead me to turn on auto discovery15:23
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_almanach master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831015:24
savvas`I couldn't find the documentation on how to define the drives per node rather than in general15:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_congress master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831115:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_masakari master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831215:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_monasca master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831315:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_searchlight master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831415:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_watcher master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831515:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_zaqar master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831615:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_zun master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831715:24
jrossermnaser: so much for patching with a script :)15:27
jrosserlet me just clean that all up....15:27
cloudnullsavvas` I have my drives defined in the openstack_user_config under host vars.15:28
savvas`similar to how I have it in user vars?15:28
savvas`I'd like to think I tried that and it gave me an error15:28
cloudnullhttps://gist.github.com/cloudnull/69bde16011d1ea94edf926252ea93933#file-openstack_user_config-yml-L115-L12715:28
*** shyambiradar has joined #openstack-ansible15:30
*** shyambiradar has quit IRC15:30
*** shyambiradar has joined #openstack-ansible15:31
savvas`cloudnull: http://paste.openstack.org/show/727160/ now it skipped the task, probably because GPT labels were already on the drives, but the playbook completed successfully15:37
savvas`so that solved it... So if you use osd_auto_discovery: true in your user_vars, this error I had turns up15:37
cloudnullI wonder why ceph-ansible auto discovers the loop device?15:38
savvas`http://paste.openstack.org/show/727161/ cluster seems healthy now15:38
cloudnullcool!15:38
savvas`very good question15:38
savvas`lol, made no sense to me15:38
cloudnullme neither, IMHO it should skip the loop device15:38
cloudnullandymccr ^15:38
cloudnullbut hey, I'm glad its fixed :)15:39
savvas`yep me too, appreciate the help!15:39
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-apt_package_pinning master: List ubuntu releases in release order  https://review.openstack.org/58829715:41
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-ceph_client master: List ubuntu releases in release order  https://review.openstack.org/58825615:42
openstackgerritLogan V proposed openstack/openstack-ansible-plugins master: Fix config_template to work with Ansible 2.6  https://review.openstack.org/58832215:42
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-memcached_server master: List ubuntu releases in release order  https://review.openstack.org/58825715:42
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-rsyslog_client master: List ubuntu releases in release order  https://review.openstack.org/58828115:43
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-rsyslog_server master: List ubuntu releases in release order  https://review.openstack.org/58828315:44
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-lxc_hosts master: List ubuntu releases in release order  https://review.openstack.org/58828515:45
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-galera_server master: List ubnutu releases in release order  https://review.openstack.org/58828915:45
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-galera_server master: List ubuntu releases in release order  https://review.openstack.org/58828915:48
*** hamzy has quit IRC15:49
*** shyambiradar has quit IRC15:54
openstackgerritLogan V proposed openstack/openstack-ansible stable/queens: Update ceph galaxy cleanup for monolithic repo  https://review.openstack.org/58832415:56
*** DanyC has joined #openstack-ansible16:04
*** cshen has quit IRC16:13
*** spotz has quit IRC16:18
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_keystone master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58447916:25
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_cinder master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58263216:27
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_neutron master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58302116:29
* jrosser struggles to understand why this fails the check but not locally http://logs.openstack.org/07/586607/4/check/openstack-ansible-deploy-aio_lxc-ubuntu-bionic/22b95de/logs/host/lxc-cache-prep-commands.log16:29
*** gyee has joined #openstack-ansible16:35
logan-jrosser: https://stackoverflow.com/questions/46673717/gpg-cant-connect-to-the-agent-ipc-connect-call-failed maybe?16:39
logan-seems unlikely that there'd be a gpg agent running on the gate nodes but who knows16:40
*** tosky has quit IRC16:41
odyssey4mewell, that's being run inside the chroot16:45
logan-maybe relevant: https://github.com/Microsoft/WSL/issues/328616:49
logan-well maybe not actually16:50
jrosserit works for me on a bionic virtualbox17:01
jrosserbut as the gate logs are a bit fubar i'm not actually sure if the same gpg stuff is installed17:02
openstackgerritLogan V proposed openstack/openstack-ansible-lxc_hosts master: Add verbose flag to container prep apt-key add  https://review.openstack.org/58834317:02
*** vnogin has joined #openstack-ansible17:02
* jrosser runs for train - will look at this again later17:03
*** vnogin has quit IRC17:07
*** hamzy has joined #openstack-ansible17:11
*** cshen has joined #openstack-ansible17:16
mnaserhas anyone seen rsyslog spamming journals with this?17:23
mnaser rsyslogd[705557]: action 'action 2' resumed (module 'builtin:omfile') [v8.24.0 try http://www.rsyslog.com/e/2359 ]17:23
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_cinder master: Add target_helper to backend_defaults  https://review.openstack.org/58460417:34
cloudnullmnaser i've not17:35
cloudnullan upraded environment ?17:35
jrossercan i download or recreate the infra bionic image somehow?17:38
jrosserso i can test exactly the same environment here17:39
logan-yes there is jrosser let me find hte link17:40
openstackgerritJacob Wagner proposed openstack/openstack-ansible-ops master: Add ability to deploy Ceph into a Multinode AIO  https://review.openstack.org/58500317:42
openstackgerritMerged openstack/openstack-ansible-ops master: correct typo  https://review.openstack.org/58830117:42
logan-jrosser: https://nb01.openstack.org/images/17:45
logan-and https://nb02.openstack.org/images/17:46
jrosserlogan-: ah excellent, looks loke some logs too so maybe a clue if anything has changed17:48
logan-yep!17:48
jrosserpreviously it got all the way to an inevitable tempest fail, so the lxc cache prep has worked in the last 24hrs or so17:50
logan-yea agreed, we had jobs passing until a day or so ago17:55
*** poopcat has joined #openstack-ansible17:55
jrosserplus ive not seen anything obvious committed that would break this17:57
jrosserperhaps theres something lurking in another broken ubuntu-18.04 -> ubuntu.yml17:57
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible master: Cleanup duplcate keys  https://review.openstack.org/58835418:01
*** electrofelix has quit IRC18:02
*** pathcl_ has quit IRC18:15
bgmccollumanyone been able to figure out whats going on with cinder in master? -- this is the tail of the cinder-scheduler.log...relevant? -- http://paste.openstack.org/show/727170/18:28
bgmccollumi know there was some shuffling of MQ setup in / out of roles...18:29
mnasercloudnull: thanks for pushing up that rebase on os_cinder18:30
cloudnullbgmccollum is that an AIO ?18:31
bgmccollumgate18:31
cloudnullok.18:32
bgmccollumhttp://logs.openstack.org/29/587229/1/check/openstack-ansible-deploy-aio_basekit-opensuse-423/009c7fd/logs/openstack/aio1_cinder_api_container-f87e97b0/cinder/cinder-scheduler.log18:32
mgariepymnaser, i've seen something similar on a release upgrade from 14.04 > 16.0418:32
mgariepybut not spammy, only a couple entry per host.18:32
mgariepynot osa managed.18:32
cloudnullbgmccollum likely due to all of the MQ config movement18:32
cloudnull^ odyssey4me18:33
odyssey4mebgmccollum: seen that and tried to figure it out with jrosser but haven't yet18:35
odyssey4meI know with certainty that https://review.openstack.org/585504 works properly.18:35
odyssey4meI've got a build up to work out how to turn off the one test that keeps failing for tempest so that we can merge patches again while we work the issue out.18:37
odyssey4meUnless someone can pinpoint the issue.18:37
*** cshen has quit IRC18:37
*** cshen has joined #openstack-ansible18:41
jrosserodyssey4me: gimme 30mins or so and i'll look at todays AIO and see if cinder started18:42
odyssey4mecloudnull: could you take a peek at https://review.openstack.org/#/c/588196/2 and see if you have any further suggestions beyond what evrardjp has put into review18:42
odyssey4meI've verified that both the tags implemented there work.18:42
odyssey4meany cores, please review https://review.openstack.org/588230 to also unblock things18:48
mgariepyodyssey4me, done.18:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Disable tempest.scenario.test_minimum_basic temporarily  https://review.openstack.org/58836018:55
odyssey4methanks mgariepy18:55
odyssey4melogan- cloudnull mgariepy evrardjp I feel that after weeks of struggling, with many patches being held back, perhaps it's time to regroup and remove the blocker for now. See https://review.openstack.org/58836018:56
odyssey4memnaser ^18:56
odyssey4mePerhaps we'll switch to using mnaser's suggestion of smoke tests, but we still need time to work out those failures too.18:56
odyssey4mepart of the motivation is because a ton of the patches waiting to merge resolve known issues - like broken gate log collection, and broken cinder mq config19:00
openstackgerritByron McCollum proposed openstack/openstack-ansible master: Fix Bootstrap AIO Passing Multiple BOOTSTRAP_OPTS  https://review.openstack.org/58722919:01
jamesdentonBefore i go down some rabbit hole, does OSA provide any NUMA CPU Pinning tunables?19:02
odyssey4mejamesdenton: typically we try to encourage using the config overrides where possible, rather than introduce new tunables19:04
*** vnogin has joined #openstack-ansible19:04
odyssey4mehttps://docs.openstack.org/openstack-ansible/latest/reference/configuration/using-overrides.html#overriding-openstack-configuration-defaults19:04
jamesdentonok fair enough. thank you19:05
odyssey4mejamesdenton: *if* some sort of wiring of sensible default makes sense then we can either hard code it into the template with a conditional, or add tunables19:05
odyssey4mebut more vars makes things harder to figure out19:05
odyssey4methat should have read wiring *or* sensible...19:06
jamesdentonright, i got ya19:06
odyssey4meconfig overrides actually override tunables anyway19:06
jamesdentonthanks Jesse19:08
openstackgerritAndy Smith proposed openstack/openstack-ansible-os_nova master: Setup oslo.messaging extra packages for optional drivers  https://review.openstack.org/58445019:12
*** mgariepy has quit IRC19:14
jrosserodyssey4me: cinder-scheduler seems to be up19:14
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Add tempest test blacklist capability  https://review.openstack.org/58836219:15
odyssey4med34dh0r53: I think https://review.openstack.org/588362 may be useful to you.19:15
*** savvas has joined #openstack-ansible19:23
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Add tempest test blacklist capability  https://review.openstack.org/58836219:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Add tempest test blacklist capability  https://review.openstack.org/58836219:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest master: Add tempest test blacklist capability  https://review.openstack.org/58836219:24
odyssey4mesorry for the spam, changes method part way through and didn't back out properly19:25
jrosserodyssey4me: this AIO fails swift though on [os_swift : Verify md5sum of builder files]19:25
odyssey4mejrosser: that does happen from time to time, the thoughts on that is that it's due to resource contention - and it's not been consistent, so it's hard to truly figure out19:26
d34dh0r53odyssey4me: that looks perfect19:26
d34dh0r53jrosser: I've been hitting that as well, quite frequently lately19:26
odyssey4meI'd like to discuss possibly reconfiguring the AIO to not bother with swift any more, and perhaps focus on NFS/Ceph as default tests instead.19:27
jrosserodyssey4me: ok no problem - cinder looks good but tbh i'd like to run this again, i had a bit of a fight applying your patches (my own fault) so it didnt get a clean run at everything19:27
odyssey4med34dh0r53: orly? any consistency in it - or just transient?19:27
bgmccollumjrosser can you check the cinder-scheduler.log for any errors?19:27
d34dh0r53odyssey4me: my current mnaio is running swift now, I'll let you know19:28
jrosserbgmccollum: i think you posted the same errors that odyssey4me and i were looking into yesterday?19:28
d34dh0r53odyssey4me: it's been rather consistent for me (as long as ansible doesn't run out of memory first)19:28
bgmccollumjrosser gotcha...19:28
jrosseri don't have those in this build19:28
bgmccollumis it this patch you were testing? -- https://review.openstack.org/#/c/585504/19:29
jrosserhttps://review.openstack.org/#/c/586607/ then https://review.openstack.org/#/c/585504/ then https://review.openstack.org/#/c/586226/19:31
bgmccollum👍19:31
openstackgerritMichael Vollman proposed openstack/ansible-config_template master: Compare dict vars to determine changed  https://review.openstack.org/58253219:32
savvasany known issues with out of the box OA Queens deployment for live migration? (Nodes running Ubuntu, not CentOS)19:33
d34dh0r53odyssey4me: my latest MNAIO just passed, but it's RPC-O stable/queens, staring an OSA one now, I'll let you know19:33
openstackgerritMerged openstack/openstack-ansible-rabbitmq_server master: Revert "Use generic vars file for ubuntu"  https://review.openstack.org/58823019:34
openstackgerritMerged openstack/openstack-ansible-memcached_server master: List ubuntu releases in release order  https://review.openstack.org/58825719:43
openstackgerritMerged openstack/openstack-ansible-rsyslog_client master: List ubuntu releases in release order  https://review.openstack.org/58828119:44
openstackgerritMerged openstack/openstack-ansible-rsyslog_server master: List ubuntu releases in release order  https://review.openstack.org/58828319:44
openstackgerritMerged openstack/openstack-ansible-apt_package_pinning master: List ubuntu releases in release order  https://review.openstack.org/58829719:44
*** vollman has quit IRC19:45
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819619:46
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder master: Allow tags to be used for MQ tasks  https://review.openstack.org/58819019:49
*** vollman has joined #openstack-ansible19:50
mnaserodyssey4me: https://review.openstack.org/#/c/587508 fyi, be good if you +A that as you did most/alot of the work for the sync stuff!19:50
openstackgerritMerged openstack/openstack-ansible-ceph_client master: List ubuntu releases in release order  https://review.openstack.org/58825619:50
openstackgerritMerged openstack/openstack-ansible-lxc_hosts master: List ubuntu releases in release order  https://review.openstack.org/58828519:50
odyssey4memnaser: haha lol19:53
odyssey4meoh crap, that's going to cause proposal bot spam19:53
mnaserodyssey4me: which is why i needed us to be sure that change was clean, haha19:54
mnaserweekend is coming up soon19:54
mnaserwe can leverage an empty gate19:54
mnaser:P19:54
odyssey4memnaser: well, discovered an important operator use-case which that didn't work with - so that whole file is going to change: https://review.openstack.org/58819619:55
odyssey4methat is, assuming everyone is ok with the new pattern19:55
*** savvas has quit IRC19:58
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: [WIP] Engage afterburner during git-clone [1]  https://review.openstack.org/58837220:03
jrosserodyssey4me: ^ i reckon you could get good mileage with that in the repo build too20:03
odyssey4meafterburner :)20:04
odyssey4meFYI that's not useful for the role clone in openstack-infra, but is perhaps outside20:05
odyssey4mewithin infra, the repositories are already there - they just get updated20:05
jrosserquite happy to have a sensible commit message if its useful :)20:07
jrosserit takes really very long indeed during boostrap-ansible and repo stuff to clone the repos20:07
jrosserwe can probably take 10mins off the build time20:08
mnaserhttps://review.openstack.org/#/c/584604/20:09
jrosseranyway i just throw that up becasue i'm out pretty much till next week and wanted people to get a look / comment20:09
mnaseranyone has ideas on how to inject $PATH without messing too much things?20:09
jrossermnaser: in an ansible task?20:10
mnaseri know we can add it to systemd env but the problem with it is that we might miss another path which is part of the default path and breaking something lse20:10
odyssey4mejrosser: yeah, thanks for sharing it!20:10
mnasercause if we do indeed add it, we might have this weird situation that we missed a path20:10
mnaserand i dont think $PATH expansion works in systemd unit files20:11
mnaserhttps://unix.stackexchange.com/questions/347873/set-path-for-a-systemd-unit20:11
mnaserwrapping in bash feels dirty20:11
mnaseri cant think of another way than hardcoding /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin :<20:12
*** savvas has joined #openstack-ansible20:12
odyssey4meok, I'm out for the night folks - chat to y'all tomorrow!20:13
jrossermnaser: https://www.freedesktop.org/software/systemd/man/systemd.exec.html#PassEnvironment=20:14
jrosseris that the sort of thing?20:14
mnaserhmm thats interesting20:15
openstackgerritJonathan Rosser proposed openstack/openstack-ansible master: [WIP] Engage afterburner during git-clone  https://review.openstack.org/58837220:20
kaiokmomnaser: any thoughts here? http://logs.openstack.org/33/586933/3/gate/openstack-ansible-functional-ubuntu-bionic/49e1e4e/job-output.txt.gz#_2018-08-02_20_06_56_18183420:24
openstackgerritMohammed Naser proposed openstack/openstack-ansible-os_cinder master: Add target_helper to backend_defaults  https://review.openstack.org/58460420:24
kaiokmoI'm also facing this same fail here http://logs.openstack.org/32/586932/2/check/openstack-ansible-functional-ubuntu-bionic/e45ba51/job-output.txt.gz#_2018-08-02_14_59_12_06264820:25
mnaserkaiokmo: when you run into these things where you have to look is in the cache prep log20:25
mnaserwhich you can find in logs/host/lxc-cache-prep-commands.log.txt.gz20:25
mnaserin this case -- right here -- http://logs.openstack.org/33/586933/3/gate/openstack-ansible-functional-ubuntu-bionic/49e1e4e/logs/host/lxc-cache-prep-commands.log.txt.gz20:25
mnasergpg: can't connect to the agent: IPC connect call failed20:26
mnaserso it looks like some sort of transient issue, but it looks like logan- proposed this to help make with visiblity on why it fails there20:27
mnaserhttps://review.openstack.org/#/c/588343/20:27
mnaserkaiokmo: so i would recheck and cross my fingers, till we find out whats the cause of this20:27
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Switch to using project-templates  https://review.openstack.org/58838020:28
mnaseryeah it's happening all over bionic20:28
jrosseri've still not found why thats happening20:28
kaiokmoah, I see it now20:28
jrosserif anyone fancies a dig then please do as i've got little time till next week now20:28
mnaserjrosser: https://review.openstack.org/#/c/588343/20:28
mnaseri think that might start giving us some more help as to what gets added and what fails20:29
openstackgerritMerged openstack/openstack-ansible-os_zun master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831720:29
kaiokmothanks mnaser20:29
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_magnum master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827020:29
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_octavia master: Add bionic to galaxy_info metadata  https://review.openstack.org/58827420:30
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_searchlight master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831420:33
logan-jrosser: surprised to see bionic pass on https://review.openstack.org/58834320:35
jrosserlogan-: yes its not every time20:35
logan-Ah20:35
jrosseri wonder if it installs and start gpg agent but it's all a bit racy20:35
jrossermnaser: i got a bunch of others here to add the CI https://review.openstack.org/#/q/topic:bionic-check+(status:open+OR+status:merged)20:36
jrosseri've just added some depends-on to the bionic-meta reviews20:37
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_zaqar master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831620:38
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_almanach master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831020:39
*** ansmith has quit IRC20:39
*** hamzy has quit IRC20:40
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_watcher master: Add bionic to galaxy_info metadata  https://review.openstack.org/58831520:43
*** vnogin has quit IRC20:45
jrosserlogan-: if you look here http://zuul.openstack.org/builds.html and filter for openstack-ansible-functional-ubuntu-bionic you see what hit/missed20:46
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-ops master: Update index retention policy to ensure its an int  https://review.openstack.org/58838920:47
*** tosky has joined #openstack-ansible20:50
*** savvas has quit IRC20:50
d34dh0r53lol, java-isms20:51
jrosserfalse = 5, sure, why not!20:53
d34dh0r53hah20:57
d34dh0r53well, it has 5 letters, so yeah, 5 let's go with that20:57
mnaseryou know what that reminds me of20:57
mnaserhttps://www.destroyallsoftware.com/talks/wat20:57
d34dh0r53:) one of my favorite talks20:58
mnaseri feel like i should do a 'wat' lightning talk but in openstack context20:58
*** vnogin has joined #openstack-ansible20:59
*** priteau has quit IRC20:59
openstackgerritMerged openstack/openstack-ansible-os_cinder master: Convert role to use a common systemd service role  https://review.openstack.org/57481721:08
openstackgerritMerged openstack/openstack-ansible stable/queens: Rename designate_galera_database_name  https://review.openstack.org/58793121:09
*** FrankZhang_ has joined #openstack-ansible21:16
*** dcdamien has quit IRC21:17
*** FrankZhang has quit IRC21:18
*** FrankZhang_ has quit IRC21:22
*** cshen has quit IRC21:23
*** dmsimard has quit IRC21:24
openstackgerritJonathan Rosser proposed openstack/openstack-ansible-os_ironic master: Add bionic to galaxy_info metadata  https://review.openstack.org/58826821:25
*** vnogin has quit IRC21:26
*** dmsimard has joined #openstack-ansible21:32
*** ansmith has joined #openstack-ansible21:33
*** vnogin has joined #openstack-ansible21:41
*** vnogin has quit IRC22:08
*** cshen has joined #openstack-ansible22:10
*** cshen has quit IRC22:14
*** vnogin has joined #openstack-ansible22:19
openstackgerritDavid Wilde proposed openstack/openstack-ansible-ops master: Deploy ELK in MNAIO  https://review.openstack.org/58661222:34
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-nspawn_hosts master: Add route when required on the local table  https://review.openstack.org/58840622:39
cloudnulld34dh0r53 jrosser :)22:40
*** lkoranda has quit IRC22:46
*** lkoranda has joined #openstack-ansible22:49
*** tosky has quit IRC22:59
openstackgerritMerged openstack/openstack-ansible stable/queens: Replace default pip index check with upper constraints check  https://review.openstack.org/58786023:02

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