| *** jcoufal has joined #openstack-ironic | 00:00 | |
| *** d0ugal has joined #openstack-ironic | 00:04 | |
| *** jcoufal has quit IRC | 00:05 | |
| *** rcernin has quit IRC | 00:09 | |
| *** bfournie has quit IRC | 00:09 | |
| *** rcernin_ has joined #openstack-ironic | 00:09 | |
| *** ijw has joined #openstack-ironic | 00:13 | |
| *** ijw has quit IRC | 00:18 | |
| *** ijw has joined #openstack-ironic | 00:26 | |
| *** yangyapeng has joined #openstack-ironic | 00:27 | |
| *** ijw has quit IRC | 00:30 | |
| *** yangyapeng has quit IRC | 00:31 | |
| *** tuanla____ has joined #openstack-ironic | 00:38 | |
| *** d0ugal has quit IRC | 00:40 | |
| *** rcernin_ has quit IRC | 00:50 | |
| *** d0ugal has joined #openstack-ironic | 00:51 | |
| *** tuanla____ has quit IRC | 00:53 | |
| *** tuanla____ has joined #openstack-ironic | 00:54 | |
| *** catintheroof has joined #openstack-ironic | 00:56 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** dgautam has quit IRC | 01:05 | |
| *** yuanying has quit IRC | 01:14 | |
| *** yangyapeng has joined #openstack-ironic | 01:16 | |
| *** yangyapeng has quit IRC | 01:17 | |
| *** yangyapeng has joined #openstack-ironic | 01:17 | |
| *** openstackgerrit has joined #openstack-ironic | 01:18 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/528402 | 01:18 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/528863 | 01:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/528864 | 01:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements https://review.openstack.org/528404 | 01:18 |
| *** zhangfei has joined #openstack-ironic | 01:18 | |
| *** rloo has quit IRC | 01:20 | |
| *** catintheroof has quit IRC | 01:21 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:22 | |
| *** hshiina has joined #openstack-ironic | 01:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-baremetal master: Updated from global requirements https://review.openstack.org/528876 | 01:25 |
| *** ijw has joined #openstack-ironic | 01:26 | |
| *** vishwanathj has joined #openstack-ironic | 01:30 | |
| *** ijw has quit IRC | 01:32 | |
| *** linkmark has quit IRC | 01:33 | |
| *** hw_wutianwei has quit IRC | 01:37 | |
| *** bfournie has joined #openstack-ironic | 01:41 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:44 | |
| *** snasir has quit IRC | 01:49 | |
| *** snasir has joined #openstack-ironic | 01:49 | |
| *** dgautam has joined #openstack-ironic | 01:51 | |
| *** trungnv has joined #openstack-ironic | 01:52 | |
| *** ijw has joined #openstack-ironic | 01:54 | |
| *** ijw has quit IRC | 02:01 | |
| *** ijw has joined #openstack-ironic | 02:06 | |
| *** penick has quit IRC | 02:07 | |
| *** wangsansui has joined #openstack-ironic | 02:10 | |
| *** rcernin has joined #openstack-ironic | 02:11 | |
| *** wangsansui has quit IRC | 02:13 | |
| *** ijw has quit IRC | 02:15 | |
| *** r-daneel has quit IRC | 02:17 | |
| *** tuanla____ has quit IRC | 02:20 | |
| *** zhangfei has quit IRC | 02:21 | |
| *** tuanla____ has joined #openstack-ironic | 02:21 | |
| *** zhangfei has joined #openstack-ironic | 02:25 | |
| *** gcb has joined #openstack-ironic | 02:27 | |
| *** trungnv has quit IRC | 02:38 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 02:38 | |
| *** AlexeyAbashkin has quit IRC | 02:43 | |
| *** dgautam has quit IRC | 02:52 | |
| *** ijw has joined #openstack-ironic | 02:54 | |
| *** ijw has quit IRC | 02:59 | |
| openstackgerrit | Merged openstack/ironic master: Use NamedExtensionManager for drivers https://review.openstack.org/528277 | 03:18 |
| *** e0ne has joined #openstack-ironic | 03:42 | |
| *** e0ne has quit IRC | 03:47 | |
| *** dpawar has joined #openstack-ironic | 03:52 | |
| *** ijw has joined #openstack-ironic | 03:55 | |
| *** ijw has quit IRC | 03:59 | |
| *** e0ne has joined #openstack-ironic | 04:01 | |
| *** e0ne has quit IRC | 04:13 | |
| *** rbudden has quit IRC | 04:15 | |
| *** e0ne has joined #openstack-ironic | 04:16 | |
| *** fragatina has quit IRC | 04:20 | |
| *** fragatina has joined #openstack-ironic | 04:20 | |
| *** e0ne has quit IRC | 04:21 | |
| *** e0ne has joined #openstack-ironic | 04:26 | |
| *** MattMan has quit IRC | 04:28 | |
| *** MattMan has joined #openstack-ironic | 04:29 | |
| *** e0ne has quit IRC | 04:31 | |
| *** datex has quit IRC | 04:39 | |
| *** e0ne has joined #openstack-ironic | 04:41 | |
| *** e0ne has quit IRC | 04:46 | |
| *** phuongnh has quit IRC | 04:51 | |
| *** phuongnh has joined #openstack-ironic | 04:51 | |
| *** tuanla____ has quit IRC | 04:54 | |
| *** tuanla____ has joined #openstack-ironic | 04:55 | |
| *** e0ne has joined #openstack-ironic | 04:56 | |
| *** gcb has quit IRC | 04:56 | |
| *** gcb has joined #openstack-ironic | 04:58 | |
| *** e0ne has quit IRC | 05:01 | |
| *** e0ne_ has joined #openstack-ironic | 05:01 | |
| *** e0ne_ has quit IRC | 05:07 | |
| *** e0ne has joined #openstack-ironic | 05:10 | |
| *** e0ne has quit IRC | 05:15 | |
| *** e0ne has joined #openstack-ironic | 05:23 | |
| *** e0ne has quit IRC | 05:28 | |
| *** e0ne has joined #openstack-ironic | 05:32 | |
| *** penick has joined #openstack-ironic | 05:32 | |
| *** tuanla____ has quit IRC | 05:33 | |
| *** penick_ has joined #openstack-ironic | 05:36 | |
| *** penick has quit IRC | 05:36 | |
| *** e0ne has quit IRC | 05:37 | |
| *** e0ne has joined #openstack-ironic | 05:44 | |
| *** links has joined #openstack-ironic | 05:47 | |
| *** e0ne has quit IRC | 05:49 | |
| *** e0ne has joined #openstack-ironic | 05:54 | |
| *** tuanla____ has joined #openstack-ironic | 05:58 | |
| *** e0ne has quit IRC | 05:59 | |
| *** e0ne has joined #openstack-ironic | 06:01 | |
| *** afazekas has quit IRC | 06:11 | |
| *** afazekas has joined #openstack-ironic | 06:11 | |
| *** penick_ has quit IRC | 06:12 | |
| *** karthiks has joined #openstack-ironic | 06:13 | |
| *** e0ne has quit IRC | 06:13 | |
| *** namnh has joined #openstack-ironic | 06:13 | |
| *** e0ne has joined #openstack-ironic | 06:16 | |
| *** jaganathan has joined #openstack-ironic | 06:18 | |
| *** e0ne has quit IRC | 06:21 | |
| *** e0ne has joined #openstack-ironic | 06:28 | |
| *** e0ne has quit IRC | 06:33 | |
| *** e0ne has joined #openstack-ironic | 06:42 | |
| *** trungnv has joined #openstack-ironic | 06:43 | |
| *** yolanda has joined #openstack-ironic | 06:47 | |
| *** e0ne has quit IRC | 06:47 | |
| *** e0ne has joined #openstack-ironic | 06:53 | |
| *** marios has joined #openstack-ironic | 06:55 | |
| *** e0ne has quit IRC | 06:58 | |
| *** e0ne has joined #openstack-ironic | 06:59 | |
| *** mjura has joined #openstack-ironic | 06:59 | |
| *** e0ne has quit IRC | 07:04 | |
| *** dpawar has quit IRC | 07:06 | |
| *** e0ne has joined #openstack-ironic | 07:14 | |
| *** e0ne has quit IRC | 07:20 | |
| *** zshi has joined #openstack-ironic | 07:22 | |
| *** dpawar has joined #openstack-ironic | 07:23 | |
| *** e0ne has joined #openstack-ironic | 07:28 | |
| *** rcernin has quit IRC | 07:31 | |
| *** e0ne has quit IRC | 07:33 | |
| *** dpawar has quit IRC | 07:35 | |
| *** dpawar has joined #openstack-ironic | 07:35 | |
| *** e0ne has joined #openstack-ironic | 07:35 | |
| *** e0ne has quit IRC | 07:41 | |
| *** vsaienk0 has joined #openstack-ironic | 07:45 | |
| *** charcol has joined #openstack-ironic | 07:48 | |
| *** fxpester has joined #openstack-ironic | 07:49 | |
| *** e0ne has joined #openstack-ironic | 07:51 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:52 | |
| *** e0ne has quit IRC | 07:55 | |
| *** e0ne has joined #openstack-ironic | 07:58 | |
| *** rcernin has joined #openstack-ironic | 08:02 | |
| *** d0ugal has quit IRC | 08:03 | |
| *** v12aml has quit IRC | 08:03 | |
| *** e0ne has quit IRC | 08:03 | |
| *** d0ugal has joined #openstack-ironic | 08:07 | |
| *** e0ne has joined #openstack-ironic | 08:07 | |
| *** tuanla____ has quit IRC | 08:10 | |
| *** tuanla____ has joined #openstack-ironic | 08:11 | |
| *** e0ne has quit IRC | 08:16 | |
| *** amarao has joined #openstack-ironic | 08:26 | |
| *** v12aml has joined #openstack-ironic | 08:27 | |
| *** e0ne has joined #openstack-ironic | 08:28 | |
| *** yolanda__ has joined #openstack-ironic | 08:34 | |
| *** yolanda has quit IRC | 08:36 | |
| *** racedo has joined #openstack-ironic | 08:37 | |
| *** ccamacho has joined #openstack-ironic | 08:39 | |
| *** jtomasek has joined #openstack-ironic | 08:39 | |
| *** dougsz has joined #openstack-ironic | 08:42 | |
| *** priteau has joined #openstack-ironic | 08:50 | |
| *** trungnv has quit IRC | 08:52 | |
| *** amarao1 has joined #openstack-ironic | 08:53 | |
| *** hshiina has quit IRC | 08:54 | |
| *** karthiks has quit IRC | 08:56 | |
| *** e0ne has quit IRC | 09:00 | |
| *** karthiks has joined #openstack-ironic | 09:08 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface https://review.openstack.org/525501 | 09:24 |
| *** amarao1 has quit IRC | 09:27 | |
| *** jaianshu has joined #openstack-ironic | 09:27 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface https://review.openstack.org/525501 | 09:30 |
| pas-ha | morning ironic :) | 09:31 |
| *** lucas-afk is now known as lucasagomes | 09:32 | |
| *** milan_ has joined #openstack-ironic | 09:35 | |
| milan_ | morning Ironic! :) | 09:35 |
| *** derekh has joined #openstack-ironic | 09:36 | |
| milan_ | folks, I've tracked down my DB issue (introducing a new state in inspector) https://review.openstack.org/#/c/510928/8/ironic_inspector/migrations/versions/18440d0834af_introducing_the_aborting_state.py doesn't update a CHECK constraint on the "Enum" type in case of sqlite: http://paste.openstack.org/show/629246/ | 09:38 |
| patchbot | patch 510928 - ironic-inspector - Terminal state transitions in transactions | 09:38 |
| *** dougsz has quit IRC | 09:38 | |
| * milan_ unable to StackOverflow this :-/ | 09:38 | |
| milan_ | any hints? | 09:38 |
| milan_ | * the constraint misses the new state 'aborting' | 09:39 |
| *** dougsz has joined #openstack-ironic | 09:42 | |
| *** yangyapeng has quit IRC | 09:45 | |
| *** mgoddard has joined #openstack-ironic | 09:54 | |
| *** dyasny__ has joined #openstack-ironic | 09:58 | |
| *** dougsz has quit IRC | 09:58 | |
| *** dyasny_ has quit IRC | 10:00 | |
| *** afazekas has quit IRC | 10:01 | |
| *** namnh has quit IRC | 10:01 | |
| *** dougsz has joined #openstack-ironic | 10:04 | |
| *** milan_ has quit IRC | 10:04 | |
| *** milan_ has joined #openstack-ironic | 10:05 | |
| *** afazekas has joined #openstack-ironic | 10:07 | |
| *** karthiks has quit IRC | 10:20 | |
| *** fragatin_ has joined #openstack-ironic | 10:22 | |
| *** fragatina has quit IRC | 10:22 | |
| *** namnh has joined #openstack-ironic | 10:25 | |
| *** namnh has quit IRC | 10:25 | |
| *** milan_ has quit IRC | 10:26 | |
| *** e0ne has joined #openstack-ironic | 10:31 | |
| *** jaganathan has quit IRC | 10:36 | |
| *** phuongnh has quit IRC | 10:37 | |
| *** jtomasek has quit IRC | 10:38 | |
| *** zhangfei has quit IRC | 10:38 | |
| *** sambetts|afk is now known as sambetts | 10:39 | |
| sambetts | Morning all | 10:39 |
| *** jtomasek has joined #openstack-ironic | 10:39 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds rescue_interface to base driver class https://review.openstack.org/509335 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface https://review.openstack.org/509336 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add API methods for [un]rescue https://review.openstack.org/350831 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue interface field to node-related notifications https://review.openstack.org/502889 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update description for config params of 'rescue' interface https://review.openstack.org/517919 | 10:41 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add documentation for rescue interface https://review.openstack.org/419606 | 10:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters https://review.openstack.org/478825 | 10:42 |
| *** early has quit IRC | 10:42 | |
| *** early has joined #openstack-ironic | 10:45 | |
| *** karthiks has joined #openstack-ironic | 10:46 | |
| *** yangyapeng has joined #openstack-ironic | 10:47 | |
| *** milan_ has joined #openstack-ironic | 10:47 | |
| *** milan_ has quit IRC | 10:48 | |
| *** milan_ has joined #openstack-ironic | 10:50 | |
| *** yangyapeng has quit IRC | 10:53 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface https://review.openstack.org/525501 | 10:56 |
| *** fragatina has joined #openstack-ironic | 10:57 | |
| *** fragatin_ has quit IRC | 10:57 | |
| *** yangyapeng has joined #openstack-ironic | 11:03 | |
| *** yangyapeng has quit IRC | 11:08 | |
| *** jaganathan has joined #openstack-ironic | 11:11 | |
| *** AlexeyAbashkin has quit IRC | 11:22 | |
| *** yangyapeng has joined #openstack-ironic | 11:24 | |
| *** jtomasek has quit IRC | 11:29 | |
| *** yangyapeng has quit IRC | 11:29 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:38 | |
| dtantsur | morning ironic! /me is back for 4 more days | 11:39 |
| *** e0ne has quit IRC | 11:41 | |
| *** yangyapeng has joined #openstack-ironic | 11:45 | |
| sambetts | hey dtantsur | 11:45 |
| dtantsur | ramineni: so, re dpkg capabilities | 11:45 |
| dtantsur | I remember some tripleo folks doing it | 11:46 |
| dtantsur | we discover CPU flags in ironic-inspector; there is also an optional NUMA collector in IPA | 11:46 |
| ramineni | dtantsur: ya , tripleO is newton spec actually , so hoping ipa functionality exists | 11:46 |
| dtantsur | I'm not sure it's enough for you or not, as I don't know much about DPDK | 11:46 |
| ramineni | dtantsur: in this , its mentioned https://specs.openstack.org/openstack/tripleo-specs/specs/newton/tripleo-ovs-dpdk.html | 11:47 |
| ramineni | CPU flags for hugepages support - If pse exists then 2MB hugepages are supported If pdpe1gb exists then 1GB hugepages are supportedCPU flags for IOMMU - If VT-d/svm exists, then IOMMU is supported, provided IOMMU support is enabled in BIOS.Compatible nics - Shall compare it with the list of NICs whitelisted for DPDK. The DPDK supported NICs are available at http://dpdk.org/doc/nics | 11:48 |
| ramineni | the third part is missing ? Compatible nics - Shall compare it with the list of NICs whitelisted for DPDK. The DPDK supported NICs are available at http://dpdk.org/doc/nics | 11:48 |
| ramineni | dtantsur: there is no comparison as such happening in inspector like above right? | 11:49 |
| *** vsaienk0 has quit IRC | 11:49 | |
| dtantsur | yep, I don't think there exists such plugin | 11:49 |
| dtantsur | maybe something outside of ironic-inspector tree? | 11:49 |
| *** yangyapeng has quit IRC | 11:49 | |
| *** yolanda__ is now known as yolanda | 11:50 | |
| ramineni | dtantsur: ya, im also not sure , how tripleO works, if they implement out of tree | 11:50 |
| ramineni | dtantsur: same I see for SR-IOV, https://specs.openstack.org/openstack/tripleo-specs/specs/newton/tripleo-sriov.html | 11:50 |
| sambetts | ramineni, dtantsur: our PCI plugin has a list of devices in the config file,wouldn't that work | 11:50 |
| sambetts | ? | 11:50 |
| *** e0ne has joined #openstack-ironic | 11:51 | |
| ramineni | sambetts: no, for sriov, dpdk , atleast according to spec, there needs to be additional check, to identify supported nics | 11:51 |
| ramineni | sambetts: dtantsur: like in proposed change for sr-iov , its mentioned like SR-IOV capable NICs: Shall read /sys/bus/pci/devices/.../sriov_totalvfs and check if its non zero, inorder to identify if the NIC is SR-IOV capable | 11:52 |
| ramineni | sambetts: dtantsur: but I don't see the code for the same in IPA | 11:52 |
| dtantsur | yeah, I don't remember this implemented | 11:52 |
| sambetts | I remeber patches about it though | 11:52 |
| sambetts | there was contention about where to store the data if I remember correctly | 11:52 |
| dtantsur | karthiks: you're mentioned as a co-author on https://specs.openstack.org/openstack/tripleo-specs/specs/newton/tripleo-sriov.html ^^^ | 11:52 |
| ramineni | dtantsur: and also, redfish driver , it also uses inband inspection right? no out of band inspection supported as of now? | 11:59 |
| dtantsur | ramineni: yep. it's in progress AFAIK, but seems stuck | 11:59 |
| ramineni | dtantsur: ok, thanks | 12:00 |
| *** yangyapeng has joined #openstack-ironic | 12:05 | |
| dtantsur | np | 12:06 |
| dtantsur | pas-ha: hi! where are we with a CI job covering the ansible driver? | 12:06 |
| *** vsaienk0 has joined #openstack-ironic | 12:10 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 12:11 |
| *** yangyapeng has quit IRC | 12:11 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add rescue_interface to node and driver https://review.openstack.org/517302 | 12:13 |
| *** milan_ has quit IRC | 12:19 | |
| *** vsaienk0 has quit IRC | 12:20 | |
| *** milan_ has joined #openstack-ironic | 12:20 | |
| *** e0ne has quit IRC | 12:22 | |
| *** bfournie has quit IRC | 12:22 | |
| *** bfournie has joined #openstack-ironic | 12:22 | |
| *** links has quit IRC | 12:23 | |
| *** dpawar has quit IRC | 12:24 | |
| *** bfournie has quit IRC | 12:27 | |
| *** vsaienk0 has joined #openstack-ironic | 12:29 | |
| *** yangyapeng has joined #openstack-ironic | 12:29 | |
| *** dpawar has joined #openstack-ironic | 12:31 | |
| *** yangyapeng has quit IRC | 12:34 | |
| karthiks | ramineni, dtantsur the totalvfs validation part is not implemented in IPA | 12:36 |
| *** tuanla____ has quit IRC | 12:39 | |
| *** lucasagomes is now known as lucas-hungry | 12:42 | |
| *** hw_wutianwei has quit IRC | 12:43 | |
| *** weshay_pto is now known as weshay | 12:48 | |
| *** yangyapeng has joined #openstack-ironic | 12:49 | |
| *** milan_ has quit IRC | 12:49 | |
| *** aarefiev has joined #openstack-ironic | 12:50 | |
| *** milan_ has joined #openstack-ironic | 12:51 | |
| *** yangyapeng has quit IRC | 12:53 | |
| *** zhurong has joined #openstack-ironic | 13:00 | |
| *** zhurong has quit IRC | 13:02 | |
| *** zhurong has joined #openstack-ironic | 13:03 | |
| *** bfournie has joined #openstack-ironic | 13:04 | |
| *** jcoufal has joined #openstack-ironic | 13:06 | |
| *** yangyapeng has joined #openstack-ironic | 13:10 | |
| pas-ha | dtantsur: re ansible deploy CI - in progress, will push some patches this week | 13:10 |
| dtantsur | pas-ha: great! I'd like a release soon(ish), but the new deploy method has to be CI tested | 13:11 |
| *** catintheroof has joined #openstack-ironic | 13:11 | |
| *** catintheroof has quit IRC | 13:12 | |
| *** catintheroof has joined #openstack-ironic | 13:12 | |
| *** d0ugal has quit IRC | 13:14 | |
| *** yangyapeng has quit IRC | 13:15 | |
| *** links has joined #openstack-ironic | 13:18 | |
| *** r-daneel has joined #openstack-ironic | 13:19 | |
| *** rcernin has quit IRC | 13:20 | |
| *** r-daneel has quit IRC | 13:20 | |
| *** d0ugal has joined #openstack-ironic | 13:23 | |
| *** zhurong has quit IRC | 13:25 | |
| *** dprince has joined #openstack-ironic | 13:25 | |
| *** dpawar has quit IRC | 13:25 | |
| *** jaianshu has quit IRC | 13:28 | |
| *** yangyapeng has joined #openstack-ironic | 13:30 | |
| *** r-daneel has joined #openstack-ironic | 13:34 | |
| *** lucas-hungry is now known as lucasagomes | 13:34 | |
| *** yangyapeng has quit IRC | 13:35 | |
| *** trown|outtypewww is now known as trown|ruck | 13:36 | |
| *** d0ugal has quit IRC | 13:37 | |
| *** yangyapeng has joined #openstack-ironic | 13:39 | |
| *** d0ugal has joined #openstack-ironic | 13:39 | |
| *** sasha2 has joined #openstack-ironic | 13:41 | |
| *** jaganathan has quit IRC | 13:42 | |
| *** yangyapeng has quit IRC | 13:44 | |
| *** mkovacik__ has joined #openstack-ironic | 13:52 | |
| *** makowals has joined #openstack-ironic | 13:53 | |
| *** milan_ has quit IRC | 13:55 | |
| *** dyasny__ has quit IRC | 13:56 | |
| *** dyasny has joined #openstack-ironic | 14:00 | |
| *** links has quit IRC | 14:03 | |
| *** yangyapeng has joined #openstack-ironic | 14:06 | |
| *** rloo has joined #openstack-ironic | 14:11 | |
| *** yangyapeng has quit IRC | 14:12 | |
| TheJulia | Good Morning everyone | 14:15 |
| mkovacik__ | morning TheJulia! :) | 14:16 |
| *** mkovacik__ is now known as milan_ | 14:16 | |
| *** milan_ has quit IRC | 14:16 | |
| *** milan_ has joined #openstack-ironic | 14:17 | |
| * milan_ has some connectivity weirdness going on | 14:17 | |
| *** links has joined #openstack-ironic | 14:17 | |
| rloo | good morning TheJulia, milan_ | 14:18 |
| milan_ | morning rloo! :) | 14:18 |
| milan_ | I've finally figured out what's wrong with https://review.openstack.org/#/c/510928/8/ironic_inspector/migrations/versions/18440d0834af_introducing_the_aborting_state.py | 14:18 |
| patchbot | patch 510928 - ironic-inspector - Terminal state transitions in transactions | 14:18 |
| * milan_ 's been trying to faceroll a solution | 14:19 | |
| milan_ | so far no luck | 14:19 |
| dtantsur | morning TheJulia, rloo, milan_ | 14:21 |
| rloo | hi dtantsur | 14:21 |
| milan_ | morning dtantsur! :) | 14:21 |
| dtantsur | pas-ha, TheJulia, wdyt about unmounting /sys and /proc with -l (lazy) in the ansible deploy? | 14:22 |
| milan_ | anyone knows how to add new value in an Enum so that sqlite DB update works? | 14:22 |
| TheJulia | dtantsur: when did that option get mounted to mount? | 14:22 |
| * milan_ no luck StackOverflow | 14:23 | |
| TheJulia | kernel 2.4.11 | 14:23 |
| dtantsur | TheJulia: no idea :) kernel 2.6.11 IIRC | 14:23 |
| dtantsur | ah, even 2.4 | 14:23 |
| *** hw_wutianwei has joined #openstack-ironic | 14:23 | |
| TheJulia | dtantsur: that should actually work I think | 14:23 |
| *** jtomasek has joined #openstack-ironic | 14:24 | |
| dtantsur | I'll try, I guess. I still see failures with "device is busy" otherwise.. | 14:24 |
| *** yangyapeng has joined #openstack-ironic | 14:24 | |
| dtantsur | another thing to try is using --rbind instead of --bind | 14:26 |
| *** yangyapeng has quit IRC | 14:29 | |
| rloo | dtantsur: are you ok with updating the Queens priorities? I don't think we've ever done so afterwards: https://review.openstack.org/#/c/528386/ | 14:32 |
| patchbot | patch 528386 - ironic-specs - Move traits from "high" to "essential" priority | 14:32 |
| dtantsur | yep (approved) | 14:33 |
| rloo | dtantsur: :) | 14:33 |
| TheJulia | rloo: replied to your bios spec comments, well, a couple of them at least :) | 14:36 |
| rloo | TheJulia: thx. How do you think we can best move forward on it? I noticed that the author didn't reply. Am wondering if we need to have a meeting to discuss/decide. | 14:38 |
| * TheJulia goes looking for the author | 14:38 | |
| openstackgerrit | Merged openstack/ironic-specs master: Move traits from "high" to "essential" priority https://review.openstack.org/528386 | 14:39 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: dd docs for ansible deploy interface https://review.openstack.org/525501 | 14:40 |
| TheJulia | zshi: o/ | 14:40 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add docs for ansible deploy interface https://review.openstack.org/525501 | 14:41 |
| TheJulia | yolanda: you around? | 14:41 |
| *** hjensas has quit IRC | 14:42 | |
| *** rbudden has joined #openstack-ironic | 14:49 | |
| rloo | dtantsur: your opinion. jlvillal has a patch that copies fixture code from nova. cinder & oslo.versionedobjects also have copies. once-upon-a-time we wouldn't allow it in our tree, but insist that it gets added to some oslo library. times have changed. what do you think, ok, to add in our tree? https://review.openstack.org/#/c/232257/ | 14:50 |
| patchbot | patch 232257 - ironic - Detect when log messages are mis-formatted | 14:50 |
| rloo | dtantsur: one issue is that jlvillal doesn't have time to pursue the oslo route. so better to have something, than nothing? | 14:51 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/499050 | 14:52 |
| *** mkovacik__ has joined #openstack-ironic | 14:52 | |
| dtantsur | rloo, jlvillal, sigh. I was going to raise it as well.. can we at least put it in one place only? | 14:52 |
| *** andyjone_ has joined #openstack-ironic | 14:55 | |
| yolanda | hi TheJulia | 14:55 |
| *** milan_ has quit IRC | 14:55 | |
| yolanda | glad to see you here :) | 14:55 |
| TheJulia | :) | 14:55 |
| *** milan_ has joined #openstack-ironic | 14:56 | |
| TheJulia | yolanda: I had some comments on https://review.openstack.org/#/c/496481/ and wondering if you could provide feedback or insight. We want to shorten the comment feedback loop as much as possible because we want traction on that spec | 14:56 |
| patchbot | patch 496481 - ironic-specs - Hardware interface for BIOS configuration | 14:56 |
| *** mkovacik__ has quit IRC | 14:57 | |
| yolanda | sure, i can take a look | 14:57 |
| TheJulia | thanks | 14:57 |
| *** yangyapeng has joined #openstack-ironic | 15:01 | |
| *** vsaienk0 has quit IRC | 15:02 | |
| *** andyjone_ has left #openstack-ironic | 15:02 | |
| *** andyjone_ has joined #openstack-ironic | 15:06 | |
| *** vsaienk0 has joined #openstack-ironic | 15:08 | |
| *** andyjone_ is now known as andyjones2001_ | 15:09 | |
| pas-ha | dtantsur: re lazy umount - IMO we should not do this. apparently there is also a bug(?) in ansible deploy around this place re mounting /sys - at least it is different from what's IPA is doing | 15:09 |
| dtantsur | yeah, trying to debug now | 15:10 |
| pas-ha | as in first mount /dev and /proc with -o bind, than mount sys w/o bind and with -t sysfs | 15:10 |
| pas-ha | umount in the same order | 15:10 |
| pas-ha | that's ^ the way IPA does it | 15:11 |
| dtantsur | mmm, yeah, lemme try this too | 15:11 |
| *** andyjones2001 has quit IRC | 15:15 | |
| *** dgautam has joined #openstack-ironic | 15:15 | |
| *** andyjones2001_ is now known as andyjones2001 | 15:16 | |
| pas-ha | and IPA also has retries on sys unmount, 3 times | 15:16 |
| dtantsur | okie, lemme make a patch | 15:17 |
| *** baha has joined #openstack-ironic | 15:19 | |
| *** aarefiev has quit IRC | 15:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: ansible: handle mount of /sys the same way IPA does it https://review.openstack.org/526402 | 15:20 |
| dtantsur | pas-ha: will try ^^^ | 15:20 |
| *** livelace has joined #openstack-ironic | 15:23 | |
| *** hjensas has joined #openstack-ironic | 15:23 | |
| *** karthiks has quit IRC | 15:25 | |
| pas-ha | I think you have a typo there, left a comment | 15:26 |
| pas-ha | s/sysfs/\/sys/ | 15:26 |
| pas-ha | dtantsur: ^ | 15:27 |
| dtantsur | pas-ha: I think it's correct. it's source, not destination | 15:28 |
| dtantsur | and source is not quite real here, as it's not a bind mount | 15:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: ansible: handle mount of /sys the same way IPA does it https://review.openstack.org/526402 | 15:28 |
| *** dgautam has quit IRC | 15:35 | |
| *** fxpester has quit IRC | 15:35 | |
| *** dgautam has joined #openstack-ironic | 15:38 | |
| *** dgautam has quit IRC | 15:43 | |
| *** hjensas has quit IRC | 15:43 | |
| *** dgautam has joined #openstack-ironic | 15:46 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: ansible: handle mount of /sys the same way IPA does it https://review.openstack.org/526402 | 15:46 |
| *** jtomasek has quit IRC | 15:50 | |
| *** amarao1 has joined #openstack-ironic | 15:54 | |
| *** vsaienk0 has quit IRC | 15:57 | |
| *** hjensas has joined #openstack-ironic | 16:02 | |
| *** vsaienk0 has joined #openstack-ironic | 16:02 | |
| *** mjura has quit IRC | 16:03 | |
| *** amarao has quit IRC | 16:05 | |
| *** amarao1 has quit IRC | 16:05 | |
| *** livelace has quit IRC | 16:06 | |
| *** dgautam has quit IRC | 16:08 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Use fake network interface in tempest VIF attach https://review.openstack.org/529099 | 16:09 |
| *** amarao has joined #openstack-ironic | 16:11 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/528402 | 16:12 |
| jroll | morning folks | 16:13 |
| dtantsur | morning jroll | 16:13 |
| *** hw_wutianwei has quit IRC | 16:14 | |
| dtantsur | pas-ha, TheJulia, with https://review.openstack.org/#/c/526402/ I no longer see 'no valid host found' on my current 4-nodes deployment | 16:15 |
| patchbot | patch 526402 - ironic - ansible: handle mount of /sys the same way IPA doe... | 16:15 |
| TheJulia | dtantsur: no valid host as in it was attempting to redeploy on other nodes upon failure? | 16:16 |
| *** charcol has quit IRC | 16:16 | |
| *** hjensas has quit IRC | 16:16 | |
| dtantsur | TheJulia: yep | 16:16 |
| dtantsur | and heat was retrying as well, but sometimes it still was not enough | 16:16 |
| *** amarao has quit IRC | 16:19 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: roles: bifrost-create-vm-nodes: Set CPU model to 'host-model' https://review.openstack.org/528677 | 16:26 |
| TheJulia | dtantsur: how many times was it retrying to deploy a node with your patch? | 16:27 |
| dtantsur | TheJulia: 1, apparently :) I don't see any 'deploy fail' in the logs after applying it | 16:28 |
| *** hjensas has joined #openstack-ironic | 16:28 | |
| *** r-daneel has quit IRC | 16:38 | |
| jlvillal | Good morning Ironic | 16:39 |
| jlvillal | rloo, That log thing is more difficult to figure out :( It is sort of a strange thing. I'm not sure why it is happening. I will have to investigate it more. | 16:40 |
| rloo | jlvillal: interesting. to be honest, i don't understand the code, which is why i figured I needed to understand/test before giving it the green light :) | 16:41 |
| dtantsur | morning jlvillal | 16:41 |
| jlvillal | rajinir, Regarding Dell CI failing with tempest removal. Let me know if you have any questions on how to switch. For the ironic projects it was about 4 lines of code changes. | 16:42 |
| rloo | jlvillal: maybe it is meant to do that and the documentation is incorrect? I'm guessing it does a simliar thing for nova? | 16:42 |
| rloo | jlvillal: and yes, GOOD morning :) | 16:42 |
| *** penick has joined #openstack-ironic | 16:42 | |
| jlvillal | rloo, No, it shouldn't be doing that. There is a unit test which proves that it captures the DEBUG message. But somehow some of the lower level ones are escaping. Odd. | 16:42 |
| rloo | jlvillal: also, don't know if you saw, I asked dtantsur what he thought of that log patch ^^ "can we at least put it in one place only" heh. | 16:43 |
| dtantsur | like, ironic-lib? | 16:43 |
| jlvillal | rloo, Okay. | 16:44 |
| *** vsaienk0 has quit IRC | 16:44 | |
| jlvillal | dtantsur, Okay, that would be easier than trying to get it into oslo. IMHO | 16:44 |
| jlvillal | I'll investigate it. | 16:44 |
| jlvillal | The strangeness in some of the log messages escaping out is blocking me at the moment. I'm worried it will take hours to figure out the issue :( | 16:45 |
| jlvillal | Or days... | 16:45 |
| jlvillal | So may not have time for awhile to get to the root cause. | 16:45 |
| *** marios has quit IRC | 16:47 | |
| *** vsaienk0 has joined #openstack-ironic | 16:49 | |
| *** andyjones2001 has quit IRC | 16:53 | |
| jlvillal | rloo, dtantsur , I guess we can go with 3 weeks for the CI change. Seems a bit long for what should be 4-line change. But I can live with it. | 16:54 |
| *** mkovacik__ has joined #openstack-ironic | 16:54 | |
| rloo | jlvillal: 3 weeks -ish could be Monday Jan 8. | 16:54 |
| *** milan_ has quit IRC | 16:55 | |
| rloo | jlvillal: also, we only have a handful of 3rd party CI, if you can catch them, ask them to switch over soon, then you can do the tempest deletion sooner. | 16:55 |
| jlvillal | rloo, True. | 16:56 |
| *** lucasagomes is now known as lucas-afk | 16:56 | |
| jlvillal | rloo, I sent out an email with a pointer to how we did it and it should be fairly simple to change. | 16:56 |
| rloo | jlvillal: yup, 'should be'. ha ha! | 16:57 |
| jlvillal | rloo, Well based on our stuff it was pretty simple. Changing the jobs to use the new location. | 16:57 |
| jlvillal | rloo, But I have no idea how they are doing their setup | 16:57 |
| rloo | jlvillal: yup, i know, i reviewed them :) | 16:57 |
| jlvillal | sambetts, Was it difficult to switch your CI over? Not sure if your CI is back up and running or not. I remember the "catastrophic failure" from a month or two ago. | 16:58 |
| sambetts | jlvillal: to Zuulv3? | 16:58 |
| jlvillal | sambetts, To the ironic-tempest-plugin | 16:58 |
| sambetts | jlvillal: oh right, I havn't actually done it yet, but it should just be a one line change in my local.conf | 16:59 |
| jlvillal | sambetts, If do get it done. Might be nice if you could reply to the email thread on what you did to change it. To help out the other CI people. | 17:00 |
| *** AlexeyAbashkin has quit IRC | 17:01 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Document the check done in "ironic-dbsync upgrade" https://review.openstack.org/529117 | 17:02 |
| *** Goneri has joined #openstack-ironic | 17:03 | |
| *** r-daneel has joined #openstack-ironic | 17:07 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Terminal state transitions in transactions https://review.openstack.org/510928 | 17:09 |
| *** ccamacho has quit IRC | 17:11 | |
| sambetts | jlvillal: hmmm just thought, the default for most third party CI's is "DEVSTACK_GATE_TEMPEST_ALL_PLUGINS=1", which will load both the plugin from /opt/stack/ironic and /opt/stack/ironic-tempest-plugin, won't they conflict? | 17:13 |
| sambetts | I thought the saving grace might be a namespace change, but we've kept it the same | 17:14 |
| jlvillal | sambetts, It doesn't appear to be conflicting | 17:14 |
| sambetts | are you using DEVSTACK_GATE_TEMPEST_ALL_PLUGINS=1 or TEMPEST_PLGUINS=... which pick specific plugins to install ? | 17:15 |
| sambetts | in the email I see TEMPEST_PLGUINS=... | 17:15 |
| rajinir | jlvillal: yes, we have made the change already. export DEVSTACK_LOCAL_CONFIG+=$'\n'"TEMPEST_PLUGINS=/opt/stack/new/ironic-tempest-plugin", we have 7 jobs , unfortunately our labs are down. So not able to do builds now | 17:15 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Terminal state transitions in transactions https://review.openstack.org/510928 | 17:15 |
| jlvillal | sambetts, yes | 17:16 |
| jlvillal | sambetts, We appear to have both | 17:16 |
| rajinir | jlvillal: We are still on ZuulV2 , so this change was also required in the job template export PROJECTS="openstack/ironic-tempest-plugin $PROJECTS" | 17:16 |
| *** vsaienk0 has quit IRC | 17:17 | |
| jlvillal | rajinir, So Dell 3rd Party CI is down? When will it be back up? | 17:17 |
| jlvillal | rajinir, Yes. I forgot to mention that. But the patch does have that. My mistake. | 17:17 |
| rajinir | jlvillal: We have some outage in the labs, no ETA yet, but a few hours it will be back up | 17:17 |
| jlvillal | rajinir, Ah okay. I thought it might be down for the holidays. | 17:18 |
| * jlvillal remembers our team in one country who would go on country holiday for a week and turn off power to lab and we needed the lab to be on... | 17:18 | |
| rajinir | jlvillal: Usually the power shuts down for a week during holidays with no exceptions, this year, we have requested for an exception to keep the CI systems up | 17:18 |
| jlvillal | rajinir, Heh, reminds me of previous situation at my company :) | 17:19 |
| *** vsaienk0 has joined #openstack-ironic | 17:19 | |
| *** trown|ruck is now known as trown|lunch | 17:22 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Prevent changes to the ironic_tempest_plugin/ directory https://review.openstack.org/529119 | 17:26 |
| jlvillal | rloo, dtantsur ^^^ Not sure what you think about that. | 17:26 |
| jlvillal | But if we are going to wait three weeks to change over I thought a check might be a good idea. | 17:26 |
| sambetts | jlvillal: is there changes that exist on the openstack/ironic-tempest-plugin but exist on openstack/ironic tempest plugin yet?? | 17:27 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: DNM/WIP: Test that PEP8 fails https://review.openstack.org/529120 | 17:27 |
| jlvillal | sambetts, I don't think any changes have been made yet since we switched over | 17:28 |
| sambetts | jlvillal: only asking because i was looking for a way to check which plugin took over: http://paste.openstack.org/show/629344/ | 17:28 |
| sambetts | jlvillal: there is no way to know where the ironic_tempest_plugin has come from if you install both ironic and ironic_tempest_plugin in the same evnironment | 17:29 |
| jlvillal | sambetts, Seems like you are getting the correct one | 17:29 |
| rloo | jlvillal: that patch looks fine to me | 17:30 |
| jlvillal | sambetts, Oh never mind. I was reading the pastebin wrong. Can't see | 17:30 |
| sambetts | if you instal ironic first, then its install and then you install the tempest plugin second, the file location is exact the same | 17:30 |
| jlvillal | sambetts, Which file location | 17:30 |
| sambetts | jlvillal: L19 and L37 | 17:30 |
| sambetts | L19 is when just ironic is installed | 17:31 |
| sambetts | and L37 is when ironic and the ironic_tempest_plugin are installed | 17:31 |
| *** vsaienk0 has quit IRC | 17:31 | |
| jlvillal | sambetts, I see that pulling from the ironic_tempest_plugin repository | 17:31 |
| jlvillal | sambetts, They look identical. Am I missing something? | 17:31 |
| sambetts | ironic_tempest_plugin isn't the repository, its the python namespace | 17:32 |
| jlvillal | Not coming from the 'ironic' repository | 17:32 |
| sambetts | aka python module name | 17:32 |
| sambetts | python doesn't install things based on the reposistory | 17:32 |
| jlvillal | But I'm not sure how "tempest' finds them though. | 17:32 |
| jlvillal | sambetts, but it is "site-packages" | 17:32 |
| jlvillal | sambetts, ironic_tempest_plugin is from 'site-packages" | 17:32 |
| jlvillal | sambetts, So that is the ironic_tempest_plugin repository | 17:33 |
| sambetts | no | 17:33 |
| sambetts | its the python module | 17:33 |
| sambetts | which is separate from both the repo and the pip package | 17:33 |
| sambetts | the ironic repo installs two python modules/namespaces, "ironic" and "ironic_tempest_plugin" | 17:34 |
| jlvillal | sambetts, Ah. | 17:34 |
| jroll | sambetts: can probably introspect the entrypoints somehow | 17:34 |
| jlvillal | sambetts, You are making me think we should move faster on the changeover to remove the plugin code.. | 17:34 |
| jlvillal | rloo, dtantsur ^^^ | 17:35 |
| sambetts | jroll, jlvillal: the entrypoints are both named the same, and refer to the python module name ironic_tempest_plugin.plugin:IronicTempestPlugin, so you could have the entrypoint from ironic loading code from ironic-tempest-plugin and visavera | 17:36 |
| jroll | :| | 17:37 |
| rloo | sambetts: is there a consistent order wrt which one is used? | 17:37 |
| jlvillal | sambetts, Okay. So sounds like we can't be sure which one gets loaded until we remove the old plugin code. Is that correct? | 17:37 |
| sambetts | tempest loads the plugins by loading all tempest.test_plugins that happen to be installed in the current environemtn | 17:37 |
| sambetts | jlvillal: correct, unless you run an environment where the ironic-tempest-plugin is installed by ironic isn't | 17:37 |
| rloo | jlvillal: i am fine delaying still. because 3rd party CI can delete so they know they are using the new repo | 17:38 |
| rloo | jlvillal: 3rd party CI that haven't done the migration thing, will get one of the two right? which is better than getting none if we delete tempest-from-ironic-master | 17:38 |
| jlvillal | rloo, The only issue is if we start adding code to openstack/ironic-tempest-plugin/ it may or may not get run. | 17:38 |
| rloo | jlvillal: yes, that we know about, they may be using the old stuff in master | 17:39 |
| jroll | sambetts: so iirc typically the gate keeps tempest (and the plugins) in a virtualenv - not sure if third party CIs typically do that though | 17:39 |
| jlvillal | rloo, We also may be using old stuff. Unless we delete it. Is what it appears to be. | 17:39 |
| rloo | jlvillal: which is better than breaking it all together if they don't have their thing set up to use the new tempest plugin repo | 17:39 |
| rloo | jlvillal: oh. that is different thing. | 17:39 |
| sambetts | jroll: the ironic CI and the third party CIs run with DEVSTACK_GATE_TEMPEST_ALL_PLUGIN=1 which pulls in all sitepackages into the virtualenv | 17:39 |
| rloo | jlvillal: just make a change to the new repo and see which gets used. unless it is random. | 17:40 |
| jlvillal | rloo, So if we add tests to openstack/ironic-tempest-plugin it may or may not run those tests. Is what I think is happening. | 17:40 |
| jroll | sambetts: blah, I thought we killed that off a while ago | 17:40 |
| sambetts | I think if we disable DEVSTACK_GATE_TEMPEST_ALL_PLUGIN and then specify all the tempest plugins using TEMPEST_PLUGINS we might be ok, but it then relys on devstack logic to inject the plugins into tempests virtual env, instead of standard tempest behaviour for loading the plugins | 17:41 |
| jlvillal | mtreinish, Do you have any ideas on the above? | 17:41 |
| jroll | sambetts: eh, it still relies on the normal tempest behavior, we just *also* need devstack to get it into the virtualenv (which shouldn't be a problem) | 17:42 |
| sambetts | jroll: thats fine as long as you only plan to use tempest against devstacks | 17:42 |
| dtantsur | yeah, outside of the gate it's not cool, I guess.. | 17:43 |
| jroll | sambetts: I mean it's just for this transition period | 17:43 |
| *** crushil__ has joined #openstack-ironic | 17:43 | |
| sambetts | or if anyone installs an ironic stable branch | 17:43 |
| sambetts | because we're not deleting it from there | 17:43 |
| jlvillal | My only issue is if we have to make that change over 30+ patches and then change it back... | 17:44 |
| jroll | blah | 17:45 |
| sambetts | conflciting with a stable branch version of ironic_tempest_plugin is going to suck too... that might already be happening in our stable gates | 17:46 |
| jroll | yeah I'm not sure what the best route is, then, I'm not up to speed on this whole project | 17:46 |
| jlvillal | sambetts, We had previously set it up to always use master for the plugin. So we shouldn't be pulling from stable. At least in the ironic gate. | 17:46 |
| *** derekh has quit IRC | 17:47 | |
| jlvillal | And by "we" I mean "jroll" :) | 17:47 |
| jroll | yeah, that's why I'm confused about this | 17:47 |
| jroll | I really think we killed tempest_all_plugins | 17:47 |
| dtantsur | pas-ha: I think we should do something like https://coderwall.com/p/arh7bq/adding-hooks-to-your-ansible-roles to simplify hooking into typical points, like grub configuration | 17:47 |
| sambetts | jroll: I think some jobs still have it :/ | 17:48 |
| jroll | I'm not seeing it in the suspected logs | 17:48 |
| sambetts | https://github.com/openstack/ironic/blob/f4a6f146e27153ddeec9718852528d94d752e485/playbooks/legacy/ironic-dsvm-functional/run.yaml#L53 | 17:48 |
| sambetts | https://github.com/openstack/ironic/blob/f4a6f146e27153ddeec9718852528d94d752e485/playbooks/legacy/tempest-dsvm-ironic-parallel/run.yaml#L38 | 17:49 |
| jlvillal | sambetts, Luckily only two of our jobs | 17:49 |
| sambetts | yeah it seems like it | 17:49 |
| jroll | $20 removing that doesn't break anything :) | 17:49 |
| jlvillal | So I think we (Ironic) our okay. | 17:49 |
| jlvillal | s/our okay/are okay/ | 17:49 |
| *** catintheroof has quit IRC | 17:50 | |
| jroll | we don't run either of those jobs, it looks like | 17:50 |
| jroll | https://review.openstack.org/#/c/350831/ | 17:50 |
| patchbot | patch 350831 - ironic - Add API methods for [un]rescue | 17:50 |
| jroll | at least not on ironic patches | 17:50 |
| jlvillal | Both of those are experimental jobs | 17:50 |
| sambetts | so third party CIs need to not use DEVSTACK_GATE_TEMPEST_ALL_PLUGIN=1 and make sure to use TEMPEST_PLUGINS= | 17:50 |
| jroll | nod | 17:50 |
| sambetts | and that will sort it for CI | 17:50 |
| jlvillal | sambetts, TEMPEST_PLUGINS=... ? | 17:50 |
| jroll | sambetts: agree, awesome | 17:50 |
| jlvillal | sambetts, As in 'not empty' | 17:51 |
| sambetts | yeah, not empty | 17:51 |
| jroll | jlvillal: e.g. TEMPEST_PLUGINS+=' /opt/stack/new/ironic-tempest-plugin' | 17:51 |
| jlvillal | Cool :) | 17:51 |
| jlvillal | sambetts, I can reply to the email thread or I can let you have the honor? | 17:51 |
| *** catintheroof has joined #openstack-ironic | 17:51 | |
| jlvillal | rajinir, ^^^^ See about do not use DEVSTACK_GATE_TEMPEST_ALL_PLUGIN | 17:52 |
| *** mkovacik__ has quit IRC | 17:52 | |
| sambetts | but we need to document somewhere that you must not have both ironic/ironic-tempest-plugin installed in the same environment where your running tempest | 17:52 |
| jlvillal | sambetts, Well soon (within 3 weeks) it should be deleted | 17:52 |
| sambetts | and maybe how to manually install ironic-tempest-plugin into tempest's virtual env (outside of the gate) | 17:52 |
| *** athomas has joined #openstack-ironic | 17:52 | |
| sambetts | jlvillal: won't fix it for stables though | 17:52 |
| jroll | remind me why we don't delete that code on stable? | 17:53 |
| jlvillal | jroll, dtantsur said he would -2 it. | 17:53 |
| jroll | but the actual reason... :) | 17:53 |
| dtantsur | that's a breaking change on stable, with all its consequences | 17:53 |
| jlvillal | sambetts, email. me? you? | 17:54 |
| dtantsur | we have to be sure that the problem we're fixing is worth than the problem of people whose stuff will break with ImportError after 'yum install' | 17:54 |
| dtantsur | s/worth/worse/ | 17:54 |
| jroll | sure | 17:54 |
| dtantsur | and s/install/update/ | 17:54 |
| dtantsur | meh, I should go get dinner.. | 17:54 |
| jroll | fair enough | 17:54 |
| *** mkovacik__ has joined #openstack-ironic | 17:54 | |
| sambetts | jlvillal: I'll follow up | 17:55 |
| jlvillal | sambetts, Thanks. You rock :) | 17:55 |
| jlvillal | rloo, https://review.openstack.org/#/c/529119/ It works :) | 17:55 |
| patchbot | patch 529119 - ironic - Prevent changes to the ironic_tempest_plugin/ dire... | 17:55 |
| dtantsur | haha, cool approach | 17:56 |
| dtantsur | another fun approach will be to create a job that always fails. and then make it run only on changes touching ironic_tempest_plugin | 17:56 |
| dtantsur | iirc zuul allows it | 17:56 |
| jroll | heh, ++ | 17:56 |
| jlvillal | dtantsur, heh. That would work too. I'm not 100% sure how to do it 'for a file'. I know how to ignore a file. | 17:57 |
| dtantsur | I seem to recall that tripleo does it | 17:58 |
| *** jtomasek has joined #openstack-ironic | 17:58 | |
| jlvillal | dtantsur, Oh yeah. Like the API docs jobs only run when changes made to the api | 17:59 |
| rloo | jlvillal: no sense spending any more time on this than needed. I'm good with your approach. Shall I +2? there was a period missing but i don't care. | 18:00 |
| mtreinish | jlvillal: what sambetts said above is what we should be moving to | 18:00 |
| sambetts | jlvillal: responded | 18:00 |
| mtreinish | using DEVSTACK_GATE_TEMPEST_ALL_PLUGIN is something we need to move away from | 18:00 |
| jlvillal | rloo, Thanks | 18:00 |
| jlvillal | sambetts, thanks | 18:00 |
| jlvillal | mtreinish, Good to know :) | 18:00 |
| mtreinish | using system site-packages, especially on stable jobs, just causes headaches in the long term | 18:01 |
| rloo | jlvillal: what did you use to test, i want to paste a link to that | 18:01 |
| *** r-daneel_ has joined #openstack-ironic | 18:01 | |
| jlvillal | rloo, It is in my comment | 18:01 |
| jlvillal | rloo, https://review.openstack.org/#/c/529120/ | 18:01 |
| patchbot | patch 529120 - ironic - DNM/WIP: Test that PEP8 fails | 18:01 |
| *** r-daneel has quit IRC | 18:01 | |
| *** r-daneel_ is now known as r-daneel | 18:01 | |
| jlvillal | rloo, The failure log: http://logs.openstack.org/20/529120/1/check/openstack-tox-pep8/aa7b7f9/job-output.txt.gz#_2017-12-19_17_45_46_258491 | 18:01 |
| rloo | jlvillal: thx | 18:01 |
| jlvillal | rloo, Basically I added an empty file ironic_tempest_plugin/hello | 18:01 |
| rloo | jlvillal: are you really sure it is an ERROR ERROR ERROR? :) | 18:02 |
| jlvillal | heh | 18:02 |
| * sambetts --> home | 18:03 | |
| *** sambetts is now known as sambetts|afk | 18:03 | |
| sambetts|afk | night all | 18:03 |
| sambetts|afk | o/ | 18:03 |
| jlvillal | sambetts|afk, Good night. Thanks! | 18:03 |
| *** ElCoyote_ has joined #openstack-ironic | 18:04 | |
| *** dyasny has quit IRC | 18:06 | |
| rloo | bye sambetts|afk | 18:06 |
| *** catintheroof has quit IRC | 18:07 | |
| *** catintheroof has joined #openstack-ironic | 18:08 | |
| *** dyasny has joined #openstack-ironic | 18:09 | |
| *** dyasny has quit IRC | 18:09 | |
| *** dyasny has joined #openstack-ironic | 18:10 | |
| *** harlowja has joined #openstack-ironic | 18:15 | |
| *** racedo has quit IRC | 18:20 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:22 | |
| *** links has quit IRC | 18:23 | |
| *** dougsz has quit IRC | 18:25 | |
| *** AlexeyAbashkin has quit IRC | 18:27 | |
| *** racedo has joined #openstack-ironic | 18:28 | |
| openstackgerrit | Michael Turek proposed openstack/ironic-python-agent master: Use lshw in place of dmidecode for the default hardware manager https://review.openstack.org/501799 | 18:29 |
| *** dgautam has joined #openstack-ironic | 18:30 | |
| dgautam | How can I force ironic to release lock on a node. Even if I delete nova instance, lock is not released. I have to restart ironic-conductor, then set maintenance of node, then delete the node and re-add the node again. | 18:30 |
| *** rmcall has joined #openstack-ironic | 18:30 | |
| TheJulia | dgautam: what is causing you to think the node is locked? | 18:31 |
| TheJulia | dgautam: also, what release of ironic are you using? | 18:31 |
| dgautam | I am using ocata. | 18:32 |
| dgautam | iscsid is not able to push the image, It was network issue (smaller MTU). but now it is currently in deploying state for many hours. | 18:33 |
| TheJulia | did you try issuing an abort? | 18:33 |
| dgautam | not yet. issue abort from where ? | 18:34 |
| dgautam | let me try abort | 18:34 |
| TheJulia | did you find the command in the openstack command? | 18:36 |
| dgautam | yes. | 18:36 |
| jlvillal | dtantsur, TheJulia: If you have a chance: https://review.openstack.org/529119 | 18:36 |
| patchbot | patch 529119 - ironic - Prevent changes to the ironic_tempest_plugin/ dire... | 18:36 |
| jlvillal | Then we don't need to -2 all those patches. | 18:36 |
| *** racedo has quit IRC | 18:37 | |
| dgautam | TheJulia: still same error. it is reporting “Error contacting Ironic server: Node c5642404-9b62-4c38-a60c-052b4acd5033 is locked by host XXXX” | 18:38 |
| *** nitishk has joined #openstack-ironic | 18:38 | |
| TheJulia | dgautam: it is stuck in deploying state correct? | 18:38 |
| dgautam | yes | 18:39 |
| TheJulia | hmmm | 18:39 |
| TheJulia | has the conductor service where the deployment hung been restarted, specifically the host XXXX | 18:40 |
| *** trown|lunch is now known as trown | 18:40 | |
| *** trown is now known as trown|ruck | 18:40 | |
| dgautam | not yet. | 18:40 |
| TheJulia | sounds like the deploy process hung where iscsi was trying to fire up (which is odd, I've never actually seen a deployment hang there before...), but restarting the conductor service will clear the lock, and it _shouldn't_ try to relock the node, which then you should be able to abort. | 18:42 |
| dgautam | let try deleting instance, then try abort | 18:43 |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:46 | |
| dgautam | TheJulia: deleting instance and restarting conductor moved node to “deploy failed” state | 18:51 |
| TheJulia | \o/ | 18:51 |
| *** dtantsur is now known as dtantsur|afk | 18:56 | |
| dtantsur|afk | o/ | 18:56 |
| *** catintheroof has quit IRC | 18:56 | |
| *** r-daneel has quit IRC | 18:57 | |
| *** AlexeyAbashkin has quit IRC | 18:58 | |
| *** r-daneel has joined #openstack-ironic | 19:03 | |
| *** mkovacik__ has quit IRC | 19:08 | |
| dgautam | How can I clean node’s provisioning-state. It is currently in error state. and apart from “maintenance set/unset” all are reporting failure that " | 19:08 |
| dgautam | The requested action "provide" can not be performed on node "c5642404-9b62-4c38-a60c-052b4acd5033" while it is in state "error” | 19:08 |
| dgautam | it was in “deploy failed”, I issued abort, it moved to error state now. | 19:09 |
| *** crushil has joined #openstack-ironic | 19:11 | |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-create-vm-nodes: Set CPU model to 'host-model' https://review.openstack.org/528677 | 19:27 |
| *** catintheroof has joined #openstack-ironic | 19:29 | |
| *** hjensas has quit IRC | 19:30 | |
| *** hjensas has joined #openstack-ironic | 19:32 | |
| vdrok | dgautam: here is the state machine of ironic https://docs.openstack.org/ironic/latest/_images/states.svg | 19:36 |
| vdrok | error state means that something went wrong during node deletion | 19:36 |
| vdrok | the easiest probably is just delete the node completely and recreate it (here I mean node-delete command) | 19:37 |
| vdrok | you could also try rebuild or deleting provision states | 19:37 |
| *** openstack has joined #openstack-ironic | 19:43 | |
| *** ChanServ sets mode: +o openstack | 19:43 | |
| openstackgerrit | Merged openstack/ironic master: Prevent changes to the ironic_tempest_plugin/ directory https://review.openstack.org/529119 | 19:47 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add baremetal neutron agent https://review.openstack.org/456235 | 19:52 |
| jlvillal | dtantsur|afk: So I think with https://review.openstack.org/529119 landing you can remove your -2 blocks. https://review.openstack.org/#/q/project:openstack/ironic+path:%255Eironic_tempest_plugin/.*+status:open | 19:53 |
| patchbot | patch 529119 - ironic - Prevent changes to the ironic_tempest_plugin/ dire... (MERGED) | 19:53 |
| *** dprince has quit IRC | 19:55 | |
| *** fragatina has quit IRC | 19:56 | |
| TheJulia | looks like the rescue patches on the whiteboard were out of order, I put the first patch in the chain at the top of the rescue list since everything is built upon it | 19:58 |
| rloo | TheJulia: aren't they the same now, the first two? | 19:59 |
| TheJulia | oh, different descriptions | 20:00 |
| TheJulia | clearly I need more coffeeeeeeee | 20:00 |
| rloo | TheJulia: heh. we could use the same description as in the commit msg :) | 20:00 |
| rloo | TheJulia: i was just about to think about whether i wanted to review that or not. staring at it... :) | 20:01 |
| TheJulia | That generally tends to help :) | 20:01 |
| rloo | TheJulia: wrt the node traits patch, i was wondering if we should -2 until all the patches or most of the patches are available, but i guess it is ok... | 20:01 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/527733 | 20:02 |
| TheJulia | I was also pondering that, I think it might not be the best idea given how we do need to get it done this cycle if at all possible, and we have a very short time window | 20:02 |
| TheJulia | I don't think we should do anything to discourage reviews | 20:03 |
| rloo | TheJulia: yeah. also, i'm assuming the cores that are reviewing are keeping an eye on it. there might be some tricky parts later. i think vdrok and i had some discussions wrt the node tag stuff ... i almost feel like we should just get the tags done, before the traits but lack of time... | 20:04 |
| *** hjensas has joined #openstack-ironic | 20:09 | |
| *** jtomasek has quit IRC | 20:15 | |
| TheJulia | well, near parallel updates and just carry forth the patchsets to merge them to "just get it done" | 20:18 |
| *** dprince has joined #openstack-ironic | 20:23 | |
| rloo | TheJulia: qq for you: https://review.openstack.org/#/c/509335/25/ironic/drivers/base.py. Do you think it is safe to move rescue? | 20:32 |
| patchbot | patch 509335 - ironic - Adds rescue_interface to base driver class | 20:32 |
| TheJulia | If they implemented their own rescue or an early copy... it would absolutely break but I don't think moving it would really break it unless someone had code expecting it. | 20:37 |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:37 | |
| TheJulia | at which point, implementation should merge conflict/break anyway | 20:37 |
| *** AlexeyAbashkin has quit IRC | 20:41 | |
| *** athomas has quit IRC | 20:44 | |
| *** penick has quit IRC | 20:53 | |
| *** catintheroof has quit IRC | 21:02 | |
| *** catintheroof has joined #openstack-ironic | 21:03 | |
| *** catintheroof has quit IRC | 21:07 | |
| rloo | TheJulia: ok, if no one else thinks it is a problem, then it is fine with me. | 21:16 |
| *** penick has joined #openstack-ironic | 21:20 | |
| *** trown|ruck is now known as trown|outtypewww | 21:43 | |
| *** priteau_ has joined #openstack-ironic | 21:46 | |
| *** priteau has quit IRC | 21:50 | |
| *** ijw has joined #openstack-ironic | 21:56 | |
| openstackgerrit | Thomas Herve proposed openstack/ironic master: Split pxe drivers https://review.openstack.org/529170 | 22:02 |
| *** priteau_ has quit IRC | 22:09 | |
| *** priteau has joined #openstack-ironic | 22:09 | |
| *** baha has quit IRC | 22:11 | |
| *** jcoufal has quit IRC | 22:13 | |
| *** priteau has quit IRC | 22:14 | |
| *** penick has quit IRC | 22:19 | |
| *** penick has joined #openstack-ironic | 22:25 | |
| *** crushil__ has quit IRC | 22:27 | |
| *** crushil has quit IRC | 22:27 | |
| *** rcernin has joined #openstack-ironic | 22:28 | |
| *** penick has quit IRC | 22:40 | |
| *** penick has joined #openstack-ironic | 22:43 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Rescue extension for TinyIPA with DHCP network https://review.openstack.org/445692 | 22:55 |
| *** lennyb has quit IRC | 22:58 | |
| *** rbudden has quit IRC | 23:00 | |
| *** lennyb has joined #openstack-ironic | 23:00 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add documentation for rescue mode https://review.openstack.org/431622 | 23:22 |
| *** bfournie has quit IRC | 23:27 | |
| *** penick has quit IRC | 23:39 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Remove /ironic-python-agent/api/app.wsgi https://review.openstack.org/523341 | 23:41 |
| *** catinthe_ has joined #openstack-ironic | 23:43 | |
| *** catintheroof has joined #openstack-ironic | 23:52 | |
| *** catintheroof has quit IRC | 23:52 | |
| *** catintheroof has joined #openstack-ironic | 23:53 | |
| *** catinthe_ has quit IRC | 23:55 | |
| *** catintheroof has quit IRC | 23:58 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [WIP] Refactor deploy interface methods https://review.openstack.org/524433 | 23:59 |
| *** bfournie has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
