Friday, 2014-05-02

*** shakayumi has joined #tripleo00:00
lifelessgeerdest: https://review.openstack.org/#/c/88153/3 - I think its a good idea to do it in undercloud too.00:01
lifelessgreghaynes: thanks for the headsup00:03
*** shakamunyi has quit IRC00:05
*** shakayumi has quit IRC00:05
lifelesshmm, missed the EU folk00:07
lifelessand spamaps is +000:07
lifelesswho can I pester00:07
lifelessbnemec: https://review.openstack.org/#/c/91198/00:08
lifelessgreghaynes: SpamapS: https://review.openstack.org/#/c/90890/ <- fix a critical00:08
*** newell has quit IRC00:14
*** TravT has quit IRC00:24
*** matsuhashi has joined #tripleo00:29
*** mestery has joined #tripleo00:37
StevenKlifeless: https://review.openstack.org/91518 https://review.openstack.org/91519 https://review.openstack.org/91520 https://review.openstack.org/9152100:45
lifelessStevenK: so we need to figure out dinner details; 1) is tchaypo coming; 2) Lynne is available in the city @ 5pm00:47
lifelessStevenK: she liked the sound of the himalayan place, if you and sarah would be up for that again (and its open, of course)00:47
openstackgerritJames Polley proposed a change to openstack/tripleo-image-elements: Nasty hack to work around pip's case-sensitivity  https://review.openstack.org/8494600:48
lifelessI need to know to sms lynne details00:48
*** chuckC has joined #tripleo00:48
StevenKlifeless: I'm guessing soonish?00:48
lifelessbefore 3pm I'd say but earlier better00:49
StevenKIt looks like Sarah has not reached her desk yet00:49
lifelessclarkb: is the change where clicking on a project name now takes you to the project details rather than the project review queued planned for fixing ?00:49
tchaypolifeless: StevenK has found something even more annoying that starting to talk just as noice cancelling is activated00:52
tchaypolifeless: I'm still on aware mode and he started whispering so I can only just hear him00:52
lifelessgreghaynes: https://review.openstack.org/#/c/91518/ is ready for a second +200:52
StevenKThat was actually thinking out loud00:52
tchayposo you're saying my headphones stop your thoughts?00:53
*** jml has quit IRC00:53
StevenKtchaypo: Coincidence, not causation00:54
lifelessor is it?00:54
StevenKI say it is, so it is.00:54
*** jml has joined #tripleo00:56
lifelessjml: poke - testtools reviews00:59
lifelessStevenK: all reviewed00:59
tchaypoI've been working on a change to cleanup-env to make it more selective - it adds the ability to only clean up a numbered environment rather than destroying everything01:01
tchaypoIt's working fine in my testing, but I'd like someone who knows about CI to take a look before it lands01:01
tchayposomeone like SpamapS would be perfect, except I don't think he's around01:02
lifelessI can look01:03
lifelessStevenK: you had a big review for me ?01:05
openstackgerritTrent Geerdes proposed a change to openstack/tripleo-incubator: Fixing typo in comment.  https://review.openstack.org/9171501:05
StevenKlifeless: The BIG review is https://review.openstack.org/8493301:07
* StevenK goes back to reading keystone code01:07
lifelessgreghaynes: happy with https://review.openstack.org/#/c/89732/ now ?01:08
morganfainbergStevenK, beware dragons >.> i mean, let me know if you have questions01:08
tchaypoTo replicate what mestery has been doing to chase https://bugs.launchpad.net/tripleo/+bug/1290486 I want to set up a devstack environment; but I'm wondering if it's safe to do that on my laptop, or if it's going to start a fight between devstack and devtest01:08
uvirtbotLaunchpad bug 1290486 in tripleo "neutron-openvswitch-agent does not recreate flows after ovsdb-server restarts" [Critical,In progress]01:08
lifelessmorganfainberg: oh hai01:09
morganfainberglifeless hi thar!01:09
lifelessmorganfainberg: did you see the mailing list thread I pointed you at a couple days back ?01:09
StevenKmorganfainberg: Oh, was just checking the order of arguments for keystone.endpoints.create(), nothing terrible01:09
morganfainberglifeless, i think i read it, but it's fleeting atm (been working on a lot of internal company stuff atm)01:10
*** yamahata has quit IRC01:10
morganfainberglifeless, that was the ID vs name one, right?01:10
lifelessmorganfainberg: yes, the one where our feedback about it being a PITA and hard to use was discounted :(01:11
morganfainberglifeless, yeah let me re-read and follow back up iwth it.01:11
lifeless''openstack-dev] [Heat] [Keystone] [TripleO] Making use of domains by name - policy and API issues?01:13
morganfainberglifeless, yep have it in front of me. - i keep IRC chat logs for just that reason :)01:14
lifeless:>01:14
lifelessthanks!01:14
StevenK144M    irclogs01:14
StevenKHm01:14
StevenKMaybe I want to compress some01:14
lifelessStevenK: or buy a real hard disk?01:15
morganfainbergStevenK, 218MB here01:15
morganfainbergStevenK, but i also store my on my Google drive :)01:15
StevenKlifeless: It's a Linode. Buying a real hard disk will not help in the slightest.01:15
morganfainbergStevenK, lets me search from anywhere i have internet01:15
lifelessmorganfainberg: yeah - the thing is having someone with keystone history weigh in; clint and I are only deployers, so carry little weight.01:15
morganfainberglifeless, i'll def re-read and try and follow up with it soon01:15
morganfainbergor at least catchup with people about it in ATL01:16
lifelessmorganfainberg: no panic - it seems to have gone silent for now.01:16
lifelessI suspect SpamapS is sitting on the volcano01:16
morganfainberglifeless, hehe01:16
tchaypoI'm not sure what "sitting on the volcano" means, but I think I'm happy to remain ignorant01:17
lifelesstoo bad01:18
lifelesssupressing frustration at the disconnect01:18
lifelessStevenK: sarah is online01:22
StevenKlifeless: Yes, I did notice. :-)01:23
tchaypolifeless: I've just noticed that I forgot to mention that I'm in01:23
StevenKlifeless: Are you drowning in my horrible Python yet?01:24
lifelesslike flynn?01:24
StevenKAs in, 'in like' ?01:25
lifeless+101:29
StevenKlifeless: Is that +1 to my comment, or my branch?01:29
lifelessyes01:32
lifeless:/01:32
lifeless:\01:32
lifeless:)01:32
lifelessproffalken: re-reviewed your three patches01:35
*** nosnos has joined #tripleo01:36
*** ramishra has joined #tripleo01:38
openstackgerritTrent Geerdes proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file  https://review.openstack.org/9171701:40
tchaypogeerdest: that sounds nice01:41
geerdestsounds nice isn’t +1 ;)01:42
tchaypoI've starred it to take a detailed look later :01:44
*** nosnos has quit IRC01:44
tchaypo:)01:44
*** nosnos has joined #tripleo01:44
* StevenK gets distracted by this whole section thing01:45
lifelessYou're in Glee?01:47
*** funzo has quit IRC01:49
*** weshay has quit IRC01:49
*** funzo has joined #tripleo01:49
*** funzo has quit IRC01:49
*** funzo has joined #tripleo01:50
*** funzo is now known as Guest7525301:50
*** nati_uen_ has joined #tripleo01:55
*** nati_uen_ has quit IRC01:56
*** eguz has joined #tripleo01:58
*** nati_ueno has quit IRC01:59
*** eghobo has quit IRC02:00
*** shakayumi has joined #tripleo02:00
lifelesstchaypo: you had a review you wanted looked at02:02
tchaypoyes02:03
tchaypohttps://review.openstack.org/9128802:03
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Perform PKI initialization for a Keystone host  https://review.openstack.org/9152002:03
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI  https://review.openstack.org/9152102:03
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Create Keystone v2 endpoint in initialize()  https://review.openstack.org/9151902:03
*** yamahata has joined #tripleo02:04
*** sdake_ has joined #tripleo02:04
*** sdake_ has quit IRC02:04
*** sdake_ has joined #tripleo02:04
*** openstackgerrit has quit IRC02:04
*** openstackgerrit has joined #tripleo02:06
tchaypolifeless waggled his finger at me02:08
tchaypoI can't wait to find out what naughty I done.02:08
lifelessSee the review :)02:09
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Perform PKI initialization for a Keystone host  https://review.openstack.org/9152002:14
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI  https://review.openstack.org/9152102:14
*** shakayumi has quit IRC02:14
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI  https://review.openstack.org/9152102:20
*** Guest75253 has quit IRC02:26
*** funzo has joined #tripleo02:26
*** funzo is now known as Guest2245702:27
*** Guest22457 has quit IRC02:31
*** shakayumi has joined #tripleo02:31
*** nati_ueno has joined #tripleo02:33
*** michchap has quit IRC02:43
*** eguz has quit IRC02:44
*** michchap has joined #tripleo02:45
lifelessbnemec: / greghaynes: - https://review.openstack.org/#/c/86523/ looks simple and ready02:49
*** eghobo has joined #tripleo02:50
*** nosnos has quit IRC02:51
*** eghobo has quit IRC02:51
*** nati_ueno has quit IRC02:55
*** eghobo has joined #tripleo03:24
*** matsuhashi has quit IRC03:27
*** tzumainn has quit IRC03:37
*** marun has joined #tripleo03:37
lifelesstchaypo: have you seed  ?https://review.openstack.org/#/c/89838/03:37
tchaypolifeless: i had not03:42
stevebakerSpamapS: could you restore this change please? https://review.openstack.org/#/c/83831/03:44
stevebakerlifeless: I really need a solution for https://review.openstack.org/#/c/83831/ so that heat software config works again03:46
lifelessstevebaker: infra can restore arbitrary changesets I believe03:47
stevebakerlifeless: I'll ask, but I was hinting at you reconsidering your -1 ;)03:48
lifelessstevebaker: we have software config working without that03:49
lifelessstevebaker: so I'll need more data03:49
stevebakerlifeless: it affects software-config when the config script is non-json, so shell, puppet, chef, ansible, salt...03:50
stevebakerlifeless: and its a regression since writing out the configs as top-levels was a new thing03:50
lifelessstevebaker: so why are you pushing that into the space oac is reading from?03:51
stevebakerI'm not, we ignore those top-levels. It was something SpamapS added to occ for tripleo config03:51
*** Rakesh6 has joined #tripleo03:52
*** nati_ueno has joined #tripleo03:53
stevebakerlifeless: maybe the fix is to not write out the configs as top-levels if it isn't a dict03:53
lifelessstevebaker: so my concern here is that this will be indistinguishable from a bad json config03:53
lifelessstevebaker: I am not a fan of quietly hiding errors except is very constrained circumstances03:54
lifelesssince the configs for a node are user supplied, this isn't a constrained circumstance03:54
lifelessstevebaker: if we can arrange to only be supplying the json configs then thats a whole lot better03:54
lifelessstevebaker: so - does software-config know that a config is metadata vs shell ?03:55
stevebakerlifeless: the config resource has a "group" attribute which is used for this, given a value like "shell", "puppet". You could set group: tripleo and only write out top-levels in this group03:57
stevebakerlifeless: see group http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::StructuredConfig03:58
lifelessstevebaker: I think that makes a lot of sense03:59
lifelessstevebaker: so an occ change to look for group= "" (backwards compat, deprecated) or "os-apply-config"03:59
stevebakeryes, except default group is "Heat::Ungrouped"04:00
lifelessok04:00
stevebakerI'll whip something up04:01
openstackgerritlifeless proposed a change to openstack/tripleo-incubator: Make bootstrapping TripleO development easier.  https://review.openstack.org/8563604:03
lifelesstchaypo: configure-tripleo repushed04:03
*** nosnos has joined #tripleo04:05
openstackgerritSteve Baker proposed a change to openstack/tripleo-heat-templates: Specify group os-apply-config for config resources  https://review.openstack.org/9173104:05
*** eghobo has quit IRC04:08
openstackgerritlifeless proposed a change to openstack/tripleo-image-elements: Tune deploy-ci-overcloud a little.  https://review.openstack.org/8407604:24
openstackgerritlifeless proposed a change to openstack/tripleo-image-elements: Fix the HP region deploy-ci-overcloud script  https://review.openstack.org/8407604:26
openstackgerritSteve Kowalik proposed a change to openstack/os-cloud-config: Add a register-nodes command and associated API  https://review.openstack.org/8493304:26
openstackgerritSteve Baker proposed a change to openstack/os-collect-config: Only top-level some deployment configs  https://review.openstack.org/9173304:37
stevebakerlifeless: ^ works for me04:37
lifelessreviewed04:42
*** nati_ueno has quit IRC04:43
*** eghobo has joined #tripleo04:44
StevenKlifeless: You commeted on https://review.openstack.org/#/c/91519 that I need to add a test for region and SSL -- er, I did?04:52
lifelessStevenK: I saw one test called ..._options04:53
lifelessStevenK: did you commit the cardinal sin of testing multiple orthogonal things at once?04:53
StevenKlifeless: So I need two tests?04:53
lifelessstevebaker: I think we need one more test for the occ patch05:00
*** matsuhashi has joined #tripleo05:03
*** nati_ueno has joined #tripleo05:04
*** shakayumi has quit IRC05:04
*** nati_ueno has quit IRC05:04
openstackgerritDmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Add NeutronControlPlaneID parameter  https://review.openstack.org/9134605:07
openstackgerritDmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Introduce configurable virtual ip in templates  https://review.openstack.org/8955605:12
openstackgerritDmitry Shulyak proposed a change to openstack/tripleo-incubator: Pass NeutronControlPlaneID to heat stack-create  https://review.openstack.org/9134905:14
*** ramishra has quit IRC05:15
openstackgerritDmitry Shulyak proposed a change to openstack/tripleo-incubator: Add keepalived to overcloud, control vip to heat  https://review.openstack.org/8961305:18
*** ramishra has joined #tripleo05:27
*** shakayumi has joined #tripleo05:30
lifelessSpamapS: so hi; when you have a chance I'd like to sync on the status of graceful...05:34
*** shakayumi has quit IRC05:35
*** ramishra has quit IRC05:36
openstackgerritlifeless proposed a change to openstack/tripleo-specs: Add .gitreview.  https://review.openstack.org/9174105:36
SpamapSlifeless: state of graceful is that I got sucked into "let's make CI stop sqawking and breaking so we can focus" and haven't poked at my POC resource since my initial super hacky 20 line attempt.. which failed.05:38
lifelessSpamapS: ok05:39
lifelessSpamapS: to answer your testenv question - sure.05:39
lifelessSpamapS: more == better, just be sure to scale the test cloud too05:39
lifelesswe need num(testenvs) <= jenkins-slaves-in-active-state05:40
lifelessalso we need to add 2 more Vms to every testenvs shortly for HA05:40
SpamapSlifeless: right, we have room for 8 more jenkins slaves right now, but only room for 40 testenvs. So we could add 2 boxes and just bump quota without adding more computes.05:40
SpamapSanyway05:41
SpamapSMore to the point.. trusty rollout.05:41
lifelessgreghaynes: dprince may need active poking on that -105:41
lifelessgreghaynes: if you disagree with it, it is in your power to approve ;P05:41
lifelessSpamapS: yes indeedy05:41
lifelessSpamapS: https://review.openstack.org/#/c/84076/ is relevant to that05:41
SpamapSlifeless: so tripleo-cd works. But rebuild fails killed it (PXE timing out presumably because of dhcp agent issues.. not 100% sure on that)05:42
* greghaynes emails dprince05:43
lifelessgreghaynes: I updated https://review.openstack.org/#/c/89732/ and it passed CI05:45
lifelessSpamapS: ^ blocker for HA.05:45
greghaynesah, finally05:45
greghayneswas waiting for CI all day05:45
lifelessSpamapS: so, not sure what to suggest on the rebuild05:45
lifelessSpamapS: in principle its 'heat DWIM' but we haven't had that bunfight yet.05:46
SpamapSlifeless: for the rebuild, nova has screwed us anyway because we just lost state.. so that's the bug I reported last night05:46
lifelessSpamapS: one thing you could do is create a stack of just hypervisors and use that to bring up fresh ones05:46
lifelesswith parameters for things needed from the control plane05:47
SpamapSlifeless: Yeah I was thinking that too... just bring up a new stack with new hypervisors and scale off the old one.05:47
SpamapSthat was originally why nova-compute-instance was kept deployable standalone05:47
lifelessor you could abandon/adopt05:48
lifelesswith less resources05:48
SpamapSthats not a terrible option05:48
SpamapShave had success with it05:48
lifelessSpamapS: greghaynes: I'd also like a review timeslice on https://review.openstack.org/#/c/90890/ which has neither passed nor failed CI, but if there are two +2's I can recheck until we do and then +A it - fixes a critical regression05:50
SpamapSok.. time for bed here05:52
tchaypothis isn't right.05:53
tchaypoGoogle tells me it's 10:53pm in US-WEST05:53
tchaypowhy are greghaynes and SpamapS here?05:54
greghaynesI operate in GST, not PST05:54
greghaynesas StevenK put it :)05:54
tchaypo6:54am in london... same comment applies :p05:54
lifelesstchaypo: not GMT, not UTC, GST.05:55
lifelesstchaypo: Greg Standard Time05:55
tchaypohahahahahah05:55
greghaynesype05:55
tchaypovery well then. Greg has my blessing to be online whenever he damn well wants.05:55
greghaynesyep, even05:55
lifelessbnemec: https://review.openstack.org/#/c/89732/ <-05:59
lifelesserm who else05:59
lifelessmarios: ^06:00
lifelessrighto, time to shove off from here for now06:00
lifelessau revoir06:00
*** morganfainberg is now known as morganfainberg_Z06:03
*** ramishra has joined #tripleo06:10
*** rwsu is now known as rwsu-away06:14
proffalkenlifeless: thanks :)06:22
*** shakayumi has joined #tripleo06:31
openstackgerritA change was merged to openstack/os-apply-config: Updated from global requirements  https://review.openstack.org/9123106:35
*** shakayumi has quit IRC06:36
openstackgerritMatthew Macdonald-Wallace proposed a change to openstack/tripleo-specs: Spec to accompany systems-monitoring blueprint  https://review.openstack.org/9175006:39
proffalkenFirst! :P06:39
openstackgerritMatthew Macdonald-Wallace proposed a change to openstack/tripleo-specs: Spec to accompany systems-monitoring blueprint  https://review.openstack.org/9175006:42
proffalkengot carried away and put it in the wrong directory... :(06:42
*** eghobo has quit IRC06:49
*** e0ne has joined #tripleo06:54
*** openstackgerrit has quit IRC06:57
GheRiveromorning all07:09
*** gcha has joined #tripleo07:17
marioslifeless: sorry, missed the ping, looking07:20
*** e0ne has quit IRC07:21
*** jml has quit IRC07:23
*** giulivo has joined #tripleo07:26
*** jml has joined #tripleo07:30
proffalkenAll: Any chance that we can get a review job that checks that scripts are using /bin/bash for the shell? On debian-derived distros, /bin/sh points to /bin/dash and therefore -o pipefail doesn't work07:31
*** shakayumi has joined #tripleo07:32
proffalken(see this trhead for more info: http://www.spinics.net/lists/dash/msg00154.html)07:37
*** shakayumi has quit IRC07:37
*** nati_ueno has joined #tripleo07:55
*** killer_prince is now known as lazy_prince07:56
*** akuznetsov has joined #tripleo07:58
*** stillbert is now known as mikal08:03
*** derekh_ has joined #tripleo08:06
*** dshulyak has joined #tripleo08:15
*** scotty95747 has joined #tripleo08:16
*** chuckC_ has joined #tripleo08:19
*** greghaynes_ has joined #tripleo08:20
*** Hazelesque_ has joined #tripleo08:20
*** dguerri_ has joined #tripleo08:21
*** greghaynes has quit IRC08:21
*** ohadlevy has quit IRC08:21
*** scott-millward has quit IRC08:21
*** NobodyCa1 has joined #tripleo08:21
*** e0ne has joined #tripleo08:21
*** DuncanT-1 has joined #tripleo08:22
*** ohadlevy has joined #tripleo08:26
*** e0ne has quit IRC08:26
*** ohadlevy is now known as Guest4269008:26
*** beekneemech has joined #tripleo08:27
*** nibalize1 has joined #tripleo08:29
*** dkehn_ has joined #tripleo08:29
*** echohead1 has joined #tripleo08:30
*** kiall_ has joined #tripleo08:31
*** jml_ has joined #tripleo08:33
*** shakayumi has joined #tripleo08:33
*** chuckC_ has quit IRC08:34
*** dshulyak has quit IRC08:34
*** akuznetsov has quit IRC08:34
*** jml has quit IRC08:34
*** dkehn has quit IRC08:34
*** jrist has quit IRC08:34
*** Shrews has quit IRC08:34
*** kiall has quit IRC08:34
*** bnemec has quit IRC08:34
*** echohead_ has quit IRC08:34
*** d0ugal has quit IRC08:34
*** nibalizer has quit IRC08:34
*** jml_ is now known as jml08:34
*** jml has quit IRC08:34
*** jml has joined #tripleo08:34
*** chuckC has quit IRC08:34
*** dguerri has quit IRC08:34
*** NobodyCam has quit IRC08:34
*** Hazelesque has quit IRC08:34
*** DuncanT- has quit IRC08:34
*** shakayumi has quit IRC08:37
goneriHi, can a core take a look on https://review.openstack.org/#/c/91430/ to get the fix for the Debian element merged.08:39
*** jrist has joined #tripleo08:40
*** hashar has joined #tripleo08:46
*** dividehex has quit IRC08:48
*** martyntaylor has joined #tripleo08:50
*** Shrews has joined #tripleo08:52
*** tzumainn has joined #tripleo08:52
*** chuckC has joined #tripleo08:53
*** dshulyak has joined #tripleo08:53
*** akuznetsov has joined #tripleo08:53
*** d0ugal has joined #tripleo08:53
*** rwsu-away has quit IRC08:56
*** nati_ueno has quit IRC08:56
derekh_the CI F20 jobs is consistently failing, appears to be because of a timeout waiting for rabbit to restart one of https://review.openstack.org/91757 or https://review.openstack.org/91758 should fix it, lets see what CI thinks08:58
*** lucasagomes has joined #tripleo08:59
derekh_kiall_: was that your patch ? adding "timeout 10 rabbitmqctl wait /var/run/rabbitmq/pid"09:02
*** matsuhashi has quit IRC09:04
*** matsuhashi has joined #tripleo09:05
*** athomas has joined #tripleo09:17
derekh_tripleo-specs docs jobs fails, needs https://review.openstack.org/#/c/91754/ befor we can merge anything09:19
*** e0ne has joined #tripleo09:22
*** dividehex has joined #tripleo09:23
*** darraghb has joined #tripleo09:27
*** e0ne has quit IRC09:27
*** chuckC has quit IRC09:28
*** chuckC has joined #tripleo09:29
*** yamahata has quit IRC09:30
proffalkenIf I wanted to add a new gate/review job to tripleo-image-elements, which repo would I do it in?09:33
*** shakayumi has joined #tripleo09:33
proffalkenI've got two I'd like to add - the first checks that scripts use /bin/bash to avoid the pipefail errors, the second is more of a functional test to confirm that my element has installed/configured correctly09:36
*** petertoft has joined #tripleo09:37
*** shakayumi has quit IRC09:38
shadowerderekh_: are there any more tests to be run for https://review.openstack.org/#/c/91754/ or can we +A?09:38
shadower(didn't see anything in zuul but want to make sure)09:39
*** jcoufal has joined #tripleo09:39
jangdaily call to get another +2 +a on this: https://review.openstack.org/#/c/86348/ ... it's _definitely_ passed the critical point in CI now.09:41
derekh_shadower: the tripleo ci test don't run over that repository so we are good to merge09:41
shadowerderekh_: approved, thanks09:41
derekh_jang: done, thanks09:42
*** gcha has quit IRC09:43
jangmerci Derek.09:46
*** nati_ueno has joined #tripleo09:57
*** nati_ueno has quit IRC10:01
*** nati_ueno has joined #tripleo10:02
*** nati_uen_ has joined #tripleo10:04
*** athomas_ has joined #tripleo10:07
*** nati_ueno has quit IRC10:07
*** nati_uen_ has quit IRC10:09
*** jcoufal has quit IRC10:13
goneriproffalken: I think you should extend bin/dib-lint first10:18
goneriproffalken: there is already a bunch of commits to extend dib-lint https://review.openstack.org/#/q/status:open+project:openstack/diskimage-builder+branch:master+topic:lint,n,z10:19
gonerihttps://review.openstack.org/#/c/88534/ and https://review.openstack.org/#/c/88531/ too10:19
*** e0ne has joined #tripleo10:21
*** hashar has quit IRC10:22
proffalkengoneri: thanks, I'll take a look now10:22
*** e0ne has quit IRC10:22
*** dguerri_ is now known as dguerri10:23
*** e0ne has joined #tripleo10:23
*** jcoufal has joined #tripleo10:23
*** giulivo has quit IRC10:24
*** hashar has joined #tripleo10:24
*** e0ne has quit IRC10:25
*** giulivo has joined #tripleo10:27
*** ramishra has quit IRC10:28
*** akuznetsov has quit IRC10:29
*** akuznetsov has joined #tripleo10:30
*** markmc has joined #tripleo10:31
*** lazy_prince is now known as killer_prince10:33
*** shakayumi has joined #tripleo10:34
*** matsuhashi has quit IRC10:35
*** matsuhashi has joined #tripleo10:37
*** shakayumi has quit IRC10:39
*** e0ne has joined #tripleo10:49
*** jcoufal has quit IRC10:53
*** nati_ueno has joined #tripleo10:53
*** nati_ueno has quit IRC10:58
*** hewbrocca has joined #tripleo11:02
*** hashar has quit IRC11:02
*** DuncanT-1 is now known as DuncanT11:17
*** matsuhashi has quit IRC11:22
derekh_ccorrigan: I'm guessing that was an accident https://review.openstack.org/#/c/91768/11:32
*** shakayumi has joined #tripleo11:35
*** nosnos has quit IRC11:37
*** shakayumi has quit IRC11:39
*** killer_prince has quit IRC11:42
*** Hazelesque_ is now known as Hazelesque11:44
*** matsuhashi has joined #tripleo11:48
*** hashar has joined #tripleo11:48
*** gcha has joined #tripleo11:51
*** nosnos has joined #tripleo11:53
*** nati_ueno has joined #tripleo11:54
*** lucasagomes is now known as lucas-hungry11:59
*** nati_ueno has quit IRC11:59
*** hewbrocca has quit IRC12:11
*** mrunge has joined #tripleo12:21
*** weshay has joined #tripleo12:29
goneriderekh_: for https://review.openstack.org/#/c/88534/ there is another review for the tab detection ( I071262ff9f6599548f869f5439ee127f64eeb46f ). Is that fine or should I drop the :space:?12:31
*** athomas_ has quit IRC12:34
*** athomas has quit IRC12:34
*** shakamunyi has joined #tripleo12:36
*** nosnos has quit IRC12:39
ccorriganderekh_:  Schoolboy error on my part, second patch abandoned, original updated...12:40
*** shakamunyi has quit IRC12:41
*** jdob has joined #tripleo12:42
*** nati_ueno has joined #tripleo12:44
*** rlandy has joined #tripleo12:45
*** nati_ueno has quit IRC12:48
*** dprince has joined #tripleo12:50
*** jdob has quit IRC12:59
*** matty_dubs|gone is now known as matty_dubs13:00
*** athomas has joined #tripleo13:01
*** ramishra has joined #tripleo13:01
*** jdob has joined #tripleo13:02
*** andrearosa has quit IRC13:03
*** matsuhashi has quit IRC13:04
*** matsuhashi has joined #tripleo13:04
*** morazi has joined #tripleo13:04
*** lucas-hungry is now known as lucasagomes13:05
*** matsuhas_ has joined #tripleo13:08
*** matsuhashi has quit IRC13:08
derekh_goneri: looking13:17
derekh_ccorrigan: thanks13:17
*** matsuhas_ has quit IRC13:18
*** hewbrocca has joined #tripleo13:22
*** Rakesh6 has quit IRC13:24
*** jdob has quit IRC13:24
*** jdob has joined #tripleo13:26
*** killer_prince has joined #tripleo13:28
*** funzo has joined #tripleo13:31
*** funzo is now known as Guest9427213:31
derekh_goneri: replied, I'm still on the side of changing to " " to avoid confusion13:32
goneriderekh_: ok, perfect. give me 10 minutes to refresh the patch :)13:33
derekh_goneri: ok, thanks13:33
kiall_derekh_ / dprince - re https://review.openstack.org/#/c/91758/ .. Any logs etc related to the timeout? If you're hitting the timeout, something else is wrong13:36
*** shakamunyi has joined #tripleo13:37
derekh_kiall_: yup, one sec13:37
dprincekiall_: not sure. my take is reverting this should be safe... and once we research it further we can re-add your patch as required13:37
derekh_kiall_: so F20 jobs have been failing all day http://goodsquishy.com/downloads/tripleo-jobs.html13:38
derekh_kiall_: taking one example13:38
*** Guest94272 is now known as funzo13:38
kiall_I'll break anyone who's running on a slow system ;) Without the wait, cluster_status returns different info during startup13:38
*** kiall_ is now known as Kiall13:38
derekh_Kiall: http://logs.openstack.org/20/91520/3/check-tripleo/check-tripleo-overcloud-f20/3ec23e3/seed_logs.tar.xz13:39
derekh_Kiall: search var/log/messages for "May  2 03:06:05"13:40
derekh_:q13:40
*** shakamunyi has quit IRC13:41
KiallI see it.. Where does F20 has the default pid file for RMQ?13:41
Kiall(Without OOO changing it..)13:41
KiallI saw something similar on ubuntu/debian, where RMQ started at boot, and `service rabbitmq-server restart`, failed to actually restart it13:42
Kiallthe bash rabbitmq-server script that wraps the erlang process got restarted, but the actual erlang process did not13:42
derekh_Kiall: looks like its the same place /var/run/rabbitmq/pid , which was created the same time erlang started, let me see how long it took13:44
KiallLooks like /run/rabbitmq/pid .. is /run a symlink/bind to /var/run on F20? (or the reverse...)13:44
*** nati_ueno has joined #tripleo13:45
*** mugsie has joined #tripleo13:45
derekh_Kiall: so locally here, the timeout triggered the first time os-collect-config ran , but its was ok the second time round,13:47
derekh_Kiall: I've also submitted a patch to increase the timeout to 20 se see if that help CI13:47
Kiallderekh_: maybe the CI is just .. slow?13:47
Kiallyea13:47
derekh_Kiall: possibly, see see once the job runs13:47
derekh_*well see13:48
derekh_*we'll see :-)13:48
derekh_phew13:48
KiallEither way .. If RMQ *actually* hasn't started as against just the timeout being triggered, you have bigger issues on F20 once you try do clustering :)13:48
KiallRMQ shows very different info during started for `rabbitmqctl cluster_status`, which causes other issues in the script13:49
Kiallduring startup*13:49
goneriderekh_: https://review.openstack.org/#/c/88534/3/bin/dib-lint13:49
*** nati_ueno has quit IRC13:50
*** rcarrill` has joined #tripleo13:52
*** shakamunyi has joined #tripleo13:52
derekh_Kiall: ok, lets see what happens when it runs and if there are other problems it should become clear13:52
derekh_goneri: looking13:52
*** rcarrillocruz has quit IRC13:53
*** openstackgerrit has joined #tripleo14:09
*** andrearosa has joined #tripleo14:10
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Expose passthrough configuration in undercloud.  https://review.openstack.org/9179214:10
*** chuckC has quit IRC14:14
*** NobodyCa1 is now known as NobodyCam14:21
*** killer_prince is now known as lazy_prince14:23
openstackgerritMatthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Install the "classic" icinga interface  https://review.openstack.org/8722314:23
openstackgerritMatthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Install check_mk server  https://review.openstack.org/8722614:24
*** lazy_prince is now known as killer_prince14:25
openstackgerritMatthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Auto-collect local checks  https://review.openstack.org/9132414:25
proffalkenlifeless: there's a some of updates/refactoring to those patches, hope they meet your high standards! :)14:26
*** shakamunyi has quit IRC14:31
*** shakamunyi has joined #tripleo14:31
*** shakamunyi has quit IRC14:32
ccrouchgiulivo: welcome to the tripleo effort!14:32
openstackgerritAlexis Lee proposed a change to openstack/tripleo-specs: Factor data out of _check_titles  https://review.openstack.org/9180114:32
openstackgerritAlexis Lee proposed a change to openstack/tripleo-specs: Allow additional titles + subtitles  https://review.openstack.org/9180214:32
*** shakamunyi has joined #tripleo14:32
giulivoccrouch , hi all :)14:32
giulivowas lurking into the chan from a while now14:32
ccrouchbut now its official :-)14:33
*** slagle has quit IRC14:33
dprincegiulivo: welcome14:34
*** slagle has joined #tripleo14:37
derekh_giulivo: hi and welcome14:40
openstackgerritCian O'Driscoll proposed a change to openstack-infra/tripleo-ci: Test the upgrade codepath works as well.  https://review.openstack.org/8775814:48
openstackgerritStuart McLaren proposed a change to openstack/tripleo-heat-templates: ssl: Add support for a CA Certificate  https://review.openstack.org/8510014:48
*** chuckC has joined #tripleo14:53
*** shakamunyi has quit IRC14:53
*** shakamunyi has joined #tripleo14:54
*** shakamunyi has quit IRC14:56
*** shakamunyi has joined #tripleo14:56
*** shakayumi has joined #tripleo14:59
*** shakamunyi has quit IRC15:01
*** nati_ueno has joined #tripleo15:02
rbrady-afkgiulivo!  Welcome!15:02
*** rbrady-afk is now known as rbrady15:02
giulivorbrady, :D15:02
*** shakayumi has quit IRC15:03
*** shakamunyi has joined #tripleo15:03
*** shakamunyi has quit IRC15:03
*** ramishra_ has joined #tripleo15:05
openstackgerrittherese-mchale proposed a change to openstack/tripleo-image-elements: Use Percona Packages for Ubuntu  https://review.openstack.org/9013415:05
*** ramishra has quit IRC15:06
*** corrigac has joined #tripleo15:10
*** andrearosa has quit IRC15:17
*** rcarrill` has quit IRC15:18
*** shakamunyi has joined #tripleo15:18
*** rameshg87 has joined #tripleo15:22
*** rcarrillocruz has joined #tripleo15:23
*** dkehn_ is now known as dkehn15:25
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files  https://review.openstack.org/8853115:25
openstackgerritDmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Introduce configurable virtual ip in templates  https://review.openstack.org/8955615:29
*** marun has quit IRC15:30
*** andrearosa has joined #tripleo15:31
goneriderekh_: it should be fine now: https://review.openstack.org/#/c/88531/4/bin/dib-lint15:31
*** marun has joined #tripleo15:31
*** nati_ueno has quit IRC15:35
openstackgerritA change was merged to openstack/tripleo-incubator: Run pull-tools before setup-clienttools  https://review.openstack.org/9143215:35
*** shakayumi has joined #tripleo15:37
openstackgerritA change was merged to openstack/tripleo-image-elements: Openvswitch 2.1 removes openvswitch-controller  https://review.openstack.org/8634815:37
openstackgerritA change was merged to openstack/diskimage-builder: debian: properly deal with Debian stable/unstable  https://review.openstack.org/9143015:37
*** shakamunyi has quit IRC15:38
openstackgerritjan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted  https://review.openstack.org/9180915:41
derekh_goneri: small nit inline, then I'm happy, sorry15:45
derekh_goneri: this begs the question, how is the pep8 test passing, I don't think its searching the elements directory in CI, will fix15:46
*** yamahata has joined #tripleo15:51
*** matty_dubs is now known as matty_dubs|lunch15:53
openstackgerritNicholas Randon proposed a change to openstack/tripleo-heat-templates: Add passthrough configuration glue.  https://review.openstack.org/9181015:56
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files  https://review.openstack.org/8853115:57
goneriderekh_: done, I dropped the two \n15:57
derekh_goneri: forget my last comment about why pep8 is not passing, I was looking at an old version of a file with tabs15:58
goneriderekh_: this \n confusion is due to the current dib-lint. It doesn't finish with a \n and git complains about that.15:59
goneriYet another check for dib-lint? :)15:59
*** eghobo has joined #tripleo15:59
derekh_goneri: sounds like it16:00
*** killer_prince is now known as lazy_prince16:05
*** lazy_prince is now known as killer_prince16:06
*** martyntaylor has quit IRC16:07
*** sdake_ has quit IRC16:08
*** killer_prince is now known as lazy_prince16:13
*** lazy_prince is now known as killer_prince16:14
openstackgerritA change was merged to openstack/tripleo-image-elements: Revert "RabbitMQ - Increase restart timeouts"  https://review.openstack.org/9175816:15
openstackgerritjan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted  https://review.openstack.org/9180916:18
*** corrigac has quit IRC16:24
*** killer_prince is now known as lazy_prince16:24
derekh_jang: was that ^ the reason the timeout didn't work on fedora?16:26
*** ramishra_ has quit IRC16:27
derekh_Kiall: both the revert and the increased timeout have passed CI https://review.openstack.org/#/c/91757/1 https://review.openstack.org/#/c/91758/16:27
derekh_Kiall: the revert has merged, should we just try your patch again with a longer timeout ?16:28
*** jogo is now known as flashgordon16:29
*** yamahata has quit IRC16:31
*** yamahata has joined #tripleo16:31
*** akuznetsov has quit IRC16:31
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files  https://review.openstack.org/8853116:32
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation  https://review.openstack.org/8853416:32
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line  https://review.openstack.org/9181816:32
*** akuznetsov has joined #tripleo16:32
*** corrigac has joined #tripleo16:34
*** e0ne has quit IRC16:34
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line  https://review.openstack.org/9181816:34
*** hashar has quit IRC16:38
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: add some missing \n at end of file  https://review.openstack.org/9182316:39
*** derekh_ has quit IRC16:43
openstackgerritjan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted  https://review.openstack.org/9180916:45
*** hashar has joined #tripleo16:47
jangderekh_: yes, I think it probably was16:49
jangfedora is a systemd os, isn't it?16:49
jangso, the idea in the latest version of that patch will probably need applying - I don't know the systemd equivalent, I'm afraid.16:50
*** saurabhs has joined #tripleo16:51
jangbut, the above patch also has the added advantage that it seems to shave about 60s off a boot on an upstart system as it doesn't muck around deciding that a misconfigured rabbit isn't going to start16:52
*** newell has joined #tripleo16:53
*** matty_dubs|lunch is now known as matty_dubs16:54
jangif Dan Prince is around he can probably turn that into fedora-isms. (Apologies if you're also fedora-savvy, I lose track)16:54
*** giulivo has quit IRC16:55
openstackgerritA change was merged to openstack/tripleo-incubator: Install and configure NTP in undercloud/overcloud  https://review.openstack.org/8864716:57
*** giulivo has joined #tripleo16:58
*** ramishra has joined #tripleo17:01
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files  https://review.openstack.org/8853117:05
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation  https://review.openstack.org/8853417:05
*** e0ne has joined #tripleo17:05
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: 4 spaces indent  https://review.openstack.org/9182917:05
*** ramishra has quit IRC17:06
*** e0ne has quit IRC17:09
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: Eliminate 'tr' in favor of inline bash.  https://review.openstack.org/9183217:11
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: 4 spaces indent  https://review.openstack.org/9182917:13
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation  https://review.openstack.org/8853417:13
*** hashar has quit IRC17:14
*** petertoft has quit IRC17:15
*** gcha has quit IRC17:16
*** e0ne has joined #tripleo17:17
*** e0ne has quit IRC17:19
*** rameshg87 has quit IRC17:20
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line  https://review.openstack.org/9181817:23
*** athomas has quit IRC17:24
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: Make sure all scripts are set -e  https://review.openstack.org/8163717:24
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: Check for set -o pipefail  https://review.openstack.org/8392917:24
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: set -u and -o pipefail everywhere  https://review.openstack.org/8486817:24
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: Ensure scripts are set -u  https://review.openstack.org/8392817:24
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: Allow excluding tests from certain files  https://review.openstack.org/9042517:24
openstackgerritGonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation  https://review.openstack.org/8853417:25
gonerisorry for the flood :/17:26
jangif you manage to get openstackgerrit kickbanned for flooding, _then_ you need to apologise17:26
gonerihopefuly I won't have to aplogise. My rebases are finished :)17:29
*** corrigac has quit IRC17:32
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-heat-templates: Replace nagios3 template with HOT rewrite  https://review.openstack.org/9066517:33
*** morganfainberg_Z is now known as morganfainberg17:37
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Separate Heat BM and VM configs for Nova-BM.  https://review.openstack.org/9003317:41
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file  https://review.openstack.org/9171717:41
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Allows user-specified PXE timeout for overcloud  https://review.openstack.org/8660517:42
* beekneemech sets a goal in life of getting the bot kicked17:42
*** eghobo has quit IRC17:43
*** markmc has quit IRC17:46
*** TravT has joined #tripleo17:48
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file  https://review.openstack.org/9171717:49
*** shakayumi has quit IRC17:52
*** e0ne has joined #tripleo17:52
beekneemech/home/bnemec/tripleo/tripleo-incubator/scripts/devtest_undercloud.sh: line 59: USE_UNDERCLOUD_UI: unbound variable17:52
beekneemechBleh, apparently devtest_variables was not run.17:52
openstackgerritA change was merged to openstack-infra/tripleo-ci: Move the testenv json file to /etc  https://review.openstack.org/8847317:53
*** rcarrill` has joined #tripleo17:53
chuckCHi all.  I'm setting up my dev environment (all on a laptop) and would like to go with ubuntu/kvm (have Windows now). Any early advice (stuff to avoid) before I get started?17:53
*** eghobo has joined #tripleo17:53
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file  https://review.openstack.org/9171717:54
*** rcarrillocruz has quit IRC17:54
beekneemechI wonder if we should re-exec devtest after updating tripleo-incubator.17:54
beekneemechI think that was my problem - devtest_variables was sourced, but it was the old version.17:55
greghaynes_yea, you have to at a minimum re-source that17:56
*** e0ne has quit IRC17:56
greghaynes_woo for env vars as parameters :/17:57
greghaynes_chuckC: just make sure youre at least on saucy if you ubuntu17:57
SpamapSyay, we're finally seeing jobs go through zuul17:57
greghaynes_er, saucy or newer17:57
SpamapSgreghaynes_: death to the environment variables!! ;)17:58
chuckCgreghaynes: thanks, I was about to go precise18:03
*** greghaynes_ is now known as greghaynes18:04
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Separate Heat BM and VM configs for Nova-BM.  https://review.openstack.org/9003318:04
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Allows user-specified PXE timeout for overcloud  https://review.openstack.org/8660518:04
*** cody-somerville has quit IRC18:06
openstackgerritNicholas Randon proposed a change to openstack/tripleo-incubator: Expose passthrough configuration in undercloud.  https://review.openstack.org/9179218:08
*** dshurtle has joined #tripleo18:12
*** corrigac has joined #tripleo18:16
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ hostname in nova.conf  https://review.openstack.org/9184618:17
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in neutron.conf  https://review.openstack.org/9184718:18
*** cody-somerville has joined #tripleo18:18
*** cody-somerville has joined #tripleo18:18
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in nova.conf  https://review.openstack.org/9184618:19
*** corrigac_ has joined #tripleo18:19
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ userid in nova.conf  https://review.openstack.org/9184618:20
*** darraghb has quit IRC18:21
*** corrigac has quit IRC18:21
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in neutron.conf  https://review.openstack.org/9184718:23
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ userid in nova.conf  https://review.openstack.org/9184618:24
* SpamapS spam spam spam spam18:24
greghaynestchaypo: You looked at concoord for picking leaders right? Curious why that didnt pan out18:24
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in glance-api.conf  https://review.openstack.org/9185218:26
SpamapSgreghaynes: leader election is really hard without built in support from the state store itself. What thing are we needing a leader for?18:26
greghaynesfor the stuff currently using the os-is-bootstrap-host hack18:27
*** akuznets_ has joined #tripleo18:27
greghaynesI suspect we were probably 'using it wrong'(TM)18:28
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in heat.conf  https://review.openstack.org/9185318:28
SpamapSgreghaynes: ah yes that18:28
*** akuznetsov has quit IRC18:28
greghaynessince concoord is paxos (not multi paxos) you dont get a strong leader, but you still get consensus which should be enough to pick who performs a certain operation18:29
greghaynesor we could just use something different18:29
SpamapSIt's as important that you get "not a leader" choices too. :)18:30
SpamapSgreghaynes: is concoord unicast based?18:30
greghaynesdont think so, but no idea18:31
greghaynesnot sure why that would matter?18:31
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in cinder.conf  https://review.openstack.org/9185518:32
SpamapSgreghaynes: harder to test broadcast systems18:32
greghaynesah18:32
SpamapSgreghaynes: and some environments constrain such things18:32
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ironic.conf  https://review.openstack.org/9185718:33
*** d0ugal has quit IRC18:34
*** d0ugal has joined #tripleo18:34
*** d0ugal has quit IRC18:34
*** d0ugal has joined #tripleo18:34
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ironic.conf  https://review.openstack.org/9185718:34
SpamapSlovely spaaaaaaaaammm18:34
*** corrigac_ has quit IRC18:34
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ceilometer.conf  https://review.openstack.org/9185918:35
dprinceSpamapS: Should we squash all these rabbit_userid changes?18:36
SpamapSdprince: why?18:37
dprinceSpamapS: all seem related to me, revertability is nice but in this case I think they are close enough18:37
SpamapSWe're not going to revert these.18:38
dprinceSpamapS: When I have lots of related changes I often try to bunch them rather than split it out. Like I did for the commen-venv stuff (I could have made that 15 changes)18:38
dprinceSpamapS: all the more reason to squash them into a single commit...18:38
SpamapSI get that. But I prefer to have them all separate so there's no rebase dance.18:38
dprinceSpamapS: right, but you'll have the get all the CI jobs passing dance :(18:38
SpamapSIf we can't get 5 passes......18:39
dprinceSpamapS: do as you like. We have ETOOMANY reviews right now so I say squash where appropriate18:39
SpamapSThese are as tiny and simple as possible.18:40
dprinceSpamapS: I'm suggesting you combine them (squash)18:40
SpamapSI am fine with a squash... really. It's just not what I do by default.18:40
dprinceSpamapS: if we won't revert these in isolation to me it is about the relatedness of the change and how big the patchset is (is it reviewable). Your squashed patch would only be 5 lines18:42
SpamapSI know18:42
SpamapSFine18:42
SpamapSmore work18:42
SpamapSgah18:42
* dprince isn't trying to pick on SpamapS, just set a good precedent18:43
SpamapSIt's my first ever "too small" patch argument18:43
SpamapSand I have about 2 minutes18:44
SpamapSso forgive me for being annoyed18:44
SpamapSnothing to do with logic or reason18:44
*** jang1 has joined #tripleo18:47
openstackgerritClint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user  https://review.openstack.org/9186118:48
SpamapSdprince: done18:50
SpamapSand all others abandoned18:50
dprinceSpamapS: :), have a good weekend18:50
SpamapSPool is clean, and 78F, and I've a case of XX ... I expect to float my way to happiness.18:51
* greghaynes jealous18:52
SpamapSgreghaynes: yes, enjoy your clouds.18:53
SpamapSand more clouds18:53
SpamapSand piles of extra money that you don't spend on housing. ;)18:53
greghaynes:( it was actually sunny this week (I think 80something yesterday) but yes, rain this weekend18:53
* beekneemech has no clouds for the first time in a week18:53
openstackgerritAdam Gandelman proposed a change to openstack/diskimage-builder: Reinstate Trusty as default for Ubuntu  https://review.openstack.org/9186418:54
adam_gSpamapS, ^18:54
greghaynesadam_g: \O/18:54
*** mrunge has quit IRC18:54
adam_gcheck-tempest-dsvm-virtual-ironic-nv should hopefully pass with that change18:55
victor_lowtherhmmm...18:55
*** e0ne has joined #tripleo18:55
victor_lowtherdid they have to change the SSH host keys in Gerrit?18:55
adam_gvictor_lowther, yeah18:56
victor_lowthergit-review is whining.18:56
*** e0ne has quit IRC18:56
adam_gvictor_lowther, http://lists.openstack.org/pipermail/openstack-dev/2014-April/033908.html18:56
beekneemechYay heartbleed18:56
victor_lowtheradam_g: OK, cool.  As long as it was not enemy action. :)18:56
SpamapSadam_g: \o/18:58
openstackgerritVictor Lowther proposed a change to openstack/diskimage-builder: Make diskimage-builder work in Docker.  https://review.openstack.org/9056018:59
SpamapSadam_g: fix that commit msg ;)18:59
openstackgerritAdam Gandelman proposed a change to openstack/diskimage-builder: Reinstate Trusty as default for Ubuntu  https://review.openstack.org/9186419:07
openstackgerritVictor Lowther proposed a change to openstack/tripleo-incubator: Let devtest.sh run in a Docker container.  https://review.openstack.org/9186719:08
victor_lowtherhm19:25
victor_lowtherIs there a awy for me to indicate that a change in a project depends on another change in a different project?19:25
*** julim_ has joined #tripleo19:25
greghaynesvictor_lowther: We usually put change-id in the commit msg and possibly WIP until the depended on change lands19:26
greghaynesbasically its all manual19:26
*** tzumainn has quit IRC19:26
victor_lowtherah, OK, because https://review.openstack.org/91867 relies on https://review.openstack.org/9056019:28
*** julim has quit IRC19:28
*** dshulyak has quit IRC19:32
*** hashar has joined #tripleo19:32
*** mrunge has joined #tripleo19:43
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: dpkg: support pkg-map in bin/install-packages  https://review.openstack.org/9160119:45
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: Yum: support pkg-map in bin/install-packages  https://review.openstack.org/9160019:45
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: opensuse: support pkg-map in bin/install-packages  https://review.openstack.org/9160219:45
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: Add pkg-map element.  https://review.openstack.org/9159819:45
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: Set DISTRO_NAME in OS environment.d  https://review.openstack.org/9159919:45
openstackgerritDan Prince proposed a change to openstack/diskimage-builder: Update base element to make use of pkg-map  https://review.openstack.org/9188019:45
*** dprince has quit IRC19:48
*** rcarrillocruz has joined #tripleo19:50
*** rcarrill` has quit IRC19:52
openstackgerritA change was merged to openstack/tripleo-image-elements: MySQL: Ensure libssl-symlink corrects exits on failure  https://review.openstack.org/9053620:13
*** e0ne has joined #tripleo20:20
*** jdob has quit IRC20:28
*** lucasagomes has quit IRC20:30
*** mrunge has quit IRC20:32
*** fandi has joined #tripleo20:35
tchaypogreghaynes: It looks like it does a lot of what we need: it looks like it would be easy to build the state object on top of it, and it handlesdetecting missing members20:41
tchaypoand broadcasting to live members nicely20:42
tchaypoFor leader election, it just picks the node with the lowest port number. That might be fine for us, we can implement put own election via the state machine20:43
*** marun has quit IRC20:45
tchaypoBigger problem is that it sets quorum to (n/2)+1, which seems good, except that n is the number of currently live nodes. If we have 7 and lose 4, quorum is now 2 rather than an unacheivable 420:45
greghaynesI guess my thinking was we could use the underlying log for picking who the leader is for a given operation20:46
greghaynesnot sure if it exposes it, but id be surprised if we cant just make something ontop that uses it20:46
tchaypoAfaict we need to either patch it to make the quorum not drop as members leave, or re-implement the liveness/quorum checking on top20:47
greghaynesah, yes thats annoying20:48
tchaypoYep, we should be able to do that. I don't think the leader election really matters: whoever makes a successful proposal is the de facto leader for that proposal20:48
tchaypoIf each node comes up and proposes itself as leader, then looks for other proposals and approves the earliest outstanding one it sees, we should arrive at consensus fairly quickly20:49
tchaypoIf the existing leader re-proposes after n seconds and everyone else proposes after 2n we should keep an existing leader unless they die20:50
tchaypoIt's not unfixable, but we didn't have a use for it ready just then so I left it alone20:51
greghayneswell, I dont think you even need to wory about that leader logic - paxos should take care of that20:53
greghaynesall we have to care about is what proposal actually gets commited to the log20:53
*** tzumainn has joined #tripleo21:04
*** e0ne has quit IRC21:08
openstackgerritJordan OMara proposed a change to openstack/tuskar-ui: Adding missing setup for instack  https://review.openstack.org/9107821:09
*** matty_dubs is now known as matty_dubs|gone21:11
*** weshay has quit IRC21:19
*** weshay has joined #tripleo21:20
*** weshay has quit IRC21:24
*** sdake_ has joined #tripleo21:24
*** sdake_ has quit IRC21:24
*** sdake_ has joined #tripleo21:24
*** weshay has joined #tripleo21:27
*** tzumainn has quit IRC21:31
*** martyntaylor has joined #tripleo21:32
*** d0ugal has quit IRC21:33
*** weshay has quit IRC21:45
*** hashar has quit IRC21:49
*** newell has quit IRC21:50
*** ccrouch has quit IRC21:54
*** ccrouch has joined #tripleo21:55
*** newell has joined #tripleo21:56
-openstackstatus- NOTICE: Zuul is being restarted with some dependency upgrades and configuration changes; ETA 221522:00
*** marun has joined #tripleo22:01
*** jang1 has quit IRC22:04
*** sdake_ has quit IRC22:20
*** rlandy has quit IRC22:32
openstackgerritAdam Gandelman proposed a change to openstack/tripleo-incubator: Do not exit 1 from devtest_variables.sh  https://review.openstack.org/9191622:40
*** chuckC has quit IRC22:45
*** rupsky has joined #tripleo23:12
*** rupsky has left #tripleo23:12
*** smulcahy has quit IRC23:21
-openstackstatus- NOTICE: paste.openstack.org is going down for a short database upgrade23:27
*** martyntaylor has quit IRC23:33
*** flashgordon is now known as jogo23:49
openstackgerritDan Prince proposed a change to openstack/tripleo-heat-templates: Configure NTP for overcloud compute  https://review.openstack.org/8869123:51

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