*** Shrews_ is now known as Shrews | 00:05 | |
jroll | JayF, NobodyCam, I +A'd https://review.openstack.org/#/c/116742/ | 00:06 |
---|---|---|
NobodyCam | :) | 00:12 |
*** penick has quit IRC | 00:13 | |
*** fishcried has quit IRC | 00:24 | |
*** chuckC has joined #openstack-ironic | 00:27 | |
*** todd_dsm has joined #openstack-ironic | 01:00 | |
*** eghobo has quit IRC | 01:25 | |
*** eghobo has joined #openstack-ironic | 01:26 | |
*** shakamunyi has quit IRC | 01:29 | |
*** mitz_ has joined #openstack-ironic | 01:31 | |
*** mitz has quit IRC | 01:33 | |
*** Poornima has joined #openstack-ironic | 01:35 | |
*** shakamunyi has joined #openstack-ironic | 01:36 | |
*** foexle has joined #openstack-ironic | 01:42 | |
*** foexle_ has quit IRC | 01:46 | |
*** todd_dsm has quit IRC | 01:52 | |
*** eguz has joined #openstack-ironic | 01:54 | |
*** todd_dsm has joined #openstack-ironic | 01:55 | |
*** nosnos has joined #openstack-ironic | 01:56 | |
*** pcrews has quit IRC | 01:57 | |
*** eghobo has quit IRC | 01:58 | |
*** foexle has quit IRC | 02:03 | |
*** todd_dsm has quit IRC | 02:15 | |
*** todd_dsm has joined #openstack-ironic | 02:19 | |
*** todd_dsm has quit IRC | 02:20 | |
*** harlowja is now known as harlowja_away | 02:44 | |
*** lazy_prince is now known as killer_prince | 02:57 | |
*** nosnos has quit IRC | 03:23 | |
*** nosnos has joined #openstack-ironic | 03:24 | |
*** nosnos has quit IRC | 03:28 | |
*** harlowja_away has quit IRC | 03:29 | |
*** killer_prince has quit IRC | 03:50 | |
*** killer_prince has joined #openstack-ironic | 03:51 | |
*** killer_prince is now known as lazy_prince | 03:51 | |
*** nosnos has joined #openstack-ironic | 03:53 | |
*** eghobo has joined #openstack-ironic | 04:04 | |
*** eghobo has quit IRC | 04:12 | |
*** ramineni has quit IRC | 04:17 | |
*** todd_dsm has joined #openstack-ironic | 04:26 | |
*** rameshg87 has joined #openstack-ironic | 04:29 | |
*** eghobo has joined #openstack-ironic | 04:42 | |
*** chuck_ has quit IRC | 04:49 | |
*** chenglch has joined #openstack-ironic | 05:08 | |
*** yuanying has joined #openstack-ironic | 05:13 | |
*** tzumainn has quit IRC | 05:25 | |
*** tzumainn has joined #openstack-ironic | 05:25 | |
*** rameshg87 has quit IRC | 05:36 | |
*** rameshg87 has joined #openstack-ironic | 05:37 | |
*** rameshg87 has quit IRC | 05:37 | |
*** rameshg87 has joined #openstack-ironic | 05:38 | |
*** todd_dsm has quit IRC | 05:51 | |
*** ramineni has joined #openstack-ironic | 05:56 | |
*** k4n0 has joined #openstack-ironic | 05:59 | |
*** rameshg87 has quit IRC | 06:19 | |
*** rameshg87 has joined #openstack-ironic | 06:20 | |
*** rakesh_hs has joined #openstack-ironic | 06:29 | |
*** jcoufal has joined #openstack-ironic | 06:38 | |
*** eghobo has quit IRC | 06:44 | |
*** foexle has joined #openstack-ironic | 07:03 | |
*** nosnos has quit IRC | 07:04 | |
*** nosnos_ has joined #openstack-ironic | 07:04 | |
*** rameshg87 has quit IRC | 07:07 | |
*** rameshg87 has joined #openstack-ironic | 07:08 | |
mrda | Night ironic! | 07:17 |
Haomeng | mrda: night:) | 07:25 |
*** vdrok has joined #openstack-ironic | 07:39 | |
*** yuanying_ has joined #openstack-ironic | 07:43 | |
*** yuanying_ has quit IRC | 07:46 | |
*** yuanying_ has joined #openstack-ironic | 07:46 | |
*** yuanying has quit IRC | 07:47 | |
*** chenglch has quit IRC | 07:52 | |
*** chenglch has joined #openstack-ironic | 07:52 | |
*** romcheg2 has joined #openstack-ironic | 07:56 | |
*** romcheg1 has quit IRC | 07:59 | |
*** jistr has joined #openstack-ironic | 08:00 | |
*** shakayumi has joined #openstack-ironic | 08:01 | |
*** dguerri_ has joined #openstack-ironic | 08:01 | |
*** zer0c00l1 has joined #openstack-ironic | 08:02 | |
*** zer0c00l has quit IRC | 08:02 | |
*** BadCub has joined #openstack-ironic | 08:03 | |
*** gilliard_ has joined #openstack-ironic | 08:05 | |
*** Hefeweiz1n has joined #openstack-ironic | 08:05 | |
*** dhellmann_ has joined #openstack-ironic | 08:08 | |
*** NobodyCa1 has joined #openstack-ironic | 08:09 | |
*** yuanying_ has quit IRC | 08:09 | |
*** yuanying has joined #openstack-ironic | 08:09 | |
*** shakamunyi has quit IRC | 08:10 | |
*** NobodyCam has quit IRC | 08:10 | |
*** BadCub_ has quit IRC | 08:10 | |
*** gilliard has quit IRC | 08:10 | |
*** tzumainn has quit IRC | 08:10 | |
*** dguerri has quit IRC | 08:10 | |
*** Hefeweizen has quit IRC | 08:10 | |
*** dhellmann has quit IRC | 08:10 | |
*** agordeev has quit IRC | 08:10 | |
dtantsur | Late morning, Ironic | 08:10 |
*** dhellmann_ has quit IRC | 08:12 | |
*** dhellmann has joined #openstack-ironic | 08:12 | |
*** derekh has joined #openstack-ironic | 08:14 | |
*** agordeev has joined #openstack-ironic | 08:16 | |
*** tzumainn has joined #openstack-ironic | 08:18 | |
romcheg2 | Morning dtantsur! | 08:35 |
dtantsur | romcheg2, hi! | 08:36 |
*** lucasagomes has joined #openstack-ironic | 08:37 | |
*** shakayumi has quit IRC | 08:40 | |
*** yuanying has quit IRC | 08:40 | |
*** Poornima has quit IRC | 08:42 | |
*** nosnos_ has quit IRC | 08:45 | |
*** nosnos has joined #openstack-ironic | 08:46 | |
*** nosnos has quit IRC | 08:50 | |
*** lucasagomes_ has joined #openstack-ironic | 08:56 | |
*** MattMan has quit IRC | 08:56 | |
*** MattMan has joined #openstack-ironic | 08:57 | |
*** lucasagomes has quit IRC | 08:58 | |
*** viktors|afk is now known as viktors | 09:01 | |
*** pelix1 has joined #openstack-ironic | 09:05 | |
rameshg87 | dtantsur, hi | 09:05 |
dtantsur | rameshg87, hi! | 09:06 |
*** igordcard has joined #openstack-ironic | 09:06 | |
rameshg87 | dtantsur, just wanted a little bit of your time to discuss something related to https://review.openstack.org/#/c/113865/ | 09:06 |
rameshg87 | dtantsur, is it okay ? | 09:06 |
dtantsur | I have some time before lunch, yes :) | 09:07 |
rameshg87 | dtantsur, okay :) | 09:07 |
rameshg87 | dtantsur, https://review.openstack.org/#/c/113865/11/ironic/drivers/modules/ilo/deploy.py (L118 - L136) | 09:07 |
rameshg87 | dtantsur, we create the boot iso and then upload to swift, you might already gone through this code once | 09:08 |
dtantsur | right | 09:08 |
rameshg87 | dtantsur, but we never remove this boot iso as of now | 09:08 |
dtantsur | you mean from swift? can we have some expiration? | 09:08 |
dtantsur | or we need it each boot? | 09:09 |
rameshg87 | dtantsur, yeah we need it for each boot | 09:09 |
dtantsur | you can delete it on node undeploy | 09:09 |
rameshg87 | dtantsur, yeah, we can do that | 09:09 |
rameshg87 | dtantsur, but right now we hash the value of kernel_id, ramdisk_id, kernel_params to generate the boot iso | 09:10 |
rameshg87 | dtantsur, that means multiple instances can use the same boot iso | 09:10 |
dtantsur | rameshg87, heh, if we don't want to invent ImageCache for swift, the only option a really see is to have unique UUID for boot iso | 09:11 |
rameshg87 | dtantsur, almost :) | 09:11 |
dtantsur | and afterwards, think how to reduce duplication | 09:11 |
rameshg87 | dtantsur, 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 iso | 09:11 |
*** nosnos has joined #openstack-ironic | 09:11 | |
rameshg87 | dtantsur, https://github.com/openstack/ironic-specs/blob/master/specs/juno/ironic-ilo-virtualmedia-driver.rst | 09:11 |
dtantsur | ok what's the problem then? | 09:11 |
rameshg87 | dtantsur, the problem is when we have multiple conductors | 09:11 |
rameshg87 | dtantsur, we don't have facility to lock something across conductors | 09:12 |
rameshg87 | dtantsur, if one conductor deletes the image while another checks if the boot_iso exists or not. that can lead to race condition. | 09:12 |
rameshg87 | dtantsur, we need some sort of driver-level lock :( | 09:12 |
dtantsur | right, but why do you need it? only one conductor should be responsible for your node | 09:12 |
dtantsur | aaaaah | 09:12 |
rameshg87 | dtantsur, but another node managed by another conductor could be using the same boot_iso, right ? | 09:13 |
dtantsur | well, for discovery I also dreamed about such locking | 09:13 |
rameshg87 | dtantsur, :) | 09:14 |
rameshg87 | dtantsur, and then ? | 09:14 |
rameshg87 | dtantsur, are you planning to think one for K ? | 09:14 |
dtantsur | but the only option i see now is to have random id | 09:14 |
dtantsur | I'm not sure | 09:14 |
rameshg87 | dtantsur, or a unique one for each node, which is essentially same as "pxe driver" before ImageCache | 09:15 |
dtantsur | discovery is already very complex | 09:15 |
dtantsur | right | 09:15 |
dtantsur | and after it lands, we can think of a better way | 09:15 |
rameshg87 | dtantsur, if i can't find anything, i am just thinking to do that way | 09:16 |
dtantsur | right | 09:16 |
rameshg87 | dtantsur, don't haash the things, just created boot_iso at time of deploy and remove it after deploy for each node | 09:16 |
rameshg87 | dtantsur, atleast for now .. | 09:16 |
dtantsur | yep | 09:16 |
rameshg87 | dtantsur, is it okay if we don't implement everything in the spec :) | 09:17 |
rameshg87 | dtantsur, ? | 09:17 |
dtantsur | well yes, if we found problems | 09:17 |
rameshg87 | dtantsur, so that requires correction of spec for now ? | 09:18 |
rameshg87 | dtantsur, should i raise a review to correct the spec ? | 09:18 |
dtantsur | I think no | 09:19 |
dtantsur | just leave a comment/todo | 09:19 |
rameshg87 | dtantsur, okay | 09:20 |
rameshg87 | dtantsur, will do that then .. thanks :) | 09:20 |
dtantsur | np | 09:20 |
*** dtantsur is now known as dtantsur|brb | 09:20 | |
*** Poornima has joined #openstack-ironic | 09:27 | |
*** Poornima has quit IRC | 09:27 | |
*** romcheg1 has joined #openstack-ironic | 09:29 | |
*** romcheg2 has quit IRC | 09:31 | |
*** soren_ has quit IRC | 09:32 | |
*** soren_ has joined #openstack-ironic | 09:32 | |
*** soren_ is now known as soren | 09:32 | |
*** krtaylor has quit IRC | 09:32 | |
*** romcheg2 has joined #openstack-ironic | 09:33 | |
*** romcheg1 has quit IRC | 09:35 | |
*** yuanying has joined #openstack-ironic | 09:40 | |
*** Alexei_987 has joined #openstack-ironic | 09:43 | |
*** yuanying has quit IRC | 09:46 | |
*** rushiagr_away is now known as rushiagr | 09:58 | |
*** chenglch has quit IRC | 10:19 | |
*** romcheg1 has joined #openstack-ironic | 10:26 | |
*** romcheg2 has quit IRC | 10:28 | |
*** rushiagr is now known as rushiagr_away | 10:30 | |
*** romcheg2 has joined #openstack-ironic | 10:31 | |
*** romcheg3 has joined #openstack-ironic | 10:34 | |
*** romcheg1 has quit IRC | 10:35 | |
*** romcheg2 has quit IRC | 10:36 | |
*** yuanying has joined #openstack-ironic | 10:42 | |
*** zul has joined #openstack-ironic | 10:43 | |
*** yuanying has quit IRC | 10:47 | |
*** zul has quit IRC | 10:51 | |
*** ifarkas has quit IRC | 10:55 | |
*** krtaylor has joined #openstack-ironic | 10:57 | |
*** rakesh_hs has quit IRC | 10:58 | |
*** ifarkas has joined #openstack-ironic | 11:02 | |
*** zul has joined #openstack-ironic | 11:04 | |
*** dhellmann has quit IRC | 11:10 | |
*** zul has quit IRC | 11:11 | |
*** dhellmann has joined #openstack-ironic | 11:13 | |
*** ifarkas has quit IRC | 11:14 | |
*** yuanying has joined #openstack-ironic | 11:15 | |
*** lucasagomes_ is now known as lucasagomes | 11:17 | |
*** lucasagomes is now known as lucas-hungry | 11:17 | |
*** nosnos has quit IRC | 11:20 | |
*** nosnos has joined #openstack-ironic | 11:20 | |
*** nosnos has quit IRC | 11:25 | |
*** ifarkas has joined #openstack-ironic | 11:27 | |
*** rameshg87 has quit IRC | 11:32 | |
*** dtantsur|brb is now known as dtantsur | 11:45 | |
dtantsur | folks, is there a way to cancel gate job approved by mistake? | 11:46 |
dtantsur | it's about https://review.openstack.org/#/c/105795/ | 11:46 |
dtantsur | yuriyz, lucas-hungry, any ideas? | 11:48 |
yuriyz | morning Ironic | 11:49 |
dtantsur | yuriyz, morning | 11:50 |
yuriyz | I see gate job not started | 11:50 |
dtantsur | yuriyz, it did, I can see it | 11:50 |
*** faizan has joined #openstack-ironic | 11:59 | |
dtantsur | ok according to #infra, removing +A is enough | 12:09 |
*** yuanying has quit IRC | 12:15 | |
*** faizan has quit IRC | 12:22 | |
*** vdrok has quit IRC | 12:23 | |
*** Isotopp_ is now known as Isotopp | 12:23 | |
*** todd_dsm has joined #openstack-ironic | 12:39 | |
lucas-hungry | dtantsur, sorry was having lunch... I see u removed the +A so it stopped | 12:44 |
lucas-hungry | I didn't know that, I thought that if u changed ur vote to -2 it would stop as well | 12:45 |
*** lucas-hungry is now known as lucasagomes | 12:45 | |
dtantsur | yeah, my epic fail cancelled ) | 12:45 |
*** yuanying has joined #openstack-ironic | 12:45 | |
*** yuanying has quit IRC | 12:51 | |
*** romcheg3 has quit IRC | 12:53 | |
*** romcheg1 has joined #openstack-ironic | 12:53 | |
*** linggao has joined #openstack-ironic | 12:55 | |
*** jasondotstar has joined #openstack-ironic | 13:02 | |
*** jcoufal has quit IRC | 13:05 | |
*** jcoufal has joined #openstack-ironic | 13:08 | |
*** lazy_prince is now known as killer_prince | 13:11 | |
*** rloo has joined #openstack-ironic | 13:28 | |
*** shakamunyi has joined #openstack-ironic | 13:41 | |
*** yuanying has joined #openstack-ironic | 13:47 | |
*** rameshg87 has joined #openstack-ironic | 13:48 | |
*** pcrews has joined #openstack-ironic | 13:52 | |
*** yuanying has quit IRC | 13:52 | |
*** rameshg87 has quit IRC | 13:56 | |
*** shakamunyi has quit IRC | 13:58 | |
*** shakamunyi has joined #openstack-ironic | 14:12 | |
rloo | hello Iroknickers | 14:14 |
rloo | lucasagomes: question about 113276, test_get_node_supported_boot_devices(). That is for a fake driver, so the expected resp is 'pxe'? | 14:15 |
lucasagomes | rloo, morning | 14:16 |
lucasagomes | rloo, so I changed that to not check for any 'pxe' stuff | 14:16 |
lucasagomes | just to check the format of the json/dict being returned | 14:16 |
lucasagomes | but yeah it's tested with the fake driver | 14:16 |
rloo | hi 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 |
rloo | lucasagomes: wanted to check with you before I +1 it. | 14:17 |
rloo | lucasagomes: 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 |
rloo | lucasagomes: so forget that i asked ;) | 14:18 |
lucasagomes | rloo, yeah... after he said I thought it was indeed better to not check for anything driver-specific | 14:18 |
lucasagomes | that's why I changed | 14:18 |
lucasagomes | I just issued a recheck there | 14:19 |
lucasagomes | there was a couple of errors on not related tests | 14:19 |
rloo | lucasagomes: 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 |
lucasagomes | rloo, /me fingers crossed | 14:20 |
rloo | lucasagomes: yup! | 14:21 |
rloo | lucasagomes: I'm off after today (til next Tues). Are there any patches that need reviews soon? | 14:21 |
lucasagomes | :O | 14:22 |
lucasagomes | hmm the J3 ones only | 14:22 |
*** shakamunyi has quit IRC | 14:22 | |
lucasagomes | lemme check if the DHCP one was updated | 14:22 |
rloo | lucasagomes: no, dhcp hsn't yet been updated. will bug jroll later. | 14:22 |
lucasagomes | rloo, not yet... I'm putting the drac one pretty soon | 14:22 |
lucasagomes | like 20 min | 14:22 |
rloo | lucasagomes: ok, will look at some of the other patches then (uefi, swit, blah blah) | 14:23 |
lucasagomes | rloo, ack | 14:23 |
lucasagomes | thanks | 14:23 |
*** shakamunyi has joined #openstack-ironic | 14:23 | |
NobodyCa1 | good morning Ironic, says the man making coffee | 14:25 |
rloo | morning NobodyCa1 | 14:28 |
yuriyz | Ironic 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 |
rloo | NobodyCal? | 14:28 |
lucasagomes | morning NobodyCa1 | 14:29 |
NobodyCa1 | gah | 14:30 |
*** NobodyCa1 is now known as NobodyCam | 14:30 | |
NobodyCam | morning rloo lucasagomes yuriyz | 14:30 |
yuriyz | morning NobodyCam | 14:30 |
rloo | yuriyz: wrt 1362114. pyghmi used to be in requirements, but deva removed it. | 14:32 |
rloo | yuriyz: https://review.openstack.org/#/c/92819/ | 14:33 |
yuriyz | rloo thanks | 14:34 |
*** ifarkas_ has joined #openstack-ironic | 14:36 | |
*** ifarkas has quit IRC | 14:38 | |
*** shakamunyi has quit IRC | 14:40 | |
dtantsur | rloo, NobodyCam, morning | 14:42 |
rloo | hi dtantsur | 14:43 |
NobodyCam | morning dtantsur | 14:44 |
*** yuanying has joined #openstack-ironic | 14:49 | |
*** HenryG has joined #openstack-ironic | 14:51 | |
*** stelfer has joined #openstack-ironic | 14:51 | |
Shrews | romcheg: are 112402 and 112575 the only reviews for the nova bm db migration? want to make sure i hit all of them | 14:54 |
Shrews | adam_g, too ^^^ | 14:54 |
romcheg1 | Shrews: yup | 14:54 |
Shrews | great | 14:54 |
*** todd_dsm has quit IRC | 14:54 | |
*** yuanying has quit IRC | 14:55 | |
*** openstackgerrit has joined #openstack-ironic | 14:55 | |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Fix IO worker tolerance of errors https://review.openstack.org/117248 | 15:01 |
*** mikal has quit IRC | 15:05 | |
*** mikal has joined #openstack-ironic | 15:06 | |
openstackgerrit | A change was merged to openstack/ironic: Updated from global requirements https://review.openstack.org/115600 | 15:17 |
*** igordcard has quit IRC | 15:17 | |
NobodyCam | morning Shrews and romcheg1 :) | 15:18 |
Shrews | NobodyCam: bonjour! | 15:18 |
*** romcheg1 has quit IRC | 15:19 | |
*** dguerri_ has quit IRC | 15:22 | |
*** kincl has quit IRC | 15:22 | |
*** yongli has quit IRC | 15:22 | |
*** tteggel has quit IRC | 15:22 | |
*** kylestev has quit IRC | 15:22 | |
*** romcheg1 has joined #openstack-ironic | 15:22 | |
*** dguerri_ has joined #openstack-ironic | 15:22 | |
*** kincl has joined #openstack-ironic | 15:22 | |
*** yongli has joined #openstack-ironic | 15:22 | |
*** tteggel has joined #openstack-ironic | 15:22 | |
*** kylestev has joined #openstack-ironic | 15:22 | |
*** jcoufal has quit IRC | 15:35 | |
*** keekz has quit IRC | 15:36 | |
*** openstack has joined #openstack-ironic | 16:19 | |
*** eghobo has joined #openstack-ironic | 16:22 | |
*** eghobo has quit IRC | 16:23 | |
*** eghobo has joined #openstack-ironic | 16:23 | |
*** penick has quit IRC | 16:29 | |
openstackgerrit | A change was merged to stackforge/pyghmi: Fix IO worker tolerance of errors https://review.openstack.org/117248 | 16:31 |
*** Hefeweiz1n is now known as Hefeweizin | 16:32 | |
*** penick has joined #openstack-ironic | 16:33 | |
*** derekh has quit IRC | 16:38 | |
*** rushiagr_away is now known as rushiagr | 16:43 | |
*** todd_dsm has joined #openstack-ironic | 16:46 | |
*** krtaylor has quit IRC | 16:47 | |
*** Alexei_987 has quit IRC | 16:48 | |
*** foexle has joined #openstack-ironic | 16:51 | |
*** chuckC has joined #openstack-ironic | 16:51 | |
*** yuanying has joined #openstack-ironic | 16:52 | |
*** yuanying has quit IRC | 16:57 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: DRAC wsman_{enumerate, invoke}() to return an ElementTree object https://review.openstack.org/116871 | 16:58 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Implements the DRAC ManagementInterface for get/set boot device https://review.openstack.org/116913 | 16:58 |
lucasagomes | aight I will call it a day | 17:01 |
lucasagomes | have a great night everyone! | 17:01 |
rloo | night lucasagomes | 17:01 |
romcheg1 | G'night lucasagomes! | 17:01 |
*** athomas has quit IRC | 17:01 | |
lucasagomes | night :) | 17:01 |
*** lucasagomes is now known as lucas-dinner | 17:01 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 17:01 |
NobodyCam | nught lucas-dinner | 17:02 |
jroll | morning ironic :) | 17:02 |
jroll | night lucas-dinner! sure you don't want to review this first? :P ^^ | 17:02 |
jroll | NobodyCam, rloo, I fixed up the dhcp patch | 17:03 |
NobodyCam | morning jroll | 17:03 |
jroll | morning NobodyCam :) | 17:03 |
lucas-dinner | jroll, hey I gotta finish some stuff here, i can review it today later | 17:03 |
rloo | jroll: thx, will look at it soon. | 17:03 |
lucas-dinner | thanks for updating that | 17:03 |
jroll | ha no rush, just letting y'all know, thanks :D | 17:03 |
*** slagle has joined #openstack-ironic | 17:03 | |
*** harlowja has joined #openstack-ironic | 17:04 | |
rloo | jroll: do you want to do some co-author whatever thingy in the commit for the dhcp patch | 17:05 |
jroll | rloo: nah, I just made a couple changes :) | 17:05 |
jroll | also, I +2'd it a few patchsets ago, still happy with the code | 17:05 |
rloo | jroll: ok, fine with me. wanted to make sure in case I +2'd it :D | 17:05 |
jroll | :) | 17:06 |
jroll | rloo: btw, I ran the conductor locally, that try/except works as expected | 17:06 |
openstackgerrit | Kyle Stevenson proposed a change to openstack/ironic-specs: Add ZooKeeper SyncManager spec to Kilo specs https://review.openstack.org/117040 | 17:06 |
rloo | jroll: thx for testing | 17:06 |
jroll | np | 17:07 |
jroll | I would test with devstack/tempest, but too lazy to set up an external dhcp server :P | 17:07 |
jroll | fyi, despite the jenkins failures (issued a recheck), I tested this with devstack and it passes tempest: https://review.openstack.org/#/c/114408/ | 17:11 |
jroll | JayF: ^^ relevant to your interests | 17:12 |
openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add support for interacting with swift https://review.openstack.org/105795 | 17:13 |
*** rloo has quit IRC | 17:16 | |
*** stelfer has quit IRC | 17:19 | |
*** penick has quit IRC | 17:23 | |
*** penick has joined #openstack-ironic | 17:31 | |
*** lazy_prince is now known as killer_prince | 17:47 | |
*** pelix1 has quit IRC | 17:53 | |
*** ramineni has quit IRC | 17:54 | |
*** yuanying has joined #openstack-ironic | 17:54 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Fix unit tests with keystoneclient master https://review.openstack.org/117314 | 17:54 |
jroll | NobodyCam: ^^ that fixes broken unit tests | 17:54 |
NobodyCam | on a conf call atm | 17:55 |
NobodyCam | is our gate broken? | 17:55 |
jroll | no worries | 17:56 |
jroll | no, it's not released yet | 17:56 |
jroll | *my* gate is broken :P | 17:56 |
jroll | writing reproduction steps now | 17:57 |
jroll | are you going to want a bug for that? | 17:57 |
* jroll will likely make one anyway | 17:57 | |
*** foexle has quit IRC | 17:59 | |
*** yuanying has quit IRC | 17:59 | |
*** mikal has quit IRC | 18:00 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Fix unit tests with keystoneclient master https://review.openstack.org/117314 | 18:00 |
jroll | updated with a bug # ^ | 18:01 |
*** mikal has joined #openstack-ironic | 18:05 | |
*** annegent- has joined #openstack-ironic | 18:15 | |
*** rushiagr is now known as rushiagr_away | 18:16 | |
openstackgerrit | A change was merged to openstack/ironic-specs: Update spec theme to show incubation status https://review.openstack.org/116306 | 18:16 |
jroll | Shrews: I'd like to land this today if you want to take a peek as well :) https://review.openstack.org/117314 | 18:17 |
NobodyCam | jroll: +2'd | 18:22 |
NobodyCam | brb...quick run to starbucks :) | 18:22 |
*** rloo has joined #openstack-ironic | 18:24 | |
openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Script to migrate Nova BM data to Ironic https://review.openstack.org/112402 | 18:29 |
openstackgerrit | Andreas Jaeger proposed a change to openstack/ironic-specs: Remove docutils pin https://review.openstack.org/117180 | 18:31 |
Shrews | jroll: will peek | 18:32 |
*** annegent- is now known as annegentle | 18:38 | |
rloo | jroll: wrt 117314, will it break our code too (besides the test)? | 18:39 |
*** zigo has quit IRC | 18:43 | |
Shrews | so, are we getting the "You need to specify either a username or user_id" message b/c of that keystoneclient change? | 18:43 |
* Shrews tests | 18:43 | |
NobodyCam | anyone 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_191 | 18:45 |
*** harlowja has quit IRC | 18:46 | |
*** zigo has joined #openstack-ironic | 18:46 | |
*** harlowja has joined #openstack-ironic | 18:46 | |
Shrews | jroll: rloo: if this breaks our tests, isn't this a backward incompatible change to keystoneclient? | 18:47 |
greghaynes | NobodyCam: There are some recent vlan patches going in tripleo | 18:48 |
greghaynes | lifeless: ^ has been hacking on that a bunch | 18:48 |
rloo | Shrews, 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 |
NobodyCam | greghaynes: Ack Ty for the info :) | 18:49 |
rloo | Shrews, 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 |
Shrews | rloo: yeah. i'm trying to understand the implications of this :/ | 18:50 |
rloo | thx Shrews! I was hoping not to understand it (trying to review some other stuff). | 18:50 |
Shrews | rloo: oh, so you just ask the hard questions and walk away :-P | 18:51 |
Shrews | lol | 18:51 |
rloo | Shrews: 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 |
Shrews | hehe | 18:53 |
*** yuanying has joined #openstack-ironic | 18:55 | |
*** yuanying has quit IRC | 19:01 | |
*** todd_dsm has quit IRC | 19:02 | |
*** krtaylor has joined #openstack-ironic | 19:08 | |
lifeless | greghaynes: speaking of vlans | 19:12 |
lifeless | greghaynes: https://review.openstack.org/#/c/103454/ really needs a second +2/+A | 19:13 |
greghaynes | mmmm | 19:14 |
Shrews | rloo: 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 ok | 19:15 |
rloo | Shrews: does your local devstack have admin_user specified? | 19:16 |
Shrews | rloo: no | 19:16 |
rloo | Shrews: that is interesting. your devstack is using keystone to authenticate, right? | 19:17 |
Shrews | rloo: it's a standard devstack setup. | 19:17 |
Shrews | rloo: nova specifies the username via it's nova.conf | 19:18 |
rloo | Shrews: hmm. oh well. | 19:18 |
rloo | Shrews: Oh, so keystone gets some user-name/id then, in devstack. | 19:18 |
Shrews | [ironic] | 19:19 |
Shrews | admin_username = admin | 19:19 |
Shrews | rloo: right | 19:19 |
rloo | Shrews: 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 that | 19:20 |
Shrews | rloo: yeah, i've always had to add the '--os-username admin' option to ironic client to do anything | 19:21 |
JayF | The change is solely in unit test setups | 19:21 |
JayF | jroll's change | 19:21 |
Shrews | JayF: right, but we were wondering about the bigger implications of the keystoneclient change | 19:21 |
JayF | it should change absolutely nothing about ironic's behavior whatsoever outside of unit tests | 19:21 |
rloo | Shrews: just looked at our client code. it wants username specified, so we're ok there. | 19:21 |
JayF | aha | 19:22 |
JayF | so you guys are just curious if we need moar change | 19:22 |
Shrews | right | 19:22 |
rloo | JayF: yeah. just paranoid is all ;) | 19:22 |
JayF | my suggestion is to land that patch as it sits | 19:22 |
NobodyCam | paranoid is a GODD thing | 19:22 |
JayF | before keystoneclient releases, breaks our gate, and we have a bad time :) | 19:22 |
NobodyCam | good even | 19:22 |
JayF | I'm paranoid they're going to release and break everything | 19:22 |
JayF | lol | 19:22 |
NobodyCam | JayF: add "again" to that line and I'll +2 it | 19:23 |
JayF | I'm paranoid they're going to release and break everything "again" | 19:23 |
NobodyCam | +2 | 19:23 |
Shrews | yeah, i'm fine with jroll's change now | 19:23 |
*** r-daneel has joined #openstack-ironic | 19:24 | |
* Shrews checks jenkins progress | 19:24 | |
rloo | Shrews: I +2'd it but leaving for you to approve ;) | 19:24 |
Shrews | rloo: ok. i'll +A after jenkins is done with it | 19:25 |
rloo | thx Shrews for looking into it. | 19:25 |
Shrews | i love that review.o.o randomly logs me out | 19:25 |
JayF | Shrews: nice to know it isn't just me with that problem | 19:25 |
JayF | and the way logins are handled are a pain if it happens when a dozen r.o.o tabs open | 19:26 |
Shrews | JayF: 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 | |
Shrews | rloo: random log outs in gerrit | 19:28 |
JayF | I don't think it's random | 19:28 |
JayF | I think it waits until I have the maximum number of gerrit tabs open | 19:28 |
JayF | then logs me out | 19:28 |
JayF | malice, not random :P | 19:28 |
Shrews | spitefully intentional? | 19:28 |
NobodyCam | lol | 19:29 |
rloo | hmm, I think I get logged out about once a week or so. Not very frequent anyway. | 19:29 |
NobodyCam | oh it may be that LogOffJayF function I landed last week | 19:29 |
NobodyCam | lol | 19:29 |
NobodyCam | just as a side note. turns out we actually have folks using https://review.openstack.org/#/c/98950 | 19:30 |
rloo | NobodyCam: tripleO folks? | 19:31 |
rloo | NobodyCam: why aren't those folks helping to review that patch? | 19:32 |
NobodyCam | hp folk :) | 19:32 |
lifeless | rloo: they are ops/deployment folk, not self identifying as programmers | 19:32 |
rloo | NobodyCam: Ohhhhh. hp. | 19:32 |
rloo | lifeless: who told them about it? :D | 19:32 |
NobodyCam | :) oh lifeless I'm look at that now | 19:32 |
NobodyCam | fyi | 19:32 |
lifeless | rloo: I wrote it for them | 19:32 |
lifeless | rloo: because they have datacentres without IPMI/iLO accessible (-long- story) | 19:33 |
lifeless | NobodyCam: thank you! | 19:33 |
Shrews | i've been meaning to pick that up and make a spec for it, but stuff gets in the way | 19:34 |
rloo | I was just wondering that. Does it need a spec? if so, it won't get in til kilo. | 19:34 |
Shrews | rloo: deva says "yes" | 19:34 |
Shrews | and i quote, "It's a driver. Drivers need specs" | 19:35 |
rloo | Shrews: ha ha. | 19:35 |
rloo | Shrews: was iboot exempt from specs? | 19:36 |
JayF | I think it predated the spec existing | 19:37 |
JayF | s/the specs/ironic-specs/ | 19:37 |
rloo | JayF: 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 |
NobodyCam | lifeless: was the log power driver planed on landing? | 19:39 |
lifeless | NobodyCam: Conceptually I'd rather have it in tree than unmaintained and floating around | 19:39 |
*** todd_dsm has joined #openstack-ironic | 19:40 | |
NobodyCam | lifeless: ack is "K" cycle ok? | 19:40 |
lifeless | NobodyCam: of course | 19:41 |
NobodyCam | :) | 19:41 |
* jroll is back | 19:42 | |
jroll | thanks for the reviews y'all :) | 19:42 |
rloo | jroll: thx for detecting and fixing! | 19:45 |
jroll | :) | 19:45 |
*** todd_dsm has quit IRC | 19:48 | |
NobodyCam | brb quick walkies | 19:51 |
Shrews | jroll: grrr. 117314 has been in the gate for 2 hours and still queued | 19:53 |
jroll | I know :/ | 19:53 |
jroll | it's ok | 19:53 |
jroll | it's not actually a problem until keystone client gets a release | 19:54 |
Shrews | jroll: i think it's acceptable if you +A yourself it since you have 3 other +2's :) | 19:54 |
jroll | yeah, waiting for jenkins though | 19:54 |
Shrews | once jenkins ok's it | 19:54 |
Shrews | yeah | 19:54 |
jroll | I'll keep an eye out | 19:54 |
*** yuanying has joined #openstack-ironic | 19:57 | |
*** todd_dsm has joined #openstack-ironic | 19:58 | |
Shrews | adam_g: how are you testing the nova bm migration script? | 19:59 |
adam_g | Shrews, grenade | 20:00 |
adam_g | Shrews, if you give me a few i can get you a database dump of a nova-bm db with some stuff in it | 20:01 |
Shrews | adam_g: ok | 20:02 |
*** tzumainn has left #openstack-ironic | 20:10 | |
*** todd_dsm has quit IRC | 20:12 | |
Shrews | So, a quick raid on my fridge has revealed to me that I have no post-work beer available. | 20:18 |
Shrews | This is, without a doubt, the worst thing ever in history | 20:18 |
kylestev | Shrews: I believe there are beer delivery services out there | 20:18 |
Shrews | well, recent history... within my home | 20:18 |
kylestev | depends on where you live, though :P | 20:18 |
Shrews | kylestev: if we had that locally, i would be a hermit, i'm sure | 20:19 |
kylestev | haha | 20:19 |
jroll | Shrews: the good news is that you haven't started drinking post-work beer, so you can still drive to fetch some :P | 20:20 |
Shrews | jroll: the post-beer scotch is calling me... i should leave quickly | 20:21 |
adam_g | Shrews, 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 it | 20:21 |
jroll | Shrews: hehehe :D | 20:21 |
Shrews | adam_g: THE HORROR! | 20:21 |
JayF | I've had IPA in the git repos for a long time | 20:21 |
JayF | and I'm not sure anyone but us has tapped it | 20:21 |
* jroll hangs his head and sighs | 20:21 | |
*** todd_dsm has joined #openstack-ironic | 20:22 | |
adam_g | ha | 20:24 |
*** yuanying has quit IRC | 20:30 | |
openstackgerrit | linggao proposed a change to openstack/ironic: Interactive console support for ipminative driver https://review.openstack.org/97331 | 20:38 |
adam_g | Shrews, http://no-carrier.net/~adam/nova_bm.sql | 20:41 |
* NobodyCam should learn to look at what he is typing :-p | 20:41 | |
Shrews | adam_g: heh, very little data there. i can probably create some random entries | 20:44 |
adam_g | Shrews, 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 API | 20:45 |
*** stevebaker has quit IRC | 20:49 | |
*** stevebaker has joined #openstack-ironic | 20:49 | |
*** stevebaker has joined #openstack-ironic | 20:49 | |
*** stevebaker has joined #openstack-ironic | 20:49 | |
*** stevebaker has joined #openstack-ironic | 20:49 | |
*** stevebaker has joined #openstack-ironic | 20:49 | |
kylestev | stevebaker is a little too eager it seems | 20:50 |
*** stevebaker has joined #openstack-ironic | 20:50 | |
*** stevebaker has joined #openstack-ironic | 20:50 | |
*** stevebaker has quit IRC | 20:50 | |
*** stevebaker has joined #openstack-ironic | 20:50 | |
*** stevebaker has joined #openstack-ironic | 20:50 | |
openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Make DHCP provider pluggable https://review.openstack.org/112351 | 20:52 |
jroll | rloo: ^ fixed pep8 failure | 20:54 |
rloo | jroll: there seem to be tempest failures check-tempest-dsvm-ironic & -postgres. not just for that patch. | 20:55 |
jroll | rloo: yeah, but they seem unrelated | 20:55 |
jroll | oh, not for that patch. great. | 20:56 |
jroll | if our gate is broken I need sharp things | 20:56 |
rloo | jroll: yeah, not just for that patch. I see 'setUpClass failed: Neutron support is required' in tempest logs. | 20:56 |
rloo | jroll: but don't know if that is normal or not. | 20:56 |
jroll | I see a lot of these tempest.scenario.test_dashboard_basic_ops.TestDashboardBasicOps.test_basic_scenario [10.154259s] ... FAILED | 20:56 |
jroll | two of them, to be specific | 20:56 |
*** linggao has quit IRC | 20:57 | |
jroll | oh, yep, I see your thing too | 20:57 |
rloo | jroll: that basic scenario has nothing to do with ironic? | 20:57 |
jroll | those look like skipped tests, though | 20:58 |
jroll | nope, we don't have a dashboard yet :) | 20:58 |
rloo | jroll: 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 |
jroll | heh, ok, have fun :) | 20:58 |
* jroll will brb and then look more | 20:59 | |
Shrews | rloo: jroll: there are horizon issues atm | 20:59 |
jroll | ah ok, thanks | 20:59 |
jroll | they are on top of them? | 20:59 |
Shrews | https://bugs.launchpad.net/horizon/+bug/1362325 | 20:59 |
Shrews | https://review.openstack.org/#/c/117350/ | 20:59 |
Shrews | dunno about an actual fix yet | 21:00 |
*** penick has quit IRC | 21:00 | |
jroll | yeah, ok | 21:00 |
Shrews | you have that in your horizon_error.txt.gz file | 21:00 |
jroll | yep | 21:00 |
jroll | thanks for the info :) | 21:00 |
Shrews | yup | 21:00 |
*** krtaylor has quit IRC | 21:04 | |
*** stevebaker has quit IRC | 21:08 | |
*** stevebaker has joined #openstack-ironic | 21:08 | |
*** foexle has joined #openstack-ironic | 21:16 | |
*** foexle has quit IRC | 21:26 | |
*** yuanying has joined #openstack-ironic | 21:27 | |
jroll | ugh horizon, you're killin me | 21:28 |
*** jasondotstar has quit IRC | 21:28 | |
NobodyCam | it a GUI what did you expect? | 21:32 |
NobodyCam | lol | 21:32 |
NobodyCam | (j/k) | 21:32 |
jroll | heh | 21:32 |
jroll | I poked their channel, they're looking into it | 21:33 |
Shrews | good news. i have secured my post-work beer. everyone can relax now. | 21:36 |
*** zer0c00l1 is now known as zer0c00l | 21:36 | |
jroll | \o/ | 21:37 |
NobodyCam | lol | 21:38 |
*** foexle has joined #openstack-ironic | 21:41 | |
*** romcheg has quit IRC | 21:52 | |
*** vdrok_ has quit IRC | 21:52 | |
*** viktors has quit IRC | 21:52 | |
*** vdrok_ has joined #openstack-ironic | 21:52 | |
*** viktors has joined #openstack-ironic | 21:53 | |
*** romcheg has joined #openstack-ironic | 21:53 | |
*** yuanying has quit IRC | 22:00 | |
rloo | fwiw, 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 |
jroll | I don't see any value in keeping remnants of old failures | 22:11 |
JayF | I do. | 22:12 |
JayF | Especially failures caused by not being in the integrated gate | 22:12 |
JayF | hopefully 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 to | 22:12 |
jroll | ok, I don't see any value in keeping that in the whiteboard | 22:13 |
*** harlowja_ has joined #openstack-ironic | 22:13 | |
rloo | jroll: 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 IRC | 22:15 | |
*** harlowja_ has joined #openstack-ironic | 22:15 | |
jroll | rloo: ok :) | 22:15 |
rloo | hmm, 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 IRC | 22:16 | |
jroll | I mean, that's kind of a given | 22:16 |
rloo | jroll: yeah, except i've seen cases in the past where people do rechecks anyway :-( | 22:17 |
jroll | wow | 22:18 |
JayF | I'm assuming $right_people know about the gate failures? | 22:20 |
rloo | JayF: yeah, someone (jroll?) updated the etherpad with links for the fixes. | 22:22 |
jroll | yeah, fixes inbound | 22:22 |
jroll | someone from infra is ninja-bumping it to the front of the gate, I guess | 22:22 |
jroll | s/it/them/ (them being the *two* patches to fix this) | 22:23 |
adam_g | https://review.openstack.org/#/c/112000 and https://review.openstack.org/#/c/112001 | 22:24 |
NobodyCam | again | 22:25 |
jroll | yah | 22:25 |
jroll | except I think the entire integrated gate might be broken :P | 22:25 |
NobodyCam | crys | 22:27 |
jroll | infra is on it | 22:27 |
NobodyCam | when they land lets just jand everything on our queue | 22:28 |
NobodyCam | lol | 22:28 |
jroll | +1 | 22:28 |
NobodyCam | :-p | 22:28 |
jroll | this is where nova approves our driver and we fail gate for days :P | 22:29 |
NobodyCam | NO DONT EVEN THINK THAT :-p | 22:29 |
jroll | lol | 22:29 |
JayF | I'm Jay Faulkner, and I approved this patch :-D | 22:32 |
JayF | wtf I still have too many gettys respawning | 22:33 |
JayF | WHAT AM I GOING TO DO WITH ALL THESE GETTIES | 22:33 |
*** keekz_ is now known as keekz | 22:35 | |
JayF | wrong channel on that one :) | 22:35 |
* JayF fighting old distributions and serial ports | 22:35 | |
*** lucas-dinner has quit IRC | 22:35 | |
jroll | lol | 22:41 |
*** krtaylor has joined #openstack-ironic | 22:45 | |
*** ZackaryDoyle has joined #openstack-ironic | 22:53 | |
*** yuanying has joined #openstack-ironic | 22:57 | |
NobodyCam | gah, what am I missing | 23:00 |
NobodyCam | to add a driver | 23:00 |
JayF | I don't know | 23:00 |
NobodyCam | add to setup.cfg | 23:00 |
JayF | but I can tell you how to prevent 4 getties from spawning on the same serial terminal in centos 6.5 | 23:00 |
JayF | if that's useful for you | 23:00 |
JayF | lol | 23:00 |
NobodyCam | :-p | 23:01 |
*** yuanying has quit IRC | 23:01 | |
jroll | NobodyCam: python setup.py (develop || install) ? | 23:04 |
jroll | NobodyCam: setup.cfg only gets parsed when you do that | 23:04 |
*** penick has joined #openstack-ironic | 23:06 | |
NobodyCam | i'm actually testing with tox. and I did a rebuild | 23:12 |
jroll | oh, hm | 23:13 |
jroll | enabled_drivers ? | 23:13 |
NobodyCam | test_fake does not fiddle with that and it passes | 23:14 |
NobodyCam | DriverNotFound: Failed to load driver pxe_logpower | 23:14 |
jroll | oh, right | 23:14 |
NobodyCam | I've done something wrong | 23:14 |
jroll | idk | 23:14 |
*** romcheg1 has left #openstack-ironic | 23:22 | |
*** yuanying has joined #openstack-ironic | 23:24 | |
*** foexle has quit IRC | 23:30 | |
*** penick has quit IRC | 23:39 | |
mrda | Morning Ironic | 23:43 |
NobodyCam | morning mrda | 23:49 |
mrda | \o | 23:51 |
openstackgerrit | Kyle Stevenson proposed a change to openstack/ironic: Add implementation of pluggable distributed sync https://review.openstack.org/115148 | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!