Tuesday, 2018-05-08

*** mjturek has quit IRC00:00
*** srart has quit IRC00:04
*** rpioso is now known as rpioso|afk00:07
*** hemna_ has quit IRC00:08
*** fragatina has quit IRC00:12
*** fragatina has joined #openstack-ironic00:15
*** fragatina has quit IRC00:19
*** srart has joined #openstack-ironic00:24
*** Goneri has quit IRC00:33
*** tuanla____ has joined #openstack-ironic00:57
*** phuongnh has joined #openstack-ironic01:00
*** trungnv has quit IRC01:14
*** trungnv has joined #openstack-ironic01:14
*** tiendc has joined #openstack-ironic01:21
*** jesusaur has quit IRC01:22
*** jesusaur has joined #openstack-ironic01:24
openstackgerritjiapei proposed openstack/ironic master: Fix XClarity parameters discrepancy  https://review.openstack.org/56140501:29
*** sapd has joined #openstack-ironic01:40
*** namnh has joined #openstack-ironic02:05
*** zhangfei has joined #openstack-ironic02:16
*** fragatina has joined #openstack-ironic02:58
*** gcb has joined #openstack-ironic03:08
*** hemna_ has joined #openstack-ironic03:14
*** fragatina has quit IRC03:20
gyeeTheJulia, I commented on https://review.openstack.org/#/c/566733. The patch does not seem to help.03:22
patchbotpatch 566733 - diskimage-builder - Add simple iscsi-boot element03:22
*** zshi has quit IRC03:27
*** gyankum has joined #openstack-ironic03:41
*** sapd has quit IRC03:48
*** gyee has quit IRC03:48
*** tuanla____ has quit IRC03:50
*** tuanla____ has joined #openstack-ironic03:51
*** namnh has quit IRC04:01
*** tiendc has quit IRC04:01
*** tiendc has joined #openstack-ironic04:01
*** namnh has joined #openstack-ironic04:01
*** fragatina has joined #openstack-ironic04:03
*** fragatina has quit IRC04:04
*** fragatina has joined #openstack-ironic04:04
openstackgerritMerged openstack/ironic-inspector master: PXE Filter dnsmasq: blacklist unknown host  https://review.openstack.org/56640704:05
*** gyee has joined #openstack-ironic04:17
*** gyee has quit IRC04:24
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Fix dnsmasq filter nits  https://review.openstack.org/56678104:36
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: PXE Filter dnsmasq: Clear filter for deleted ports  https://review.openstack.org/56675704:36
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: PXE Filter dnsmasq: Purge cleared files  https://review.openstack.org/56678204:37
*** links has joined #openstack-ironic04:38
*** jesusaur has quit IRC05:14
*** hemna_ has quit IRC05:15
*** e0ne has joined #openstack-ironic05:18
*** jesusaur has joined #openstack-ironic05:20
*** jtomasek has joined #openstack-ironic05:22
*** jtomasek has quit IRC05:22
*** jtomasek has joined #openstack-ironic05:23
*** e0ne has quit IRC05:29
*** zshi has joined #openstack-ironic05:48
*** marios has joined #openstack-ironic05:51
*** mjura has joined #openstack-ironic05:59
*** pcaruana has joined #openstack-ironic06:01
*** rbartal has joined #openstack-ironic06:07
*** penick has quit IRC06:17
*** mjura has quit IRC06:21
*** mjura has joined #openstack-ironic06:24
*** liuzz_ has joined #openstack-ironic06:29
*** liuzz has quit IRC06:29
*** rajinir has quit IRC06:35
*** penick has joined #openstack-ironic06:36
*** jesusaur has quit IRC06:45
*** jesusaur has joined #openstack-ironic06:46
*** vabada has joined #openstack-ironic06:47
*** rcernin has quit IRC07:08
*** sapd has joined #openstack-ironic07:11
*** Chandra has joined #openstack-ironic07:16
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOSInterface  https://review.openstack.org/50779307:17
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching  https://review.openstack.org/51220007:20
*** rwsu has quit IRC07:32
*** gcb has quit IRC07:41
*** parlos has joined #openstack-ironic07:42
*** serlex has joined #openstack-ironic07:46
*** rwsu has joined #openstack-ironic08:02
*** marlinc has quit IRC08:13
vdrokgood morning ironic08:15
*** lucas-pto is now known as lucasagomes08:16
*** dtantsur|afk is now known as dtantsur08:27
dtantsurmorning ironic08:28
*** marlinc has joined #openstack-ironic08:29
vdrokgood morning dtantsur08:33
*** arnewiebalck_ has joined #openstack-ironic08:38
*** hshiina_ has joined #openstack-ironic08:39
*** _mmethot_ has joined #openstack-ironic08:39
*** bfournie1 has joined #openstack-ironic08:39
*** bfournie has quit IRC08:39
*** arnewiebalck has quit IRC08:39
*** arnewiebalck_ is now known as arnewiebalck08:39
*** e0ne has joined #openstack-ironic08:40
*** mmethot_ has quit IRC08:40
*** hshiina has quit IRC08:40
*** zigo_ has quit IRC08:40
*** linkmark has quit IRC08:41
*** linkmark has joined #openstack-ironic08:42
*** derekh has joined #openstack-ironic08:44
*** zigo has joined #openstack-ironic08:45
*** vabada has left #openstack-ironic08:55
*** vabada has quit IRC08:55
*** vabada has joined #openstack-ironic08:55
*** d0ugal__ has quit IRC08:57
*** d0ugal has joined #openstack-ironic08:58
*** hshiina_ has quit IRC09:04
*** goutham has joined #openstack-ironic09:12
*** goutham has left #openstack-ironic09:13
*** sambetts|afk is now known as sambetts09:22
sambettsMorning all09:22
dtantsurhey sambetts09:24
*** vabada has quit IRC09:46
*** vabada has joined #openstack-ironic09:46
*** mgoddard has joined #openstack-ironic09:59
*** rcernin has joined #openstack-ironic10:35
*** namnh has quit IRC10:36
*** tuanla_____ has joined #openstack-ironic10:40
*** yolanda_ has joined #openstack-ironic10:40
*** tuanla____ has quit IRC10:42
*** yolanda has quit IRC10:44
*** MattMan has quit IRC11:02
*** MattMan has joined #openstack-ironic11:02
serlexHi, "cable is unplugged" during pxe boot...but it isn't...anyone else seen that?11:10
sambettsserlex: you'll see cable unplugged if the physical link is down, which includes if it is set administratively down on the switch11:11
sambettsor if it has been set down on the switch due to some error or protocol11:12
serlexSure, will check, but it is PXE booting fine11:13
sambettsserlex: its sucessfully booting into the ironic ramdisk?11:13
serlexyes, I can see both ramdisk and kernel getting loaded up11:14
*** phuongnh has quit IRC11:14
serlexbut half way through console is showing cable unplugged11:14
serlexclearlt local configuration issue, just trying to figure out what11:15
*** tuanla_____ has quit IRC11:15
sambettssounds like the switch could be shutting down the port for some reason or other, worth watching that while it boots11:16
sambettsalso check your spanning tree settings on the switch and ensure your ports are in portfast or port type edge mode11:16
sambettshttps://docs.openstack.org/ironic/pike/admin/troubleshooting.html#dhcp-during-pxe-or-ipxe-is-inconsistent-or-unreliable11:16
*** lucasagomes is now known as lucas-hungry11:20
TheJuliagood morning11:20
serlexcheers sambetts, will do11:20
sambettsTheJulia: morning11:21
*** zhangfei has quit IRC11:23
*** tiendc has quit IRC11:31
arnewiebalckHi, anyone running Ironic in Queens? We’re struggling somewhat with placement/scheduling in nova, and we’re not entirely sure if this is a general issue or specific to our deployment (as we have some backports in nova).11:32
arnewiebalckNova is in Queens as well, btw, with cells v2 and multiple cells.11:33
arnewiebalckGood morning :-D11:33
*** bfournie1 has quit IRC11:34
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: PXE Filter dnsmasq: Clear filter for deleted ports  https://review.openstack.org/56675711:36
jrollmorning everyone11:39
jrollarnewiebalck: I haven't run queens personally, but if you describe the problem, someone may be able to help11:39
*** dprince has joined #openstack-ironic11:40
dtantsurmorning jroll11:47
hjensasarnewiebalck: There is a warning in the docs about scheduling using resource classes is replaced by scheduling using properties. https://docs.openstack.org/ironic/pike/install/enrollment.html#adding-scheduling-information11:47
jrollyes, but queens still works with properties11:48
dtantsurright, though both pike and queens work less stable with them11:49
arnewiebalckhjensas: it’s the other way round, no? :)11:49
dtantsurbecause we stopped doing some ugly hacks to prevent nova from double-assigning nodes11:50
jrollah right11:50
dtantsurarnewiebalck: correct :)11:50
hjensasarnewiebalck: yes it is, sorry.11:50
jrolland yes, the other way around :)11:50
arnewiebalckour nova (on Queens) is using some backports from Rocky so that things work with v2 and multiple cells11:51
arnewiebalckthese backports use aggregates and seems to be related to our issues11:51
arnewiebalckone issue is that the aggregates do not get updated in case there are new compute nodes11:52
arnewiebalckanother is that resource providers get deleted by name (which deletes valid entries that have moved between nodes)11:53
arnewiebalckshutting down compute nodes make their resources disappear (as there is no mapping to the correpsonding aggregates)11:54
arnewiebalckwe’re still trying to get the full picture of all issues :)11:54
arnewiebalckI was just wondering if there are prod deployments on Queens that work fine ;)11:55
arnewiebalckMany of these issues are probably related to our special setup.11:55
jrollhrm11:56
jrollcurious which backports you have, I guess11:57
jrollsounds like bugs we could hit in master11:57
arnewiebalckit’s mostly the code to enable nova v2 to work with multiple cells11:57
arnewiebalckour nova experts thinks these are mostly issues in nova11:57
arnewiebalckand will only hit a wider audience with Rocky :)11:58
jrollI would agree with all of that :)11:58
arnewiebalckjroll: lol11:58
jrollarnewiebalck: would it be possible to get a test environment up on master, and see if you can reproduce?11:59
arnewiebalckIronic worked pretty well until nova upgraded to Queens :)11:59
jrollthen we can get bugs filed and such11:59
arnewiebalckjroll: Ironic on master you mean?11:59
jrollor if you can write something up on how someone else could try to reproduce12:00
jrollarnewiebalck: ironic and nova12:00
jrollor mostly nova I guess12:00
jrolltrying to find out if it's currently a bug on master; if it's not we can track down the fixes and you could backport those12:00
arnewiebalckyes, I guess we’ll do that12:01
*** rbartal has quit IRC12:01
arnewiebalcknot sure we manage before the summit, though, as we’re busy bringing the service to a usable state12:01
jrollyeah, I understand that12:02
jrollI guess the other option is to fix it yourself in your deployment12:02
jrolland if the fix applies cleanly on master, it's probably needed :)12:03
arnewiebalck:-D12:03
arnewiebalckI’ll bring the current state of things to the summit in case someone is around and interested to have a chat.12:04
arnewiebalcklet’s see how we get until then ...12:04
*** rbartal has joined #openstack-ironic12:04
arnewiebalckhow far12:04
jroll++12:05
jrollI believe there is some sort of ironic ops session12:05
arnewiebalckoh, yeah?12:05
jrollI won't be there, but I know the community would love to hear about it (and I hope notes are taken because I'd also like to run multi-cell ironic soonish)12:05
* jroll tries to find it12:05
jrollsure enough: https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21753/ironic-ops-and-user-feedback12:06
arnewiebalckawesome, thx!12:06
jrollno problem :)12:06
*** kiseok7 has quit IRC12:10
*** trown|outtypewww is now known as trown12:13
*** bfournie has joined #openstack-ironic12:14
openstackgerritMerged openstack/networking-baremetal master: Do not run functional (API) tests in the CI  https://review.openstack.org/56658412:15
dtantsurfolks, yet another easy CI patch: https://review.openstack.org/#/c/566677/12:16
patchbotpatch 566677 - ironic-tempest-plugin - Add the functional (API) tests job12:16
*** rbudden has joined #openstack-ironic12:23
*** lucas-hungry is now known as lucasagomes12:24
openstackgerritMerged openstack/ironic stable/queens: Remove vifs upon teardown  https://review.openstack.org/56231412:26
*** parlos has quit IRC12:32
openstackgerritSam Betts proposed openstack/ironic master: [WIP] Direct deploy serve HTTP images from conductor  https://review.openstack.org/56547412:37
*** openstackgerrit has quit IRC12:49
*** arxcruz has quit IRC12:50
*** rloo has joined #openstack-ironic12:51
*** gyankum has quit IRC12:53
*** arxcruz has joined #openstack-ironic12:57
*** arxcruz has quit IRC13:00
*** arxcruz has joined #openstack-ironic13:01
*** openstackgerrit has joined #openstack-ironic13:02
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Bump coreos version to latest stable  https://review.openstack.org/56609413:02
*** jcoufal has joined #openstack-ironic13:02
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Bump coreos version to latest stable  https://review.openstack.org/56609413:06
openstackgerritSam Betts proposed openstack/ironic master: Make validation failure on node deploy a 4XX code  https://review.openstack.org/56688313:14
*** Goneri has joined #openstack-ironic13:17
*** mjturek has joined #openstack-ironic13:25
*** dtantsur is now known as dtantsur|bbl13:36
*** racedo_ has joined #openstack-ironic13:36
*** racedo has quit IRC13:37
*** mjura has quit IRC13:46
*** amotoki has quit IRC13:51
*** lbragstad has quit IRC14:05
*** lbragstad has joined #openstack-ironic14:05
*** e0ne_ has joined #openstack-ironic14:05
*** e0ne has quit IRC14:05
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: PXE Filter dnsmasq: Clear filter for deleted ports  https://review.openstack.org/56675714:10
*** zhangfei has joined #openstack-ironic14:13
*** hamzy has quit IRC14:18
*** hamzy has joined #openstack-ironic14:18
openstackgerritSam Betts proposed openstack/ironic-python-agent master: Attempt to read the partition table after writing an image  https://review.openstack.org/55768714:25
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Bump coreos version to latest stable  https://review.openstack.org/56609414:40
*** rbudden has quit IRC14:41
openstackgerritMerged openstack/ironic-inspector master: Fix dnsmasq filter nits  https://review.openstack.org/56678114:43
*** rcernin has quit IRC14:47
*** rajinir has joined #openstack-ironic14:49
*** hemna_ has joined #openstack-ironic14:55
*** r-daneel has joined #openstack-ironic14:55
*** rbudden has joined #openstack-ironic14:55
*** gyankum has joined #openstack-ironic15:00
openstackgerritMerged openstack/ironic-inspector stable/queens: PXE Filter dnsmasq: blacklist unknown host  https://review.openstack.org/56668215:01
*** zhangfei has quit IRC15:06
*** r-daneel has quit IRC15:08
*** links has quit IRC15:08
*** exodusftw has quit IRC15:36
TheJuliaipa gate is getting worse :(15:39
*** srart has quit IRC15:41
* TheJulia re-runs with tox -r since now pep8 failures with no python changes15:41
*** srart has joined #openstack-ironic15:41
*** lucasagomes is now known as lucas-bbl15:42
TheJuliaand debian package checksum mispmatches15:43
TheJuliamismatches15:43
TheJulia\o/15:43
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: PXE Filter dnsmasq: Purge cleared files  https://review.openstack.org/56678215:47
*** serlex has quit IRC15:49
*** gyan_ has joined #openstack-ironic15:51
*** gyankum has quit IRC15:54
*** e0ne_ has quit IRC15:58
*** dtantsur|bbl is now known as dtantsur16:04
dtantsurTheJulia: ugh :(16:04
*** gyee has joined #openstack-ironic16:04
NobodyCamGood Morning Ironic'ers16:05
TheJuliaand it didn't let me cheat with noqa... :\16:05
TheJuliaand... bashate now fails16:05
dtantsurmorning NobodyCam16:06
dtantsurTheJulia: is it Monday again?16:06
NobodyCamMorning dtantsur :)16:06
TheJuliadtantsur: apparently... clearly I need to take my laptop to the local bar16:06
dtantsur++16:06
*** marios has quit IRC16:06
TheJuliaold tinyipa build it looks like16:08
* TheJulia re-runs16:08
*** trown is now known as trown|lunch16:11
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix gate and bump CoreOS version to latest stable.  https://review.openstack.org/56609416:13
TheJuliadtantsur: ^^^16:13
*** cjloader has joined #openstack-ironic16:14
TheJulialooks like ironic-inspector as well16:15
TheJuliaand ironic16:15
*** harlowja has joined #openstack-ironic16:17
TheJuliaoh this is awesome16:18
dtantsurTheJulia: fun >_< well, I was sure we ignored these pep8 errors explicitly16:19
TheJulialooks like a new hacking 1.1.0 got released two hours ago16:21
dtantsuroh yeah, I'm catching it locally WUT16:21
*** arnewiebalck has quit IRC16:21
* dtantsur thinks of capping it again16:21
dtantsurdidn't they say new checks will be opt-in only?16:21
TheJuliaI really don't remember16:22
TheJuliabut for ironic it is not as easy as fixing ipa... because it also tosses future reserved python 3.7 keywords16:22
TheJulianot just a DeprecationWarning16:22
*** Nisha_Agarwal has joined #openstack-ironic16:24
Nisha_Agarwalpmannidi16:24
rloodtantsur, TheJulia: wrt the previous time we capped. that caused problems with our stable branches. jlvillal fixed the issues by updating the code and/or adding in tox.ini, to ignore certain ones.16:24
*** Nisha_Agarwal has quit IRC16:24
rloonot sure what this new hacky thing is checking for.16:25
dtantsurrloo: it's not a scalable approach though.. I vaguely remember that starting with hacking 1.0 all new checks are opt-in.16:25
rloodtantsur: it must be causing issues for other projects too.16:26
dtantsurI warned #openstack-release (not sure where exactly to go with it)16:26
TheJuliabasically if one wanted to force things like... fixing keywords for 3.7, this would be the way to do it16:27
TheJuliabreak everything and let everyone clean up the dust16:27
TheJuliait is also not very.... friendly.16:27
dtantsurhaven't we fixed the keywords?16:27
TheJuliaapparently not according to what I got out for ironic16:28
TheJuliaI've got like ~1000 lines16:28
dtantsurO_____O16:28
TheJulia./ironic/tests/unit/drivers/modules/irmc/test_boot.py:733:53: W606 'async' and 'await' are reserved keywords starting with Python 3.7 <-- just one16:28
* TheJulia wonders if we just disable pep8 gate checking for a few days... 16:30
* TheJulia thinks that is also a bad idea16:30
dtantsurTheJulia: wanna start an ML thread or should I?16:31
TheJuliayeah16:32
dtantsurTheJulia: yeah to which variant? :)16:32
TheJuliahuh?16:32
dtantsurTheJulia: the question is who should start an ML thread :) I can do it, but you may have more context now.16:33
TheJuliaNot a huge amount, but enough to click send :)16:34
dtantsurcool :)16:35
*** fragatina has quit IRC16:36
dtantsurTheJulia: the release team confirms that from their point of view new checks should be opt-in16:36
mgoddardhi folks, I have a boot from volume question16:42
mgoddardso particularly hi TheJulia :)16:43
TheJuliao/16:43
TheJuliahi mgoddard16:43
mgoddardwe're trying out the standard ipxe boot + cinder lvm/iscsi setup16:43
TheJuliamgoddard: okay16:44
mgoddardwhat is the normal way to allow access to network boot & volume services from tenant networks? Is there one?16:44
hjensasok, so I take it pep8 is b0rked?16:46
*** akhilaki has joined #openstack-ironic16:46
TheJuliathere is not a standard or normal way. It is ultimately an environmental/architectural decision that we don't really have a good grasp on. From an OOB perspective, dedicated interfaces, but that doesn't mix with ipxe16:47
*** jtomasek has quit IRC16:47
TheJuliahjensas: super b0rked16:47
hjensasTheJulia: sigh ...16:48
*** EmilienM is now known as EmilienM_PTO16:49
* TheJulia thinks EmilienM_PTO has the right idea....16:49
TheJuliaso for ironic, I'm thinking for the short term, we pin down hacking, and turn around and create a new changeset to unpin it. Land the pin, and then can interrate on the unpinning16:51
*** gyan_ has quit IRC16:52
rlooTheJulia: that will work. just watch out for stable branches.16:54
mgoddardTheJulia: thanks. That's the answer I was expecting :)16:55
mgoddardTheJulia: seen any 'good' / 'least bad' approaches?16:56
*** derekh has quit IRC16:57
openstackgerritSam Betts proposed openstack/ironic master: Make validation failure on node deploy a 4XX code  https://review.openstack.org/56688316:58
*** rbartal has quit IRC16:59
mgoddardanyways, office kick out time - good night everyone17:00
TheJuliamgoddard: not really, most people seem to allow ip connetivity to cinder for iscsi17:00
TheJuliasince there is the whole utility to mount iscsi volumes from within guests17:00
sambettsmgoddard: one of the main difficulties with allowing network boot etc to work in a tenant network is that tenant networks are allowed to have overlapping IP ranges as long as they aren't routed together, one way I played with was to put a tftp proxy into each tenant network like the metadata proxy17:00
sambettswhich worked just about but probably wouldn't be the best at scale17:01
*** efried has joined #openstack-ironic17:01
* sambetts --> home too 17:02
sambettsnight all17:02
*** sambetts is now known as sambetts|afk17:02
TheJuliagreat point sambetts|afk, I was also just thinking that in some cases people have it so that actually works too :(17:03
efriedTheJulia: Not sure if you were aware of this, but you can disable pep rules in tox.ini, see e.g. https://github.com/openstack/nova/blob/master/tox.ini#L194-L21017:03
efriedTheJulia: That might be a better quick fix while you go in and correct the individual violations.17:03
TheJuliaefried: Yeah, I was just thinking about that looking at the library release timelines and thinking that pinning wouldn't actually do anything17:04
*** mgoddard has quit IRC17:04
openstackgerritKaifeng Wang proposed openstack/ironic master: Follow up to inspect wait implementation  https://review.openstack.org/56694817:05
*** fragatina has joined #openstack-ironic17:09
*** e0ne has joined #openstack-ironic17:10
*** e0ne has quit IRC17:12
*** trown|lunch is now known as trown17:16
openstackgerritJulia Kreger proposed openstack/ironic master: Ignore new errors until we're able to fix them  https://review.openstack.org/56695017:16
*** gyee has quit IRC17:17
openstackgerritJulia Kreger proposed openstack/ironic master: Ignore new errors until we're able to fix them  https://review.openstack.org/56695017:25
*** hamzy has quit IRC17:28
*** harlowja has quit IRC17:31
*** e0ne has joined #openstack-ironic17:32
*** dprince has quit IRC17:46
*** mjturek has quit IRC17:50
*** jlvillal is now known as jlvacation17:51
*** r-daneel has joined #openstack-ironic17:52
*** sekharvajjula has joined #openstack-ironic17:52
*** Chandra has quit IRC17:53
*** jtomasek has joined #openstack-ironic17:54
jlvacationTheJulia, You probably already know this. But maybe can just pin hacking to below the new release version.17:54
jlvacationTheJulia, Sorry I am in the middle of preparing to travel. I leave today.17:55
TheJuliajlvacation: enjoy vacation17:55
jlvacationTheJulia, Thanks!17:55
*** jiapei has quit IRC17:55
TheJuliasome of the changes need to be made in the long run, so it kind of makes sense just to ignore them and peel them off one by one17:56
jroll++18:01
TheJuliaalso should go for MUCH faster reviews18:02
TheJuliasince the changes should be fairly uniform and consistent across change sets18:02
jrollagree18:02
*** racedo_ has quit IRC18:02
dtantsur++18:04
*** sshank has joined #openstack-ironic18:09
*** Goneri has quit IRC18:12
*** baha has joined #openstack-ironic18:12
*** dtantsur is now known as dtantsur|afk18:19
*** gyee has joined #openstack-ironic18:19
dtantsur|afksee you18:19
TheJuliagoodnight18:19
*** hamzy has joined #openstack-ironic18:20
*** harlowja has joined #openstack-ironic18:25
*** vabada2 has joined #openstack-ironic18:26
*** vabada has quit IRC18:27
*** bfournie has quit IRC18:28
*** hwoarang has quit IRC18:28
*** bfournie1 has joined #openstack-ironic18:28
*** hwoarang has joined #openstack-ironic18:29
*** strigazi has quit IRC18:30
*** Goneri has joined #openstack-ironic18:30
*** strigazi has joined #openstack-ironic18:31
*** e0ne has quit IRC18:38
*** jtomasek has quit IRC18:48
*** jaypipes has joined #openstack-ironic18:48
*** EranBD has joined #openstack-ironic18:48
*** openstackgerrit has quit IRC18:49
jaypipesTheJulia: Yo yo yo! Hey, quick request for you. Check out https://review.openstack.org/#/c/565805/. We'd like to ditch the ironic host manager, since it's not used for Ironic node scheduling after Pike. However... based on our conversation the other day where you described Triple-O's decision to not use node resource classes, I'm concerned that triple-o is still dependent on the ironic scheduler host manager. Mind having a looksie?18:50
patchbotpatch 565805 - nova - Remove IronicHostManager and baremetal scheduling ...18:50
jaypipesany triple-o folks around (mwhahaha?), also feel free to check out the above.18:51
TheJuliajaypipes: ack, I'll try to look today. Also dtantsur|afk likely shoudl take a look since he is cognizant of more of the tripleo side than I presently18:52
mwhahahai think matt had ligned up patches everywhere18:52
TheJulialol18:52
jaypipesmwhahaha: mriedem? mdbooth? another matt? :)18:52
mwhahahamriedem18:52
mwhahahain the puppet world and stuff18:53
jaypipesmwhahaha: ah, gotcha.18:53
mwhahahaif it breaks stuff, i'll just blame dtantsur|afk later18:53
jaypipesmwhahaha: ah, indeed I see he does indeed have a bunch of puppet and kolla-ish things.18:53
TheJuliamwhahaha: excellent plan!18:53
jaypipeslol, there's even a fuel-plugin-ironic patch.18:54
jaypipeshe's thorough if nothing else :)18:54
mwhahahaindeed18:54
*** e0ne has joined #openstack-ironic19:02
*** sshank has quit IRC19:17
*** rpioso|afk is now known as rpioso19:17
*** openstackgerrit has joined #openstack-ironic19:22
openstackgerritJulia Kreger proposed openstack/ironic master: Fix W504 errors  https://review.openstack.org/56700919:22
*** e0ne has quit IRC19:24
openstackgerritJulia Kreger proposed openstack/ironic master: Fix W504 errors  https://review.openstack.org/56700919:25
openstackgerritJulia Kreger proposed openstack/ironic master: Fix E305 Errors  https://review.openstack.org/56702520:02
openstackgerritJulia Kreger proposed openstack/ironic master: Fix W605 Errors  https://review.openstack.org/56702620:02
openstackgerritJulia Kreger proposed openstack/ironic master: Silence F405 errors  https://review.openstack.org/56702720:02
openstackgerritJulia Kreger proposed openstack/ironic master: Fix E501 errors  https://review.openstack.org/56702820:02
TheJuliaLooks like W606 is going through deprecation20:02
TheJulianext cycle we can clean up the calls20:02
rlooTheJulia: thx!20:03
TheJuliaand with that, I think I'm going to get a shower and pour a glass of wine20:04
rlooAny cores out there, this could be approved: https://review.openstack.org/#/c/56695020:04
patchbotpatch 566950 - ironic - Ignore new errors until we're able to fix them20:04
anupnHello TheJulia, rloo : I have a quick question, my devstack installtion is failing while provisioning ironic nodes. I can see inside the ironic-bm-logs "No bootable device". But not sure why, because I can see DHCP server running but may be ironic is not sending correctly requests to DHCP server, could you point where should I look for understanding what may be going wrong20:06
anupnalso to note this devstack setup is inside vagrant20:07
*** e0ne has joined #openstack-ironic20:20
*** pcaruana has quit IRC20:21
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix gate and bump CoreOS version to latest stable.  https://review.openstack.org/56609420:22
TheJuliaanupn: At a high level, ironic ships the pertinant information to neutron which updates the dnsmasq configuration20:24
TheJuliaanupn: so what your encountering could be how the networks the vm are plugged into is wired into neutron, and in all liklihood the dhcp server that neutorn has created on the provisioning network is just not bridged to that vm network... at least that is most commonly what has occured.20:25
anupnTheJulia: hmm, yes in ir-cond logs I see the error while creating neutron ports but I believe those errors were because no bootable device was found20:27
TheJuliaanupn: order of operations is wrong, neutron port error would be the path you should investigate since ironic has no idea about the bootable devices yet20:28
anupnTheJulia: Ah okie20:29
anupnTheJulia: is there a way to check on what network dhcp server has been created by Neutron?20:29
anupnTheJulia: You were right, I see an error in q-dhcp.service "Network interface tap48c846a4-ba not found in namespace qdhcp-97dd254e-510b-4e3d-b9cc-e1196cd9f966."20:34
TheJuliaanupn: looks like your on the right path  :)20:38
anupnTheJulia: I am wondering these tap interfaces and all should be handled during devstack neutron setup, I didn't get any error earlier when Devstack was setting up neutron20:39
anupnmay be destroying the vagrant and bringing back might help20:39
*** hemna_ has quit IRC20:41
*** e0ne has quit IRC20:45
bdoddSo today I've been setting up my dev env using the guide here: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html20:49
bdoddI found it quite helpful!20:49
bdoddOne step that seemed to be missing was the install of the python-openstackclient20:50
bdoddSeems like in section "Step 1: Create a Python virtualenv" where it says to "pip install python-ironicclient", it should include installing python-openstackclient20:52
bdoddI was able to successfully create a Redfish node and send it a power on command. So, yay!20:53
anupnbdodd: cool20:56
*** trown is now known as trown|outtypewww20:57
anupnTheJulia: thanks for the pointers :)21:02
openstackgerritMerged openstack/ironic master: Ignore new errors until we're able to fix them  https://review.openstack.org/56695021:02
*** Goneri has quit IRC21:04
*** hemna_ has joined #openstack-ironic21:13
*** ianychoi has quit IRC21:15
*** rbudden has quit IRC21:19
*** baha has quit IRC21:21
TheJuliaanupn: they are dynamicly created an ddestroyed21:25
TheJuliabdodd: ahh, minor oversight given the evolution of docs over time. Luckily it is a super easy fix21:26
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Update wording used in removal of VIFs  https://review.openstack.org/56231521:27
bdoddTheJulia: that's the best kind of fix21:27
openstackgerritJulia Kreger proposed openstack/ironic master: Install OSC during quickstart  https://review.openstack.org/56706121:34
TheJuliabdodd: ^^^21:35
TheJuliahmm... the coreos container still doesn't like me21:41
* TheJulia fires off a new local build with the hope that it will finish21:41
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Fix pycodestyle warnings/errors now visible with hacking 1.1.0  https://review.openstack.org/56706621:42
TheJuliahjensas: thanks, and go get some sleep :)21:42
hjensasTheJulia: yeah, I did that with a gin and tonic as comfort. (consider it a friendly review warning ;))21:43
TheJuliahjensas: I completely approve ;)21:43
TheJuliaspeaking of which, I'm going to go open a bottle of red wine and unwind with kerbals!21:44
jroll++21:45
openstackgerritHarald Jensås proposed openstack/ironic-inspector master: Fix pycodestyle warnings/errors now visible with hacking 1.1.0  https://review.openstack.org/56706621:46
*** jcoufal has quit IRC21:47
hjensasTheJulia: enjoy!21:48
* hjensas steps out on the balcony to look at starts. o/21:48
*** hjensas is now known as hjensas|afk21:49
*** bfournie1 has quit IRC21:51
*** bfournie has joined #openstack-ironic21:51
*** bfournie has quit IRC21:51
*** akhilaki has quit IRC22:03
*** cjloader has quit IRC22:04
*** cjloader_ has joined #openstack-ironic22:04
bdoddTheJulia: Thanks! That fix looks good. Should I add a comment or review or anything to the change?22:08
*** cjloader_ has quit IRC22:09
*** akhilaki has joined #openstack-ironic22:13
*** rwsu has quit IRC22:13
anupnTheJulia: Yes I understand but I don't see any particular error that explains why it was unable to create the tap interface. I am running devstack again inside new vagrant VM, hoping to get pass that error22:21
*** rcernin has joined #openstack-ironic22:25
*** bfournie has joined #openstack-ironic22:27
openstackgerritJim Rollenhagen proposed openstack/ironic master: Make validation failure on node deploy a 4XX code  https://review.openstack.org/56688322:52
jrollrebased for pep8, TheJulia ^ you previously +2'd that22:52
jrollclarkb: is this waiting for a larger decision on the path forward or just for you to get time to do it? I can take it over if the latter: https://review.openstack.org/#/c/559811/22:54
patchbotpatch 559811 - ironic - Preserve env when running vbmc22:54
clarkbjroll: I think its largely waiting on us (big collective openstack /devstack users) to decide it is the way forward22:55
jrollok, thanks22:55
clarkbthat said I think its a reasonable improvement to ironic's tooling that should be useable even as is22:55
clarkbif you want to take it over and get it in before devstack moves that should be fine (then you'd just be ready if it happens)22:56
jrollclarkb: is the wip procedural then?22:56
jrollwould rather just +2 :)22:56
clarkbjroll: ya I didn't awnt to give the impression that devstack was absolutely headed that direction22:56
clarkbshould be updated to address ilya's concerns before +2'ing but is fine otherwise I think22:57
jrollnot entirely sure I understand the difference22:57
jrollbut I'm fine with what you have, honestly22:58
* jroll rebases and +222:58
clarkbheh22:58
openstackgerritJim Rollenhagen proposed openstack/ironic master: Preserve env when running vbmc  https://review.openstack.org/55981122:58
jrollIIRC sudo -u didn't work22:58
clarkbya it is because sudo sanitizes the env but now that we set PATH directly I think it would work22:59
clarkbit just took me far too long to understand that22:59
jrollah I see, makes sense22:59
jrolletingof: re 559811 I'm fine as is, but would also +2 if you update it :)23:00
clarkband right now with summit so soon I'm swamped with a bunch of not devstack stuff :/23:00
*** pmannidi has joined #openstack-ironic23:03
*** rloo has quit IRC23:15
*** EranBD has quit IRC23:20
*** cjloader has joined #openstack-ironic23:23
*** r-daneel has quit IRC23:26
*** cjloader has quit IRC23:27
gyeeTheJulia, I finally make bfv work with ubuntu23:33
gyeebut I had to make a bunch of changes to the element23:34

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