*** hoangcx has quit IRC | 00:01 | |
*** mbound has quit IRC | 00:04 | |
*** harshs has quit IRC | 00:10 | |
openstackgerrit | Merged openstack/python-ironicclient: Fix coverage target https://review.openstack.org/348043 | 00:14 |
---|---|---|
*** lujinluo has joined #openstack-ironic | 00:16 | |
*** wajdi has joined #openstack-ironic | 00:17 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add metrics support to IPA https://review.openstack.org/119981 | 00:23 |
*** aNupoisc has quit IRC | 00:31 | |
*** bigjools has quit IRC | 00:43 | |
*** rama_y has quit IRC | 00:43 | |
*** bigjools has joined #openstack-ironic | 00:44 | |
*** hoangcx2 has quit IRC | 00:50 | |
*** mtanino has joined #openstack-ironic | 00:55 | |
bodepd | is this the right channel for glean questions? | 00:56 |
bodepd | or is it infra? | 00:56 |
*** jjohnson2 has joined #openstack-ironic | 00:59 | |
*** adu has quit IRC | 00:59 | |
*** hoangcx has joined #openstack-ironic | 01:02 | |
*** mbound has joined #openstack-ironic | 01:03 | |
*** chlong has joined #openstack-ironic | 01:06 | |
*** mbound has quit IRC | 01:08 | |
*** adu has joined #openstack-ironic | 01:09 | |
*** Sukhdev has quit IRC | 01:18 | |
*** harshs has joined #openstack-ironic | 01:21 | |
*** ahqa has quit IRC | 01:22 | |
*** harshs has quit IRC | 01:25 | |
*** harshs has joined #openstack-ironic | 01:27 | |
*** jjohnson2 has quit IRC | 01:37 | |
*** joprovost has joined #openstack-ironic | 01:49 | |
*** joprovost1 has joined #openstack-ironic | 02:01 | |
*** joprovost has quit IRC | 02:03 | |
*** joprovost1 is now known as joprovost | 02:03 | |
*** ChrisAusten has joined #openstack-ironic | 02:05 | |
*** baoli has quit IRC | 02:06 | |
*** amotoki has quit IRC | 02:10 | |
jroll | bodepd: probably infra is best | 02:12 |
jroll | JayF: I think we should do it | 02:12 |
jroll | morgabra: all I really have to say right now is 'yikes', and probably +1 for hitting the ML | 02:12 |
*** thrash is now known as thrash|g0ne | 02:25 | |
*** ayoung has quit IRC | 02:25 | |
*** adu has quit IRC | 02:26 | |
*** ayoung has joined #openstack-ironic | 02:27 | |
*** amotoki has joined #openstack-ironic | 02:32 | |
*** ayoung has quit IRC | 02:32 | |
*** spandhe has joined #openstack-ironic | 02:35 | |
*** amotoki has quit IRC | 02:36 | |
*** ayoung has joined #openstack-ironic | 02:36 | |
*** amotoki has joined #openstack-ironic | 02:40 | |
*** yuanying has quit IRC | 02:47 | |
*** jrist has quit IRC | 02:55 | |
*** rloo has quit IRC | 02:58 | |
*** amotoki has quit IRC | 03:01 | |
*** adu has joined #openstack-ironic | 03:01 | |
*** amotoki has joined #openstack-ironic | 03:10 | |
*** vishwanathj has joined #openstack-ironic | 03:11 | |
*** ayoung has quit IRC | 03:14 | |
*** xavierr-afk has quit IRC | 03:15 | |
*** Nisha has joined #openstack-ironic | 03:15 | |
*** sdake has joined #openstack-ironic | 03:15 | |
*** xavierr-afk has joined #openstack-ironic | 03:15 | |
*** Nisha is now known as Guest54664 | 03:15 | |
*** sabeen has quit IRC | 03:20 | |
*** sabeen has joined #openstack-ironic | 03:22 | |
*** lujinluo has quit IRC | 03:25 | |
*** sdake_ has joined #openstack-ironic | 03:25 | |
*** sdake has quit IRC | 03:27 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Combine multiple warning logs into one in create_ports https://review.openstack.org/346578 | 03:35 |
*** mtanino has quit IRC | 03:39 | |
*** bvandewa has quit IRC | 03:44 | |
*** sdake_ has quit IRC | 03:46 | |
*** sdake has joined #openstack-ironic | 03:46 | |
*** yuanying has joined #openstack-ironic | 03:50 | |
*** xavierr-afk has quit IRC | 03:57 | |
*** xavierr-afk has joined #openstack-ironic | 03:57 | |
*** harshs has quit IRC | 03:58 | |
*** lujinluo has joined #openstack-ironic | 03:58 | |
*** links has joined #openstack-ironic | 04:03 | |
*** harshs has joined #openstack-ironic | 04:12 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/230924 | 04:14 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: adding back the inspect_ports as an optional parameter https://review.openstack.org/325230 | 04:15 |
*** Sukhdev has joined #openstack-ironic | 04:15 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Add hpssa support only in raid mode https://review.openstack.org/347978 | 04:19 |
*** spandhe has quit IRC | 04:20 | |
*** spandhe_ has joined #openstack-ironic | 04:20 | |
*** harshs has quit IRC | 04:28 | |
*** harlowja_at_home has joined #openstack-ironic | 04:30 | |
*** adu has quit IRC | 04:32 | |
*** deray has joined #openstack-ironic | 04:39 | |
*** harshs has joined #openstack-ironic | 04:48 | |
*** Sukhdev has quit IRC | 04:58 | |
*** vmud213 has joined #openstack-ironic | 05:00 | |
deray | g'morning Ironicers! | 05:01 |
*** jrist has joined #openstack-ironic | 05:03 | |
*** ChubYann has quit IRC | 05:15 | |
*** harshs has quit IRC | 05:23 | |
*** harshs has joined #openstack-ironic | 05:28 | |
*** bks has joined #openstack-ironic | 05:31 | |
*** lujinluo has quit IRC | 05:36 | |
*** lujinluo has joined #openstack-ironic | 05:36 | |
*** ChrisAusten has quit IRC | 05:39 | |
*** joprovost has quit IRC | 05:45 | |
*** joprovost has joined #openstack-ironic | 05:45 | |
*** amotoki_ has joined #openstack-ironic | 05:48 | |
*** joprovost has quit IRC | 05:50 | |
*** amotoki has quit IRC | 05:51 | |
*** rama_y has joined #openstack-ironic | 06:00 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Add hpssa support only in raid mode https://review.openstack.org/347978 | 06:02 |
*** spandhe_ has quit IRC | 06:07 | |
*** spandhe has joined #openstack-ironic | 06:07 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 06:10 |
*** harlowja_at_home has quit IRC | 06:16 | |
*** pcaruana has joined #openstack-ironic | 06:17 | |
*** mjura has joined #openstack-ironic | 06:23 | |
*** mjura has quit IRC | 06:32 | |
*** mjura has joined #openstack-ironic | 06:32 | |
*** rbartal has joined #openstack-ironic | 06:36 | |
*** tesseract- has joined #openstack-ironic | 06:42 | |
*** sdake has quit IRC | 06:42 | |
*** moshele has joined #openstack-ironic | 06:42 | |
*** rama_y has quit IRC | 06:45 | |
*** spandhe has quit IRC | 06:48 | |
*** vishwanathj has quit IRC | 06:49 | |
*** jtomasek has joined #openstack-ironic | 07:02 | |
*** Fdaisuke_ has joined #openstack-ironic | 07:05 | |
*** hoangcx has quit IRC | 07:05 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available https://review.openstack.org/347791 | 07:07 |
*** Fdaisuke has quit IRC | 07:07 | |
*** hoangcx has joined #openstack-ironic | 07:08 | |
*** hshiina has joined #openstack-ironic | 07:15 | |
*** ifarkas_away is now known as ifarkas | 07:15 | |
*** pece has joined #openstack-ironic | 07:29 | |
*** hoangcx has quit IRC | 07:34 | |
*** Guest54664 has quit IRC | 07:39 | |
*** bvandewa has joined #openstack-ironic | 07:39 | |
*** bvandewa has quit IRC | 07:40 | |
*** chlong has quit IRC | 07:47 | |
*** jpich has joined #openstack-ironic | 07:56 | |
*** bvandewa has joined #openstack-ironic | 07:56 | |
*** zzzeek has quit IRC | 08:00 | |
*** hoangcx has joined #openstack-ironic | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** e0ne has joined #openstack-ironic | 08:01 | |
*** mgoddard has joined #openstack-ironic | 08:06 | |
*** _Fdaisuke_ has joined #openstack-ironic | 08:09 | |
*** Fdaisuke_ has quit IRC | 08:10 | |
*** hparekh has quit IRC | 08:11 | |
*** Romanenko_K has joined #openstack-ironic | 08:13 | |
*** derekh has joined #openstack-ironic | 08:14 | |
*** stendulker has joined #openstack-ironic | 08:16 | |
*** lujinluo has quit IRC | 08:19 | |
*** lucas-dinner is now known as lucasagomes | 08:21 | |
*** lujinluo has joined #openstack-ironic | 08:22 | |
*** hparekh has joined #openstack-ironic | 08:24 | |
*** edand has joined #openstack-ironic | 08:25 | |
*** harshs has quit IRC | 08:27 | |
*** yibo has quit IRC | 08:34 | |
*** yibo has joined #openstack-ironic | 08:35 | |
*** yibo has quit IRC | 08:35 | |
*** yibo has joined #openstack-ironic | 08:36 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/225115 | 08:37 |
*** bvandewa has quit IRC | 08:39 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available https://review.openstack.org/347791 | 08:40 |
*** sambetts|afk is now known as sambetts | 08:46 | |
sambetts | JayF: I'm happy with making the tinyipa 256 job the default if it works | 08:47 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive https://review.openstack.org/296466 | 09:03 |
*** daemontool has joined #openstack-ironic | 09:03 | |
openstackgerrit | David Lenwell proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 09:10 |
*** e0ne has quit IRC | 09:17 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add erase_devices_metadata cleaning step https://review.openstack.org/344164 | 09:17 |
*** lujinluo has quit IRC | 09:20 | |
*** hshiina has quit IRC | 09:22 | |
*** electrofelix has joined #openstack-ironic | 09:26 | |
openstackgerrit | Merged openstack/ironic: Add api-ref for new port fields https://review.openstack.org/325299 | 09:32 |
*** lujinluo has joined #openstack-ironic | 09:32 | |
*** e0ne has joined #openstack-ironic | 09:32 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 09:33 |
stendulker | lucasagomes: Hi | 09:36 |
lucasagomes | stendulker, hi there | 09:36 |
stendulker | lucasagomes: regarding review https://review.openstack.org/#/c/344164/ | 09:36 |
stendulker | lucasagomes: You are throwing exception upon encountering first error on any device | 09:37 |
stendulker | lucasagomes: Wont it be better to complete operation for all devices and then throw an exception listing all the devices | 09:38 |
stendulker | this would help user to fix things if any, instead getting incremetal errors on subsequent run of this clean step | 09:39 |
*** electrofelix has quit IRC | 09:39 | |
lucasagomes | stendulker, could be... I will update the patch | 09:39 |
stendulker | ok. just wanted to check if there is any other rational, will log comment accordingly | 09:40 |
lucasagomes | cool thanks a lot for the review | 09:40 |
stendulker | tahnk you. | 09:41 |
stendulker | thank you :) | 09:41 |
*** sabeen has quit IRC | 09:41 | |
*** fumie has quit IRC | 09:45 | |
*** lujinluo has quit IRC | 09:50 | |
*** mgould|afk is now known as mgould | 09:54 | |
mgould | morning Ironic! | 09:54 |
*** mbound has joined #openstack-ironic | 09:56 | |
*** lujinluo has joined #openstack-ironic | 09:57 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD notifications to Ironic https://review.openstack.org/347242 | 10:00 |
*** electrofelix has joined #openstack-ironic | 10:05 | |
*** Guest54664 has joined #openstack-ironic | 10:06 | |
*** hoangcx has quit IRC | 10:08 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
vmud213 | morning mgould | 10:10 |
*** yibo has quit IRC | 10:14 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add erase_devices_metadata cleaning step https://review.openstack.org/344164 | 10:14 |
*** hwcomcn has joined #openstack-ironic | 10:21 | |
*** pionar has joined #openstack-ironic | 10:22 | |
openstackgerrit | Merged openstack/python-ironicclient: Add baremetal port show command to OSC plugin https://review.openstack.org/344732 | 10:30 |
*** mbound has quit IRC | 10:32 | |
*** deray_ has joined #openstack-ironic | 10:32 | |
*** hshiina has joined #openstack-ironic | 10:35 | |
*** deray has quit IRC | 10:35 | |
*** milan has quit IRC | 10:54 | |
*** milan has joined #openstack-ironic | 10:55 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options https://review.openstack.org/297312 | 10:56 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 10:56 |
*** deray_ has quit IRC | 11:05 | |
*** deray_ has joined #openstack-ironic | 11:05 | |
lucasagomes | stendulker, re https://review.openstack.org/#/c/230924/ | 11:06 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-inspector: Combine multiple warning logs into one in create_ports https://review.openstack.org/346578 | 11:06 |
lucasagomes | stendulker, how was that releasenote generated? ironic-lib do not have support for it yet | 11:06 |
stendulker | oh, i use reno command directly | 11:07 |
stendulker | never used ironic version | 11:07 |
stendulker | will remove it | 11:07 |
*** deray_ has quit IRC | 11:08 | |
*** deray_ has joined #openstack-ironic | 11:08 | |
Romanenko_K | Thanks for review and merge of "baremetal port show command to OSC plugin"! :) | 11:08 |
stendulker | lucasagomes: it was quiet a while. it was generated when reno was first introduced :( | 11:08 |
*** deray_ has quit IRC | 11:08 | |
lucasagomes | stendulker, right, because there's no config for that in ironic-lib | 11:09 |
stendulker | lucasagomes: yes | 11:09 |
lucasagomes | so I bet that release note will conflict with something when we add support for it | 11:09 |
stendulker | yes, will clean it up right away | 11:09 |
lucasagomes | stendulker, perhaps remove it for now and we can add later when reno is avaialble? | 11:09 |
stendulker | lucasagomes: thank you for the review | 11:09 |
stendulker | lucasagomes: sure thing | 11:09 |
lucasagomes | stendulker, I'm preparing a patch for it at the moment | 11:09 |
stendulker | lucasagomes: ok, that would be useful | 11:10 |
TheJulia | Good morning | 11:10 |
sambetts | o/ TheJulia | 11:11 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/230924 | 11:11 |
lucasagomes | stendulker, +2;d | 11:13 |
lucasagomes | TheJulia, morning | 11:13 |
stendulker | lucasagomes: thank you :) | 11:13 |
*** stendulker has quit IRC | 11:20 | |
mgould | TheJulia: morning! | 11:21 |
openstackgerrit | Aparna proposed openstack/proliantutils: Add a clean step 'hardware_disk_erase' https://review.openstack.org/336148 | 11:23 |
jroll | morning y'all | 11:34 |
jroll | lucasagomes: thanks for catching the resource_class length thing, updating shortly | 11:34 |
lucasagomes | jroll, morning, no problem :-) | 11:35 |
*** vmud213 has quit IRC | 11:40 | |
*** ccamacho is now known as ccamacho|lunch | 11:41 | |
*** hshiina has quit IRC | 11:44 | |
*** hshiina has joined #openstack-ironic | 11:45 | |
derekh | Is there a irc channel for pyghmi related stuff? | 11:46 |
TheJulia | /win 8 | 11:47 |
TheJulia | doh | 11:47 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add node.resource_class field https://review.openstack.org/345055 | 11:49 |
jroll | ^ with tests :D | 11:49 |
jroll | derekh: here is best, the main author doesn't seem to be here now, but ask away and we'll try to help | 11:50 |
openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Add support for node.resource_class https://review.openstack.org/345080 | 11:54 |
*** lucasagomes is now known as lucas-hungry | 11:54 | |
*** chlong has joined #openstack-ironic | 11:55 | |
derekh | jroll: thanks, I'm looking for a release if possible https://bugs.launchpad.net/tripleo/+bug/1599299 | 11:55 |
openstack | Launchpad bug 1599299 in tripleo "openstackbmc still failing with concatenation errors" [High,Triaged] | 11:55 |
jroll | derekh: ah yeah, that's in our governance, I can roll one now | 11:56 |
derekh | jroll: great, thanks | 11:56 |
*** hoonetorg has quit IRC | 11:56 | |
jroll | ... or is it | 11:57 |
jroll | maybe not | 11:57 |
* jroll does it anyway maybe? | 11:57 | |
jroll | derekh: meh, I need devananda or jarrod to release it, gonna be a bit | 11:58 |
derekh | jroll: ok | 11:58 |
*** lujinluo has quit IRC | 12:00 | |
*** amotoki_ has quit IRC | 12:06 | |
*** xavierr-afk is now known as xavierr | 12:09 | |
openstackgerrit | Merged openstack/python-ironicclient: Add CLI tests to check JSON response body https://review.openstack.org/291239 | 12:11 |
*** mtanino has joined #openstack-ironic | 12:12 | |
*** amotoki has joined #openstack-ironic | 12:13 | |
*** thrash|g0ne is now known as thrash | 12:14 | |
*** MattMan has quit IRC | 12:17 | |
*** MattMan has joined #openstack-ironic | 12:17 | |
*** pionar has quit IRC | 12:21 | |
*** fellypefca has joined #openstack-ironic | 12:22 | |
*** edand has quit IRC | 12:24 | |
jroll | holding off on the policy stuff for a recheck, multitenant job has never passed on it, quite curious | 12:26 |
jroll | (devstack errors) | 12:26 |
*** hoonetorg has joined #openstack-ironic | 12:27 | |
*** hwcomcn has quit IRC | 12:31 | |
*** amotoki has quit IRC | 12:34 | |
*** amotoki has joined #openstack-ironic | 12:39 | |
*** links has quit IRC | 12:40 | |
*** trown|outtypewww is now known as trown | 12:43 | |
*** amotoki has quit IRC | 12:47 | |
*** Guest54664 has quit IRC | 12:48 | |
*** mbound has joined #openstack-ironic | 12:49 | |
*** thiagop has joined #openstack-ironic | 12:50 | |
thiagop | Good morning, Ironicers | 12:50 |
TheJulia | good morning | 12:51 |
fellypefca | thiagop good morning :) | 12:51 |
thiagop | TheJulia fellypefca o/ | 12:52 |
*** baoli has joined #openstack-ironic | 12:53 | |
*** baoli_ has joined #openstack-ironic | 12:54 | |
*** ccamacho|lunch is now known as ccamacho | 12:55 | |
*** baoli has quit IRC | 12:58 | |
*** hoangcx has joined #openstack-ironic | 13:02 | |
*** lucas-hungry is now known as lucasagomes | 13:03 | |
*** ppiela has joined #openstack-ironic | 13:07 | |
*** rloo has joined #openstack-ironic | 13:08 | |
*** amotoki has joined #openstack-ironic | 13:12 | |
openstackgerrit | Merged openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images https://review.openstack.org/230924 | 13:17 |
*** bks has quit IRC | 13:19 | |
jroll | rloo: mind a re-review on https://review.openstack.org/#/c/345055/ for something lucas found? | 13:19 |
jroll | lucasagomes: mind reviewing the client patch to go with it? https://review.openstack.org/#/c/345080/ | 13:19 |
rloo | jroll: looking... | 13:19 |
lucasagomes | jroll, not at all, will do in a min | 13:20 |
jroll | thanks | 13:20 |
*** jcoufal has joined #openstack-ironic | 13:21 | |
*** daemontool has quit IRC | 13:21 | |
rloo | jroll: +2'd. I'm going to wait a few hours in case any one else wants to look at it. i or someone else will +A it today I think :) | 13:24 |
jroll | rloo: thanks :) | 13:25 |
jroll | rloo: my goal is to get the client release submitted early-ish today so that release team can approve it | 13:25 |
jroll | they don't tend to do friday releases | 13:25 |
rloo | jroll: oh, right. monday is probably fine too. nova folks don't really need it this week? | 13:26 |
thiagop | jroll rloo lucasagomes if you have some review slots to spend on https://review.openstack.org/#/c/286192/, we'd appreciate | 13:26 |
jroll | rloo: the earlier the better, but monday could be fine | 13:26 |
* thiagop don't want it to be pushed back on the last minutes again :/ | 13:27 | |
jroll | thiagop: honest question, oneview patches often have lots of co-authors, did 6 people really write that code? | 13:27 |
lucasagomes | thiagop, will do | 13:27 |
thiagop | jroll: yup. We were a team of six. Some people put the hands on the code at some point in time, others give maintenance later... | 13:28 |
jroll | thiagop: interesting | 13:28 |
jroll | I guess that's 200+ lines each, seems reasonable | 13:29 |
jroll | just always curious :) | 13:29 |
thiagop | jroll: you know, these stuff of the oneview driver hangs a lot before being merged. Maybe because it's a lot of code in a single patch, but... that's it | 13:29 |
thiagop | jroll: this one is hanging since february... :/ | 13:29 |
jroll | thiagop: well, it's usually because we have other priorities, unfortunately | 13:29 |
thiagop | jroll: I know that. That's why I'm always looking for breaches on your review agenda :) | 13:30 |
jroll | hehe | 13:30 |
*** wajdi has quit IRC | 13:34 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add reno support for release notes management https://review.openstack.org/348318 | 13:35 |
lucasagomes | should be a quick review ^ adding reno support to ironic-lib | 13:35 |
*** sdake has joined #openstack-ironic | 13:36 | |
jroll | lucasagomes: nice, I was going to file a bug for that today :) | 13:36 |
lucasagomes | jroll, btw the patch to oslo.utils adding > and < operators to the spec_matcher is already merged | 13:37 |
jroll | we'll also need a project-config update | 13:37 |
jroll | oh nice | 13:37 |
lucasagomes | jroll, yeah I'm putting a patch up for the project-config now | 13:37 |
lucasagomes | I just noted the reno support was missing due: https://review.openstack.org/#/c/230924/22/releasenotes/notes/config-drive-support-for-whole-disk-images-in-iscsi-deploy-10d74d74dcca807f.yaml | 13:37 |
jroll | awesome, thanks man | 13:37 |
jroll | right, saw that in scrollback | 13:37 |
*** ayoung has joined #openstack-ironic | 13:43 | |
*** jcoufal has quit IRC | 13:46 | |
*** jcoufal has joined #openstack-ironic | 13:47 | |
rloo | jroll, lucasagomes: wrt ironic-lib & reno. I submitted a patch for that awhile ago, and in discussion with someone, we decided it wasn't needed. | 13:49 |
rloo | jroll, lucasagomes: might have been dmitry. I think I asked oslo folks, and they don't use reno. | 13:49 |
sambetts | I guess if we continue to say that ironic-lib isn't for use outside of Ironic projects then there is noone that needs informing about changes | 13:50 |
*** PollyZ has joined #openstack-ironic | 13:51 | |
rloo | jroll, lucasagomes: here you go: https://review.openstack.org/#/c/335205/ | 13:51 |
rloo | sambetts: yeah, dmitry's point was that it wasn't a user-facing library | 13:51 |
lucasagomes | rloo, oh hmm it's a good point | 13:52 |
*** thiagop is now known as thiagop-afk | 13:52 | |
lucasagomes | rloo, tho even for us it would be easier to know which version includes what by looking at the release notes | 13:52 |
lucasagomes | than commits in the code | 13:52 |
lucasagomes | I don't think it does any harm, not sure... let's see what others think | 13:53 |
rloo | lucasagomes: only harm is extra work for folks i guess. i am not against it cuz obviously i submitted a patch for it, but i also don't see the use of adding extra work given everything that we have to do. clearly, dtantsur|pto was against it :) | 13:55 |
lucasagomes | rloo, true, I've added a comment to the patch to see what other reviewers think | 13:59 |
lucasagomes | thanks for pointing it out | 13:59 |
rloo | lucasagomes: thx! | 14:00 |
rloo | lucasagomes: after thinking about it for 10 seconds more, maybe we shouldn't cuz of what sambetts reminded me. the reno's will be available via openstack's release notes url. but it isn't even like oslo's library. it isn't for consumption outside of ironic project. | 14:01 |
lucasagomes | rloo, hmm true :-/ | 14:02 |
jroll | rloo: good points | 14:02 |
lucasagomes | yeah let's abandon that patch | 14:02 |
jroll | that said reno makes it super easy for me to figure out what version a release needs to be | 14:02 |
jroll | I'm fine without it though | 14:02 |
lucasagomes | jroll, yeah I find it easier to figure out what version includes/removes what as well | 14:03 |
sambetts | perhaps we'll just add a note into the release note that uses the new funtionality in Ironic/IPA? | 14:03 |
rloo | jroll, lucasagomes: i agree. how do the oslo folks deal with it? maybe they have fancy scripts or warped minds (just kidding) | 14:03 |
rloo | sambetts: yes | 14:04 |
jroll | rloo: they release weekly so that it's not many commits and easy to figure out :P | 14:04 |
rloo | jroll: we can release weekly too :D | 14:04 |
*** jvgrant has quit IRC | 14:04 | |
jroll | indeed :) | 14:04 |
*** bvandewa has joined #openstack-ironic | 14:05 | |
*** causten has joined #openstack-ironic | 14:06 | |
*** thrash is now known as thrash|biab | 14:07 | |
lucasagomes | abandoned | 14:07 |
*** jvgrant has joined #openstack-ironic | 14:08 | |
*** wajdi has joined #openstack-ironic | 14:08 | |
*** edand has joined #openstack-ironic | 14:09 | |
*** wajdi_ has joined #openstack-ironic | 14:11 | |
*** wajdi has quit IRC | 14:13 | |
*** hshiina has quit IRC | 14:26 | |
*** wajdi_ has quit IRC | 14:29 | |
*** jistr is now known as jistr|call | 14:35 | |
*** thrash|biab is now known as thrash | 14:36 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Clean up provision ports when reattempting deploy https://review.openstack.org/348392 | 14:37 |
*** sabeen has joined #openstack-ironic | 14:37 | |
*** wajdi has joined #openstack-ironic | 14:39 | |
*** clenimar has joined #openstack-ironic | 14:42 | |
*** spandhe has joined #openstack-ironic | 14:43 | |
*** spandhe_ has joined #openstack-ironic | 14:45 | |
vdrok | morning all! | 14:47 |
*** ijw has joined #openstack-ironic | 14:47 | |
vdrok | another super easy change that would be good to go into the client release - https://review.openstack.org/341552 | 14:47 |
*** spandhe has quit IRC | 14:47 | |
*** spandhe_ is now known as spandhe | 14:47 | |
*** ijw has quit IRC | 14:47 | |
*** ijw has joined #openstack-ironic | 14:47 | |
abashmak | Hi folks, would someone kindly review this: https://review.openstack.org/#/c/346832/ | 14:48 |
*** jistr|call is now known as jistr | 14:49 | |
mgould | vdrok: LGTM | 14:49 |
*** rama_y has joined #openstack-ironic | 14:50 | |
vdrok | thx mgould | 14:51 |
*** ijw has quit IRC | 14:52 | |
*** sabeen1 has joined #openstack-ironic | 14:55 | |
*** sabeen has quit IRC | 14:57 | |
*** ifarkas is now known as ifarkas_afk | 14:58 | |
*** causten has quit IRC | 14:59 | |
*** causten has joined #openstack-ironic | 14:59 | |
*** rama_y has quit IRC | 15:01 | |
*** harshs has joined #openstack-ironic | 15:01 | |
*** mjura has quit IRC | 15:01 | |
*** rama_y has joined #openstack-ironic | 15:01 | |
*** causten has quit IRC | 15:03 | |
*** joprovost has joined #openstack-ironic | 15:05 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: Add node serial console installation howto https://review.openstack.org/293872 | 15:05 |
*** PollyZ has quit IRC | 15:06 | |
*** ccamacho is now known as ccamacho|afk | 15:08 | |
*** jmiu_ is now known as jmiu | 15:09 | |
*** mgoddard has quit IRC | 15:09 | |
*** mgoddard_ has joined #openstack-ironic | 15:09 | |
NobodyCam | Good Morning Ironicers | 15:13 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on https://review.openstack.org/330003 | 15:15 |
TheJulia | vdrok: re, 341552, is that in parity with osc? | 15:17 |
*** harshs has quit IRC | 15:17 | |
*** openstackgerrit has quit IRC | 15:18 | |
*** ijw has joined #openstack-ironic | 15:18 | |
*** openstackgerrit has joined #openstack-ironic | 15:18 | |
thiagop-afk | morning NobodyCam | 15:19 |
NobodyCam | morning thiagop-afk | 15:19 |
*** yuanying_ has joined #openstack-ironic | 15:20 | |
*** pcaruana has quit IRC | 15:22 | |
*** yuanying has quit IRC | 15:22 | |
*** ijw has quit IRC | 15:23 | |
*** bvandewa has quit IRC | 15:27 | |
*** Guest54664 has joined #openstack-ironic | 15:28 | |
*** PollyZ has joined #openstack-ironic | 15:30 | |
*** edand has quit IRC | 15:30 | |
*** ijw has joined #openstack-ironic | 15:34 | |
openstackgerrit | Moshe Levi proposed openstack/ironic: Adding InfiniBand Support https://review.openstack.org/264263 | 15:36 |
*** harshs has joined #openstack-ironic | 15:37 | |
devananda | good morning, all. | 15:38 |
*** ijw has quit IRC | 15:39 | |
*** bvandewa has joined #openstack-ironic | 15:39 | |
*** bvandewa has quit IRC | 15:39 | |
*** bvandewa has joined #openstack-ironic | 15:39 | |
*** jistr is now known as jistr|afk | 15:42 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 15:44 |
*** e0ne has quit IRC | 15:45 | |
*** e0ne has joined #openstack-ironic | 15:46 | |
vdrok | TheJulia: iirc there is no port show yet in osc plugin, but will check now | 15:46 |
jroll | morning devananda | 15:48 |
vdrok | TheJulia: ah, there is now | 15:48 |
rloo | vdrok: i think the port show just merged this morning | 15:48 |
vdrok | TheJulia: but it seems it just outputs whatever is returned by port.py | 15:48 |
jroll | devananda: unless jarrod pops up, can you release pyghmi today? it's on the old tagging system and only you two have access. derekh needs it to fix a tripleo bug | 15:48 |
vdrok | TheJulia: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/v1/baremetal_port.py#L128 | 15:49 |
vdrok | rloo: yup, morning :) | 15:49 |
devananda | jroll: been at least a year since I did - let me see if I still rememer how | 15:50 |
*** ijw has joined #openstack-ironic | 15:50 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port unset command to OSC plugin https://review.openstack.org/347402 | 15:50 |
*** PollyZ has quit IRC | 15:52 | |
*** mtanino has quit IRC | 15:54 | |
TheJulia | vdrok: thank you :) | 15:55 |
*** rloo has quit IRC | 15:55 | |
*** comstud has quit IRC | 15:55 | |
*** ijw has quit IRC | 15:55 | |
*** rloo has joined #openstack-ironic | 15:55 | |
vdrok | yeah, just confirmed locally, np | 15:55 |
jroll | this nova patch lgtm if someone else wants to look https://review.openstack.org/#/c/341253 | 15:56 |
jroll | JayF: lucasagomes: I know you two are interested in that interaction (cleaning up failed deploys) | 15:57 |
lucasagomes | jroll, looking | 15:57 |
*** causten has joined #openstack-ironic | 15:58 | |
*** pece has quit IRC | 15:58 | |
lucasagomes | interesting... | 15:59 |
* lucasagomes really wanted to have a /deploy endpoint to pass all the info for a deployment at once | 15:59 | |
jroll | yeah | 15:59 |
jroll | I brought that up at nova midcycle | 15:59 |
jroll | they want to parallelize some things, so like plug_vifs and any volume stuff would still need to be separate to do those at the same time before deploying | 16:00 |
rloo | JayF: did that gate troubleshooting session happen on July 13? I missed it. | 16:02 |
JayF | No it didn't :X | 16:02 |
JayF | man, I failboated a lot, that never got on my calendar | 16:02 |
rloo | JayF: ha ha. I forgot to put it on my calendar too! | 16:03 |
rloo | JayF: maybe we should try this in Sept or October :) | 16:03 |
*** PollyZ has joined #openstack-ironic | 16:06 | |
*** harshs has quit IRC | 16:07 | |
*** e0ne has quit IRC | 16:07 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Dont review https://review.openstack.org/348457 | 16:08 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Log full config only once in conductor https://review.openstack.org/348459 | 16:10 |
cinerama | rloo: i have a question for you about the volume connection work, if you have time for it | 16:11 |
rloo | cinerama: oh. ugh. you can ask. just in the middle of two things and not sure i can swap my brain to somethign else but can try. | 16:13 |
*** harshs has joined #openstack-ironic | 16:13 | |
cinerama | rloo: yikes, i know that feel | 16:14 |
cinerama | rloo: anyway this is pretty simple: currently, the RPC API version and the REST API version get an increment for both volume connector and volume target...we were wondering whether it should stay that way, or whether they can both be introduced in the same version | 16:15 |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Dont review https://review.openstack.org/348457 | 16:17 |
rloo | cinerama: same version of ironic? like ironic release? | 16:17 |
cinerama | rloo: same API version | 16:18 |
*** rama_y has quit IRC | 16:18 | |
rloo | cinerama: the RPC version is diff from the API version. | 16:18 |
rloo | cinerama: if the RPC version is bumped, it doesn't mean the API version is bumped. | 16:19 |
*** baoli_ has quit IRC | 16:19 | |
rloo | cinerama: i haven't had a chance to look at that volume patch. should i take a quick look? | 16:21 |
*** adu has joined #openstack-ironic | 16:21 | |
*** ijw has joined #openstack-ironic | 16:21 | |
cinerama | rloo: that'd be great. all the work is currently under bug/1526231, the first patch in the series is https://review.openstack.org/#/c/200983/ | 16:21 |
*** mbound has quit IRC | 16:22 | |
rloo | cinerama: ok, in a bit. going to grab something to eat first. | 16:22 |
cinerama | rloo: the REST API patches and the client patches are still pretty WIP because there are some differences with the spec we need to address | 16:22 |
*** mbound has joined #openstack-ironic | 16:23 | |
cinerama | rloo: do eat (i need to after this call i'm on). and thanks for looking | 16:23 |
*** ijw has quit IRC | 16:26 | |
*** Romanenko_K has quit IRC | 16:28 | |
*** moshele has quit IRC | 16:29 | |
*** jistr|afk is now known as jistr | 16:30 | |
abashmak | vdrok: thanks for the review, I will address your comments | 16:31 |
*** tesseract- has quit IRC | 16:32 | |
vdrok | abashmak: thank you too :) | 16:32 |
*** derekh has quit IRC | 16:39 | |
NobodyCam | morning rloo vdrok abashmak cinerama jroll JayF lucasagomes :) | 16:42 |
*** jpich has quit IRC | 16:42 | |
*** aNupoisc has joined #openstack-ironic | 16:42 | |
abashmak | Good morning NobodyCam | 16:42 |
JayF | o/ | 16:42 |
jroll | hey NobodyCam | 16:42 |
NobodyCam | :) | 16:42 |
lucasagomes | NobodyCam, hi there | 16:43 |
NobodyCam | xavierr: question: when is support for NON-dynamic allocation planed for obsolescence | 16:43 |
*** mtanino has joined #openstack-ironic | 16:45 | |
*** gfhellma has joined #openstack-ironic | 16:45 | |
vdrok | morning! :) | 16:45 |
*** aNupoisc has quit IRC | 16:46 | |
*** linuxgeek has quit IRC | 16:46 | |
mgould | morning NobodyCam | 16:48 |
NobodyCam | morning mogul :) | 16:48 |
NobodyCam | gah | 16:48 |
lucasagomes | folks, I'm calling it a day | 16:48 |
NobodyCam | mgould: even | 16:48 |
NobodyCam | lol | 16:48 |
lucasagomes | have a great evening all | 16:48 |
NobodyCam | have a good night lucasagomes | 16:48 |
*** PollyZ has quit IRC | 16:48 | |
mgould | good night lucasagomes | 16:50 |
mgould | NobodyCam: one of these days you'll spell my name right :-) | 16:50 |
openstackgerrit | Merged openstack/ironic: Add node.resource_class field https://review.openstack.org/345055 | 16:50 |
NobodyCam | I just have the new laptop "learn" how to do just that :p | 16:51 |
mgould | heh | 16:51 |
jmiu | tab completion is a beautiful thing | 16:51 |
NobodyCam | jmiu: thats what I use but the new mac keeps auto correcting the names :p | 16:52 |
jmiu | NobodyCam: can you turn that off for specifically irc? | 16:54 |
mgould | NobodyCam: https://twitter.com/qntm/status/755460779319685120/photo/1 | 16:54 |
*** amotoki has quit IRC | 16:54 | |
mgould | https://twitter.com/qntm/status/755460779319685120, rather | 16:55 |
* sambetts -> home | 16:56 | |
*** sambetts is now known as sambetts|afk | 16:56 | |
NobodyCam | mgould: :) | 16:56 |
sambetts|afk | night all | 16:56 |
NobodyCam | night sambetts|afk | 16:56 |
NobodyCam | jmiu: that would turn off auto correct safari which would be bad (at least for me) :p | 16:57 |
mgould | night sambetts|afk | 16:59 |
*** amotoki has joined #openstack-ironic | 17:00 | |
*** causten has joined #openstack-ironic | 17:01 | |
*** PollyZ has joined #openstack-ironic | 17:01 | |
*** amotoki has quit IRC | 17:03 | |
*** e0ne has joined #openstack-ironic | 17:04 | |
*** causten_ has joined #openstack-ironic | 17:05 | |
*** causten has quit IRC | 17:05 | |
jmiu | NobodyCam: oh so no specifically off for irc | 17:06 |
NobodyCam | nope | 17:07 |
*** loki_ has joined #openstack-ironic | 17:07 | |
*** trown is now known as trown|lunch | 17:08 | |
JayF | devananda: is it possible the oslo policy generator is generating YAML | 17:08 |
JayF | devananda: instead of JSON? | 17:08 |
JayF | devananda: I know YAML is now supported by oslo policy iirc | 17:08 |
*** causten has joined #openstack-ironic | 17:08 | |
*** causten_ has quit IRC | 17:10 | |
devananda | JayF: does YAML allow items to be separated only by a newline ? | 17:10 |
*** Kelso has joined #openstack-ironic | 17:10 | |
JayF | yes | 17:11 |
JayF | http://www.yaml.org/spec/1.2/spec.html#id2759963 | 17:11 |
JayF | unless I'm misreading that | 17:11 |
*** causten has quit IRC | 17:12 | |
devananda | ok - yea, it's valid YAML | 17:12 |
*** causten has joined #openstack-ironic | 17:12 | |
devananda | lacks the opening --- but it seems to pass parse tests for YAML ... | 17:12 |
JayF | So perhaps that's what it's doing then? | 17:13 |
devananda | perhaps. then we shouldn't call the file *.json | 17:13 |
JayF | ++ | 17:14 |
*** jcoufal_ has joined #openstack-ironic | 17:14 | |
devananda | ugh. parking expired - bbiab | 17:14 |
*** causten has quit IRC | 17:17 | |
*** jcoufal has quit IRC | 17:17 | |
*** fragatina has joined #openstack-ironic | 17:17 | |
*** fragatina has quit IRC | 17:19 | |
*** mgould is now known as mgould|afk | 17:20 | |
vdrok | good night everyone! | 17:20 |
*** PollyZ has quit IRC | 17:21 | |
NobodyCam | night vdrok | 17:21 |
*** baoli has joined #openstack-ironic | 17:22 | |
*** rbartal has quit IRC | 17:25 | |
*** causten has joined #openstack-ironic | 17:25 | |
jroll | oh I'm totally okay with yaml | 17:26 |
jroll | because comments make invalid json \o/ | 17:26 |
*** fragatina has joined #openstack-ironic | 17:26 | |
*** fragatina has quit IRC | 17:27 | |
*** rbartal has joined #openstack-ironic | 17:27 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Dont review https://review.openstack.org/348457 | 17:27 |
*** rama_y has joined #openstack-ironic | 17:27 | |
*** rbartal has quit IRC | 17:27 | |
*** rama_y has quit IRC | 17:28 | |
cinerama | do we not use ramdisk-image-create for DIB IPA builds anymore? our docs seem to suggest not | 17:30 |
cinerama | dib folks are wondering about it as we are looking at retooling the special casing for the IPA build not creating an image | 17:33 |
*** causten has quit IRC | 17:33 | |
NobodyCam | thiagop-afk: wow great stuff with the dynamic allocation stuff. I did leave a question on the review. | 17:34 |
*** causten has joined #openstack-ironic | 17:34 | |
*** causten has quit IRC | 17:38 | |
*** adu has quit IRC | 17:38 | |
*** fragatina has joined #openstack-ironic | 17:39 | |
*** fragatina has quit IRC | 17:40 | |
*** fragatina has joined #openstack-ironic | 17:41 | |
*** Kelso has quit IRC | 17:44 | |
*** Kelso has joined #openstack-ironic | 17:45 | |
*** fragatina has quit IRC | 17:46 | |
thiagop-afk | NobodyCam: seeing now | 17:48 |
NobodyCam | :) | 17:48 |
*** Kelso has quit IRC | 17:50 | |
*** maurosr has quit IRC | 17:52 | |
*** thiagop-afk is now known as thiagop | 17:53 | |
thiagop | NobodyCam: I think that'll depend on the state machine. When cleaning is disabled, the node remains in deployfail or does it move the machine back to available? | 17:53 |
thiagop | NobodyCam: I have not considered that, indeed.. | 17:54 |
NobodyCam | I think its direct to available which could to confusion | 17:56 |
NobodyCam | s/could/could lead / | 17:56 |
thiagop | NobodyCam: If it goes to available, the first periodic will take care of it | 17:57 |
thiagop | NobodyCam: if it reamains in deployfail, we'll need to add a new filter there... | 17:57 |
NobodyCam | can you dig in to that | 17:57 |
NobodyCam | dig / look | 17:58 |
* thiagop looks | 17:59 | |
*** causten has joined #openstack-ironic | 18:00 | |
NobodyCam | :) thou even if the first periodic task would catch the node. it could be (at max) 2xperiodic_check_interval? | 18:02 |
NobodyCam | thats ten minutes in available when in fact its taken? | 18:03 |
thiagop | NobodyCam: it depends on the configured time. If an operator expects that these kind of stuff is frequent, he should use a shorter time | 18:04 |
thiagop | NobodyCam: and if the instance is booted, it'll fail validation | 18:04 |
*** causten has quit IRC | 18:04 | |
*** causten has joined #openstack-ironic | 18:05 | |
NobodyCam | is that documented (or in the process of being documented)? | 18:05 |
thiagop | NobodyCam: more like "in the process": https://review.openstack.org/#/c/305441/ | 18:07 |
*** Sukhdev has joined #openstack-ironic | 18:07 | |
*** trown|lunch is now known as trown | 18:07 | |
NobodyCam | thiagop: Nice!!!! :) | 18:08 |
*** causten has quit IRC | 18:10 | |
*** Sukhdev has quit IRC | 18:10 | |
*** PollyZ has joined #openstack-ironic | 18:13 | |
hamzy | I have question about IPA hardware managers such as https://github.com/jayofdoom/ipa-example-hardware-managers . I have been looking at diskimage-builder and it isn't obvious to me how I would install a git repo of a python package into the image being built. IIUC the setup.cfg of an installed repo inside of the image would then be picked up by IPA. | 18:15 |
JayF | I'm not super familiar with DIB, I'd imagine you'd have to do a custom element? | 18:15 |
*** Kelso has joined #openstack-ironic | 18:16 | |
JayF | I know there's a prolientutils element, and they used to ship a hardware manager | 18:16 |
JayF | TheJulia: cinerama ^ can you help hamzy? | 18:16 |
hamzy | I don't think it would be under package-installs http://docs.openstack.org/developer/diskimage-builder/elements/package-installs/README.html | 18:16 |
TheJulia | hamzy: looking for an example | 18:16 |
*** Sukhdev has joined #openstack-ironic | 18:17 | |
rajinir | sambetts: thiagop: have a question on the thirdparty CI, are you running it with Devstack vm on Devstack? | 18:17 |
thiagop | rajinir: devstack-gate | 18:17 |
rajinir | sambetts: thiagop: devstack-gate on the Devstack VM ? | 18:17 |
thiagop | thiagop: https://github.com/openstack-infra/devstack-gate | 18:18 |
thiagop | rajinir: ^ | 18:18 |
TheJulia | hamzy: I think this is the magic that will answer your question: https://github.com/openstack/diskimage-builder/blob/master/elements/proliant-tools/install.d/65-proliant-tools-install#L40 and https://github.com/openstack/proliantutils/blob/master/setup.cfg#L25 | 18:19 |
cinerama | JayF: unless i'm missing something that element just installs hpssacli rpm & proliantutils from pip now | 18:19 |
JayF | Yeah that's what I'd imagine | 18:19 |
rajinir | sambetts: thiagop: I see. We are using devstack-vm-gate, now I see why we are seeing networking issues | 18:19 |
JayF | all you'd have to do is install the python package in | 18:19 |
rajinir | sambetts: thiagop: Thanks , let me switch and see if that solves the issues | 18:19 |
cinerama | JayF: oh hey while you're here do you remember why we stopped using ramdisk-image-create vs disk-image-create for the IPA element? | 18:20 |
thiagop | rajinir: it really depends on your script and underlying infra. Never used devstack-vm-gate but... maybe :) | 18:20 |
JayF | cinerama: no idea whatsoever. I have run a dIB IPA image exactly once ever. | 18:20 |
cinerama | JayF: relevant change here - https://review.openstack.org/#/c/334042/ | 18:20 |
cinerama | heh | 18:20 |
*** loki_ has quit IRC | 18:21 | |
rajinir | thiagop: Confused. We are running devstack-gate/devstack-vm-gate-wrap.sh from https://github.com/openstack-infra/devstack-gate | 18:21 |
thiagop | rajinir: so you're using devstack-gate | 18:22 |
hamzy | cinerama, TheJulia, hrm... I want to install an arbitrary git repo of my own manager | 18:22 |
rajinir | thiagop:yes | 18:22 |
TheJulia | hamzy: oh | 18:22 |
JayF | hamzy: you can pip install from a git repository | 18:22 |
TheJulia | ahh | 18:22 |
TheJulia | hamzy: what jayF said, and there is a way to do it in dib specificaly | 18:22 |
thiagop | rajinir: if you describe what's happening, maybe I can give some ideas in how to debug | 18:22 |
rajinir | thiagop: The dhcp packets are not routed to the tap. This is our main issue | 18:22 |
hamzy | yeah I was looking for a dib specific way since I am using dib to create the IPA boot image | 18:22 |
TheJulia | hamzy: example of pulling the package down via git https://github.com/openstack/diskimage-builder/blob/master/elements/simple-init/source-repository-simple-init | 18:23 |
TheJulia | or the repo, and then you'd have to install it in your dib element | 18:23 |
thiagop | rajinir: you can see them arriving at the physical port with tcpdump? | 18:23 |
*** causten has joined #openstack-ironic | 18:23 | |
thiagop | can you** | 18:23 |
rajinir | thiagop: Would love too. We are running out of ideas and thinnking of abandoning devstak-gate. Any pointer will help | 18:23 |
*** causten_ has joined #openstack-ironic | 18:24 | |
*** ayoung has quit IRC | 18:24 | |
hamzy | TheJulia, thanks... I will try messing around with it | 18:25 |
rajinir | thiagop: yes, tcpdump shows the packets forwarded until the last step. Hardware Ironic Nodes -> Devstack Host(physicalinterface)->Devstack Slave VM (phrical interface) -> Tap | 18:25 |
devananda | JayF: after looking into changing policy.json to policy.yaml, I'm disinclined to do that. oslo things still default to callig the file policy.json in many places | 18:25 |
TheJulia | hamzy: enjoy! | 18:25 |
JayF | devananda: so, call it policy.json even though it's yaml? And maybe add a patch to the policy generator to put a comment at the top saying it's yaml | 18:26 |
JayF | devananda: not shipping it is also fine with me | 18:26 |
rajinir | thiagop: The devstack slave VM networking is what I see is not setup right | 18:26 |
*** causten_ has quit IRC | 18:26 | |
thiagop | rajinir: you may need to hack into devstack-gate's (or some of devstack's) options to make it right | 18:26 |
devananda | JayF: wdyt about: add tox target, include policy.json.sample, update docs and devstack to indicate that it's not needed | 18:27 |
thiagop | rajinir: let-me paste you some of the configs I'm using here... | 18:27 |
*** causten_ has joined #openstack-ironic | 18:27 | |
JayF | devananda: +1, and as a bonus we could patch the oslo generator to add a comment at the top saying it's yaml (or perhaps file a bug indicating the confusion and leave it to oslo folks to sort) | 18:28 |
*** causten has quit IRC | 18:28 | |
thiagop | rajinir: http://paste.openstack.org/show/543419/ | 18:28 |
TheJulia | devananda: JayF you know... its not much code to determine if it is yaml or json and load it *ducks* | 18:28 |
*** baoli has quit IRC | 18:29 | |
jroll | devananda: I think the generator can output json | 18:29 |
jroll | devananda: I have a present for you, but first am curious if you got around to tagging pyghmi? | 18:30 |
rajinir | thiagop: Ah, we have something like this http://paste.openstack.org/show/543420/ | 18:30 |
*** spandhe has quit IRC | 18:30 | |
devananda | jroll: nothing in the --help indicates that it can generate json | 18:30 |
jroll | really? :/ | 18:30 |
devananda | jroll: and, no, I haven't. | 18:30 |
devananda | I did look at it, but then needed to move my car | 18:30 |
jroll | ok | 18:30 |
devananda | I think I just tag -s 1.0.8 && git push 1.0.8 | 18:31 |
rajinir | thiagop: looks similar, but when the bridge happens the ip add is del and moved to the ovs bridge, node goes offline | 18:31 |
*** causten_ has quit IRC | 18:31 | |
jroll | devananda: that sounds right | 18:31 |
thiagop | rajinir: ugh, you are using a single interface VM to run devstack-gate? | 18:32 |
rajinir | thiagop: yes, is that the issue | 18:32 |
rajinir | thiagop: Was just thinking about it this morning to add another interface | 18:32 |
thiagop | rajinir: here we use two different networks. | 18:32 |
*** electrofelix has quit IRC | 18:33 | |
rajinir | thiagop: should we give it a try with two different networks? | 18:33 |
thiagop | rajinir: one interface on the cloud's private network, and another to the physical network that is where the physical blades are | 18:33 |
*** vishwanathj has joined #openstack-ironic | 18:34 | |
rajinir | thiagop: and the one you are bridging in the devstack-gate is the physical? | 18:34 |
*** catintheroof has joined #openstack-ironic | 18:34 | |
thiagop | rajinir: I think so. If public_interface is the connection jenkins(?!) is using to connect to the node via ssh, it'll break | 18:34 |
thiagop | rajinir: yup. The other one is just used to ssh commands to devstack | 18:35 |
jroll | devananda: anyway, the present: https://review.openstack.org/#/c/348443/ | 18:35 |
jroll | it totally works | 18:35 |
jroll | there's one todo in there | 18:35 |
rajinir | thiagop: ok, will check with my labmanager and add another network and give it a try | 18:36 |
thiagop | rajinir: good luck | 18:36 |
*** fellypefca has quit IRC | 18:36 | |
rajinir | thiagop: Before I go off, did you customize the nodepool to create the vm to add two networks? | 18:37 |
thiagop | rajinir: yes | 18:37 |
rajinir | thiagop: Any doc on how to? | 18:37 |
rajinir | thiagop: Or do we have to hack the nodepool code | 18:37 |
thiagop | rajinir: not handy, but I can paste it too | 18:38 |
rajinir | thiagop: cool, that would be very helpful. Thanks a bunch | 18:38 |
thiagop | rajinir: http://paste.openstack.org/show/543422/ | 18:39 |
thiagop | rajinir: first provider connects only on the private net, second one connects on private AND ironic's | 18:39 |
*** rama_y has joined #openstack-ironic | 18:40 | |
devananda | jroll: very interesting. that todo might be a bit of work... | 18:40 |
*** gfhellma has quit IRC | 18:41 | |
jroll | devananda: shouldn't be bad, though maybe a bit db heavy, and this is the worst place to put db heavy things :( | 18:41 |
*** ChubYann has joined #openstack-ironic | 18:41 | |
rajinir | thiagop: question, what is the first-provider used for? do you have another job that uses it? | 18:42 |
thiagop | rajinir: yes. We use it for a power&management only job with fake_oneview. It tests a bunch of operations without real hardware | 18:43 |
devananda | jroll: agreed. I'll take a look at that with my perf hat on, when you add it | 18:44 |
jroll | devananda: I figure it can't be worse than writing compute_nodes records for every node :) | 18:45 |
rajinir | thiagop: Cool, gives me some clarity and hope. Thank you. Will keep you posted if this works | 18:45 |
thiagop | rajinir: ;) | 18:45 |
*** maurosr has joined #openstack-ironic | 18:48 | |
devananda | jroll: I have pushed a 1.0.8 tag | 18:48 |
jroll | devananda: thank you | 18:49 |
jroll | devananda: I wonder if you need to do stuff to pypi too | 18:51 |
*** gfhellma has joined #openstack-ironic | 18:51 | |
*** baoli has joined #openstack-ironic | 18:54 | |
*** ElCoyote_ has quit IRC | 18:54 | |
*** ElCoyote_ has joined #openstack-ironic | 18:54 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic https://review.openstack.org/325599 | 18:57 |
*** causten has joined #openstack-ironic | 18:59 | |
thiagop | NobodyCam: yeah Chris, it's what I expected. The driver will go into "deployfail", then nova calls "delete" and the instance is deleted, going thru teard_down to available | 19:00 |
NobodyCam | so in standalone mode (i.e.. with out nova) we're left in deploy fail? | 19:01 |
thiagop | jroll devananda , can you confirm this ^? (we are talking about what happens to the node if it fails deploy and cleaning is disabled) | 19:01 |
jroll | that sounds correct | 19:01 |
thiagop | NobodyCam: yep. So far... | 19:02 |
jroll | thiagop: in fact, the same thing happens if cleaning is enabled | 19:02 |
jroll | cleaning won't happen until delete is called | 19:02 |
thiagop | Would that^ be a problem to bifrost TheJulia? | 19:02 |
*** rbudden has joined #openstack-ironic | 19:02 | |
devananda | TheJulia, NobodyCam, jroll: addressed all of your comments on the policy patch. thanks for the review! | 19:02 |
NobodyCam | :) | 19:03 |
devananda | thiagop: that sounds like what I expect -- node is left in deployfail until user initiates something else | 19:03 |
devananda | thiagop: which could be "delete" or "rebuild", for example | 19:04 |
rloo | cinerama: wrt your question about REST api and RPC apis in the volume connector work, the way it is coded seems fine. | 19:04 |
rloo | cinerama: although i did wonder if you could combine the two REST API patches (volume connector operation & volume target operation) into one patch. | 19:04 |
thiagop | devananda jroll gotcha, thanks | 19:04 |
jroll | devananda: multitenant job still hasn't passed on that, I'm curious to see if it does this round. couldn't tell if the previous failures were "real", the most recent one was not | 19:04 |
rloo | cinerama: cuz is it useful to use one w/o the other? | 19:04 |
jroll | devananda: thanks for fixing though :D | 19:04 |
cinerama | rloo: we were kind of pondering that | 19:05 |
cinerama | rloo: (combining those patches) | 19:05 |
rloo | cinerama: i didn't look at the code but if they are fairly simple, it is fine to combine them. but that's my opinion. | 19:06 |
cinerama | rloo: thanks for the advice. will be good to have your (& others') full review at some point | 19:07 |
rloo | cinerama: at some point. soooo many patches (i guess that is a good thing) and not enough people reviewing :-( | 19:08 |
*** jcoufal_ has quit IRC | 19:13 | |
devananda | jroll: multitenant job has been failing on all patches for a bit ... hrmmm | 19:13 |
*** harshs has quit IRC | 19:13 | |
devananda | looks like http://logs.openstack.org/65/321865/14/check/gate-tempest-dsvm-ironic-multitenant-network-nv/a0d8ae6/ was the first run it started failing on | 19:14 |
* devananda investigates briefly | 19:14 | |
jroll | ah you know what, I bet it got caught in that big refactor | 19:14 |
rloo | jroll: you're going to push out a client release soon? | 19:16 |
jroll | rloo: yes, thanks for reminder, was waiting for the merge | 19:16 |
devananda | 2016-07-27 01:09:18.814 | + functions-common:git_clone:533 : echo 'The /opt/stack/new/ironic project was not found; if this is a gate job, add' | 19:16 |
devananda | that's the first time I've seen that error | 19:17 |
jroll | devananda: seems like we're missing a PROJECTS=openstack/ironic thing | 19:17 |
jroll | but, the other jobs don't have it explicit | 19:17 |
jroll | so idk | 19:17 |
devananda | yea... | 19:17 |
*** Sukhdev has quit IRC | 19:17 | |
jroll | looks like it includes devstack-virtual-ironic, which does have it | 19:18 |
*** jcoufal has joined #openstack-ironic | 19:18 | |
jroll | rloo: zuul didn't pick up the dependent patch merging, I added a +A to kick it | 19:23 |
rloo | jroll: good thing you like to kick :) | 19:23 |
* jroll has a strong foot | 19:24 | |
openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros https://review.openstack.org/346894 | 19:28 |
devananda | jroll: it looks like dsg-setup-workspace isn't including ironic in the builds that are failing | 19:28 |
devananda | http://logs.openstack.org/92/348392/1/check/gate-tempest-dsvm-ironic-multitenant-network-nv/386fc05/logs/devstack-gate-setup-workspace-new.txt.gz | 19:28 |
jroll | hrm, firefox broke ctrl-f | 19:28 |
jroll | O_o | 19:28 |
devananda | I need to run - delivering soup to a sick friend. bbiab | 19:29 |
openstackgerrit | Akilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros https://review.openstack.org/346894 | 19:29 |
*** Kelso has quit IRC | 19:32 | |
*** Kelso has joined #openstack-ironic | 19:45 | |
*** jcoufal has quit IRC | 19:49 | |
* rloo wonders if devananda *made* the soup... | 19:49 | |
*** jcoufal has joined #openstack-ironic | 19:50 | |
*** ccamacho has joined #openstack-ironic | 19:51 | |
openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient: Workaround to OneView pagination https://review.openstack.org/348532 | 20:00 |
*** gfhellma1 has joined #openstack-ironic | 20:01 | |
*** gfhellma has quit IRC | 20:04 | |
*** baoli has quit IRC | 20:04 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 20:07 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification base classes and docs https://review.openstack.org/298461 | 20:07 |
mariojv | fixed the releasenotes test failure above | 20:07 |
*** gfhellma1 has quit IRC | 20:07 | |
*** mbound has quit IRC | 20:08 | |
*** Kelso has quit IRC | 20:14 | |
*** adu has joined #openstack-ironic | 20:14 | |
*** e0ne has quit IRC | 20:18 | |
openstackgerrit | Merged openstack/python-ironicclient: Add support for node.resource_class https://review.openstack.org/345080 | 20:25 |
*** greghaynes has quit IRC | 20:25 | |
*** fragatina has joined #openstack-ironic | 20:27 | |
NobodyCam | TheJulia: https://bugs.launchpad.net/ironic/+bug/1607527 | 20:28 |
openstack | Launchpad bug 1607527 in Ironic "ssh power driver can not use a key with a passphrase" [Undecided,New] | 20:28 |
*** greghaynes has joined #openstack-ironic | 20:37 | |
thiagop | NobodyCam: answered in the review just to keep record | 20:39 |
*** ccamacho has quit IRC | 20:39 | |
*** ccamacho|afk has quit IRC | 20:39 | |
NobodyCam | thiagop: awesome thank you | 20:40 |
openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Raise Not Authorized Exception when connection request is forbidden https://review.openstack.org/343550 | 20:43 |
*** jralph has joined #openstack-ironic | 20:45 | |
JayF | NobodyCam: I suspect that bug should likely end up wontfix since that driver is going out of tree this cycle | 20:47 |
NobodyCam | :( they are still going to live in the staging repo. figured I should file it | 20:48 |
*** Kelso has joined #openstack-ironic | 20:48 | |
*** jralph_ has quit IRC | 20:48 | |
*** Kelso has quit IRC | 20:49 | |
*** gmmaha has left #openstack-ironic | 20:57 | |
*** trown is now known as trown|outtypewww | 20:57 | |
*** gmmaha has joined #openstack-ironic | 20:58 | |
*** causten has quit IRC | 21:00 | |
*** causten has joined #openstack-ironic | 21:01 | |
*** causten_ has joined #openstack-ironic | 21:02 | |
jroll | rloo: dims_ is going to process that client release today | 21:02 |
*** causten has quit IRC | 21:05 | |
rloo | jroll: sweet! | 21:06 |
*** causten_ has quit IRC | 21:07 | |
*** mtanino has quit IRC | 21:08 | |
mrda | Morning Ironic | 21:08 |
*** causten has joined #openstack-ironic | 21:08 | |
thiagop | morning mrda | 21:08 |
*** mbound has joined #openstack-ironic | 21:09 | |
thiagop | Calling it a day | 21:09 |
thiagop | good night folks | 21:09 |
*** thiagop has quit IRC | 21:09 | |
mrda | o/ | 21:10 |
*** causten has quit IRC | 21:12 | |
*** mbound has quit IRC | 21:14 | |
*** causten has joined #openstack-ironic | 21:24 | |
*** wajdi has quit IRC | 21:24 | |
NobodyCam | morning mrda | 21:28 |
mrda | hey NobodyCam | 21:29 |
NobodyCam | :) | 21:30 |
*** causten has quit IRC | 21:31 | |
*** causten has joined #openstack-ironic | 21:31 | |
*** PollyZ has quit IRC | 21:32 | |
*** causten has quit IRC | 21:33 | |
*** causten has joined #openstack-ironic | 21:34 | |
*** baoli has joined #openstack-ironic | 21:34 | |
*** causten has quit IRC | 21:39 | |
*** causten has joined #openstack-ironic | 21:39 | |
*** adu has quit IRC | 21:46 | |
*** causten has quit IRC | 21:53 | |
*** harshs has joined #openstack-ironic | 21:53 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP https://review.openstack.org/343947 | 21:55 |
*** harshs has quit IRC | 21:57 | |
*** harshs has joined #openstack-ironic | 21:58 | |
*** jaybeale has joined #openstack-ironic | 22:03 | |
*** PollyZ has joined #openstack-ironic | 22:04 | |
*** jaybeale has quit IRC | 22:15 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros https://review.openstack.org/346894 | 22:21 |
*** harshs has quit IRC | 22:27 | |
*** krtaylor has quit IRC | 22:31 | |
*** harshs has joined #openstack-ironic | 22:46 | |
*** harshs has quit IRC | 22:56 | |
*** PollyZ has quit IRC | 22:56 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP https://review.openstack.org/343947 | 23:01 |
*** harshs has joined #openstack-ironic | 23:02 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 23:05 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 23:16 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 23:16 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [agent] https://review.openstack.org/309603 | 23:16 |
*** fumie has joined #openstack-ironic | 23:21 | |
openstackgerrit | Chris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP https://review.openstack.org/343947 | 23:24 |
*** rbudden has quit IRC | 23:40 | |
*** hoangcx2 has joined #openstack-ironic | 23:46 | |
*** hoangcx has quit IRC | 23:47 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!