Wednesday, 2014-08-27

*** Shrews_ is now known as Shrews00:05
jrollJayF, NobodyCam, I +A'd https://review.openstack.org/#/c/116742/00:06
NobodyCam:)00:12
*** penick has quit IRC00:13
*** fishcried has quit IRC00:24
*** chuckC has joined #openstack-ironic00:27
*** todd_dsm has joined #openstack-ironic01:00
*** eghobo has quit IRC01:25
*** eghobo has joined #openstack-ironic01:26
*** shakamunyi has quit IRC01:29
*** mitz_ has joined #openstack-ironic01:31
*** mitz has quit IRC01:33
*** Poornima has joined #openstack-ironic01:35
*** shakamunyi has joined #openstack-ironic01:36
*** foexle has joined #openstack-ironic01:42
*** foexle_ has quit IRC01:46
*** todd_dsm has quit IRC01:52
*** eguz has joined #openstack-ironic01:54
*** todd_dsm has joined #openstack-ironic01:55
*** nosnos has joined #openstack-ironic01:56
*** pcrews has quit IRC01:57
*** eghobo has quit IRC01:58
*** foexle has quit IRC02:03
*** todd_dsm has quit IRC02:15
*** todd_dsm has joined #openstack-ironic02:19
*** todd_dsm has quit IRC02:20
*** harlowja is now known as harlowja_away02:44
*** lazy_prince is now known as killer_prince02:57
*** nosnos has quit IRC03:23
*** nosnos has joined #openstack-ironic03:24
*** nosnos has quit IRC03:28
*** harlowja_away has quit IRC03:29
*** killer_prince has quit IRC03:50
*** killer_prince has joined #openstack-ironic03:51
*** killer_prince is now known as lazy_prince03:51
*** nosnos has joined #openstack-ironic03:53
*** eghobo has joined #openstack-ironic04:04
*** eghobo has quit IRC04:12
*** ramineni has quit IRC04:17
*** todd_dsm has joined #openstack-ironic04:26
*** rameshg87 has joined #openstack-ironic04:29
*** eghobo has joined #openstack-ironic04:42
*** chuck_ has quit IRC04:49
*** chenglch has joined #openstack-ironic05:08
*** yuanying has joined #openstack-ironic05:13
*** tzumainn has quit IRC05:25
*** tzumainn has joined #openstack-ironic05:25
*** rameshg87 has quit IRC05:36
*** rameshg87 has joined #openstack-ironic05:37
*** rameshg87 has quit IRC05:37
*** rameshg87 has joined #openstack-ironic05:38
*** todd_dsm has quit IRC05:51
*** ramineni has joined #openstack-ironic05:56
*** k4n0 has joined #openstack-ironic05:59
*** rameshg87 has quit IRC06:19
*** rameshg87 has joined #openstack-ironic06:20
*** rakesh_hs has joined #openstack-ironic06:29
*** jcoufal has joined #openstack-ironic06:38
*** eghobo has quit IRC06:44
*** foexle has joined #openstack-ironic07:03
*** nosnos has quit IRC07:04
*** nosnos_ has joined #openstack-ironic07:04
*** rameshg87 has quit IRC07:07
*** rameshg87 has joined #openstack-ironic07:08
mrdaNight ironic!07:17
Haomengmrda: night:)07:25
*** vdrok has joined #openstack-ironic07:39
*** yuanying_ has joined #openstack-ironic07:43
*** yuanying_ has quit IRC07:46
*** yuanying_ has joined #openstack-ironic07:46
*** yuanying has quit IRC07:47
*** chenglch has quit IRC07:52
*** chenglch has joined #openstack-ironic07:52
*** romcheg2 has joined #openstack-ironic07:56
*** romcheg1 has quit IRC07:59
*** jistr has joined #openstack-ironic08:00
*** shakayumi has joined #openstack-ironic08:01
*** dguerri_ has joined #openstack-ironic08:01
*** zer0c00l1 has joined #openstack-ironic08:02
*** zer0c00l has quit IRC08:02
*** BadCub has joined #openstack-ironic08:03
*** gilliard_ has joined #openstack-ironic08:05
*** Hefeweiz1n has joined #openstack-ironic08:05
*** dhellmann_ has joined #openstack-ironic08:08
*** NobodyCa1 has joined #openstack-ironic08:09
*** yuanying_ has quit IRC08:09
*** yuanying has joined #openstack-ironic08:09
*** shakamunyi has quit IRC08:10
*** NobodyCam has quit IRC08:10
*** BadCub_ has quit IRC08:10
*** gilliard has quit IRC08:10
*** tzumainn has quit IRC08:10
*** dguerri has quit IRC08:10
*** Hefeweizen has quit IRC08:10
*** dhellmann has quit IRC08:10
*** agordeev has quit IRC08:10
dtantsurLate morning, Ironic08:10
*** dhellmann_ has quit IRC08:12
*** dhellmann has joined #openstack-ironic08:12
*** derekh has joined #openstack-ironic08:14
*** agordeev has joined #openstack-ironic08:16
*** tzumainn has joined #openstack-ironic08:18
romcheg2Morning dtantsur!08:35
dtantsurromcheg2, hi!08:36
*** lucasagomes has joined #openstack-ironic08:37
*** shakayumi has quit IRC08:40
*** yuanying has quit IRC08:40
*** Poornima has quit IRC08:42
*** nosnos_ has quit IRC08:45
*** nosnos has joined #openstack-ironic08:46
*** nosnos has quit IRC08:50
*** lucasagomes_ has joined #openstack-ironic08:56
*** MattMan has quit IRC08:56
*** MattMan has joined #openstack-ironic08:57
*** lucasagomes has quit IRC08:58
*** viktors|afk is now known as viktors09:01
*** pelix1 has joined #openstack-ironic09:05
rameshg87dtantsur, hi09:05
dtantsurrameshg87, hi!09:06
*** igordcard has joined #openstack-ironic09:06
rameshg87dtantsur, just wanted a little bit of your time to discuss something related to https://review.openstack.org/#/c/113865/09:06
rameshg87dtantsur, is it okay ?09:06
dtantsurI have some time before lunch, yes :)09:07
rameshg87dtantsur, okay :)09:07
rameshg87dtantsur, https://review.openstack.org/#/c/113865/11/ironic/drivers/modules/ilo/deploy.py (L118 - L136)09:07
rameshg87dtantsur, we create the boot iso and then upload to swift, you might already gone through this code once09:08
dtantsurright09:08
rameshg87dtantsur, but we never remove this boot iso as of now09:08
dtantsuryou mean from swift? can we have some expiration?09:08
dtantsuror we need it each boot?09:09
rameshg87dtantsur, yeah we need it for each boot09:09
dtantsuryou can delete it on node undeploy09:09
rameshg87dtantsur, yeah, we can do that09:09
rameshg87dtantsur, but right now we hash the value of kernel_id, ramdisk_id, kernel_params to generate the boot iso09:10
rameshg87dtantsur, that means multiple instances can use the same boot iso09:10
dtantsurrameshg87, heh, if we don't want to invent ImageCache for swift, the only option a really see is to have unique UUID for boot iso09:11
rameshg87dtantsur, almost :)09:11
dtantsurand afterwards, think how to reduce duplication09:11
rameshg87dtantsur, so i had proposed in the spec to use a usage_count meta data in the swift object to track how many instances are using boot iso09:11
*** nosnos has joined #openstack-ironic09:11
rameshg87dtantsur, https://github.com/openstack/ironic-specs/blob/master/specs/juno/ironic-ilo-virtualmedia-driver.rst09:11
dtantsurok what's the problem then?09:11
rameshg87dtantsur, the problem is when we have multiple conductors09:11
rameshg87dtantsur, we don't have facility to lock something across conductors09:12
rameshg87dtantsur, if one conductor deletes the image while another checks if the boot_iso exists or not. that can lead to race condition.09:12
rameshg87dtantsur, we need some sort of driver-level lock :(09:12
dtantsurright, but why do you need it? only one conductor should be responsible for your node09:12
dtantsuraaaaah09:12
rameshg87dtantsur, but another node managed by another conductor could be using the same boot_iso, right ?09:13
dtantsurwell, for discovery I also dreamed about such locking09:13
rameshg87dtantsur, :)09:14
rameshg87dtantsur, and then ?09:14
rameshg87dtantsur, are you planning to think one for K ?09:14
dtantsurbut the only option i see now is to have random id09:14
dtantsurI'm not sure09:14
rameshg87dtantsur, or a unique one for each node, which is essentially same as "pxe driver" before ImageCache09:15
dtantsurdiscovery is already very complex09:15
dtantsurright09:15
dtantsurand after it lands, we can think of a better way09:15
rameshg87dtantsur, if i can't find anything, i am just thinking to do that way09:16
dtantsurright09:16
rameshg87dtantsur, don't haash the things, just created boot_iso at time of deploy and remove it after deploy for each node09:16
rameshg87dtantsur, atleast for now ..09:16
dtantsuryep09:16
rameshg87dtantsur, is it okay if we don't implement everything in the spec :)09:17
rameshg87dtantsur, ?09:17
dtantsurwell yes, if we found problems09:17
rameshg87dtantsur, so that requires correction of spec for now ?09:18
rameshg87dtantsur, should i raise a review to correct the spec ?09:18
dtantsurI think no09:19
dtantsurjust leave a comment/todo09:19
rameshg87dtantsur, okay09:20
rameshg87dtantsur, will do that then .. thanks :)09:20
dtantsurnp09:20
*** dtantsur is now known as dtantsur|brb09:20
*** Poornima has joined #openstack-ironic09:27
*** Poornima has quit IRC09:27
*** romcheg1 has joined #openstack-ironic09:29
*** romcheg2 has quit IRC09:31
*** soren_ has quit IRC09:32
*** soren_ has joined #openstack-ironic09:32
*** soren_ is now known as soren09:32
*** krtaylor has quit IRC09:32
*** romcheg2 has joined #openstack-ironic09:33
*** romcheg1 has quit IRC09:35
*** yuanying has joined #openstack-ironic09:40
*** Alexei_987 has joined #openstack-ironic09:43
*** yuanying has quit IRC09:46
*** rushiagr_away is now known as rushiagr09:58
*** chenglch has quit IRC10:19
*** romcheg1 has joined #openstack-ironic10:26
*** romcheg2 has quit IRC10:28
*** rushiagr is now known as rushiagr_away10:30
*** romcheg2 has joined #openstack-ironic10:31
*** romcheg3 has joined #openstack-ironic10:34
*** romcheg1 has quit IRC10:35
*** romcheg2 has quit IRC10:36
*** yuanying has joined #openstack-ironic10:42
*** zul has joined #openstack-ironic10:43
*** yuanying has quit IRC10:47
*** zul has quit IRC10:51
*** ifarkas has quit IRC10:55
*** krtaylor has joined #openstack-ironic10:57
*** rakesh_hs has quit IRC10:58
*** ifarkas has joined #openstack-ironic11:02
*** zul has joined #openstack-ironic11:04
*** dhellmann has quit IRC11:10
*** zul has quit IRC11:11
*** dhellmann has joined #openstack-ironic11:13
*** ifarkas has quit IRC11:14
*** yuanying has joined #openstack-ironic11:15
*** lucasagomes_ is now known as lucasagomes11:17
*** lucasagomes is now known as lucas-hungry11:17
*** nosnos has quit IRC11:20
*** nosnos has joined #openstack-ironic11:20
*** nosnos has quit IRC11:25
*** ifarkas has joined #openstack-ironic11:27
*** rameshg87 has quit IRC11:32
*** dtantsur|brb is now known as dtantsur11:45
dtantsurfolks, is there a way to cancel gate job approved by mistake?11:46
dtantsurit's about https://review.openstack.org/#/c/105795/11:46
dtantsuryuriyz, lucas-hungry, any ideas?11:48
yuriyzmorning Ironic11:49
dtantsuryuriyz, morning11:50
yuriyzI see gate job not started11:50
dtantsuryuriyz, it did, I can see it11:50
*** faizan has joined #openstack-ironic11:59
dtantsurok according to #infra, removing +A is enough12:09
*** yuanying has quit IRC12:15
*** faizan has quit IRC12:22
*** vdrok has quit IRC12:23
*** Isotopp_ is now known as Isotopp12:23
*** todd_dsm has joined #openstack-ironic12:39
lucas-hungrydtantsur, sorry was having lunch... I see u removed the +A so it stopped12:44
lucas-hungryI didn't know that, I thought that if u changed ur vote to -2 it would stop as well12:45
*** lucas-hungry is now known as lucasagomes12:45
dtantsuryeah, my epic fail cancelled )12:45
*** yuanying has joined #openstack-ironic12:45
*** yuanying has quit IRC12:51
*** romcheg3 has quit IRC12:53
*** romcheg1 has joined #openstack-ironic12:53
*** linggao has joined #openstack-ironic12:55
*** jasondotstar has joined #openstack-ironic13:02
*** jcoufal has quit IRC13:05
*** jcoufal has joined #openstack-ironic13:08
*** lazy_prince is now known as killer_prince13:11
*** rloo has joined #openstack-ironic13:28
*** shakamunyi has joined #openstack-ironic13:41
*** yuanying has joined #openstack-ironic13:47
*** rameshg87 has joined #openstack-ironic13:48
*** pcrews has joined #openstack-ironic13:52
*** yuanying has quit IRC13:52
*** rameshg87 has quit IRC13:56
*** shakamunyi has quit IRC13:58
*** shakamunyi has joined #openstack-ironic14:12
rloohello Iroknickers14:14
rloolucasagomes: question about 113276, test_get_node_supported_boot_devices(). That is for a fake driver, so the expected resp is 'pxe'?14:15
lucasagomesrloo, morning14:16
lucasagomesrloo, so I changed that to not check for any 'pxe' stuff14:16
lucasagomesjust to check the format of the json/dict being returned14:16
lucasagomesbut yeah it's tested with the fake driver14:16
rloohi lucasagomes. yeah, cuz of that other person's comment. your change is fine with me. but you could check for 'pxe'. if you want.14:17
rloolucasagomes: wanted to check with you before I +1 it.14:17
rloolucasagomes: it is probably better not checking for 'pxe' in case we change fake to support other ones. we don't want to update the tempest test if we do that.14:18
rloolucasagomes: so forget that i asked ;)14:18
lucasagomesrloo, yeah... after he said I thought it was indeed better to not check for anything driver-specific14:18
lucasagomesthat's why I changed14:18
lucasagomesI just issued a recheck there14:19
lucasagomesthere was a couple of errors on not related tests14:19
rloolucasagomes: yeah, i saw those failures. but with grenade and I wasn't going to look into it ;) Hopefully it'll get approved soon.14:20
lucasagomesrloo, /me fingers crossed14:20
rloolucasagomes: yup!14:21
rloolucasagomes: I'm off after today (til next Tues). Are there any patches that need reviews soon?14:21
lucasagomes:O14:22
lucasagomeshmm the J3 ones only14:22
*** shakamunyi has quit IRC14:22
lucasagomeslemme check if the DHCP one was updated14:22
rloolucasagomes: no, dhcp hsn't yet been updated. will bug jroll later.14:22
lucasagomesrloo, not yet... I'm putting the drac one pretty soon14:22
lucasagomeslike 20 min14:22
rloolucasagomes: ok, will look at some of the other patches then (uefi, swit, blah blah)14:23
lucasagomesrloo, ack14:23
lucasagomesthanks14:23
*** shakamunyi has joined #openstack-ironic14:23
NobodyCa1good morning Ironic, says the man making coffee14:25
rloomorning NobodyCa114:28
yuriyzIronic people, please look at https://bugs.launchpad.net/ironic/+bug/1362114 I prefer add pyghmi to requirements because this is not vendor-specific lib, what you think?14:28
rlooNobodyCal?14:28
lucasagomesmorning NobodyCa114:29
NobodyCa1gah14:30
*** NobodyCa1 is now known as NobodyCam14:30
NobodyCammorning rloo lucasagomes yuriyz14:30
yuriyzmorning NobodyCam14:30
rlooyuriyz: wrt 1362114. pyghmi used to be in requirements, but deva removed it.14:32
rlooyuriyz: https://review.openstack.org/#/c/92819/14:33
yuriyzrloo thanks14:34
*** ifarkas_ has joined #openstack-ironic14:36
*** ifarkas has quit IRC14:38
*** shakamunyi has quit IRC14:40
dtantsurrloo, NobodyCam, morning14:42
rloohi dtantsur14:43
NobodyCammorning dtantsur14:44
*** yuanying has joined #openstack-ironic14:49
*** HenryG has joined #openstack-ironic14:51
*** stelfer has joined #openstack-ironic14:51
Shrewsromcheg: are 112402 and 112575 the only reviews for the nova bm db migration? want to make sure i hit all of them14:54
Shrewsadam_g, too ^^^14:54
romcheg1Shrews: yup14:54
Shrewsgreat14:54
*** todd_dsm has quit IRC14:54
*** yuanying has quit IRC14:55
*** openstackgerrit has joined #openstack-ironic14:55
openstackgerritJarrod Johnson proposed a change to stackforge/pyghmi: Fix IO worker tolerance of errors  https://review.openstack.org/11724815:01
*** mikal has quit IRC15:05
*** mikal has joined #openstack-ironic15:06
openstackgerritA change was merged to openstack/ironic: Updated from global requirements  https://review.openstack.org/11560015:17
*** igordcard has quit IRC15:17
NobodyCammorning Shrews and romcheg1 :)15:18
ShrewsNobodyCam: bonjour!15:18
*** romcheg1 has quit IRC15:19
*** dguerri_ has quit IRC15:22
*** kincl has quit IRC15:22
*** yongli has quit IRC15:22
*** tteggel has quit IRC15:22
*** kylestev has quit IRC15:22
*** romcheg1 has joined #openstack-ironic15:22
*** dguerri_ has joined #openstack-ironic15:22
*** kincl has joined #openstack-ironic15:22
*** yongli has joined #openstack-ironic15:22
*** tteggel has joined #openstack-ironic15:22
*** kylestev has joined #openstack-ironic15:22
*** jcoufal has quit IRC15:35
*** keekz has quit IRC15:36
*** openstack has joined #openstack-ironic16:19
*** eghobo has joined #openstack-ironic16:22
*** eghobo has quit IRC16:23
*** eghobo has joined #openstack-ironic16:23
*** penick has quit IRC16:29
openstackgerritA change was merged to stackforge/pyghmi: Fix IO worker tolerance of errors  https://review.openstack.org/11724816:31
*** Hefeweiz1n is now known as Hefeweizin16:32
*** penick has joined #openstack-ironic16:33
*** derekh has quit IRC16:38
*** rushiagr_away is now known as rushiagr16:43
*** todd_dsm has joined #openstack-ironic16:46
*** krtaylor has quit IRC16:47
*** Alexei_987 has quit IRC16:48
*** foexle has joined #openstack-ironic16:51
*** chuckC has joined #openstack-ironic16:51
*** yuanying has joined #openstack-ironic16:52
*** yuanying has quit IRC16:57
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: DRAC wsman_{enumerate, invoke}() to return an ElementTree object  https://review.openstack.org/11687116:58
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Implements the DRAC ManagementInterface for get/set boot device  https://review.openstack.org/11691316:58
lucasagomesaight I will call it a day17:01
lucasagomeshave a great night everyone!17:01
rloonight lucasagomes17:01
romcheg1G'night lucasagomes!17:01
*** athomas has quit IRC17:01
lucasagomesnight :)17:01
*** lucasagomes is now known as lucas-dinner17:01
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Make DHCP provider pluggable  https://review.openstack.org/11235117:01
NobodyCamnught lucas-dinner17:02
jrollmorning ironic :)17:02
jrollnight lucas-dinner! sure you don't want to review this first? :P ^^17:02
jrollNobodyCam, rloo, I fixed up the dhcp patch17:03
NobodyCammorning jroll17:03
jrollmorning NobodyCam :)17:03
lucas-dinnerjroll, hey I gotta finish some stuff here, i can review it today later17:03
rloojroll: thx, will look at it soon.17:03
lucas-dinnerthanks for updating that17:03
jrollha no rush, just letting y'all know, thanks :D17:03
*** slagle has joined #openstack-ironic17:03
*** harlowja has joined #openstack-ironic17:04
rloojroll: do you want to do some co-author whatever thingy in the commit for the dhcp patch17:05
jrollrloo: nah, I just made a couple changes :)17:05
jrollalso, I +2'd it a few patchsets ago, still happy with the code17:05
rloojroll: ok, fine with me. wanted to make sure in case I +2'd it :D17:05
jroll:)17:06
jrollrloo: btw, I ran the conductor locally, that try/except works as expected17:06
openstackgerritKyle Stevenson proposed a change to openstack/ironic-specs: Add ZooKeeper SyncManager spec to Kilo specs  https://review.openstack.org/11704017:06
rloojroll: thx for testing17:06
jrollnp17:07
jrollI would test with devstack/tempest, but too lazy to set up an external dhcp server :P17:07
jrollfyi, despite the jenkins failures (issued a recheck), I tested this with devstack and it passes tempest: https://review.openstack.org/#/c/114408/17:11
jrollJayF: ^^ relevant to your interests17:12
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Add support for interacting with swift  https://review.openstack.org/10579517:13
*** rloo has quit IRC17:16
*** stelfer has quit IRC17:19
*** penick has quit IRC17:23
*** penick has joined #openstack-ironic17:31
*** lazy_prince is now known as killer_prince17:47
*** pelix1 has quit IRC17:53
*** ramineni has quit IRC17:54
*** yuanying has joined #openstack-ironic17:54
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Fix unit tests with keystoneclient master  https://review.openstack.org/11731417:54
jrollNobodyCam: ^^ that fixes broken unit tests17:54
NobodyCamon a conf call atm17:55
NobodyCamis our gate broken?17:55
jrollno worries17:56
jrollno, it's not released yet17:56
jroll*my* gate is broken :P17:56
jrollwriting reproduction steps now17:57
jrollare you going to want a bug for that?17:57
* jroll will likely make one anyway17:57
*** foexle has quit IRC17:59
*** yuanying has quit IRC17:59
*** mikal has quit IRC18:00
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Fix unit tests with keystoneclient master  https://review.openstack.org/11731418:00
jrollupdated with a bug # ^18:01
*** mikal has joined #openstack-ironic18:05
*** annegent- has joined #openstack-ironic18:15
*** rushiagr is now known as rushiagr_away18:16
openstackgerritA change was merged to openstack/ironic-specs: Update spec theme to show incubation status  https://review.openstack.org/11630618:16
jrollShrews: I'd like to land this today if you want to take a peek as well :) https://review.openstack.org/11731418:17
NobodyCamjroll: +2'd18:22
NobodyCambrb...quick run to starbucks :)18:22
*** rloo has joined #openstack-ironic18:24
openstackgerritAdam Gandelman proposed a change to openstack/ironic: Script to migrate Nova BM data to Ironic  https://review.openstack.org/11240218:29
openstackgerritAndreas Jaeger proposed a change to openstack/ironic-specs: Remove docutils pin  https://review.openstack.org/11718018:31
Shrewsjroll: will peek18:32
*** annegent- is now known as annegentle18:38
rloojroll: wrt 117314, will it break our code too (besides the test)?18:39
*** zigo has quit IRC18:43
Shrewsso, are we getting the "You need to specify either a username or user_id" message b/c of that keystoneclient change?18:43
* Shrews tests18:43
NobodyCamanyone seen this b4? http://logs.openstack.org/02/112402/6/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/853324e/console.html#_2014-08-26_19_16_56_19118:45
*** harlowja has quit IRC18:46
*** zigo has joined #openstack-ironic18:46
*** harlowja has joined #openstack-ironic18:46
Shrewsjroll: rloo: if this breaks our tests, isn't this a backward incompatible change to keystoneclient?18:47
greghaynesNobodyCam: There are some recent vlan patches going in tripleo18:48
greghayneslifeless: ^ has been hacking on that a bunch18:48
rlooShrews, jroll: that's what I would think but I don't know what keystone folks think. They had a typo in their patch too :-(18:49
NobodyCamgreghaynes: Ack Ty for the info :)18:49
rlooShrews, jroll: it doesn't hurt to add the patch that jroll put in. I'm just wondering if it might break our code (or how our users use our code)18:49
Shrewsrloo: yeah. i'm trying to understand the implications of this  :/18:50
rloothx Shrews! I was hoping not to understand it (trying to review some other stuff).18:50
Shrewsrloo: oh, so you just ask the hard questions and walk away   :-P18:51
Shrewslol18:51
rlooShrews: well, I ask questions and *wait* for an answer before taking any action. So since I got additional info that you're looking into it, I don't feel guilty walking away ;)18:51
Shrewshehe18:53
*** yuanying has joined #openstack-ironic18:55
*** yuanying has quit IRC19:01
*** todd_dsm has quit IRC19:02
*** krtaylor has joined #openstack-ironic19:08
lifelessgreghaynes: speaking of vlans19:12
lifelessgreghaynes: https://review.openstack.org/#/c/103454/ really needs a second +2/+A19:13
greghaynesmmmm19:14
Shrewsrloo: ok, so actual value of the admin_user doesn't matter it seems. and it doesn't break our code since my local devstack has that keystoneclient change and things appear ok19:15
rlooShrews: does your local devstack have admin_user specified?19:16
Shrewsrloo: no19:16
rlooShrews: that is interesting. your devstack is using keystone to authenticate, right?19:17
Shrewsrloo: it's a standard devstack setup.19:17
Shrewsrloo: nova specifies the username via it's nova.conf19:18
rlooShrews: hmm. oh well.19:18
rlooShrews: Oh, so keystone gets some user-name/id then, in devstack.19:18
Shrews[ironic]19:19
Shrewsadmin_username = admin19:19
Shrewsrloo: right19:19
rlooShrews: so wrt ironic, it might only be a problem if someone issues ironic commands that need to authenticate against keystone. but you'd need to specify the user/admin name for that19:20
Shrewsrloo: yeah, i've always had to add the '--os-username admin' option to ironic client to do anything19:21
JayFThe change is solely in unit test setups19:21
JayFjroll's change19:21
ShrewsJayF: right, but we were wondering about the bigger implications of the keystoneclient change19:21
JayFit should change absolutely nothing about ironic's behavior whatsoever outside of unit tests19:21
rlooShrews: just looked at our client code. it wants username specified, so we're ok there.19:21
JayFaha19:22
JayFso you guys are just curious if we need moar change19:22
Shrewsright19:22
rlooJayF: yeah. just paranoid is all ;)19:22
JayFmy suggestion is to land that patch as it sits19:22
NobodyCamparanoid is a GODD thing19:22
JayFbefore keystoneclient releases, breaks our gate, and we have a bad time :)19:22
NobodyCamgood even19:22
JayFI'm paranoid they're going to release and break everything19:22
JayFlol19:22
NobodyCamJayF: add "again" to that line and I'll +2 it19:23
JayFI'm paranoid they're going to release and break everything "again"19:23
NobodyCam+219:23
Shrewsyeah, i'm fine with jroll's change now19:23
*** r-daneel has joined #openstack-ironic19:24
* Shrews checks jenkins progress19:24
rlooShrews: I +2'd it but leaving for you to approve ;)19:24
Shrewsrloo: ok. i'll +A after jenkins is done with it19:25
rloothx Shrews for looking into it.19:25
Shrewsi love that review.o.o randomly logs me out19:25
JayFShrews: nice to know it isn't just me with that problem19:25
JayFand the way logins are handled are a pain if it happens when a dozen r.o.o tabs open19:26
ShrewsJayF: by "love", i mean "hate". and by "hate", i mean "really really want to destroy"19:26
* rloo wonders what Shrews and JayF are talking about but doesn't want to find out. Knocking on wood...19:27
Shrewsrloo: random log outs in gerrit19:28
JayFI don't think it's random19:28
JayFI think it waits until I have the maximum number of gerrit tabs open19:28
JayFthen logs me out19:28
JayFmalice, not random :P19:28
Shrewsspitefully intentional?19:28
NobodyCamlol19:29
rloohmm, I think I get logged out about once a week or so. Not very frequent anyway.19:29
NobodyCamoh it may be that LogOffJayF function I landed last week19:29
NobodyCamlol19:29
NobodyCamjust as a side note. turns out we actually have folks using https://review.openstack.org/#/c/9895019:30
rlooNobodyCam: tripleO folks?19:31
rlooNobodyCam: why aren't those folks helping to review that patch?19:32
NobodyCamhp folk :)19:32
lifelessrloo: they are ops/deployment folk, not self identifying as programmers19:32
rlooNobodyCam: Ohhhhh. hp.19:32
rloolifeless: who told them about it? :D19:32
NobodyCam:) oh lifeless I'm look at that now19:32
NobodyCamfyi19:32
lifelessrloo: I wrote it for them19:32
lifelessrloo: because they have datacentres without IPMI/iLO accessible (-long- story)19:33
lifelessNobodyCam: thank you!19:33
Shrewsi've been meaning to pick that up and make a spec for it, but stuff gets in the way19:34
rlooI was just wondering that. Does it need a spec? if so, it won't get in til kilo.19:34
Shrewsrloo: deva says "yes"19:34
Shrewsand i quote, "It's a driver. Drivers need specs"19:35
rlooShrews: ha ha.19:35
rlooShrews: was iboot exempt from specs?19:36
JayFI think it predated the spec existing19:37
JayFs/the specs/ironic-specs/19:37
rlooJayF: yeah, that might have been it. The logpower driver was drafted in June, after the specs gates were open. Sorry, should have noticed/noted that on the patch.19:38
NobodyCamlifeless: was the log power driver planed on landing?19:39
lifelessNobodyCam: Conceptually I'd rather have it in tree than unmaintained and floating around19:39
*** todd_dsm has joined #openstack-ironic19:40
NobodyCamlifeless: ack is "K" cycle ok?19:40
lifelessNobodyCam: of course19:41
NobodyCam:)19:41
* jroll is back19:42
jrollthanks for the reviews y'all :)19:42
rloojroll: thx for detecting and fixing!19:45
jroll:)19:45
*** todd_dsm has quit IRC19:48
NobodyCambrb quick walkies19:51
Shrewsjroll: grrr. 117314 has been in the gate for 2 hours and still queued19:53
jrollI know :/19:53
jrollit's ok19:53
jrollit's not actually a problem until keystone client gets a release19:54
Shrewsjroll: i think it's acceptable if you +A yourself it since you have 3 other +2's  :)19:54
jrollyeah, waiting for jenkins though19:54
Shrewsonce jenkins ok's it19:54
Shrewsyeah19:54
jrollI'll keep an eye out19:54
*** yuanying has joined #openstack-ironic19:57
*** todd_dsm has joined #openstack-ironic19:58
Shrewsadam_g: how are you testing the nova bm migration script?19:59
adam_gShrews, grenade20:00
adam_gShrews, if you give me a few i can get you a database dump of a nova-bm db with some stuff in it20:01
Shrewsadam_g: ok20:02
*** tzumainn has left #openstack-ironic20:10
*** todd_dsm has quit IRC20:12
ShrewsSo, a quick raid on my fridge has revealed to me that I have no post-work beer available.20:18
ShrewsThis is, without a doubt, the worst thing ever in history20:18
kylestevShrews: I believe there are beer delivery services out there20:18
Shrewswell, recent history... within my home20:18
kylestevdepends on where you live, though :P20:18
Shrewskylestev: if we had that locally, i would be a hermit, i'm sure20:19
kylestevhaha20:19
jrollShrews: the good news is that you haven't started drinking post-work beer, so you can still drive to fetch some :P20:20
Shrewsjroll: the post-beer scotch is calling me... i should leave quickly20:21
adam_gShrews, if it makes you feel any better, we've had a quarter keg of IPA in the fridge for the last week and no way of tapping it20:21
jrollShrews: hehehe :D20:21
Shrewsadam_g: THE HORROR!20:21
JayFI've had IPA in the git repos for a long time20:21
JayFand I'm not sure anyone but us has tapped it20:21
* jroll hangs his head and sighs20:21
*** todd_dsm has joined #openstack-ironic20:22
adam_gha20:24
*** yuanying has quit IRC20:30
openstackgerritlinggao proposed a change to openstack/ironic: Interactive console support for ipminative driver  https://review.openstack.org/9733120:38
adam_gShrews, http://no-carrier.net/~adam/nova_bm.sql20:41
* NobodyCam should learn to look at what he is typing :-p20:41
Shrewsadam_g: heh, very little data there. i can probably create some random entries20:44
adam_gShrews, thats populated with the BM nodes we'd use in a devstack run. the grenade test inserts them into the nova db, then migrates them directly into ironic's instead of enrolling them via API20:45
*** stevebaker has quit IRC20:49
*** stevebaker has joined #openstack-ironic20:49
*** stevebaker has joined #openstack-ironic20:49
*** stevebaker has joined #openstack-ironic20:49
*** stevebaker has joined #openstack-ironic20:49
*** stevebaker has joined #openstack-ironic20:49
kylestevstevebaker is a little too eager it seems20:50
*** stevebaker has joined #openstack-ironic20:50
*** stevebaker has joined #openstack-ironic20:50
*** stevebaker has quit IRC20:50
*** stevebaker has joined #openstack-ironic20:50
*** stevebaker has joined #openstack-ironic20:50
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Make DHCP provider pluggable  https://review.openstack.org/11235120:52
jrollrloo: ^ fixed pep8 failure20:54
rloojroll: there seem to be tempest failures check-tempest-dsvm-ironic & -postgres. not just for that patch.20:55
jrollrloo: yeah, but they seem unrelated20:55
jrolloh, not for that patch. great.20:56
jrollif our gate is broken I need sharp things20:56
rloojroll: yeah, not just for that patch. I see 'setUpClass failed: Neutron support is required' in tempest logs.20:56
rloojroll: but don't know if that is normal or not.20:56
jrollI see a lot of these tempest.scenario.test_dashboard_basic_ops.TestDashboardBasicOps.test_basic_scenario [10.154259s] ... FAILED20:56
jrolltwo of them, to be specific20:56
*** linggao has quit IRC20:57
jrolloh, yep, I see your thing too20:57
rloojroll: that basic scenario has nothing to do with ironic?20:57
jrollthose look like skipped tests, though20:58
jrollnope, we don't have a dashboard yet :)20:58
rloojroll: sigh. gotta take off for a bit too. not sure if that is good or bad but whee, can't look into it now ;)20:58
jrollheh, ok, have fun :)20:58
* jroll will brb and then look more20:59
Shrewsrloo: jroll: there are horizon issues atm20:59
jrollah ok, thanks20:59
jrollthey are on top of them?20:59
Shrewshttps://bugs.launchpad.net/horizon/+bug/136232520:59
Shrewshttps://review.openstack.org/#/c/117350/20:59
Shrewsdunno about an actual fix yet21:00
*** penick has quit IRC21:00
jrollyeah, ok21:00
Shrewsyou have that in your horizon_error.txt.gz file21:00
jrollyep21:00
jrollthanks for the info :)21:00
Shrewsyup21:00
*** krtaylor has quit IRC21:04
*** stevebaker has quit IRC21:08
*** stevebaker has joined #openstack-ironic21:08
*** foexle has joined #openstack-ironic21:16
*** foexle has quit IRC21:26
*** yuanying has joined #openstack-ironic21:27
jrollugh horizon, you're killin me21:28
*** jasondotstar has quit IRC21:28
NobodyCamit a GUI what did you expect?21:32
NobodyCamlol21:32
NobodyCam(j/k)21:32
jrollheh21:32
jrollI poked their channel, they're looking into it21:33
Shrewsgood news. i have secured my post-work beer. everyone can relax now.21:36
*** zer0c00l1 is now known as zer0c00l21:36
jroll\o/21:37
NobodyCamlol21:38
*** foexle has joined #openstack-ironic21:41
*** romcheg has quit IRC21:52
*** vdrok_ has quit IRC21:52
*** viktors has quit IRC21:52
*** vdrok_ has joined #openstack-ironic21:52
*** viktors has joined #openstack-ironic21:53
*** romcheg has joined #openstack-ironic21:53
*** yuanying has quit IRC22:00
rloofwiw, I updated our IronicWhiteBoard with the latest status. I am wondering if we should keep a list of all the gate failures, but that may be depressing.22:09
jrollI don't see any value in keeping remnants of old failures22:11
JayFI do.22:12
JayFEspecially failures caused by not being in the integrated gate22:12
JayFhopefully we can get some kind of RCA from the TC as to how to make other new projects not slurg through some of the crap we've had to22:12
jrollok, I don't see any value in keeping that in the whiteboard22:13
*** harlowja_ has joined #openstack-ironic22:13
rloojroll: yeah, not in the whiteboard. I wasn't going to do anything about it; just thought I'd throw out the question in case someone else was interested ;)22:15
*** harlowja_ has quit IRC22:15
*** harlowja_ has joined #openstack-ironic22:15
jrollrloo: ok :)22:15
rloohmm, I should put a note in the whiteboard that there's no use doing a recheck until it is fixed.22:16
*** harlowja has quit IRC22:16
jrollI mean, that's kind of a given22:16
rloojroll: yeah, except i've seen cases in the past where people do rechecks anyway :-(22:17
jrollwow22:18
JayFI'm assuming $right_people know about the gate failures?22:20
rlooJayF: yeah, someone (jroll?) updated the etherpad with links for the fixes.22:22
jrollyeah, fixes inbound22:22
jrollsomeone from infra is ninja-bumping it to the front of the gate, I guess22:22
jrolls/it/them/ (them being the *two* patches to fix this)22:23
adam_ghttps://review.openstack.org/#/c/112000 and https://review.openstack.org/#/c/11200122:24
NobodyCamagain22:25
jrollyah22:25
jrollexcept I think the entire integrated gate might be broken :P22:25
NobodyCamcrys22:27
jrollinfra is on it22:27
NobodyCamwhen they land lets just jand everything on our queue22:28
NobodyCamlol22:28
jroll+122:28
NobodyCam:-p22:28
jrollthis is where nova approves our driver and we fail gate for days :P22:29
NobodyCamNO DONT EVEN THINK THAT :-p22:29
jrolllol22:29
JayFI'm Jay Faulkner, and I approved this patch :-D22:32
JayFwtf I still have too many gettys respawning22:33
JayFWHAT AM I GOING TO DO WITH ALL THESE GETTIES22:33
*** keekz_ is now known as keekz22:35
JayFwrong channel on that one :)22:35
* JayF fighting old distributions and serial ports22:35
*** lucas-dinner has quit IRC22:35
jrolllol22:41
*** krtaylor has joined #openstack-ironic22:45
*** ZackaryDoyle has joined #openstack-ironic22:53
*** yuanying has joined #openstack-ironic22:57
NobodyCamgah, what am I missing23:00
NobodyCamto add a driver23:00
JayFI don't know23:00
NobodyCamadd to setup.cfg23:00
JayFbut I can tell you how to prevent 4 getties from spawning on the same serial terminal in centos 6.523:00
JayFif that's useful for you23:00
JayFlol23:00
NobodyCam:-p23:01
*** yuanying has quit IRC23:01
jrollNobodyCam: python setup.py (develop || install) ?23:04
jrollNobodyCam: setup.cfg only gets parsed when you do that23:04
*** penick has joined #openstack-ironic23:06
NobodyCami'm actually testing with tox. and I did a rebuild23:12
jrolloh, hm23:13
jrollenabled_drivers ?23:13
NobodyCamtest_fake does not fiddle with that and it passes23:14
NobodyCamDriverNotFound: Failed to load driver pxe_logpower23:14
jrolloh, right23:14
NobodyCamI've done something wrong23:14
jrollidk23:14
*** romcheg1 has left #openstack-ironic23:22
*** yuanying has joined #openstack-ironic23:24
*** foexle has quit IRC23:30
*** penick has quit IRC23:39
mrdaMorning Ironic23:43
NobodyCammorning mrda23:49
mrda\o23:51
openstackgerritKyle Stevenson proposed a change to openstack/ironic: Add implementation of pluggable distributed sync  https://review.openstack.org/11514823:52

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