*** shakayumi has joined #tripleo | 00:00 | |
lifeless | geerdest: https://review.openstack.org/#/c/88153/3 - I think its a good idea to do it in undercloud too. | 00:01 |
---|---|---|
lifeless | greghaynes: thanks for the headsup | 00:03 |
*** shakamunyi has quit IRC | 00:05 | |
*** shakayumi has quit IRC | 00:05 | |
lifeless | hmm, missed the EU folk | 00:07 |
lifeless | and spamaps is +0 | 00:07 |
lifeless | who can I pester | 00:07 |
lifeless | bnemec: https://review.openstack.org/#/c/91198/ | 00:08 |
lifeless | greghaynes: SpamapS: https://review.openstack.org/#/c/90890/ <- fix a critical | 00:08 |
*** newell has quit IRC | 00:14 | |
*** TravT has quit IRC | 00:24 | |
*** matsuhashi has joined #tripleo | 00:29 | |
*** mestery has joined #tripleo | 00:37 | |
StevenK | lifeless: https://review.openstack.org/91518 https://review.openstack.org/91519 https://review.openstack.org/91520 https://review.openstack.org/91521 | 00:45 |
lifeless | StevenK: so we need to figure out dinner details; 1) is tchaypo coming; 2) Lynne is available in the city @ 5pm | 00:47 |
lifeless | StevenK: 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 |
openstackgerrit | James Polley proposed a change to openstack/tripleo-image-elements: Nasty hack to work around pip's case-sensitivity https://review.openstack.org/84946 | 00:48 |
lifeless | I need to know to sms lynne details | 00:48 |
*** chuckC has joined #tripleo | 00:48 | |
StevenK | lifeless: I'm guessing soonish? | 00:48 |
lifeless | before 3pm I'd say but earlier better | 00:49 |
StevenK | It looks like Sarah has not reached her desk yet | 00:49 |
lifeless | clarkb: 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 |
tchaypo | lifeless: StevenK has found something even more annoying that starting to talk just as noice cancelling is activated | 00:52 |
tchaypo | lifeless: I'm still on aware mode and he started whispering so I can only just hear him | 00:52 |
lifeless | greghaynes: https://review.openstack.org/#/c/91518/ is ready for a second +2 | 00:52 |
StevenK | That was actually thinking out loud | 00:52 |
tchaypo | so you're saying my headphones stop your thoughts? | 00:53 |
*** jml has quit IRC | 00:53 | |
StevenK | tchaypo: Coincidence, not causation | 00:54 |
lifeless | or is it? | 00:54 |
StevenK | I say it is, so it is. | 00:54 |
*** jml has joined #tripleo | 00:56 | |
lifeless | jml: poke - testtools reviews | 00:59 |
lifeless | StevenK: all reviewed | 00:59 |
tchaypo | I'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 everything | 01:01 |
tchaypo | It's working fine in my testing, but I'd like someone who knows about CI to take a look before it lands | 01:01 |
tchaypo | someone like SpamapS would be perfect, except I don't think he's around | 01:02 |
lifeless | I can look | 01:03 |
lifeless | StevenK: you had a big review for me ? | 01:05 |
openstackgerrit | Trent Geerdes proposed a change to openstack/tripleo-incubator: Fixing typo in comment. https://review.openstack.org/91715 | 01:05 |
StevenK | lifeless: The BIG review is https://review.openstack.org/84933 | 01:07 |
* StevenK goes back to reading keystone code | 01:07 | |
lifeless | greghaynes: happy with https://review.openstack.org/#/c/89732/ now ? | 01:08 |
morganfainberg | StevenK, beware dragons >.> i mean, let me know if you have questions | 01:08 |
tchaypo | To 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 devtest | 01:08 |
uvirtbot | Launchpad bug 1290486 in tripleo "neutron-openvswitch-agent does not recreate flows after ovsdb-server restarts" [Critical,In progress] | 01:08 |
lifeless | morganfainberg: oh hai | 01:09 |
morganfainberg | lifeless hi thar! | 01:09 |
lifeless | morganfainberg: did you see the mailing list thread I pointed you at a couple days back ? | 01:09 |
StevenK | morganfainberg: Oh, was just checking the order of arguments for keystone.endpoints.create(), nothing terrible | 01:09 |
morganfainberg | lifeless, 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 IRC | 01:10 | |
morganfainberg | lifeless, that was the ID vs name one, right? | 01:10 |
lifeless | morganfainberg: yes, the one where our feedback about it being a PITA and hard to use was discounted :( | 01:11 |
morganfainberg | lifeless, 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 |
morganfainberg | lifeless, yep have it in front of me. - i keep IRC chat logs for just that reason :) | 01:14 |
lifeless | :> | 01:14 |
lifeless | thanks! | 01:14 |
StevenK | 144M irclogs | 01:14 |
StevenK | Hm | 01:14 |
StevenK | Maybe I want to compress some | 01:14 |
lifeless | StevenK: or buy a real hard disk? | 01:15 |
morganfainberg | StevenK, 218MB here | 01:15 |
morganfainberg | StevenK, but i also store my on my Google drive :) | 01:15 |
StevenK | lifeless: It's a Linode. Buying a real hard disk will not help in the slightest. | 01:15 |
morganfainberg | StevenK, lets me search from anywhere i have internet | 01:15 |
lifeless | morganfainberg: yeah - the thing is having someone with keystone history weigh in; clint and I are only deployers, so carry little weight. | 01:15 |
morganfainberg | lifeless, i'll def re-read and try and follow up with it soon | 01:15 |
morganfainberg | or at least catchup with people about it in ATL | 01:16 |
lifeless | morganfainberg: no panic - it seems to have gone silent for now. | 01:16 |
lifeless | I suspect SpamapS is sitting on the volcano | 01:16 |
morganfainberg | lifeless, hehe | 01:16 |
tchaypo | I'm not sure what "sitting on the volcano" means, but I think I'm happy to remain ignorant | 01:17 |
lifeless | too bad | 01:18 |
lifeless | supressing frustration at the disconnect | 01:18 |
lifeless | StevenK: sarah is online | 01:22 |
StevenK | lifeless: Yes, I did notice. :-) | 01:23 |
tchaypo | lifeless: I've just noticed that I forgot to mention that I'm in | 01:23 |
StevenK | lifeless: Are you drowning in my horrible Python yet? | 01:24 |
lifeless | like flynn? | 01:24 |
StevenK | As in, 'in like' ? | 01:25 |
lifeless | +1 | 01:29 |
StevenK | lifeless: Is that +1 to my comment, or my branch? | 01:29 |
lifeless | yes | 01:32 |
lifeless | :/ | 01:32 |
lifeless | :\ | 01:32 |
lifeless | :) | 01:32 |
lifeless | proffalken: re-reviewed your three patches | 01:35 |
*** nosnos has joined #tripleo | 01:36 | |
*** ramishra has joined #tripleo | 01:38 | |
openstackgerrit | Trent Geerdes proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file https://review.openstack.org/91717 | 01:40 |
tchaypo | geerdest: that sounds nice | 01:41 |
geerdest | sounds nice isn’t +1 ;) | 01:42 |
tchaypo | I've starred it to take a detailed look later : | 01:44 |
*** nosnos has quit IRC | 01:44 | |
tchaypo | :) | 01:44 |
*** nosnos has joined #tripleo | 01:44 | |
* StevenK gets distracted by this whole section thing | 01:45 | |
lifeless | You're in Glee? | 01:47 |
*** funzo has quit IRC | 01:49 | |
*** weshay has quit IRC | 01:49 | |
*** funzo has joined #tripleo | 01:49 | |
*** funzo has quit IRC | 01:49 | |
*** funzo has joined #tripleo | 01:50 | |
*** funzo is now known as Guest75253 | 01:50 | |
*** nati_uen_ has joined #tripleo | 01:55 | |
*** nati_uen_ has quit IRC | 01:56 | |
*** eguz has joined #tripleo | 01:58 | |
*** nati_ueno has quit IRC | 01:59 | |
*** eghobo has quit IRC | 02:00 | |
*** shakayumi has joined #tripleo | 02:00 | |
lifeless | tchaypo: you had a review you wanted looked at | 02:02 |
tchaypo | yes | 02:03 |
tchaypo | https://review.openstack.org/91288 | 02:03 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Perform PKI initialization for a Keystone host https://review.openstack.org/91520 | 02:03 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI https://review.openstack.org/91521 | 02:03 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Create Keystone v2 endpoint in initialize() https://review.openstack.org/91519 | 02:03 |
*** yamahata has joined #tripleo | 02:04 | |
*** sdake_ has joined #tripleo | 02:04 | |
*** sdake_ has quit IRC | 02:04 | |
*** sdake_ has joined #tripleo | 02:04 | |
*** openstackgerrit has quit IRC | 02:04 | |
*** openstackgerrit has joined #tripleo | 02:06 | |
tchaypo | lifeless waggled his finger at me | 02:08 |
tchaypo | I can't wait to find out what naughty I done. | 02:08 |
lifeless | See the review :) | 02:09 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Perform PKI initialization for a Keystone host https://review.openstack.org/91520 | 02:14 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI https://review.openstack.org/91521 | 02:14 |
*** shakayumi has quit IRC | 02:14 | |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Add a init-keystone CLI https://review.openstack.org/91521 | 02:20 |
*** Guest75253 has quit IRC | 02:26 | |
*** funzo has joined #tripleo | 02:26 | |
*** funzo is now known as Guest22457 | 02:27 | |
*** Guest22457 has quit IRC | 02:31 | |
*** shakayumi has joined #tripleo | 02:31 | |
*** nati_ueno has joined #tripleo | 02:33 | |
*** michchap has quit IRC | 02:43 | |
*** eguz has quit IRC | 02:44 | |
*** michchap has joined #tripleo | 02:45 | |
lifeless | bnemec: / greghaynes: - https://review.openstack.org/#/c/86523/ looks simple and ready | 02:49 |
*** eghobo has joined #tripleo | 02:50 | |
*** nosnos has quit IRC | 02:51 | |
*** eghobo has quit IRC | 02:51 | |
*** nati_ueno has quit IRC | 02:55 | |
*** eghobo has joined #tripleo | 03:24 | |
*** matsuhashi has quit IRC | 03:27 | |
*** tzumainn has quit IRC | 03:37 | |
*** marun has joined #tripleo | 03:37 | |
lifeless | tchaypo: have you seed ?https://review.openstack.org/#/c/89838/ | 03:37 |
tchaypo | lifeless: i had not | 03:42 |
stevebaker | SpamapS: could you restore this change please? https://review.openstack.org/#/c/83831/ | 03:44 |
stevebaker | lifeless: I really need a solution for https://review.openstack.org/#/c/83831/ so that heat software config works again | 03:46 |
lifeless | stevebaker: infra can restore arbitrary changesets I believe | 03:47 |
stevebaker | lifeless: I'll ask, but I was hinting at you reconsidering your -1 ;) | 03:48 |
lifeless | stevebaker: we have software config working without that | 03:49 |
lifeless | stevebaker: so I'll need more data | 03:49 |
stevebaker | lifeless: it affects software-config when the config script is non-json, so shell, puppet, chef, ansible, salt... | 03:50 |
stevebaker | lifeless: and its a regression since writing out the configs as top-levels was a new thing | 03:50 |
lifeless | stevebaker: so why are you pushing that into the space oac is reading from? | 03:51 |
stevebaker | I'm not, we ignore those top-levels. It was something SpamapS added to occ for tripleo config | 03:51 |
*** Rakesh6 has joined #tripleo | 03:52 | |
*** nati_ueno has joined #tripleo | 03:53 | |
stevebaker | lifeless: maybe the fix is to not write out the configs as top-levels if it isn't a dict | 03:53 |
lifeless | stevebaker: so my concern here is that this will be indistinguishable from a bad json config | 03:53 |
lifeless | stevebaker: I am not a fan of quietly hiding errors except is very constrained circumstances | 03:54 |
lifeless | since the configs for a node are user supplied, this isn't a constrained circumstance | 03:54 |
lifeless | stevebaker: if we can arrange to only be supplying the json configs then thats a whole lot better | 03:54 |
lifeless | stevebaker: so - does software-config know that a config is metadata vs shell ? | 03:55 |
stevebaker | lifeless: 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 group | 03:57 |
stevebaker | lifeless: see group http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::StructuredConfig | 03:58 |
lifeless | stevebaker: I think that makes a lot of sense | 03:59 |
lifeless | stevebaker: so an occ change to look for group= "" (backwards compat, deprecated) or "os-apply-config" | 03:59 |
stevebaker | yes, except default group is "Heat::Ungrouped" | 04:00 |
lifeless | ok | 04:00 |
stevebaker | I'll whip something up | 04:01 |
openstackgerrit | lifeless proposed a change to openstack/tripleo-incubator: Make bootstrapping TripleO development easier. https://review.openstack.org/85636 | 04:03 |
lifeless | tchaypo: configure-tripleo repushed | 04:03 |
*** nosnos has joined #tripleo | 04:05 | |
openstackgerrit | Steve Baker proposed a change to openstack/tripleo-heat-templates: Specify group os-apply-config for config resources https://review.openstack.org/91731 | 04:05 |
*** eghobo has quit IRC | 04:08 | |
openstackgerrit | lifeless proposed a change to openstack/tripleo-image-elements: Tune deploy-ci-overcloud a little. https://review.openstack.org/84076 | 04:24 |
openstackgerrit | lifeless proposed a change to openstack/tripleo-image-elements: Fix the HP region deploy-ci-overcloud script https://review.openstack.org/84076 | 04:26 |
openstackgerrit | Steve Kowalik proposed a change to openstack/os-cloud-config: Add a register-nodes command and associated API https://review.openstack.org/84933 | 04:26 |
openstackgerrit | Steve Baker proposed a change to openstack/os-collect-config: Only top-level some deployment configs https://review.openstack.org/91733 | 04:37 |
stevebaker | lifeless: ^ works for me | 04:37 |
lifeless | reviewed | 04:42 |
*** nati_ueno has quit IRC | 04:43 | |
*** eghobo has joined #tripleo | 04:44 | |
StevenK | lifeless: 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 |
lifeless | StevenK: I saw one test called ..._options | 04:53 |
lifeless | StevenK: did you commit the cardinal sin of testing multiple orthogonal things at once? | 04:53 |
StevenK | lifeless: So I need two tests? | 04:53 |
lifeless | stevebaker: I think we need one more test for the occ patch | 05:00 |
*** matsuhashi has joined #tripleo | 05:03 | |
*** nati_ueno has joined #tripleo | 05:04 | |
*** shakayumi has quit IRC | 05:04 | |
*** nati_ueno has quit IRC | 05:04 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Add NeutronControlPlaneID parameter https://review.openstack.org/91346 | 05:07 |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Introduce configurable virtual ip in templates https://review.openstack.org/89556 | 05:12 |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/tripleo-incubator: Pass NeutronControlPlaneID to heat stack-create https://review.openstack.org/91349 | 05:14 |
*** ramishra has quit IRC | 05:15 | |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/tripleo-incubator: Add keepalived to overcloud, control vip to heat https://review.openstack.org/89613 | 05:18 |
*** ramishra has joined #tripleo | 05:27 | |
*** shakayumi has joined #tripleo | 05:30 | |
lifeless | SpamapS: so hi; when you have a chance I'd like to sync on the status of graceful... | 05:34 |
*** shakayumi has quit IRC | 05:35 | |
*** ramishra has quit IRC | 05:36 | |
openstackgerrit | lifeless proposed a change to openstack/tripleo-specs: Add .gitreview. https://review.openstack.org/91741 | 05:36 |
SpamapS | lifeless: 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 |
lifeless | SpamapS: ok | 05:39 |
lifeless | SpamapS: to answer your testenv question - sure. | 05:39 |
lifeless | SpamapS: more == better, just be sure to scale the test cloud too | 05:39 |
lifeless | we need num(testenvs) <= jenkins-slaves-in-active-state | 05:40 |
lifeless | also we need to add 2 more Vms to every testenvs shortly for HA | 05:40 |
SpamapS | lifeless: 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 |
SpamapS | anyway | 05:41 |
SpamapS | More to the point.. trusty rollout. | 05:41 |
lifeless | greghaynes: dprince may need active poking on that -1 | 05:41 |
lifeless | greghaynes: if you disagree with it, it is in your power to approve ;P | 05:41 |
lifeless | SpamapS: yes indeedy | 05:41 |
lifeless | SpamapS: https://review.openstack.org/#/c/84076/ is relevant to that | 05:41 |
SpamapS | lifeless: 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 dprince | 05:43 | |
lifeless | greghaynes: I updated https://review.openstack.org/#/c/89732/ and it passed CI | 05:45 |
lifeless | SpamapS: ^ blocker for HA. | 05:45 |
greghaynes | ah, finally | 05:45 |
greghaynes | was waiting for CI all day | 05:45 |
lifeless | SpamapS: so, not sure what to suggest on the rebuild | 05:45 |
lifeless | SpamapS: in principle its 'heat DWIM' but we haven't had that bunfight yet. | 05:46 |
SpamapS | lifeless: for the rebuild, nova has screwed us anyway because we just lost state.. so that's the bug I reported last night | 05:46 |
lifeless | SpamapS: one thing you could do is create a stack of just hypervisors and use that to bring up fresh ones | 05:46 |
lifeless | with parameters for things needed from the control plane | 05:47 |
SpamapS | lifeless: Yeah I was thinking that too... just bring up a new stack with new hypervisors and scale off the old one. | 05:47 |
SpamapS | that was originally why nova-compute-instance was kept deployable standalone | 05:47 |
lifeless | or you could abandon/adopt | 05:48 |
lifeless | with less resources | 05:48 |
SpamapS | thats not a terrible option | 05:48 |
SpamapS | have had success with it | 05:48 |
lifeless | SpamapS: 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 regression | 05:50 |
SpamapS | ok.. time for bed here | 05:52 |
tchaypo | this isn't right. | 05:53 |
tchaypo | Google tells me it's 10:53pm in US-WEST | 05:53 |
tchaypo | why are greghaynes and SpamapS here? | 05:54 |
greghaynes | I operate in GST, not PST | 05:54 |
greghaynes | as StevenK put it :) | 05:54 |
tchaypo | 6:54am in london... same comment applies :p | 05:54 |
lifeless | tchaypo: not GMT, not UTC, GST. | 05:55 |
lifeless | tchaypo: Greg Standard Time | 05:55 |
tchaypo | hahahahahah | 05:55 |
greghaynes | ype | 05:55 |
tchaypo | very well then. Greg has my blessing to be online whenever he damn well wants. | 05:55 |
greghaynes | yep, even | 05:55 |
lifeless | bnemec: https://review.openstack.org/#/c/89732/ <- | 05:59 |
lifeless | erm who else | 05:59 |
lifeless | marios: ^ | 06:00 |
lifeless | righto, time to shove off from here for now | 06:00 |
lifeless | au revoir | 06:00 |
*** morganfainberg is now known as morganfainberg_Z | 06:03 | |
*** ramishra has joined #tripleo | 06:10 | |
*** rwsu is now known as rwsu-away | 06:14 | |
proffalken | lifeless: thanks :) | 06:22 |
*** shakayumi has joined #tripleo | 06:31 | |
openstackgerrit | A change was merged to openstack/os-apply-config: Updated from global requirements https://review.openstack.org/91231 | 06:35 |
*** shakayumi has quit IRC | 06:36 | |
openstackgerrit | Matthew Macdonald-Wallace proposed a change to openstack/tripleo-specs: Spec to accompany systems-monitoring blueprint https://review.openstack.org/91750 | 06:39 |
proffalken | First! :P | 06:39 |
openstackgerrit | Matthew Macdonald-Wallace proposed a change to openstack/tripleo-specs: Spec to accompany systems-monitoring blueprint https://review.openstack.org/91750 | 06:42 |
proffalken | got carried away and put it in the wrong directory... :( | 06:42 |
*** eghobo has quit IRC | 06:49 | |
*** e0ne has joined #tripleo | 06:54 | |
*** openstackgerrit has quit IRC | 06:57 | |
GheRivero | morning all | 07:09 |
*** gcha has joined #tripleo | 07:17 | |
marios | lifeless: sorry, missed the ping, looking | 07:20 |
*** e0ne has quit IRC | 07:21 | |
*** jml has quit IRC | 07:23 | |
*** giulivo has joined #tripleo | 07:26 | |
*** jml has joined #tripleo | 07:30 | |
proffalken | All: 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 work | 07:31 |
*** shakayumi has joined #tripleo | 07:32 | |
proffalken | (see this trhead for more info: http://www.spinics.net/lists/dash/msg00154.html) | 07:37 |
*** shakayumi has quit IRC | 07:37 | |
*** nati_ueno has joined #tripleo | 07:55 | |
*** killer_prince is now known as lazy_prince | 07:56 | |
*** akuznetsov has joined #tripleo | 07:58 | |
*** stillbert is now known as mikal | 08:03 | |
*** derekh_ has joined #tripleo | 08:06 | |
*** dshulyak has joined #tripleo | 08:15 | |
*** scotty95747 has joined #tripleo | 08:16 | |
*** chuckC_ has joined #tripleo | 08:19 | |
*** greghaynes_ has joined #tripleo | 08:20 | |
*** Hazelesque_ has joined #tripleo | 08:20 | |
*** dguerri_ has joined #tripleo | 08:21 | |
*** greghaynes has quit IRC | 08:21 | |
*** ohadlevy has quit IRC | 08:21 | |
*** scott-millward has quit IRC | 08:21 | |
*** NobodyCa1 has joined #tripleo | 08:21 | |
*** e0ne has joined #tripleo | 08:21 | |
*** DuncanT-1 has joined #tripleo | 08:22 | |
*** ohadlevy has joined #tripleo | 08:26 | |
*** e0ne has quit IRC | 08:26 | |
*** ohadlevy is now known as Guest42690 | 08:26 | |
*** beekneemech has joined #tripleo | 08:27 | |
*** nibalize1 has joined #tripleo | 08:29 | |
*** dkehn_ has joined #tripleo | 08:29 | |
*** echohead1 has joined #tripleo | 08:30 | |
*** kiall_ has joined #tripleo | 08:31 | |
*** jml_ has joined #tripleo | 08:33 | |
*** shakayumi has joined #tripleo | 08:33 | |
*** chuckC_ has quit IRC | 08:34 | |
*** dshulyak has quit IRC | 08:34 | |
*** akuznetsov has quit IRC | 08:34 | |
*** jml has quit IRC | 08:34 | |
*** dkehn has quit IRC | 08:34 | |
*** jrist has quit IRC | 08:34 | |
*** Shrews has quit IRC | 08:34 | |
*** kiall has quit IRC | 08:34 | |
*** bnemec has quit IRC | 08:34 | |
*** echohead_ has quit IRC | 08:34 | |
*** d0ugal has quit IRC | 08:34 | |
*** nibalizer has quit IRC | 08:34 | |
*** jml_ is now known as jml | 08:34 | |
*** jml has quit IRC | 08:34 | |
*** jml has joined #tripleo | 08:34 | |
*** chuckC has quit IRC | 08:34 | |
*** dguerri has quit IRC | 08:34 | |
*** NobodyCam has quit IRC | 08:34 | |
*** Hazelesque has quit IRC | 08:34 | |
*** DuncanT- has quit IRC | 08:34 | |
*** shakayumi has quit IRC | 08:37 | |
goneri | Hi, 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 #tripleo | 08:40 | |
*** hashar has joined #tripleo | 08:46 | |
*** dividehex has quit IRC | 08:48 | |
*** martyntaylor has joined #tripleo | 08:50 | |
*** Shrews has joined #tripleo | 08:52 | |
*** tzumainn has joined #tripleo | 08:52 | |
*** chuckC has joined #tripleo | 08:53 | |
*** dshulyak has joined #tripleo | 08:53 | |
*** akuznetsov has joined #tripleo | 08:53 | |
*** d0ugal has joined #tripleo | 08:53 | |
*** rwsu-away has quit IRC | 08:56 | |
*** nati_ueno has quit IRC | 08: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 thinks | 08:58 |
*** lucasagomes has joined #tripleo | 08:59 | |
derekh_ | kiall_: was that your patch ? adding "timeout 10 rabbitmqctl wait /var/run/rabbitmq/pid" | 09:02 |
*** matsuhashi has quit IRC | 09:04 | |
*** matsuhashi has joined #tripleo | 09:05 | |
*** athomas has joined #tripleo | 09:17 | |
derekh_ | tripleo-specs docs jobs fails, needs https://review.openstack.org/#/c/91754/ befor we can merge anything | 09:19 |
*** e0ne has joined #tripleo | 09:22 | |
*** dividehex has joined #tripleo | 09:23 | |
*** darraghb has joined #tripleo | 09:27 | |
*** e0ne has quit IRC | 09:27 | |
*** chuckC has quit IRC | 09:28 | |
*** chuckC has joined #tripleo | 09:29 | |
*** yamahata has quit IRC | 09:30 | |
proffalken | If 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 #tripleo | 09:33 | |
proffalken | I'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 correctly | 09:36 |
*** petertoft has joined #tripleo | 09:37 | |
*** shakayumi has quit IRC | 09:38 | |
shadower | derekh_: 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 #tripleo | 09:39 | |
jang | daily 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 merge | 09:41 |
shadower | derekh_: approved, thanks | 09:41 |
derekh_ | jang: done, thanks | 09:42 |
*** gcha has quit IRC | 09:43 | |
jang | merci Derek. | 09:46 |
*** nati_ueno has joined #tripleo | 09:57 | |
*** nati_ueno has quit IRC | 10:01 | |
*** nati_ueno has joined #tripleo | 10:02 | |
*** nati_uen_ has joined #tripleo | 10:04 | |
*** athomas_ has joined #tripleo | 10:07 | |
*** nati_ueno has quit IRC | 10:07 | |
*** nati_uen_ has quit IRC | 10:09 | |
*** jcoufal has quit IRC | 10:13 | |
goneri | proffalken: I think you should extend bin/dib-lint first | 10:18 |
goneri | proffalken: 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,z | 10:19 |
goneri | https://review.openstack.org/#/c/88534/ and https://review.openstack.org/#/c/88531/ too | 10:19 |
*** e0ne has joined #tripleo | 10:21 | |
*** hashar has quit IRC | 10:22 | |
proffalken | goneri: thanks, I'll take a look now | 10:22 |
*** e0ne has quit IRC | 10:22 | |
*** dguerri_ is now known as dguerri | 10:23 | |
*** e0ne has joined #tripleo | 10:23 | |
*** jcoufal has joined #tripleo | 10:23 | |
*** giulivo has quit IRC | 10:24 | |
*** hashar has joined #tripleo | 10:24 | |
*** e0ne has quit IRC | 10:25 | |
*** giulivo has joined #tripleo | 10:27 | |
*** ramishra has quit IRC | 10:28 | |
*** akuznetsov has quit IRC | 10:29 | |
*** akuznetsov has joined #tripleo | 10:30 | |
*** markmc has joined #tripleo | 10:31 | |
*** lazy_prince is now known as killer_prince | 10:33 | |
*** shakayumi has joined #tripleo | 10:34 | |
*** matsuhashi has quit IRC | 10:35 | |
*** matsuhashi has joined #tripleo | 10:37 | |
*** shakayumi has quit IRC | 10:39 | |
*** e0ne has joined #tripleo | 10:49 | |
*** jcoufal has quit IRC | 10:53 | |
*** nati_ueno has joined #tripleo | 10:53 | |
*** nati_ueno has quit IRC | 10:58 | |
*** hewbrocca has joined #tripleo | 11:02 | |
*** hashar has quit IRC | 11:02 | |
*** DuncanT-1 is now known as DuncanT | 11:17 | |
*** matsuhashi has quit IRC | 11:22 | |
derekh_ | ccorrigan: I'm guessing that was an accident https://review.openstack.org/#/c/91768/ | 11:32 |
*** shakayumi has joined #tripleo | 11:35 | |
*** nosnos has quit IRC | 11:37 | |
*** shakayumi has quit IRC | 11:39 | |
*** killer_prince has quit IRC | 11:42 | |
*** Hazelesque_ is now known as Hazelesque | 11:44 | |
*** matsuhashi has joined #tripleo | 11:48 | |
*** hashar has joined #tripleo | 11:48 | |
*** gcha has joined #tripleo | 11:51 | |
*** nosnos has joined #tripleo | 11:53 | |
*** nati_ueno has joined #tripleo | 11:54 | |
*** lucasagomes is now known as lucas-hungry | 11:59 | |
*** nati_ueno has quit IRC | 11:59 | |
*** hewbrocca has quit IRC | 12:11 | |
*** mrunge has joined #tripleo | 12:21 | |
*** weshay has joined #tripleo | 12:29 | |
goneri | derekh_: 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 IRC | 12:34 | |
*** athomas has quit IRC | 12:34 | |
*** shakamunyi has joined #tripleo | 12:36 | |
*** nosnos has quit IRC | 12:39 | |
ccorrigan | derekh_: Schoolboy error on my part, second patch abandoned, original updated... | 12:40 |
*** shakamunyi has quit IRC | 12:41 | |
*** jdob has joined #tripleo | 12:42 | |
*** nati_ueno has joined #tripleo | 12:44 | |
*** rlandy has joined #tripleo | 12:45 | |
*** nati_ueno has quit IRC | 12:48 | |
*** dprince has joined #tripleo | 12:50 | |
*** jdob has quit IRC | 12:59 | |
*** matty_dubs|gone is now known as matty_dubs | 13:00 | |
*** athomas has joined #tripleo | 13:01 | |
*** ramishra has joined #tripleo | 13:01 | |
*** jdob has joined #tripleo | 13:02 | |
*** andrearosa has quit IRC | 13:03 | |
*** matsuhashi has quit IRC | 13:04 | |
*** matsuhashi has joined #tripleo | 13:04 | |
*** morazi has joined #tripleo | 13:04 | |
*** lucas-hungry is now known as lucasagomes | 13:05 | |
*** matsuhas_ has joined #tripleo | 13:08 | |
*** matsuhashi has quit IRC | 13:08 | |
derekh_ | goneri: looking | 13:17 |
derekh_ | ccorrigan: thanks | 13:17 |
*** matsuhas_ has quit IRC | 13:18 | |
*** hewbrocca has joined #tripleo | 13:22 | |
*** Rakesh6 has quit IRC | 13:24 | |
*** jdob has quit IRC | 13:24 | |
*** jdob has joined #tripleo | 13:26 | |
*** killer_prince has joined #tripleo | 13:28 | |
*** funzo has joined #tripleo | 13:31 | |
*** funzo is now known as Guest94272 | 13:31 | |
derekh_ | goneri: replied, I'm still on the side of changing to " " to avoid confusion | 13:32 |
goneri | derekh_: ok, perfect. give me 10 minutes to refresh the patch :) | 13:33 |
derekh_ | goneri: ok, thanks | 13: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 wrong | 13:36 |
*** shakamunyi has joined #tripleo | 13:37 | |
derekh_ | kiall_: yup, one sec | 13:37 |
dprince | kiall_: not sure. my take is reverting this should be safe... and once we research it further we can re-add your patch as required | 13:37 |
derekh_ | kiall_: so F20 jobs have been failing all day http://goodsquishy.com/downloads/tripleo-jobs.html | 13:38 |
derekh_ | kiall_: taking one example | 13:38 |
*** Guest94272 is now known as funzo | 13:38 | |
kiall_ | I'll break anyone who's running on a slow system ;) Without the wait, cluster_status returns different info during startup | 13:38 |
*** kiall_ is now known as Kiall | 13:38 | |
derekh_ | Kiall: http://logs.openstack.org/20/91520/3/check-tripleo/check-tripleo-overcloud-f20/3ec23e3/seed_logs.tar.xz | 13:39 |
derekh_ | Kiall: search var/log/messages for "May 2 03:06:05" | 13:40 |
derekh_ | :q | 13:40 |
*** shakamunyi has quit IRC | 13:41 | |
Kiall | I see it.. Where does F20 has the default pid file for RMQ? | 13:41 |
Kiall | (Without OOO changing it..) | 13:41 |
Kiall | I saw something similar on ubuntu/debian, where RMQ started at boot, and `service rabbitmq-server restart`, failed to actually restart it | 13:42 |
Kiall | the bash rabbitmq-server script that wraps the erlang process got restarted, but the actual erlang process did not | 13: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 took | 13:44 |
Kiall | Looks like /run/rabbitmq/pid .. is /run a symlink/bind to /var/run on F20? (or the reverse...) | 13:44 |
*** nati_ueno has joined #tripleo | 13:45 | |
*** mugsie has joined #tripleo | 13: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 CI | 13:47 |
Kiall | derekh_: maybe the CI is just .. slow? | 13:47 |
Kiall | yea | 13:47 |
derekh_ | Kiall: possibly, see see once the job runs | 13:47 |
derekh_ | *well see | 13:48 |
derekh_ | *we'll see :-) | 13:48 |
derekh_ | phew | 13:48 |
Kiall | Either 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 |
Kiall | RMQ shows very different info during started for `rabbitmqctl cluster_status`, which causes other issues in the script | 13:49 |
Kiall | during startup* | 13:49 |
goneri | derekh_: https://review.openstack.org/#/c/88534/3/bin/dib-lint | 13:49 |
*** nati_ueno has quit IRC | 13:50 | |
*** rcarrill` has joined #tripleo | 13:52 | |
*** shakamunyi has joined #tripleo | 13:52 | |
derekh_ | Kiall: ok, lets see what happens when it runs and if there are other problems it should become clear | 13:52 |
derekh_ | goneri: looking | 13:52 |
*** rcarrillocruz has quit IRC | 13:53 | |
*** openstackgerrit has joined #tripleo | 14:09 | |
*** andrearosa has joined #tripleo | 14:10 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Expose passthrough configuration in undercloud. https://review.openstack.org/91792 | 14:10 |
*** chuckC has quit IRC | 14:14 | |
*** NobodyCa1 is now known as NobodyCam | 14:21 | |
*** killer_prince is now known as lazy_prince | 14:23 | |
openstackgerrit | Matthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Install the "classic" icinga interface https://review.openstack.org/87223 | 14:23 |
openstackgerrit | Matthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Install check_mk server https://review.openstack.org/87226 | 14:24 |
*** lazy_prince is now known as killer_prince | 14:25 | |
openstackgerrit | Matthew Macdonald-Wallace proposed a change to openstack/tripleo-image-elements: Auto-collect local checks https://review.openstack.org/91324 | 14:25 |
proffalken | lifeless: there's a some of updates/refactoring to those patches, hope they meet your high standards! :) | 14:26 |
*** shakamunyi has quit IRC | 14:31 | |
*** shakamunyi has joined #tripleo | 14:31 | |
*** shakamunyi has quit IRC | 14:32 | |
ccrouch | giulivo: welcome to the tripleo effort! | 14:32 |
openstackgerrit | Alexis Lee proposed a change to openstack/tripleo-specs: Factor data out of _check_titles https://review.openstack.org/91801 | 14:32 |
openstackgerrit | Alexis Lee proposed a change to openstack/tripleo-specs: Allow additional titles + subtitles https://review.openstack.org/91802 | 14:32 |
*** shakamunyi has joined #tripleo | 14:32 | |
giulivo | ccrouch , hi all :) | 14:32 |
giulivo | was lurking into the chan from a while now | 14:32 |
ccrouch | but now its official :-) | 14:33 |
*** slagle has quit IRC | 14:33 | |
dprince | giulivo: welcome | 14:34 |
*** slagle has joined #tripleo | 14:37 | |
derekh_ | giulivo: hi and welcome | 14:40 |
openstackgerrit | Cian O'Driscoll proposed a change to openstack-infra/tripleo-ci: Test the upgrade codepath works as well. https://review.openstack.org/87758 | 14:48 |
openstackgerrit | Stuart McLaren proposed a change to openstack/tripleo-heat-templates: ssl: Add support for a CA Certificate https://review.openstack.org/85100 | 14:48 |
*** chuckC has joined #tripleo | 14:53 | |
*** shakamunyi has quit IRC | 14:53 | |
*** shakamunyi has joined #tripleo | 14:54 | |
*** shakamunyi has quit IRC | 14:56 | |
*** shakamunyi has joined #tripleo | 14:56 | |
*** shakayumi has joined #tripleo | 14:59 | |
*** shakamunyi has quit IRC | 15:01 | |
*** nati_ueno has joined #tripleo | 15:02 | |
rbrady-afk | giulivo! Welcome! | 15:02 |
*** rbrady-afk is now known as rbrady | 15:02 | |
giulivo | rbrady, :D | 15:02 |
*** shakayumi has quit IRC | 15:03 | |
*** shakamunyi has joined #tripleo | 15:03 | |
*** shakamunyi has quit IRC | 15:03 | |
*** ramishra_ has joined #tripleo | 15:05 | |
openstackgerrit | therese-mchale proposed a change to openstack/tripleo-image-elements: Use Percona Packages for Ubuntu https://review.openstack.org/90134 | 15:05 |
*** ramishra has quit IRC | 15:06 | |
*** corrigac has joined #tripleo | 15:10 | |
*** andrearosa has quit IRC | 15:17 | |
*** rcarrill` has quit IRC | 15:18 | |
*** shakamunyi has joined #tripleo | 15:18 | |
*** rameshg87 has joined #tripleo | 15:22 | |
*** rcarrillocruz has joined #tripleo | 15:23 | |
*** dkehn_ is now known as dkehn | 15:25 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files https://review.openstack.org/88531 | 15:25 |
openstackgerrit | Dmitry Shulyak proposed a change to openstack/tripleo-heat-templates: Introduce configurable virtual ip in templates https://review.openstack.org/89556 | 15:29 |
*** marun has quit IRC | 15:30 | |
*** andrearosa has joined #tripleo | 15:31 | |
goneri | derekh_: it should be fine now: https://review.openstack.org/#/c/88531/4/bin/dib-lint | 15:31 |
*** marun has joined #tripleo | 15:31 | |
*** nati_ueno has quit IRC | 15:35 | |
openstackgerrit | A change was merged to openstack/tripleo-incubator: Run pull-tools before setup-clienttools https://review.openstack.org/91432 | 15:35 |
*** shakayumi has joined #tripleo | 15:37 | |
openstackgerrit | A change was merged to openstack/tripleo-image-elements: Openvswitch 2.1 removes openvswitch-controller https://review.openstack.org/86348 | 15:37 |
openstackgerrit | A change was merged to openstack/diskimage-builder: debian: properly deal with Debian stable/unstable https://review.openstack.org/91430 | 15:37 |
*** shakamunyi has quit IRC | 15:38 | |
openstackgerrit | jan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted https://review.openstack.org/91809 | 15:41 |
derekh_ | goneri: small nit inline, then I'm happy, sorry | 15: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 fix | 15:46 |
*** yamahata has joined #tripleo | 15:51 | |
*** matty_dubs is now known as matty_dubs|lunch | 15:53 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-heat-templates: Add passthrough configuration glue. https://review.openstack.org/91810 | 15:56 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files https://review.openstack.org/88531 | 15:57 |
goneri | derekh_: done, I dropped the two \n | 15:57 |
derekh_ | goneri: forget my last comment about why pep8 is not passing, I was looking at an old version of a file with tabs | 15:58 |
goneri | derekh_: this \n confusion is due to the current dib-lint. It doesn't finish with a \n and git complains about that. | 15:59 |
goneri | Yet another check for dib-lint? :) | 15:59 |
*** eghobo has joined #tripleo | 15:59 | |
derekh_ | goneri: sounds like it | 16:00 |
*** killer_prince is now known as lazy_prince | 16:05 | |
*** lazy_prince is now known as killer_prince | 16:06 | |
*** martyntaylor has quit IRC | 16:07 | |
*** sdake_ has quit IRC | 16:08 | |
*** killer_prince is now known as lazy_prince | 16:13 | |
*** lazy_prince is now known as killer_prince | 16:14 | |
openstackgerrit | A change was merged to openstack/tripleo-image-elements: Revert "RabbitMQ - Increase restart timeouts" https://review.openstack.org/91758 | 16:15 |
openstackgerrit | jan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted https://review.openstack.org/91809 | 16:18 |
*** corrigac has quit IRC | 16:24 | |
*** killer_prince is now known as lazy_prince | 16:24 | |
derekh_ | jang: was that ^ the reason the timeout didn't work on fedora? | 16:26 |
*** ramishra_ has quit IRC | 16: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 flashgordon | 16:29 | |
*** yamahata has quit IRC | 16:31 | |
*** yamahata has joined #tripleo | 16:31 | |
*** akuznetsov has quit IRC | 16:31 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files https://review.openstack.org/88531 | 16:32 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation https://review.openstack.org/88534 | 16:32 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line https://review.openstack.org/91818 | 16:32 |
*** akuznetsov has joined #tripleo | 16:32 | |
*** corrigac has joined #tripleo | 16:34 | |
*** e0ne has quit IRC | 16:34 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line https://review.openstack.org/91818 | 16:34 |
*** hashar has quit IRC | 16:38 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: add some missing \n at end of file https://review.openstack.org/91823 | 16:39 |
*** derekh_ has quit IRC | 16:43 | |
openstackgerrit | jan grant proposed a change to openstack/tripleo-image-elements: Ensure rabbitmq can be successfully restarted https://review.openstack.org/91809 | 16:45 |
*** hashar has joined #tripleo | 16:47 | |
jang | derekh_: yes, I think it probably was | 16:49 |
jang | fedora is a systemd os, isn't it? | 16:49 |
jang | so, 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 #tripleo | 16:51 | |
jang | but, 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 start | 16:52 |
*** newell has joined #tripleo | 16:53 | |
*** matty_dubs|lunch is now known as matty_dubs | 16:54 | |
jang | if 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 IRC | 16:55 | |
openstackgerrit | A change was merged to openstack/tripleo-incubator: Install and configure NTP in undercloud/overcloud https://review.openstack.org/88647 | 16:57 |
*** giulivo has joined #tripleo | 16:58 | |
*** ramishra has joined #tripleo | 17:01 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: check for tab indent in files https://review.openstack.org/88531 | 17:05 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation https://review.openstack.org/88534 | 17:05 |
*** e0ne has joined #tripleo | 17:05 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: 4 spaces indent https://review.openstack.org/91829 | 17:05 |
*** ramishra has quit IRC | 17:06 | |
*** e0ne has quit IRC | 17:09 | |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: Eliminate 'tr' in favor of inline bash. https://review.openstack.org/91832 | 17:11 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: 4 spaces indent https://review.openstack.org/91829 | 17:13 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation https://review.openstack.org/88534 | 17:13 |
*** hashar has quit IRC | 17:14 | |
*** petertoft has quit IRC | 17:15 | |
*** gcha has quit IRC | 17:16 | |
*** e0ne has joined #tripleo | 17:17 | |
*** e0ne has quit IRC | 17:19 | |
*** rameshg87 has quit IRC | 17:20 | |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: dib-lint: ensure file finish with a new line https://review.openstack.org/91818 | 17:23 |
*** athomas has quit IRC | 17:24 | |
openstackgerrit | Ben Nemec proposed a change to openstack/diskimage-builder: Make sure all scripts are set -e https://review.openstack.org/81637 | 17:24 |
openstackgerrit | Ben Nemec proposed a change to openstack/diskimage-builder: Check for set -o pipefail https://review.openstack.org/83929 | 17:24 |
openstackgerrit | Ben Nemec proposed a change to openstack/diskimage-builder: set -u and -o pipefail everywhere https://review.openstack.org/84868 | 17:24 |
openstackgerrit | Ben Nemec proposed a change to openstack/diskimage-builder: Ensure scripts are set -u https://review.openstack.org/83928 | 17:24 |
openstackgerrit | Ben Nemec proposed a change to openstack/diskimage-builder: Allow excluding tests from certain files https://review.openstack.org/90425 | 17:24 |
openstackgerrit | Gonéri Le Bouder proposed a change to openstack/diskimage-builder: indent: search for !=4 spaces indentation https://review.openstack.org/88534 | 17:25 |
goneri | sorry for the flood :/ | 17:26 |
jang | if you manage to get openstackgerrit kickbanned for flooding, _then_ you need to apologise | 17:26 |
goneri | hopefuly I won't have to aplogise. My rebases are finished :) | 17:29 |
*** corrigac has quit IRC | 17:32 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-heat-templates: Replace nagios3 template with HOT rewrite https://review.openstack.org/90665 | 17:33 |
*** morganfainberg_Z is now known as morganfainberg | 17:37 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Separate Heat BM and VM configs for Nova-BM. https://review.openstack.org/90033 | 17:41 |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file https://review.openstack.org/91717 | 17:41 |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Allows user-specified PXE timeout for overcloud https://review.openstack.org/86605 | 17:42 |
* beekneemech sets a goal in life of getting the bot kicked | 17:42 | |
*** eghobo has quit IRC | 17:43 | |
*** markmc has quit IRC | 17:46 | |
*** TravT has joined #tripleo | 17:48 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file https://review.openstack.org/91717 | 17:49 |
*** shakayumi has quit IRC | 17:52 | |
*** e0ne has joined #tripleo | 17:52 | |
beekneemech | /home/bnemec/tripleo/tripleo-incubator/scripts/devtest_undercloud.sh: line 59: USE_UNDERCLOUD_UI: unbound variable | 17:52 |
beekneemech | Bleh, apparently devtest_variables was not run. | 17:52 |
openstackgerrit | A change was merged to openstack-infra/tripleo-ci: Move the testenv json file to /etc https://review.openstack.org/88473 | 17:53 |
*** rcarrill` has joined #tripleo | 17:53 | |
chuckC | Hi 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 #tripleo | 17:53 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Move most undercloud params to an environment file https://review.openstack.org/91717 | 17:54 |
*** rcarrillocruz has quit IRC | 17:54 | |
beekneemech | I wonder if we should re-exec devtest after updating tripleo-incubator. | 17:54 |
beekneemech | I 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 that | 17:56 |
*** e0ne has quit IRC | 17:56 | |
greghaynes_ | woo for env vars as parameters :/ | 17:57 |
greghaynes_ | chuckC: just make sure youre at least on saucy if you ubuntu | 17:57 |
SpamapS | yay, we're finally seeing jobs go through zuul | 17:57 |
greghaynes_ | er, saucy or newer | 17:57 |
SpamapS | greghaynes_: death to the environment variables!! ;) | 17:58 |
chuckC | greghaynes: thanks, I was about to go precise | 18:03 |
*** greghaynes_ is now known as greghaynes | 18:04 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Separate Heat BM and VM configs for Nova-BM. https://review.openstack.org/90033 | 18:04 |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Allows user-specified PXE timeout for overcloud https://review.openstack.org/86605 | 18:04 |
*** cody-somerville has quit IRC | 18:06 | |
openstackgerrit | Nicholas Randon proposed a change to openstack/tripleo-incubator: Expose passthrough configuration in undercloud. https://review.openstack.org/91792 | 18:08 |
*** dshurtle has joined #tripleo | 18:12 | |
*** corrigac has joined #tripleo | 18:16 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ hostname in nova.conf https://review.openstack.org/91846 | 18:17 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in neutron.conf https://review.openstack.org/91847 | 18:18 |
*** cody-somerville has joined #tripleo | 18:18 | |
*** cody-somerville has joined #tripleo | 18:18 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in nova.conf https://review.openstack.org/91846 | 18:19 |
*** corrigac_ has joined #tripleo | 18:19 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ userid in nova.conf https://review.openstack.org/91846 | 18:20 |
*** darraghb has quit IRC | 18:21 | |
*** corrigac has quit IRC | 18:21 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ username in neutron.conf https://review.openstack.org/91847 | 18:23 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ userid in nova.conf https://review.openstack.org/91846 | 18:24 |
* SpamapS spam spam spam spam | 18:24 | |
greghaynes | tchaypo: You looked at concoord for picking leaders right? Curious why that didnt pan out | 18:24 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in glance-api.conf https://review.openstack.org/91852 | 18:26 |
SpamapS | greghaynes: leader election is really hard without built in support from the state store itself. What thing are we needing a leader for? | 18:26 |
greghaynes | for the stuff currently using the os-is-bootstrap-host hack | 18:27 |
*** akuznets_ has joined #tripleo | 18:27 | |
greghaynes | I suspect we were probably 'using it wrong'(TM) | 18:28 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in heat.conf https://review.openstack.org/91853 | 18:28 |
SpamapS | greghaynes: ah yes that | 18:28 |
*** akuznetsov has quit IRC | 18:28 | |
greghaynes | since 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 operation | 18:29 |
greghaynes | or we could just use something different | 18:29 |
SpamapS | It's as important that you get "not a leader" choices too. :) | 18:30 |
SpamapS | greghaynes: is concoord unicast based? | 18:30 |
greghaynes | dont think so, but no idea | 18:31 |
greghaynes | not sure why that would matter? | 18:31 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in cinder.conf https://review.openstack.org/91855 | 18:32 |
SpamapS | greghaynes: harder to test broadcast systems | 18:32 |
greghaynes | ah | 18:32 |
SpamapS | greghaynes: and some environments constrain such things | 18:32 |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ironic.conf https://review.openstack.org/91857 | 18:33 |
*** d0ugal has quit IRC | 18:34 | |
*** d0ugal has joined #tripleo | 18:34 | |
*** d0ugal has quit IRC | 18:34 | |
*** d0ugal has joined #tripleo | 18:34 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ironic.conf https://review.openstack.org/91857 | 18:34 |
SpamapS | lovely spaaaaaaaaammm | 18:34 |
*** corrigac_ has quit IRC | 18:34 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user in ceilometer.conf https://review.openstack.org/91859 | 18:35 |
dprince | SpamapS: Should we squash all these rabbit_userid changes? | 18:36 |
SpamapS | dprince: why? | 18:37 |
dprince | SpamapS: all seem related to me, revertability is nice but in this case I think they are close enough | 18:37 |
SpamapS | We're not going to revert these. | 18:38 |
dprince | SpamapS: 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 |
dprince | SpamapS: all the more reason to squash them into a single commit... | 18:38 |
SpamapS | I get that. But I prefer to have them all separate so there's no rebase dance. | 18:38 |
dprince | SpamapS: right, but you'll have the get all the CI jobs passing dance :( | 18:38 |
SpamapS | If we can't get 5 passes...... | 18:39 |
dprince | SpamapS: do as you like. We have ETOOMANY reviews right now so I say squash where appropriate | 18:39 |
SpamapS | These are as tiny and simple as possible. | 18:40 |
dprince | SpamapS: I'm suggesting you combine them (squash) | 18:40 |
SpamapS | I am fine with a squash... really. It's just not what I do by default. | 18:40 |
dprince | SpamapS: 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 lines | 18:42 |
SpamapS | I know | 18:42 |
SpamapS | Fine | 18:42 |
SpamapS | more work | 18:42 |
SpamapS | gah | 18:42 |
* dprince isn't trying to pick on SpamapS, just set a good precedent | 18:43 | |
SpamapS | It's my first ever "too small" patch argument | 18:43 |
SpamapS | and I have about 2 minutes | 18:44 |
SpamapS | so forgive me for being annoyed | 18:44 |
SpamapS | nothing to do with logic or reason | 18:44 |
*** jang1 has joined #tripleo | 18:47 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed a change to openstack/tripleo-image-elements: Use configured RabbitMQ user https://review.openstack.org/91861 | 18:48 |
SpamapS | dprince: done | 18:50 |
SpamapS | and all others abandoned | 18:50 |
dprince | SpamapS: :), have a good weekend | 18:50 |
SpamapS | Pool is clean, and 78F, and I've a case of XX ... I expect to float my way to happiness. | 18:51 |
* greghaynes jealous | 18:52 | |
SpamapS | greghaynes: yes, enjoy your clouds. | 18:53 |
SpamapS | and more clouds | 18:53 |
SpamapS | and 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 weekend | 18:53 |
* beekneemech has no clouds for the first time in a week | 18:53 | |
openstackgerrit | Adam Gandelman proposed a change to openstack/diskimage-builder: Reinstate Trusty as default for Ubuntu https://review.openstack.org/91864 | 18:54 |
adam_g | SpamapS, ^ | 18:54 |
greghaynes | adam_g: \O/ | 18:54 |
*** mrunge has quit IRC | 18:54 | |
adam_g | check-tempest-dsvm-virtual-ironic-nv should hopefully pass with that change | 18:55 |
victor_lowther | hmmm... | 18:55 |
*** e0ne has joined #tripleo | 18:55 | |
victor_lowther | did they have to change the SSH host keys in Gerrit? | 18:55 |
adam_g | victor_lowther, yeah | 18:56 |
victor_lowther | git-review is whining. | 18:56 |
*** e0ne has quit IRC | 18:56 | |
adam_g | victor_lowther, http://lists.openstack.org/pipermail/openstack-dev/2014-April/033908.html | 18:56 |
beekneemech | Yay heartbleed | 18:56 |
victor_lowther | adam_g: OK, cool. As long as it was not enemy action. :) | 18:56 |
SpamapS | adam_g: \o/ | 18:58 |
openstackgerrit | Victor Lowther proposed a change to openstack/diskimage-builder: Make diskimage-builder work in Docker. https://review.openstack.org/90560 | 18:59 |
SpamapS | adam_g: fix that commit msg ;) | 18:59 |
openstackgerrit | Adam Gandelman proposed a change to openstack/diskimage-builder: Reinstate Trusty as default for Ubuntu https://review.openstack.org/91864 | 19:07 |
openstackgerrit | Victor Lowther proposed a change to openstack/tripleo-incubator: Let devtest.sh run in a Docker container. https://review.openstack.org/91867 | 19:08 |
victor_lowther | hm | 19:25 |
victor_lowther | Is 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 #tripleo | 19:25 | |
greghaynes | victor_lowther: We usually put change-id in the commit msg and possibly WIP until the depended on change lands | 19:26 |
greghaynes | basically its all manual | 19:26 |
*** tzumainn has quit IRC | 19:26 | |
victor_lowther | ah, OK, because https://review.openstack.org/91867 relies on https://review.openstack.org/90560 | 19:28 |
*** julim has quit IRC | 19:28 | |
*** dshulyak has quit IRC | 19:32 | |
*** hashar has joined #tripleo | 19:32 | |
*** mrunge has joined #tripleo | 19:43 | |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: dpkg: support pkg-map in bin/install-packages https://review.openstack.org/91601 | 19:45 |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: Yum: support pkg-map in bin/install-packages https://review.openstack.org/91600 | 19:45 |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: opensuse: support pkg-map in bin/install-packages https://review.openstack.org/91602 | 19:45 |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: Add pkg-map element. https://review.openstack.org/91598 | 19:45 |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: Set DISTRO_NAME in OS environment.d https://review.openstack.org/91599 | 19:45 |
openstackgerrit | Dan Prince proposed a change to openstack/diskimage-builder: Update base element to make use of pkg-map https://review.openstack.org/91880 | 19:45 |
*** dprince has quit IRC | 19:48 | |
*** rcarrillocruz has joined #tripleo | 19:50 | |
*** rcarrill` has quit IRC | 19:52 | |
openstackgerrit | A change was merged to openstack/tripleo-image-elements: MySQL: Ensure libssl-symlink corrects exits on failure https://review.openstack.org/90536 | 20:13 |
*** e0ne has joined #tripleo | 20:20 | |
*** jdob has quit IRC | 20:28 | |
*** lucasagomes has quit IRC | 20:30 | |
*** mrunge has quit IRC | 20:32 | |
*** fandi has joined #tripleo | 20:35 | |
tchaypo | greghaynes: 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 members | 20:41 |
tchaypo | and broadcasting to live members nicely | 20:42 |
tchaypo | For 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 machine | 20:43 |
*** marun has quit IRC | 20:45 | |
tchaypo | Bigger 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 4 | 20:45 |
greghaynes | I guess my thinking was we could use the underlying log for picking who the leader is for a given operation | 20:46 |
greghaynes | not sure if it exposes it, but id be surprised if we cant just make something ontop that uses it | 20:46 |
tchaypo | Afaict we need to either patch it to make the quorum not drop as members leave, or re-implement the liveness/quorum checking on top | 20:47 |
greghaynes | ah, yes thats annoying | 20:48 |
tchaypo | Yep, 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 proposal | 20:48 |
tchaypo | If 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 quickly | 20:49 |
tchaypo | If the existing leader re-proposes after n seconds and everyone else proposes after 2n we should keep an existing leader unless they die | 20:50 |
tchaypo | It's not unfixable, but we didn't have a use for it ready just then so I left it alone | 20:51 |
greghaynes | well, I dont think you even need to wory about that leader logic - paxos should take care of that | 20:53 |
greghaynes | all we have to care about is what proposal actually gets commited to the log | 20:53 |
*** tzumainn has joined #tripleo | 21:04 | |
*** e0ne has quit IRC | 21:08 | |
openstackgerrit | Jordan OMara proposed a change to openstack/tuskar-ui: Adding missing setup for instack https://review.openstack.org/91078 | 21:09 |
*** matty_dubs is now known as matty_dubs|gone | 21:11 | |
*** weshay has quit IRC | 21:19 | |
*** weshay has joined #tripleo | 21:20 | |
*** weshay has quit IRC | 21:24 | |
*** sdake_ has joined #tripleo | 21:24 | |
*** sdake_ has quit IRC | 21:24 | |
*** sdake_ has joined #tripleo | 21:24 | |
*** weshay has joined #tripleo | 21:27 | |
*** tzumainn has quit IRC | 21:31 | |
*** martyntaylor has joined #tripleo | 21:32 | |
*** d0ugal has quit IRC | 21:33 | |
*** weshay has quit IRC | 21:45 | |
*** hashar has quit IRC | 21:49 | |
*** newell has quit IRC | 21:50 | |
*** ccrouch has quit IRC | 21:54 | |
*** ccrouch has joined #tripleo | 21:55 | |
*** newell has joined #tripleo | 21:56 | |
-openstackstatus- NOTICE: Zuul is being restarted with some dependency upgrades and configuration changes; ETA 2215 | 22:00 | |
*** marun has joined #tripleo | 22:01 | |
*** jang1 has quit IRC | 22:04 | |
*** sdake_ has quit IRC | 22:20 | |
*** rlandy has quit IRC | 22:32 | |
openstackgerrit | Adam Gandelman proposed a change to openstack/tripleo-incubator: Do not exit 1 from devtest_variables.sh https://review.openstack.org/91916 | 22:40 |
*** chuckC has quit IRC | 22:45 | |
*** rupsky has joined #tripleo | 23:12 | |
*** rupsky has left #tripleo | 23:12 | |
*** smulcahy has quit IRC | 23:21 | |
-openstackstatus- NOTICE: paste.openstack.org is going down for a short database upgrade | 23:27 | |
*** martyntaylor has quit IRC | 23:33 | |
*** flashgordon is now known as jogo | 23:49 | |
openstackgerrit | Dan Prince proposed a change to openstack/tripleo-heat-templates: Configure NTP for overcloud compute https://review.openstack.org/88691 | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!