*** baoli has joined #openstack-ironic | 00:00 | |
*** jaypipes has quit IRC | 00:02 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Add missing 'autospec' to unit tests - /unit/dhcp/ https://review.openstack.org/509672 | 00:12 |
---|---|---|
*** crushil has quit IRC | 00:18 | |
*** rama_y_ has quit IRC | 00:18 | |
*** crushil has joined #openstack-ironic | 00:25 | |
*** ijw has joined #openstack-ironic | 00:26 | |
*** charcol has quit IRC | 00:52 | |
*** charcol has joined #openstack-ironic | 00:57 | |
*** crushil has quit IRC | 01:15 | |
*** hshiina has joined #openstack-ironic | 01:18 | |
*** ijw has quit IRC | 01:21 | |
*** bnemec-backlater has quit IRC | 01:24 | |
*** bnemec has joined #openstack-ironic | 01:24 | |
*** jlabarre has quit IRC | 01:35 | |
*** phuongnh has joined #openstack-ironic | 01:46 | |
*** charcol has quit IRC | 01:49 | |
*** jhesketh has quit IRC | 01:51 | |
*** jhesketh has joined #openstack-ironic | 01:51 | |
*** cdearborn_ has quit IRC | 01:52 | |
*** charcol has joined #openstack-ironic | 01:53 | |
*** CharlesXavier has quit IRC | 01:58 | |
*** CharlesXavier has joined #openstack-ironic | 01:58 | |
*** IgorYozhikov has quit IRC | 01:59 | |
*** CharlesXavier has quit IRC | 01:59 | |
*** livelace-link has quit IRC | 01:59 | |
*** adreznec has quit IRC | 02:00 | |
*** CharlesXavier has joined #openstack-ironic | 02:01 | |
*** hoonetorg has quit IRC | 02:02 | |
*** afazekas has quit IRC | 02:02 | |
*** afazekas has joined #openstack-ironic | 02:02 | |
*** adreznec has joined #openstack-ironic | 02:02 | |
*** hoonetorg has joined #openstack-ironic | 02:04 | |
*** CharlesXavier has quit IRC | 02:04 | |
*** IgorYozhikov has joined #openstack-ironic | 02:04 | |
*** CharlesXavier has joined #openstack-ironic | 02:09 | |
*** CharlesXavier has quit IRC | 02:12 | |
*** CharlesXavier has joined #openstack-ironic | 02:30 | |
*** vishwanathj has quit IRC | 02:44 | |
*** jkilpatr has quit IRC | 02:46 | |
*** ijw has joined #openstack-ironic | 02:53 | |
*** ijw has quit IRC | 02:57 | |
*** ijw has joined #openstack-ironic | 03:02 | |
*** baoli has quit IRC | 03:02 | |
*** ijw has quit IRC | 03:08 | |
*** ijw has joined #openstack-ironic | 03:11 | |
*** ijw has quit IRC | 03:16 | |
*** baoli has joined #openstack-ironic | 03:17 | |
*** dpawar has joined #openstack-ironic | 03:25 | |
*** baoli has quit IRC | 03:26 | |
*** dpawar has quit IRC | 03:26 | |
*** dpawar has joined #openstack-ironic | 03:26 | |
*** rbudden has quit IRC | 03:27 | |
*** udesale has joined #openstack-ironic | 03:31 | |
*** stendulker has joined #openstack-ironic | 03:33 | |
*** hshiina has quit IRC | 03:37 | |
*** tonyb has quit IRC | 03:42 | |
*** zzzeek has quit IRC | 03:46 | |
*** links has joined #openstack-ironic | 03:48 | |
*** tonyb has joined #openstack-ironic | 03:56 | |
*** hshiina has joined #openstack-ironic | 03:57 | |
*** jdwidari has quit IRC | 04:11 | |
*** sasha2 has quit IRC | 04:36 | |
*** dpawar has quit IRC | 04:37 | |
*** zzzeek has joined #openstack-ironic | 04:38 | |
*** noam_ has joined #openstack-ironic | 04:43 | |
*** stendulker_ has joined #openstack-ironic | 04:45 | |
*** stendulker has quit IRC | 04:49 | |
*** chlong_ has joined #openstack-ironic | 04:51 | |
*** dpawar has joined #openstack-ironic | 05:06 | |
*** jtomasek has joined #openstack-ironic | 05:08 | |
*** marios has joined #openstack-ironic | 05:18 | |
*** ChubYann has quit IRC | 05:41 | |
*** udesale__ has joined #openstack-ironic | 05:42 | |
*** udesale has quit IRC | 05:45 | |
*** mins1 has quit IRC | 05:46 | |
*** aparnav has joined #openstack-ironic | 05:50 | |
*** karimb has joined #openstack-ironic | 05:51 | |
*** vishwanathj has joined #openstack-ironic | 05:54 | |
*** mjura has joined #openstack-ironic | 06:08 | |
*** mjura has quit IRC | 06:08 | |
*** mjura has joined #openstack-ironic | 06:08 | |
*** chlong_ has quit IRC | 06:13 | |
pas-ha | morning ironic :) | 06:19 |
*** vishwana_ has joined #openstack-ironic | 06:19 | |
*** vishwanathj has quit IRC | 06:21 | |
*** karthiks_afk is now known as karthiks | 06:22 | |
stendulker_ | Good morning pas-ha :) | 06:24 |
*** rcernin has joined #openstack-ironic | 06:24 | |
*** vsaienk0 has joined #openstack-ironic | 06:32 | |
*** etingof has quit IRC | 06:33 | |
*** spectr has quit IRC | 06:34 | |
*** spectr has joined #openstack-ironic | 06:34 | |
*** udesale has joined #openstack-ironic | 06:46 | |
*** jaganathan has joined #openstack-ironic | 06:48 | |
*** udesale__ has quit IRC | 06:48 | |
*** vishwana_ has quit IRC | 06:48 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent master: Build tiny disk partitioned image with grub https://review.openstack.org/504888 | 06:59 |
*** etingof has joined #openstack-ironic | 07:00 | |
*** stendulker has joined #openstack-ironic | 07:03 | |
*** vishwanathj has joined #openstack-ironic | 07:04 | |
*** stendulker_ has quit IRC | 07:06 | |
*** tesseract has joined #openstack-ironic | 07:07 | |
*** pcaruana has joined #openstack-ironic | 07:08 | |
*** stendulker has quit IRC | 07:09 | |
*** e0ne has joined #openstack-ironic | 07:34 | |
*** charcol has quit IRC | 07:36 | |
*** lucas-afk is now known as lucasagomes | 07:46 | |
*** vsaienk0 has quit IRC | 07:48 | |
*** vsaienk0 has joined #openstack-ironic | 07:51 | |
*** spectr has quit IRC | 08:09 | |
*** spectr has joined #openstack-ironic | 08:09 | |
*** etingof has quit IRC | 08:13 | |
*** spectr has quit IRC | 08:17 | |
*** spectr has joined #openstack-ironic | 08:18 | |
*** etingof has joined #openstack-ironic | 08:25 | |
*** ralonsoh has joined #openstack-ironic | 08:28 | |
*** trungnv has quit IRC | 08:29 | |
*** trungnv has joined #openstack-ironic | 08:30 | |
*** vsaienk0 has quit IRC | 08:38 | |
*** vsaienk0 has joined #openstack-ironic | 08:41 | |
*** fxpester has joined #openstack-ironic | 08:57 | |
*** namnh has joined #openstack-ironic | 09:04 | |
*** inc0 has quit IRC | 09:13 | |
*** ccamacho has joined #openstack-ironic | 09:18 | |
*** etingof has quit IRC | 09:28 | |
*** gmonteiro has quit IRC | 09:31 | |
*** nicodemos has quit IRC | 09:31 | |
*** CharlesXavier has quit IRC | 09:34 | |
*** mrtenio has quit IRC | 09:34 | |
*** inc0 has joined #openstack-ironic | 09:35 | |
*** etingof has joined #openstack-ironic | 09:40 | |
*** derekh has joined #openstack-ironic | 09:47 | |
*** trungnv has quit IRC | 09:47 | |
*** mrch has joined #openstack-ironic | 09:49 | |
*** milan has joined #openstack-ironic | 09:54 | |
aparnav | hi vsaienk0, yesterday I have provided the details you have asked for the error I was facing on CI. Did you get any chance to see that? | 10:02 |
vsaienk0 | aparnav: I found that in your ironic.conf keystone_authtoken section is completely missing | 10:03 |
aparnav | vsaienk0, please find the ironic.conf file here http://15.184.0.52/ironic.conf. http://paste.openstack.org is accepting only the first 500 lines | 10:05 |
*** amarao has joined #openstack-ironic | 10:26 | |
vsaienk0 | aparnav: the thing looks wrong to me is auth_version = 2 under keystone_authsection, please have a try just remove it | 10:26 |
*** dpawar_ has joined #openstack-ironic | 10:28 | |
*** dpawar has quit IRC | 10:28 | |
*** dpawar has joined #openstack-ironic | 10:33 | |
*** dpawar_ has quit IRC | 10:36 | |
*** udesale has quit IRC | 10:38 | |
*** noam_ has quit IRC | 10:42 | |
*** dtantsur|afk is now known as dtantsur | 10:44 | |
dtantsur | morning ironic | 10:44 |
aparnav | vsaienk0, thank you I will try | 10:46 |
*** spectr has quit IRC | 10:54 | |
vsaienk0 | morning dtantsur | 10:57 |
*** priteau has joined #openstack-ironic | 11:05 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Make starting state non-reentrant https://review.openstack.org/509754 | 11:07 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Ignore race conditions in finish with error https://review.openstack.org/509408 | 11:08 |
milan | morning aparnav, dtantsur, vsaienk0! :) | 11:08 |
milan | and morning Ironic! | 11:08 |
milan | #pixiesay -mflexing insufficient coffee beans | 11:08 |
PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: insufficient coffee beans | 11:08 |
*** jaganathan has quit IRC | 11:10 | |
dtantsur | o/ | 11:11 |
*** noam_ has joined #openstack-ironic | 11:12 | |
*** jkilpatr has joined #openstack-ironic | 11:15 | |
*** lucasagomes is now known as lucas-hungry | 11:17 | |
*** jdwidari has joined #openstack-ironic | 11:19 | |
*** gmonteiro has joined #openstack-ironic | 11:21 | |
*** jkilpatr has quit IRC | 11:24 | |
*** bfournie has quit IRC | 11:32 | |
*** bfournie has joined #openstack-ironic | 11:33 | |
openstackgerrit | Merged openstack/ironic-inspector master: Use ostestr unit test runner https://review.openstack.org/508122 | 11:33 |
*** bfournie has quit IRC | 11:37 | |
*** jkilpatr has joined #openstack-ironic | 11:38 | |
*** marios has quit IRC | 11:38 | |
*** srart has quit IRC | 11:38 | |
*** marios has joined #openstack-ironic | 11:41 | |
*** jdwidari has quit IRC | 11:43 | |
*** ianychoi has quit IRC | 11:46 | |
*** phuongnh has quit IRC | 11:48 | |
*** priteau has quit IRC | 11:53 | |
*** priteau has joined #openstack-ironic | 11:53 | |
*** priteau has quit IRC | 11:54 | |
*** priteau has joined #openstack-ironic | 11:54 | |
*** aparnav has quit IRC | 12:07 | |
*** dprince has joined #openstack-ironic | 12:08 | |
*** bfournie has joined #openstack-ironic | 12:09 | |
*** namnh has quit IRC | 12:10 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/509632 | 12:11 |
*** srart has joined #openstack-ironic | 12:16 | |
*** lucas-hungry is now known as lucasagomes | 12:17 | |
*** vsaienk0 has quit IRC | 12:18 | |
*** jaganathan has joined #openstack-ironic | 12:19 | |
*** vsaienk0 has joined #openstack-ironic | 12:20 | |
*** jdwidari has joined #openstack-ironic | 12:22 | |
*** trown|outtypewww is now known as trown | 12:25 | |
*** jcoufal has joined #openstack-ironic | 12:27 | |
*** nicodemos has joined #openstack-ironic | 12:29 | |
*** vsaienk0 has quit IRC | 12:31 | |
*** vsaienk0 has joined #openstack-ironic | 12:32 | |
*** fellypefca has joined #openstack-ironic | 12:37 | |
fellypefca | Good morning :) | 12:38 |
gmonteiro | Good morning everyone o/ | 12:41 |
*** namnh has joined #openstack-ironic | 12:42 | |
*** namnh has quit IRC | 12:43 | |
*** namnh has joined #openstack-ironic | 12:43 | |
*** links has quit IRC | 12:44 | |
*** jdwidari is now known as plan908 | 12:48 | |
*** plan908 is now known as jdwidari | 12:49 | |
*** catintheroof has joined #openstack-ironic | 12:50 | |
*** hshiina has quit IRC | 12:50 | |
*** catintheroof has quit IRC | 12:51 | |
*** catintheroof has joined #openstack-ironic | 12:51 | |
*** rloo has joined #openstack-ironic | 12:51 | |
*** jdwidari has left #openstack-ironic | 12:57 | |
*** cdearborn has joined #openstack-ironic | 12:57 | |
*** udesale has joined #openstack-ironic | 12:58 | |
*** sasha2 has joined #openstack-ironic | 13:00 | |
*** spectr has joined #openstack-ironic | 13:02 | |
*** Goneri has joined #openstack-ironic | 13:03 | |
*** fellypefca has quit IRC | 13:14 | |
*** namnh has quit IRC | 13:15 | |
*** namnh has joined #openstack-ironic | 13:16 | |
*** milan has quit IRC | 13:20 | |
*** namnh has quit IRC | 13:20 | |
*** baoli has joined #openstack-ironic | 13:21 | |
*** noam_ has quit IRC | 13:22 | |
rloo | hi and good morning all | 13:24 |
rloo | Is anyone looking into the gate failures? | 13:24 |
*** jlabarre has joined #openstack-ironic | 13:24 | |
*** xavierr has joined #openstack-ironic | 13:26 | |
*** rbudden has joined #openstack-ironic | 13:28 | |
dtantsur | morning rloo; we may feel better after the partial revert of zuul v3 | 13:29 |
dtantsur | I haven't checked the most recent status, but I saw some patches passing CI | 13:29 |
dtantsur | essentially, Zuul is non-voting now | 13:29 |
rloo | dtantsur: ok, i looked at our whiteboard and it indicates that there are issues. i saw a bunch of jenkin failures so was wondering. guess it is random then :-( | 13:30 |
dtantsur | it may be.. I'm waiting for the things to settle | 13:31 |
rloo | dtantsur: ok then, i'll do other stuff then :) | 13:31 |
dtantsur | I see jenkins passing on an IPA patch 2 hours ago | 13:31 |
patchbot | No data found for patch 2 | 13:31 |
dtantsur | lol | 13:31 |
dtantsur | actually, zuul passed on it too, modulo one apparently random failure | 13:32 |
rloo | dtantsur: this is the most recent failure i saw: https://review.openstack.org/#/c/353156/ | 13:33 |
patchbot | patch 353156 - ironic - Add rescuewait timeout periodic task | 13:33 |
dtantsur | jenkins has to be fine, these seem random | 13:33 |
dtantsur | the same patch passed jenkins before the recheck | 13:33 |
rloo | dtantsur: yeah, random. but our random test failures could become less random ... so for now, we are recheck'ing? | 13:34 |
dtantsur | rloo: I'd say, recheck important stuff only | 13:36 |
dtantsur | to avoid overloading the CI now that people are trying to repair it | 13:37 |
rloo | dtantsur: gotcha. vdrok isn't around so i'm guessing we want to try to do a client release next week? | 13:37 |
rloo | dtantsur: as opposed to last week/this week? :) | 13:37 |
dtantsur | I'm fine with the next week | 13:38 |
dtantsur | I'm overloaded with relocation stuff too :( | 13:38 |
*** MattMan has quit IRC | 13:38 | |
*** MattMan has joined #openstack-ironic | 13:38 | |
rloo | dtantsur: oh yuck. hopefully it'll be all done soon and then you can have viele bier (sp) | 13:38 |
dtantsur | I'm done with urgent stuff, hopefully | 13:39 |
*** spectr has quit IRC | 13:40 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/509779 | 13:41 |
jlvillal | dtantsur, rloo: I had updated the whiteboard yesterday that we seem to be seeing a very high failure rate on the multinode job | 13:43 |
jlvillal | I do believe there is something wrong. But I don't know what it is. | 13:44 |
jlvillal | Sometimes the job passes. | 13:44 |
rloo | jlvillal: thx for looking into it. is it timing out on something? | 13:44 |
jlvillal | rloo, I'm not sure what is causing it :( | 13:45 |
rloo | jlvillal: sigh. let's keep an eye on it then... | 13:45 |
jlvillal | rloo, My guess is something network related. Sometimes the nodes don't boot from PXE | 13:46 |
*** chlong_ has joined #openstack-ironic | 13:46 | |
jlvillal | rloo, I'm hoping vsaienk0 or vdrok might have an idea :) | 13:47 |
rloo | jlvillal: i think vdrok is on vaca this week. vsaienk0 ^^ if you have any ideas/time to look... | 13:47 |
vsaienk0 | jlvillal: rloo I looked on some failure and saw that IPA can't connect to ironic API when do lookup but didn't investigate more deeply. If failures will occur more I will check more deeply | 13:49 |
jlvillal | vsaienk0, Thanks | 13:49 |
rloo | vsaienk0: +1 thx! I'll update the whiteboard if no one beats me to it ;) | 13:50 |
*** jaypipes has joined #openstack-ironic | 13:52 | |
dtantsur | jlvillal: I've seen it failing too, both on zuul and jenkins | 13:53 |
*** spectr has joined #openstack-ironic | 13:57 | |
*** crushil has joined #openstack-ironic | 13:58 | |
*** srobert has joined #openstack-ironic | 13:58 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Update tests to do not use deprecated test.services() https://review.openstack.org/509788 | 14:01 |
vsaienk0 | dtantsur: rloo jlvillal interested in your opinion here https://review.openstack.org/#/c/506173/ should we introduce a new config option now or not? | 14:02 |
patchbot | patch 506173 - ironic - Change pxe dhcp options name to codes. | 14:02 |
dtantsur | MOAR CONFIG OPTIONZZ! | 14:03 |
*** srobert has quit IRC | 14:03 | |
dtantsur | I hate introducing options, if it can be avoided. sometimes it cannot. | 14:03 |
* dtantsur will take a look | 14:03 | |
*** zzzeek has quit IRC | 14:03 | |
* rloo will look in a few min... | 14:04 | |
* dtantsur will take a look, if his window on the 4th floor does not get hit by a tree | 14:05 | |
dtantsur | windy weather in Berlin today :D | 14:05 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Use oslotest for base test case https://review.openstack.org/508925 | 14:05 |
*** hjensas has quit IRC | 14:06 | |
*** zzzeek has joined #openstack-ironic | 14:07 | |
*** hjensas has joined #openstack-ironic | 14:07 | |
*** hjensas has joined #openstack-ironic | 14:07 | |
*** spectr has quit IRC | 14:07 | |
*** spectr has joined #openstack-ironic | 14:08 | |
*** fellypefca has joined #openstack-ironic | 14:13 | |
*** namnh has joined #openstack-ironic | 14:16 | |
*** jheroux has joined #openstack-ironic | 14:16 | |
etingof | it seems that we do label files at puppet modules rather than having a policy for our stuff in openstack-selinux. what would be the right way to tackle selinux things? | 14:18 |
etingof | EmilienM, ^ | 14:19 |
dtantsur | etingof: I suggest moving to #puppet-openstack | 14:19 |
EmilienM | etingof: it's better to do it in openstack-selinux | 14:19 |
etingof | okay, looks like we do not have anything there yet. or do we? | 14:20 |
EmilienM | selinux things shouldn't be managed in puppet modules | 14:20 |
EmilienM | etingof: you tell me, I don't know | 14:20 |
etingof | +1 | 14:20 |
jlvillal | dtantsur, rloo, vsaienk0: As a note: http://ci-watch.tintri.com/project?project=ironic&time=7+days | 14:22 |
jlvillal | You can see all the failures for: gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial | 14:22 |
* TheJulia semi-re-appears from ghost like status | 14:23 | |
rloo | jlvillal: did I ever say that I liked the colour red?... | 14:23 |
jlvillal | rloo, heh ;) | 14:23 |
rloo | jlvillal: yeah, that doesn't look good. | 14:24 |
* jlvillal decides it is time to get out of bed and go make breakfast | 14:24 | |
* TheJulia wonders if she is in a medication induced alternate reality since rloo is saying that she likes red suddenly. | 14:24 | |
rloo | TheJulia is in an alternate reality :) | 14:24 |
TheJulia | woot \o/ | 14:25 |
rloo | Honestly, I do like red. and green. and blue. but it does depend on where I see the colour. (but i think I don't like pink, regardless of where...) | 14:25 |
TheJulia | I do love purple, but the gray+red cobo is a bit hard on the eyes. Stands out really well though. | 14:25 |
rloo | TheJulia: +1 for purple! | 14:27 |
TheJulia | So, I've been stupidly sick this week. Anything I've missed that is important or that needs my attention? | 14:27 |
rloo | TheJulia: i was away yesterday. Am catching up now. I think the gate issues have caused a slowdown in stuff. | 14:28 |
TheJulia | Yeah, that is kind of what I remember seeing when glancing. | 14:29 |
rloo | TheJulia: if there is anything important/urgent, I'm sure someone will let you know. If you're still not 100%, take it easy, I don't think much will get done this week | 14:31 |
*** trown is now known as trown|brb | 14:34 | |
TheJulia | Far from 100% :( But I have coffee! | 14:35 |
*** etingof has quit IRC | 14:36 | |
dtantsur | hi TheJulia, I hope you feel better | 14:37 |
TheJulia | :) | 14:38 |
TheJulia | <3 all of you | 14:38 |
*** etingof has joined #openstack-ironic | 14:39 | |
rloo | TheJulia: :) | 14:40 |
*** trown|brb is now known as trown | 14:40 | |
dtantsur | ^_^ | 14:41 |
*** ccamacho has quit IRC | 14:45 | |
*** aparnav has joined #openstack-ironic | 14:45 | |
*** etingof has quit IRC | 14:47 | |
NobodyCam | Good Morning Ironic'ers | 14:50 |
*** aparnav has quit IRC | 14:52 | |
dtantsur | morning NobodyCam | 14:52 |
NobodyCam | :) hey hey dtantsur Good Morning :) | 14:53 |
rloo | good morning NobodyCam | 14:54 |
*** jaganathan has quit IRC | 14:54 | |
NobodyCam | Good Morning rloo How are you ding today? | 14:55 |
rloo | NobodyCam: I'm dinging well today. how are you doing? | 14:55 |
*** vsaienk0 has quit IRC | 14:56 | |
*** xavierr has quit IRC | 14:56 | |
rloo | NobodyCam: I've got my dingle hopper handy (http://www.urbandictionary.com/define.php?term=dingle%20hopper) :D | 14:56 |
NobodyCam | doing okay, only on first cup of coffee | 14:56 |
NobodyCam | hehehe :) | 14:57 |
NobodyCam | sounds like its breakfast time :) | 14:58 |
rloo | NobodyCam: enjoy! :) | 14:58 |
*** karthiks has quit IRC | 14:59 | |
NobodyCam | hehehe but I don't have a dingle hopper | 14:59 |
* rloo hands NobodyCam a dingle hopper | 15:00 | |
NobodyCam | hehehehe :) | 15:00 |
*** penick has joined #openstack-ironic | 15:02 | |
*** tuanla_fujitsu has joined #openstack-ironic | 15:05 | |
*** tuanla__ has joined #openstack-ironic | 15:05 | |
*** mat128 has joined #openstack-ironic | 15:05 | |
*** rpioso has joined #openstack-ironic | 15:08 | |
*** radez has joined #openstack-ironic | 15:09 | |
*** rama_y has joined #openstack-ironic | 15:30 | |
*** dpawar has quit IRC | 15:30 | |
*** markguz has joined #openstack-ironic | 15:31 | |
*** mjura has quit IRC | 15:32 | |
*** dpawar has joined #openstack-ironic | 15:35 | |
*** markguz_ has joined #openstack-ironic | 15:39 | |
*** srobert has joined #openstack-ironic | 15:39 | |
*** pcaruana has quit IRC | 15:39 | |
*** srobert has quit IRC | 15:40 | |
*** racedo_ has quit IRC | 15:40 | |
*** markguz has quit IRC | 15:42 | |
rpioso | dtantsur: Regarding https://review.openstack.org/#/c/507942/, that fixes a bug that was introduced when the fix to https://bugs.launchpad.net/ironic/+bug/1691808 was reverted. That was reverted, because a bug fix to python-dracclient more broadly applied the fix; a Dell Lifecycle Controller best practice was adopted that fixes a number of bugs that had been observed. The surgical fix was no longer needed. | 15:45 |
openstack | Launchpad bug 1691808 in Ironic "Manual cleaning fails when using iDRAC driver" [High,Fix released] - Assigned to Christopher Dearborn (cdearbor) | 15:45 |
patchbot | patch 507942 - ironic - python-dracclient driver version update | 15:45 |
rpioso | dtantsur: The change that reverted https://bugs.launchpad.net/ironic/+bug/1691808 is https://review.openstack.org/#/c/481318/. Note that driver-requirements.txt was changed to bump the version of python-dracclient. | 15:47 |
openstack | Launchpad bug 1691808 in Ironic "Manual cleaning fails when using iDRAC driver" [High,Fix released] - Assigned to Christopher Dearborn (cdearbor) | 15:47 |
patchbot | patch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED) | 15:47 |
*** srobert has joined #openstack-ironic | 15:47 | |
dtantsur | rpioso: bumping a driver version does not fix a bug, a new release of dracclient does. | 15:48 |
rpioso | dtantsur: While the implementation of the Lifecycle Controller best practice appears to be solid on 14G servers, it is not on 13G servers. 13G servers consistently fail to deploy. | 15:48 |
rpioso | dtantsur: A new release of python-dracclient was created before the ironic bug fix was proposed. | 15:48 |
rpioso | dtantsur: The new release of python-dracclient's has the tag 1.3.1. | 15:49 |
dtantsur | essentially, I don't like people bumping requirements for bug fixes that are purely contained in their driver libraries | 15:49 |
dtantsur | in addition to being incorrect, this also makes patches not backportable | 15:49 |
rpioso | dtantsur: But, deployment of supported hardware consistently fails. | 15:50 |
dtantsur | like, you fix it for Queens now, but the same problem also hits Pike, right? and you cannot apply this change to pike | 15:50 |
dtantsur | rpioso: right, and we have fixed it, no? it's fixed in dracclient 1.3.1, right? | 15:50 |
rpioso | dtantsur: Yes, it's also broken in Pike. | 15:50 |
* TheJulia opens the pike driver-reqs | 15:51 | |
*** spectr has quit IRC | 15:51 | |
rpioso | dtantsur: This doesn't seem much different from ironic change https://review.openstack.org/#/c/481318/. | 15:51 |
patchbot | patch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED) | 15:51 |
rpioso | dtantsur: The only difference is that a prior surgical fix in ironic proper was reverted, because a new release of python-dracclient made it redundant and confusing. | 15:52 |
dtantsur | rpioso: I'm sorry, I'm losing the thread of this conversation.. what exactly do you propose? | 15:52 |
TheJulia | https://github.com/openstack/ironic/blob/stable/pike/driver-requirements.txt#L13 <- == any version >= version 1.3.0, which m eans 1.3.1 should be picked up if not present. Seems like the library needs to just to be upgraded locally, or uninstalled and re-installed | 15:52 |
*** namnh has quit IRC | 15:52 | |
*** namnh has joined #openstack-ironic | 15:53 | |
*** amarao has quit IRC | 15:54 | |
rpioso | dtantsur, TheJuila: The equal sign is an issue. | 15:54 |
dtantsur | rpioso: sorry, where? I see >=, not == | 15:54 |
rpioso | dtantsur, TheJulia: If it's equal to 1.3.0, 13G servers will not consistently deploy. | 15:55 |
rpioso | dtantsur, TheJulia: What am I missing? | 15:55 |
dtantsur | that's why people have to upgrade dracclient, no? | 15:55 |
TheJulia | yes, but pip will always try to meet the requirements, but it is aware of upgrading versions with the range | 15:55 |
TheJulia | if the requirement is satisfied, it does nothing. | 15:55 |
dtantsur | I just don't get what you try to achieve by bumping driver-requirements. This does not do what you think it does. | 15:55 |
dtantsur | Well, first it does nothing for many people. E.g. RDO does not sync with it, neither does RHOSP. | 15:56 |
TheJulia | which means, if 1.3.0 is present, then it is broken. If they explicitly upgrade to 1.3.1, then they will get it. If 1.3.1 is available in the universe, it should be installed before 1.3.0. | 15:56 |
*** hshiina has joined #openstack-ironic | 15:56 | |
rpioso | dtantsur: To be >= 1.3.1. | 15:56 |
TheJulia | rpioso: you want to change the minimum base version, correct? | 15:57 |
dtantsur | requirements are bumped when we introduce an explicit dependency. like, using a new function argument or a new function | 15:57 |
*** dpawar has quit IRC | 15:57 | |
*** mrtenio has joined #openstack-ironic | 15:57 | |
dtantsur | when you use dracclient.foo() which was introduced in 1.4.0, you cannot say that you require >=1.3.0 | 15:57 |
*** namnh has quit IRC | 15:57 | |
dtantsur | however, when there is a bug, we expect operators/distributions to pick fixes without our intervention | 15:57 |
dtantsur | we simply cannot track all bug fixes on all stable branches in a way you expect us to | 15:58 |
dtantsur | according to openstack policies, changing the minimum requirement counts as a feature, so it cannot be applied to stable branches as well | 15:58 |
dtantsur | TheJulia: are we having a BFV meeting now? | 15:59 |
TheJulia | ^--- Was just going to bring that up | 15:59 |
*** baoli has quit IRC | 15:59 | |
TheJulia | dtantsur: my brain is quite literally scrambled | 15:59 |
TheJulia | I don't see a mjturek... | 15:59 |
rpioso | dtantsur: So this is different from what was done for change https://review.openstack.org/#/c/481318/, right? | 15:59 |
patchbot | patch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED) | 15:59 |
dtantsur | TheJulia: I can chair it if needed, but I don't have anything on agenda | 15:59 |
TheJulia | hshiina is around, I guess we could hold a BFV meeting | 15:59 |
*** baoli has joined #openstack-ironic | 16:00 | |
hshiina | TheJulia, hi | 16:00 |
TheJulia | dtantsur: It could be a very quick meeting "anyone have anything to discuss?" "no" "#endmeeting" | 16:00 |
dtantsur | rpioso: I think the bump was an oversight from us | 16:00 |
rpioso | dtantsur, TheJulia: Okay. Then I'm good. | 16:01 |
rpioso | dtantsur, TheJulia: Thank you for clarifying. | 16:01 |
dtantsur | you're always welcome | 16:02 |
rpioso | dtantsur: :) | 16:02 |
*** markguz_ has quit IRC | 16:03 | |
*** markguz has joined #openstack-ironic | 16:03 | |
*** dprince has quit IRC | 16:03 | |
*** dprince has joined #openstack-ironic | 16:05 | |
*** markguz has quit IRC | 16:08 | |
*** xavierr has joined #openstack-ironic | 16:08 | |
*** rcernin has quit IRC | 16:13 | |
* dtantsur does not get 47 minutes back, as he has another meeting in parallel | 16:13 | |
*** rcernin has joined #openstack-ironic | 16:13 | |
*** rcernin has quit IRC | 16:14 | |
*** marios has quit IRC | 16:15 | |
*** e0ne has quit IRC | 16:17 | |
*** aparnav has joined #openstack-ironic | 16:18 | |
TheJulia | doh! :( | 16:19 |
TheJulia | well, don't worry, I'm discussing project update stuff with hshiina so we can start getting the slide decks together for sydney ;) | 16:19 |
TheJulia | so, we didn't get it back either. :) | 16:20 |
dtantsur | heh | 16:20 |
dtantsur | TheJulia: have you got milan's slides? | 16:20 |
openstackgerrit | Pierre Riteau proposed openstack/ironic stable/ocata: Skip PortNotFound when unbinding port https://review.openstack.org/505367 | 16:21 |
TheJulia | dtantsur: so, funny thing is he sent it to my openstack centric gmail address as a drive attached document. So, I can't access it due to some weird constraint that only I seem to have related to multiple sign-ins. | 16:22 |
dtantsur | ugh :( | 16:22 |
TheJulia | dtantsur: I asked him to send it over to my personal address where I'll be able to open it | 16:22 |
*** namnh has joined #openstack-ironic | 16:22 | |
*** xavierr has quit IRC | 16:23 | |
*** srobert_ has joined #openstack-ironic | 16:23 | |
dtantsur | okay, I see | 16:25 |
hshiina | good night, everyone | 16:25 |
*** srobert has quit IRC | 16:26 | |
*** etingof has joined #openstack-ironic | 16:26 | |
*** hshiina has quit IRC | 16:27 | |
*** tesseract has quit IRC | 16:29 | |
*** ralonsoh has quit IRC | 16:31 | |
rloo | dtantsur, TheJulia, others: for the new rescue_interface, are classic drivers going to support it, or only hardware types? | 16:35 |
dtantsur | rloo: I think we decidd that classic drivers will have no-rescue hardcoded.. | 16:36 |
openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes https://review.openstack.org/473957 | 16:36 |
rloo | dtantsur: ok | 16:37 |
*** baoli has quit IRC | 16:39 | |
*** trown is now known as trown|lunch | 16:40 | |
*** baoli has joined #openstack-ironic | 16:44 | |
*** etingof has quit IRC | 16:49 | |
*** udesale has quit IRC | 16:49 | |
*** tesseract has joined #openstack-ironic | 16:50 | |
*** derekh has quit IRC | 16:51 | |
TheJulia | dtantsur: That is what I remember as well | 16:54 |
rloo | thx TheJulia. I've added a comment, to get the spec updated with that. | 16:55 |
TheJulia | rloo: thanks! | 16:55 |
openstackgerrit | Merged openstack/bifrost stable/ocata: scripts: test-bifrost: Start MySQL service before configuring the DB https://review.openstack.org/506138 | 16:57 |
openstackgerrit | Merged openstack/bifrost stable/newton: scripts: test-bifrost: Start MySQL service before configuring the DB https://review.openstack.org/506139 | 16:58 |
*** srobert_ has quit IRC | 17:02 | |
*** srobert has joined #openstack-ironic | 17:03 | |
*** etingof has joined #openstack-ironic | 17:03 | |
*** tuanla_fujitsu has quit IRC | 17:05 | |
*** tuanla__ has quit IRC | 17:05 | |
*** ijw has joined #openstack-ironic | 17:07 | |
rloo | TheJulia, dtantsur, jlvillal: any of you going to be around the next few days? This can't land before the rolling upgrades stuff, but I'm going to be away til Tues so I don't want to -2 it: https://review.openstack.org/#/c/509335/ | 17:07 |
patchbot | patch 509335 - ironic - Adds rescue_interface to base driver class | 17:07 |
dtantsur | rloo: I'm around, yes | 17:08 |
TheJulia | I'll be around | 17:08 |
rloo | dtantsur, TheJulia: thx, first one gets the honours :) | 17:09 |
dtantsur | I'm not sure what you want us to do though :) just watch it? ask the author to rebase on top of some other patch? | 17:09 |
*** srobert has quit IRC | 17:09 | |
TheJulia | rebase is likely the best course of action | 17:10 |
*** srobert has joined #openstack-ironic | 17:10 | |
rloo | dtantsur, TheJulia: -2 until it is rebased? | 17:10 |
rloo | dtantsur, TheJulia: i just don't wnat it to land before the other stuff | 17:10 |
rloo | dtantsur, TheJulia: even if we do nothing, I am doubtful that it will land. but procedurally, just in case... | 17:11 |
*** lucasagomes is now known as lucas-afk | 17:11 | |
dtantsur | I'm not the biggest fan of -2 until something is fixed for the very reason you mention: that requires me to take a look at the patch every time it is updated.. | 17:12 |
openstackgerrit | Merged openstack/python-ironicclient master: Update README https://review.openstack.org/507898 | 17:12 |
dtantsur | rloo: I think your -1 with a firm "please do" will be enough | 17:12 |
rloo | dtantsur, TheJulia: ok, will -1 then. I had been reserving that for when i actually reviewed it, but that is fine :-) | 17:13 |
openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/509632 | 17:16 |
dtantsur | okay folks, time to go :) see you tomorrow, see you on Tue rloo | 17:20 |
rloo | bye dtantsur! | 17:20 |
* dtantsur means a virtual "go", as he work from home, and the outside is damaged by a storm to a serious extent | 17:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/509779 | 17:22 |
openstackgerrit | Sam Betts proposed openstack/ironic master: Separate baremetal simulation from devstack-plugin https://review.openstack.org/509844 | 17:22 |
*** tesseract has quit IRC | 17:22 | |
NobodyCam | dtantsur: oh wow hope all is okay there. | 17:22 |
NobodyCam | have a good night! | 17:22 |
dtantsur | I'm fine, but overall the things could be better: https://www.thelocal.de/20171005/storm-xavier-brings-train-services-in-much-of-north-germany-to-a-stop | 17:23 |
TheJulia | wow | 17:24 |
sambetts|afk | night al | 17:27 |
rloo | stay safe dtantsur. | 17:28 |
rloo | good night sambetts|afk | 17:28 |
NobodyCam | wow | 17:31 |
NobodyCam | night sambetts|afk | 17:31 |
*** dtantsur is now known as dtantsur|afk | 17:32 | |
NobodyCam | be safe dtantsur|afk | 17:32 |
*** penick has quit IRC | 17:32 | |
*** gmonteiro has quit IRC | 17:33 | |
*** aparnav has quit IRC | 17:35 | |
*** e0ne has joined #openstack-ironic | 17:35 | |
*** trown|lunch is now known as trown | 17:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/509779 | 17:38 |
*** namnh has quit IRC | 17:38 | |
*** penick has joined #openstack-ironic | 17:39 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-ironicclient master: Do not use urljoin in base http client https://review.openstack.org/509851 | 17:41 |
*** e0ne has quit IRC | 17:42 | |
openstackgerrit | Merged openstack/bifrost master: Use 'none' auth plugin w/o keystone https://review.openstack.org/499085 | 17:48 |
*** aparnav has joined #openstack-ironic | 17:53 | |
*** noam_ has joined #openstack-ironic | 17:55 | |
*** hamzy has quit IRC | 17:57 | |
openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Deprecate the ironic CLI https://review.openstack.org/508218 | 18:04 |
*** ijw has quit IRC | 18:16 | |
*** e0ne has joined #openstack-ironic | 18:18 | |
*** e0ne has quit IRC | 18:23 | |
*** sambetts|afk has quit IRC | 18:33 | |
*** sambetts_ has joined #openstack-ironic | 18:36 | |
*** ijw has joined #openstack-ironic | 18:39 | |
*** ChubYann has joined #openstack-ironic | 18:40 | |
*** ijw has quit IRC | 18:41 | |
*** hamzy has joined #openstack-ironic | 18:41 | |
*** chlong_ has quit IRC | 18:44 | |
openstackgerrit | Merged openstack/ironic master: Use oslotest for base test case https://review.openstack.org/508925 | 18:44 |
openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver https://review.openstack.org/507632 | 18:49 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/509779 | 18:51 |
*** markguz has joined #openstack-ironic | 18:57 | |
*** chlong_ has joined #openstack-ironic | 19:00 | |
*** baoli has quit IRC | 19:02 | |
*** baoli has joined #openstack-ironic | 19:02 | |
*** fellypefca has quit IRC | 19:07 | |
TheJulia | johnthetubaguy: You around? | 19:21 |
rloo | oh, look at that, is it our first merge in daze? ^^ 508925? | 19:34 |
TheJulia | woot | 19:35 |
openstackgerrit | Xavier proposed openstack/ironic stable/newton: [DNM|WIP] Fix agent deploy on OneView https://review.openstack.org/509431 | 19:41 |
*** aparnav has quit IRC | 19:43 | |
*** chlong_ has quit IRC | 19:44 | |
*** baoli has quit IRC | 19:48 | |
rloo | TheJulia: wrt bfv, I'm a bit confused, should StorageInterface.detach_volumes() have connector & aborting_attach parameters? | 19:49 |
rloo | TheJulia: https://github.com/openstack/ironic/blob/4c82308ebed809352674959a3892f8b3bac01d1b/ironic/drivers/base.py#L1093 | 19:49 |
rloo | TheJulia: cuz cinder.py has: https://github.com/openstack/ironic/blob/05922a4098a42add937b098b9defafb7dd8b3943/ironic/drivers/modules/storage/cinder.py#L281 | 19:49 |
*** baoli has joined #openstack-ironic | 19:49 | |
rloo | TheJulia: but noop.py doesn't? https://github.com/openstack/ironic/blob/c491e93a082ede262e617cebfd3022090e3e715b/ironic/drivers/modules/storage/noop.py#L28 | 19:50 |
rloo | TheJulia: nor does FakeStorage. | 19:51 |
* TheJulia context switches | 19:55 | |
openstackgerrit | Yusef Shaban proposed openstack/sushy master: Implement Redfish Sessions https://review.openstack.org/471942 | 19:57 |
*** srobert has quit IRC | 19:57 | |
*** ianw|pto is now known as ianw | 20:00 | |
*** jkilpatr has quit IRC | 20:03 | |
TheJulia | rloo: I don't think so, at least off hand since they are only used to back out of a failure case which is just how that code ended up for that driver. The parent class doesn't necessarily need them nor do we need to prescribe them to any other interfaces because their mechanics may differ. | 20:05 |
*** jlabarre has quit IRC | 20:05 | |
rloo | TheJulia: but that's the interface API. | 20:05 |
TheJulia | true, and the only place it is used inside attach_ to cleanup after something going sideways | 20:06 |
TheJulia | I mean, we could refactor, put a wrapper in and hide the two extra fields that are used for that as a private method. The wrapper would then just pass just the task down to the private method | 20:07 |
rloo | TheJulia: I didn't look at the code, but maybe you could put that info in some internal* field. | 20:07 |
rloo | TheJulia: or refactor. Again, I'm not familiar with the code. But I think the API has to be the same... | 20:07 |
TheJulia | runtime-ish data and db writes are costly :\ | 20:08 |
rloo | TheJulia: yup. and we do it. whatever works... | 20:09 |
TheJulia | I think it is more style because I think I've nova is where some overriden classes methods begin to gain some additional args | 20:10 |
TheJulia | speaking of nova... sticking brain back into the networking code of it | 20:12 |
*** jcoufal has quit IRC | 20:18 | |
openstackgerrit | Stenio Araujo proposed openstack/ironic stable/newton: [DNM|WIP] Fix agent deploy on OneView https://review.openstack.org/509431 | 20:19 |
*** jtomasek has quit IRC | 20:20 | |
*** jkilpatr has joined #openstack-ironic | 20:22 | |
TheJulia | johnthetubaguy: I've been talking to markguz regarding some networking issues he has been having, and his use case is a little different when initiated through nova. I'd love to get a little bit of your time to discuss it tomorrow because I feel like there are/were two bugs. | 20:22 |
*** mat128 has quit IRC | 20:35 | |
*** priteau has quit IRC | 20:49 | |
*** priteau has joined #openstack-ironic | 20:50 | |
*** hamzy has quit IRC | 20:52 | |
*** trown is now known as trown|outtypewww | 20:56 | |
*** catintheroof has quit IRC | 20:56 | |
*** priteau has quit IRC | 20:59 | |
*** crushil has quit IRC | 21:06 | |
*** rajinir is now known as rajinir-afk | 21:11 | |
*** jheroux has quit IRC | 21:11 | |
*** Goneri has quit IRC | 21:19 | |
*** crushil has joined #openstack-ironic | 21:19 | |
*** baoli has quit IRC | 21:19 | |
*** rcernin has joined #openstack-ironic | 21:22 | |
*** dprince has quit IRC | 21:23 | |
*** mrtenio has quit IRC | 21:23 | |
*** crushil has quit IRC | 21:24 | |
openstackgerrit | Merged openstack/ironic master: Stop passing raw Exceptions as the reasons for ironic Image exceptions https://review.openstack.org/508225 | 21:26 |
jlvillal | Woo hoo! Something merged :) | 21:27 |
*** bfournie has quit IRC | 21:30 | |
*** bfournie has joined #openstack-ironic | 21:30 | |
*** bfournie has quit IRC | 21:34 | |
*** noam_ has quit IRC | 21:42 | |
*** markguz_ has joined #openstack-ironic | 21:47 | |
*** cdearborn has quit IRC | 21:51 | |
*** charcol has joined #openstack-ironic | 21:52 | |
*** rcernin has quit IRC | 22:06 | |
*** baoli has joined #openstack-ironic | 22:08 | |
*** baoli has quit IRC | 22:12 | |
*** baoli has joined #openstack-ironic | 22:12 | |
*** rpioso has quit IRC | 22:25 | |
openstackgerrit | Merged openstack/bifrost stable/newton: Fix the dhcp checks on centos https://review.openstack.org/500487 | 22:38 |
openstackgerrit | Merged openstack/bifrost master: scripts: install-deps.sh: Set noninteractive for DEBIAN_FRONTEND https://review.openstack.org/509430 | 22:44 |
*** penick has quit IRC | 22:54 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/pike: Remove hard-coded path to grub binaries https://review.openstack.org/507468 | 23:04 |
TheJulia | \o/ | 23:07 |
*** karimb has quit IRC | 23:09 | |
*** rloo has quit IRC | 23:11 | |
*** baoli has quit IRC | 23:24 | |
*** namnh has joined #openstack-ironic | 23:28 | |
*** markguz_ has quit IRC | 23:33 | |
*** hamzy has joined #openstack-ironic | 23:43 | |
*** jlabarre has joined #openstack-ironic | 23:55 | |
*** dyasny has quit IRC | 23:59 | |
*** charcol has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!