Monday, 2016-07-25

*** rstarmer has quit IRC00:13
*** omiday has joined #openstack-ansible00:19
*** asettle has joined #openstack-ansible00:21
*** elo has quit IRC00:21
*** david-lyle_ has joined #openstack-ansible00:22
*** elo has joined #openstack-ansible00:24
*** asettle has quit IRC00:26
*** david-lyle_ has quit IRC00:27
*** eric_lopez has joined #openstack-ansible00:32
*** elo has quit IRC00:33
*** elo has joined #openstack-ansible00:33
*** eric_lopez has quit IRC00:34
*** chandanc__ has joined #openstack-ansible00:42
*** thorst has joined #openstack-ansible00:50
*** chandanc__ has quit IRC00:53
*** thorst has quit IRC00:58
*** wadeholl_ has joined #openstack-ansible01:04
*** wadeholl_ has quit IRC01:04
*** wadeholler has quit IRC01:06
*** thorst has joined #openstack-ansible01:13
*** thorst has quit IRC01:15
*** wadeholler has joined #openstack-ansible01:20
*** wadeholler has quit IRC01:20
*** wadeholler has joined #openstack-ansible01:20
*** v1k0d3n has joined #openstack-ansible01:25
v1k0d3nthis may be a bit of a long shot, but does anyone do development with a muli-node cluster, with a single NIC (or a primary bond)?01:27
*** elo has quit IRC01:27
*** elo has joined #openstack-ansible01:28
v1k0d3nreason i ask is because i'm looking for any possible pitfalls with using keepalive/haproxy...and wondering if anyone has a sample config for user_var/os_user_config01:28
*** wadeholler has quit IRC01:52
*** abelur has joined #openstack-ansible01:56
*** winggundamth has quit IRC02:11
*** thorst has joined #openstack-ansible02:16
*** winggundamth has joined #openstack-ansible02:20
*** thorst has quit IRC02:24
*** david-lyle_ has joined #openstack-ansible02:25
*** david-lyle_ has quit IRC02:29
*** retreved_ has quit IRC02:33
*** woodard has quit IRC02:39
*** elo has quit IRC02:43
*** elo has joined #openstack-ansible02:43
*** thorst has joined #openstack-ansible03:25
*** thorst has quit IRC03:30
*** elo has quit IRC03:33
*** elo has joined #openstack-ansible03:34
*** v1k0d3n has quit IRC03:45
*** elo has quit IRC04:21
*** elo has joined #openstack-ansible04:23
*** thorst has joined #openstack-ansible04:26
*** david-lyle_ has joined #openstack-ansible04:26
*** david-lyle_ has quit IRC04:32
*** thorst has quit IRC04:35
*** elo has quit IRC04:43
*** elo has joined #openstack-ansible04:43
*** elo has quit IRC04:55
*** elo has joined #openstack-ansible04:56
*** janki has joined #openstack-ansible05:17
*** abelur__ has joined #openstack-ansible05:20
*** abelur has quit IRC05:24
*** eric_lopez has joined #openstack-ansible05:24
*** elo has quit IRC05:25
*** thorst has joined #openstack-ansible05:33
*** thorst has quit IRC05:40
*** eric_lopez has quit IRC05:50
*** eric_lopez has joined #openstack-ansible05:50
*** McMurlock1 has joined #openstack-ansible05:58
*** eric_lopez has quit IRC06:08
*** elo has joined #openstack-ansible06:08
openstackgerritNeill Cox proposed openstack/openstack-ansible: [WIP] Add Horizon Ironic dashboard plugin  https://review.openstack.org/33250806:18
*** elo has quit IRC06:19
*** elo has joined #openstack-ansible06:19
odyssey4mejmccrory automagically cloudnull d34dh0r53 stevelle mattt hughsaunders andymccr evrardjp when you're in https://review.openstack.org/346386 / https://review.openstack.org/346388 (backports) need urgent review06:21
*** v1k0d3n has joined #openstack-ansible06:25
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Revert "Allow a deployer to disable container restarts"  https://review.openstack.org/34660806:27
*** david-lyle_ has joined #openstack-ansible06:29
*** david-lyle_ has quit IRC06:33
*** jascott1_tmp has joined #openstack-ansible06:33
*** rstarmer has joined #openstack-ansible06:34
openstackgerritMerged openstack/openstack-ansible-repo_build: Fix setting of upper_constraints fact  https://review.openstack.org/34638606:36
*** thorst has joined #openstack-ansible06:37
*** pcaruana has joined #openstack-ansible06:37
*** asettle has joined #openstack-ansible06:38
*** jiteka has joined #openstack-ansible06:38
*** rstarmer has quit IRC06:39
odyssey4mejmccrory automagically cloudnull d34dh0r53 stevelle mattt hughsaunders andymccr evrardjp mhayden when you're in https://review.openstack.org/346608 reverts a patch which causes container restarts unexpectedly - please review asap06:41
odyssey4memancdaz ^ note that we'll have to rather iron that functionality out in master and properly test it before re-considering it for backport06:41
openstackgerritMerged openstack/openstack-ansible-os_ironic: Define keystone region name.  https://review.openstack.org/34638706:41
odyssey4mewe can't introduce bad behaviour like that in a stable branch06:42
*** jiteka has quit IRC06:43
*** thorst has quit IRC06:44
openstackgerritNeill Cox proposed openstack/openstack-ansible-os_horizon: Enable Horizon Ironic dashboard plugin  https://review.openstack.org/33250706:45
odyssey4meneillc some feedback in your patches :)07:02
neillcsaw that - thank you looking at it now07:02
*** eric_lopez has joined #openstack-ansible07:03
*** elo has quit IRC07:03
odyssey4meneillc make sense?07:09
neillcodyssey4me: just to make sure I'm clear on this: os_horizon role should include a default for horizon_enable_ironic and openstack-ansible should set that same variable based on the length of  groups['horizon_all']07:09
neillcwas just typing that question when you responded :-)07:09
odyssey4mebased on the length of ironic_all07:09
neillcsorry that's what I meant07:09
odyssey4methe plugin must be enabled if there are ironic hosts07:09
neillcyep07:10
odyssey4meyes - the convention is that the role is dumb, and the group_vars make them smart07:10
neillccool07:10
neillcI shall make those changes and follow Kevin's suggestion re pypi07:10
odyssey4methe role does what it's told, the group_vars know the environment, the inventory and they tie things together in a sensible way07:11
odyssey4mekevin's suggestion re pypi?07:11
odyssey4mefor now I don't think the role should have the git bits for the additional ui components... the plugins are on pypi, that will do07:11
neillche's seggesting using pypi rather than git07:12
odyssey4meheh, that was me07:12
odyssey4mein the role you are already using pypi07:12
neillcI think I need to go have a coffee07:12
odyssey4mehaha07:13
odyssey4meif you can tweak the patches per my suggestions, I think you're pretty much good to go07:13
neillcbtw - did you see my email re searchlight? Are you happy for me to poke at that next? I know sigmavirus has started a role but noit sure how much osa has changed in the meantime.07:13
odyssey4meoh yeah - one more thing, the horizon patch should also include a release note07:13
neillcok07:13
odyssey4mesimilar to https://github.com/openstack/openstack-ansible-os_horizon/blob/master/releasenotes/notes/lbaasv2-horizon-panel-8f99026b025ca2fd.yaml is fine07:14
*** elo has joined #openstack-ansible07:15
*** mikelk has joined #openstack-ansible07:15
odyssey4meneillc yeah, I'll reply to your email today07:15
*** eric_lopez has quit IRC07:16
odyssey4meafk while I relocate07:17
*** elo has quit IRC07:19
*** elo has joined #openstack-ansible07:19
*** eric_lopez has joined #openstack-ansible07:26
*** elo has quit IRC07:26
evrardjpgood morning everyone07:28
openstackgerritNeill Cox proposed openstack/openstack-ansible: Add Horizon Ironic dashboard plugin  https://review.openstack.org/33250807:29
openstackgerritNeill Cox proposed openstack/openstack-ansible: Add Horizon Ironic dashboard plugin  https://review.openstack.org/33250807:31
*** jascott1_tmp has quit IRC07:32
*** asettle has quit IRC07:32
mancdazodyssey4me wut07:35
evrardjpinteresting thing happened apparently07:36
evrardjpthings07:36
*** eric_lopez has quit IRC07:37
*** elo has joined #openstack-ansible07:37
openstackgerritNeill Cox proposed openstack/openstack-ansible-os_horizon: Enable Horizon Ironic dashboard plugin  https://review.openstack.org/33250707:39
*** Oku_OS-away is now known as Oku_OS07:41
*** thorst has joined #openstack-ansible07:43
*** thorst has quit IRC07:50
evrardjpodyssey4me: about this: https://review.openstack.org/#/c/346608/ wouldn't be easier to fix the module instead?07:52
evrardjpfor the conversation about uwsgi you had on last week meeting, due to the way federation works for now, the move to uwsgi seems a radical change. Deployers will have to adapt, and that's something that should be done carefully07:57
evrardjpIMO07:57
evrardjpnot in a few days07:57
evrardjpwith a proper coordination with keystone ppl07:58
evrardjpcloudnull: you're right about callback plugins, I had some quirks in the past too08:02
evrardjpcloudnull: jmccrory about the slowdown on some tasks, it's maybe worth a further analysis - whether it's connection plugin or the fact caching08:04
evrardjpcloudnull: you had access on a cluster that can be useful to test different connection plugins? I'd be happy to know the speed/reliability of paramiko 2/other connection plugins.08:05
evrardjpalso could someone handle the bug triage of tomorrow? I'm gonna be in the train :/08:09
*** eric_lopez has joined #openstack-ansible08:15
*** elo has quit IRC08:16
*** david-lyle_ has joined #openstack-ansible08:31
*** david-lyle_ has quit IRC08:36
odyssey4memancdaz https://review.openstack.org/34606408:37
*** electrofelix has joined #openstack-ansible08:40
odyssey4memancdaz another thing that will affect upgrades: https://review.openstack.org/34633508:42
openstackgerritMerged openstack/openstack-ansible: Fix setting of upper_constraints fact  https://review.openstack.org/34638808:43
*** thorst has joined #openstack-ansible08:48
*** thorst has quit IRC08:54
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Update all SHAs for 13.3.0  https://review.openstack.org/34230908:54
*** vnogin has joined #openstack-ansible09:02
odyssey4meevrardjp with regards to https://review.openstack.org/346608 fixing the module is an upstream activity, I've registered an issue for it https://github.com/ansible/ansible-modules-extras/issues/261209:09
odyssey4megetting it fixed upstream will not help liberty or mitaka, as the ansible version there is 1.909:09
evrardjpmakes sense09:10
odyssey4meso basically we either need to do https://review.openstack.org/346608 or get https://review.openstack.org/346064 into a working state09:10
evrardjpnot tracked in the commit message/code though09:10
odyssey4meevrardjp yes, that patch is a revert09:10
evrardjpyup I saw that09:11
odyssey4methe issue can be noted once that forward ports to master09:11
evrardjpfine for me09:11
odyssey4meevrardjp any thoughts on https://review.openstack.org/346335 ? I'd like to establish it as a consistent pattern across all roles09:12
odyssey4methis would allow finer control of when packages are updated09:12
evrardjpI was enclined to have a state present in the past09:13
*** elo has joined #openstack-ansible09:13
*** eric_lopez has quit IRC09:13
evrardjpI'll check the behavior with keepalived, because it has the same kind of impact09:14
evrardjpthis way we can have a consistent *generic* pattern09:14
odyssey4meyep09:14
evrardjpwe didn't have a generic one, it was keepalived_use_latest_stable, which defaults to false09:15
evrardjphttps://github.com/evrardjp/ansible-keepalived/blob/master/tasks/keepalived_install_apt.yml#L6709:16
evrardjphere we have the "upgrade" issue to manage, too09:16
evrardjpI'll think about it, in depth09:16
odyssey4meno worries, when I do the group_var I'll connect those two vars together09:16
odyssey4mebut yes - the intent is that in upgrades we then set the var to upgrade the packages09:17
odyssey4meor if a deployer wants to do package updates another time, that can be done09:17
odyssey4mebut right now we have a mix of 'present' and 'latest' which is bad09:17
odyssey4mewhenever you run any playbook, the mariadb client gets updated09:18
evrardjpmy general opinion was: it's better to have it "present" at all times, to avoid restarts. Latest can be useful for non-ppa packages (stable LTS packages), this way you bring security updates and restart BECAUSE it's useful09:19
evrardjpthe rest is a more complex topic09:19
odyssey4mein other news, this works very nicely: https://review.openstack.org/34432909:19
odyssey4meyes exactly, this is why I want to set the pattern so that present is how we do it everywhere and we can control when 'latest' is applied09:19
evrardjpodyssey4me: we can have another "overarching" preference (like the debug one), which sets for every role the package state, and define present as default09:20
odyssey4meevrardjp yes, that's the intent09:20
odyssey4mebut first all the roles need the var09:21
mancdazodyssey4me I'm spinning a liberty to see why cloudnull might have made the lineinfile change like that09:21
odyssey4methen I'll do a patch for the group_vars to get a value from a single var in all09:21
odyssey4methanks mancdaz09:21
evrardjpodyssey4me: agreed09:21
mancdazhe might be trying to take care of additional whitespace surrounding the lines09:21
evrardjpit's possible to fine tune stuff with group vars now odyssey4me09:21
evrardjpyou can assign some hosts to a group that has apt to latest09:22
evrardjpso you can fine tune which one needs an upgrade of the package09:22
evrardjpwithout setting a generic var for all cases09:22
openstackgerritWang Qing wu proposed openstack/openstack-ansible-security: Add audit rules to support ppc64le architecture.  https://review.openstack.org/34666609:23
evrardjpodyssey4me: reviewing your other link09:23
odyssey4meevrardjp the generic var is for ease of use, so that if a deployer wants to upgrade all packages across all roles, it only takes setting one var with -e09:23
evrardjpoh ok cool09:23
evrardjpodyssey4me: yes, we keep that09:24
evrardjpbut in addition to it09:24
evrardjpyou can fine tune per group09:24
odyssey4mebut yes, I hear you regarding the application of group_vars and special settings09:24
evrardjpI'll explain to you with a white board :D09:24
odyssey4metime for me to get some coffee and breakfast09:25
evrardjpenjoy :D09:30
*** david-lyle_ has joined #openstack-ansible09:32
*** david-lyle_ has quit IRC09:36
evrardjpodyssey4me: may I edit https://review.openstack.org/#/c/346335 by adding quotes for the "present" line ?09:48
*** javeriak has joined #openstack-ansible09:49
*** hwcomcn has joined #openstack-ansible09:50
*** hwcomcn has quit IRC09:51
*** hwcomcn has joined #openstack-ansible09:52
*** thorst has joined #openstack-ansible09:52
*** hwcomcn has quit IRC09:55
odyssey4meevrardjp not yet - let's get more feedback first09:58
*** hwcomcn has joined #openstack-ansible09:58
evrardjpok09:58
*** thorst has quit IRC09:59
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Add Horizon Ironic dashboard plugin  https://review.openstack.org/33250810:00
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon: Enable Horizon Ironic dashboard plugin  https://review.openstack.org/33250710:02
*** hwcomcn has quit IRC10:03
*** hwcomcn has joined #openstack-ansible10:04
*** hwcomcn has quit IRC10:05
*** hwcomcn has joined #openstack-ansible10:06
*** gparaskevas has joined #openstack-ansible10:07
*** asettle has joined #openstack-ansible10:14
*** gparaskevas has quit IRC10:27
*** wadeholler has joined #openstack-ansible10:32
*** wadeholler has quit IRC10:33
*** wadeholler has joined #openstack-ansible10:33
*** javeriak has quit IRC10:36
*** javeriak has joined #openstack-ansible10:36
openstackgerritMerged openstack/openstack-ansible: Move LXC AppArmor profile setting to the inventory  https://review.openstack.org/34432910:42
*** johnmilton has quit IRC10:46
*** asettle has quit IRC10:48
*** asettle has joined #openstack-ansible10:50
*** thorst has joined #openstack-ansible10:50
*** thorst has quit IRC10:55
*** thorst has joined #openstack-ansible10:56
*** smatzek has joined #openstack-ansible11:04
*** thorst has quit IRC11:05
*** elo has quit IRC11:12
*** elo has joined #openstack-ansible11:12
csmartis the correct way to locally build the docs with this? tox -e docs11:14
mattoliveraucsmart: prob, I just look at the tox11:21
mattoliverau*tox.ini11:21
mattoliverauCause I'm not super familiar with OSA ;)11:22
*** javeriak has quit IRC11:22
csmartmattoliverau: thanks, I'm getting this: https://paste.fedoraproject.org/395267/44578514/11:23
mattoliverauHave you done a: pip install -r test-requirements11:24
mattoliverauNoting I'm basing this off what is do in swift ;)11:25
mattoliverau*i11:25
* mattoliverau is failing at typing on phone11:25
csmartmattoliverau: lol11:26
csmartlemme see11:26
csmartCould not open requirements file: [Errno 2] No such file or directory: 'test-requirements'11:26
csmartI'll hit up asettle11:26
* csmart waves11:27
asettleyo yo yo yo yo11:27
* asettle puts on cape11:27
asettleWassup?11:27
asettlecsmart: yep, tox11:27
csmartasettle: I don't know what I'm doing, but given that I have a doco change for https://review.openstack.org/#/c/346510 I figured I should also build it locally to make sure it works and looks good11:28
mattoliveraucsmart, you have to be in the root of the repo (where requirements.txt and test-requirements.txt live)11:29
csmartmattoliverau: yeah I am... :-S11:29
csmartjust no test-requirements11:29
csmartoh, test-requirements.txt11:30
mattoliveraucsmart also test-requirements.txt11:30
asettlecsmart: well, as you can see, you've passed the docs gate there - implying it does build. To be completely honest, unless you're writing huge chunks of doc, I personally do not run tox each change. It should be fairly simply to pick up on the syntax, and ensure you have no whitespace.11:30
mattoliverauI forgot the txt.. Also not being there would mean it'll fail :p11:30
csmartmattoliverau: rgr, that wants a new version of pip now.. you may be on to something11:30
*** thorst has joined #openstack-ansible11:31
csmartasettle: understood, although odyssey4me wants a further change to the docs (with examples) which I have locally, but haven't pushed yet.. also I'm keen to learn so figured knowing how to build the docs would be useful11:31
mattoliveraucsmart: you can activate the tox virtual env and then up grade if you need to11:31
asettleSo, csmartthis is all the information you'll need: http://docs.openstack.org/contributor-guide/docs-builds.html11:32
asettleOSA just changed over to using the same docs build as openstack-manuals11:32
mattoliverausource .tox/docs/bin/activate; pip install -U pip; deactivate11:32
mattoliverauOr something like that11:32
*** javeriak has joined #openstack-ansible11:33
* csmart looks11:33
mattoliverauOh and also... Hey asettle o/ Australia is a little quiet without ya!11:34
*** david-lyle_ has joined #openstack-ansible11:34
asettlemattoliverau: hey hey! Aww silence in rcbops11:34
asettle?*11:34
odyssey4mecsmart yes: 'tox -e docs'11:35
* csmart notes it's even quiet in Canberra without asettle11:35
odyssey4mehttp://docs.openstack.org/developer/openstack-ansible/developer-docs/scripts.html#documentation-build11:35
csmartodyssey4me: thanks, that's what mcarden gave me too, but I seem to hit this (even on master): https://paste.fedoraproject.org/395267/44578514/11:36
asettlecsmart: nawww you can hear me down in the middle of nowhere too11:36
*** retreved has joined #openstack-ansible11:36
*** retreved has quit IRC11:37
odyssey4meheh, asettle it looks like there are more lint things to fix11:37
odyssey4mecsmart you could patch those fails up too11:37
*** retreved has joined #openstack-ansible11:37
odyssey4meit's a little odd that the linters test isn't picking those up11:37
csmartodyssey4me: yeah was planning too.. ;-) just need to get it working first11:37
odyssey4mehmm11:37
* odyssey4me goes to check the linters test11:37
asettleodyssey4me: reh. Really? Bummer.11:38
odyssey4mecsmart those errors are problems with the existing docs which need fixing11:38
csmartodyssey4me: ok, so that's stopping it.. sweet.. I can fix those first then11:38
csmartasettle: this is what I get: https://paste.fedoraproject.org/395267/44578514/11:38
odyssey4methe trouble is that the openstack-ci process for building docs doesn't do 'tox -e docs' so we don't get an exact match in results11:38
asettleYeah, ha, sorry csmart I thought you were trying to get it BUILT not fix the errors :P that's just some lines too long etc :) easy fixes11:39
*** david-lyle_ has quit IRC11:39
csmartasettle: yeah, well I do just want it to build... just master doesnt' build..?11:39
csmartasettle: I'm happy to fix those11:39
* csmart assumes that once those are fixed, tox -e docs will just work(TM)11:40
csmart?11:40
odyssey4mecsmart ok if you want to patch those errors up then just include a change to the https://github.com/openstack/openstack-ansible/blob/master/tox.ini#L165 section to include the docs tox target in the commands series11:40
asettleBecause you have the errors :) that's okay. Your text HAS to wrap at 79 characters - see lines 13, 45, and 4611:40
odyssey4methat way we'll catch these errors properly11:40
asettlecsmart: yep11:40
* csmart thanks asettle with a pack of TimTams11:40
asettlecsmart: this is the important stuff: https://paste.fedoraproject.org/395271/44686114/11:41
asettleLines 15 (in the paste) is just hte summary. Ensure you go through the lines 1-9 and just fix the errors :) if you have any questions about what the errors may mean, just ping. Most are fairly self explanatory11:41
csmartasettle: ok cool, I'll start a new patch to fix those and put it up for review11:41
asettle:) easy as11:41
* csmart thanks asettle mattoliverau odyssey4me11:41
*** johnmilton has joined #openstack-ansible11:42
mattoliveraunps.. I can always guess the answers ;p11:42
mhaydenmorning folks11:43
csmartmhayden: \o11:43
mattoliverauOr assume all projects are the same :p11:43
mattoliveraumhayden: o/11:43
csmartmattoliverau: hehe, I do appreciate it all the same11:43
* csmart feels so at home in openstack land, fondly recalls LCA 201611:44
csmart(apart from the twisted ankle..)11:44
mattoliverauSometimes OpenStack does feel like a twisted ankle :p11:44
csmartasettle: I'll hit you up in #openstack-doc re lint, if needed11:44
csmartlol11:45
mhaydenhey wait, shouldn't australians be in bed by now? :)11:45
*** wadeholl_ has joined #openstack-ansible11:45
asettlecsmart: best ankle twist everrrrr11:45
asettlecsmart: do :)11:45
asettlemhayden: they've all come to me, they've flocked11:45
csmart"it's a cricket bat"11:45
csmartmhayden: yeah I miss asettle...11:45
csmart21:45, still some time to get some patches in!11:45
mattoliverauIts true, like moths to a flame is Australians to an asettle11:46
asettlehahahah thanks for getting on board team.11:46
csmartasettle: ^ that's why mattoliverau isn't in docs ;-)11:46
asettlecsmart: awww11:46
mattoliverauThat's why I like to vulentold asettle into things :p11:47
csmartlol11:47
asettlemattoliverau: you haven't done that in a while... where you be at man!?11:47
* csmart gets back to some patches11:47
asettleI've been telling MYSELF to do things.11:47
*** weshay_afk is now known as weshay11:47
*** wadeholler has quit IRC11:48
mattoliverauasettle: do I need to stay up late some nights and make sure you vollentold, cause I can do that :)11:48
asettlemattoliverau: nawww friend, yeah, maybe :p11:49
mattoliverauk, well csmart may be a machine, but I'm only human, so I'm calling it a night, night world o/11:51
csmartmattoliverau: you have a baby... you're excused ;-)11:51
csmartmattoliverau: thanks and ttyl11:51
*** sdake has joined #openstack-ansible11:54
*** psilvad has joined #openstack-ansible12:03
*** messy has joined #openstack-ansible12:09
*** elo has quit IRC12:10
*** elo has joined #openstack-ansible12:10
*** kjw3 has quit IRC12:13
*** timl0101 has joined #openstack-ansible12:20
asettleodyssey4me: can you help csmart with a build error? I haven't seen it before12:22
asettle"openstack-ansible/doc/source/index.rst:: WARNING: "table row spanning" not supported"12:22
asettleFull output here: https://paste.fedoraproject.org/395280/69449271/12:22
csmartodyssey4me: it's just a warning, the build works - was just wondering if I should take a look at that also, while I'm at it12:22
*** cfarquhar has joined #openstack-ansible12:28
*** cfarquhar has quit IRC12:28
*** cfarquhar has joined #openstack-ansible12:28
*** cfarquhar has quit IRC12:33
openstackgerritChris Smart proposed openstack/openstack-ansible: [DOCS] Fix up validation failures  https://review.openstack.org/34674412:37
*** cfarquhar has joined #openstack-ansible12:38
*** cfarquhar has quit IRC12:38
*** cfarquhar has joined #openstack-ansible12:38
evrardjpasettle: I think we have sphinx warning with table row/column span even if we are using the standard rst markup12:41
*** elo has quit IRC12:41
evrardjpso I wouldn't care about these :p12:41
odyssey4mecsmart do you mind if I tweak https://review.openstack.org/346744 with a tox config change to catch these in the future?12:42
csmartodyssey4me: I was _just_ about to ask that very thing..12:42
csmartplease go ahead12:42
asettleevrardjp: good to know! Thanks :) I've never seen them before.12:42
*** woodard has joined #openstack-ansible12:44
*** woodard_ has joined #openstack-ansible12:45
*** elo has joined #openstack-ansible12:47
*** woodard has quit IRC12:49
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: [DOCS] Fix up validation failures  https://review.openstack.org/34674412:50
odyssey4measettle csmart ^ there you go12:50
asettleWoohooo12:50
odyssey4meonce that merges we will never get another patch in which allows those failures to happen12:50
csmartodyssey4me: yep! thanks, looks good12:52
csmartodyssey4me: once that's in I'll update ANSIBLE_PACKAGE patch12:52
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-memcached_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675012:55
*** kjw3 has joined #openstack-ansible13:00
csmart23:00 here, so I'll check back tomorrow morning before work13:00
csmart\o13:00
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-apt_package_pinning: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675313:02
odyssey4meapologoies in advance for the noise13:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_client: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675413:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675513:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_container_create: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675613:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_hosts: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675713:03
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_hosts: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675813:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_openrc: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675913:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_aodh: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676113:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_barbican: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676213:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676313:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676513:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cloudkitty: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676713:04
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_designate: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676813:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_glance: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676913:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677013:05
*** elo has quit IRC13:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677113:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677213:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677313:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677413:05
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677513:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_neutron: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677613:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677713:06
*** elo has joined #openstack-ansible13:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677813:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677913:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_swift: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678013:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_tempest: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678113:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_zaqar: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678313:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678413:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_lock_down: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678513:07
*** elo has quit IRC13:07
*** bootsha has joined #openstack-ansible13:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678613:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678713:07
*** elo has joined #openstack-ansible13:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678813:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rsyslog_client: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678913:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rsyslog_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34679013:07
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-security: Ensure that doc linting is included in the linters test  https://review.openstack.org/34679113:07
*** janki has quit IRC13:08
* mhayden wonders when odyssey4me is going to overload stackalytics :P13:08
odyssey4mehaha13:08
mhaydenhah gertty is sitting there twirling around13:08
odyssey4mefree review stats for all13:08
*** Trident has quit IRC13:08
mhaydenlaptop fan activated :P13:09
odyssey4meof course with the current queue depth of zuul we won't see results for those patches until tomorrow I don't think13:09
mhaydenso this review should clear up the intermittent rabbitmq server failures on centos 713:11
mhaydenhttps://review.openstack.org/34628713:12
odyssey4memhayden lemme fire up a test instance to see what's going on there - looks reasonable13:15
mhaydenodyssey4me: the problem happens on centos + rabbitmq13:15
mhaydenthe /etc/hosts has: 127.0.0.1 localhost infra113:15
mhaydenhaving 'infra1' there makes rabbitmq get grumpy when the client connects13:16
mhaydenit can't figure out the hostname13:16
mhaydenbut the jinja template doesn't get applied to /etc/hosts for localhost because the split on the period fails13:16
odyssey4meah, seems sensible to catch that one out13:17
mhaydenmy jinja fix might not be gorgeous, but it seems to work13:17
* mhayden is open to suggestions ;)13:17
mhaydenthanks for those linting fixes -- i kept wondering why tox was blowing up locally on my machine but not in the gate13:18
*** elo has quit IRC13:22
*** bootsha has quit IRC13:22
*** elo has joined #openstack-ansible13:24
*** elo has quit IRC13:26
*** ngraf has joined #openstack-ansible13:26
*** elo has joined #openstack-ansible13:26
*** elo has quit IRC13:31
*** elo has joined #openstack-ansible13:31
*** weshay is now known as weshay_brb13:32
*** bootsha has joined #openstack-ansible13:33
cloudnullmornings13:35
mhaydenugh, lots of doc lint failures in the security role for lines being too long13:35
mhaydenis it possible to make section headings go over multiple lines?13:35
*** david-lyle_ has joined #openstack-ansible13:36
*** elo has quit IRC13:36
*** elo has joined #openstack-ansible13:37
*** eric_lopez has joined #openstack-ansible13:39
*** elo has quit IRC13:39
*** david-lyle_ has quit IRC13:41
odyssey4memhayden please go ahead and modify the patch for the role to add the fixes in to make it pass :)13:44
*** weshay_brb is now known as weshay13:44
odyssey4memorning cloudnull - please see feedback in https://review.openstack.org/34606413:44
*** sdake_ has joined #openstack-ansible13:44
*** sdake has quit IRC13:45
mhaydenodyssey4me: i can do that -- trying to figure out if i can do multiline sectionheadings :(13:45
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Enable the use of a package manager cache  https://review.openstack.org/34551413:45
odyssey4mecloudnull also note https://bugs.launchpad.net/openstack-ansible/+bug/1605846 which jmccrory found on the weekend13:45
openstackLaunchpad bug 1605846 in openstack-ansible mitaka "Upper constraints are not applied correctly in repo_build" [Critical,Fix committed] - Assigned to Jesse Pretorius (jesse-pretorius)13:45
*** smatzek has quit IRC13:47
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-lxc_container_create: Allow a deployer to disable container restarts  https://review.openstack.org/34604013:48
*** javeriak has quit IRC13:49
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Fix config entry regression  https://review.openstack.org/34606413:52
*** BjoernT has joined #openstack-ansible13:55
cloudnullodyssey4me: so jmccrory fix the constraint bits right?13:55
cloudnullnothing to do there ?13:55
odyssey4mecloudnull  yep - I just wanted you to be aware13:56
odyssey4meand to shout if you think it's all wrong13:56
odyssey4mebut it seems right, and tests right13:56
odyssey4mecloudnull you need another escape in https://review.openstack.org/#/c/346040/13:56
odyssey4memancdaz in master the change makes more sense, because we have settings in group_vars, so changing the format of the items would involve quite a few patches to different places13:57
cloudnullthe change looks good to me.13:58
cloudnullgood catch jmccrory13:58
odyssey4mevery good catch, yes13:58
openstackgerritAdam Reznechek proposed openstack/openstack-ansible: [WIP] Add discovery of build servers per architecture  https://review.openstack.org/34682913:59
mancdazodyssey4me ok, so for liberty we could just add spaces to the items? But we're going to leave it using the filter?14:00
mancdazoh wait14:00
mancdazI see the new patchset now14:00
odyssey4memancdaz yeah, so liberty just has the spaces in the items... master uses the filters14:00
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-lxc_container_create: Allow a deployer to disable container restarts  https://review.openstack.org/34604014:00
mancdazso things are less flexible in master14:01
cloudnullmancdaz: things are more flexable in master.14:02
odyssey4memancdaz not really, but in master we're allowing user-configurable options... so we're enforcing through the filters14:02
*** javeriak has joined #openstack-ansible14:02
mancdazlols I was trolling14:02
cloudnullah.14:02
*** michaelgugino has joined #openstack-ansible14:02
odyssey4melolsmashyoface14:02
mancdazodyssey4me so we're going to kill the revert patch?14:02
mancdazif we kill that, and then this current one passes, I will submit the patch for more controlled galera upgrades that relies on this new functionality14:03
odyssey4memancdaz already done14:03
*** hwcomcn has quit IRC14:04
odyssey4memancdaz you can always submit your patch now, based on cloudnull's patch14:04
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Refactor run-playbooks  https://review.openstack.org/34683314:06
*** javeriak has quit IRC14:06
*** retreved has quit IRC14:07
cloudnull^ I got asked about run-playbooks again today for mitaka, so that should help clear things up to future folks that want to run that script.14:07
odyssey4mehaha, yeah - I've actually stopped using it altogether now14:07
*** retreved has joined #openstack-ansible14:08
odyssey4meI do bootstrap-ansible, bootstrap-aio, setup-everything14:08
odyssey4meI think with the changes going into master now the need for the successerator is fast diminishing.14:08
cloudnullagreeded.14:09
cloudnullor rather i hope :)14:09
*** ngraf has quit IRC14:10
*** galstrom_zzz is now known as galstrom14:11
*** Mudpuppy has joined #openstack-ansible14:11
*** eric_lopez has quit IRC14:12
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: [TESTING] Run using Ansible "v2.1.1.0-0.5.rc5".  https://review.openstack.org/34683914:12
evrardjplet's hope at least14:13
*** bootsha has quit IRC14:13
*** elo has joined #openstack-ansible14:16
*** bootsha has joined #openstack-ansible14:17
*** eric_lopez has joined #openstack-ansible14:21
*** elo has quit IRC14:21
*** catintheroof has joined #openstack-ansible14:24
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_sahara: Support service-specific rabbitmq clusters  https://review.openstack.org/34308914:27
*** spotz_zzz is now known as spotz14:27
*** spotz is now known as spotz_zzz14:29
*** GMAzrael has joined #openstack-ansible14:30
*** spotz_zzz is now known as spotz14:32
*** jmckind has joined #openstack-ansible14:32
*** eric_lopez has quit IRC14:34
*** eric_lopez has joined #openstack-ansible14:35
*** jmckind_ has joined #openstack-ansible14:36
openstackgerritAdam Reznechek proposed openstack/openstack-ansible-repo_build: [WIP] Add support for multi-repo/multi-arch repo builds  https://review.openstack.org/34686314:38
*** jmckind has quit IRC14:39
*** BjoernT has quit IRC14:41
openstackgerritAdam Reznechek proposed openstack/openstack-ansible-repo_build: [WIP] Add support for multi-repo/multi-arch repo builds  https://review.openstack.org/34686314:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build: [WIP] Allow the use of a pre-staged git cache  https://review.openstack.org/34686714:42
antonymnoticed that https://github.com/openstack/openstack-ansible/blob/6bf159ff07da57ad3428d5d33221d3fc101d6aa3/playbooks/lxc-hosts-setup.yml seems to stop if /var/log/lxc exists, was able to get around it by nuking the dirs, so probably wouldn't be an issue on a new build14:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build: [WIP] Allow the use of a pre-staged git cache  https://review.openstack.org/34686714:43
odyssey4meantonym oh really? that's weird - the 'force' option is supposed to recklessly replace a dir with a link IIRC14:45
antonymmaybe it doesn't work with symlinks?14:46
odyssey4mecloudnull any thoughts on the discussion in https://review.openstack.org/346335 ?14:47
antonymmaybe it's ok with files and not directories14:47
* cloudnull looking14:48
prometheanfireodyssey4me: can you share the updated link for https://bugs.launchpad.net/neutron/+bug/1509312/comments/2314:49
openstackLaunchpad bug 1509312 in openstack-ansible trunk "unable to use tenant network after kilo to liberty update due to port security extension" [High,Fix released] - Assigned to Jesse Pretorius (jesse-pretorius)14:49
cloudnullodyssey4me antonym: it'll fail if its a dir.14:49
prometheanfirecloudnull: or I think you fixed the upgrade piece of it (to keep it disabled on upgrade)14:49
prometheanfireif you have the correct link that'd be helpful14:49
odyssey4meprometheanfire hmm, that's entirely the wrong link :p14:50
prometheanfireyep14:51
odyssey4mecloudnull yeah, I see the ansible docs aren't very explicit, but mention files only14:51
prometheanfireI wanted that for our docs14:51
odyssey4meprometheanfire I'm sure you're capable of git blaming ;)14:51
prometheanfireI'm going though that now14:52
odyssey4meprometheanfire https://review.openstack.org/#/c/294243/14:52
prometheanfirethanks14:54
prometheanfireodyssey4me: looks like that's only for upgrading to liberty?14:55
*** jmckind_ has quit IRC14:55
prometheanfirenot for L->M14:56
odyssey4meprometheanfire uh yeah, for L->M you should either have that config in-place already, or you started fresh in L and therefore it doesn't matter14:56
openstackgerritAdam Reznechek proposed openstack/openstack-ansible: [WIP] Add discovery and build for multiple CPU architectures  https://review.openstack.org/34682914:56
odyssey4meit only ever matters for K->L14:57
*** jmckind has joined #openstack-ansible14:57
odyssey4meunless you did something stupid like setup your L with l2pop... in which case you can clearly manage your own config and the upgrade process doesn't have to do it for you14:57
prometheanfireah, ya, it's adding the variable15:00
*** GMAzrael has quit IRC15:03
odyssey4megit-harry FYI for liberty, please do not SHA bump RPC until https://review.openstack.org/346064 merges15:06
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Fix config entry regression introduced in container restart patch  https://review.openstack.org/34606415:08
openstackgerritMerged openstack/openstack-ansible-rabbitmq_server: Write localhost line in /etc/hosts when testing  https://review.openstack.org/34628715:09
*** GMAzrael has joined #openstack-ansible15:10
openstackgerritMerged openstack/openstack-ansible-openstack_openrc: Define region name variables in openrc and clouds.  https://review.openstack.org/34639215:10
openstackgerritMerged openstack/openstack-ansible-rabbitmq_server: Fix D005 error  https://review.openstack.org/34628615:13
*** bootsha has quit IRC15:15
openstackgerritAdam Reznechek proposed openstack/openstack-ansible: [WIP] Support pulling architecture-tagged venv artifacts  https://review.openstack.org/34598615:15
openstackgerritNolan Brubaker proposed openstack/openstack-ansible: Allow empty container dicts in env overrides  https://review.openstack.org/34050715:16
openstackgerritMerged openstack/openstack-ansible-os_glance: Define keystone region name.  https://review.openstack.org/34444415:19
*** eric_lopez has quit IRC15:20
openstackgerritMerged openstack/openstack-ansible-os_cinder: Define keystone region name.  https://review.openstack.org/34444515:20
openstackgerritMerged openstack/openstack-ansible-os_neutron: Define keystone region name.  https://review.openstack.org/34444915:24
*** elo has joined #openstack-ansible15:25
*** pcaruana has quit IRC15:26
openstackgerritMerged openstack/openstack-ansible-os_swift: Define region name.  https://review.openstack.org/34445315:26
*** TxGirlGeek has joined #openstack-ansible15:27
*** McMurlock1 has quit IRC15:30
*** derekjhyang has quit IRC15:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Confirm container data destroys  https://review.openstack.org/34689515:37
*** david-lyle_ has joined #openstack-ansible15:37
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Confirm container data destroys  https://review.openstack.org/34689515:38
*** javeriak has joined #openstack-ansible15:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: Confirm container data destroys  https://review.openstack.org/34689515:39
*** TxGirlGeek has quit IRC15:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_horizon: Enable Horizon Ironic dashboard plugin  https://review.openstack.org/33250715:41
*** TxGirlGeek has joined #openstack-ansible15:41
openstackgerritMerged openstack/openstack-ansible: Update the sources-branch-updater  https://review.openstack.org/33099415:42
*** david-lyle_ has quit IRC15:43
*** mikelk has quit IRC15:44
openstackgerritMichael Gugino proposed openstack/openstack-ansible: Add nova-lxd virt driver git repo  https://review.openstack.org/34690115:44
*** adrian_otto has joined #openstack-ansible15:45
openstackgerritMajor Hayden proposed openstack/openstack-ansible-security: Ensure that doc linting is included in the linters test  https://review.openstack.org/34679115:45
*** sdake has joined #openstack-ansible15:46
*** sdake_ has quit IRC15:49
*** TxGirlGeek has quit IRC15:50
*** TxGirlGeek has joined #openstack-ansible15:51
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Implement git server HAP backend  https://review.openstack.org/34690715:56
cloudnulljmccrory: ^ in support of your repo server change to remove the fastcgi implementation of the git server.15:57
*** TxGirlGeek has quit IRC15:57
*** TxGirlGeek has joined #openstack-ansible15:58
jmccrorycloudnull: cool, thanks a lot. i think the only other thing needed was updating os_tempest to remove or auto allow hostkeys for git clones15:59
jmccrorythat's remove the git clone itself, not sure if it's actually needed15:59
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible: Implement git server HAP backend  https://review.openstack.org/34690716:00
*** TxGirlGeek has quit IRC16:00
cloudnull^ added release note16:00
*** TxGirlGeek has joined #openstack-ansible16:01
cloudnullLooks like we can remove https://github.com/openstack/openstack-ansible-os_tempest/blob/master/tasks/tempest_install.yml#L137-L15316:02
cloudnullbecause of all of the other changes.16:02
*** TxGirlGeek has quit IRC16:04
cloudnullbut https://github.com/openstack/openstack-ansible/blob/master/playbooks/os-tempest-install.yml#L24 this should pick up the repo url change and work regardless16:04
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_glance: Implement CentOS 7 support in os_glance  https://review.openstack.org/32054216:06
*** javeriak_ has joined #openstack-ansible16:07
jmccrorycloudnull: yep, that's the task. i think if it stays i'll need 'accept_hostkey: yes', like nova consoles did https://review.openstack.org/#/c/316984/16:08
*** TxGirlGeek has joined #openstack-ansible16:09
*** javeriak has quit IRC16:09
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_keystone: Implement CentOS 7 support in os_keystone  https://review.openstack.org/32021616:12
*** kstev has joined #openstack-ansible16:13
*** TxGirlGeek has quit IRC16:13
mhaydenthe gate is having a case of the mondays16:13
*** TxGirlGeek has joined #openstack-ansible16:14
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_tempest: Accept host keys for console git tasks  https://review.openstack.org/34691616:14
cloudnulldone.16:14
cloudnull^ jmccrory16:14
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_tempest: Accept host keys for tempest git task  https://review.openstack.org/34691616:15
cloudnullmhayden: idk if you have bw but i'd really appreciate your eyes on https://review.openstack.org/#/c/320216/ -- I seem to be having issues with Cent7 and keystone w/ that patch and I'm not 100% sure why...16:16
mhaydencloudnull: i'll gander16:16
cloudnullbeing that youre a RHEL guru I'd appreciate your feedback.16:17
cloudnull:)16:17
mhaydencloudnull: is something wrong with my rabbitmq patch16:17
cloudnullalso mhayden you16:17
palendaeCertified and everything!16:17
cloudnull*you're running ipv6 in mitaka right ?16:17
mhaydeni am running v6 in mitakatown16:17
cloudnullmhayden: no i've not seen any issue with rabbit16:17
cloudnullin your rabbitmq patch that is16:18
mhaydencloudnull: that's an unusual failure in your keystone patch16:18
cloudnullIKR .16:18
cloudnulland its consistent.16:18
mhaydenYARLY16:18
jmccrorycloudnull : awesome thanks again!16:18
cloudnullso idk . why its pissed at me.16:18
cloudnullRE:v6 in mitakatown.16:18
cloudnulldid you have to do anything out-of-band to make it go ?16:19
mhaydencloudnull: are we still sticking those keys into memcached?16:19
cloudnulli don't think so.16:19
cloudnullit looks like for federation we still are16:20
mhaydencloudnull: no -- i let my router do SLAAC though16:21
cloudnullbut thats not being tested in the gate.16:21
cloudnullokiedokie16:21
mhaydencloudnull: if you want neutron to do SLAAC and/or DHCPv6 for you, things get a little wild16:21
mhaydeni had some issues there16:21
cloudnullok.16:23
cloudnullthanks.16:23
cloudnullI'm getting an env up w/ ipv6 and was just curious. thanks again16:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: If /var/log/lxc exists, move it to the log aggregation parent  https://review.openstack.org/34691916:24
odyssey4meantonym ^ please verify that works16:24
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: If /var/log/lxc exists, move it to the log aggregation parent  https://review.openstack.org/34691916:29
*** elo has quit IRC16:31
*** eil397 has joined #openstack-ansible16:33
*** jwitko has joined #openstack-ansible16:33
*** eil397_ has joined #openstack-ansible16:34
*** david-lyle_ has joined #openstack-ansible16:35
*** asettle has quit IRC16:37
*** daneyon has joined #openstack-ansible16:38
*** kstev has quit IRC16:39
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova: If /var/lib/libvirt/qemu/save exists, move it to nova_libvirt_save_path  https://review.openstack.org/34692416:40
*** daneyon_ has joined #openstack-ansible16:40
eil397_good morning16:42
cloudnullo/ eil397_16:42
*** daneyon has quit IRC16:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_build: Allow the use of a pre-staged git cache  https://review.openstack.org/34686716:47
odyssey4mecloudnull ^ that should speed up the repo build a bit in OpenStack-CI, and is perhaps useful for AIO's too16:49
odyssey4melogan- ^ may be of interest to you too16:49
*** elo has joined #openstack-ansible16:53
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_nova: If /var/lib/libvirt/qemu/save exists, move it to nova_libvirt_save_path  https://review.openstack.org/34692416:56
*** eric_lopez has joined #openstack-ansible16:56
mrhillsmanhey16:57
*** elo has quit IRC17:00
*** david-lyle_ has quit IRC17:01
*** asettle has joined #openstack-ansible17:01
*** david-lyle_ has joined #openstack-ansible17:07
*** dmsimard is now known as dmsimard|afk17:09
cloudnullo/ mrhillsman17:12
v1k0d3nhey cloudnull, are you super busy atm?17:13
*** TxGirlGeek has quit IRC17:13
cloudnullnot super busy  .17:13
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_cloudkitty: Implement Multi-OS support in os_cloudkitty  https://review.openstack.org/34695217:13
cloudnullhows it going ?17:13
v1k0d3ngood man. your vacation looked amazing...i want to go there so bad.17:14
v1k0d3ntalking to the wife about taking a trip soon.17:14
cloudnullit was fun17:14
cloudnullwhere you planning on going?17:14
cloudnullsouth america?17:14
v1k0d3nyeah buenos aires...machu picchu17:16
cloudnullawesome !17:16
v1k0d3nplanning now.17:16
v1k0d3nnever been want to see it so bad. i hear great things about BA. my wife is mexican and studied SA culture...so for her it would be a real treat.17:17
v1k0d3nshe's huge into LA authors17:17
cloudnullBA is awesome !17:18
cloudnulli totally recommend it17:18
cloudnullwe stayed in palermo using AirBnB .17:18
cloudnullventured out all over the city .17:18
cloudnullmass transit is kick ass there.17:19
v1k0d3ni was wondering if AireBnB was the way to go...was looking at places to stay out there. really european looking. people have good taste down there.17:19
v1k0d3nso, i could use a little guidance, i haven't been able to get back to single-nic deployment since before...but having some issues. just wanted to ask some questions.17:21
v1k0d3nwith the single nic, are you using tr_tables for the different gateways on the interfaces?17:22
*** pcaruana has joined #openstack-ansible17:23
v1k0d3nalso are there any modules that i need to have other than loop, bonding, 8021q? for some reason the network takes foooooorever to come up.17:23
*** abelur__ has quit IRC17:23
v1k0d3ncloudnull: exactly what i have one node03: https://gist.github.com/v1k0d3n/e5567a11dd680248ce58b29fefe6a7c817:23
-cloudnull- -- We have a few patches in "Merge Conflict" that need attention: [ https://review.openstack.org/#/q/project:%255Eopenstack/openstack-ansible.*+-is:mergeable+is:open,n,z ]. If you're working on one of these patch-sets and would like to get more feedback on them please rebase the change as soon as possible. If the work is no longer 100% valid and/or needs updating it'd be great to see the reviews cleaned up / abandoned soo17:25
cloudnullv1k0d3n: for a deployment with a single nic I just do bridges and vlans17:26
cloudnullIE: eth0 > br-int > br-int.100 > br-mgmt, etc...17:27
v1k0d3nso just pull off the bonds i guess?17:27
cloudnullif you only have a sinle nic yea.17:27
cloudnulllike so https://gist.github.com/cloudnull/e81115119dddee5e2a0617:28
cloudnulljust without the bonds.17:28
*** mikelk has joined #openstack-ansible17:28
cloudnullif you only have a single interface.17:29
mrhillsmancloudnull nvm :)17:29
mrhillsmanwe got it resolved17:29
v1k0d3ncloudnull: tried that (i think you sent that a while back), and i replaced the bond0 statements with the int eth0...and it bombed out.17:30
v1k0d3nlet me look at and try again i guess.17:30
v1k0d3nmaybe i overlooked something...maybe i'll step in up slowly with each interface.17:30
cloudnullv1k0d3n: http://cdn.pasteraw.com/95i1ixw313otqxr6gxppdiubi5a9qjn17:35
*** TxGirlGeek has joined #openstack-ansible17:37
cloudnullIn that last paste, eth0 is plugged directly into br-vlan which has a native vlan and then additional tagged interfaces are built off of that.17:39
*** sdake has quit IRC17:41
cloudnulli think bofu2u has a similar setup.17:41
*** electrofelix has quit IRC17:42
v1k0d3nthat makes sense. tired and modified the ip to match what is connected at the other end of that interface...still taking forever to come up.17:42
v1k0d3ncloudnull: i almost wonder if i'm overlooking a module or something? perhaps the interfaces are fighting for default gw?17:42
cloudnullv1k0d3n: very well could be, remove the gateway entries from all of the interfaces except br-vlan17:43
cloudnullthe vlan module needs to be loaded.17:44
*** omiday has quit IRC17:44
*** omiday has joined #openstack-ansible17:44
v1k0d3ni have loop, vlan, and 8021q,17:45
cloudnullas well as br_netfilter17:45
cloudnull^ depending on kernel17:45
v1k0d3nah! gateway....that did the trick17:45
v1k0d3ncame right up17:45
v1k0d3nso that was it! that's where having the rt_table entries could really help.17:45
cloudnullindeed.17:46
cloudnullWe leave the host networking VERY open-ended, which is great for flexibility but can be hard to get started with.17:47
v1k0d3nspoke a little too soon. came up fast, but still can't reach device.17:47
cloudnullyou cant ping it on the physical host ?17:48
cloudnullor from another host?17:48
*** asettle has quit IRC17:48
v1k0d3ncloudnull: yeah...that's the great thing about osad though. you guys do an excellent job at opening up options and limiting opinion. as a community user, it's amazing.17:48
cloudnulldo you see tagged traffic on the various hosts?17:48
v1k0d3ncan't ping from other hosts on the same broadcast domain. looking into it now.17:49
v1k0d3ni mean, routes look good.17:49
v1k0d3nno arp though. so no L2.17:49
cloudnullhost to host on the eth0 interface isn't working either ?17:50
cloudnulleth0 > plugged into br-vlan in my example case.17:51
v1k0d3ncorrect. this is what threw me off when i tried this before...and attempted to annoy the crap out of you :)17:51
v1k0d3nsorry about that btw.17:51
cloudnullno worries.17:51
*** eil397 has left #openstack-ansible17:51
cloudnullyou have a native vlan ?17:51
eil397_leave17:51
*** eil397_ has quit IRC17:51
v1k0d3nlet me get this straight....eth0 doesn't have the ip for my upstream connected net, right? that's vlan....do i have that correctly?17:52
cloudnullchao eil397_ :)17:52
cloudnullcorrect17:52
v1k0d3nok, then configured as expected...17:52
cloudnullbr-vlan has that ip17:53
v1k0d3none sec...i think i made a L8 issue....a complete oversight.17:54
v1k0d3noh wait...nevermind...vlan is in modules...all good.17:54
cloudnullso eth0 (no ip), br-vlan (ip on native vlan)17:55
*** david-lyle has quit IRC17:57
*** david-lyle_ is now known as david-lyle17:57
v1k0d3ncloudnull: br-mgmt and br-vlan have the same ip?17:59
v1k0d3n248 maybe?17:59
cloudnullno.18:00
cloudnullthey should have seperate IP addresses and vlans.18:00
v1k0d3nok...that confused me. look at the paste again.18:01
cloudnull** The should be on different subnets.18:01
v1k0d3njust making sure.18:01
*** TxGirlGeek has quit IRC18:01
*** eil397_ has joined #openstack-ansible18:03
*** albertcard has joined #openstack-ansible18:03
*** dotplus has joined #openstack-ansible18:04
*** eil397 has joined #openstack-ansible18:09
eil397cloudnull: sorry for spaming in chat. slash has disappeared : - )18:10
*** adrian_otto has quit IRC18:11
*** javeriak has joined #openstack-ansible18:16
*** javeriak_ has quit IRC18:18
prometheanfireodyssey4me: https://bugs.launchpad.net/bugs/158532518:18
openstackLaunchpad bug 1585325 in openstack-ansible trunk "libvirt save directory saves to the rootfs" [Medium,In progress] - Assigned to Jesse Pretorius (jesse-pretorius)18:18
openstackgerritTim Laszlo proposed openstack/openstack-ansible-os_heat: Define default region name.  https://review.openstack.org/34444618:18
prometheanfireodyssey4me: didn't we already fix that?18:18
v1k0d3ncloudnull: i almost wonder if the nic i am using is too new or doesn't support these features. :(18:19
v1k0d3ni'm running on an intel nuc, but the new ones...the skull canyons.18:19
cloudnulli updated this paste http://cdn.pasteraw.com/tnxpl6wzk10nzs2zkzfv6hyk79l06i218:19
v1k0d3noh just for the ip?18:20
v1k0d3nor was there something else?18:20
cloudnullip and added a couple comments which should cover when a non-native vlan18:20
*** TxGirlGeek has joined #openstack-ansible18:21
*** KLevenstein has joined #openstack-ansible18:22
*** woodard_ has quit IRC18:23
*** thorst has quit IRC18:23
*** woodard has joined #openstack-ansible18:23
cloudnullv1k0d3n: there's really no special features.18:24
cloudnullits just bridging w/ a vlan18:24
cloudnulland most if not all nics should support that.18:24
openstackgerritJimmy McCrory proposed openstack/openstack-ansible-os_cloudkitty: Implement Multi-OS support in os_cloudkitty  https://review.openstack.org/34695218:26
*** jascott1 is now known as jascott1_away18:26
cloudnullv1k0d3n: still no connectivity from bridge to bridge between the nucs ?18:27
*** mikelk has quit IRC18:27
cloudnullI assume the switch supports vlan tagging?18:27
*** thorst has joined #openstack-ansible18:30
*** thorst has quit IRC18:34
*** jamesdenton has joined #openstack-ansible18:34
*** sdake has joined #openstack-ansible18:44
*** sirfry has joined #openstack-ansible18:46
sirfryHey I'm working on back porting the fix for bug 1579612, but I am new to OSA. Could someone help me understand the relationship between openstack-ansible-os-heat (and other service specific OSA projects) and openstack-ansible proper? Are the repositories nested?18:49
openstackbug 1579612 in openstack-ansible "heat domain setup fails with keystone using self signed certs" [Medium,Fix released] https://launchpad.net/bugs/1579612 - Assigned to Miguel Alejandro Cantu (miguel-cantu)18:49
v1k0d3ncloudnull: i'm not even getting a /proc/net/vlan/config file which i think should be there if things were working correctly, right18:51
v1k0d3n?18:51
*** retreved has quit IRC18:55
*** sirfry has quit IRC18:55
cloudnullv1k0d3n: are you sure the vlan module is loaded?18:55
*** rstarmer has joined #openstack-ansible18:55
*** retreved has joined #openstack-ansible18:55
cloudnullmodprobe vlan18:55
*** ngraf has joined #openstack-ansible18:56
cloudnullsorry 8021q18:56
cloudnullvlan is the package name18:56
jmccrorysirfry: os_heat is an ansible role. in liberty there was a single monolithic repo (openstack-ansible, with roles in playbooks/roles/), in mitaka+ the roles were separated into their own repos18:56
*** ngraf has quit IRC18:57
*** retreved has quit IRC18:57
*** retreved has joined #openstack-ansible18:58
cloudnullv1k0d3n: from my env http://cdn.pasteraw.com/itq6g0dbhysh3aemjsycikho27jxltm18:58
odyssey4meprometheanfire read the patch18:58
*** ngraf has joined #openstack-ansible18:58
odyssey4meprometheanfire essentially the link process does not replace an existing dir, even with the force option... so the patch you put in will only work on a fresh environment18:59
v1k0d3ncloudnull: let me check, one sec. thanks! the compare is extremely helpful.18:59
openstackgerritMichael Gugino proposed openstack/openstack-ansible-os_nova: WIP: Implement nova-lxd driver  https://review.openstack.org/34698419:00
*** johnmilton has quit IRC19:01
*** TxGirlGeek has quit IRC19:02
*** KLevenstein has quit IRC19:22
*** jascott1_away is now known as jascott119:24
openstackgerritMichael Gugino proposed openstack/openstack-ansible-os_nova: WIP: Implement nova-lxd driver  https://review.openstack.org/34698419:24
michaelguginoanyone around that can help me solve some pip problems?  Our pip script is not installing the latest versions of requests and urllib19:25
*** dmsimard|afk is now known as dmsimard19:26
*** messy has quit IRC19:30
*** messy has joined #openstack-ansible19:31
*** messy_ has joined #openstack-ansible19:33
*** messy has quit IRC19:33
*** messy_ has quit IRC19:34
*** messy has joined #openstack-ansible19:35
*** jimbaker has quit IRC19:35
*** jimbaker has joined #openstack-ansible19:40
*** jimbaker has quit IRC19:40
*** jimbaker has joined #openstack-ansible19:40
odyssey4memichaelgugino by 'pip script' what do you mean?19:41
odyssey4memichaelgugino everything a service needs goes into a virtualenv, so whatever's on the host is immaterial19:41
odyssey4me(and probably old)19:41
odyssey4meso if you want to see what the service has access to you need to activate the venv first19:42
michaelguginoyeah, I'm talking about the repo_build part19:42
odyssey4methe repo_build will build up to the upper-constraint that the requirements SHA is fixed to19:42
michaelguginoso, the repo_build process is failing on building the wheel for pylxd.  This seems to be due old version of requests and urllib319:43
odyssey4meit's not supposed to do the 'latest' - it's supposed to do the 'latest allowed in openstack requirements and tested successfully by openstack'19:43
michaelguginowhere is the upper-constraint sha list?19:43
odyssey4methe u-c comes from the requirements repo19:43
michaelguginoare we pull the u-c directly from master for newton?19:44
odyssey4memichaelgugino yes, master==newton19:44
odyssey4mein the integrated build we have a fixed SHA19:44
michaelguginookay, I'm still running into some issues then, I think those versions should be adequate19:45
odyssey4mein the role repositories we just use the head of the branch19:45
michaelguginoI can confirm, requests=2.10.0 is valid in UC's, but pylxd is complaining of requests 2.2.x19:46
michaelguginoI'm doing this locally, so, I'm not worried about gate checks at this point.  It's something in a the repo_build role, I think.19:46
*** KLevenstein has joined #openstack-ansible19:47
odyssey4memichaelgugino the repo is building a later version than that: http://logs.openstack.org/94/330994/15/gate/gate-openstack-ansible-dsvm-commit/3123f81/console.html#_2016-07-25_15_35_21_78137119:47
odyssey4methe repo_build will use the u-c version unless there is another constraint that reduces it19:47
michaelguginohmm, I see.  That's really strange.  I don't think it's being applied correctly locally for some reason.  It always has 2.2.x installed when I run this book, and i've been trying for over a week now19:48
michaelguginoI have to manually run pip install --upgrade requests19:49
michaelguginoon the repo container.  That will allow pylxd to build (along with doing the same for urllib3)19:50
*** v1k0d3n has quit IRC19:51
michaelguginoI think we should pass --upgrade to the pip script, I think requests is probably already installed on the host, so it's not being updated locally.19:53
michaelguginoany ideas?19:55
odyssey4menope, you must be doing something wrong19:57
odyssey4meeven the wheel build doesn't use the host's python packages - it uses a venv19:57
odyssey4medo you have a review with a log to look through?19:58
*** asettle has joined #openstack-ansible19:58
*** asettle has quit IRC19:58
michaelguginono, ci is taking forever today, but there's some in pipe.  I'm interested to see if they actually pass in the gates19:58
michaelguginohttps://review.openstack.org/#/c/346984/19:59
odyssey4meis there a particular reason why you need to use the openstack git source rather than just use the package from pypi?20:07
openstackgerritMajor Hayden proposed openstack/openstack-ansible-os_glance: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676920:07
*** rstarmer has quit IRC20:12
michaelguginoodyssey4me: how do you mean?  for nova-lxd, yes, I need from git source because that project has not tagged an newton branch and published to pypi.  From my testing, mitaka nova-lxd + newton nova does not work20:14
odyssey4memichaelgugino I don't think it ever will create an openstack-named branch.20:14
michaelguginothey have stable/mitaka20:15
odyssey4mebut ok, no matter anyway20:15
odyssey4meah ok, interesting20:15
*** sdake has quit IRC20:16
*** javeriak has quit IRC20:19
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_container_create: Allow extra LXC command options via group/host vars  https://review.openstack.org/34701420:21
odyssey4meok michaelgugino - if you haven't figured out the solution by tomorrow, I'll take a look-see20:21
michaelguginook.  I was just looking at their gerrit, and I found this:  https://review.openstack.org/#/c/339245/2/devstack/nova-lxd-functions.sh20:22
michaelguginoso, it seems there is a conflict with the apt installed python packages from trusty20:22
michaelguginoI might cut a patch to that effect locally.20:22
odyssey4melines 15-17 are irrelevant, as those are system packages being removed20:22
michaelguginoright, but something is causing this conflict, and it doesn't seem to be anything I'm doing / we're doing20:23
odyssey4meas said before, the reason we python venv everything is to allow us to ignore system packages installed by apt20:23
odyssey4meso anything installed by apt is irrelevant20:23
odyssey4mehow recent is your master checkout?20:24
michaelguginodoes that apply to the wheel build process on the repo server?20:24
michaelguginowithin the last week or two20:24
odyssey4meyou need to update your checkout20:24
mhaydencloudnull: getting to the bottom of these rabbit shenanigans on centos 720:24
mhaydensomething is mangling /etc/hosts  -- trying to find the task20:25
odyssey4memichaelgugino also, the wheel compile process sources only from the applicable sources - being pypi20:26
odyssey4memichaelgugino is the wheel build failing, or the venv build?20:26
michaelguginowheel build20:27
odyssey4memichaelgugino ok, let me fire up a build quickly20:29
*** thorst has joined #openstack-ansible20:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677020:34
*** severion has joined #openstack-ansible20:34
severioncloudnull: man...i'm having one heck of a time here.20:34
*** severion has quit IRC20:34
*** v1k0d3n has joined #openstack-ansible20:35
v1k0d3ni may have to think of something else...like grabbing a bunch of usb/ethernet dongles and replanning this layout or sometihng.20:35
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677120:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_heat: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677120:37
cloudnullv1k0d3n: still no love ?20:37
*** TxGirlGeek has joined #openstack-ansible20:37
cloudnulljust to recap, you have a collection of nucs, connected back to some switch ?20:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_gnocchi: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677020:38
cloudnulland basic l2 connectivity between the nucs is not working ?20:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-memcached_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675020:40
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_cinder: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676520:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_lock_down: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678520:42
v1k0d3ncloudnull: no...i have no idea why though...really strange.20:43
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_rally: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677820:43
cloudnullv1k0d3n: does it still fail when not using a bridge?20:44
odyssey4memichaelgugino the repo build works just fine for me20:44
v1k0d3none sec...let me check. got pulled into other things unfortunately.20:44
v1k0d3ni know i have to get this running. have a huge demo coming up next week and wanted to show kubernetes 1.3 openstack added features.20:44
cloudnullno worries.20:44
michaelguginook, maybe my environment is messed up/too old.  But I was seeing these issues as soon as I created the image about a week ago, so I don't think outdated plays is the issue.20:44
*** ngraf has quit IRC20:46
odyssey4mehang on, I didn't patch the nova role so it didn't bother trying to build the git source20:49
odyssey4meinteresting, I didn't know that would happen20:49
openstackgerritKevin Carter (cloudnull) proposed openstack/openstack-ansible-os_keystone: Implement CentOS 7 support in os_keystone  https://review.openstack.org/32021620:49
cloudnullodyssey4me michaelgugino what happened?20:52
michaelguginowhat do mean?20:52
cloudnullRE: [15:49] <odyssey4me> interesting, I didn't know that would happen20:52
michaelguginoI'm not quite sure what he meant20:53
odyssey4mecloudnull I didn't realise that a remote package part would not be built unless referenced by a role specifically.20:54
cloudnullyou mean the python bits in a git repo not getting built?20:55
michaelguginothat repo build is quite confusing.  Lots of secret sauce.20:55
*** Mudpuppy has quit IRC20:56
cloudnullWhats not building?20:56
*** omiday has quit IRC20:56
* cloudnull coming in late to the convo sorry20:56
*** omiday has joined #openstack-ansible20:57
michaelguginopylxd cloudnull20:58
odyssey4memichaelgugino are you sure that pylxd is required? it's not expressed in any of the requirements files20:58
odyssey4mepkg_resources.VersionConflict: (requests 2.2.1 (/usr/lib/python2.7/dist-packages), Requirement.parse('requests!=2.8.0,>=2.5.2'))20:59
odyssey4meso the issue is that the requests version in the repo is not new enough20:59
*** messy has quit IRC20:59
*** rstarmer has joined #openstack-ansible20:59
michaelguginoodyssey4me: https://github.com/openstack/nova-lxd/blob/cda09db0969107b2fd6a69bf4d32b043050de1e7/nova/virt/lxd/driver.py#L3920:59
odyssey4meah, it's because this is pylxd which is not managed by openstack in any way20:59
odyssey4meso I expect that you will need to reduce/cap the pylxd version20:59
*** messy has joined #openstack-ansible20:59
cloudnullodyssey4me michaelgugino -- https://review.openstack.org/#/c/346984/2/defaults/main.yml@381  ?21:00
cloudnullare wanting to build https://github.com/lxc/pylxd ?21:00
*** jamesdenton has quit IRC21:00
michaelguginocloudnull: https://github.com/openstack/nova-lxd/blob/cda09db0969107b2fd6a69bf4d32b043050de1e7/nova/virt/lxd/driver.py#L3921:00
michaelguginowe need pylxd, I'm not sure the best way of going about getting it.21:00
michaelguginobut, 'requests!=2.8.0,>=2.5.2' is valid for the upper limit constraints with openstack21:01
cloudnullcan we add the git config to the defaults within the nova role to pull it from a given sha?21:02
odyssey4meheh, right from the start is required a higher version21:02
cloudnullsimilar to https://github.com/openstack/openstack-ansible-os_nova/blob/master/defaults/main.yml#L22-L29 but for pylxd ?21:03
openstackgerritMajor Hayden proposed openstack/openstack-ansible-rabbitmq_server: Fix localhost line in /etc/hosts  https://review.openstack.org/34702921:04
odyssey4mehmm, requirements_constraints has requests<=2.10.021:04
odyssey4meso why are we ending up with 2.221:04
michaelguginothat's my question, lol21:04
michaelguginoit doesn't make any sense whatsoever21:04
michaelguginoand by updating the requests and urllib3 directly in the repo_server container, it's fine21:05
*** retreved has quit IRC21:05
odyssey4meok, but we have the right one: /var/www/repo/links/requests-2.10.0-py2.py3-none-any.whl21:05
odyssey4meok, afk for a bit... it seems that we may have some sort of system usage leak here21:05
michaelguginoI also faced this problem with the version of nova-lxd from pypi21:05
odyssey4meit does kinda make sense, because I don't think we install requests on the repo server... so we may have to21:06
cloudnullmichaelgugino: that should build the needed bits from a given git source http://cdn.pasteraw.com/lgc7yvx4royo36kzfpwyii3z154v5iw21:07
michaelguginois that going to impact hosts that aren't utilizing lxd?21:08
cloudnullsorry http://cdn.pasteraw.com/fmpdq7vtqn5y777qpqda7k74symmhdt21:08
odyssey4methere should be no need for the pylxd bits cloudnull, as those should rather come from pypi21:08
mrdaMorning all21:09
cloudnullmorning mrda21:09
odyssey4mebut yes, the other bits will be needed for the nova role test21:09
michaelguginookay, I'll remove the pylxd bits.  I'm not sure why I added it in the first place.  I think it was from the nova-lxd package from pypi not declaring the dependencies correctly.21:10
cloudnullmichaelgugino: maybe? the nova-lxd project would get built, which would be added to constraints at build time so if its requirements break other things its possible.21:10
cloudnullhowever nova-lxd is in the openstack namespace21:10
cloudnullso its requirements should play nice with everything else21:10
michaelguginookay, works for me.21:11
michaelgugino(the idea, haven't tried the edits)21:11
cloudnullok21:11
cloudnullme neither :)21:11
odyssey4meyeah, that's why the nova_lxd bits are fine, not the pylxd21:12
odyssey4memichaelgugino it would appear that adding 'requests' to the repo_build role's python package list solves the problem for the pyrax build because it installs the latest version of requests that complies with the u-c21:13
odyssey4meI'm just confirming a build now, but it definitely upgrades requests21:13
michaelguginothe problem is that pylxd isn't listed in the requirements file in nova-lxd21:14
*** messy has quit IRC21:15
odyssey4meheh, then it fails on urllib21:15
odyssey4meurllib321:15
michaelguginoright21:15
michaelguginothose are the only two though, don't worry, you're not going down the rabbit hole.21:15
openstackgerritMajor Hayden proposed openstack/openstack-ansible-rabbitmq_server: Really fix localhost line in /etc/hosts  https://review.openstack.org/34702921:17
odyssey4memichaelgugino ok, that solves it and I think the fix is generally appropriate anyway21:21
spotznice title there mhayden:)21:21
mhaydenyeah, fun times :P21:21
odyssey4meso put in a review to add 'requests' and 'urllib3' to https://github.com/openstack/openstack-ansible-repo_build/blob/master/defaults/main.yml#L6821:21
michaelguginowill do21:22
odyssey4meunless there's a way to tell https://github.com/openstack/openstack-ansible-repo_build/blob/master/tasks/repo_pre_build.yml#L16 to upgrade all the deps for the packages it installs21:24
odyssey4melemme try something21:24
*** rstarmer has quit IRC21:25
*** v1k0d3n has quit IRC21:26
michaelguginoI thought adding --upgrade to the script would be helpful21:26
michaelguginosince the constraints should stop it.  But I haven't tested it.  And, if it works, it will catch any of these issues in the future21:26
odyssey4menope, it doesn't appear to do it - just tried it21:29
odyssey4meit seems like it only installs the latest of the packages in the list21:29
*** rstarmer has joined #openstack-ansible21:32
*** bsv has joined #openstack-ansible21:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_client: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675421:38
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_hosts: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675821:40
*** johnmilton has joined #openstack-ansible21:41
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_container_create: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675621:43
cloudnullodyssey4me: did this work out https://review.openstack.org/#/c/345448/ ?21:43
odyssey4mecloudnull I haven't had a chance to try it out yet - did you implement the unique merge with the global pins going over the requirements?21:44
cloudnullyes21:44
odyssey4meok, I'll try it out again tomorrow21:45
*** johnmilton has quit IRC21:45
*** johnmilton has joined #openstack-ansible21:46
cloudnullthis is the new output http://cdn.pasteraw.com/kaaumhif1209znderf75q7j39fcrjuy21:46
*** adrian_otto has joined #openstack-ansible21:46
odyssey4mecloudnull looks good21:47
odyssey4meI was thinking - basically the precedence can be worked out in py_pkgs or in the template21:47
odyssey4mebut in py_pkgs is faster21:47
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_hosts: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675721:48
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-openstack_openrc: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675921:50
cloudnullso this passed, even in ovh -- https://review.openstack.org/#/c/345514/ -- idk what our general run times have been there but the last couple I checked seem to all be timeouts21:50
odyssey4mecloudnull they've been passing with the removal of all the telemetry21:51
*** ngraf has joined #openstack-ansible21:51
odyssey4mebut yeah, the cache doesn't seem to hurt21:51
odyssey4meanyway, I'm out for tonight21:51
cloudnullah you might be right, telemetry isn't gating right now...21:51
odyssey4meif any of the doc8 lint checks fail, leave them be - I know what needs fixing21:52
cloudnulllooks like the gate still has bout an 8 hour lead time.21:52
odyssey4mecloudnull this works in local testing - a review would be good: https://review.openstack.org/34701421:52
cloudnullso you might be able to check in on the run when you wake up in the morning ;)21:52
odyssey4meI'm out - night all21:52
cloudnullhave a good one odyssey4me21:52
*** ngraf has quit IRC21:53
michaelguginocloudnull: okay, where did we leave off.  I had to step away a bit.21:55
michaelguginooh, I see you guys were talking about something else, nvm21:57
cloudnullha!21:57
cloudnulldid the build bits work ?21:57
michaelguginoI didn't try, I'm about to pack up.  I was just going to go with Jesse's suggestion and add the packages to the repo build process21:57
*** KLevenstein has quit IRC22:00
openstackgerritMichael Gugino proposed openstack/openstack-ansible-repo_build: Fix out-dated requests and urllib3  https://review.openstack.org/34704222:01
*** mkrish004 has joined #openstack-ansible22:01
*** michaelgugino has quit IRC22:06
cloudnullok.22:06
cloudnullwell. i have to run also.22:06
cloudnullbbl22:06
cloudnullwill chekc in on things a little later.22:06
openstackgerritMerged openstack/openstack-ansible-os_sahara: Support service-specific rabbitmq clusters  https://review.openstack.org/34308922:08
*** jamesdenton has joined #openstack-ansible22:09
*** jamesden_ has joined #openstack-ansible22:11
*** jamesdenton has quit IRC22:15
*** evilt0ne has joined #openstack-ansible22:15
csmartodyssey4me: \o22:16
csmartnight22:17
* csmart heads to work22:18
*** spotz is now known as spotz_zzz22:24
*** TxGirlGeek has quit IRC22:27
openstackgerritMerged openstack/openstack-ansible: [DOCS] Fix up validation failures  https://review.openstack.org/34674422:32
*** galstrom is now known as galstrom_zzz22:37
*** thorst has quit IRC22:43
*** thorst has joined #openstack-ansible22:43
*** v1k0d3n has joined #openstack-ansible22:51
*** thorst has quit IRC22:52
*** jmckind has quit IRC22:54
*** asettle has joined #openstack-ansible23:03
*** asettle has quit IRC23:08
*** bsv has quit IRC23:15
*** thorst has joined #openstack-ansible23:26
odyssey4meI can't get no sleep. :/23:26
stevelleodyssey4me: may I recommend a concussion?23:29
odyssey4melol23:29
odyssey4medoes that work for you stevelle ?23:29
stevellebeen a while since I have tried23:29
stevelleI'd like to keep it that way23:30
stevelleI just try to read and then suffer through the following afternoon23:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-galera_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34675523:30
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ceilometer: Ensure that doc linting is included in the linters test  https://review.openstack.org/34676323:32
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_magnum: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677523:33
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_ironic: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677323:34
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_keystone: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677423:36
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-os_sahara: Ensure that doc linting is included in the linters test  https://review.openstack.org/34677923:37
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-pip_install: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678423:39
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-repo_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678823:40
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-rabbitmq_server: Ensure that doc linting is included in the linters test  https://review.openstack.org/34678623:42
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible: If /var/log/lxc exists, move it to the log aggregation parent  https://review.openstack.org/34691923:47
odyssey4mestevelle if you have a moment, another review of https://review.openstack.org/346064 would be useful23:50
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/openstack-ansible-lxc_container_create: Allow a deployer to disable container restarts  https://review.openstack.org/34604023:59
openstackgerritMerged openstack/openstack-ansible-os_tempest: Accept host keys for tempest git task  https://review.openstack.org/34691623:59

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