*** aNuposic has quit IRC | 00:03 | |
*** soliosg has joined #openstack-ironic | 00:06 | |
*** soliosg has quit IRC | 00:23 | |
*** cdearborn has joined #openstack-ironic | 00:36 | |
*** mtan_____ has joined #openstack-ironic | 00:36 | |
*** mtanino has quit IRC | 00:37 | |
openstackgerrit | Merged openstack/ironic: Fix exception handling in iscsi_deploy.continue_deploy https://review.openstack.org/374881 | 00:52 |
---|---|---|
*** SerenaFeng has joined #openstack-ironic | 00:54 | |
*** causten has joined #openstack-ironic | 00:57 | |
*** rama_y has quit IRC | 01:00 | |
*** rbrud has quit IRC | 01:04 | |
*** pmannidi_ has joined #openstack-ironic | 01:09 | |
*** rloo has quit IRC | 01:10 | |
*** pmannidi has quit IRC | 01:11 | |
*** causten_ has joined #openstack-ironic | 01:12 | |
*** tuanluong has joined #openstack-ironic | 01:15 | |
*** causten has quit IRC | 01:15 | |
*** penick has quit IRC | 01:20 | |
*** Sukhdev has quit IRC | 01:23 | |
*** moshele has joined #openstack-ironic | 01:25 | |
*** zhangjl has joined #openstack-ironic | 01:26 | |
*** vsaienko has quit IRC | 01:27 | |
*** SerenaFeng has quit IRC | 01:30 | |
*** causten has joined #openstack-ironic | 01:31 | |
*** causten_ has quit IRC | 01:34 | |
*** gcb has joined #openstack-ironic | 01:35 | |
*** penick has joined #openstack-ironic | 01:37 | |
*** alexpilotti has joined #openstack-ironic | 01:44 | |
*** SerenaFeng has joined #openstack-ironic | 01:46 | |
*** alexpilotti has quit IRC | 01:49 | |
*** mtanino has joined #openstack-ironic | 01:56 | |
*** mtan_____ has quit IRC | 01:57 | |
*** alexpilotti has joined #openstack-ironic | 01:58 | |
*** aNuposic has joined #openstack-ironic | 01:59 | |
*** alexpilotti has quit IRC | 02:03 | |
*** Michael-ZTE has joined #openstack-ironic | 02:04 | |
*** Michael-ZTE has quit IRC | 02:04 | |
*** mtanino has quit IRC | 02:04 | |
*** zhangjl has quit IRC | 02:08 | |
*** gcb has quit IRC | 02:09 | |
*** zhangjl has joined #openstack-ironic | 02:09 | |
*** yufei_ has quit IRC | 02:09 | |
*** yufei has joined #openstack-ironic | 02:11 | |
*** penick has quit IRC | 02:18 | |
*** yufei_ has joined #openstack-ironic | 02:18 | |
openstackgerrit | Ghanshyam Mann proposed openstack/ironic: Keep CONF.baremetal.driver_enabled for Tempest plugin https://review.openstack.org/400032 | 02:19 |
*** gcb has joined #openstack-ironic | 02:22 | |
openstackgerrit | Merged openstack/ironic: Remove deprecated Neutron DHCP provider methods https://review.openstack.org/398861 | 02:25 |
*** syed__ has quit IRC | 02:36 | |
*** soliosg has joined #openstack-ironic | 02:37 | |
*** Alagar has joined #openstack-ironic | 02:38 | |
*** soliosg has quit IRC | 02:41 | |
*** moshele has quit IRC | 02:46 | |
*** causten_ has joined #openstack-ironic | 02:54 | |
*** moshele has joined #openstack-ironic | 02:54 | |
*** Michael-ZTE has joined #openstack-ironic | 02:54 | |
*** causten__ has joined #openstack-ironic | 02:56 | |
*** causten has quit IRC | 02:57 | |
*** Michael-ZTE has quit IRC | 02:58 | |
*** causten_ has quit IRC | 03:00 | |
*** rama_y has joined #openstack-ironic | 03:07 | |
*** aNuposic has quit IRC | 03:11 | |
tonyb | dtantsur|afk: working through that list now. the python-ironic-inspector-client release is going to need tom some thought so I'll do it last | 03:15 |
*** vikrant has joined #openstack-ironic | 03:31 | |
*** Nisha has joined #openstack-ironic | 03:34 | |
*** Nisha is now known as Guest27425 | 03:34 | |
*** fragatin_ has joined #openstack-ironic | 03:36 | |
*** fragatin_ has quit IRC | 03:38 | |
*** fragati__ has joined #openstack-ironic | 03:38 | |
*** fragatina has quit IRC | 03:39 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-inspector: Use function is_valid_mac from oslo.utils https://review.openstack.org/400489 | 03:40 |
*** fragati__ has quit IRC | 03:41 | |
*** SerenaFeng has quit IRC | 03:44 | |
*** ionutbalutoiu has quit IRC | 03:52 | |
*** Guest27425 has quit IRC | 03:53 | |
*** Guest27425 has joined #openstack-ironic | 03:53 | |
*** ionutbalutoiu has joined #openstack-ironic | 03:55 | |
*** fragatina has joined #openstack-ironic | 03:57 | |
*** cdearborn has quit IRC | 04:03 | |
*** fragatina has quit IRC | 04:06 | |
*** fragatina has joined #openstack-ironic | 04:07 | |
*** links has joined #openstack-ironic | 04:08 | |
*** aparnav has joined #openstack-ironic | 04:08 | |
*** fragatina has quit IRC | 04:11 | |
*** alexpilotti has joined #openstack-ironic | 04:14 | |
*** causten has joined #openstack-ironic | 04:14 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient: Replace six.iteritems() with .items() https://review.openstack.org/400505 | 04:15 |
*** moshele has quit IRC | 04:15 | |
*** causten__ has quit IRC | 04:17 | |
*** alexpilotti has quit IRC | 04:18 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/bifrost: Replace six.iteritems() with .items() https://review.openstack.org/400506 | 04:19 |
*** ijw has quit IRC | 04:20 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 04:23 |
*** fragatina has joined #openstack-ironic | 04:29 | |
*** fragatina has quit IRC | 04:29 | |
*** fragatina has joined #openstack-ironic | 04:29 | |
*** Alagar has quit IRC | 04:33 | |
*** Alagar has joined #openstack-ironic | 04:33 | |
*** pmannidi_ is now known as pmannidi | 04:37 | |
*** Sukhdev has joined #openstack-ironic | 04:57 | |
*** causten_ has joined #openstack-ironic | 05:00 | |
*** causten has quit IRC | 05:03 | |
*** Alagar has quit IRC | 05:04 | |
*** Alagar has joined #openstack-ironic | 05:06 | |
*** Guest27425 has quit IRC | 05:35 | |
*** SerenaFeng has joined #openstack-ironic | 05:45 | |
*** yufei has quit IRC | 06:00 | |
*** SerenaFeng has quit IRC | 06:03 | |
*** rama_y has quit IRC | 06:03 | |
*** yufei has joined #openstack-ironic | 06:04 | |
*** SerenaFeng has joined #openstack-ironic | 06:08 | |
*** SerenaFeng has quit IRC | 06:11 | |
*** SerenaFeng has joined #openstack-ironic | 06:12 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Modprobe IPMI modules before chroot in coreos image https://review.openstack.org/399987 | 06:17 |
*** jaosorior has joined #openstack-ironic | 06:17 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Modprobe IPMI modules before chroot in coreos image https://review.openstack.org/399987 | 06:19 |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Modprobe IPMI modules before chroot in coreos image https://review.openstack.org/399987 | 06:22 |
*** ChubYann has quit IRC | 06:27 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Modprobe IPMI modules before chroot in CoreOS image https://review.openstack.org/399987 | 06:30 |
*** causten_ has quit IRC | 06:41 | |
*** mjura has joined #openstack-ironic | 06:46 | |
*** jaosorior has quit IRC | 06:49 | |
*** awidders_ has joined #openstack-ironic | 06:49 | |
*** awiddersheim has quit IRC | 06:49 | |
*** jaosorior has joined #openstack-ironic | 06:50 | |
*** yufei has quit IRC | 06:50 | |
*** mjura has quit IRC | 06:50 | |
*** mjura has joined #openstack-ironic | 06:51 | |
*** moshele has joined #openstack-ironic | 06:52 | |
*** fxpester has joined #openstack-ironic | 06:57 | |
*** pester has joined #openstack-ironic | 07:00 | |
*** fxpester has quit IRC | 07:04 | |
*** rbartal has joined #openstack-ironic | 07:19 | |
*** yufei has joined #openstack-ironic | 07:19 | |
*** gcb has quit IRC | 07:22 | |
moshele | jroll, JayF: can you review again https://review.openstack.org/#/c/334816/ I addressed you comments | 07:29 |
patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs | 07:29 |
*** yufei has quit IRC | 07:30 | |
*** yufei has joined #openstack-ironic | 07:30 | |
*** jtomasek has joined #openstack-ironic | 07:32 | |
*** gcb has joined #openstack-ironic | 07:35 | |
*** ccamacho has quit IRC | 07:38 | |
openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 07:40 |
_milan_ | morning Ironic! :) | 07:43 |
*** jtomasek has quit IRC | 07:44 | |
*** jtomasek_ has joined #openstack-ironic | 07:44 | |
_milan_ | aarefiev, dtantsur|afk w/r the non-running smoke test I've managed to run it http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/grenade.sh.txt.gz#_2016-11-21_23_38_58_044 but now http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/grenade.sh.txt.gz#_2016-11-21_23_28_29_691 fails on me with http://logs.openstack.org/10/400410/1/check/gate-gren | 07:45 |
_milan_ | ade-dsvm-ironic-inspector/87eafe7/logs/old/screen-n-cpu.txt.gz?level=ERROR#_2016-11-21_23_28_20_803 | 07:45 |
_milan_ | see also: https://review.openstack.org/#/c/400410/ | 07:46 |
patchbot | patch 400410 - ironic-inspector - TODO: Tempest config update | 07:46 |
*** Nisha has joined #openstack-ironic | 07:47 | |
*** Nisha is now known as Guest92303 | 07:47 | |
*** daemontool has joined #openstack-ironic | 07:48 | |
_milan_ | see also https://bugs.launchpad.net/ironic-inspector/+bug/1643683/ | 07:49 |
openstack | Launchpad bug 1643683 in Ironic Inspector "Inspector smoke test skipped in grenade" [Undecided,In progress] - Assigned to milan k (vetrisko) | 07:49 |
_milan_ | aarefiev, dtantsur|afk ^ | 07:49 |
*** ralonsoh has joined #openstack-ironic | 07:51 | |
*** pcaruana has joined #openstack-ironic | 07:56 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Python 2 and 3 Compatibility https://review.openstack.org/400577 | 08:04 |
*** Sukhdev has quit IRC | 08:06 | |
*** vmud213 has joined #openstack-ironic | 08:10 | |
aarefiev | morning! | 08:11 |
aarefiev | _milan_: hey, thanks for update patch | 08:11 |
_milan_ | aarefiev, morning! :) | 08:11 |
tuanluong | morning ironic | 08:12 |
_milan_ | morning tuanluong! :) | 08:12 |
aarefiev | morning tuanluong | 08:12 |
_milan_ | aarefiev, not sure what happens with the basic_network_ops test, it fails to create new bm instance in the last 2 steps :-/ | 08:13 |
aarefiev | _milan_: hmm, I'm looking | 08:13 |
_milan_ | aarefiev, thx | 08:13 |
*** rcernin_ has joined #openstack-ironic | 08:14 | |
*** alexpilotti has joined #openstack-ironic | 08:15 | |
*** alexpilotti has quit IRC | 08:19 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-inspector: Use function is_valid_mac from oslo.utils https://review.openstack.org/400489 | 08:21 |
* _milan_ looks at https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L401 < reasons are wrong state | 08:21 | |
_milan_ | but I've not seen any ironic conductor error | 08:21 |
sergek | Morning Ironicers! | 08:35 |
*** vmud213 has quit IRC | 08:35 | |
*** daemontool has quit IRC | 08:37 | |
*** hogepodge has quit IRC | 08:45 | |
*** vsaienko has joined #openstack-ironic | 08:45 | |
*** alexpilotti has joined #openstack-ironic | 08:45 | |
*** vmud213 has joined #openstack-ironic | 08:47 | |
*** alexpilotti has quit IRC | 08:50 | |
*** alexpilo_ has joined #openstack-ironic | 08:53 | |
*** vsaienko has quit IRC | 08:54 | |
*** jpich has joined #openstack-ironic | 08:57 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:01 | |
*** vsaienko has joined #openstack-ironic | 09:02 | |
*** vsaienko has quit IRC | 09:03 | |
*** athomas has joined #openstack-ironic | 09:04 | |
_milan_ | morning sergek! :) | 09:07 |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Python 2 and 3 Compatibility https://review.openstack.org/400577 | 09:09 |
aarefiev | _milan_: http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/grenade.sh.txt.gz#_2016-11-21_23_38_59_846 | 09:10 |
aarefiev | morning sergek | 09:10 |
aarefiev | _milan_: test delete instance when it's in spawning state yet | 09:11 |
_milan_ | aarefiev, yeah seems like that :-/ | 09:12 |
_milan_ | so the error in nova-cpu isn't related? | 09:12 |
aarefiev | _milan_: it's the reason n-cpu error | 09:13 |
_milan_ | ah so n-cpu error causes the test to timeout | 09:13 |
_milan_ | then most likely I'm missing the https://github.com/openstack/ironic/commit/ab6f73f7a06a9c9e497762e1dc917a9626cccd26#diff-3e7370b7b0ad85701e43800afdd4c305R41 config somewhere | 09:13 |
_milan_ | or not | 09:14 |
_milan_ | it's vice versa isn't it? | 09:14 |
aarefiev | hm whats the default value here | 09:16 |
* _milan_ checks | 09:16 | |
*** dtantsur|afk is now known as dtantsur | 09:18 | |
_milan_ | aarefiev, https://github.com/openstack/tempest/blame/ca8be4ecb0700850a030acbfc9b9672ac5ddb687/tempest/config.py#L234 | 09:18 |
dtantsur | Morning Ironic | 09:18 |
_milan_ | seems 300x1s | 09:18 |
_milan_ | morning dtantsur! :) | 09:18 |
tuanluong | Mornig dtantsur | 09:18 |
tuanluong | mornig _milan_ | 09:19 |
*** Guest92303 has quit IRC | 09:19 | |
aarefiev | morning dtantsur | 09:19 |
tuanluong | mornig aarefiev | 09:19 |
*** bcornec1 has joined #openstack-ironic | 09:20 | |
tuanluong | dtantsur, Do you know about cloubase-init, I don't know what IRC channel to discuss or communicate with development team | 09:21 |
dtantsur | no idea, sorry | 09:21 |
*** SerenaFeng has quit IRC | 09:22 | |
_milan_ | aarefiev, the log of spawning the instance seems ~20:04--28:19 8min seems OK to me | 09:23 |
aarefiev | _milan_: I thought timeout 5 min, no ? | 09:24 |
_milan_ | yeah | 09:24 |
_milan_ | should be that case | 09:24 |
_milan_ | but at least low timeout value doesn't seem to be the issue here | 09:25 |
_milan_ | if it were e.g 30s | 09:25 |
aarefiev | why it takes so long ( | 09:25 |
_milan_ | not sure why it's building 8m | 09:25 |
_milan_ | yeah | 09:25 |
_milan_ | also, there's no node3 log in bm logs | 09:26 |
_milan_ | 1, 2, 4, 5 are present | 09:26 |
_milan_ | but may be unrelated | 09:26 |
*** ohamada has joined #openstack-ironic | 09:27 | |
aarefiev | _milan_: looks like because it wasn't booted | 09:29 |
openstackgerrit | Bharath kumar proposed openstack/ironic-python-agent: Moving Reboot bashscript to python https://review.openstack.org/372478 | 09:29 |
*** bks has joined #openstack-ironic | 09:29 | |
_milan_ | aarefiev, yeah, it's not an issue I guess ; e.g http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_20_12_428 shows all 7 nodes | 09:30 |
*** lucas-afk is now known as lucasagomes | 09:31 | |
lucasagomes | morning all | 09:31 |
_milan_ | morning lucasagomes! :) | 09:31 |
aarefiev | morning lucasagomes | 09:31 |
lucasagomes | o/ | 09:31 |
_milan_ | aarefiev, http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_20_12_999 is odd though | 09:32 |
_milan_ | so the node is deploying while powered off? | 09:32 |
*** SerenaFeng has joined #openstack-ironic | 09:33 | |
aarefiev | _milan_: it will be turned on later http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_20_33_473 | 09:35 |
_milan_ | ah OK, was just searching for the node being actually powered on in the interval 20:04--28:19 | 09:36 |
*** derekh has joined #openstack-ironic | 09:36 | |
*** _milan_ has quit IRC | 09:38 | |
*** _milan_ has joined #openstack-ironic | 09:38 | |
*** Romanenko_K has joined #openstack-ironic | 09:41 | |
pas-ha | morning Ironic :) | 09:43 |
*** ccamacho has joined #openstack-ironic | 09:45 | |
_milan_ | aarefiev, http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_24_39_623 < deployed successfully, then just power syncs | 09:45 |
_milan_ | and http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_28_21_403 tear down | 09:45 |
_milan_ | nothing odd :-/ | 09:45 |
_milan_ | morning pas-ha! :) | 09:45 |
_milan_ | so how come nova compute is still building? | 09:46 |
aarefiev | _milan_: wrong node? instance on ***201 | 09:50 |
* _milan_ facepalm | 09:50 | |
*** tuanluong has quit IRC | 09:56 | |
*** athomas has quit IRC | 09:57 | |
*** athomas has joined #openstack-ironic | 09:57 | |
*** athomas has quit IRC | 09:57 | |
openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 09:59 |
*** snozhka has quit IRC | 10:01 | |
*** baoli has quit IRC | 10:02 | |
_milan_ | aarefiev, tempest creates server 20:08 but nothing happens tillhttp://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_25_33_582 | 10:03 |
_milan_ | 25:33 | 10:03 |
_milan_ | *w/ 306c5569-bf81-4bb0-9418-44a12fd6f201 | 10:03 |
*** sambetts|afk is now known as sambetts | 10:05 | |
sambetts | Morning team o/ | 10:05 |
_milan_ | morning sambetts! :) | 10:05 |
sambetts | hey _milan_ | 10:05 |
aarefiev | _milan_: yeah, it's too long | 10:06 |
aarefiev | morning sambetts | 10:06 |
*** athomas has joined #openstack-ironic | 10:07 | |
_milan_ | aarefiev, moreover I don't see the "successfully deployed" message in that time interval for that node :-/ | 10:07 |
_milan_ | just power syncs and then tear down | 10:08 |
aarefiev | _milan_: delete was requested before deploy finish | 10:08 |
_milan_ | the node last state seems wait_callback | 10:08 |
_milan_ | so there's no call back from 26:00 http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_26_00_787 | 10:10 |
_milan_ | till http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/87eafe7/logs/old/screen-ir-cond.txt.gz#_2016-11-21_23_28_20_974 | 10:10 |
_milan_ | 2:20min | 10:10 |
_milan_ | it's very slow | 10:10 |
_milan_ | I almost feel like recheck might make it pass now | 10:11 |
aarefiev | _milan_: lets try | 10:11 |
_milan_ | ack | 10:11 |
*** gcb_ has joined #openstack-ironic | 10:16 | |
*** gcb_ has quit IRC | 10:16 | |
*** snozhka has joined #openstack-ironic | 10:16 | |
*** gcb_ has joined #openstack-ironic | 10:17 | |
*** gcb_ has quit IRC | 10:17 | |
vdrok | good morning ironic, _milan_ aarefiev sergek dtantsur lucasagomes pas-ha and sambetts | 10:18 |
lucasagomes | hey there | 10:18 |
dtantsur | o/ | 10:18 |
_milan_ | morning vdrok! :) | 10:18 |
* _milan_ relocates | 10:18 | |
*** gcb has quit IRC | 10:20 | |
aarefiev | morning vdrok | 10:20 |
*** hshiina has joined #openstack-ironic | 10:21 | |
*** gcb has joined #openstack-ironic | 10:22 | |
openstackgerrit | Derek Higgins proposed openstack/ironic: mask credentials for the ssh power driver. https://review.openstack.org/400273 | 10:23 |
*** yufei has quit IRC | 10:24 | |
*** _milan_ has quit IRC | 10:26 | |
*** e0ne has joined #openstack-ironic | 10:30 | |
*** zhangjl has left #openstack-ironic | 10:31 | |
*** yolanda has quit IRC | 10:35 | |
*** yolanda has joined #openstack-ironic | 10:38 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 10:44 |
*** jtomasek_ is now known as jtomasek | 10:45 | |
*** hoonetorg has quit IRC | 10:53 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types https://review.openstack.org/336626 | 10:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Introduce generic hardware types https://review.openstack.org/400678 | 10:58 |
*** SerenaFeng has quit IRC | 10:58 | |
*** yufei has joined #openstack-ironic | 11:01 | |
pas-ha | hi all, who's responsible for ibm powerkvm ci? kurt taylor? | 11:03 |
pas-ha | because it is broken atm since it tries to cherry-pick a non-related patch from gerrit (and fails to apply due to conflicts). I actually suspect why they are doing so (pxe on xenial) but this is fixed already, and the patch they are using is abandoned long ago | 11:06 |
jroll | krtaylor: hamzy: ^^ | 11:06 |
jroll | also mjturek but they aren't online | 11:07 |
jroll | morning y'all | 11:07 |
dtantsur | morning jroll | 11:12 |
*** mgould|afk is now known as mgould | 11:12 | |
mgould | morning Ironic! | 11:12 |
pas-ha | morning jroll | 11:12 |
dtantsur | hey mgould, pas-ha and vdrok | 11:12 |
pas-ha | morning dtantsur mgould | 11:13 |
mgould | morning dtantsur jroll pas-ha aarefiev vdrok sergek sambetts lucasagomes | 11:13 |
lucasagomes | hi there | 11:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types https://review.openstack.org/336626 | 11:13 |
dtantsur | this can use some reviews now ^^^ | 11:13 |
aarefiev | morning mgould | 11:13 |
*** cdearborn has joined #openstack-ironic | 11:14 | |
*** foexle has joined #openstack-ironic | 11:14 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface APIs https://review.openstack.org/327046 | 11:16 |
*** ccamacho has quit IRC | 11:17 | |
*** ccamacho has joined #openstack-ironic | 11:20 | |
vdrok | morning jroll and mgould | 11:21 |
openstackgerrit | Anton Arefiev proposed openstack/ironic: Grenade: configure tempest with Ironic plugin https://review.openstack.org/400687 | 11:21 |
*** hshiina has quit IRC | 11:23 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Grenade: test https://review.openstack.org/400690 | 11:26 |
*** ccamacho has quit IRC | 11:26 | |
openstackgerrit | Derek Higgins proposed openstack/ironic: mask private keys for the ssh power driver. https://review.openstack.org/400273 | 11:26 |
*** ccamacho has joined #openstack-ironic | 11:31 | |
*** bks has quit IRC | 11:34 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 11:34 |
*** Goneri has quit IRC | 11:38 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups https://review.openstack.org/396610 | 11:42 |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add portgroup support to osc plugin https://review.openstack.org/362130 | 11:46 |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Add possibility to work with portgroups https://review.openstack.org/335964 | 11:46 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 11:49 |
ricardoas | morning everybody | 11:53 |
*** cmyster has left #openstack-ironic | 11:54 | |
vdrok | good morning ricardoas | 11:56 |
mgould | morning ricardoas | 11:59 |
charlledaniel | morning all | 12:01 |
mgould | morning charlledaniel | 12:01 |
*** milan has joined #openstack-ironic | 12:02 | |
*** jkilpatr has quit IRC | 12:03 | |
*** jkilpatr has joined #openstack-ironic | 12:03 | |
nicodemos | morning, ricardoas, mgould, vdrok, charlledaniel and all | 12:03 |
vdrok | morning charlledaniel and nicodemos | 12:04 |
*** pester has quit IRC | 12:14 | |
openstackgerrit | Markos Chandras proposed openstack/bifrost: playbooks: bifrost-ironic-install: Fix regexp for pxe_bootfile_name https://review.openstack.org/400712 | 12:15 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Stop disabling the Neutron tempest plugin https://review.openstack.org/400714 | 12:16 |
*** tesseract has joined #openstack-ironic | 12:19 | |
*** tesseract is now known as Guest3787 | 12:19 | |
*** vmud213 has quit IRC | 12:21 | |
*** vmud213 has joined #openstack-ironic | 12:21 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic: Grenade: configure tempest with Ironic plugin https://review.openstack.org/400687 | 12:23 |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Load IPMI modules before chroot in CoreOS image https://review.openstack.org/399987 | 12:23 |
yufei | good morning,ironicers | 12:25 |
mgould | morning yufei | 12:25 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Grenade: test https://review.openstack.org/400690 | 12:25 |
yufei | :D | 12:25 |
gabriel-bezerra | lucasagomes, TheJulia, sambetts, jroll: from yesterday's discussion: do you have a point on why the agent itself should not power off the machine before changing the boot device? | 12:25 |
nicodemos | morning, yufei. | 12:25 |
openstackgerrit | Merged openstack/ironic: Move `deploy_forces_oob_reboot` to deploy drivers https://review.openstack.org/397843 | 12:25 |
gabriel-bezerra | pas-ha: ^ | 12:27 |
lucasagomes | gabriel-bezerra, I haven't put a lot of thoughts into it... Does the agent *always* reboot the node after changing the boot device ? | 12:27 |
vdrok | morning yufei | 12:28 |
*** ccamacho is now known as ccamacho|lunch | 12:28 | |
gabriel-bezerra | lucasagomes: I should look further to give you a better answer, but the tests in the CI server dont break with the change that shuts the machine off and don't turn it back on. | 12:29 |
gabriel-bezerra | we run tempest | 12:29 |
gabriel-bezerra | so seems like all set boot device comes with a power on/reboot afterwards | 12:30 |
dtantsur | set boot device is exposed in the API unfortunately... | 12:30 |
* dtantsur wishes we exposed it as a reboot argument instead... | 12:31 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Update to newer hacking https://review.openstack.org/399546 | 12:31 |
gabriel-bezerra | I'm looking for keeping the set-boot-device without powering off, and changing the agent to do a power off before changing the boot device. | 12:31 |
gabriel-bezerra | dtantsur: ^ | 12:32 |
dtantsur | ah, I see | 12:32 |
gabriel-bezerra | I'm checking for reasons not to do it | 12:32 |
jroll | so you just want to change the order of those in the deploy process | 12:33 |
gabriel-bezerra | lucasagomes: something I don't like in the lazy approach is when we further have actions in the driver, will it be based on which state? (target, current), (target)? | 12:33 |
gabriel-bezerra | jroll: yes, basically that | 12:33 |
gabriel-bezerra | instead of: set-b-device, reboot; do power-off, set-b-device, power-on | 12:34 |
*** dtantsur is now known as dtantsur|brb | 12:35 | |
jroll | yeah, so the weird thing is the code organization here | 12:36 |
jroll | the boot interface has prepare_instance which sets boot device to pxe if we aren't local booting | 12:37 |
jroll | but the agent deploy driver sets it to disk if we are | 12:37 |
sambetts | sounds like the agent deploy driver shoudn't be touching the boot device really | 12:38 |
jroll | actually, we should move that 'if iwdi' block into prepare_instance_to_boot as well | 12:38 |
jroll | yep | 12:38 |
jroll | and then bah, part of agent_base_vendor's prepare_instance_for_boot hits the agent to set up the bootloader | 12:39 |
jroll | so we can't move all of that to after the power off | 12:39 |
sambetts | shouldn't the workflow be: agent deploy does its thing to deploy -> power_off -> boot.prepare_instance -> power_on | 12:40 |
milan | morning gabriel-bezerra jroll nicodemos yufei and all! :) | 12:41 |
gabriel-bezerra | good morning, milan! | 12:41 |
jroll | sambetts: yeah, just need to move the set_boot_device out of prepare_instance, which is... weird | 12:41 |
milan | aarefiev, so seems the node was just too slow :) | 12:42 |
milan | thanks for the help btw | 12:42 |
lucasagomes | gabriel-bezerra, right... well we would always base it in the current state, since the power_off runs async you can talk to the hardware and wait the state of the node to be completetly powered off | 12:43 |
nicodemos | hey, milan | 12:43 |
lucasagomes | before setting the boot device | 12:43 |
aarefiev | milan: yeah, odd, I'm trying to fix this configure staff in ironic now | 12:43 |
sambetts | jroll: move it out of prepare_instance? surely it should be moved into there? | 12:43 |
sambetts | jroll: the deploy code (outside of installing the bootloader) shouldn't care about how the node is booted right? | 12:44 |
jroll | sambetts: oh yeah, my fault | 12:44 |
gabriel-bezerra | lucasagomes: I mean: something depends on boot device. now it has to consider the current and the eventual state. it may take forever before reaching the target state. | 12:44 |
sambetts | jroll: ah got it :) | 12:44 |
milan | aarefiev, so it should be set through ironic? | 12:44 |
*** sborkows has joined #openstack-ironic | 12:44 | |
jroll | sambetts: need to move prepare_instance *out* of agent's prepare_instance_to_boot | 12:44 |
sambetts | jroll: ah yeah *mind splodes* | 12:45 |
aarefiev | milan: it's in ironic, but we doesn't configure tempest after upgrade | 12:46 |
* milan checks | 12:47 | |
jroll | sambetts: so anyway I think switching the order is fine, idk | 12:47 |
lucasagomes | gabriel-bezerra hmm idk, sorry I wasn't paying much attention cause I'm looking at something else... But, with the lazy approach Ironic will be layering it for you. One just call node-set-boot-device <boot device>, and Ironic will apply it depending on the current power state: #1. node is power off, apply it now! #2. node is on, save to driver_internal_info and apply on the next time power_off/reboot is called | 12:48 |
sambetts | jroll: yeah, I don't see a problem with it at first glance | 12:48 |
gabriel-bezerra | jroll, sambetts: I don't have enough knowledge of the code do follow the discussion. By moving the prepare_instance out of the agent, where should it be put? | 12:48 |
lucasagomes | and all the logic will be contained inside the oneview driver, where moving things around affects every single driver in Ironic | 12:49 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: dnm, showing gabriel-bezerra some thoughts https://review.openstack.org/400733 | 12:51 |
jroll | gabriel-bezerra: ^^ something like that | 12:51 |
*** aparnav has quit IRC | 12:51 | |
*** vmud213 has quit IRC | 12:52 | |
gabriel-bezerra | lucasagomes: I see the point. We have to find a balance between: complexity of the states oneview driver has to deal with vs checking if changing the order of power-off and set boot will break other driver | 12:52 |
*** vmud213 has joined #openstack-ironic | 12:52 | |
gabriel-bezerra | From the nature of the change, I guess the latter will be easier/cheaper for all of us. | 12:52 |
jroll | sambetts: ^ sanity check me on 400733, obviously it would need more work and I didn't think through if that breaks the pxe driver, but that's basically what we said, right? | 12:52 |
gabriel-bezerra | jroll: thank you for the patch | 12:53 |
jroll | gabriel-bezerra: fyi, I'm not going to work on that to finish it, just giving you an idea | 12:53 |
lucasagomes | jroll, that's a bit odd... now "prepare_instance_to_boot()" does not call prepare_instance() from the boot interface | 12:53 |
*** dprince has joined #openstack-ironic | 12:53 | |
jroll | lucasagomes: naming aside... | 12:53 |
* jroll brb | 12:53 | |
lucasagomes | jroll, might break out of tree drivers as well | 12:54 |
gabriel-bezerra | jroll: I'd expect that. That's why I was asking for more guidance on what you were discussing. | 12:54 |
*** yolanda has quit IRC | 12:58 | |
* lucasagomes brb, will grab some food | 12:58 | |
*** lucasagomes is now known as lucas-hungry | 12:59 | |
sambetts | jroll: yeah pretty much | 12:59 |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-inspector: Use function is_valid_mac from oslo.utils https://review.openstack.org/400489 | 12:59 |
sambetts | jroll: although I wondering if the boot.prepare_instance function should be the one handling the "if iwdi" | 13:00 |
*** gcb has quit IRC | 13:00 | |
*** moshele has quit IRC | 13:01 | |
*** mjura has quit IRC | 13:01 | |
*** Michael-ZTE has joined #openstack-ironic | 13:04 | |
*** Michael-ZTE has quit IRC | 13:04 | |
*** foexle has quit IRC | 13:05 | |
*** galyna has quit IRC | 13:05 | |
jroll | lucas-hungry: yeah, out of tree is a fair point | 13:07 |
jroll | sambetts: it should, yeah | 13:07 |
*** trown|afk is now known as trown | 13:07 | |
*** vmud213 has quit IRC | 13:14 | |
*** vmud213 has joined #openstack-ironic | 13:14 | |
*** vmud213 has quit IRC | 13:16 | |
*** vmud213 has joined #openstack-ironic | 13:16 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 13:24 |
*** vmud213 has quit IRC | 13:26 | |
*** vmud213 has joined #openstack-ironic | 13:26 | |
*** bfournie has quit IRC | 13:28 | |
*** jcoufal has joined #openstack-ironic | 13:30 | |
*** chlong has joined #openstack-ironic | 13:36 | |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for X710 network card https://review.openstack.org/391836 | 13:38 |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 13:38 |
pas-ha | gabriel-bezerra: as for why not power-off -> change-boot-dev -> power-on - I suspect that in some cases 'reboot' (as single command) is lighter on HW than full power cycle | 13:38 |
pas-ha | so change-bood-dev -> reboot is better | 13:38 |
*** kun_huang has left #openstack-ironic | 13:38 | |
jroll | pas-ha: we don't do reboot today, though | 13:38 |
jroll | it's power off, power on | 13:39 |
gabriel-bezerra | yeah, it is power off -> change networking -> power on | 13:44 |
pas-ha | krtaylor: here? have a question on IBM CI | 13:45 |
*** Guest92303 has joined #openstack-ironic | 13:46 | |
*** Shashi has joined #openstack-ironic | 13:47 | |
Shashi | Provision state to deleted has issue on Newton release The state of a node always is in "Clean wait" state | 13:49 |
openstackgerrit | Merged openstack/bifrost: playbooks: bifrost-ironic-install: Fix regexp for pxe_bootfile_name https://review.openstack.org/400712 | 13:49 |
Shashi | While running "ironic node-set-provision-state <node> deleted | 13:52 |
patchbot | Error: No closing quotation | 13:52 |
*** yufei has quit IRC | 13:54 | |
*** lucas-hungry is now known as lucasagomes | 13:56 | |
*** bfournie has joined #openstack-ironic | 14:00 | |
*** ricardoas has quit IRC | 14:01 | |
*** ricardoas has joined #openstack-ironic | 14:03 | |
*** ccamacho|lunch is now known as ccamacho | 14:03 | |
*** lindycoder has joined #openstack-ironic | 14:06 | |
*** rloo has joined #openstack-ironic | 14:07 | |
krtaylor | pas-ha, whats your question? | 14:08 |
krtaylor | nm, read the scrollback | 14:08 |
pas-ha | 2016-11-22 03:00:48.132 | 2016-11-22 03:00:48.130 | https://review.openstack.org/#/c/264213/ Replace syslinux dependency by pxelinux and syslinux-common | 14:08 |
patchbot | patch 264213 - ironic - Replace syslinux dependency by pxelinux and syslin... (ABANDONED) | 14:08 |
TheJulia | Shashi: I think that makes sense because while disks are being cleaned on the baremetal node, the node should be in CLEAN_WAIT | 14:08 |
pas-ha | this is what ibm CI is doing | 14:08 |
pas-ha | and it fails to apply the cherry-pick | 14:09 |
*** moshele has joined #openstack-ironic | 14:09 | |
pas-ha | I kind of suspect why you were doing that - but we think we fixed PXE job setup in DevStack under Xenial, so I propose you try to remove that hack | 14:10 |
pas-ha | http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/44/397844/6/check/tempest-dsvm-ironic-agent_ipmitool/085cbad/ | 14:10 |
*** Goneri has joined #openstack-ironic | 14:10 | |
pas-ha | that ^^.. line was from here | 14:10 |
*** glonlas has joined #openstack-ironic | 14:11 | |
*** baoli has joined #openstack-ironic | 14:12 | |
*** glonlas has quit IRC | 14:12 | |
krtaylor | mmedvede, ^^ | 14:12 |
mmedvede | pas-ha: thanks for heads up, looking into that | 14:13 |
*** daemontool has joined #openstack-ironic | 14:13 | |
krtaylor | pas-ha, yes, thanks | 14:13 |
milan | aarefiev, I've just updated https://review.openstack.org/#/c/400410/2 to depend on your https://review.openstack.org/#/c/400687/2 let's see ;) | 14:14 |
patchbot | patch 400410 - ironic-inspector - [WIP] Tempest config update | 14:14 |
patchbot | patch 400687 - ironic - Grenade: configure tempest with Ironic plugin | 14:14 |
pas-ha | np - I urgently need all third-party CI working :) as I remove old vendor passthru and need to be sure I do not break vendors | 14:14 |
*** alex_xu has quit IRC | 14:14 | |
krtaylor | pas-ha, ah, understood, we want it working too :) | 14:14 |
*** glonlas has joined #openstack-ironic | 14:14 | |
*** glonlas has quit IRC | 14:14 | |
aarefiev | milan: I've already tested, the problem is "upgrade project" runs before tempest upgrade | 14:15 |
*** glonlas has joined #openstack-ironic | 14:15 | |
krtaylor | pas-ha, what it the patch for vendor passthru? | 14:15 |
* krtaylor looks | 14:15 | |
pas-ha | krtaylor: the one probably conserning IBM is https://review.openstack.org/#/c/397844/6 | 14:16 |
patchbot | patch 397844 - ironic - Remove agent vendor passthru from most drivers | 14:16 |
pas-ha | (as you do not seem to use any "vedor"ish drivers, just agent_ipmitool | 14:16 |
krtaylor | yep, looking at it now | 14:16 |
krtaylor | no, we should be ok | 14:16 |
*** yolanda has joined #openstack-ironic | 14:17 | |
pas-ha | just to be clear (as many people freak out :) ) - this is not removing vendor passthru at all, only the old stuff that agent was using for lookup and heartbeats before they were promoted to core API | 14:17 |
* pas-ha hints that fresh IPA will not even call into those any longer | 14:18 | |
*** alex_xu has joined #openstack-ironic | 14:18 | |
*** mjturek has joined #openstack-ironic | 14:18 | |
krtaylor | I won't freak out, I promise :) | 14:19 |
milan | aarefiev, so if your change puts the config in place while in upgrade.sh, the inspector smoke won't be skipped anymore ? | 14:19 |
pas-ha | krtaylor: :) | 14:19 |
mjturek | hey pas-ha, I don't have any history on this channel. Can you point out the patch in our CI you're referring to? | 14:19 |
TheJulia | gabriel-bezerra: to your question earlier, from my point of view, the agent may not be the only deployment mechanism, so we can't just assume that in all cases the agent can run and take care of the operation. | 14:19 |
pas-ha | mjturek: https://review.openstack.org/#/c/397844/6 | 14:19 |
patchbot | patch 397844 - ironic - Remove agent vendor passthru from most drivers | 14:19 |
krtaylor | mjturek, https://review.openstack.org/#/c/397844/ | 14:19 |
patchbot | patch 397844 - ironic - Remove agent vendor passthru from most drivers | 14:19 |
pas-ha | i'll post a new patch-set now, so use CI report from patch-set 6 | 14:20 |
*** Guest92303 has quit IRC | 14:21 | |
mjturek | pas-ha: krtaylor mentioned a non-related cherry pick. Which one are you referring to? | 14:21 |
krtaylor | mjturek, see the pastebin I shared | 14:22 |
mjturek | krtaylor: ahh I see, pas-ha you're referring to https://review.openstack.org/#/c/264213/ | 14:22 |
patchbot | patch 264213 - ironic - Replace syslinux dependency by pxelinux and syslin... (ABANDONED) | 14:22 |
pas-ha | around timestamp 016-11-22 03:00:52.631 at http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/44/397844/6/check/tempest-dsvm-ironic-agent_ipmitool/085cbad/console.html | 14:22 |
pas-ha | yes | 14:23 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update Ansible version to stable-2.1 https://review.openstack.org/378041 | 14:23 |
mjturek | unfortunately, I still think our CI needs this patch. the arch we use doesn't have a syslinux package | 14:23 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 14:23 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 14:23 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add support to update keystone for bifrost https://review.openstack.org/380226 | 14:23 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 14:24 |
pas-ha | then you have to refresh it - it fails to cherry-pick and fails the whole job | 14:24 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone install and clouds.yaml to testing https://review.openstack.org/380228 | 14:24 |
aarefiev | milan: no (, the tempest upgrade step runs after project upgrade | 14:24 |
mjturek | pas-ha: right will do | 14:24 |
pas-ha | mjturek: what distro are you on? just curious | 14:25 |
milan | aarefiev, ah ok :-/ | 14:25 |
*** rbartal has quit IRC | 14:26 | |
pas-ha | may be we can start thinking of moving to bindep or such as the distro pinning in devstack is lacking some abilities like '!=' | 14:26 |
openstackgerrit | Aline Bousquet proposed openstack/ironic: Add tests for the node update API https://review.openstack.org/400792 | 14:26 |
mjturek | pas-ha: Ubuntu 16.04 | 14:26 |
pas-ha | it has syslinux | 14:26 |
mjturek | not for ppc64 | 14:26 |
pas-ha | ouch | 14:26 |
mjturek | pas-ha: https://launchpad.net/ubuntu/xenial/+source/syslinux | 14:27 |
gabriel-bezerra | TheJulia: indeed, but it is what is causing the break today, as it is changing the device with the machine turned on. | 14:27 |
mjturek | we use syslinux-common and pxelinux instead | 14:27 |
pas-ha | so you now can change the patch to just remove that line from 'debs' file | 14:27 |
gabriel-bezerra | TheJulia: our drivers: agent_pxe_oneview, iscsi_pxe_oneview, use it only. | 14:28 |
pas-ha | as pxelinux and syslinux-common are there already and pinned to 'xenial' | 14:28 |
gabriel-bezerra | TheJulia: and I recognize that driver composition removes this assumption. | 14:28 |
pas-ha | (or just run a manual bash script...) | 14:28 |
mjturek | pas-ha: oh cool | 14:28 |
*** daemontool has quit IRC | 14:28 | |
mjturek | thanks :) | 14:29 |
pas-ha | mjturek: merged today https://review.openstack.org/#/c/399485/ | 14:29 |
patchbot | patch 399485 - ironic - Fix PXE setup for fresh Ubuntu Xenial (MERGED) | 14:29 |
mjturek | awesome | 14:29 |
*** vikrant has quit IRC | 14:29 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 14:30 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add Keystone support to ironic.conf https://review.openstack.org/382469 | 14:30 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 14:30 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 14:30 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 14:30 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 14:30 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 14:30 |
*** rbartal has joined #openstack-ironic | 14:33 | |
TheJulia | gabriel-bezerra: I guess I'm not exactly getting the sticking point as to where the disagreement really lies and what our options are, along with shared context. Admittedly, I've not read the entire buffer yet this morning, I'm trying to get some rebasing done first thing :) | 14:33 |
*** yolanda has quit IRC | 14:36 | |
*** betherly has quit IRC | 14:37 | |
*** rbartal has quit IRC | 14:37 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Adding keystone support to inspector config https://review.openstack.org/383425 | 14:38 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add test script for Bifrost using keystone https://review.openstack.org/383426 | 14:38 |
*** mjura has joined #openstack-ironic | 14:39 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add test script for Bifrost using keystone https://review.openstack.org/383426 | 14:40 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update the task label for inspection https://review.openstack.org/383427 | 14:41 |
*** daemontool has joined #openstack-ironic | 14:41 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone capability to base install https://review.openstack.org/385040 | 14:41 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 14:41 |
openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 14:43 |
*** yolanda has joined #openstack-ironic | 14:43 | |
*** mjura has quit IRC | 14:47 | |
*** jaosorior has quit IRC | 14:49 | |
*** rbartal has joined #openstack-ironic | 14:50 | |
*** jaosorior has joined #openstack-ironic | 14:50 | |
*** rama_y has joined #openstack-ironic | 14:51 | |
*** jaosorior has quit IRC | 14:51 | |
*** jaosorior has joined #openstack-ironic | 14:51 | |
*** vmud213 has quit IRC | 14:52 | |
*** vmud213 has joined #openstack-ironic | 14:52 | |
*** vmud213 has quit IRC | 14:55 | |
*** vmud213 has joined #openstack-ironic | 14:55 | |
*** yufei has joined #openstack-ironic | 14:56 | |
*** vmud213 has quit IRC | 14:57 | |
*** vmud213 has joined #openstack-ironic | 14:57 | |
*** rama_y has quit IRC | 14:58 | |
*** glonlas_ has joined #openstack-ironic | 14:59 | |
*** vmud213 has quit IRC | 14:59 | |
lucasagomes | sambetts, (maybe a silly question) the attach/detach API work is most an "enhancement" to make it better to use the network isolation feature, right ? | 15:00 |
*** mjura has joined #openstack-ironic | 15:00 | |
*** anush has joined #openstack-ironic | 15:00 | |
*** vmud213 has joined #openstack-ironic | 15:00 | |
openstackgerrit | Michael Turek proposed openstack/ironic: [WIP] Remove syslinux dependency for POWER https://review.openstack.org/264213 | 15:02 |
sambetts | lucasagomes: It makes the iteraction with the NetworkInterfaces less implict (e.g. setting a json field on a port) and more explict (e.g. an actual API) | 15:02 |
*** dnikishov has joined #openstack-ironic | 15:02 | |
lucasagomes | sambetts, cool, ty | 15:03 |
Shashi | TheJulia: understood, but that state is not changing from clean wait to available, I have waited approx 1 hour. | 15:03 |
*** glonlas has quit IRC | 15:03 | |
TheJulia | Shashi: do you see a clean_step in progress? | 15:04 |
*** karthiks has quit IRC | 15:04 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: WIP: Full-featured objects. POC https://review.openstack.org/400335 | 15:06 |
*** glonlas_ has quit IRC | 15:07 | |
Shashi | TheJulia: No, clean_steps not happening. | 15:07 |
*** glonlas has joined #openstack-ironic | 15:08 | |
Shashi | TheJulia:I have waited to long but no progress | 15:08 |
*** rbartal has quit IRC | 15:08 | |
*** dtantsur|brb is now known as dtantsur | 15:10 | |
TheJulia | Shashi: Well, erase_devices can take quite some time with disks that do not support ATA secure erase. Is the node powered on? Do you see anything on the console? | 15:11 |
*** yolanda has quit IRC | 15:12 | |
Shashi | TheJulia: Yes I could see but after one specific message the operation were paused | 15:13 |
TheJulia | how so? | 15:14 |
Shashi | TheJulia: If you want I can share the message as well from console | 15:14 |
TheJulia | Shashi: That would be helpful for us | 15:15 |
Shashi | ok | 15:15 |
milan | sambetts, mind another review? https://review.openstack.org/#/c/344921 got +2 from dtantsur fixed your comments from yesterday ;) | 15:15 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 15:15 |
*** anush has quit IRC | 15:17 | |
rloo | hi and morning all, milan, Shashi, TheJulia, dtantsur, lucasagomes, sambetts | 15:18 |
TheJulia | good morning rloo | 15:18 |
sborkows | morning rloo | 15:18 |
milan | morning rloo! :) | 15:18 |
Shashi | Good morning rloo | 15:18 |
lucasagomes | rloo, hi there | 15:18 |
rloo | sambetts, vsaienk0, vdrok: wrt mandatory (or not) portgroups.address, I'll leave it for you three to decide, cuz I'm not sure I grok your comments at https://review.openstack.org/#/c/396610/8/specs/approved/ironic-ml2-integration.rst | 15:19 |
patchbot | patch 396610 - ironic-specs - Add bonding configuration fields to portgroups | 15:19 |
* TheJulia loads it up | 15:19 | |
sborkows | I have a question: is cleaning executed during the inspection? I mean, when I start the node inspection, will the cleaning steps defined in hardware managers will be executed? | 15:19 |
dtantsur | morning rloo, TheJulia, sborkows and everyone I missed | 15:20 |
*** karthiks has joined #openstack-ironic | 15:20 | |
dtantsur | sborkows, no | 15:20 |
vdrok | morning TheJulia rloo and sborkows | 15:20 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 15:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 15:21 |
*** rbartal has joined #openstack-ironic | 15:22 | |
snozhka | jroll, ping | 15:23 |
jroll | snozhka: hi | 15:24 |
snozhka | jroll, hi, do you have couple mins talk about root hints tests? | 15:25 |
jroll | snozhka: sure, lucasagomes may also be able to help, he knows more than me about that feature | 15:25 |
jroll | but go ahead | 15:25 |
lucasagomes | o/ | 15:26 |
snozhka | jroll, i think we should be able to create VM's with couple of disks for that | 15:26 |
snozhka | lucasagomes, jroll am i right > | 15:26 |
openstackgerrit | Merged openstack/ironic: mask private keys for the ssh power driver. https://review.openstack.org/400273 | 15:27 |
jroll | snozhka: yeah, that seems like the right thing to do, don't see why not | 15:27 |
lucasagomes | snozhka, that wouldn't be a bad idea, given that we could set "wwn", "serial", etc... in the libvirt xml | 15:27 |
snozhka | lucasagomes, jroll the upstream ci uses devstack installation afaik | 15:27 |
snozhka | lucasagomes, jroll so we need to prepare VM with several disks and try to boot each of them | 15:29 |
jroll | snozhka: yes, I think that makes sense... the hard part is, tempest shouldn't be coupled to devstack | 15:29 |
jroll | so we'll need some way for tempest to know about the disks | 15:29 |
snozhka | jroll, and the actual node's ids... | 15:30 |
jroll | yeah | 15:30 |
jroll | hrm | 15:30 |
jroll | maybe it'll just need to be a config, I can't imagine a way to find that via the api | 15:31 |
TheJulia | rloo: sambetts: vsaienk0: vdrok: I tossed two cents in on the mandatory address field option. Feel free to take it as such. | 15:32 |
*** syed__ has joined #openstack-ironic | 15:32 | |
rloo | thx TheJulia. The scale is shifting towards the non-mandatory camp. Maybe we can have a vote at our next meeting, how exciting :D | 15:34 |
sambetts | TheJulia: thanks :) | 15:34 |
snozhka | jroll, for node id looks like it should be definitely property | 15:35 |
*** rpioso has joined #openstack-ironic | 15:36 | |
snozhka | jroll, but for wwn/serial/... we can try to create some api that will get these info from libvirt | 15:36 |
* TheJulia wonders if it is too early for lunch | 15:36 | |
*** hoonetorg has joined #openstack-ironic | 15:37 | |
jroll | snozhka: well, that doesn't help with testing non-devstack clouds | 15:37 |
snozhka | jroll, then specify these data in configs | 15:38 |
sambetts | jroll, snozhka: I don't think tempest needs to be aware of the fact it has multiple disks and the root device hints are set, can't we just configure the node in Ironic and then have tempest nova boot it, and if its successful its worked right? if we deploy to the wrong disk it won't boot will it? | 15:39 |
sambetts | s/we just configure/devstack configure/ | 15:39 |
sambetts | jroll, snozhka: or are we planned on adding addtional logic to check the disk when we ssh in ? | 15:40 |
lucasagomes | sambetts, https://review.openstack.org/#/c/400673 LGTM, but it triggered a bug in the code rendering the release note (it fails when starting with `) | 15:40 |
patchbot | patch 400673 - ironic - Rename and redocument the Cisco UCSM driver | 15:40 |
sambetts | lucasagomes: ah shoot | 15:41 |
jroll | sambetts: we want to verify that it chose the disk, no? | 15:41 |
snozhka | sambetts, jroll, lucasagomes i think we should add this logic | 15:41 |
*** ccamacho has quit IRC | 15:41 | |
snozhka | sambetts, jroll lucasagomes to make sure that correct disk was selected | 15:41 |
snozhka | and others remains empty | 15:42 |
sambetts | jroll: if it doesn't choose the bootable disk it will fail to become sshable right? or do we change the boot flag on the disk? | 15:42 |
lucasagomes | snozhka, ++ yeah, that's how we can guarantee that it worked | 15:42 |
jroll | sambetts: I assume it would still work | 15:42 |
lucasagomes | once we ssh into the instance we can check the "mount" command and see if the root partitions is coming from the right disk | 15:43 |
sambetts | ah :/ I was thinking 1 bootable disk and 1 not bootable disk, and we use the hints to force the bootable disk (but I'm probably missing something) | 15:43 |
sambetts | then it won't boot if it picks the wrong disk | 15:44 |
lucasagomes | sambetts, what would be a not bootable disk ? | 15:44 |
lucasagomes | unless we make a disk that is so small that will fail the deployment if it was pick | 15:44 |
lucasagomes | like 2MiB | 15:44 |
*** Alagar has quit IRC | 15:44 | |
*** Alagar has joined #openstack-ironic | 15:45 | |
*** jkilpatr has quit IRC | 15:45 | |
*** jkilpatr has joined #openstack-ironic | 15:46 | |
*** daemontool has quit IRC | 15:46 | |
*** vmud213 has quit IRC | 15:46 | |
sambetts | lucasagomes: if the boot flag isn't set on the device or it wasn't the device selected in the boot order, (although I guess we probably handle both these cases) | 15:47 |
lucasagomes | sambetts, the boot flag is in the partition table, so whole disk image will have it set or Ironic will set it for partition images | 15:48 |
lucasagomes | it's not in the disk itself | 15:49 |
NobodyCam | good morning IRonic'ers | 15:50 |
dtantsur | morning NobodyCam | 15:50 |
lucasagomes | NobodyCam, morning | 15:50 |
sambetts | lucasagomes: ah thats what I'm missing | 15:50 |
lucasagomes | sambetts, yeah... I wonder if QEMU have a faulty disk driver or something | 15:51 |
milan | aarefiev, you were right: http://logs.openstack.org/10/400410/2/check/gate-grenade-dsvm-ironic-inspector/b326e21/console.html#_2016-11-22_15_28_21_852610 | 15:51 |
openstackgerrit | Merged openstack/bifrost: Update Ansible version to stable-2.1 https://review.openstack.org/378041 | 15:52 |
*** rloo has quit IRC | 15:52 | |
sambetts | lucasagomes: when we set the boot order and there are multiple disks do we specify a disk or just try all disks? | 15:52 |
*** rloo has joined #openstack-ironic | 15:52 | |
*** yolanda has joined #openstack-ironic | 15:52 | |
vdrok | TheJulia: sambetts rloo thanks for reviews! maybe making it not mandatory indeed makes sense. more checks on the api side make me a bit sad tho :( | 15:53 |
Shashi | TheJulia: Sorry for delay, but colud not able to send attachment | 15:53 |
vdrok | morning NobodyCam | 15:53 |
lucasagomes | sambetts, we do not, I think it will pick the first disk. Protocols like IPMI for example doesn't even have a way to specify which disk you want | 15:53 |
rloo | vdrok: heh. don't be sad. it is just code, think of how happy our users will be :) | 15:54 |
lucasagomes | it's only "pxe", "hd", "network" etc... | 15:54 |
Shashi | TheJulia: Also I trying to find the workaround | 15:54 |
sambetts | lucasagomes: ah :/ that won't work either then | 15:54 |
vdrok | rloo: and who's going to think about the developers? :D | 15:54 |
lucasagomes | sambetts, more complex BMCs like DRAC/iLO/UCS would allow it, but our API doesn't have a way to specify which one | 15:54 |
lucasagomes | :-/ | 15:54 |
rloo | morning NobodyCam | 15:54 |
lucasagomes | sambetts, yeah... maybe making a really tiny disk would do it tho | 15:54 |
milan | morning NobodyCam! :) | 15:55 |
*** jaosorior has quit IRC | 15:55 | |
lucasagomes | sambetts, if pick the deployment will fail cause it's too small to the image to be copied onto | 15:55 |
sambetts | lucasagomes: doesn't our code avoid those disks automatically though? | 15:55 |
lucasagomes | sambetts, only when guessing the disk (== no root device hints specified) | 15:56 |
lucasagomes | then it looks for a disk >= 4GiB | 15:56 |
rloo | vdrok: the developers. Isn't a 'Thank you' enough? :D | 15:56 |
openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: WIP: Full-featured objects. POC https://review.openstack.org/400335 | 15:57 |
sambetts | lucasagomes: ah ok, so we could do a negative test, e.g. boot fails because the root device hint you've picked points to a disk that is too small | 15:57 |
*** Guest92303 has joined #openstack-ironic | 15:57 | |
lucasagomes | sambetts, ++ | 15:57 |
NobodyCam | Morning rloo milan vdrok dtantsur, lucasagomes :) | 15:58 |
snozhka | sambetts, lucasagomes nice case! | 15:58 |
nicodemos | morning, NobodyCam. o/ | 15:58 |
NobodyCam | morning nicodemos :) | 15:58 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add Keystone support to ironic.conf https://review.openstack.org/382469 | 16:00 |
openstackgerrit | Merged openstack/ironic: Convert iPXE boot script to Jinja template https://review.openstack.org/394396 | 16:01 |
*** glonlas has quit IRC | 16:01 | |
*** anush has joined #openstack-ironic | 16:01 | |
*** soliosg has joined #openstack-ironic | 16:02 | |
moshele | lucasagomes: hi can you review https://review.openstack.org/#/c/334816/ | 16:02 |
patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs | 16:02 |
lucasagomes | moshele, hi, will take a look in a few | 16:02 |
moshele | lucasagomes: thanks | 16:02 |
*** links has quit IRC | 16:04 | |
*** yolanda has quit IRC | 16:04 | |
sborkows | regarding this patch - I have made a mistake when pushing changes in my manager and that's why I am there as a commiter | 16:06 |
*** rcernin_ has quit IRC | 16:06 | |
Shashi | TheJulia: after updating erase_device_priority=0 in /etc/ironic/ironic.conf, hat works well, Any suggestion regarding this config param? | 16:07 |
openstackgerrit | Merged openstack/ironic-inspector: Use function is_valid_mac from oslo.utils https://review.openstack.org/400489 | 16:08 |
lucasagomes | Shashi, that will disable the clean step that shred/zero's the disk but won't disable cleaning in general | 16:09 |
* lucasagomes if that's what you want, it seems fine | 16:09 | |
*** glonlas has joined #openstack-ironic | 16:09 | |
*** glonlas has quit IRC | 16:09 | |
Shashi | Ok, but is this suggested? | 16:10 |
*** glonlas has joined #openstack-ironic | 16:10 | |
Shashi | lucasagomes: Is this suggested? packstack doesnot enable it by default? | 16:11 |
lucasagomes | Shashi, depends on your use case really, shreding the disk will get rid of all data left by the previous tenants | 16:11 |
lucasagomes | Shashi, overall it's a (very) good thing. But that also increases a lot of the time it takes for a node to go back to the pool of available nodes for deployment | 16:12 |
lucasagomes | so it's trade-off | 16:12 |
lucasagomes | a trade-off* | 16:12 |
Shashi | lucasagomes: So I can go with this? | 16:13 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal port commands https://review.openstack.org/365692 | 16:13 |
lucasagomes | Shashi, since you are using packstack, I assume you only wants to test Ironic. Right ? It should be fine then | 16:13 |
Shashi | lucasagomes: Thanks :) | 16:14 |
lucasagomes | if you have a cloud with untrusted tenants using it I would suggest you leave it enabled | 16:14 |
*** rbartal has quit IRC | 16:14 | |
lucasagomes | and so on | 16:14 |
*** Guest92303 has quit IRC | 16:15 | |
*** Guest3787 has quit IRC | 16:16 | |
snozhka | jroll, regarding disk's wwn/serial/.... | 16:17 |
snozhka | jroll, we can use introspection to get these data afaiu from http://docs.openstack.org/developer/tripleo-docs/advanced_deployment/root_device.html#using-introspection-data-to-find-the-root-device | 16:17 |
*** mjturek has quit IRC | 16:18 | |
*** mjura has quit IRC | 16:19 | |
snozhka | lucasagomes, am i right ? ^^^ | 16:20 |
lucasagomes | snozhka, right | 16:21 |
snozhka | lucasagomes, thank you! | 16:21 |
* milan relocates | 16:21 | |
lucasagomes | snozhka, for virtio disks tho it's most going to return "null" (except from vendor/name/size) | 16:21 |
lucasagomes | if you want a wwn/serial etc... you need to add to the libvirt xlm | 16:22 |
lucasagomes | xml* | 16:22 |
*** Shashi has quit IRC | 16:22 | |
snozhka | lucasagomes, got it! | 16:22 |
* snozhka start reading kvm docs | 16:22 | |
jroll | snozhka: you can't assume the cloud you are running tempest against has inspector, though | 16:25 |
*** milan has quit IRC | 16:26 | |
snozhka | jroll, agree, but can handle it in properties, like if inspector is missing try getting wwns from config, otherwise fail | 16:27 |
snozhka | jroll, does it make sense ? | 16:27 |
jroll | snozhka: hm, maybe | 16:28 |
*** yolanda has joined #openstack-ironic | 16:29 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Grenade: test https://review.openstack.org/400690 | 16:30 |
snozhka | jroll, imo it would be better to add this, to avoid hardcoding in some cases | 16:30 |
*** Alagar has quit IRC | 16:30 | |
*** Alagar has joined #openstack-ironic | 16:31 | |
*** [1]rpioso has joined #openstack-ironic | 16:32 | |
*** mchasal has joined #openstack-ironic | 16:32 | |
jroll | snozhka: yeah, trying to think about it, in meetings right now | 16:32 |
snozhka | jroll, thank you! | 16:32 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal chassis commands https://review.openstack.org/366158 | 16:32 |
jroll | snozhka: the weird part to me would be putting code to talk to inspector in the ironic tempest plugin | 16:33 |
*** rpioso has quit IRC | 16:33 | |
snozhka | jroll, also thinking about it | 16:33 |
TheJulia | jroll: Of course, we could just rename it to the baremetal tempest plugin *ducks* | 16:34 |
snozhka | jroll, probably we can use inspector tempest plugin, but generally yeah, it's little bit weird, need to think about it additionally | 16:34 |
*** moshele has quit IRC | 16:34 | |
jroll | TheJulia: lawd | 16:34 |
aarefiev | milan: imo grenade should call plugins to configure this, also looks like timeout again ( | 16:35 |
*** rbrud has joined #openstack-ironic | 16:39 | |
*** aNuposic has joined #openstack-ironic | 16:42 | |
*** snozhka is now known as snozhka|afk | 16:42 | |
*** yolanda has quit IRC | 16:42 | |
sborkows | dtantsur, would you mind looking at my comment on https://bugs.launchpad.net/ironic-python-agent/+bug/1623659 ? It is about that X710 Hardware manager. | 16:44 |
openstack | Launchpad bug 1623659 in ironic-python-agent "LLDP discovery is not working when IPA is run on system with X710 network cards" [Medium,In progress] - Assigned to Szymon Borkowski (szymon-borkowski) | 16:44 |
*** Guest92303 has joined #openstack-ironic | 16:44 | |
dtantsur | sborkows, not right now, sorry - in the middle of coding | 16:44 |
sborkows | dtantsur, I don't need it for now, just look at it when you can | 16:46 |
dtantsur | sure | 16:46 |
*** rama_y has joined #openstack-ironic | 16:50 | |
*** yolanda has joined #openstack-ironic | 16:51 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 17:00 |
*** betherly has joined #openstack-ironic | 17:01 | |
*** Alagar has quit IRC | 17:02 | |
JayF | jroll: do we have any tempest tests today around maintenance afayk? | 17:04 |
*** anush has quit IRC | 17:04 | |
jroll | JayF: $ ag maintenance ironic_tempest_plugin | 17:04 |
jroll | gives me nothing | 17:05 |
JayF | that's what I thought. | 17:05 |
*** athomas has quit IRC | 17:05 | |
*** fragatina has quit IRC | 17:06 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 17:06 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Implement interfaces defaults calculation logic https://review.openstack.org/400881 | 17:07 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Introduce generic hardware types https://review.openstack.org/400678 | 17:08 |
*** athomas has joined #openstack-ironic | 17:09 | |
* jroll -> lunch before I bash my head in over this tempest thing | 17:10 | |
dtantsur | sborkows, responded, dunno how cleaning is related at all here | 17:10 |
lucasagomes | sambetts, sorry I missed something in my previous review of ^ | 17:11 |
lucasagomes | sambetts, the new drivers in the setup.cfg are pointing to the wrong class | 17:11 |
lucasagomes | (they are pointing to the old ones which will now log a warning) | 17:11 |
sambetts | lucasagomes: woops! Thanks!! | 17:12 |
*** glonlas has quit IRC | 17:12 | |
sborkows | dtantsur: thanks for response, I will discuss it with Tomasz | 17:13 |
*** tuan_luong has joined #openstack-ironic | 17:14 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 17:15 |
*** [1]rpioso has quit IRC | 17:17 | |
dtantsur | o/ | 17:18 |
*** dtantsur is now known as dtantsur|afk | 17:18 | |
sambetts | o/ dtantsur|afk | 17:18 |
mgould | snozhka|afk, jroll: reanimating your discussion from 2hrs ago, I'm wondering if root device hints can be sensibly tested within ironic-tempest-plugin at all | 17:23 |
mgould | or if it's something we should test in non-tempest functional tests | 17:24 |
mgould | you can't control the wwns unless you're backing your Ironic with VMs, which won't be true of a general cloud | 17:24 |
lucasagomes | calling it a day, have a great evening all! | 17:25 |
*** lucasagomes is now known as lucas-afk | 17:25 | |
mgould | do we expose "give me a node with >1 disk and its WWNs" via api? | 17:25 |
mgould | gnight lucas-afk | 17:25 |
*** penick has joined #openstack-ironic | 17:28 | |
*** dprince has quit IRC | 17:29 | |
*** moshele has joined #openstack-ironic | 17:29 | |
*** fellypefca has joined #openstack-ironic | 17:29 | |
*** dprince has joined #openstack-ironic | 17:30 | |
*** tuan_luong has quit IRC | 17:31 | |
*** jvgrant_ has quit IRC | 17:32 | |
*** jvgrant has joined #openstack-ironic | 17:37 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work If no api url configured https://review.openstack.org/399857 | 17:38 |
*** Alagar has joined #openstack-ironic | 17:39 | |
openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 17:39 |
*** yolanda has quit IRC | 17:40 | |
openstackgerrit | Merged openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 17:41 |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured https://review.openstack.org/399857 | 17:44 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 17:45 |
*** sborkows has quit IRC | 17:46 | |
openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured https://review.openstack.org/399857 | 17:49 |
*** anush has joined #openstack-ironic | 17:49 | |
openstackgerrit | Merged openstack/python-ironicclient: Add tests for maintenance mode commands https://review.openstack.org/337259 | 17:50 |
*** moshele has quit IRC | 17:51 | |
*** yufei has quit IRC | 17:52 | |
*** dprince has quit IRC | 17:58 | |
*** dprince has joined #openstack-ironic | 17:59 | |
*** fragatina has joined #openstack-ironic | 17:59 | |
*** derekh has quit IRC | 18:01 | |
*** mgould is now known as mgould|afk | 18:02 | |
*** penick has quit IRC | 18:02 | |
*** baoli has quit IRC | 18:02 | |
*** ralonsoh has quit IRC | 18:03 | |
*** baoli has joined #openstack-ironic | 18:03 | |
*** penick has joined #openstack-ironic | 18:04 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface DB field and object https://review.openstack.org/348005 | 18:06 |
*** Alagar has quit IRC | 18:11 | |
*** rama_y has quit IRC | 18:14 | |
*** Alagar has joined #openstack-ironic | 18:14 | |
*** vinsh has quit IRC | 18:15 | |
*** glonlas has joined #openstack-ironic | 18:15 | |
*** glonlas has quit IRC | 18:15 | |
*** glonlas has joined #openstack-ironic | 18:16 | |
*** [1]rpioso has joined #openstack-ironic | 18:16 | |
*** Romanenko_K has quit IRC | 18:16 | |
*** Guest92303 has quit IRC | 18:17 | |
*** yolanda has joined #openstack-ironic | 18:20 | |
*** rama_y has joined #openstack-ironic | 18:21 | |
*** e0ne has quit IRC | 18:22 | |
*** yolanda has quit IRC | 18:23 | |
*** yolanda has joined #openstack-ironic | 18:23 | |
*** glonlas has quit IRC | 18:26 | |
*** anush has quit IRC | 18:27 | |
*** penick has quit IRC | 18:29 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface to base driver class https://review.openstack.org/348006 | 18:32 |
*** glonlas has joined #openstack-ironic | 18:33 | |
*** yolanda has quit IRC | 18:34 | |
*** penick has joined #openstack-ironic | 18:34 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add base cinder common interface https://review.openstack.org/355625 | 18:34 |
*** abalfour has joined #openstack-ironic | 18:35 | |
*** trown is now known as trown|lunch | 18:43 | |
*** soliosg has quit IRC | 18:43 | |
*** strigazi is now known as strigazi_AFK | 18:46 | |
*** Alagar has quit IRC | 18:46 | |
*** penick has quit IRC | 18:47 | |
*** ohamada has quit IRC | 18:48 | |
*** Sukhdev has joined #openstack-ironic | 18:48 | |
*** Alagar has joined #openstack-ironic | 18:49 | |
*** anush has joined #openstack-ironic | 18:49 | |
*** penick has joined #openstack-ironic | 18:50 | |
abalfour | Not sure where to go with this one, and I couldn't find any open bugs, and could use some advice as I don't know pecan at all... Is this an ironic bug? a configuration issue? operator error? http://paste.openstack.org/show/590118/ | 18:51 |
*** dprince has quit IRC | 18:53 | |
*** yolanda has joined #openstack-ironic | 18:55 | |
*** ijw has joined #openstack-ironic | 18:56 | |
*** yolanda has quit IRC | 18:59 | |
*** glonlas_ has joined #openstack-ironic | 19:02 | |
*** ChubYann has joined #openstack-ironic | 19:02 | |
*** glonlas_ has quit IRC | 19:03 | |
*** ijw has quit IRC | 19:03 | |
*** glonlas_ has joined #openstack-ironic | 19:03 | |
*** glonlas has quit IRC | 19:05 | |
*** glonlas has joined #openstack-ironic | 19:07 | |
*** dprince has joined #openstack-ironic | 19:08 | |
*** glonlas_ has quit IRC | 19:10 | |
rloo | abalfour: looks like a bug to me. Would you mind opening a bug about it: https://bugs.launchpad.net/ironic | 19:10 |
abalfour | sure thing. wasn't sure if it was an ironic bug, a pecan bug or a me bug. :) | 19:10 |
TheJulia | abalfour: additionally, please detail how your running the api. | 19:11 |
rloo | abalfour: i was wondering what it would do if you put the name in quotes. i don't recall the details of the code but it shouldn't be hard to track down. | 19:11 |
abalfour | Yeah, it's pecan/core.py in find_controller() | 19:12 |
rloo | abalfour: pecan, oh pecan, why do you do this to us... | 19:12 |
*** ijw has joined #openstack-ironic | 19:12 | |
rloo | TheJulia: wrt volume connector work, i have this link but I suspect it isn't correct? https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/1526231 | 19:14 |
abalfour | http://paste.openstack.org/show/590126/ | 19:14 |
rloo | abalfour: thx! | 19:15 |
JayF | abalfour: you should go and file the best bug ever now, that's great research, and toss that in the bug and I'm sure someone will get it fixed | 19:15 |
rloo | abalfour: please add that info to the bug report. and feel free to fix it too if you'd like! | 19:15 |
TheJulia | rloo: ugh, fixing tags, they got mucked up in rebasing fun | 19:15 |
JayF | abalfour: alternatively, I'm glad to review any patches you put up to fix the issue :D | 19:15 |
*** yolanda has joined #openstack-ironic | 19:16 | |
abalfour | Don't know pecan or ironic well enough to figure out where the fix should go. :( | 19:16 |
rloo | TheJulia: ah, ok, let me know when they are fixed. In the meantime, is this the first in the chain: https://review.openstack.org/#/c/200983/ | 19:16 |
patchbot | patch 200983 - ironic - Add volume_connector table to save connector infor... | 19:16 |
JayF | that's fine, filing a bug with all that informatino will be super valuable | 19:16 |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 19:17 |
penick | pro-tip: Make sure you’re not leaving stale leases lying around. The isc-dhcpd omapi runs reeallllllyyyyyyy slow if you’ve got around 30K leases. | 19:17 |
TheJulia | rloo: topics for volume connector fixed | 19:19 |
rloo | penick: whoops. Or make sure you don't have so many :) There should be an Ops FAQ or somewhere to add that | 19:19 |
rloo | TheJulia: thx! | 19:19 |
penick | Yeah there’s a bug in our isc-dhcpd driver, as well as our horizontal import code. Even manually cleaning up the lease isn’t working. Something, somewhere, is re-adding them. | 19:20 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Use kvm for ironic VMs when possible https://review.openstack.org/400941 | 19:20 |
*** jtomasek has quit IRC | 19:21 | |
rloo | penick: :-( | 19:21 |
openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 19:22 |
*** jtomasek has joined #openstack-ironic | 19:22 | |
*** Alagar has quit IRC | 19:22 | |
*** e0ne has joined #openstack-ironic | 19:26 | |
*** trown|lunch is now known as trown | 19:26 | |
*** awiddersheim has joined #openstack-ironic | 19:28 | |
openstackgerrit | Merged openstack/ironic-inspector: Stop disabling the Neutron tempest plugin https://review.openstack.org/400714 | 19:28 |
*** awidders_ has quit IRC | 19:31 | |
*** yolanda has quit IRC | 19:32 | |
*** moshele has joined #openstack-ironic | 19:34 | |
*** moshele has quit IRC | 19:41 | |
*** yolanda has joined #openstack-ironic | 19:42 | |
*** glonlas_ has joined #openstack-ironic | 19:43 | |
abalfour | bug filed; if anyone wants to chime in that the suggested workaround would be a valid fix, I can code it up and submit the patch... I just have no idea if that's the right thing to do. | 19:46 |
abalfour | https://bugs.launchpad.net/ironic/+bug/1643995 | 19:46 |
openstack | Launchpad bug 1643995 in Ironic "ironic can't find nodes whose name ends in ".<mimetype>"" [Undecided,New] | 19:46 |
*** glonlas has quit IRC | 19:47 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Replace six.iteritems() with .items() https://review.openstack.org/400506 | 19:47 |
*** yolanda has quit IRC | 19:54 | |
*** chlong has quit IRC | 19:55 | |
*** ppiela has quit IRC | 19:57 | |
*** glonlas_ has quit IRC | 19:59 | |
*** chlong has joined #openstack-ironic | 20:01 | |
*** betherly has quit IRC | 20:01 | |
*** baoli has quit IRC | 20:01 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 20:02 |
*** vsaienko has joined #openstack-ironic | 20:02 | |
*** glonlas has joined #openstack-ironic | 20:03 | |
*** vsaienko has quit IRC | 20:05 | |
*** sambetts is now known as sambetts|afk | 20:05 | |
*** baoli has joined #openstack-ironic | 20:05 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 20:05 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 20:06 |
*** Goneri has quit IRC | 20:13 | |
pas-ha | abalfour: O_O | 20:14 |
abalfour | is that good? or bad? :) | 20:15 |
jroll | I think that means "omg pecan y u do this" | 20:15 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 20:16 |
abalfour | heh. | 20:16 |
abalfour | it's being helpful! | 20:16 |
JayF | This thanksgiving | 20:17 |
JayF | we should take pecan, chop it up | 20:17 |
JayF | and turn it into pie. | 20:17 |
abalfour | <homer>mmm. pie.</homer> | 20:18 |
jroll | lol | 20:18 |
JayF | ^^ that specific fault support spec is ready for review at this point | 20:19 |
JayF | if anyone wants any, uh, light reading before the holiday | 20:19 |
*** jcoufal has quit IRC | 20:20 | |
*** jschwarz has quit IRC | 20:20 | |
TheJulia | JayF: So... perfect for awkward family get togethers on Thanksgiving? | 20:21 |
JayF | TheJulia: did you see the SNL Target thing from this weeks' episode? | 20:21 |
TheJulia | no, I don't watch SNL... really don't watch much TV actually | 20:22 |
JayF | TheJulia: https://www.youtube.com/watch?v=VnzRFyOMD4U hit the nail on the head re: family get togethers during thanksgiving.. especially this year | 20:23 |
*** [1]rpioso has quit IRC | 20:24 | |
*** jschwarz has joined #openstack-ironic | 20:26 | |
TheJulia | JayF: omg yes | 20:27 |
*** vinsh has joined #openstack-ironic | 20:30 | |
*** [1]rpioso has joined #openstack-ironic | 20:31 | |
*** glonlas has quit IRC | 20:45 | |
openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 20:47 |
*** penick has quit IRC | 20:50 | |
*** penick has joined #openstack-ironic | 20:51 | |
*** glonlas has joined #openstack-ironic | 20:59 | |
*** MattMan has quit IRC | 21:05 | |
*** dprince has quit IRC | 21:05 | |
*** MattMan has joined #openstack-ironic | 21:05 | |
openstackgerrit | Merged openstack/bifrost: Enable inspector discovery by default https://review.openstack.org/386701 | 21:05 |
*** Goneri has joined #openstack-ironic | 21:05 | |
*** baoli has quit IRC | 21:08 | |
-openstackstatus- NOTICE: Gerrit is offline until 21:30 UTC for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107379.html | 21:09 | |
*** ChanServ changes topic to "Gerrit is offline until 21:30 UTC for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107379.html" | 21:09 | |
jroll | welp | 21:09 |
jroll | sounds like time to call it a week | 21:09 |
JayF | jroll: o/ have a good holiday | 21:13 |
rloo | jroll: happy Thanksgiving! | 21:16 |
*** srobert has joined #openstack-ironic | 21:22 | |
*** jpich has quit IRC | 21:23 | |
*** shobhar_ has joined #openstack-ironic | 21:30 | |
TheJulia | jroll: have a wonderful Thanksgiving | 21:31 |
*** vinsh has quit IRC | 21:39 | |
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 21:40 | |
*** jkilpatr has quit IRC | 21:40 | |
*** e0ne has quit IRC | 21:45 | |
*** baoli has joined #openstack-ironic | 21:50 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Devstack will die if default driver is not enabled https://review.openstack.org/387657 | 21:51 |
*** jkilpatr has joined #openstack-ironic | 21:52 | |
*** vinsh has joined #openstack-ironic | 21:55 | |
*** yolanda has joined #openstack-ironic | 21:57 | |
*** trown is now known as trown|outtypewww | 22:02 | |
*** vinsh has quit IRC | 22:08 | |
*** vinsh has joined #openstack-ironic | 22:09 | |
*** vinsh has quit IRC | 22:13 | |
*** anush has quit IRC | 22:21 | |
*** soliosg has joined #openstack-ironic | 22:24 | |
*** ElCoyote_ has left #openstack-ironic | 22:30 | |
*** rama_y has quit IRC | 22:33 | |
*** srobert has quit IRC | 22:35 | |
mariojv | have a great holiday o/ | 22:39 |
JayF | o/ | 22:40 |
*** ElCoyote_ has joined #openstack-ironic | 22:42 | |
*** anush has joined #openstack-ironic | 22:44 | |
*** vinsh has joined #openstack-ironic | 22:47 | |
aNuposic | have a great thanksgiving :) | 22:48 |
*** chlong has quit IRC | 22:52 | |
*** yolanda has quit IRC | 22:58 | |
*** glonlas has quit IRC | 23:01 | |
*** glonlas has joined #openstack-ironic | 23:04 | |
*** glonlas has quit IRC | 23:08 | |
*** lindycoder has quit IRC | 23:15 | |
*** baoli has quit IRC | 23:18 | |
*** rbrud has quit IRC | 23:31 | |
*** [1]cdearborn has joined #openstack-ironic | 23:32 | |
*** watanabe_isao has joined #openstack-ironic | 23:38 | |
*** baoli has joined #openstack-ironic | 23:40 | |
*** baoli has quit IRC | 23:40 | |
*** lindycoder has joined #openstack-ironic | 23:40 | |
*** nicodemos has quit IRC | 23:42 | |
*** nicodemos has joined #openstack-ironic | 23:42 | |
*** shobhar_ has quit IRC | 23:42 | |
*** watanabe_isao has quit IRC | 23:45 | |
*** cdearborn has quit IRC | 23:46 | |
*** lindycoder has quit IRC | 23:52 | |
*** penick has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!