| *** ijw has joined #openstack-ironic | 00:01 | |
| *** ijw has quit IRC | 00:01 | |
| *** ijw has joined #openstack-ironic | 00:02 | |
| *** sasha2 has quit IRC | 00:09 | |
| *** rama_y_ has quit IRC | 00:12 | |
| *** Sukhdev_ has quit IRC | 00:16 | |
| *** r-daneel has quit IRC | 00:20 | |
| *** bfournie has joined #openstack-ironic | 00:21 | |
| *** chas has joined #openstack-ironic | 00:24 | |
| *** chas has quit IRC | 00:29 | |
| *** litao__ has joined #openstack-ironic | 00:31 | |
| *** tuanluong has joined #openstack-ironic | 00:36 | |
| openstackgerrit | Anup Navare proposed openstack/ironic-ui master: Display enabled interfaces for underlying driver https://review.openstack.org/467804 | 00:36 |
|---|---|---|
| *** causten has quit IRC | 00:38 | |
| *** ijw has quit IRC | 00:39 | |
| *** fragatina has joined #openstack-ironic | 00:54 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** srobert has joined #openstack-ironic | 01:05 | |
| *** SerenaFeng has joined #openstack-ironic | 01:05 | |
| *** srobert has quit IRC | 01:08 | |
| *** srobert has joined #openstack-ironic | 01:08 | |
| *** SerenaFeng has quit IRC | 01:13 | |
| *** SerenaFeng has joined #openstack-ironic | 01:13 | |
| *** fragatina has quit IRC | 01:24 | |
| *** crushil has quit IRC | 01:24 | |
| *** crushil has joined #openstack-ironic | 01:27 | |
| *** links has joined #openstack-ironic | 01:30 | |
| *** mtanino__ has joined #openstack-ironic | 01:44 | |
| *** annp has joined #openstack-ironic | 01:49 | |
| *** mtanino__ has quit IRC | 01:59 | |
| *** ram5391 has joined #openstack-ironic | 02:04 | |
| *** zhurong has joined #openstack-ironic | 02:05 | |
| *** nicodemos has quit IRC | 02:08 | |
| *** ijw has joined #openstack-ironic | 02:15 | |
| *** ijw has quit IRC | 02:18 | |
| *** rloo has quit IRC | 02:18 | |
| *** ijw_ has joined #openstack-ironic | 02:19 | |
| *** ijw__ has joined #openstack-ironic | 02:20 | |
| *** ijw_ has quit IRC | 02:24 | |
| *** chas has joined #openstack-ironic | 02:24 | |
| *** jaganathan has quit IRC | 02:29 | |
| *** chas has quit IRC | 02:30 | |
| *** mtanino_ has joined #openstack-ironic | 02:37 | |
| *** rbudden has quit IRC | 02:38 | |
| *** jaganathan has joined #openstack-ironic | 02:42 | |
| *** tuanluong has quit IRC | 02:50 | |
| *** mtanino_ has quit IRC | 02:56 | |
| *** yuanying has joined #openstack-ironic | 03:24 | |
| *** hamzy has joined #openstack-ironic | 03:31 | |
| *** Sukhdev has joined #openstack-ironic | 03:32 | |
| *** crushil has quit IRC | 03:36 | |
| *** vikrant has joined #openstack-ironic | 03:42 | |
| *** tuanluong has joined #openstack-ironic | 03:47 | |
| *** crushil has joined #openstack-ironic | 03:54 | |
| *** tuanluong has quit IRC | 03:59 | |
| *** vikrant is now known as vikrant|brb | 04:00 | |
| *** zhurong has quit IRC | 04:08 | |
| *** SerenaFeng has quit IRC | 04:11 | |
| *** Sukhdev has quit IRC | 04:11 | |
| *** vikrant|brb is now known as vikrant | 04:14 | |
| *** zhurong has joined #openstack-ironic | 04:16 | |
| *** jaganathan has quit IRC | 04:18 | |
| *** chas has joined #openstack-ironic | 04:25 | |
| *** chas has quit IRC | 04:30 | |
| *** jaganathan has joined #openstack-ironic | 04:32 | |
| *** crushil has quit IRC | 04:38 | |
| *** jtomasek has quit IRC | 04:46 | |
| *** crushil has joined #openstack-ironic | 04:50 | |
| *** karthiks has joined #openstack-ironic | 05:00 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:04 | |
| *** mikal_ is now known as mikal | 05:06 | |
| Nisha_Agarwal | morning Ironic'iers!!! | 05:07 |
| *** jaganathan has quit IRC | 05:08 | |
| *** jaganathan has joined #openstack-ironic | 05:09 | |
| *** crushil has quit IRC | 05:13 | |
| *** udesale has joined #openstack-ironic | 05:17 | |
| *** mvpnitesh has joined #openstack-ironic | 05:18 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 05:21 |
| *** rbartal has joined #openstack-ironic | 05:28 | |
| *** ram5391 has quit IRC | 05:35 | |
| openstackgerrit | paresh sao proposed openstack/ironic-python-agent master: Adds UEFI support for building coreos image https://review.openstack.org/467894 | 05:37 |
| *** SerenaFeng has joined #openstack-ironic | 05:38 | |
| openstackgerrit | paresh sao proposed openstack/ironic-python-agent master: Adds UEFI support for building coreos image https://review.openstack.org/467894 | 05:46 |
| *** root has joined #openstack-ironic | 05:52 | |
| *** root is now known as Guest24428 | 05:52 | |
| *** jaganathan has quit IRC | 05:55 | |
| *** root____6 has quit IRC | 05:56 | |
| *** sayantan_ has quit IRC | 06:02 | |
| *** Dinesh_Bhor has quit IRC | 06:06 | |
| *** jaganathan has joined #openstack-ironic | 06:07 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 06:11 | |
| *** rcernin has joined #openstack-ironic | 06:18 | |
| *** jaganathan has quit IRC | 06:18 | |
| *** jaganathan has joined #openstack-ironic | 06:19 | |
| *** pcaruana has joined #openstack-ironic | 06:20 | |
| *** Nisha_ has joined #openstack-ironic | 06:22 | |
| *** Nisha_Agarwal has quit IRC | 06:22 | |
| *** chas has joined #openstack-ironic | 06:26 | |
| *** chas has quit IRC | 06:30 | |
| *** sayantan_ has joined #openstack-ironic | 06:34 | |
| *** Nisha_ has quit IRC | 06:43 | |
| *** sayantan_ has quit IRC | 06:47 | |
| *** sayantan_ has joined #openstack-ironic | 06:47 | |
| *** sayantan_ has quit IRC | 06:47 | |
| *** sayantan_ has joined #openstack-ironic | 06:48 | |
| *** adisky_ has joined #openstack-ironic | 06:50 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Enable cinder storage interface for generic hardware https://review.openstack.org/463908 | 06:50 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add storage interface field to node-related notifications https://review.openstack.org/463972 | 06:51 |
| *** ralonsoh has joined #openstack-ironic | 06:55 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface https://review.openstack.org/467911 | 06:59 |
| *** gcb has quit IRC | 07:10 | |
| *** SerenaFeng has quit IRC | 07:11 | |
| *** SerenaFeng has joined #openstack-ironic | 07:11 | |
| *** sayantan_ has quit IRC | 07:13 | |
| *** mjura has joined #openstack-ironic | 07:13 | |
| *** gcb has joined #openstack-ironic | 07:14 | |
| openstackgerrit | vinay kumar muddu proposed openstack/proliantutils master: [WIP] Added secure boot support through redfish https://review.openstack.org/467912 | 07:15 |
| *** rbartal_ has joined #openstack-ironic | 07:19 | |
| *** rbartal has quit IRC | 07:22 | |
| *** Serlex has joined #openstack-ironic | 07:25 | |
| *** zhurong has quit IRC | 07:30 | |
| *** SerenaFeng has quit IRC | 07:32 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 07:33 |
| *** dtantsur|afk is now known as dtantsur | 07:38 | |
| dtantsur | morning ironic | 07:38 |
| *** fragatina has joined #openstack-ironic | 07:38 | |
| *** SerenaFeng has joined #openstack-ironic | 07:43 | |
| *** fragatina has quit IRC | 07:44 | |
| *** zhurong has joined #openstack-ironic | 07:45 | |
| *** kromanenko has joined #openstack-ironic | 07:49 | |
| *** mgoddard has joined #openstack-ironic | 07:51 | |
| *** rbartal_ has quit IRC | 07:55 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** mvpnitesh has quit IRC | 08:01 | |
| *** mvpnitesh has joined #openstack-ironic | 08:01 | |
| *** zzzeek has joined #openstack-ironic | 08:02 | |
| *** fragatina has joined #openstack-ironic | 08:05 | |
| *** lucas-pto is now known as lucasagomes | 08:08 | |
| pas-ha | morning dtantsur and ironic :) | 08:08 |
| dtantsur | o/ | 08:09 |
| *** jaianshu has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [install-guide] explain the defaults calculation for hardware types https://review.openstack.org/466741 | 08:14 |
| dtantsur | vdrok: ^^^ | 08:14 |
| vdrok | good morning ironic, dtantsur pas-ha :) | 08:15 |
| vdrok | looking | 08:15 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Update ipmitool installation and usage documentation https://review.openstack.org/466734 | 08:22 |
| *** chas has joined #openstack-ironic | 08:27 | |
| *** athomas has joined #openstack-ironic | 08:29 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adding 'get_one_time_boot' method using redfish https://review.openstack.org/467520 | 08:31 |
| *** chas has quit IRC | 08:31 | |
| *** mvpnitesh has quit IRC | 08:34 | |
| *** mvpnitesh has joined #openstack-ironic | 08:34 | |
| *** Lambert has joined #openstack-ironic | 08:36 | |
| *** phuongnh has quit IRC | 08:38 | |
| *** phuongnh has joined #openstack-ironic | 08:39 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Change volume metadata not to use nested dicts https://review.openstack.org/467930 | 08:49 |
| *** mgoddard has quit IRC | 08:53 | |
| *** zhurong has quit IRC | 08:54 | |
| *** e0ne has joined #openstack-ironic | 08:56 | |
| *** baoli has joined #openstack-ironic | 09:02 | |
| *** racedo has joined #openstack-ironic | 09:02 | |
| *** zhurong has joined #openstack-ironic | 09:04 | |
| *** baoli has quit IRC | 09:06 | |
| *** derekh has joined #openstack-ironic | 09:09 | |
| *** pmannidi__ has quit IRC | 09:10 | |
| *** rbartal_ has joined #openstack-ironic | 09:16 | |
| *** priteau has joined #openstack-ironic | 09:17 | |
| *** phuongnh has quit IRC | 09:25 | |
| *** phuongnh has joined #openstack-ironic | 09:26 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Update ipmitool installation and usage documentation https://review.openstack.org/466734 | 09:29 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: WIP Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 09:32 |
| *** udesale has quit IRC | 09:34 | |
| yolanda | hi , good morning. vdrok , do you know if there has been some update in the sql fixes for ocata? | 09:35 |
| *** zhurong has quit IRC | 09:35 | |
| vdrok | yolanda: morning, hm, nope, I don't have +2 on stable branches | 09:36 |
| yolanda | ironic-cores, can you help on that? we are broken on bifrost for stable/ocata | 09:36 |
| vdrok | dtantsur: could you take a look at https://review.openstack.org/459455 ? | 09:37 |
| patchbot | patch 459455 - ironic (stable/ocata) - Don't pass sqlite_db in db_options.set_defaults() | 09:37 |
| *** udesale has joined #openstack-ironic | 09:39 | |
| * dtantsur looks | 09:41 | |
| openstackgerrit | Merged openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 09:43 |
| yolanda | thx | 09:44 |
| dtantsur | yolanda, vdrok, this patch is not correct, the option seems present in Ocata.. | 09:45 |
| *** SerenaFeng has quit IRC | 09:45 | |
| yolanda | so sqlite problems are caused by another issue? | 09:45 |
| dtantsur | I'm not sure, but the argument is still there: https://github.com/openstack/oslo.db/blob/stable/ocata/oslo_db/options.py#L149 | 09:46 |
| vdrok | yolanda: maybe the version installed for bifrost is incorrect? | 09:46 |
| dtantsur | yolanda: are you sure bifrost respects the branches properly? | 09:46 |
| dtantsur | yes, this | 09:46 |
| sambetts|afk | it might be a versioning issue then, e.g. upper constraints on the oslo.db version | 09:46 |
| *** sambetts|afk is now known as sambetts | 09:46 | |
| vdrok | good morning sambetts | 09:47 |
| dtantsur | sambetts: morning | 09:47 |
| sambetts | hey vdrok dtantsur | 09:47 |
| *** udesale has quit IRC | 09:48 | |
| yolanda | dtantsur, i see on logs http://logs.openstack.org/82/464782/1/check/gate-bifrost-integration-tinyipa-ubuntu-xenial/18ec14b/console.html.gz that it installed ocata | 09:50 |
| yolanda | http://logs.openstack.org/82/464782/1/check/gate-bifrost-integration-tinyipa-ubuntu-xenial/18ec14b/console.html.gz#_2017-05-19_08_11_22_466214 | 09:50 |
| *** udesale has joined #openstack-ironic | 09:53 | |
| *** root has joined #openstack-ironic | 09:53 | |
| *** root is now known as Guest17849 | 09:53 | |
| *** Guest24428 has quit IRC | 09:56 | |
| dtantsur | yeah, seems right. interesting | 10:01 |
| dtantsur | TheJulia: hi! would you like to request (or for me to request) a Bifrost mid-cycle release soon(ish)? | 10:15 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:19 | |
| dtantsur | ditto re ironic-ui | 10:21 |
| Nisha_Agarwal | dtantsur, hi | 10:22 |
| dtantsur | o/ | 10:22 |
| Nisha_Agarwal | wanted to know your opinion on https://bugs.launchpad.net/ironic/+bug/1681320 | 10:22 |
| openstack | Launchpad bug 1681320 in Ironic "[RFE] Allow inspection to discover pci devices" [Wishlist,Triaged] | 10:22 |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adds 'get_pending_boot_mode' in ILO redfish https://review.openstack.org/467982 | 10:23 |
| *** udesale has quit IRC | 10:23 | |
| *** udesale has joined #openstack-ironic | 10:23 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adds 'get_pending_boot_mode' in ILO redfish https://review.openstack.org/467982 | 10:24 |
| dtantsur | Nisha_Agarwal: I think we already discover PCI devices.. | 10:26 |
| dtantsur | or is it about OOB? | 10:26 |
| Nisha_Agarwal | it is about OOB | 10:27 |
| *** annp has quit IRC | 10:27 | |
| Nisha_Agarwal | currently PCI devices are discovered as part of inspector | 10:27 |
| Nisha_Agarwal | Now OOB inspection wants to do the same so inspector solution needs to partially sit in ironic | 10:28 |
| Nisha_Agarwal | and half in driver libraries | 10:28 |
| *** chas has joined #openstack-ironic | 10:28 | |
| dtantsur | Nisha_Agarwal: yeah, but what's the use case? inspector one was introduced for populating capabilities. I'm not sure about the fate of capabilities now.. | 10:28 |
| Nisha_Agarwal | dtantsur, I explain the steps need to be done for it | 10:28 |
| Nisha_Agarwal | dtantsur, it will populate the "alias" properties in in node.properties which can be read by nova directly | 10:29 |
| Nisha_Agarwal | the count etc is same as it is in inspector | 10:29 |
| Nisha_Agarwal | pci thing doesnt change in nova as far as i know | 10:30 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Stop sending custom context values over RPC https://review.openstack.org/467985 | 10:30 |
| *** chas has quit IRC | 10:33 | |
| *** milan has joined #openstack-ironic | 10:36 | |
| *** milan has left #openstack-ironic | 10:36 | |
| vdrok | dtantsur: yolanda upper constraints for ocata say that 4.17.0 should be installed but the logs seem to indicate 4.21.0 being installed | 10:36 |
| Nisha_Agarwal | dtantsur, ^^^^ | 10:41 |
| dtantsur | Nisha_Agarwal: do you have nova blueprint/spec for the nova side? | 10:42 |
| dtantsur | I'm not sure they're going to be fond of it, we need to check with them first | 10:42 |
| Nisha_Agarwal | dtantsur, nova side we dont need any change unless we want pcipasthorugh filter to be default filter for ironic | 10:43 |
| Nisha_Agarwal | dtantsur, user need to modify nova.conf for adding the filter name and add the alias on the nova side | 10:44 |
| Nisha_Agarwal | rest all changes are needed only on ironic and driver libraries | 10:44 |
| Nisha_Agarwal | dtantsur, i have explained the end to end steps in the bug above | 10:45 |
| Nisha_Agarwal | dtantsur, i have the blueprint on the nova side but i will abandon it after we finalize on it | 10:46 |
| *** lucasagomes is now known as lucas-hungry | 10:48 | |
| dtantsur | Nisha_Agarwal: sorry, but how does Nova become aware of node.properties['alias']? | 10:49 |
| dtantsur | I don't think we support it now | 10:50 |
| Nisha_Agarwal | dtantsur, thats the part of pcipassthrough filter | 10:50 |
| dtantsur | Nisha_Agarwal: how does this value get to that filter? | 10:50 |
| dtantsur | we seem to have a fixed list of properties we use: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L212-L236 | 10:51 |
| Nisha_Agarwal | dtantsur, i experimented it without any change in ironic, i manually added alias field to ironic node.properties and did the nova.conf changes for alias and pcipassthrough filter. This made the node to be selected given flavor has pci devices alias in its flavor | 10:53 |
| dtantsur | I believe you, but I'd like to understand the code path that leads to it | 10:53 |
| dtantsur | again, the previous inspector change was concerning capabilities. this is quite obvious, we know how it works. | 10:54 |
| sambetts | I am also sceptical that this worked | 10:54 |
| dtantsur | if you want to do it with capabilities - fine (just keep in mind the ongoing nova work on traits) | 10:54 |
| dtantsur | but what you're describing is something I'd like Nova folks to buy into | 10:54 |
| sambetts | (and also on going work in nova with the placements API which is going to change everything regarding PCI devices too | 10:55 |
| Nisha_Agarwal | dtantsur, we can add it into capabilities but i am just doing the way nova schedules VMs for pci devices | 10:56 |
| dtantsur | fine, but we need to figure out the nova side then | 10:57 |
| Nisha_Agarwal | dtantsur, i didnt get what we want to figure on nova side....Nova side we follow their documents and use it for ironic nodes | 10:57 |
| *** jkilpatr has quit IRC | 10:57 | |
| sambetts | what we're saying is that there needs to be nova changes because the ironic nova driver currently will ignore any properties you add to the ironic node unless they are cpu_arch etc | 10:58 |
| dtantsur | Nisha_Agarwal: this needs a blueprint for the ironic virt driver. please present it to the nova community (specifically, jaypipes, I guess). | 10:58 |
| dtantsur | if they don't reject it, we can proceed with discussing the ironic part | 10:59 |
| sambetts | dtantsur: IMO i think it'll need to part of placement API reporting | 10:59 |
| Nisha_Agarwal | dtantsur, sambetts i experimented it without any chnage to ironci virt driver and it works | 10:59 |
| dtantsur | Nisha_Agarwal: please show me the code dealing with node.properties['alias'] in the ironic virt driver | 11:00 |
| Nisha_Agarwal | dtantsur, sambetts i also thought it reauires ironic virt driver changes but just adding the filter name to the "baremetal_enabled_filters" enables the filter for ironci nodes | 11:01 |
| Nisha_Agarwal | dtantsur, wait i will show | 11:01 |
| Nisha_Agarwal | s/reauires/requires | 11:01 |
| dtantsur | there seems to be a separate resource class for PCI devices, which we'll have to expose: https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L467 | 11:02 |
| Nisha_Agarwal | dtantsur, yes true its a seperate resource class | 11:03 |
| Nisha_Agarwal | dtantsur, but we just need the way inspector has | 11:04 |
| sambetts | dtantsur: the way PCI devices are represented in the placemnt API is that they will be child resource providers of the resource provider representing the baremetal node | 11:04 |
| dtantsur | oh, interesting | 11:04 |
| Nisha_Agarwal | dtantsur, i dont think we need more than what inspector does apart from that we populate it as alias in node.properties | 11:05 |
| dtantsur | yeah, but the latter is something we can't agree on | 11:05 |
| Nisha_Agarwal | dtantsur, why? | 11:05 |
| dtantsur | for the reasons explained above | 11:06 |
| *** jaianshu has quit IRC | 11:06 | |
| dtantsur | here's what the ironic nova driver seems to track: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L618-L676 | 11:06 |
| Nisha_Agarwal | dtantsur, pcipassthrough filter looks into the node.properties by default and schedules the node | 11:06 |
| dtantsur | I don't see PCI devices there | 11:06 |
| Nisha_Agarwal | dtantsur, yes PCI devices are not there | 11:06 |
| Nisha_Agarwal | but it works.... | 11:07 |
| *** mtanino_ has joined #openstack-ironic | 11:07 | |
| dtantsur | please get an ack from the Nova team on your blueprint. if they confirm that there's no work to do - I'm happy to discuss further. | 11:07 |
| Nisha_Agarwal | dtantsur, i actually just followed https://docs.openstack.org/admin-guide/compute-pci-passthrough.html | 11:07 |
| Nisha_Agarwal | dtantsur, just following above steps made it work for ironic nodes | 11:09 |
| Nisha_Agarwal | after adding filter name to the baremetal_enabled_filters | 11:09 |
| sambetts | that looks like its just hard coding the available pci devices from the nova compute | 11:10 |
| dtantsur | yep | 11:10 |
| dtantsur | Nisha_Agarwal: we can't find the code looking it node.properties['alias']. this is strange, we need to understand how it works before we commit to it. | 11:11 |
| dtantsur | otherwise it may end up broken in a months, because nova folks change some internals.. | 11:11 |
| dtantsur | or in Queens when we move to resource classes (as you confirmed, we do not expose PCI_DEVICE resources) | 11:11 |
| Nisha_Agarwal | dtantsur, this gets called through pci passthorugh filter | 11:12 |
| Nisha_Agarwal | https://github.com/openstack/nova/blob/master/nova/pci/request.py#L100 | 11:12 |
| Nisha_Agarwal | which validates the Alias | 11:12 |
| sambetts | that only reads nova.conf | 11:12 |
| dtantsur | Nisha_Agarwal: yes, I understand this bit. I don't understand where exactly the ironic virt driver exposes the field. | 11:12 |
| dtantsur | it *has* to be in nova/virt/ironic/ directory, please don't look anywhere else | 11:13 |
| Nisha_Agarwal | dtantsur, ironic virt driver doesnt expose that | 11:13 |
| dtantsur | ok, here we have a problem. you populate the field that nova has no clue about. why do you need to populate it then? | 11:13 |
| *** jkilpatr has joined #openstack-ironic | 11:14 | |
| Nisha_Agarwal | dtantsur, i didnt get ...nova knows about the field thats why filter is working, right? | 11:14 |
| sambetts | all the code you've linked to only looks at the pci_alias option in nova.cond | 11:15 |
| sambetts | conf* | 11:15 |
| dtantsur | Nisha_Agarwal: no, nova does not know about this field. you've just confirmed it. I think you're misinterpreting the results of your testing. | 11:15 |
| Nisha_Agarwal | dtantsur, sambetts i dont think so. if nova doesnt know it cannot work or schedule different nodes for pci passthorugh filter based on flavor with pcipassthorugh filter | 11:16 |
| Nisha_Agarwal | give me few min. i will just show where it works | 11:17 |
| dtantsur | right. and it does not know. so you need a nova blueprint to make it aware of it. | 11:17 |
| *** phuongnh has quit IRC | 11:20 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 11:22 |
| *** mgoddard has joined #openstack-ironic | 11:29 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Better driver interface isolation https://review.openstack.org/468000 | 11:30 |
| *** Nisha_Agarwal has quit IRC | 11:32 | |
| *** Lambert has quit IRC | 11:38 | |
| *** jtomasek has joined #openstack-ironic | 11:46 | |
| *** amarao has joined #openstack-ironic | 11:46 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:53 | |
| *** lucas-hungry is now known as lucasagomes | 11:55 | |
| mvpnitesh | Hey Sambetts, should i change anything in this patch https://review.openstack.org/#/c/462931/ | 11:58 |
| patchbot | patch 462931 - ironic - Ironic conductor works correctly on changes of net... | 11:58 |
| *** rook is now known as rook-pto | 11:59 | |
| *** udesale has quit IRC | 11:59 | |
| *** rook-pto is now known as rook | 11:59 | |
| *** udesale has joined #openstack-ironic | 11:59 | |
| *** udesale has quit IRC | 11:59 | |
| *** jkilpatr has quit IRC | 12:00 | |
| *** jkilpatr has joined #openstack-ironic | 12:01 | |
| *** Nisha_Agarwal has quit IRC | 12:01 | |
| *** mat128 has joined #openstack-ironic | 12:05 | |
| *** xavierr has joined #openstack-ironic | 12:08 | |
| *** rbartal_ has quit IRC | 12:12 | |
| *** mtanino_ has quit IRC | 12:14 | |
| *** jkilpatr has quit IRC | 12:16 | |
| *** mvpnitesh has quit IRC | 12:16 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:18 | |
| *** Nisha_Agarwal has quit IRC | 12:19 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:22 | |
| *** karthiks has quit IRC | 12:24 | |
| *** chas has joined #openstack-ironic | 12:29 | |
| *** rbudden has joined #openstack-ironic | 12:29 | |
| *** dprince has joined #openstack-ironic | 12:31 | |
| *** chas has quit IRC | 12:33 | |
| *** srobert has quit IRC | 12:34 | |
| *** jkilpatr has joined #openstack-ironic | 12:43 | |
| *** vikrant has quit IRC | 12:44 | |
| *** chas has joined #openstack-ironic | 12:45 | |
| *** mat128 has quit IRC | 12:46 | |
| *** chas has quit IRC | 12:49 | |
| *** hw_wutianwei has quit IRC | 12:50 | |
| *** links has quit IRC | 12:54 | |
| *** trown|outtypewww is now known as trown | 12:57 | |
| *** nicodemos has joined #openstack-ironic | 12:58 | |
| *** Goneri has joined #openstack-ironic | 12:59 | |
| *** sasha2 has joined #openstack-ironic | 13:01 | |
| *** mat128 has joined #openstack-ironic | 13:02 | |
| *** baoli has joined #openstack-ironic | 13:05 | |
| TheJulia | Good morning everyone | 13:06 |
| *** chlong has quit IRC | 13:06 | |
| sambetts | hey TheJulia | 13:06 |
| lucasagomes | TheJulia, sambetts morning | 13:09 |
| *** Nisha_Agarwal has quit IRC | 13:10 | |
| robcresswell | This channel is like an ARP storm when someone says hello | 13:10 |
| robcresswell | :p | 13:10 |
| dtantsur | morning TheJulia, hi lucasagomes | 13:10 |
| dtantsur | :) | 13:10 |
| lucasagomes | dtantsur, yo o/ | 13:10 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Setup JavaScript test environment https://review.openstack.org/453730 | 13:10 |
| TheJulia | ^^^ rebase | 13:11 |
| lucasagomes | robcresswell, hah yeah, and it's always morning (-: (http://www.total-knowledge.com/~ilya/mips/ugt.html) | 13:11 |
| dtantsur | TheJulia: I have issues releasing ironic-ui stable, could you please check? https://review.openstack.org/467979 | 13:11 |
| patchbot | patch 467979 - releases - Release ironic-ui 2.2.1 for Ocata | 13:11 |
| openstackgerrit | Merged openstack/ironic-ui master: Fix Javascript unit tests https://review.openstack.org/466273 | 13:11 |
| robcresswell | lucasagomes: Ha, I like that | 13:11 |
| TheJulia | dtantsur: interesting | 13:12 |
| lucasagomes | we should document that in ironic somewhere, probably in the wiki page | 13:12 |
| *** baoli has quit IRC | 13:13 | |
| *** baoli has joined #openstack-ironic | 13:13 | |
| *** rloo has joined #openstack-ironic | 13:14 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Remove explicit package version https://review.openstack.org/468029 | 13:17 |
| dtantsur | lucasagomes: patches are welcome :D | 13:17 |
| lucasagomes | hah yeah, will leave as a todo here :D | 13:18 |
| *** ccamacho|out has quit IRC | 13:18 | |
| openstackgerrit | Merged openstack/bifrost master: Fix when clause in pymysql check on CentOS https://review.openstack.org/467357 | 13:18 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui stable/ocata: Remove explicit package version https://review.openstack.org/468030 | 13:19 |
| TheJulia | dtantsur: ^^^ I think that should fix it | 13:19 |
| TheJulia | Sorry for being absent so much the last few days, i didn't expect shifting my schedule and the time zones to consume me as much as it did :( | 13:20 |
| dtantsur | TheJulia: no worries, thanks! are you sure that these versions are ok? | 13:24 |
| TheJulia | dtantsur: I think so yes, I double checked horizon's config and it was set to 0.0.0. And I seem to remember having to do this before | 13:24 |
| TheJulia | so *crosses fingers* | 13:24 |
| *** fragatina has quit IRC | 13:25 | |
| TheJulia | If anyone would take a quick sanity check look at https://review.openstack.org/#/c/453730, it would be appreciated. | 13:27 |
| patchbot | patch 453730 - ironic-ui - Setup JavaScript test environment | 13:27 |
| robcresswell | dtantsur: There is a fix here: https://review.openstack.org/#/c/467688/ | 13:31 |
| patchbot | patch 467688 - releases - handle mixed-language repos packaged as python | 13:31 |
| robcresswell | dtantsur: Horizon is blocked by it too. Its a new issue caused by releases adding some new npm gate jobs. | 13:32 |
| robcresswell | So all mixed-language things (Horizon + plugins) are blocked | 13:32 |
| robcresswell | I believe the solution is basically to estimate whether it is a python module or JS, and then skip appropriately | 13:32 |
| dtantsur | robcresswell: ouch.. thanks! | 13:33 |
| TheJulia | fun, thanks robcresswell | 13:38 |
| TheJulia | dtantsur: I guess those two patches I threw up might as well still land, since we shouldn't really have hard coded 1.0.0 versions anyway | 13:38 |
| rloo | hi and good morning all, TheJulia, dtantsur, robcresswell, lucasagomes, vdrok | 13:39 |
| lucasagomes | rloo, o/ morning | 13:39 |
| rloo | TheJulia: have you seen this? bifrost ocata gate broken, and vdrok's comment about why: https://review.openstack.org/#/c/459455/ | 13:40 |
| patchbot | patch 459455 - ironic (stable/ocata) - Don't pass sqlite_db in db_options.set_defaults() | 13:40 |
| vdrok | morning TheJulia and rloo | 13:40 |
| *** baoli_ has joined #openstack-ironic | 13:40 | |
| dtantsur | TheJulia++ | 13:41 |
| dtantsur | morning rloo | 13:42 |
| *** baoli has quit IRC | 13:43 | |
| TheJulia | rloo: Yeah, I'm thinking upper constraints is not getting pulled in somewhere along the way and with that breaking change we really need it to be honoring upper constraints. I need to go through the package installs and see what is actually causing it | 13:44 |
| rloo | TheJulia: thx for looking into it! | 13:44 |
| rloo | TheJulia: makes me wonder what changed recently to cause it to break | 13:44 |
| TheJulia | Yeah :( | 13:46 |
| *** mtanino_ has joined #openstack-ironic | 13:48 | |
| dtantsur | is there anyone here using hybrid bm-vm clouds? I want to hear from you how you're doing it :) | 13:50 |
| dtantsur | we're using host aggregates, and we've got severely broken by nova apparently.. | 13:51 |
| sambetts | dtantsur: we are, we're also using host aggregates right now | 13:51 |
| dtantsur | I wonder what to put to our reference architecture docs even, except for "trying something, maybe it will work" | 13:51 |
| dtantsur | sambetts: in which way? similar to http://tripleo.org/advanced_deployment/baremetal_overcloud.html#creating-flavors-and-host-aggregates and http://tripleo.org/advanced_deployment/baremetal_overcloud.html#assigning-host-aggregates ? | 13:52 |
| sambetts | dtantsur: we should encourage people to use the resource classes now though which should make it better | 13:52 |
| sambetts | dtantsur: yup like that | 13:52 |
| dtantsur | hmm, I see | 13:52 |
| dtantsur | sambetts: apparently, nova no longer likes adding hosts without any ironic nodes attached to them | 13:53 |
| dtantsur | which gives some pain, especially with testing | 13:53 |
| sambetts | dtantsur: there is an easy "fix" | 13:53 |
| dtantsur | when you have 1-2 nodes and you to guess somehow which of three controllers they map to | 13:53 |
| dtantsur | sambetts: really? what's that? | 13:53 |
| openstackgerrit | Merged openstack/ironic-ui master: Fix Javascript unassignable errors https://review.openstack.org/447532 | 13:54 |
| *** mnaser has joined #openstack-ironic | 13:55 | |
| sambetts | you have to run the nova-manage discover-hosts command after you've added your nodes to ironic OR what we've started doing is enrolling a fake driver node with 1 ram 1 disk 1 cpu, and then running discover-hosts as part of the install and then we delete the fake nodes after we've been discovered | 13:55 |
| mnaser | o/ hi folks. would i be correct in my reading that at the moment ironic requires that all VMs run on the same L2 when for introspection and provisioning? | 13:56 |
| dtantsur | sambetts: oh, I see... | 13:56 |
| dtantsur | mnaser: not really requires, but it's the default mode of operation yes | 13:56 |
| openstackgerrit | Merged openstack/ironic-ui master: Setup JavaScript test environment https://review.openstack.org/453730 | 13:56 |
| mnaser | dtantsur: is there a way around that? i've been looking :x | 13:56 |
| mnaser | search keywords or links welcome :) | 13:57 |
| *** crushil has joined #openstack-ironic | 13:57 | |
| dtantsur | mnaser: ironic can use multiple networks: https://docs.openstack.org/developer/ironic/deploy/multitenancy.html but configuring an ML2 driver for VMs may be funny | 13:58 |
| dtantsur | mnaser: sambetts or vdrok (or vsaienk0 who's IIRC on PTO) may help with it. we do it for our CI. | 13:58 |
| sambetts | dtantsur: the hosts with no resources doesn't just break host-aggs it breaks ironic nova compute period, I think we had to add a work around in our devstack plugin somewhere already | 13:58 |
| dtantsur | ouch. OUCH! | 13:59 |
| sambetts | mnaser: are you using ironic to boot VMs for testing? I | 13:59 |
| TheJulia | Wow, bifrost's broken gate is downright bizzar | 13:59 |
| dtantsur | a broken gate is often like that | 13:59 |
| sambetts | mnaser: in the Ironic CI we have a neutron ML2 driver that allows neutron to control OVS for like a real TOR for baremetal testing | 14:00 |
| mnaser | sambetts: nope, looking at using ironic to maintain a fleet of physical infra. we have many racks and they don't currently share a single L2 domain | 14:00 |
| sambetts | mnaser: why did you say VMs then? or did you mean BMs? | 14:01 |
| mnaser | gah | 14:01 |
| mnaser | not enough coffee | 14:01 |
| mnaser | im sorry | 14:01 |
| mnaser | "that all physical nodes" | 14:01 |
| dtantsur | BMs are easier :) you still need an ML2 plugin suitable for your switches | 14:02 |
| sambetts | mnaser: ah ok cool, Ironic can support dynamic network managment through neutron, if there is an ML2 driver that supports your physical TORs | 14:02 |
| mnaser | so there would be a neutron "provisioning" network and then ironic will handle switching back and forth i assume? | 14:02 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost stable/ocata: DNM: Test to gather data for broken gate https://review.openstack.org/468043 | 14:02 |
| dtantsur | sambetts: fyi https://review.openstack.org/#/c/460616/ | 14:03 |
| patchbot | patch 460616 - nova - re-Allow adding computes with no ComputeNodes to a... | 14:03 |
| dtantsur | mnaser: right | 14:03 |
| sambetts | dtantsur: ooooooooo!!! | 14:03 |
| mnaser | i guess its back to the drawing board for a bit to think about it | 14:04 |
| mnaser | thanks dtantsur and sambetts ! | 14:04 |
| sambetts | mnaser: in the dynamic network case we (downstream) run inspection has a completely separate network, and we manually setup the TOR into that network when we rack the servers, we do an inspection on that network, then Ironic + Neutron take over the switch and baremetal for provisioning + tenant networks | 14:05 |
| mnaser | the thing is we come from nova world where our switches aren't smart and openvswitch does all the magic | 14:05 |
| sambetts | mnaser: ah yeah that makes things a little more difficult | 14:06 |
| mnaser | i was hoping (and hey im sure contribution are welcome) that ironic had something similar to maas' rackd or foreman's "smart proxy" | 14:06 |
| mnaser | which really is a simple agent that runs inside an L2 which communicates with the "master" node and proxies dhcp/pxe/etc | 14:06 |
| sambetts | you still end up with all the baremetals in the same L2 though right? | 14:07 |
| *** ccamacho has joined #openstack-ironic | 14:07 | |
| mnaser | sambetts we run a spine/leaf network so each rack has it's own L2 | 14:07 |
| mnaser | so the L2 is limited to a max of ~40 servers | 14:08 |
| hjensas | dtantsur: I've been working on OOO spine-leaf support, we need to use Neutron Segments (routed-networks model) to make DHCP for provisioning work. Now I realize we need https://bugs.launchpad.net/ironic/+bug/1658964 implemented. | 14:08 |
| openstack | Launchpad bug 1658964 in Ironic "[RFE] Implement neutron routed networks support in Ironic" [Wishlist,Confirmed] | 14:08 |
| mnaser | rather than however many racks you have | 14:08 |
| hjensas | dtantsur: related bug: https://bugs.launchpad.net/ironic/+bug/1693284 led me to the RFE. | 14:08 |
| openstack | Launchpad bug 1693284 in neutron "Routed Networks - Ironic Port Binding - baremetal node on remote non-local subnet" [Undecided,In progress] - Assigned to Harald Jensås (harald-jensas) | 14:08 |
| sambetts | mnaser: you'd need support for neutron L3 networks in that csae | 14:09 |
| sambetts | case | 14:09 |
| sambetts | there is an RFE in place for that, and work on going | 14:09 |
| *** jaypipes has joined #openstack-ironic | 14:09 | |
| mnaser | yeah i've done a bit of reading about that. i dont know if hjensas just brought it up out of coincidence or in context of the convo :p | 14:09 |
| sambetts | yeah, seems concidental ;) | 14:10 |
| hjensas | mnaser: coincidence. :) | 14:10 |
| dtantsur | hjensas: the guys are working on it IIUC | 14:10 |
| dtantsur | help is appreciated, as always | 14:10 |
| hjensas | sambetts: do you know who is working on it? The RFE bug 1658964 is in wishlist, noone assigned. | 14:11 |
| openstack | bug 1658964 in Ironic "[RFE] Implement neutron routed networks support in Ironic" [Wishlist,Confirmed] https://launchpad.net/bugs/1658964 | 14:11 |
| sambetts | hjensas: it has a dependecy on a couple of things the most important one is ironic physical network awareness | 14:12 |
| sambetts | hjensas: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/physical-network-awareness.html | 14:12 |
| sambetts | mgoddard__: is working on that peice and there are patches up for it | 14:13 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost stable/ocata: DNM: Test to gather data for broken gate https://review.openstack.org/468043 | 14:13 |
| sambetts | hjensas: I should probably assign myself to that RFE as I have code up for it | 14:13 |
| dtantsur | sambetts: please do | 14:13 |
| hjensas | sambetts: thanks, I will read the spec and have a look at any gerrit reviews. | 14:14 |
| sambetts | hjensas: there is a peice that is required to feed the physical awareness info into neutron, that is the code I have up | 14:14 |
| dtantsur | sambetts: this will also require an ML2 driver and "neutron" network interface, won't it? | 14:14 |
| hjensas | sambetts: does that by chance change how the binding:host_id is used? | 14:14 |
| *** mgoddard has quit IRC | 14:14 | |
| *** ijw__ has quit IRC | 14:15 | |
| *** jtomasek has quit IRC | 14:16 | |
| sambetts | hjensas: binding:host_id is today and will continue to be the ironic node UUID, so then with the peice I'm writing to feed to phys net info into, neutron can work out what segments are avaible to which ironic nodes | 14:17 |
| sambetts | *magic* http://www.reactiongifs.com/r/mgc.gif | 14:18 |
| dtantsur | lol | 14:19 |
| dtantsur | so, I guess it's "neutron", not "flat".. which we don't yet support in OOO, I guess | 14:19 |
| hjensas | sambetts: :) sounds good. | 14:20 |
| sambetts | dtantsur: I think this will provide support for statically configure l3 networks too | 14:20 |
| sambetts | configured* | 14:20 |
| dtantsur | ah, good then | 14:20 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Consolidate code for setting node maintenance state https://review.openstack.org/465972 | 14:20 |
| TheJulia | ppiela: I marked 463852 -1, If you feel differently, please let me know | 14:21 |
| sambetts | dtantsur: BTW if you start using the resource classes you should be able to stop using host aggs for baremetal+vms today | 14:22 |
| sambetts | dtantsur: you can specify the resource class as an extra_spec and jaypipes has written it so that if it sees that extra_spec it'll ignore everything else from the flavor | 14:23 |
| dtantsur | sambetts: yeah, but I think something was missing from resource class support still.. and dunno about Ocata | 14:23 |
| *** udesale has joined #openstack-ironic | 14:23 | |
| sambetts | unsure, jaypipes gave me the impression at the summit that it should be useable today | 14:23 |
| jaypipes | sambetts: s/has written it/is writing it/ :) | 14:23 |
| *** udesale has quit IRC | 14:23 | |
| sambetts | ah | 14:24 |
| jaypipes | sambetts: sorry! | 14:24 |
| dtantsur | that's the problem, I see :) | 14:24 |
| *** udesale has joined #openstack-ironic | 14:24 | |
| jaypipes | dtantsur: sorry, man, still working on it. | 14:24 |
| dtantsur | hah, no prob :) | 14:24 |
| openstackgerrit | Merged openstack/ironic-ui master: Modify regex pattern used to validate node ids https://review.openstack.org/465982 | 14:24 |
| dtantsur | jaypipes: btw, do you need any additional input from ironic side while jroll is enjoying beaches and cocktails? | 14:25 |
| *** r-daneel has joined #openstack-ironic | 14:25 | |
| ppiela | TheJulia: I agree with adding a release note. I will review, and make a recommendation on unit tests. | 14:25 |
| TheJulia | ppiela: Thanks! | 14:25 |
| sambetts | jaypipes: does that mean dropping the RAM, CPU and DISK in queens is still up for discussion ;) /me isn't sold on doing that over just correctly setting the min_unit in get_inventory | 14:25 |
| dtantsur | I also wonder what do to about "get me any node" pattern that tripleo actively uses | 14:26 |
| sambetts | dtantsur: thats an easy one to solve | 14:27 |
| dtantsur | how? | 14:27 |
| jaypipes | sambetts: that ship sailed a while ago, I'm afraid.. | 14:27 |
| *** chlong has joined #openstack-ironic | 14:27 | |
| TheJulia | Regarding bifrost's gate... apparently zuul-cloner is not actually setting the branch properly Oo | 14:27 |
| dtantsur | TheJulia: Oo | 14:28 |
| sambetts | dtantsur: either give all the nodes the same resource class OR have inspection rules that sets the ram,disk,cpu to the same thing on all the nodes | 14:28 |
| dtantsur | sambetts: 1. they have different resource classes, 2. ram,disk,cpu are going to be ignored, no? | 14:29 |
| sambetts | dtantsur: why do they have different resource classes? you choose then | 14:29 |
| sambetts | if you want to treat all the nodes the same, treat them the same | 14:29 |
| dtantsur | sambetts: well, they have to. they are different. just *sometimes* for some role a user may not care | 14:29 |
| dtantsur | I'm not saying it's great, but this is how things work today | 14:30 |
| sambetts | dtantsur: it shouldn't be... flavors aren't suppose to work like that | 14:30 |
| dtantsur | e.g. in tripleo you can ask "just deploy any two nodes" or you can ask "deploy controllers on this class, and computes on this" | 14:30 |
| dtantsur | sambetts: well, we're simulating resource classes with capabilities nowadays, and it does work | 14:30 |
| dtantsur | with a change to resource classes, the simple case goes away | 14:31 |
| sambetts | dtantsur: you could acheive what your doing with traits/capbilities even if resource classes | 14:31 |
| TheJulia | dtantsur: http://logs.openstack.org/43/468043/2/check/gate-bifrost-integration-tinyipa-ubuntu-xenial/51f524d/console.html for 8| when your brain has free cycle | 14:31 |
| sambetts | dtantsur: ever node has the same resource class, and then different traits/capabilities | 14:31 |
| sambetts | dtantsur: then you can ask for 1 baremetal_silver, or 1 baremetal_siver with SSD=true | 14:32 |
| sambetts | you don't need to have baremetal_silver and baremetal_silver_ssd | 14:32 |
| sambetts | as separate resource classes | 14:32 |
| dtantsur | TheJulia: eeeek, wut? | 14:33 |
| TheJulia | dtantsur: yeah, exactly | 14:33 |
| dtantsur | sambetts: doable, yes. but this is essentially working around resource classes. and it assumes the folks won't change the capabilities/traits model | 14:33 |
| TheJulia | the git log output is master branch :( | 14:33 |
| dtantsur | TheJulia: I guess it's good time to poke #infra | 14:34 |
| TheJulia | yup | 14:34 |
| * dtantsur needs fresh air | 14:36 | |
| *** dtantsur is now known as dtantsur|brb | 14:36 | |
| *** ram5391 has joined #openstack-ironic | 14:37 | |
| sambetts | dtantsur: this is why I am kind of -1 on the removing support for ram/disk/cpu for baremetal, but accord to jaypipes that ship has sailed :/ I wish I'd managed to be more involved when it was orginally being designed (was too busy making networking work), but with the min_unit, max_unit, and total fields on resource classes I don't see why it needs to be removed, because we can acheive | 14:37 |
| sambetts | atomicness by setting them all to the same thing, and it won't affect anyones else because all that will happen inside the ironic virt driver's get_inventory function :/ | 14:37 |
| *** MattMan_1 has quit IRC | 14:38 | |
| *** MattMan_1 has joined #openstack-ironic | 14:39 | |
| sambetts | and the scheduler and placement should continue to work as normal with no specialness required for ironic | 14:40 |
| sambetts | surely that makes maintaing this stuff waaay easier | 14:41 |
| openstackgerrit | Merged openstack/ironic-inspector master: Bump pydot2 to pydot3 https://review.openstack.org/460938 | 14:43 |
| openstackgerrit | Wang Xuguang proposed openstack/ironic master: Replace test.attr with decorators.attr https://review.openstack.org/468055 | 14:43 |
| *** chas has joined #openstack-ironic | 14:45 | |
| *** ram5391 has quit IRC | 14:46 | |
| *** udesale has quit IRC | 14:48 | |
| *** mgoddard has joined #openstack-ironic | 14:49 | |
| *** karthiks has joined #openstack-ironic | 14:49 | |
| *** chas has quit IRC | 14:50 | |
| *** ijw has joined #openstack-ironic | 15:00 | |
| *** ijw has quit IRC | 15:05 | |
| *** fragatina has joined #openstack-ironic | 15:06 | |
| *** Serlex has quit IRC | 15:07 | |
| *** derekh has quit IRC | 15:10 | |
| *** kromanenko has quit IRC | 15:10 | |
| *** ram5391 has joined #openstack-ironic | 15:12 | |
| TheJulia | mjturek: I'm looking at the bfv meeting notes from last week and wondering if my wiki page update just didn't get saved. Did you update it in the past week by chance? | 15:17 |
| TheJulia | s/notes/agenda/ | 15:17 |
| *** penick has joined #openstack-ironic | 15:18 | |
| TheJulia | dtantsur|brb: rloo: w/r/t bifrost's gate, https://review.openstack.org/#/c/468061/ + one other patch that I'll post once that lands which will change where the scripts look. I'll then back port that to the stable branches. | 15:21 |
| patchbot | patch 468061 - openstack-infra/project-config - Explicitly clone requirements for bifrost | 15:21 |
| rloo | TheJulia: ok. i don't know much about jenkins etc, will trust you :-) | 15:22 |
| TheJulia | tl;dr, bifrost was doing it wrong :) | 15:22 |
| mjturek | TheJulia: I haven't updated it this week! | 15:22 |
| TheJulia | mjturek: thanks | 15:23 |
| rloo | TheJulia: that's what I figured. Am assuming your fix is doing it 'right' :-) right-er anyway ;) | 15:23 |
| TheJulia | Oh, and now I know what the next thing will be :) | 15:26 |
| rloo | TheJulia: so a dumb question wrt https://review.openstack.org/#/c/468061/1/jenkins/jobs/bifrost.yaml. why do you add requirements for one job only? | 15:33 |
| patchbot | patch 468061 - openstack-infra/project-config - Explicitly clone requirements for bifrost | 15:33 |
| TheJulia | rloo: great catch, I'm juggling too much at the moment | 15:35 |
| TheJulia | :( | 15:35 |
| rloo | TheJulia: thx. Seems like we need a macro for the templates. ha ha. | 15:36 |
| TheJulia | Heh, just about | 15:37 |
| TheJulia | although apparently zuulv3 will be changing some of these things so it won't really matter | 15:37 |
| *** pcaruana has quit IRC | 15:38 | |
| TheJulia | note to self: always push code asap if I'm in the air. | 15:38 |
| *** pcaruana has joined #openstack-ironic | 15:39 | |
| TheJulia | I need to go peel back an onion of what I did between some revisions that mjturek also changed over the past week :( | 15:39 |
| mjturek | TheJulia: the wirein patch? I only added cinder to the default driver list | 15:40 |
| mjturek | and rebased :) | 15:40 |
| TheJulia | Yeah, the wire-in. Okay | 15:41 |
| TheJulia | I'll be able to actually dedicate cycles to looking at that after the meeting and I get travel booked for next week :( | 15:41 |
| *** racedo has quit IRC | 15:44 | |
| mjturek | \o/ yeeaah sounds like you've been pretty busy | 15:44 |
| *** leifmadsen has quit IRC | 15:45 | |
| *** lucasagomes is now known as lucas-afk | 15:47 | |
| TheJulia | Yeah which is good... and bad. | 15:47 |
| *** hshiina has joined #openstack-ironic | 15:48 | |
| *** leifmadsen has joined #openstack-ironic | 15:49 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM rework keystone auth for glance https://review.openstack.org/467728 | 15:49 |
| *** dtantsur|brb is now known as dtantsur | 15:49 | |
| *** crushil has quit IRC | 15:50 | |
| dtantsur | TheJulia: travel again? omg | 15:50 |
| *** e0ne has quit IRC | 15:51 | |
| TheJulia | dtantsur: yup :( Hopefully this is the last of it for a month and a half or so, the last half of july is going to be crazy though. | 15:51 |
| dtantsur | wow.. I mean, not wow at all | 15:52 |
| dtantsur | you're incredibly strong, I can't imagine travelling so much | 15:52 |
| TheJulia | s/strong/crazy/ :) | 15:52 |
| dtantsur | this often goes in pair, right? :) | 15:53 |
| TheJulia | possibly | 15:53 |
| dtantsur | TheJulia: the infra patch looks correct. (/me did not know how bifrost CI works) | 15:54 |
| TheJulia | I guess on a plus side, I'm getting better at packing light | 15:54 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:55 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add CRUD notification objects for volume connector and volume target https://review.openstack.org/463930 | 15:55 |
| *** phuongnh has joined #openstack-ironic | 15:56 | |
| dtantsur | TheJulia: true. summits already taught me to not end up with a huge check-in bag :) | 15:57 |
| *** ram5391 has quit IRC | 15:58 | |
| sambetts | +1 small hand lugguage == best | 15:58 |
| dtantsur | unless they really ban laptops from getting on board... | 15:58 |
| sambetts | -_- yeah that'll be dumb | 15:59 |
| *** penick has quit IRC | 15:59 | |
| TheJulia | I'm still over-packing the tech, but I like a clear separation between work and personal computing | 15:59 |
| TheJulia | laptops, tablets, etc :( | 15:59 |
| dtantsur | I have a mess of everything | 15:59 |
| TheJulia | That will be a major productivity killer, but bring massive boom to the airport book stores | 15:59 |
| dtantsur | probably related to the fact that I simply don't have a personal computer | 15:59 |
| *** penick has joined #openstack-ironic | 15:59 | |
| dtantsur | heh | 16:00 |
| *** Nisha_Agarwal has quit IRC | 16:00 | |
| *** penick has quit IRC | 16:04 | |
| *** Goneri has quit IRC | 16:07 | |
| *** Goneri has joined #openstack-ironic | 16:07 | |
| *** causten has joined #openstack-ironic | 16:10 | |
| *** sayantan_ has joined #openstack-ironic | 16:18 | |
| *** crushil has joined #openstack-ironic | 16:25 | |
| *** retr0h has quit IRC | 16:26 | |
| *** retr0h has joined #openstack-ironic | 16:26 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 16:26 |
| *** pcaruana has quit IRC | 16:30 | |
| *** pcaruana has joined #openstack-ironic | 16:30 | |
| *** causten has quit IRC | 16:33 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:37 | |
| *** causten has joined #openstack-ironic | 16:37 | |
| *** udesale has joined #openstack-ironic | 16:39 | |
| *** rcernin has quit IRC | 16:40 | |
| *** jheroux has joined #openstack-ironic | 16:44 | |
| *** chas has joined #openstack-ironic | 16:46 | |
| *** ppiela_ has joined #openstack-ironic | 16:48 | |
| *** skazi has quit IRC | 16:48 | |
| *** ppiela has quit IRC | 16:49 | |
| *** chas has quit IRC | 16:52 | |
| *** jtomasek has joined #openstack-ironic | 16:53 | |
| NobodyCam | Good Morning Ironic'ers | 16:55 |
| dtantsur | morning NobodyCam | 16:56 |
| mjturek | gm NobodyCam | 16:56 |
| dtantsur | and good night everyone | 16:56 |
| mjturek | cya dtantsur | 16:56 |
| TheJulia | Goodnight! | 16:56 |
| NobodyCam | hey dtantsur Good morning. | 16:56 |
| NobodyCam | and I have a quick question | 16:56 |
| dtantsur | to me or to everyone? | 16:56 |
| NobodyCam | to you... any opposition to adding a "noadd" option to add_ports | 16:57 |
| dtantsur | NobodyCam: like, do not add ports at all? | 16:57 |
| NobodyCam | yep! | 16:57 |
| dtantsur | mmm, why? | 16:57 |
| * TheJulia feels like RFE with use case context.... | 16:58 | |
| NobodyCam | because I don;t know the network that is needed at discovery time | 16:58 |
| *** baoli_ has quit IRC | 16:59 | |
| dtantsur | NobodyCam: I don't quite get it.. MAC addresses do not depend on networks, do they? | 16:59 |
| NobodyCam | well for us it does! | 17:00 |
| dtantsur | wow, interesting | 17:00 |
| NobodyCam | :) | 17:00 |
| dtantsur | to be honest, I wish add_ports was an API parameter, not an option. but otherwise I don't have any objections. add_ports=none, ok | 17:01 |
| dtantsur | please do file an RFE | 17:01 |
| *** ralonsoh has quit IRC | 17:01 | |
| *** ijw has joined #openstack-ironic | 17:01 | |
| NobodyCam | once I get thing working here! | 17:01 |
| NobodyCam | *things | 17:01 |
| NobodyCam | :) thank and have a good night! | 17:02 |
| dtantsur | o/ | 17:03 |
| *** dtantsur is now known as dtantsur|afk | 17:03 | |
| *** baoli has joined #openstack-ironic | 17:04 | |
| *** sayantan_ has quit IRC | 17:07 | |
| *** sayantan_ has joined #openstack-ironic | 17:08 | |
| *** trown is now known as trown|lunch | 17:08 | |
| *** sayantan_ has quit IRC | 17:14 | |
| *** sayantan_ has joined #openstack-ironic | 17:14 | |
| *** sayantan_ has quit IRC | 17:17 | |
| *** sayantan_ has joined #openstack-ironic | 17:17 | |
| *** sambetts is now known as sambetts|afk | 17:18 | |
| sambetts|afk | night all | 17:18 |
| *** udesale has quit IRC | 17:20 | |
| *** Nisha_ has joined #openstack-ironic | 17:20 | |
| *** Nisha_Agarwal has quit IRC | 17:20 | |
| *** penick has joined #openstack-ironic | 17:21 | |
| *** Nisha_ has quit IRC | 17:21 | |
| *** phuongnh has quit IRC | 17:21 | |
| *** Nisha_ has joined #openstack-ironic | 17:22 | |
| *** sayantan_ has quit IRC | 17:22 | |
| *** Nisha_ has quit IRC | 17:22 | |
| *** sayantan_ has joined #openstack-ironic | 17:22 | |
| *** Nisha_ has joined #openstack-ironic | 17:22 | |
| *** Nisha_ has quit IRC | 17:24 | |
| *** Nisha_ has joined #openstack-ironic | 17:24 | |
| *** Nisha_brb has joined #openstack-ironic | 17:29 | |
| *** Nisha_ has quit IRC | 17:29 | |
| *** Nisha_ has joined #openstack-ironic | 17:31 | |
| *** Nisha_brb has quit IRC | 17:31 | |
| *** Nisha_brb has joined #openstack-ironic | 17:32 | |
| *** Nisha_ has quit IRC | 17:32 | |
| *** Nisha_brb has quit IRC | 17:34 | |
| *** Nisha_brb has joined #openstack-ironic | 17:34 | |
| *** Nisha_ has joined #openstack-ironic | 17:38 | |
| *** Nisha_brb has quit IRC | 17:38 | |
| *** Nisha_ has quit IRC | 17:39 | |
| *** sayantan_ has quit IRC | 17:39 | |
| *** Nisha_ has joined #openstack-ironic | 17:39 | |
| *** sayantan_ has joined #openstack-ironic | 17:40 | |
| *** Nisha_ has quit IRC | 17:40 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 17:41 | |
| *** crushil has quit IRC | 17:45 | |
| *** Nisha_ has joined #openstack-ironic | 17:46 | |
| *** Nisha_Agarwal has quit IRC | 17:46 | |
| *** fragatina has quit IRC | 17:47 | |
| *** Nisha_brb has joined #openstack-ironic | 17:48 | |
| *** Nisha_ has quit IRC | 17:48 | |
| *** Goneri has quit IRC | 17:48 | |
| *** amarao has quit IRC | 17:49 | |
| *** Nisha_ has joined #openstack-ironic | 17:51 | |
| *** Nisha_brb has quit IRC | 17:51 | |
| *** racedo has joined #openstack-ironic | 17:55 | |
| *** sayantan_ has quit IRC | 18:00 | |
| *** mjura has quit IRC | 18:02 | |
| *** Nisha_ has quit IRC | 18:02 | |
| *** crushil has joined #openstack-ironic | 18:03 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:03 | |
| *** causten has quit IRC | 18:07 | |
| *** baoli has quit IRC | 18:08 | |
| *** Nisha_Agarwal has quit IRC | 18:08 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 18:08 | |
| *** Nisha_ has joined #openstack-ironic | 18:12 | |
| *** Nisha_Agarwal has quit IRC | 18:12 | |
| *** Nisha_brb has joined #openstack-ironic | 18:14 | |
| *** Nisha_ has quit IRC | 18:14 | |
| *** baoli has joined #openstack-ironic | 18:14 | |
| *** baoli has quit IRC | 18:15 | |
| *** Nisha_ has joined #openstack-ironic | 18:16 | |
| *** Nisha_brb has quit IRC | 18:16 | |
| *** hshiina has quit IRC | 18:19 | |
| *** Nisha_ has quit IRC | 18:19 | |
| *** Nisha_ has joined #openstack-ironic | 18:19 | |
| *** rbudden has quit IRC | 18:22 | |
| *** Nisha_brb has joined #openstack-ironic | 18:24 | |
| *** Nisha_ has quit IRC | 18:24 | |
| *** racedo has quit IRC | 18:27 | |
| *** baoli has joined #openstack-ironic | 18:27 | |
| *** fragatina has joined #openstack-ironic | 18:28 | |
| *** fragatina has quit IRC | 18:30 | |
| *** fragatina has joined #openstack-ironic | 18:30 | |
| *** baoli has quit IRC | 18:30 | |
| *** trown|lunch is now known as trown | 18:31 | |
| *** Nisha_brb has quit IRC | 18:32 | |
| *** Nisha_brb has joined #openstack-ironic | 18:32 | |
| *** baoli has joined #openstack-ironic | 18:32 | |
| *** Nisha_brb has quit IRC | 18:33 | |
| *** Nisha_brb has joined #openstack-ironic | 18:33 | |
| *** mgoddard has quit IRC | 18:34 | |
| *** Nisha_ has joined #openstack-ironic | 18:35 | |
| *** Nisha_brb has quit IRC | 18:35 | |
| *** causten has joined #openstack-ironic | 18:35 | |
| *** fragatina has quit IRC | 18:36 | |
| *** Nisha_ has quit IRC | 18:36 | |
| *** Nisha_ has joined #openstack-ironic | 18:36 | |
| *** Nisha_brb has joined #openstack-ironic | 18:38 | |
| *** Nisha_ has quit IRC | 18:38 | |
| *** Nisha_brb has quit IRC | 18:39 | |
| *** Nisha_brb has joined #openstack-ironic | 18:40 | |
| *** Sukhdev has joined #openstack-ironic | 18:42 | |
| *** Nisha_brb has quit IRC | 18:42 | |
| *** Nisha_brb has joined #openstack-ironic | 18:43 | |
| *** Goneri has joined #openstack-ironic | 18:44 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: WIP: Introduce hpOneView to the OneView driver https://review.openstack.org/468129 | 18:47 |
| *** sayantan_ has joined #openstack-ironic | 18:47 | |
| *** chas has joined #openstack-ironic | 18:47 | |
| *** Nisha_brb has quit IRC | 18:48 | |
| *** chas has quit IRC | 18:52 | |
| *** frontrunner has joined #openstack-ironic | 19:00 | |
| *** causten has quit IRC | 19:01 | |
| *** e0ne has joined #openstack-ironic | 19:01 | |
| *** causten has joined #openstack-ironic | 19:05 | |
| *** penick has quit IRC | 19:06 | |
| *** dsneddon is now known as dsneddon_pto | 19:06 | |
| *** pcaruana has quit IRC | 19:09 | |
| *** ppiela_ has quit IRC | 19:10 | |
| *** e0ne has quit IRC | 19:17 | |
| *** jtomasek_ has joined #openstack-ironic | 19:17 | |
| *** baoli has quit IRC | 19:23 | |
| *** causten has quit IRC | 19:25 | |
| *** baoli has joined #openstack-ironic | 19:25 | |
| *** e0ne has joined #openstack-ironic | 19:27 | |
| *** trown is now known as trown|brb | 19:38 | |
| *** trown|brb is now known as trown | 19:40 | |
| *** ppiela has joined #openstack-ironic | 19:45 | |
| *** priteau has quit IRC | 19:55 | |
| *** Sukhdev has quit IRC | 19:57 | |
| *** amotoki is now known as amotoki_away | 19:58 | |
| *** amotoki_away is now known as amotoki | 19:59 | |
| *** amotoki is now known as amotoki_away | 19:59 | |
| *** mgoddard has joined #openstack-ironic | 20:03 | |
| *** chlong has quit IRC | 20:15 | |
| *** sayantan_ has quit IRC | 20:20 | |
| *** sayantan_ has joined #openstack-ironic | 20:20 | |
| *** rbudden has joined #openstack-ironic | 20:27 | |
| *** mgoddard has quit IRC | 20:27 | |
| *** jkilpatr has quit IRC | 20:29 | |
| *** chlong has joined #openstack-ironic | 20:33 | |
| *** chlong has quit IRC | 20:37 | |
| *** chlong has joined #openstack-ironic | 20:38 | |
| *** crushil has quit IRC | 20:39 | |
| *** athomas has quit IRC | 20:46 | |
| *** chas has joined #openstack-ironic | 20:48 | |
| *** yolanda has quit IRC | 20:49 | |
| *** jkilpatr has joined #openstack-ironic | 20:49 | |
| *** chas has quit IRC | 20:52 | |
| openstackgerrit | Xavier proposed openstack/ironic master: you shall not pass https://review.openstack.org/466872 | 20:54 |
| *** trown is now known as trown|outtypewww | 20:59 | |
| *** fragatina has joined #openstack-ironic | 21:00 | |
| *** mgoddard has joined #openstack-ironic | 21:00 | |
| * TheJulia thinks that wins at the commit message subject game | 21:01 | |
| *** xavierr has quit IRC | 21:01 | |
| *** frontrunner has quit IRC | 21:02 | |
| *** jtomasek_ has quit IRC | 21:04 | |
| *** e0ne has quit IRC | 21:05 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Allow logical name in port creation https://review.openstack.org/405586 | 21:08 |
| *** chas has joined #openstack-ironic | 21:09 | |
| *** dprince has quit IRC | 21:10 | |
| *** chlong has quit IRC | 21:11 | |
| *** mgoddard has quit IRC | 21:11 | |
| *** chas has quit IRC | 21:14 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost stable/ocata: Use WORKSPACE for requirements https://review.openstack.org/468043 | 21:19 |
| anupn | Hi TheJulia: wrt https://review.openstack.org/#/c/467804/ nodejs4-npm-run-lint gate is failing, and seems missing due to some docstrings and minor nits, can i run them locally? | 21:21 |
| patchbot | patch 467804 - ironic-ui - Display enabled interfaces for underlying driver | 21:21 |
| anupn | TheJulia: is that something i can run with tox like pep8? | 21:22 |
| TheJulia | anupn: npm test | 21:22 |
| TheJulia | I think | 21:22 |
| TheJulia | npm lint may also be an option | 21:22 |
| * anupn checking | 21:23 | |
| anupn | seems npm-lint or npm-test is not in tox.ini | 21:23 |
| anupn | not sure if i can use tox for that then | 21:23 |
| *** jheroux has quit IRC | 21:25 | |
| TheJulia | anupn: it is not a tox target | 21:27 |
| TheJulia | npm is actually the equivalent node tool | 21:27 |
| TheJulia | well, Node Package Manager | 21:27 |
| TheJulia | so different, but has targets such as test | 21:28 |
| anupn | TheJulia: Ah i see, so will have to get that tool | 21:28 |
| TheJulia | yes | 21:28 |
| anupn | cool | 21:29 |
| anupn | :) | 21:29 |
| *** chas has joined #openstack-ironic | 21:30 | |
| *** crushil has joined #openstack-ironic | 21:30 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Use WORKSPACE for requirements https://review.openstack.org/468170 | 21:32 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost stable/ocata: Use WORKSPACE for requirements https://review.openstack.org/468043 | 21:33 |
| *** mgoddard has joined #openstack-ironic | 21:34 | |
| *** chas has quit IRC | 21:34 | |
| *** crushil has quit IRC | 21:37 | |
| *** chas has joined #openstack-ironic | 21:40 | |
| *** rpioso has joined #openstack-ironic | 21:42 | |
| *** rpioso has quit IRC | 21:43 | |
| *** fragatina has quit IRC | 21:48 | |
| openstackgerrit | Merged openstack/ironic-lib master: Prevent tests from using utils.execute() https://review.openstack.org/464074 | 21:48 |
| NobodyCam | anyone happen to know off the top of their head if one can use "data://" in a inspector rule condition? | 21:49 |
| *** chas has quit IRC | 21:54 | |
| *** chas has joined #openstack-ironic | 21:54 | |
| *** mgoddard has quit IRC | 21:55 | |
| *** chas has quit IRC | 21:59 | |
| TheJulia | NobodyCam: No clue :( | 22:03 |
| NobodyCam | TheJulia: | 22:07 |
| NobodyCam | {"op": "contains", "field": "data://inventory.system_vendor.manufacturer", "value": "Dell"} for the win!!!! | 22:07 |
| TheJulia | \o/ | 22:09 |
| NobodyCam | :) | 22:09 |
| JayF | NobodyCam: data://? | 22:10 |
| NobodyCam | the data collected from the ram disk | 22:11 |
| NobodyCam | ie. add these creds for dell and these other ones for HP and still these other ones for super micro | 22:11 |
| JayF | nice | 22:12 |
| NobodyCam | :) | 22:12 |
| *** chas has joined #openstack-ironic | 22:15 | |
| *** chas has quit IRC | 22:20 | |
| TheJulia | JayF: by chance have stickers made it to you yet? | 22:21 |
| TheJulia | dtantsur|afk: Looks like they are in customs there. | 22:22 |
| *** Goneri has quit IRC | 22:26 | |
| JayF | TheJulia: oh yeah! they did! I forgot to thank you | 22:27 |
| JayF | THANK YOU | 22:27 |
| *** baoli has quit IRC | 22:29 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Add option for building IPMItool into TinyIPA https://review.openstack.org/465932 | 22:30 |
| TheJulia | no problem :) | 22:32 |
| TheJulia | Thank you for sending out the original pixie boots stickers :) | 22:32 |
| JayF | hah, no problem | 22:35 |
| *** ricardoas is now known as ricardoas-afk | 22:49 | |
| *** chas has joined #openstack-ironic | 22:56 | |
| *** chas has quit IRC | 23:01 | |
| *** chas has joined #openstack-ironic | 23:37 | |
| *** sayantan_ has quit IRC | 23:42 | |
| *** chas has quit IRC | 23:42 | |
| *** sayantan_ has joined #openstack-ironic | 23:42 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
