Friday, 2016-05-13

*** alexpilo_ has quit IRC00:01
*** alexpilotti has joined #openstack-ironic00:01
openstackgerritMerged openstack/bifrost: Updated from global requirements  https://review.openstack.org/31371200:03
*** jvgrant_ has joined #openstack-ironic00:05
*** jvgrant has quit IRC00:07
*** sacharya has quit IRC00:09
*** Sukhdev has joined #openstack-ironic00:16
harlowjadtantsur where is the code in ironic that converts the old periodic format to the new (i'm gonna see about what a layer in oslo.service that uses the older stuff would look like)00:19
*** ayoung has joined #openstack-ironic00:20
*** rcernin has quit IRC00:24
openstackgerritMerged openstack/ironic: devstack: Fetch tarball images via https  https://review.openstack.org/31566300:27
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/31505700:27
*** catintheroof has quit IRC00:27
*** jvgrant__ has joined #openstack-ironic00:32
*** jvgrant_ has quit IRC00:34
*** mbound has quit IRC00:44
*** vishwanathj has quit IRC00:45
*** YiboCai has joined #openstack-ironic00:49
*** shu-mutou has joined #openstack-ironic00:54
*** hoangcx has joined #openstack-ironic00:55
*** shu-mutou has left #openstack-ironic00:55
*** openstackgerrit has quit IRC01:02
*** openstackgerrit has joined #openstack-ironic01:03
*** mtanino has quit IRC01:05
*** rbudden has quit IRC01:07
*** sacharya has joined #openstack-ironic01:10
*** chlong has joined #openstack-ironic01:13
*** sacharya has quit IRC01:15
*** vishwanathj has joined #openstack-ironic01:18
*** alex_xu_ has joined #openstack-ironic01:30
*** chlong has quit IRC01:31
*** ijw has quit IRC01:33
*** M4g1c5t0rM has joined #openstack-ironic01:40
*** mbound has joined #openstack-ironic01:45
*** rama_y has joined #openstack-ironic01:45
*** mbound has quit IRC01:49
*** alexpilotti has quit IRC02:01
*** alexpilotti has joined #openstack-ironic02:03
openstackgerritwatanabe isao proposed openstack/ironic: Update neutron services' phrase  https://review.openstack.org/31583702:03
*** mbound has joined #openstack-ironic02:46
*** mbound has quit IRC02:50
*** sivaramakrishna has joined #openstack-ironic02:51
*** mtanino has joined #openstack-ironic02:53
*** thrash is now known as thrash|g0ne02:53
*** harlowja_at_home has joined #openstack-ironic02:56
*** absubram has joined #openstack-ironic02:57
*** absubram_ has joined #openstack-ironic02:59
*** absubram has quit IRC03:02
*** absubram_ is now known as absubram03:02
*** chlong has joined #openstack-ironic03:08
*** afaranha has quit IRC03:08
*** sacharya has joined #openstack-ironic03:12
*** sacharya has quit IRC03:16
*** rama_y has quit IRC03:26
*** rama_y has joined #openstack-ironic03:26
*** sacharya has joined #openstack-ironic03:28
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624403:38
*** absubram has quit IRC03:39
*** gcb has joined #openstack-ironic03:41
*** szhsong has joined #openstack-ironic03:43
openstackgerritClint 'SpamapS' Byrum proposed openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586203:49
*** rama_y has quit IRC03:58
*** rama_y has joined #openstack-ironic04:07
*** ayoung has quit IRC04:07
*** vmud213 has joined #openstack-ironic04:07
*** vmud213 has quit IRC04:15
*** vmud213 has joined #openstack-ironic04:16
*** mtanino has quit IRC04:19
*** bodepd has quit IRC04:21
*** links has joined #openstack-ironic04:23
*** gcb has quit IRC04:39
lazy_princeHi.. Can someone from core +A https://review.openstack.org/#/c/315623/04:45
*** vishwanathj has quit IRC04:45
*** rama_y has quit IRC04:48
zer0c00lpip 8.1.2 crap!04:50
zer0c00lis it fixed yet?04:51
*** gcb has joined #openstack-ironic04:52
*** joprovost has quit IRC04:56
*** gcb has quit IRC04:58
*** alexpilotti has quit IRC05:00
*** alexpilotti has joined #openstack-ironic05:01
*** fragatina has quit IRC05:10
*** ChubYann has quit IRC05:17
jlvillalzer0c00l, What about pip 8.1.2?05:18
*** hamzy has joined #openstack-ironic05:19
*** gcb has joined #openstack-ironic05:20
*** amotoki has quit IRC05:23
*** chlong has quit IRC05:29
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Add network drivers  https://review.openstack.org/28585205:30
*** harlowja_at_home has quit IRC05:34
*** rcernin has joined #openstack-ironic05:50
*** vmud213 has quit IRC05:50
*** vmud213 has joined #openstack-ironic05:51
*** amotoki has joined #openstack-ironic05:52
*** fragatina has joined #openstack-ironic05:54
*** szhsong has quit IRC05:56
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Add network drivers  https://review.openstack.org/28585205:56
*** fragatina has quit IRC05:58
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326206:03
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849606:05
*** fragatin_ has joined #openstack-ironic06:11
*** fragatin_ has quit IRC06:11
*** fragatin_ has joined #openstack-ironic06:12
*** mjura has joined #openstack-ironic06:14
*** e0ne has joined #openstack-ironic06:15
*** wajdi has quit IRC06:15
*** fragatin_ has quit IRC06:16
*** mjura has quit IRC06:20
*** mjura has joined #openstack-ironic06:21
*** parlos has joined #openstack-ironic06:27
openstackgerritVasyl Saienko proposed openstack/ironic: TEST Checking Gates  https://review.openstack.org/31110906:27
*** fragatina has joined #openstack-ironic06:32
*** fragatina has quit IRC06:33
*** fragatina has joined #openstack-ironic06:33
*** wajdi has joined #openstack-ironic06:33
parlosGood Morning06:45
*** parlos has quit IRC06:45
*** amotoki has quit IRC06:51
*** sacharya has quit IRC06:54
*** Sukhdev has quit IRC06:55
*** tesseract has joined #openstack-ironic06:56
*** amotoki has joined #openstack-ironic07:01
*** amotoki has quit IRC07:01
*** amotoki has joined #openstack-ironic07:07
*** amotoki has quit IRC07:07
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Wait for all interfaces to be up before node lookup  https://review.openstack.org/30350307:16
*** amotoki has joined #openstack-ironic07:17
*** athomas has joined #openstack-ironic07:23
*** hoangcx has quit IRC07:25
*** jcoufal has joined #openstack-ironic07:44
*** szhsong has joined #openstack-ironic07:53
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:02
*** ohamada has joined #openstack-ironic08:06
*** ohamada has quit IRC08:08
*** ohamada has joined #openstack-ironic08:09
*** kromanenko_ has joined #openstack-ironic08:19
*** lucas-dinner is now known as lucasagomes08:31
lucasagomesgood morning all08:32
*** vmud213 has quit IRC08:32
*** vmud213 has joined #openstack-ironic08:32
*** lintan_ has quit IRC08:33
*** alex_xu_ has quit IRC08:33
*** lintan_ has joined #openstack-ironic08:34
*** alex_xu has joined #openstack-ironic08:37
aarefievmorning all!08:39
vdrokmorning ironic, lucasagomes aarefiev08:39
aarefievmorning vdrok08:40
*** parlos has joined #openstack-ironic08:40
*** jtomasek_ has joined #openstack-ironic08:46
pas-hamorning aarefiev vdrok lucasagomes08:46
*** jistr has joined #openstack-ironic08:46
vsaienkomorning Ironic'ers08:48
*** jtomasek_ has quit IRC08:51
*** e0ne has quit IRC08:52
*** sacharya has joined #openstack-ironic08:54
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Add network drivers  https://review.openstack.org/28585208:55
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326208:55
sivaramakrishnamorning all!08:57
*** sacharya has quit IRC08:59
*** electrofelix has joined #openstack-ironic09:07
*** dmk0202 has joined #openstack-ironic09:09
*** oberezovskyi has quit IRC09:10
jrollwelp, it's friday alright09:11
jrollhttp://logs.openstack.org/35/313035/4/check/gate-tempest-dsvm-ironic-agent_ssh/b98e41d/logs/ironic-bm-logs/node-0_console_12-05-2016-22:24:40.txt.gz09:11
jrollhttp://logs.openstack.org/35/313035/4/check/gate-tempest-dsvm-ironic-agent_ssh/b98e41d/logs/screen-ir-cond.txt.gz?level=WARNING09:11
jroll:|09:11
jrollsambetts: something's suddenly up with tinyipa09:13
jrollI think just whole disk images? it's unclear to me09:13
sambettsjroll: ?!09:15
jrollright?09:15
jrollso this does bring up a point about tinyipa09:15
jrollif this was ubuntu, I'd be like "oh apt-get install lib-whatever"09:15
jrollnfi what to do here, but then again haven't really looked around09:16
sambettsjroll: you just need to add it to build_files/finalreqs.lst if it exists in the tiny core repo09:16
sambettsjroll: which I'm pretty sure this does, however it should actually be being pulled in by gdisk as a requirement...09:17
jrollya09:17
sambettshuh... well... ok, so when they updated gdisk for us to remove the requirement on icu they also removed the whole dep file, because unless you use sgdisk its an optional requirement09:18
jroll$ curl -s distro.ibiblio.org/tinycorelinux/7.x/x86/tcz/gdisk.tcz.dep | grep pop                                          1 ↵09:18
jrollpopt.tcz09:18
jrollhm09:18
sambettsjroll: thats the x86 deps09:19
sambettsyou need x86_6409:19
jrollso we need to add popt-dev?09:19
jrollah09:19
sambettsgdisk.tcz.dep doesn't exist in x86_64 :/09:19
jrollyep :|09:19
sambettsnot sure why it wasn't caught by the gate when we merged the patch...09:20
jrollyeah, that's interesting09:21
jrollunless they removed it between merge and post job that built it09:21
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: tinyipa: add popt to deps  https://review.openstack.org/31598209:22
jrollthat should do it09:22
sambettsI guess because we weren't testing agent deploy with tinyIPA?09:22
sambettsjroll: ha you beat me to it!09:22
*** vmud213 has quit IRC09:22
jrollthat could be too?09:22
jrollI think this is just agent driver whole disk images09:22
sambettsjroll: did JayF's patch merge switching to tinyIPA for all things and thats why we now see it09:23
jrollso you might be right09:23
lucasagomesmorning sambetts jroll09:23
jrollbut post queues are long lately09:23
jrollum09:23
jrolloh no, that's where I saw the break https://review.openstack.org/#/c/313035/09:23
jrollso maybe rest of gate is fine til that lands :)09:23
sambettsooooh yeah post queue might have delayed the build09:23
sambettsso the change to 7.x wasn't picked up until that was done09:24
*** vmud213 has joined #openstack-ironic09:24
jrollhttp://status.openstack.org/openstack-health/#/?searchProject=ironic&groupKey=project&resolutionKey=hour&end=2016-05-13T09:24:08.130Z09:24
jrolldoesn't look like we're totally broken09:24
openstackgerritSivaramakrishna Garimella proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849609:25
jrolloh, well maybe http://status.openstack.org/openstack-health/#/?searchProject=ironic&groupKey=project&resolutionKey=hour&end=2016-05-13T09:24:08.130Z&duration=P1D09:25
jrollthere http://status.openstack.org/openstack-health/#/g/project/openstack~2Fironic?groupKey=project&resolutionKey=hour&duration=P1D&end=2016-05-13T09:24:08.130Z09:25
sambetts:/ yeah we should get that landed ASAP while the post queue is short09:25
jrollnot enough load to tell09:25
jrollyep, lucasagomes mind hitting 315982?09:26
sambettshey lucasagomes09:26
jrolland morning lucasagomes :)09:26
* lucasagomes looks09:26
lucasagomesjroll, done, +2+a09:27
jrollthanks man09:27
sambettsyour up super early aren't you?09:27
jrollso TIL openstack-health is suddenly awesome09:27
jrollalso, we're failing oslo master09:27
jrollhttps://github.com/openstack/oslo.log/commit/7c671ea77f4d7701d0def9a9e17b6c49310ca43809:27
jrollHA09:27
lucasagomesyeah I didn't know about it09:28
jrollsambetts: ish, couldn't sleep09:28
sambetts:(09:28
openstackgerritMerged openstack/python-ironic-inspector-client: Bump supported API version to 1.5  https://review.openstack.org/31504709:28
* lucasagomes likes the failopotamus project for specific jobs failures as well09:28
openstackgerritJim Rollenhagen proposed openstack/ironic: Make iPXE + TinyIPA the defaults for devstack  https://review.openstack.org/31303509:28
jrolloh ugh, depends-on won't help there09:29
jrollwhoops >.>09:29
sambettslucasagomes: failopotamus?09:29
openstackgerritJim Rollenhagen proposed openstack/ironic: Make iPXE + TinyIPA the defaults for devstack  https://review.openstack.org/31303509:29
lucasagomessambetts, yeah, it's a tiny script https://github.com/sdague/failopotamus09:29
sambettsjroll: because its not a src job right?09:30
lucasagomesit greats a graph with the failures of a specific job in gate09:30
* lucasagomes internally it just calls graphite09:30
lucasagomesbut yeah, it's nice09:30
sambettsha thats cool!09:30
jrollsambetts: yep09:31
jrolloh wow, gerrit recognized I put it back correctly and reapplied the votes09:31
jrollO_o09:31
sambettswoah?! I didn't know that was a thing!09:32
*** e0ne has joined #openstack-ironic09:33
jrollright?09:33
*** zhihaisong has joined #openstack-ironic09:37
lucasagomesaarefiev, the nova bug seems tricky, I'm not sure yet, but it seems that since we create all the nodes with the same amount of cpus, ram and disk they all have the same resource_class_id in the inventories table09:38
lucasagomesso once one is changed to 0, all the rest is updated09:38
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove verbose option  https://review.openstack.org/31599009:38
jroll^ this should unbreak us on oslo master09:38
sambettswow... our tests actual test a URL we don't support http://logs.openstack.org/66/315766/2/check/gate-tempest-dsvm-ironic-pxe_ssh/695c53b/console.html09:39
sambettsactually*09:39
sambettsaccording to our API ref anyway...09:39
*** jcoufal has quit IRC09:39
jrollsambetts: O_o09:40
jrollthat feels like a bug / another broken gate thing09:40
*** szhsong has quit IRC09:40
sambettsthats on my patch https://review.openstack.org/#/c/315766/09:40
jrolllike, clearly something happened there, we didn't magically get a new broken test09:40
jrollsure you didn't break that? :)09:40
jrollI don't quite trust our api ref yet :/09:41
sambettsI broke it if the test is trying to do /v1/nodes/states/console, because the URL we define and I made it support in my patch is v1/nodes/<node_id>/states/console09:42
jrollright, apparently we test that09:42
* jroll wonders what that does09:42
*** zhihais has joined #openstack-ironic09:42
sambettswoah?! no please no ...09:44
sambettswe're actually testing /v1/nodes/states/console/<node_id>09:44
jrollhahahahahaha09:44
jrollT_T09:45
sambettsewwwwwwwwwwwwwwwwwwwwwwwwwwwww!!!!!09:45
jroll(that does totally work)09:45
*** zhihaisong has quit IRC09:45
jroll/v1/nodes/states/console gives "error_message": "{\"debuginfo\": null, \"faultcode\": \"Client\", \"faultstring\": \"Missing argument: \\\"node_ident\\\"\"}"09:45
* sambetts claws his eyes out 09:45
* sambetts also thinks that is a bug 09:45
*** hoangcx has joined #openstack-ironic09:45
aarefievlucasagomes: hey, I have found the root cause, it's issue in db code while updating inventories09:46
jrollfwiw v1/nodes/states/power/uuid doesn't work09:46
lucasagomesaarefiev, yeah that's what I was heading to09:46
lucasagomesaarefiev, thanks for that09:46
lucasagomesaarefiev, do you have a patch for it already?09:46
aarefievlucasagomes: I'm working on it now, it's one line fix09:47
sambettsjroll: .... oh man, these controllers really need somework...09:48
jrollsambetts: curl -v -X PUT  http://localhost:6385/v1/nodes/maintenance/45a6b4e2-fb9f-4ab6-8760-74e231d441b3 -> 20209:48
* jroll cries09:48
*** chopmann has joined #openstack-ironic09:48
sambettsyeah... that should be fixed by my patch too09:49
jrollthis is a really interesting case of breaking an api09:49
jrolltechnically we shouldn't do that, but...09:49
sambettsI say if we never supported it in the first place its a bug09:49
lucasagomesaarefiev, cool09:50
jrollwell, how do you define supported09:50
sambettsdocumented support09:50
jrollwell than all of our apis are a bug ;)09:50
jrollI do get what you mean09:50
sambettsheh, http://docs.openstack.org/developer/ironic/webapi/v1.html#get--v1-nodes-(node_ident)-states-console <- supported09:50
lucasagomesyeah, it's too many magic with controllers09:51
lucasagomesparameters can be inputed as part of the URL or body request09:51
jrollyeah, it's a bit of a dumpster fire09:51
sambettslucasagomes: having worked with pecan and RoR a lot in the past, we've used and abused the controllers  really badly09:52
lucasagomessambetts, right, the controllers way AFAIUI is the wsme way of doing it right?09:52
jrollnah, controllers are a well-understood thing in most frameworks09:52
jrollmodel-view-controller09:53
sambettsnot as far as know, the controllers come from pecan, wsme just fudges some validation stuff in there09:53
*** chopmann has quit IRC09:53
lucasagomesgotcha09:53
lucasagomesyeah apparently we did it wrong from the start then :-/09:53
jrollmodel == internal (object) representation, view == transformation to the API representation, controller == api logic09:54
jrollimo the magic routing and the way pecan exposes parameters, as well as unfamiliarity with the things, is what made us shoot ourselver in the foot09:54
jrollourselves*09:54
lucasagomesyeah09:55
lucasagomesyeah, it's funny that this problem may affect other OS projects using wsme09:55
*** sacharya has joined #openstack-ironic09:55
lucasagomescause we looked at them to make our api09:56
jrollI kind of want to rewrite it in $other_framework, but we don't have great api testing so we'll almost certainly break something09:56
sambettsthe thing that breaks the pecan RestController magic is that sub-resource functions are expecting to receive idents from all its parents, e.g. def get_one(node_id, state_id): ... but we've written weird functions that only receive one parameter in e.g. def raid(node_id): and so when it tries to do its matching its like oh I've got enough info to call that function so I must be able too09:57
*** amotoki has quit IRC09:58
lucasagomesjroll, yeah agreed, loads of corner cases there09:58
jrollI so want to get the network stuff done, make our testign amazing, and then just do tech debt cleanup for a cycle09:59
jrollif I was a dictator, I think that's what I would do10:00
*** sacharya has quit IRC10:00
lucasagomesjroll, heh don't need to be one, I mean, it's arguable10:00
lucasagomesand in fairness we do need to do it yes10:01
jrolllucasagomes: yeah, just saying not everyone would agree and I can't force it10:01
lucasagomesyeah... there's a bunch of spaghetti code (/me thinks about the pxe config generation bits)10:02
openstackgerritAparna proposed openstack/ironic: Modify doc for RAID clean steps in manual cleaning  https://review.openstack.org/31599810:02
sambettsjroll: wow... just wow... https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/services/baremetal/v1/json/baremetal_client.py#L331 does it one way, then the function below it does it the right way ...10:06
*** gcb has quit IRC10:06
jrollsambetts: yeah, so we intend for /v1/nodes/states/console?node_id=foo to work10:06
jroll(apparently)10:06
sambettsdo we?10:07
*** e0ne has quit IRC10:07
jrollwell10:08
jrollif we have a test for it...10:09
sambettsthat call creates a URL that looks like /v1/nodes/states/console/<node_id>10:10
*** YiboCai has quit IRC10:11
sambettsalso I'm not sure if that file is actually a test is it? its just used by the tests? and its the only one of its kind looking through the fiel10:11
jrollaha, I see10:12
jrollhrm10:12
sambettsis the OrphanedObjectError thing fixed yet? I still see it in the n-cpu logs for this patches tests10:14
sambettspython-ironiclient does it /v1/nodes/<node id>/states/console10:15
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576610:15
*** amotoki has joined #openstack-ironic10:18
jrollsambetts: should be as of like... 2200 utc?10:19
*** hoangcx has quit IRC10:19
sambettsjroll: hmmm that test run posted results at 22:29 UTC so it might have been the last one to run without the fix10:20
*** d0ugal has quit IRC10:20
*** d0ugal has joined #openstack-ironic10:21
jrollsambetts: that time is totally an estimate fwiw :)10:22
jroll21:46:41            JayF | yep just landed10:22
jrollto be exact10:22
sambetts:-P the new run I've just pushed should pass then, cool10:23
*** kromanenko__ has joined #openstack-ironic10:26
openstackgerritJim Rollenhagen proposed openstack/ironic: Test post don't upvote  https://review.openstack.org/31186510:29
openstackgerritVasyl Saienko proposed openstack/ironic: TEST Checking Gates  https://review.openstack.org/31110910:30
*** jtomasek_ has joined #openstack-ironic10:33
jrollvery interesting http://lists.openstack.org/pipermail/openstack-dev/2016-May/094867.html10:34
*** amotoki has quit IRC10:36
jrollbbiab10:36
*** jtomasek_ has quit IRC10:42
*** amotoki has joined #openstack-ironic10:43
*** amotoki has quit IRC10:44
openstackgerritMerged openstack/ironic-python-agent: tinyipa: add popt to deps  https://review.openstack.org/31598210:52
sambettswoooop!! ^10:52
lucasagomessambetts, w00t10:57
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Add unittests for the config.py module  https://review.openstack.org/31602110:57
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Set IPMI session timeout for the virtual BMCs  https://review.openstack.org/31602210:57
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Set IPMI session timeout for the virtual BMCs  https://review.openstack.org/31602211:00
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Bump pyghmi version to 1.0.3  https://review.openstack.org/31602311:00
*** alexpilotti has quit IRC11:00
*** mkovacik__ has joined #openstack-ironic11:02
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [WIP] [POC] Callback from ramdisk support for Ansible driver  https://review.openstack.org/31081611:02
*** mkovacik__ is now known as _milan_11:03
*** devananda has quit IRC11:03
*** baoli has joined #openstack-ironic11:04
sivaramakrishnavsaienko: can you rebase the hwinfo patch https://review.openstack.org/#/c/256364/61 ?11:04
*** sivaramakrishna has quit IRC11:05
*** alexpilotti has joined #openstack-ironic11:07
*** baoli has quit IRC11:09
*** mgould has joined #openstack-ironic11:14
*** devananda has joined #openstack-ironic11:14
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Fix local boot issue with fedora in uefi mode  https://review.openstack.org/30214311:14
mgouldmorning Ironic11:14
*** blinky_ghost has joined #openstack-ironic11:17
lucasagomesmgould, g'morning11:17
sambettsjroll: right now we make DELETE v1/chassis/<chassis id>/nodes/<node id> an operation not permitted, I assume this was done because right now the routing would make v1/chassis/<chassis id>/nodes pass the chassis id into the delete function, with my changes the right IDs are passed into the right places, e.g. if you do DELETE v1/chassis/<chassis id>/nodes it complains you havn't pass in a node11:18
sambettsid, which is the correct behavior, can we make that action valid now?11:18
sambettslucasagomes: ^^11:18
mgouldmorning lucasagomes sambetts jrist11:19
mgould*jroll11:19
sambettshey mgould11:19
lucasagomessambetts, yeah the old api has a bunch of from_<resource> preventing certain actions11:19
*** thrash|g0ne is now known as thrash11:20
lucasagomesdue the way it was programmed (aka wrongly) :-/11:20
blinky_ghosthi all, can anybody tell me if ironic supports vmware esxi based vms? I'm trying to test tripleo on my lab u using pxe_ssh driver and virt_type=vmware but I have this errors when deploying the overcloud:  File "/usr/lib/python2.7/site-packages/ironic/drivers/modules/ssh.py", line 275, in _set_boot_device  ironic-conductor[1216]: raise NotImplementedError()  ironic-conductor[1216]: NotImplementedError11:20
sambettslucasagomes: yeah, with my changes that we can remove a lot of them :/11:20
lucasagomesnow problem is that, by changing this behaviors we also change the API :-/ not sure if we now should allow things like DELETE /chassis/<uuid>/nodes/<uuid>11:20
sambettsblinky_ghost: I think all supported drivers are here -> http://docs.openstack.org/developer/ironic/deploy/drivers.html11:21
lucasagomesI mean, I wouldn't mind, but that's something that wasn't allowed for older versions and now it is allowed11:21
lucasagomessounds a bit inconsistent11:21
sambettslucasagomes: additional things should be fine though right? as long as we're not removing anything that was supported/11:21
lucasagomesblinky_ghost, I don't think we support vmware esxi11:21
sambetts?11:21
lucasagomesblinky_ghost, BUT, if you use the libvirt driver form ironic-staging-drivers11:22
lucasagomesor virtualBMC that will work11:22
lucasagomesbeucase the use python-libvirt underneath and AFAIK, libvirt works with vmware esxi11:22
* lucasagomes checks11:22
lucasagomesblinky_ghost, yeah, seems so http://libvirt.org/11:22
blinky_ghostlucasgomes: I'm using pxe_ssh driver, which should I use for libvirt?11:23
lucasagomessambetts, right, it won't break anyone, but that discussion may fall into the microversion-ish realm11:23
*** jaoh has joined #openstack-ironic11:24
lucasagomeswhere APIs of the same version (if we don't bump) will be inconsistent across deployments11:24
lucasagomesit's a fine line tho...11:24
lucasagomesbut we also can argue that it's was a bug and should be fixed across all versions... so I don't know, I'm good with it but we need to have a consensus I guess11:24
sambettsblinky_ghost: we're transfering over to virtualBMC, which is a translation layer between IPMI and libvirt, so you can test the real pxe_ipmitool driver11:24
lucasagomesblinky_ghost, right, so the libvirt driver is not in the ironic tree11:25
lucasagomesblinky_ghost, you have to do a "pip install ironic-staging-drivers"11:25
sambettslucasagomes: yeah, its a hard one :/11:25
lucasagomesblinky_ghost, when then you can import "pxe_libvirt_iscsi" or "pxe_libvirt_agent" depending on ur deploy method11:26
aarefievlucasagomes: nova fix here https://review.openstack.org/#/c/316031/11:26
lucasagomesunfortunately we lack documentation for this driver :-/11:26
blinky_ghostlucasgomes I can use pxe_ipmitool also? for vmware?11:26
lucasagomesblinky_ghost, you after you enroll a node with it, you can then look at "ironic node-get-properties" command11:27
lucasagomesblinky_ghost, yes, if you use VirtualBMC to translate the ipmitool commands to libvirt you can11:27
lucasagomesaarefiev, thanks11:27
lucasagomessambetts, yeah, it's just politics. But totally arguable I mean11:27
lucasagomeswe should build the api in the correctly!11:28
sambettslucasagomes: yeah, why do I always managed to get into the political parts of OpenStack XD11:28
lucasagomessambetts, heh there are many11:28
lucasagomesin the correct way*11:28
sambettslucasagomes: I'll leave the not permitted exceptions in place for now11:29
lucasagomessambetts, ++11:29
*** dprince has quit IRC11:29
sambettslucasagomes: although my changes still end up changing DELETE v1/chassis/<uuid>/nodes from operation not permitted to bad request missing node id, because the expose validate catches it before it even makes it into the function11:30
lucasagomes:-/ yeah some corner cases11:32
lucasagomesI wonder if it worth to send an email ML about it, with the api folks. Or at least bring these corner cases up to the next meeting ?11:33
lucasagomessee what people think about it11:33
sambettslucasagomes: right now POST v1/chassis/<uuid>/nodes is operation not permitted, and I think that one would be super useful to enable11:33
sambettslucasagomes: for creating a node belonging to a chassis11:33
lucasagomesyeah to create a node that belongs to a chassis, right now the only way is to pass chassis_uuid in the body request when creating the node directly11:34
sambettsyeah11:34
jrollsambetts: well, I think we decided /v1/chassis/<chassis id>/nodes was duplicate and we didn't want duplicate things11:37
blinky_ghostlucasagomes: to test the virtual bmc, I just run pip install vrtualbmc?11:38
jrollsambetts: lucasagomes: I think the right thing to do here is make a list of bad/inconsistent things, and propose a spec with what to do to fix them11:38
sambettsis it a dup though? because it should autopopulate chassis_id into the node right?11:38
jrolland then we have lots of bikeshedding there11:38
jrollsambetts: I meant with e.g. DELETE11:38
lucasagomesblinky_ghost, yeah, "pip install virtualbmc" that will gives you a "vbmc" command which you can use to register the vms11:38
lucasagomesblinky_ghost, https://github.com/umago/virtualbmc#usage11:39
sambettsjroll: ah, right, well ideally  it shouldn't be a dup because it should validation the node your trying to delete belongs to that chassis11:39
sambettsvalidate*11:39
lucasagomescheck vbmc --help as well for additional parameters such as "--port" (which should be unique for each virtual bmc) etc...11:39
sambettsman ... I'm back to Friday typing hands...11:39
lucasagomesjroll, yeah, that works too11:39
jrollsambetts: sure, I guess11:39
*** jaoh has quit IRC11:41
*** derekh has joined #openstack-ironic11:42
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Set IPMI session timeout for the virtual BMCs  https://review.openstack.org/31602211:43
* lucasagomes goes grab a quick lunch11:45
*** lucasagomes is now known as lucas-hungry11:45
*** baoli has joined #openstack-ironic11:45
*** baoli has quit IRC11:47
*** baoli has joined #openstack-ironic11:47
*** baoli_ has joined #openstack-ironic11:48
blinky_ghostlucasagomes: how do I integrate this with ironic?11:48
*** baoli has quit IRC11:52
*** cdearborn has joined #openstack-ironic11:54
*** sacharya has joined #openstack-ironic11:56
jrollJayF: jcook: you'll be interested in https://bugs.launchpad.net/ironic/+bug/158093111:59
openstackLaunchpad bug 1580931 in Ironic "[RFE] Tooling for recovering from -WAIT and -ING state" [Wishlist,Incomplete] - Assigned to Tan Lin (tan-lin-good)11:59
*** sacharya has quit IRC12:01
*** tloeb has joined #openstack-ironic12:07
mat128jroll: that'd be great ^12:09
jrollyeah12:10
*** links has quit IRC12:14
*** mbound has joined #openstack-ironic12:16
*** clif_h has quit IRC12:17
*** jroll has quit IRC12:17
zhihaisHi, guys, i am using liberty, and trying to build a cloud that have both kvm compute nodes and bare metal nodes. But for bare metal, nova scheduler has its own config like 'host_manager'. How should i setup nova-scheduler to support the co-existance of kvm vms and bare metal node?12:19
*** aarefiev has quit IRC12:20
*** amotoki has joined #openstack-ironic12:20
*** aarefiev has joined #openstack-ironic12:21
*** raildo-afk is now known as raildo12:22
*** jroll has joined #openstack-ironic12:22
*** jroll has quit IRC12:22
*** jroll has joined #openstack-ironic12:22
*** raildo has left #openstack-ironic12:22
*** joprovost has joined #openstack-ironic12:26
*** joprovost has quit IRC12:26
*** alaski is now known as lascii12:29
*** clif_h has joined #openstack-ironic12:30
*** dprince has joined #openstack-ironic12:34
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: grenade checking  https://review.openstack.org/31110912:38
*** fellypefca has joined #openstack-ironic12:39
*** mtanino has joined #openstack-ironic12:41
*** Goneri has joined #openstack-ironic12:41
*** rcernin_ has joined #openstack-ironic12:42
*** rcernin has quit IRC12:42
*** jaypipes is now known as leakypipes12:42
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: OneView client with managers  https://review.openstack.org/28680312:46
*** mtanino has quit IRC12:49
*** lucas-hungry is now known as lucasagomes12:53
*** tloeb has quit IRC12:54
*** e0ne has joined #openstack-ironic12:57
*** jaoh has joined #openstack-ironic12:57
lucasagomeszhihais, hi there, the host manager is fine because it will check for the hypervisor type when doing the sched things12:58
lucasagomeszhihais, but other stuff such as tweaking the {ram,disk,cpu}_allocation_ratio etc will really depend on ur use case12:58
lucasagomesif you ended up seeing that using one n-sch does not fit into ur model, I believe you will have to use nova cells to separate the virtual machines and the baremetal ones12:59
lucasagomesand then you configure the scheduler differently on each cell12:59
*** ayoung has joined #openstack-ironic13:00
*** amotoki has quit IRC13:00
*** alexpilotti has quit IRC13:02
*** piet has joined #openstack-ironic13:02
*** chopmann has joined #openstack-ironic13:04
openstackgerritVasyl Saienko proposed openstack/ironic: Add tests for cleaning  https://review.openstack.org/30446913:06
*** alexpilotti has joined #openstack-ironic13:06
*** tloeb has joined #openstack-ironic13:09
*** rbudden has joined #openstack-ironic13:10
*** dprince has quit IRC13:11
*** dprince has joined #openstack-ironic13:11
*** rcernin_ has quit IRC13:16
mgouldlucasagomes, I thought you needed cells to have VM and BM in the same installation?13:16
lucasagomesmgould, I don't think you required it, you can use host aggregates to separate both types and use the same scheduler13:17
lucasagomesI believe it's mostly up to the business logic, say for example, that for baremetal you must use the exact filters13:18
vsaienkomorning dtantsur, lucasagomes, jroll13:19
jroll\o vsaienko13:20
lucasagomesmgould, then you will need a dedicated scheduler for the baremetal loads (since the vms will use different filters)13:20
vsaienkojroll once you have a time could you please review https://review.openstack.org/#/c/304473/13:20
vsaienkothe patch adds experimental job to test cleaning at the gates13:21
*** odyssey4me has quit IRC13:21
*** odyssey4me has joined #openstack-ironic13:21
NobodyCamgood morning Ironicers13:22
jrollvsaienko: one comment, else lgtm13:22
jrollhey NobodyCam13:22
lucasagomesvsaienko, IRONIC_VM_SPEC_DISK=1 works?! Casue IPA will look for >=4 GB no?13:22
lucasagomesI might be wrong13:22
jrolloh good point13:22
NobodyCamhey hey jroll :)13:22
jrollwe could lay down root device hints no?13:22
NobodyCamoh ya TGIF!!!13:22
lucasagomesjroll, yup13:23
lucasagomeschecking our devstack/lib/ironic13:23
vsaienkojroll, lucasagomes it has been already fixed https://github.com/openstack/ironic/commit/3295a7e8ec0312b84694a02aaf4f1d6a0228d6e313:23
mgouldNobodyCam, morning!13:23
lucasagomesvsaienko, a-ha!13:24
lucasagomesok13:24
NobodyCamgood morning mgould :)13:24
jrollvsaienko: oh neat13:24
*** amotoki has joined #openstack-ironic13:24
openstackgerritMerged openstack/virtualbmc: Add unittests for the config.py module  https://review.openstack.org/31602113:24
jrollvsaienko: yeah just the one comment then13:24
NobodyCamand lucasagomes too13:24
persiaNobodyCam: So, this is the kind of Friday that is not usually considered ideal13:24
lucasagomesNobodyCam, hi there13:24
vsaienkojroll fixed13:25
jrollthanks13:25
NobodyCampersia: ??? may I ask why.. (and good morning)13:25
*** vmud213 has quit IRC13:26
* lucasagomes remember to update the RFE about root device hints... 13:26
* lucasagomes does that13:26
*** rcernin has joined #openstack-ironic13:27
lucasagomesbtw folks, if you have some spare time here's a quick patch https://review.openstack.org/#/c/315593/13:27
lucasagomesplumbing work for gating the ironic job against the virtualbmc gate13:27
persiaNobodyCam: I don't actually know.  WIkipedia claims it is a 19th century conflation of the misfortune of the numeric value of the day of the month and the day of the week, surrounding some obscure christian myths, but there are other origin stories going back to the 14th century.13:27
lucasagomesto make sure we won't break anything13:27
mat128lucasagomes: not that I'm stalking you, but reviewed it already :)13:28
lucasagomesmat128, a-ha! awesome thank you a lot!13:28
*** absubram has joined #openstack-ironic13:28
NobodyCampersia: oh wow. I had no idea.13:29
NobodyCammat128: there is always https://review.openstack.org/#/c/272658 which could use s few eyes :p13:30
jrollpersia: hehehe13:30
persiaFor me, it's a fun coincidence that happens every six or seven months (usually), but reading "TGIF" today was particularly amusing.13:30
NobodyCam:p13:31
*** absubram_ has joined #openstack-ironic13:32
*** jcoufal has joined #openstack-ironic13:32
*** absubram has quit IRC13:32
*** absubram_ is now known as absubram13:32
*** wajdi has quit IRC13:33
*** wajdi has joined #openstack-ironic13:33
NobodyCamlucasagomes: +213:34
chopmannhi hi folks, someone from Ironic Projetc here? We (tloeb, chopmann, jaoh) have some question regarding ironic development... Anyone got around 10-20 Min time? :-)13:34
lucasagomesNobodyCam, cheerios13:35
*** wajdi_ has joined #openstack-ironic13:39
*** athomas has quit IRC13:41
*** wajdi has quit IRC13:43
*** openstack has joined #openstack-ironic13:46
*** jcoufal has quit IRC13:47
chopmannwe just saw the team meeting is on monday.  The "Open Discussion" Block is for that sort of things right?13:47
*** athomas has joined #openstack-ironic13:48
*** parlos has quit IRC13:48
mgouldchopmann, kinda13:48
chopmannmgould: we are planning to build a Network Devices Provisioner. Most Switches/Routers use dhcp+tftp for Zero Touch Config/ Initial Provisioning13:48
mgouldOK13:49
chopmannwe dont know if ironic is the right place for such a feature13:49
*** mtanino has joined #openstack-ironic13:49
mgouldchopmann, that sounds like more of a Neutron feature to me13:49
mgouldbut I'm not very familiar with Neutron, so I could be totally wrong13:49
*** joprovost has joined #openstack-ironic13:50
mgouldgenerally Ironic only provisions baremetal compute nodes13:50
mgouldbut sure, I guess you could treat your switches as baremetal nodes and provision them with Ironic13:50
chopmannit's less about setting up networks...we want to do the initial config on some OpenFlow switches so they can connect to the Controller13:50
mgouldoh, OK13:50
mat128chopmann: it could be done13:51
jrollI would love for ironic to provision network gear13:51
mat128chopmann: in our internal backlog of things we have something similar to this13:51
jrollswitches, firewalls, etc13:51
mat128yup13:51
chopmannyup something like it13:51
sambetts++ for deploying network gear13:51
jaohcool13:51
mgoulddoes OpenStack currently have anything for deploying network gear, or does it have to be done manually?13:52
chopmannnothing we know of13:52
mat128mgould: I don't think theres anything to *deploy* network gear13:52
sambettsIronic + https://en.wikipedia.org/wiki/Converged_infrastructure == Win13:52
*** openstack has joined #openstack-ironic13:54
mgouldsounds like chopmann already has network gear that can boot from an image over the network13:55
chopmannsort of13:55
*** ChrisAusten has joined #openstack-ironic13:55
chopmannyou can boot them up, and the get the config from the tftp server13:55
mgouldah, OK13:55
chopmannor you can "tell them" to update the switch.13:56
mgouldcan we treat "switch config file" as an image format in Glance?13:56
chopmannthat sort of stuff. BlackBox Switches13:56
mat128hook that into -inspector and you have datacenter profiling13:56
mat128mgould: I dont think you would want to have a static config in glance13:56
mat128depending on what is connected13:56
*** ChrisAusten has quit IRC13:57
mgouldmat128, oh, good point13:57
mat128you might want to use something like https://github.com/internap/netman to control the switch once it's up and running13:57
chopmannsomething more like a template based --> say something like a C***O Switch Family13:57
*** sacharya has joined #openstack-ironic13:57
mgoulddoes neutron assume you've already configured your physical network infrastructure?13:57
sambettstbh, unless your deploying an OS image onto a blank switch I don't think Ironic is the right tool for the job13:57
chopmannfor whitebox switches, Ironic fits the bill13:58
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: grenade checking  https://review.openstack.org/31110913:58
chopmann(sort of)13:58
mat128mgould: https://github.com/openstack/networking-generic-switch/13:58
mat128would make neutron use the switch13:58
mat128but for initial provisioning13:58
mgouldmat128, thanks!13:58
sambettsmat128, mgould13:58
mat128I planned to integrate netman in networking-generic-switch13:58
mgouldchopmann, first time I've heard "whitebox" used like that, but I like it :-)13:59
*** derekh has quit IRC13:59
sambettsor you could use any of our (Cisco) ml2 drivers13:59
sambettsfor our switches13:59
mgouldor other vendor-specific drivers, if you'd bought someone else's gear in a moment of craziness? :-)13:59
sambettsits an interesting problem, because it depends if your wanting to configure undercloud/OOB switches or overcloud switches13:59
mgouldpresumably "I've just plugged a new rack into my datacenter, please add it to my overcloud" is an important use case14:00
sambettsnot sure if you can run neutron in a tripleo undercloud14:00
sambettsI assume you can14:00
mat128sambetts: Oh true, you're from Cisco :)14:00
sambetts;)14:00
mat128but yeah, undercloud vs overcloud14:01
mgouldsambetts, I think you can run neutron in the undercloud, let me check...14:01
*** dansmith is now known as superdan14:02
*** joprovost has quit IRC14:02
*** joprovost1 has joined #openstack-ironic14:02
sambettsif you can run neutron in your undercloud, then you can use your undercloud to bootstrap everything into your overcloud via ml2 I would expect14:02
*** sacharya has quit IRC14:02
*** mjura has quit IRC14:03
sambetts(not tried personally)14:03
*** ametts has joined #openstack-ironic14:04
* mgould spins up a tripleo box14:04
*** joprovost1 is now known as joprovost14:05
chopmannwe want to host some SDN Controllers in OpenStack and use "Ironic" to provision/configure the HW-Switches to talk to the SDN controllers inside openstack14:05
chopmann:-)14:05
mat128thats good14:05
mgouldsambetts, don't you need neutron in the undercloud for deploying Ironic nodes?14:06
* mgould thought we relied on neutron's dhcp service for that14:06
sambettsmgould: good point!14:06
mat128chopmann: I think it fits the bill14:06
mariojvgood morning yuriyz - i had a question about your comment on https://review.openstack.org/#/c/226058/15/ironic/conductor/manager.py about the "or get driver status"14:06
*** absubram has quit IRC14:06
mariojvit looks like current master has that particular comment: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L28214:06
*** absubram has joined #openstack-ironic14:06
yuriyzhi mariojv14:06
mariojvso i am not sure whether i should remove it14:07
mgouldmat128, chopmann so the only problem is that the "image file" is actually a config file for the switch, which we might want to generate dynamically?14:07
chopmannjep14:07
mat128chopmann: if you switches are on PDUs and/or have IPMI, you will be able to power control them14:07
mat128then you can add them a port, for the management MAC address and get them to fetch their config via TFTP14:07
yuriyzthis is not directly related to your change, but looks like artefact of old code14:07
jrollhonestly how I would do it is a base "kick" config file that ironic puts down, and then use neutron from there14:08
mat128mgould: I think the "image file" would only be the base setup for it to be reachable14:08
mariojvok - i may as well remove it then since i'm editing the docstring14:08
mariojvthank you14:08
mgouldjroll, so just enough for the switch to talk to Neutron?14:08
mat128mgould: then they enroll the switch in their SDN controller *or* neutron14:08
mgouldnice14:08
sambettsso I think a custom deploy driver in Ironic would work that doesn't do image deployment14:08
jrollmgould: enough for neutron to configure the switch correctly, yeah14:08
mgouldyeah, that shouldn't change too often14:08
sambettsbut it wouldn't be able to be driven from nova14:08
yuriyzmariojv yes better remove it14:08
mat128sambetts: thats correct14:08
jrollsambetts: why not?14:09
jrollconfig file could be a raw image14:09
mat128sambetts: nova boot --flavor=Cisco3750 --image=iOS ?14:09
mgouldsambetts, personally I'm happy to consider a config file as code for a *very* high level instruction set architecture :-)14:09
mat128;)14:09
jroll(which is actually just text)14:09
mariojvi will put the word "synchronously" back again as well, since that's the default behavior14:09
sambettsmat128: for that I think it would work, but for a config file?14:09
mat128sambetts: --image=my_basic_template ?14:10
jrollsambetts: mv my_config.txt my_config.img && glance create --file my_config.img14:10
mat128then maybe use config drive data to assign the management ip?14:10
mat128jroll: oh god14:10
mat128so simple14:10
chopmannthe base config file tells the switch --> go to IP xxx.xxx.xxx.xxx (SDN Controller), wich OpenFlow Version it should speak and so on...14:10
mat128why not14:10
jrollmat128: :)14:10
sambettsewwww, but  ok :-P14:10
mat128well14:10
mat128images don't have to be x86 images14:10
mat128we already have whole disk images, partition images, etc14:11
* mgould points you all at http://langsec.org/papers/Bratus.pdf, which is about treating input languages as code for implicit abstract "weird machines"14:11
mgouldvery cool and rather scary stuff14:11
mat128mgould: there's this one guy I was interviewing who considered literally *everything* as data transformation14:12
mat128we didnt hire him,but I still think of this all the time14:12
sambettsthen a custom deploy driver to set the right DHCP_OPTs and a weird config image should work today I guess14:12
mgouldmat128, yeah, that can be a very powerful perspective14:13
mgouldnot always the most appropriate one, though :-)14:13
chopmannsambetts: we'll have a look at that14:13
mat128mgould: I think he used to write switch/firewall firmwares, so I think he took it from there14:13
mgouldmat128, right, I can see that14:14
mat128mgould: He wasn't very comfortable with anything high level, unfortunately14:14
mgouldmultiple levels of interpretation14:14
*** absubram has quit IRC14:14
mgouldheh, I have the opposite problem :-)14:14
lucasagomesthe provisioning network gear makes me think of www.onie.org14:14
mat128lucasagomes: interesting, goes back to "whitebox" switches that chopmann was mentioning14:15
lucasagomesyeah14:15
jaohonie uses the same protocols14:15
jaohit could also be done via tftp dhcp14:15
mgouldlucasagomes, oh, cool14:15
mat128lucasagomes: I remember seeing Cumulus Networks' demo in Vancouver where neutron-agent was simply running on their switches14:15
sambettsmat128: ^^ thats who I was reminded of by this convo14:16
*** jcoufal has quit IRC14:16
lucasagomesmat128, oh sweet14:16
mat128lucasagomes: It was literally a linux box, running specialized hardware14:16
mgouldmat128, cool! I guess that's a lot more efficient than VM -> physical switch -> virtual switch -> physical switch -> second VM...14:17
mat128https://twitter.com/MathieuMitchell/status/60038716668698214414:17
mat128before I knew he was from Cumulus and he did a demo in the PM14:17
*** vishwanathj has joined #openstack-ironic14:17
mgouldmat128, hahah14:18
sambettshahaha brilliant!14:18
mat128mgould: apparently it was very hard to get the box through customs14:18
lucasagomeslol14:18
NobodyCammat128: lol14:19
*** links has joined #openstack-ironic14:19
* sambetts feels sorry for the tech reviewers that got nivida gtx1080s for review at the press release and then had customs unbox them for them on the way home 14:19
mgould:-(14:20
chopmannwe'll have a look at the info you guys gave us and probably come to the meeting and hopefully we get a slot on "Open Discussion" :-)14:20
*** fellypefca has quit IRC14:20
chopmannty all!14:20
mat128chopmann: make sure you stay around, I'm very interested by your project14:20
openstackgerritMario Villaplana proposed openstack/ironic: Allow vendor drivers to acquire shared locks  https://review.openstack.org/22605814:20
sambettslooking forward to it, /me interested too14:20
mat128Have to leave for a few hours14:21
*** mat128 is now known as mat128|away14:21
*** joprovost1 has joined #openstack-ironic14:21
sambettso/ mat128|away14:21
JayFjroll: ty for the bug link14:22
wajdi_sambetts: "customs unbox them for them on the way home"....that's brutal!14:23
*** joprovost has quit IRC14:23
*** joprovost1 is now known as joprovost14:23
vsaienkojroll, lucasagomes could you please have look one more time https://review.openstack.org/#/c/304473/. I have missed to add some parameters after rebasing to master14:23
* jlvillal reads the backlog and would be interested to see actual hardware for 'whitebox' switches. Or for ONIE stuff14:23
lucasagomesvsaienko, sure14:24
JayFI would really like some ironicers to review https://review.openstack.org/#/c/313800/ -- as much as we've  talked about wanting the gate to be tinyipa, this has been up two days with no reviews to make the change14:24
*** jaybeale has joined #openstack-ironic14:25
*** piet has quit IRC14:25
*** piet has joined #openstack-ironic14:26
vsaienkojroll, lucasagomes thanks14:26
jaohhi jlvillal are you looking for something like this14:27
jaohhttps://www.deltacomputer.com/edge-core-as4610-30p-poe-bms.html14:27
jrollkrotscheck: lucasagomes: is it possible for one of you to invert this for me? it's going on a black background https://wiki.openstack.org/wiki/File:Bear_metal.svg14:27
JayFanyone else seen gate failures like http://logs.openstack.org/35/313035/6/check/gate-tempest-dsvm-ironic-agent_ssh/698f36c/logs/screen-ir-cond.txt.gz#_2016-05-13_10_04_54_58214:27
jroll("open up gimp and do X" is acceptable)14:27
openstackgerritChris Krelle proposed openstack/ironic: Add support for the audit middleware  https://review.openstack.org/27265814:27
jrollJayF: yep https://github.com/openstack/ironic-python-agent/commit/085e622945e273cc759ff4bff7c08d15fc1e15c614:27
lucasagomesjroll, yup, 1 sec14:27
lucasagomeslooking at JayF's patch14:27
JayFjroll: so that means my ramdisk defaults change is impacting the gate14:27
JayFhttps://review.openstack.org/#/c/313035/14:28
jrollsambetts: hmm http://logs.openstack.org/08/085e622945e273cc759ff4bff7c08d15fc1e15c6/post/ironic-python-agent-buildimage-tinyipa/27deb9e/console.html14:28
JayFjroll: because agent_ssh shouldn't have been running with tinyipa14:28
*** jaoh has quit IRC14:28
sambettsjroll: yeah I was just trying to work out why that didn't build14:28
jrollJayF: are you sure?14:28
JayFI'll make sure it's tinyipa that's booting there14:28
jrollit is14:28
JayFthen my patch is wrong14:28
*** d0ugal has quit IRC14:29
*** links has quit IRC14:29
jrollwhat should agent_ssh be using?14:29
JayFproject-config should've isolated the defaults change from the gate14:29
JayFagent_ssh should be using coreos until my project-config change lands14:29
jrollah14:29
JayFbasically trying to split "devstack default" from "gate config"14:29
jrollJayF: yep, project-config matches what you say14:30
JayFbut obviously that's not what's happening14:30
JayFmaybe a devstack bug as much as anything else?14:30
JayFor perhaps it's like ipxe, with the default built in14:30
JayFI'll figure it out14:30
*** d0ugal has joined #openstack-ironic14:30
jrollp-c looks right14:30
jrollJayF: OH14:31
openstackgerritSam Betts proposed openstack/ironic-python-agent: Make sure tinyipa du takes into account branch  https://review.openstack.org/31612414:31
jrollyou have export IRONIC_RAMDISK_TYPE={ramdisk-type}14:31
sambettsjroll: ^^^^^14:31
JayFwell *I* don't, lol14:31
jrollshoulc be export DEVSTACK_LOCAL_CONFIG+=$'\n'"..."14:31
jrollno never mind I'm an idiot14:32
JayFthat's been configured that way in the gate14:32
JayFfor forever maybe14:32
jrollat any rate, that isn't making it to the gate for the thing you linked14:32
jrollyeah I missed the next line14:32
sambettsjroll, JayF: we need this to pass post https://review.openstack.org/316124 :(14:32
JayFjroll: ^ I already +214:33
jroll+A14:33
jrollsigh14:33
sambettsmucho sigh...14:33
sambettsagain not caught by the gate because that branch value is set in our normal gate14:34
mgouldJayF, does agent_ssh require PXE?14:34
mgouldalso libvirt, those seem to be the two that you haven't changed ipxe-enabled on14:35
JayFmgould: agent_ssh is using the old bash ramdisk14:35
mgouldah, gotcha14:35
JayFmgould: also I was aksed to keep one job running normal-pxe14:35
JayFmgould: libvirt, I honestly had nfi what that job was or did so I left it be14:35
mgouldyeah, that makes sense14:35
lucasagomesjroll, https://wiki.openstack.org/wiki/File:Bear_metal_white.svg14:36
mgouldI think you could probably run tinyipa on that one too14:36
lucasagomeslooks like a white square but the bear is there14:36
*** jcoufal has joined #openstack-ironic14:37
jrolllucasagomes: thanks man14:37
*** d0ugal has quit IRC14:38
jlvillaljaoh thanks!  Though I think you left14:40
jlvillallucasagomes, So is the movie Sing Street popular where you are?14:41
jlvillalvdrok, You have spent more time looking in the devstack code than me. Do we ever not set this to true?  https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L9214:43
mgouldJayF, left some comments/questions14:43
JayFmgould: ty14:43
mgouldmost can probably be answered with "it's fine, you've just misunderstood how jobs are configured"14:43
lucasagomesjlvillal, hi there! hmmm never heard of it heh14:43
* lucasagomes looks14:43
jlvillallucasagomes, Just wondering. It is supposed to be about a fictional Dublin band...14:44
* jlvillal will try to drag his wife to go see it this weekend14:44
lucasagomesjlvillal, oh right, yeah I don't know... I don't think it's very popular here14:44
JayFmgould: well, I'm not sure I understand how they work now, given my patch broke so bad (to change devstack defaults14:44
jlvillallucasagomes, Thanks :)14:44
mgouldJayF, heh14:44
lucasagomesjlvillal, OTOH the commitments is14:44
lucasagomesjlvillal, https://en.wikipedia.org/wiki/The_Commitments_%28film%2914:45
jlvillallucasagomes, That is a good movie :)14:45
* mgould hasn't heard of Sing Street either, but it looks fun14:45
vdrokjlvillal: setting to false will disable vm creation and network setup I guess14:45
jlvillalvdrok, Yeah I was just wondering when that would be a valid case.14:45
lucasagomesjlvillal, yeah... if you are looking for irish movie/tv stuff you may want to check "love/hate"14:45
lucasagomesor "the fall"14:46
lucasagomesjlvillal, https://en.wikipedia.org/wiki/Love/Hate_%28TV_series%2914:46
jlvillallucasagomes, Thanks.  Or is that: go raibh maith agat14:46
* jlvillal thinks it looks complicated to say 'thank you' in Irish.14:46
lucasagomesheh yeah I don't even know how to pronounce that14:47
jlvillalMe either! :)14:47
* lucasagomes knows very few irish words14:47
jlvillallucasagomes, beoir  ?14:47
jlvillal= beer14:47
vdrokjlvillal: the only case I can recall right now is ironicclient functional tests job - https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L4314:47
*** mbound has quit IRC14:48
lucasagomesjlvillal, hah yeah or slainte14:48
vdrokas we just do the api calls14:48
jlvillalvdrok, Ah so there is at least one case. That is good.14:48
lucasagomesmadra rua = red dog = fox14:48
lucasagomesonly random stuff heh14:48
jlvillal:)14:48
* mgould has a friend who jokes that H in Gaelic has a silent ^ before it14:48
* jlvillal better get ready to go to work...14:48
mgouldso go raib^H mait^H agat14:49
mgouldgets you reasonably close, AIUI14:49
JayFjroll: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L683 requires deploy-with-ipa to be true to set ramdisk type. https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L808 has it set to false, as does agent_ssh and agent_ipmitool14:49
jrollO_o14:49
JayFjroll: I'm thinking we should unconditionally set the RAMDISK_TYPE14:49
JayFjroll: unless it causes harm for other setups?14:49
jrollJayF: curious what that does for bash ramdisk14:50
jrollbut that's my only concern14:50
JayFwhich job is supposed to be bash ramdisk?14:50
mgouldwhen are we EOLing the bash ramdisk?14:50
JayFpxe_ssh?14:50
JayFif so I'm 99% sure it's not running bash ramdisk today14:50
jrollyes, pxe_ssh14:50
mgouldrename it to *_bash_ramdisk* for greater clarity, maybe14:50
jrollwat14:50
jrollsince when14:50
JayFIDK; but my patch wouldn't have modified the behavior of the pxe_ssh job14:51
JayFif it had been running the bash ramdisk14:51
JayFbecause that's a completely diff codepath14:51
JayFgoing to doublecheck the logs14:51
JayFoh, agent_ssh was my failure14:51
lucasagomesJayF, in ur patch you may end up passing an empty IRONIC_RAMDISK_TYPE= no?14:51
lucasagomesfor stable/{kilo, liberty}14:51
JayFlucasagomes: yes, and that's desired, so it uses the default14:51
JayFlucasagomes: since for k and l we only had 1 ramdisk14:51
JayFlucasagomes: otherwsie k and l jobs would fail post-change with inability to fetch ipa14:52
lucasagomesJayF, strange... I mean, yeah it should use the default but export an empty one seems odd14:52
JayFyeah jroll this isn't as bad as I thought, one character fix14:52
lucasagomesbetter to not export anything, right?14:52
JayFlucasagomes: I think that's what I'm doing, right?14:52
JayFif you think not please put it on the patch so the knowledge isn't lost14:52
JayFI'll have to rebase it on this fix anyway14:52
jrollJayF: cool14:53
lucasagomesJayF, ish yeah, https://review.openstack.org/#/c/313800/3/jenkins/jobs/devstack-gate.yaml (see L89)14:53
lucasagomesI would just put that inside the if conditional14:53
lucasagomesor get rid of it and make it a export DEVSTACK_LOCAL_CONFIG+=$'\n'"IRONIC_RAMDISK_TYPE=${ramdisk-type}"14:53
lucasagomesinside the if14:53
lucasagomesL68914:54
JayFoh, gotcha14:54
lucasagomeswhat I mean is, $IRONIC_RAMDISK_TYPE may be unbonded14:54
JayFgood catch14:54
JayFjust put it on a comment and I'll fix it with the next patchset14:54
JayFI see what you mean sinec opening the code14:54
lucasagomesJayF, done14:54
JayFjroll: https://review.openstack.org/#/c/316138/14:55
*** rama_y has joined #openstack-ironic14:56
jrollJayF: thanks14:57
JayFonce tests pass I'll go beg for reviews from infra14:57
JayFand once this lands and takes effect, we can recheck my devstack defaults change patch14:57
JayFI thought that changing those defaults might unearth some craziness :)14:58
*** sacharya has joined #openstack-ironic14:58
lucasagomesjroll, btw, re root device hints operators... should we just wait for the oslo spec matcher thing and use it instead?14:58
jrolllucasagomes: possibly/probably?14:59
lucasagomesit's hard to map the other because eq , ==, s== ?!14:59
jrollyeah, insanity14:59
lucasagomesthe type awareness of the nova spec match Vs. the non-type aware of inspector makes it crazy14:59
lucasagomesyeah14:59
lucasagomesdtantsur, ^ btw, re mapping the root device hints operators with inspector and nova15:01
lucasagomesseems like a overcomplex interface, I would rather chose 115:02
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576615:03
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614915:03
*** d0ugal has joined #openstack-ironic15:05
*** dmk0202 has quit IRC15:13
*** openstackgerrit has quit IRC15:17
*** openstackgerrit has joined #openstack-ironic15:18
blinky_ghostanyone can help me with this error when I'm trying to deploy overcloud: "No configuration found for any of the present NICs"? thanks15:31
*** absubram has joined #openstack-ironic15:31
*** mgould has quit IRC15:38
*** yarkot has quit IRC15:43
*** anush has quit IRC15:44
*** absubram has quit IRC15:47
*** mbound has joined #openstack-ironic15:48
*** anush has joined #openstack-ironic15:49
*** mgould has joined #openstack-ironic15:49
*** ifarkas has quit IRC15:51
JayFjroll: that project-config change is landing now, if you don't mind, recheck my devstack patch once it takes effect so it can land? I put explicit instructions on how to make sure the changes took effect in the gerrit comment15:51
* JayF off the rest of the day, see you on on Monday o/15:51
sambettso/ JayF15:51
*** mbound has quit IRC15:53
*** MattMan has quit IRC15:54
*** MattMan has joined #openstack-ironic15:54
*** tesseract has quit IRC15:56
*** absubram has joined #openstack-ironic15:56
* leakypipes shakes fist at JayF for causing him many IRC highlights... :)15:57
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add tests for openstackclient plugin  https://review.openstack.org/30168715:57
*** jcoufal has quit IRC16:00
blinky_ghostlucasgomes: can I compile this https://github.com/umago/ironic-staging-drivers-packaging/blob/master/openstack-ironic-staging-drivers-0.1.0-1.src.rpm to tryout libvirt? thanks16:01
*** alexpilotti has quit IRC16:01
openstackgerritgreghaynes proposed openstack/bifrost: Add bonding support to bifrost  https://review.openstack.org/31616916:01
*** rcernin has quit IRC16:02
*** alexpilotti has joined #openstack-ironic16:04
mgouldsambetts, just ran `openstack service list` on my undercloud VM, and neutron is indeed there :-)16:07
sambettsmgould: sweet :)16:07
openstackgerritMerged openstack/virtualbmc: Bump pyghmi version to 1.0.3  https://review.openstack.org/31602316:11
sambettsmgould: your still better at the englishing than I am XD16:14
mgould:-)16:15
mgouldsambetts, short sentences Are Your Friends16:16
sambettsheh16:16
lucasagomesblinky_ghost, sorry missed ur ping16:17
lucasagomesblinky_ghost, so the libvirt driver is part of the 0.2.0 version16:17
* lucasagomes needs to change his nick16:17
* mgould wishes he'd watched https://www.youtube.com/watch?v=C9-o1gLHHWo six months ago :-/16:18
blinky_ghostlucasagomes: how about this error?  "No configuration found for any of the present NICs"? I'm testing with fake_pxe driver16:18
lucasagomesblinky_ghost, do you have any port registered in the node?16:19
blinky_ghostlucasagomes: how do I check that? ironic node-show?16:19
lucasagomesblinky_ghost, ironic port-list16:20
blinky_ghostlucasagomes: Yes I have 2 ports, I'm testing with 2 nodes. One of them, the compute node booted correctly, this one is stuck here, and the disk seems corrupted: "error: not a correct XFS inode".  grub rescue16:22
lucasagomesblinky_ghost, right, because that error is coming from the node booting not being able to find the [i]PXE configuration option16:22
lucasagomesblinky_ghost, the mac address of the port is correct? Does it match with the node being booted?16:23
blinky_ghostlucasagomes: yes, the port is correct16:23
blinky_ghostlucasagomes: I did the exactly same test with OSP 8 and it works fine, could be a bug I guess16:24
*** e0ne has quit IRC16:24
openstackgerritMerged openstack/ironic-python-agent: Make sure tinyipa du takes into account branch  https://review.openstack.org/31612416:25
klindgrenSo I am working on getting introspection to actually use lldp to report the switches that it is connected to vs's none.  I am noticing some work currently in progress that it looks like ironic is trying to standardize on a dict for ports attached to switches.16:25
klindgrenone of those items appears to be the switchid? which looks to be nothing more than the switches mac-address?16:26
lucasagomesblinky_ghost, right, when deploying can you check if the configuration exist under /httpboot/pxelinux.cfg/<mac addresses> ? (Or if you are using devstack /opt/stack/data/ironic/httpboot/pxelinux.cfg/...)16:26
blinky_ghostlucasagomes: it's empty16:26
lucasagomesblinky_ghost, right, but there's no node in "wait call-back" or "deploying" right?16:27
lucasagomescause it will be created at that stage16:27
lucasagomesand cleaned up if the deployment fails etc...16:27
klindgrenSo if siwtch_id == switch mac addy(eg: xx:xx:xx:xx:xx:xx), switch_info == switch name(eg: switch.fqdn), port_id == switch port (eg eth1/101/5)16:28
blinky_ghostlucasagomes: I start the node by hand when it asks for wait call-back, it start's deploying then it shutdowns the nodes, I see a message "emergency shutdown". After this, the wait call-back never shows up again, so I start them by hand, and that's where I see the error.16:28
*** fragatin_ has joined #openstack-ironic16:29
sambettsklindgren: So the format is switch_id == mac_addr/openflow_id, switch_info == any string that can be customisable per vendor, and port_id == switch port16:30
lucasagomesblinky_ghost, oh ok, so deployment is failing... that's why after it failed16:30
lucasagomesblinky_ghost, ironic will remove the PXE configs, so after you restart it manually it will not boot anymore16:30
lucasagomesblinky_ghost, so, we need to figure out why it's failing in the first place. Any hints in the ironic-conductor log16:30
lucasagomes?16:30
sambettsklindgren: if your planning to tackle this work, please look at the existing work, and also at the comment my colleage Robert Li left on the existing work16:30
*** sabeen has joined #openstack-ironic16:31
blinky_ghostlucasagomes: in OSP8 I don't see the emergency shutdown, actually it never shutdowns the node, I have to shut it down by hand as it's fake_pxe16:31
blinky_ghostlucasagomes: I'll redeploy and check the the ironic-conductor.log16:32
*** fragatina has quit IRC16:32
lucasagomesblinky_ghost, ok, and you are using the same image for both deployments ?16:32
lucasagomesramdisks and tenant image?16:32
sambettsklindgren: we have this working down stream now, and switch_info is designed allow for vendor specific info, because of that the IPA side of it needs to do very little processing of the lldp packet, allowing plugging of vendor specific procssing hooks into inspector to handle different vendors switch processing16:33
blinky_ghostlucasagomes: no, I built one to Centos, from OSP8 I've installed a rpm package16:33
lucasagomesblinky_ghost, right so you built a new image for the tenant? Which is a CentOS based one?16:34
blinky_ghostlucasagomes: can I copy deploy-ramdisk-ironic.kernel deploy-ramdisk-ironic.initramfs from OSP8 and use from Centos?16:34
*** athomas has quit IRC16:34
blinky_ghostlucasagomes: Yes, correct16:35
lucasagomesblinky_ghost, you can yes16:35
lucasagomesblinky_ghost, so to build the tenant image, it may worth taking a look at http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements16:35
lucasagomese.g, if you want to use local boot, make sure grub2 is installed in the iamge16:35
lucasagomesimage*16:35
lucasagomessame for EFI, make sure the modules are present in the image16:35
lucasagomesand things like that16:35
blinky_ghostIucasagomes: Can I use everything  from OSP8 except overcloud-full.qcow2 to test or is better to buld a new one?16:36
klindgrensambetts, I am curious what one would do with the switch_id?  Which neutron mechanism driver uses that as identification for the switch that a change needs to be made on?16:38
lucasagomesblinky_ghost, its really up to you, for testing you can even use the cirros image16:39
lucasagomeswhich is tiny :-)16:39
lucasagomes(that's what we use in gate as well)16:39
lucasagomesblinky_ghost, what I mean is, if you want to build ur image just make sure it contains all the required bits (following http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements)16:40
sambettsklindgren: I can't comment on which specific ml2 drivers use mac-addr or open-flow id, because ours doesn't it uses the mgnt ip, which we include in switch_info, which we have as a string encoded json16:40
blinky_ghostlucasagomes: OK, I'll test both ways thanks :)16:40
lucasagomesnp :-)16:40
sambettsklindgren: I've campained to make it a generic string, but I was fought against16:41
*** jistr has quit IRC16:42
*** Goneri has quit IRC16:44
lucasagomesok folks, I'm calling it a day16:44
lucasagomeshave a great evening all, enjoy the weekend!16:44
lucasagomestalk to you all on monday :-)16:44
*** daemontool_ has joined #openstack-ironic16:45
*** lucasagomes is now known as lucas-afk16:45
sambettso/ lucas-afk16:45
lucas-afksambetts, see ya!16:45
NobodyCamhave a good night16:45
*** kromanenko_ has quit IRC16:47
*** kromanenko__ has quit IRC16:47
*** daemontool has quit IRC16:48
*** tloeb has quit IRC16:52
*** chopmann has quit IRC16:54
*** mgould has quit IRC16:56
*** romcheg has quit IRC16:56
*** kozhukalov has quit IRC16:56
*** agordeev has quit IRC16:56
*** yonglihe has quit IRC16:56
*** rajinir has quit IRC16:56
*** evgenyl has quit IRC16:56
*** sergek has quit IRC16:56
*** ChrisMead has quit IRC16:56
*** electrofelix has quit IRC16:59
*** catintheroof has joined #openstack-ironic16:59
*** catintheroof has quit IRC17:00
*** catintheroof has joined #openstack-ironic17:00
*** alexpilotti has quit IRC17:01
*** alexpilotti has joined #openstack-ironic17:03
*** fragatin_ has quit IRC17:07
*** ijw has joined #openstack-ironic17:08
blinky_ghostlucasagomes: after copying ironic-python-agent.initramfs ironic-python-agent.kernel overcloud-full.initrd overcloud-full.vmlinuz from OSP817:10
blinky_ghostit works fine17:10
*** ohamada has quit IRC17:10
blinky_ghostlucasagomes: it uses overcloud-full.qcow2 from centos and the rest from OSP8 so it's some kind of bug on ramdisk or something17:11
*** rajinir has joined #openstack-ironic17:12
*** kozhukalov has joined #openstack-ironic17:14
*** agordeev has joined #openstack-ironic17:14
*** evgenyl has joined #openstack-ironic17:15
*** amotoki has quit IRC17:21
*** romcheg has joined #openstack-ironic17:22
*** Sukhdev has joined #openstack-ironic17:25
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: [WIP] Tempest: add basic test  https://review.openstack.org/29392617:25
*** ijw has quit IRC17:25
*** _milan_ has quit IRC17:28
*** athomas has joined #openstack-ironic17:28
openstackgerritSam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances  https://review.openstack.org/27785317:28
*** yonglihe has joined #openstack-ironic17:29
* sambetts is heading off, see everyone Monday17:31
*** sambetts is now known as sambetts|afk17:31
*** sergek has joined #openstack-ironic17:32
openstackgerritJarrod Johnson proposed openstack/pyghmi: Disable futile attempts to correct uncorrectable time  https://review.openstack.org/31621817:32
*** blinky_ghost has quit IRC17:34
*** ChrisMead has joined #openstack-ironic17:39
*** mat128|away is now known as mat12817:42
*** piet has quit IRC17:47
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Create a early_create grenade phase  https://review.openstack.org/31623417:48
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Adding dynamic allocation operations  https://review.openstack.org/25869917:48
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Adding dynamic allocation operations  https://review.openstack.org/25869917:54
*** alexpilotti has quit IRC18:00
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614918:01
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576618:01
*** alexpilotti has joined #openstack-ironic18:06
*** keedya has joined #openstack-ironic18:13
*** linuxgeek has quit IRC18:17
*** spandhe has joined #openstack-ironic18:17
*** fragatina has joined #openstack-ironic18:31
*** jtomasek_ has joined #openstack-ironic18:37
*** ijw has joined #openstack-ironic18:38
*** jtomasek_ has quit IRC18:38
*** mbound has joined #openstack-ironic18:38
*** jaybeale has quit IRC18:39
*** jtomasek_ has joined #openstack-ironic18:39
*** ayoung has quit IRC18:42
*** e0ne has joined #openstack-ironic18:43
*** ChubYann has joined #openstack-ironic18:45
*** anush has quit IRC18:46
*** linuxgeek has joined #openstack-ironic18:46
*** Goneri has joined #openstack-ironic18:56
*** daemontool_ has quit IRC19:06
*** Sukhdev has quit IRC19:12
*** Sukhdev has joined #openstack-ironic19:15
*** yarkot has joined #openstack-ironic19:17
*** ijw has quit IRC19:24
*** jtomasek_ has quit IRC19:27
*** sabeen has quit IRC19:27
*** Sukhdev has quit IRC19:28
*** Sukhdev has joined #openstack-ironic19:28
*** sabeen has joined #openstack-ironic19:29
*** jaybeale has joined #openstack-ironic19:30
*** ayoung has joined #openstack-ironic19:30
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Fix misformatted releasenotes of merged patch #314016  https://review.openstack.org/31624919:34
openstackgerritJohn L. Villalovos proposed openstack/ironic: Setup for using the Grenade 'early_create' phase  https://review.openstack.org/31623419:43
*** rbudden has quit IRC19:44
jlvillalcinerama: Thanks for the patch.  You beat me to it :)19:47
cineramaheh, no problem19:47
cineramajust poking at this a bit on a quiet afternoon19:47
jlvillalcinerama: I just pushed a new change.19:52
* jlvillal trying it out locally now19:53
*** e0ne has quit IRC19:59
cineramanice i should try it too20:00
*** ijw has joined #openstack-ironic20:00
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Fix formatting error in releasenotes  https://review.openstack.org/31624920:02
*** alexpilo_ has joined #openstack-ironic20:02
*** alexpilotti has quit IRC20:02
klindgrenOne final question re: ironic-python-agent + coreos build stuff and using it for on boarding of nodes via ironic_inspector.  I am testing this out using kilo right now (but the IPA part is from master).  But as currently packaged and release the IPA stuff flat out wont work on anything using IPMI?  This is because the coreos image that is configured to use doesn't have the IPMI drivers and - atleast on my hardware.  I needed t20:02
klindgreno load additional kernel modules to get the ipmi devices created so that ipmitool would function and find the BMC IP address.20:02
*** dmk0202 has joined #openstack-ironic20:03
*** joprovost has quit IRC20:03
klindgrenis the idea that basically every deployment needs to have their own branch to customize the IPA agent to work with their hardware?  Or is updating to newer versions of coreos mainly blocked due to the gate stuff?20:03
mat128klindgren: interesting that you had to load additional modules20:05
mat128I would file a bug20:05
mat128We are not expecting operators to fork this to simply "get it working"20:06
mat128I think we are fetching latest CoreOS nowadays20:06
klindgrenI mean asside from the additional modules - that fact that its still using coreos 3xx - where it doesn't even have the ipmi modules built20:07
klindgrenhttps://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/coreos-oem-inject.py#L2420:07
klindgrenstill targeting 367.1.0 ?20:07
mat128oh, the work I was thinking of got reverted20:08
mat128https://github.com/openstack/ironic-python-agent/commit/943d2c00e94cff38c15bca5ecad40f9a9b27008f20:08
mat128it was working on 5xx IIRC20:08
mat128maybe we can pin to something newer, rather than completely rolling this back20:08
klindgrenok - so basically this in on hold until gate gets move to tinyipa20:08
mat128we will still be supporting CoreOS20:09
mat128it's just that TinyIPA is so much smaller that it gives us quicker feedback and allows us to run more instances on the same host20:09
mat128are your modules part of 367 or you needed a fork to get a newer CoreOS?20:09
klindgrenI moved to the latest stable just to get any ipmi modules (ipmi_si) - not sure when it was built - I didn't bother tracking it down.  But inorder for ipmi0 to be created I also needed to do:20:12
klindgrenwrite_files:20:12
klindgren  - path: /etc/modules-load.d/ipmi.conf20:12
klindgren    content: ipmi_devintf20:12
klindgrenalong with a:20:12
klindgren    - name: systemd-modules-load.service20:12
klindgren      command: restart20:12
mat128Interesting20:13
mat128klindgren: I would submit a review for using a newer (but pinned) CoreOS20:13
mat128if it proves to work with the gate, we can at least have this20:13
mat128or maybe we can re-revert the "autoroll" version we had in place20:13
*** rbudden has joined #openstack-ironic20:14
mat128klindgren: we now have virtualBMC which we will use to test against IPMI in the gates20:14
mat128but in-band IPMI will be harder20:14
mat128:)20:14
*** spandhe has quit IRC20:14
klindgrenyea - hardware :-/20:14
klindgrenstandards? whats that?20:15
mat128Haha20:15
mat128We'd need IPMI support in QEMU20:15
mat128everything is possible my friend :)20:15
klindgrenone thing I was curious about - is how are people hanlding where for SOL one BMC needs to be set to com1 and another set to com2?20:15
mat128I remember seeing an implementation of this, it also tried to provide out-of-band access (network) but it didnt really work well20:16
klindgrenbooting the deploymentr ram disk I can't set both ttys0 and ttys1 as only the first console of each type is used20:16
mat128s/this/ ipmi in qemu/20:16
mat128klindgren: we (as an operator) aren't using serial over LAN at all so I don't know20:17
mat128but for sure this means you can't hardcode anything in the template20:17
mat128maybe you can pull this information from the node properties?20:17
klindgrenyea - but it also means I need things changed in the images as well20:18
klindgrenso I can't have a generic image20:18
klindgrenI guess I can fixup after boot20:18
klindgrenand get a patch in to pull console terminal from a node property20:18
mat128that's ugly but yeah it would work20:19
klindgrenso process for adding that feature within ironic.  bug -> rfe? or will a spec be required (Pulling which console to use from the node properties)20:20
mat128bug tagged "rfe" then if it requires more details / more design a spec will be requested20:20
mat128http://lists.openstack.org/pipermail/openstack-dev/2016-May/094658.html20:21
mat128one of the priorities for this cycle is nova-compatible serial console20:21
mat128your point fits very well within that topic20:21
openstackgerritJohn L. Villalovos proposed openstack/ironic: Setup for using the Grenade 'early_create' phase  https://review.openstack.org/31623420:21
*** dmk0202 has quit IRC20:22
mat128make sure you also file a bug / open a review to switch to a newer CoreOS as I think this can be valuable to the group20:22
mat128especially if you did the work20:24
jlvillalNobodyCam: jroll TheJulia: Pretty easy review here: https://review.openstack.org/#/c/316249/    Fixes weirdness seen in the bug-fixes section of: http://docs.openstack.org/releasenotes/ironic/current-series.html20:28
jlvillaldevananda: ^^^20:29
*** e0ne has joined #openstack-ironic20:31
NobodyCamJlvillal +a :-)20:33
jlvillalThanks NobodyCam :)20:34
*** joprovost has joined #openstack-ironic20:35
*** ametts has quit IRC20:36
openstackgerritMerged openstack/ironic: Fix formatting error in releasenotes  https://review.openstack.org/31624920:40
*** daemontool has joined #openstack-ironic20:46
*** e0ne has quit IRC20:51
*** daemontool has quit IRC20:52
*** joprovost has quit IRC20:53
jlvillaljroll: Hmmm. I'm thinking releasenotes when doing stable/mitaka is pulling things from other git history.20:55
jrolljlvillal: yeah, there's known weirdness there afaik20:55
jrollyou seeing anything super troubling? or rather what are you seeing?20:55
jlvillaljroll: Like this patch: https://review.openstack.org/#/c/316029/20:56
jlvillaljroll: http://docs-draft.openstack.org/29/316029/1/check/gate-ironic-releasenotes/38ac413//releasenotes/build/html/mitaka.html20:56
jlvillalSo there is breakage in master. But it is also breaking stable/mitaka20:56
jrolljlvillal: hm, published version looks okay http://docs.openstack.org/releasenotes/ironic/mitaka.html20:57
jrollweird20:57
*** Sukhdev has quit IRC20:57
jlvillalI reproduced locally just pulling stable/mitaka and nothing else.20:57
devanandaklindgren: it sounds like you'd need to set different pxe append parameters per-node, instead of per-conductor20:57
jrolljlvillal: I think it may be something odd with like, what's in the local git tree, maybe the publish job only pulls in the branch20:58
jlvillalmaybe20:58
jrollI'm running away right now though, feel free to file a bug if you track something down20:58
jlvillaljroll: Thanks. Have a great weekend20:59
jrollyou too20:59
mat128devananda: is this already possible?21:01
klindgrendevananda, yea.  I was hoping we could move more to an override per node model.21:01
klindgrenLike setting the IPA image per conductor vs's per node.  and setting the cleaning network per node vs's per conductor21:01
klindgrendevananda, +1 to what mat128 asked.  Is this already possible?  I didn't see anything in the documentation that led me to believe it was possible.21:03
mat128klindgren, devananda we're using a totally static pxe here, so I'd be missing out on the new dynamic templated pxe stuff :)21:04
*** jaybeale_ has joined #openstack-ironic21:08
*** jaybeale has quit IRC21:08
mjturek1jlvillal: fyi I ran into this while running your grenade test today http://paste.openstack.org/show/497128/ wondering if you've seen it?21:09
openstackgerritJohn L. Villalovos proposed openstack/ironic: Setup for using the Grenade 'early_create' phase  https://review.openstack.org/31623421:15
jlvillal mjturek1 Looks like a network issue to me.21:17
jlvillalmjturek1: As in it couldn't download something and then died.21:19
mjturek1jlvillal: oh duh, I see know.21:19
jlvillalmjturek1: :)21:23
*** sabeen has quit IRC21:25
*** lascii is now known as alaski21:28
*** thrash is now known as thrash|wknd21:41
*** keedya has quit IRC21:43
*** dprince has quit IRC21:45
*** jaybeale_ has quit IRC21:51
*** dmk0202 has joined #openstack-ironic21:53
*** keedya has joined #openstack-ironic21:56
*** Sukhdev has joined #openstack-ironic21:57
*** daemontool has joined #openstack-ironic21:59
*** alexpilo_ has quit IRC22:01
*** alexpilotti has joined #openstack-ironic22:04
*** dmk0202 has quit IRC22:06
*** fragatina has quit IRC22:11
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Setup for using the Grenade 'early_create' phase  https://review.openstack.org/31623422:19
*** sacharya has quit IRC22:20
openstackgerritKris Lindgren proposed openstack/ironic-python-agent: Changes to allow ipmi to work under the CoreOS IPA image  https://review.openstack.org/31630622:24
*** fragatina has joined #openstack-ironic22:26
*** fragatina has quit IRC22:26
*** fragatina has joined #openstack-ironic22:27
*** cdearborn has quit IRC22:27
*** alex_xu has quit IRC22:30
*** lintan_ has quit IRC22:31
*** daemontool has quit IRC22:31
*** lintan_ has joined #openstack-ironic22:31
*** alex_xu has joined #openstack-ironic22:33
*** alexpilotti has quit IRC22:35
*** alexpilotti has joined #openstack-ironic22:38
*** keedya has quit IRC22:41
*** catintheroof has quit IRC22:49
*** rbudden has quit IRC22:51
openstackgerritKris Lindgren proposed openstack/ironic-python-agent: Make ipmi to work under the CoreOS IPA image  https://review.openstack.org/31630622:54
*** ppiela has quit IRC23:00
*** alexpilotti has quit IRC23:01
*** alexpilotti has joined #openstack-ironic23:02
*** lucas-afk has quit IRC23:05
*** lucasagomes has joined #openstack-ironic23:12
*** sacharya has joined #openstack-ironic23:20
*** baoli_ has quit IRC23:23
*** ekarlso has quit IRC23:23
*** sacharya has quit IRC23:25
*** absubram has quit IRC23:27
*** ekarlso has joined #openstack-ironic23:32
*** mtanino has quit IRC23:41
*** mbound has quit IRC23:49

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