Thursday, 2017-10-05

*** baoli has joined #openstack-ironic00:00
*** jaypipes has quit IRC00:02
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Add missing 'autospec' to unit tests - /unit/dhcp/  https://review.openstack.org/50967200:12
*** crushil has quit IRC00:18
*** rama_y_ has quit IRC00:18
*** crushil has joined #openstack-ironic00:25
*** ijw has joined #openstack-ironic00:26
*** charcol has quit IRC00:52
*** charcol has joined #openstack-ironic00:57
*** crushil has quit IRC01:15
*** hshiina has joined #openstack-ironic01:18
*** ijw has quit IRC01:21
*** bnemec-backlater has quit IRC01:24
*** bnemec has joined #openstack-ironic01:24
*** jlabarre has quit IRC01:35
*** phuongnh has joined #openstack-ironic01:46
*** charcol has quit IRC01:49
*** jhesketh has quit IRC01:51
*** jhesketh has joined #openstack-ironic01:51
*** cdearborn_ has quit IRC01:52
*** charcol has joined #openstack-ironic01:53
*** CharlesXavier has quit IRC01:58
*** CharlesXavier has joined #openstack-ironic01:58
*** IgorYozhikov has quit IRC01:59
*** CharlesXavier has quit IRC01:59
*** livelace-link has quit IRC01:59
*** adreznec has quit IRC02:00
*** CharlesXavier has joined #openstack-ironic02:01
*** hoonetorg has quit IRC02:02
*** afazekas has quit IRC02:02
*** afazekas has joined #openstack-ironic02:02
*** adreznec has joined #openstack-ironic02:02
*** hoonetorg has joined #openstack-ironic02:04
*** CharlesXavier has quit IRC02:04
*** IgorYozhikov has joined #openstack-ironic02:04
*** CharlesXavier has joined #openstack-ironic02:09
*** CharlesXavier has quit IRC02:12
*** CharlesXavier has joined #openstack-ironic02:30
*** vishwanathj has quit IRC02:44
*** jkilpatr has quit IRC02:46
*** ijw has joined #openstack-ironic02:53
*** ijw has quit IRC02:57
*** ijw has joined #openstack-ironic03:02
*** baoli has quit IRC03:02
*** ijw has quit IRC03:08
*** ijw has joined #openstack-ironic03:11
*** ijw has quit IRC03:16
*** baoli has joined #openstack-ironic03:17
*** dpawar has joined #openstack-ironic03:25
*** baoli has quit IRC03:26
*** dpawar has quit IRC03:26
*** dpawar has joined #openstack-ironic03:26
*** rbudden has quit IRC03:27
*** udesale has joined #openstack-ironic03:31
*** stendulker has joined #openstack-ironic03:33
*** hshiina has quit IRC03:37
*** tonyb has quit IRC03:42
*** zzzeek has quit IRC03:46
*** links has joined #openstack-ironic03:48
*** tonyb has joined #openstack-ironic03:56
*** hshiina has joined #openstack-ironic03:57
*** jdwidari has quit IRC04:11
*** sasha2 has quit IRC04:36
*** dpawar has quit IRC04:37
*** zzzeek has joined #openstack-ironic04:38
*** noam_ has joined #openstack-ironic04:43
*** stendulker_ has joined #openstack-ironic04:45
*** stendulker has quit IRC04:49
*** chlong_ has joined #openstack-ironic04:51
*** dpawar has joined #openstack-ironic05:06
*** jtomasek has joined #openstack-ironic05:08
*** marios has joined #openstack-ironic05:18
*** ChubYann has quit IRC05:41
*** udesale__ has joined #openstack-ironic05:42
*** udesale has quit IRC05:45
*** mins1 has quit IRC05:46
*** aparnav has joined #openstack-ironic05:50
*** karimb has joined #openstack-ironic05:51
*** vishwanathj has joined #openstack-ironic05:54
*** mjura has joined #openstack-ironic06:08
*** mjura has quit IRC06:08
*** mjura has joined #openstack-ironic06:08
*** chlong_ has quit IRC06:13
pas-hamorning ironic :)06:19
*** vishwana_ has joined #openstack-ironic06:19
*** vishwanathj has quit IRC06:21
*** karthiks_afk is now known as karthiks06:22
stendulker_Good morning pas-ha :)06:24
*** rcernin has joined #openstack-ironic06:24
*** vsaienk0 has joined #openstack-ironic06:32
*** etingof has quit IRC06:33
*** spectr has quit IRC06:34
*** spectr has joined #openstack-ironic06:34
*** udesale has joined #openstack-ironic06:46
*** jaganathan has joined #openstack-ironic06:48
*** udesale__ has quit IRC06:48
*** vishwana_ has quit IRC06:48
openstackgerritVasyl Saienko proposed openstack/ironic-python-agent master: Build tiny disk partitioned image with grub  https://review.openstack.org/50488806:59
*** etingof has joined #openstack-ironic07:00
*** stendulker has joined #openstack-ironic07:03
*** vishwanathj has joined #openstack-ironic07:04
*** stendulker_ has quit IRC07:06
*** tesseract has joined #openstack-ironic07:07
*** pcaruana has joined #openstack-ironic07:08
*** stendulker has quit IRC07:09
*** e0ne has joined #openstack-ironic07:34
*** charcol has quit IRC07:36
*** lucas-afk is now known as lucasagomes07:46
*** vsaienk0 has quit IRC07:48
*** vsaienk0 has joined #openstack-ironic07:51
*** spectr has quit IRC08:09
*** spectr has joined #openstack-ironic08:09
*** etingof has quit IRC08:13
*** spectr has quit IRC08:17
*** spectr has joined #openstack-ironic08:18
*** etingof has joined #openstack-ironic08:25
*** ralonsoh has joined #openstack-ironic08:28
*** trungnv has quit IRC08:29
*** trungnv has joined #openstack-ironic08:30
*** vsaienk0 has quit IRC08:38
*** vsaienk0 has joined #openstack-ironic08:41
*** fxpester has joined #openstack-ironic08:57
*** namnh has joined #openstack-ironic09:04
*** inc0 has quit IRC09:13
*** ccamacho has joined #openstack-ironic09:18
*** etingof has quit IRC09:28
*** gmonteiro has quit IRC09:31
*** nicodemos has quit IRC09:31
*** CharlesXavier has quit IRC09:34
*** mrtenio has quit IRC09:34
*** inc0 has joined #openstack-ironic09:35
*** etingof has joined #openstack-ironic09:40
*** derekh has joined #openstack-ironic09:47
*** trungnv has quit IRC09:47
*** mrch has joined #openstack-ironic09:49
*** milan has joined #openstack-ironic09:54
aparnavhi 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
vsaienk0aparnav: I found that in your ironic.conf keystone_authtoken section is completely missing10:03
aparnavvsaienk0, please find the ironic.conf file here http://15.184.0.52/ironic.conf.  http://paste.openstack.org is accepting only the first 500 lines10:05
*** amarao has joined #openstack-ironic10:26
vsaienk0aparnav: the thing looks wrong to me is auth_version = 2 under keystone_authsection, please have a try just remove it10:26
*** dpawar_ has joined #openstack-ironic10:28
*** dpawar has quit IRC10:28
*** dpawar has joined #openstack-ironic10:33
*** dpawar_ has quit IRC10:36
*** udesale has quit IRC10:38
*** noam_ has quit IRC10:42
*** dtantsur|afk is now known as dtantsur10:44
dtantsurmorning ironic10:44
aparnavvsaienk0, thank you I will try10:46
*** spectr has quit IRC10:54
vsaienk0morning dtantsur10:57
*** priteau has joined #openstack-ironic11:05
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Make starting state non-reentrant  https://review.openstack.org/50975411:07
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Ignore race conditions in finish with error  https://review.openstack.org/50940811:08
milanmorning aparnav, dtantsur, vsaienk0! :)11:08
milanand morning Ironic!11:08
milan#pixiesay -mflexing insufficient coffee beans11:08
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: insufficient coffee beans11:08
*** jaganathan has quit IRC11:10
dtantsuro/11:11
*** noam_ has joined #openstack-ironic11:12
*** jkilpatr has joined #openstack-ironic11:15
*** lucasagomes is now known as lucas-hungry11:17
*** jdwidari has joined #openstack-ironic11:19
*** gmonteiro has joined #openstack-ironic11:21
*** jkilpatr has quit IRC11:24
*** bfournie has quit IRC11:32
*** bfournie has joined #openstack-ironic11:33
openstackgerritMerged openstack/ironic-inspector master: Use ostestr unit test runner  https://review.openstack.org/50812211:33
*** bfournie has quit IRC11:37
*** jkilpatr has joined #openstack-ironic11:38
*** marios has quit IRC11:38
*** srart has quit IRC11:38
*** marios has joined #openstack-ironic11:41
*** jdwidari has quit IRC11:43
*** ianychoi has quit IRC11:46
*** phuongnh has quit IRC11:48
*** priteau has quit IRC11:53
*** priteau has joined #openstack-ironic11:53
*** priteau has quit IRC11:54
*** priteau has joined #openstack-ironic11:54
*** aparnav has quit IRC12:07
*** dprince has joined #openstack-ironic12:08
*** bfournie has joined #openstack-ironic12:09
*** namnh has quit IRC12:10
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/50963212:11
*** srart has joined #openstack-ironic12:16
*** lucas-hungry is now known as lucasagomes12:17
*** vsaienk0 has quit IRC12:18
*** jaganathan has joined #openstack-ironic12:19
*** vsaienk0 has joined #openstack-ironic12:20
*** jdwidari has joined #openstack-ironic12:22
*** trown|outtypewww is now known as trown12:25
*** jcoufal has joined #openstack-ironic12:27
*** nicodemos has joined #openstack-ironic12:29
*** vsaienk0 has quit IRC12:31
*** vsaienk0 has joined #openstack-ironic12:32
*** fellypefca has joined #openstack-ironic12:37
fellypefcaGood morning :)12:38
gmonteiroGood morning everyone o/12:41
*** namnh has joined #openstack-ironic12:42
*** namnh has quit IRC12:43
*** namnh has joined #openstack-ironic12:43
*** links has quit IRC12:44
*** jdwidari is now known as plan90812:48
*** plan908 is now known as jdwidari12:49
*** catintheroof has joined #openstack-ironic12:50
*** hshiina has quit IRC12:50
*** catintheroof has quit IRC12:51
*** catintheroof has joined #openstack-ironic12:51
*** rloo has joined #openstack-ironic12:51
*** jdwidari has left #openstack-ironic12:57
*** cdearborn has joined #openstack-ironic12:57
*** udesale has joined #openstack-ironic12:58
*** sasha2 has joined #openstack-ironic13:00
*** spectr has joined #openstack-ironic13:02
*** Goneri has joined #openstack-ironic13:03
*** fellypefca has quit IRC13:14
*** namnh has quit IRC13:15
*** namnh has joined #openstack-ironic13:16
*** milan has quit IRC13:20
*** namnh has quit IRC13:20
*** baoli has joined #openstack-ironic13:21
*** noam_ has quit IRC13:22
rloohi and good morning all13:24
rlooIs anyone looking into the gate failures?13:24
*** jlabarre has joined #openstack-ironic13:24
*** xavierr has joined #openstack-ironic13:26
*** rbudden has joined #openstack-ironic13:28
dtantsurmorning rloo; we may feel better after the partial revert of zuul v313:29
dtantsurI haven't checked the most recent status, but I saw some patches passing CI13:29
dtantsuressentially, Zuul is non-voting now13:29
rloodtantsur: 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
dtantsurit may be.. I'm waiting for the things to settle13:31
rloodtantsur: ok then, i'll do other stuff then :)13:31
dtantsurI see jenkins passing on an IPA patch 2 hours ago13:31
patchbotNo data found for patch 213:31
dtantsurlol13:31
dtantsuractually, zuul passed on it too, modulo one apparently random failure13:32
rloodtantsur: this is the most recent failure i saw: https://review.openstack.org/#/c/353156/13:33
patchbotpatch 353156 - ironic - Add rescuewait timeout periodic task13:33
dtantsurjenkins has to be fine, these seem random13:33
dtantsurthe same patch passed jenkins before the recheck13:33
rloodtantsur: yeah, random. but our random test failures could become less random ... so for now, we are recheck'ing?13:34
dtantsurrloo: I'd say, recheck important stuff only13:36
dtantsurto avoid overloading the CI now that people are trying to repair it13:37
rloodtantsur: gotcha. vdrok isn't around so i'm guessing we want to try to do a client release next week?13:37
rloodtantsur: as opposed to last week/this week? :)13:37
dtantsurI'm fine with the next week13:38
dtantsurI'm overloaded with relocation stuff too :(13:38
*** MattMan has quit IRC13:38
*** MattMan has joined #openstack-ironic13:38
rloodtantsur: oh yuck. hopefully it'll be all done soon and then you can have viele bier (sp)13:38
dtantsurI'm done with urgent stuff, hopefully13:39
*** spectr has quit IRC13:40
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/50977913:41
jlvillaldtantsur, rloo: I had updated the whiteboard yesterday that we seem to be seeing a very high failure rate on the multinode job13:43
jlvillalI do believe there is something wrong. But I don't know what it is.13:44
jlvillalSometimes the job passes.13:44
rloojlvillal: thx for looking into it. is it timing out on something?13:44
jlvillalrloo, I'm not sure what is causing it :(13:45
rloojlvillal: sigh. let's keep an eye on it then...13:45
jlvillalrloo, My guess is something network related. Sometimes the nodes don't boot from PXE13:46
*** chlong_ has joined #openstack-ironic13:46
jlvillalrloo, I'm hoping vsaienk0 or vdrok might have an idea :)13:47
rloojlvillal: i think vdrok is on vaca this week. vsaienk0 ^^ if you have any ideas/time to look...13:47
vsaienk0jlvillal: 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 deeply13:49
jlvillalvsaienk0, Thanks13:49
rloovsaienk0: +1 thx! I'll update the whiteboard if no one beats me to it ;)13:50
*** jaypipes has joined #openstack-ironic13:52
dtantsurjlvillal: I've seen it failing too, both on zuul and jenkins13:53
*** spectr has joined #openstack-ironic13:57
*** crushil has joined #openstack-ironic13:58
*** srobert has joined #openstack-ironic13:58
openstackgerritVasyl Saienko proposed openstack/ironic master: Update tests to do not use deprecated test.services()  https://review.openstack.org/50978814:01
vsaienk0dtantsur: 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
patchbotpatch 506173 - ironic - Change pxe dhcp options name to codes.14:02
dtantsurMOAR CONFIG OPTIONZZ!14:03
*** srobert has quit IRC14:03
dtantsurI hate introducing options, if it can be avoided. sometimes it cannot.14:03
* dtantsur will take a look14:03
*** zzzeek has quit IRC14: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 tree14:05
dtantsurwindy weather in Berlin today :D14:05
openstackgerritRuby Loo proposed openstack/ironic master: Use oslotest for base test case  https://review.openstack.org/50892514:05
*** hjensas has quit IRC14:06
*** zzzeek has joined #openstack-ironic14:07
*** hjensas has joined #openstack-ironic14:07
*** hjensas has joined #openstack-ironic14:07
*** spectr has quit IRC14:07
*** spectr has joined #openstack-ironic14:08
*** fellypefca has joined #openstack-ironic14:13
*** namnh has joined #openstack-ironic14:16
*** jheroux has joined #openstack-ironic14:16
etingofit 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
etingofEmilienM, ^14:19
dtantsuretingof: I suggest moving to #puppet-openstack14:19
EmilienMetingof: it's better to do it in openstack-selinux14:19
etingofokay, looks like we do not have anything there yet. or do we?14:20
EmilienMselinux things shouldn't be managed in puppet modules14:20
EmilienMetingof: you tell me, I don't know14:20
etingof+114:20
jlvillaldtantsur, rloo, vsaienk0: As a note: http://ci-watch.tintri.com/project?project=ironic&time=7+days14:22
jlvillalYou can see all the failures for: gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial14:22
* TheJulia semi-re-appears from ghost like status14:23
rloojlvillal: did I ever say that I liked the colour red?...14:23
jlvillalrloo, heh ;)14:23
rloojlvillal: yeah, that doesn't look good.14:24
* jlvillal decides it is time to get out of bed and go make breakfast14:24
* TheJulia wonders if she is in a medication induced alternate reality since rloo is saying that she likes red suddenly.14:24
rlooTheJulia is in an alternate reality :)14:24
TheJuliawoot \o/14:25
rlooHonestly, 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
TheJuliaI do love purple, but the gray+red cobo is a bit hard on the eyes.  Stands out really well though.14:25
rlooTheJulia: +1 for purple!14:27
TheJuliaSo, I've been stupidly sick this week. Anything I've missed that is important or that needs my attention?14:27
rlooTheJulia: i was away yesterday. Am catching up now. I think the gate issues have caused a slowdown in stuff.14:28
TheJuliaYeah, that is kind of what I remember seeing when glancing.14:29
rlooTheJulia: 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 week14:31
*** trown is now known as trown|brb14:34
TheJuliaFar from 100% :(  But I have coffee!14:35
*** etingof has quit IRC14:36
dtantsurhi TheJulia, I hope you feel better14:37
TheJulia:)14:38
TheJulia<3 all of you14:38
*** etingof has joined #openstack-ironic14:39
rlooTheJulia: :)14:40
*** trown|brb is now known as trown14:40
dtantsur^_^14:41
*** ccamacho has quit IRC14:45
*** aparnav has joined #openstack-ironic14:45
*** etingof has quit IRC14:47
NobodyCamGood Morning Ironic'ers14:50
*** aparnav has quit IRC14:52
dtantsurmorning NobodyCam14:52
NobodyCam:) hey hey dtantsur Good Morning :)14:53
rloogood morning NobodyCam14:54
*** jaganathan has quit IRC14:54
NobodyCamGood Morning rloo How are you ding today?14:55
rlooNobodyCam: I'm dinging well today. how are you doing?14:55
*** vsaienk0 has quit IRC14:56
*** xavierr has quit IRC14:56
rlooNobodyCam: I've got my dingle hopper handy (http://www.urbandictionary.com/define.php?term=dingle%20hopper) :D14:56
NobodyCamdoing okay, only on first cup of coffee14:56
NobodyCamhehehe :)14:57
NobodyCamsounds like its breakfast time :)14:58
rlooNobodyCam: enjoy! :)14:58
*** karthiks has quit IRC14:59
NobodyCamhehehe but I don't have a dingle hopper14:59
* rloo hands NobodyCam a dingle hopper15:00
NobodyCamhehehehe :)15:00
*** penick has joined #openstack-ironic15:02
*** tuanla_fujitsu has joined #openstack-ironic15:05
*** tuanla__ has joined #openstack-ironic15:05
*** mat128 has joined #openstack-ironic15:05
*** rpioso has joined #openstack-ironic15:08
*** radez has joined #openstack-ironic15:09
*** rama_y has joined #openstack-ironic15:30
*** dpawar has quit IRC15:30
*** markguz has joined #openstack-ironic15:31
*** mjura has quit IRC15:32
*** dpawar has joined #openstack-ironic15:35
*** markguz_ has joined #openstack-ironic15:39
*** srobert has joined #openstack-ironic15:39
*** pcaruana has quit IRC15:39
*** srobert has quit IRC15:40
*** racedo_ has quit IRC15:40
*** markguz has quit IRC15:42
rpiosodtantsur: 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
openstackLaunchpad bug 1691808 in Ironic "Manual cleaning fails when using iDRAC driver" [High,Fix released] - Assigned to Christopher Dearborn (cdearbor)15:45
patchbotpatch 507942 - ironic - python-dracclient driver version update15:45
rpiosodtantsur: 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
openstackLaunchpad bug 1691808 in Ironic "Manual cleaning fails when using iDRAC driver" [High,Fix released] - Assigned to Christopher Dearborn (cdearbor)15:47
patchbotpatch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED)15:47
*** srobert has joined #openstack-ironic15:47
dtantsurrpioso: bumping a driver version does not fix a bug, a new release of dracclient does.15:48
rpiosodtantsur: 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
rpiosodtantsur: A new release of python-dracclient was created before the ironic bug fix was proposed.15:48
rpiosodtantsur: The new release of python-dracclient's has the tag 1.3.1.15:49
dtantsuressentially, I don't like people bumping requirements for bug fixes that are purely contained in their driver libraries15:49
dtantsurin addition to being incorrect, this also makes patches not backportable15:49
rpiosodtantsur: But, deployment of supported hardware consistently fails.15:50
dtantsurlike, you fix it for Queens now, but the same problem also hits Pike, right? and you cannot apply this change to pike15:50
dtantsurrpioso: right, and we have fixed it, no? it's fixed in dracclient 1.3.1, right?15:50
rpiosodtantsur: Yes, it's also broken in Pike.15:50
* TheJulia opens the pike driver-reqs15:51
*** spectr has quit IRC15:51
rpiosodtantsur: This doesn't seem much different from ironic change https://review.openstack.org/#/c/481318/.15:51
patchbotpatch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED)15:51
rpiosodtantsur: 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
dtantsurrpioso: I'm sorry, I'm losing the thread of this conversation.. what exactly do you propose?15:52
TheJuliahttps://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-installed15:52
*** namnh has quit IRC15:52
*** namnh has joined #openstack-ironic15:53
*** amarao has quit IRC15:54
rpiosodtantsur, TheJuila: The equal sign is an issue.15:54
dtantsurrpioso: sorry, where? I see >=, not ==15:54
rpiosodtantsur, TheJulia: If it's equal to 1.3.0, 13G servers will not consistently deploy.15:55
rpiosodtantsur, TheJulia: What am I missing?15:55
dtantsurthat's why people have to upgrade dracclient, no?15:55
TheJuliayes, but pip will always try to meet the requirements, but it is aware of upgrading versions with the range15:55
TheJuliaif the requirement is satisfied, it does nothing.15:55
dtantsurI just don't get what you try to achieve by bumping driver-requirements. This does not do what you think it does.15:55
dtantsurWell, first it does nothing for many people. E.g. RDO does not sync with it, neither does RHOSP.15:56
TheJuliawhich 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-ironic15:56
rpiosodtantsur: To be >= 1.3.1.15:56
TheJuliarpioso: you want to change the minimum base version, correct?15:57
dtantsurrequirements are bumped when we introduce an explicit dependency. like, using a new function argument or a new function15:57
*** dpawar has quit IRC15:57
*** mrtenio has joined #openstack-ironic15:57
dtantsurwhen you use dracclient.foo() which was introduced in 1.4.0, you cannot say that you require >=1.3.015:57
*** namnh has quit IRC15:57
dtantsurhowever, when there is a bug, we expect operators/distributions to pick fixes without our intervention15:57
dtantsurwe simply cannot track all bug fixes on all stable branches in a way you expect us to15:58
dtantsuraccording to openstack policies, changing the minimum requirement counts as a feature, so it cannot be applied to stable branches as well15:58
dtantsurTheJulia: are we having a BFV meeting now?15:59
TheJulia^--- Was just going to bring that up15:59
*** baoli has quit IRC15:59
TheJuliadtantsur: my brain is quite literally scrambled15:59
TheJuliaI don't see a mjturek...15:59
rpiosodtantsur: So this is different from what was done for change  https://review.openstack.org/#/c/481318/, right?15:59
patchbotpatch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... (MERGED)15:59
dtantsurTheJulia: I can chair it if needed, but I don't have anything on agenda15:59
TheJuliahshiina is around, I guess we could hold a BFV meeting15:59
*** baoli has joined #openstack-ironic16:00
hshiinaTheJulia, hi16:00
TheJuliadtantsur: It could be a very quick meeting "anyone have anything to discuss?" "no" "#endmeeting"16:00
dtantsurrpioso: I think the bump was an oversight from us16:00
rpiosodtantsur, TheJulia: Okay.  Then I'm good.16:01
rpiosodtantsur, TheJulia: Thank you for clarifying.16:01
dtantsuryou're always welcome16:02
rpiosodtantsur: :)16:02
*** markguz_ has quit IRC16:03
*** markguz has joined #openstack-ironic16:03
*** dprince has quit IRC16:03
*** dprince has joined #openstack-ironic16:05
*** markguz has quit IRC16:08
*** xavierr has joined #openstack-ironic16:08
*** rcernin has quit IRC16:13
* dtantsur does not get 47 minutes back, as he has another meeting in parallel16:13
*** rcernin has joined #openstack-ironic16:13
*** rcernin has quit IRC16:14
*** marios has quit IRC16:15
*** e0ne has quit IRC16:17
*** aparnav has joined #openstack-ironic16:18
TheJuliadoh! :(16:19
TheJuliawell, don't worry, I'm discussing project update stuff with hshiina so we can start getting the slide decks together for sydney ;)16:19
TheJuliaso, we didn't get it back either. :)16:20
dtantsurheh16:20
dtantsurTheJulia: have you got milan's slides?16:20
openstackgerritPierre Riteau proposed openstack/ironic stable/ocata: Skip PortNotFound when unbinding port  https://review.openstack.org/50536716:21
TheJuliadtantsur: 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
dtantsurugh :(16:22
TheJuliadtantsur: I asked him to send it over to my personal address where I'll be able to open it16:22
*** namnh has joined #openstack-ironic16:22
*** xavierr has quit IRC16:23
*** srobert_ has joined #openstack-ironic16:23
dtantsurokay, I see16:25
hshiinagood night, everyone16:25
*** srobert has quit IRC16:26
*** etingof has joined #openstack-ironic16:26
*** hshiina has quit IRC16:27
*** tesseract has quit IRC16:29
*** ralonsoh has quit IRC16:31
rloodtantsur, TheJulia, others: for the new rescue_interface, are classic drivers going to support it, or only hardware types?16:35
dtantsurrloo: I think we decidd that classic drivers will have no-rescue hardcoded..16:36
openstackgerritPeter Piela proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes  https://review.openstack.org/47395716:36
rloodtantsur: ok16:37
*** baoli has quit IRC16:39
*** trown is now known as trown|lunch16:40
*** baoli has joined #openstack-ironic16:44
*** etingof has quit IRC16:49
*** udesale has quit IRC16:49
*** tesseract has joined #openstack-ironic16:50
*** derekh has quit IRC16:51
TheJuliadtantsur: That is what I remember as well16:54
rloothx TheJulia. I've added a comment, to get the spec updated with that.16:55
TheJuliarloo: thanks!16:55
openstackgerritMerged openstack/bifrost stable/ocata: scripts: test-bifrost: Start MySQL service before configuring the DB  https://review.openstack.org/50613816:57
openstackgerritMerged openstack/bifrost stable/newton: scripts: test-bifrost: Start MySQL service before configuring the DB  https://review.openstack.org/50613916:58
*** srobert_ has quit IRC17:02
*** srobert has joined #openstack-ironic17:03
*** etingof has joined #openstack-ironic17:03
*** tuanla_fujitsu has quit IRC17:05
*** tuanla__ has quit IRC17:05
*** ijw has joined #openstack-ironic17:07
rlooTheJulia, 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
patchbotpatch 509335 - ironic - Adds rescue_interface to base driver class17:07
dtantsurrloo: I'm around, yes17:08
TheJuliaI'll be around17:08
rloodtantsur, TheJulia: thx, first one gets the honours :)17:09
dtantsurI'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 IRC17:09
TheJuliarebase is likely the best course of action17:10
*** srobert has joined #openstack-ironic17:10
rloodtantsur, TheJulia: -2 until it is rebased?17:10
rloodtantsur, TheJulia: i just don't wnat it to land before the other stuff17:10
rloodtantsur, 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-afk17:11
dtantsurI'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
openstackgerritMerged openstack/python-ironicclient master: Update README  https://review.openstack.org/50789817:12
dtantsurrloo: I think your -1 with a firm "please do" will be enough17:12
rloodtantsur, TheJulia: ok, will -1 then. I had been reserving that for when i actually reviewed it, but that is fine :-)17:13
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/50963217:16
dtantsurokay folks, time to go :) see you tomorrow, see you on Tue rloo17:20
rloobye dtantsur!17:20
* dtantsur means a virtual "go", as he work from home, and the outside is damaged by a storm to a serious extent17:21
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/50977917:22
openstackgerritSam Betts proposed openstack/ironic master: Separate baremetal simulation from devstack-plugin  https://review.openstack.org/50984417:22
*** tesseract has quit IRC17:22
NobodyCamdtantsur: oh wow hope all is okay there.17:22
NobodyCamhave a good night!17:22
dtantsurI'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-stop17:23
TheJuliawow17:24
sambetts|afknight al17:27
rloostay safe dtantsur.17:28
rloogood night sambetts|afk17:28
NobodyCamwow17:31
NobodyCamnight sambetts|afk17:31
*** dtantsur is now known as dtantsur|afk17:32
NobodyCambe safe dtantsur|afk17:32
*** penick has quit IRC17:32
*** gmonteiro has quit IRC17:33
*** aparnav has quit IRC17:35
*** e0ne has joined #openstack-ironic17:35
*** trown|lunch is now known as trown17:36
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/50977917:38
*** namnh has quit IRC17:38
*** penick has joined #openstack-ironic17:39
openstackgerritPavlo Shchelokovskyy proposed openstack/python-ironicclient master: Do not use urljoin in base http client  https://review.openstack.org/50985117:41
*** e0ne has quit IRC17:42
openstackgerritMerged openstack/bifrost master: Use 'none' auth plugin w/o keystone  https://review.openstack.org/49908517:48
*** aparnav has joined #openstack-ironic17:53
*** noam_ has joined #openstack-ironic17:55
*** hamzy has quit IRC17:57
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Deprecate the ironic CLI  https://review.openstack.org/50821818:04
*** ijw has quit IRC18:16
*** e0ne has joined #openstack-ironic18:18
*** e0ne has quit IRC18:23
*** sambetts|afk has quit IRC18:33
*** sambetts_ has joined #openstack-ironic18:36
*** ijw has joined #openstack-ironic18:39
*** ChubYann has joined #openstack-ironic18:40
*** ijw has quit IRC18:41
*** hamzy has joined #openstack-ironic18:41
*** chlong_ has quit IRC18:44
openstackgerritMerged openstack/ironic master: Use oslotest for base test case  https://review.openstack.org/50892518:44
openstackgerritFellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver  https://review.openstack.org/50763218:49
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/50977918:51
*** markguz has joined #openstack-ironic18:57
*** chlong_ has joined #openstack-ironic19:00
*** baoli has quit IRC19:02
*** baoli has joined #openstack-ironic19:02
*** fellypefca has quit IRC19:07
TheJuliajohnthetubaguy: You around?19:21
rloooh, look at that, is it our first merge in daze? ^^ 508925?19:34
TheJuliawoot19:35
openstackgerritXavier proposed openstack/ironic stable/newton: [DNM|WIP] Fix agent deploy on OneView  https://review.openstack.org/50943119:41
*** aparnav has quit IRC19:43
*** chlong_ has quit IRC19:44
*** baoli has quit IRC19:48
rlooTheJulia: wrt bfv, I'm a bit confused, should StorageInterface.detach_volumes() have connector & aborting_attach parameters?19:49
rlooTheJulia: https://github.com/openstack/ironic/blob/4c82308ebed809352674959a3892f8b3bac01d1b/ironic/drivers/base.py#L109319:49
rlooTheJulia: cuz cinder.py has: https://github.com/openstack/ironic/blob/05922a4098a42add937b098b9defafb7dd8b3943/ironic/drivers/modules/storage/cinder.py#L28119:49
*** baoli has joined #openstack-ironic19:49
rlooTheJulia: but noop.py doesn't? https://github.com/openstack/ironic/blob/c491e93a082ede262e617cebfd3022090e3e715b/ironic/drivers/modules/storage/noop.py#L2819:50
rlooTheJulia: nor does FakeStorage.19:51
* TheJulia context switches19:55
openstackgerritYusef Shaban proposed openstack/sushy master: Implement Redfish Sessions  https://review.openstack.org/47194219:57
*** srobert has quit IRC19:57
*** ianw|pto is now known as ianw20:00
*** jkilpatr has quit IRC20:03
TheJuliarloo: 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 IRC20:05
rlooTheJulia: but that's the interface API.20:05
TheJuliatrue, and the only place it is used inside attach_ to cleanup after something going sideways20:06
TheJuliaI 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 method20:07
rlooTheJulia: I didn't look at the code, but maybe you could put that info in some internal* field.20:07
rlooTheJulia: or refactor. Again, I'm not familiar with the code. But I think the API has to be the same...20:07
TheJuliaruntime-ish data and db writes are costly :\20:08
rlooTheJulia: yup. and we do it. whatever works...20:09
TheJuliaI think it is more style because I think I've nova is where some overriden classes methods begin to gain some additional args20:10
TheJuliaspeaking of nova... sticking brain back into the networking code of it20:12
*** jcoufal has quit IRC20:18
openstackgerritStenio Araujo proposed openstack/ironic stable/newton: [DNM|WIP] Fix agent deploy on OneView  https://review.openstack.org/50943120:19
*** jtomasek has quit IRC20:20
*** jkilpatr has joined #openstack-ironic20:22
TheJuliajohnthetubaguy: 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 IRC20:35
*** priteau has quit IRC20:49
*** priteau has joined #openstack-ironic20:50
*** hamzy has quit IRC20:52
*** trown is now known as trown|outtypewww20:56
*** catintheroof has quit IRC20:56
*** priteau has quit IRC20:59
*** crushil has quit IRC21:06
*** rajinir is now known as rajinir-afk21:11
*** jheroux has quit IRC21:11
*** Goneri has quit IRC21:19
*** crushil has joined #openstack-ironic21:19
*** baoli has quit IRC21:19
*** rcernin has joined #openstack-ironic21:22
*** dprince has quit IRC21:23
*** mrtenio has quit IRC21:23
*** crushil has quit IRC21:24
openstackgerritMerged openstack/ironic master: Stop passing raw Exceptions as the reasons for ironic Image exceptions  https://review.openstack.org/50822521:26
jlvillalWoo hoo! Something merged :)21:27
*** bfournie has quit IRC21:30
*** bfournie has joined #openstack-ironic21:30
*** bfournie has quit IRC21:34
*** noam_ has quit IRC21:42
*** markguz_ has joined #openstack-ironic21:47
*** cdearborn has quit IRC21:51
*** charcol has joined #openstack-ironic21:52
*** rcernin has quit IRC22:06
*** baoli has joined #openstack-ironic22:08
*** baoli has quit IRC22:12
*** baoli has joined #openstack-ironic22:12
*** rpioso has quit IRC22:25
openstackgerritMerged openstack/bifrost stable/newton: Fix the dhcp checks on centos  https://review.openstack.org/50048722:38
openstackgerritMerged openstack/bifrost master: scripts: install-deps.sh: Set noninteractive for DEBIAN_FRONTEND  https://review.openstack.org/50943022:44
*** penick has quit IRC22:54
openstackgerritMerged openstack/ironic-python-agent stable/pike: Remove hard-coded path to grub binaries  https://review.openstack.org/50746823:04
TheJulia\o/23:07
*** karimb has quit IRC23:09
*** rloo has quit IRC23:11
*** baoli has quit IRC23:24
*** namnh has joined #openstack-ironic23:28
*** markguz_ has quit IRC23:33
*** hamzy has joined #openstack-ironic23:43
*** jlabarre has joined #openstack-ironic23:55
*** dyasny has quit IRC23:59
*** charcol has quit IRC23:59

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