| *** AlexeyAbashkin has joined #openstack-ironic | 00:01 | |
| *** AlexeyAbashkin has quit IRC | 00:05 | |
| *** penick has quit IRC | 00:09 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor deploy interface methods https://review.openstack.org/524433 | 00:10 |
|---|---|---|
| *** e0ne has quit IRC | 00:18 | |
| *** yangyapeng has quit IRC | 00:27 | |
| *** yangyapeng has joined #openstack-ironic | 00:27 | |
| *** yangyapeng has quit IRC | 00:31 | |
| *** liuzz has joined #openstack-ironic | 00:34 | |
| *** andymccr has quit IRC | 00:36 | |
| *** andymccr has joined #openstack-ironic | 00:40 | |
| *** ricardoas has joined #openstack-ironic | 00:51 | |
| *** andymccr has quit IRC | 00:51 | |
| *** ricardoas-away has quit IRC | 00:51 | |
| *** jlabarre has quit IRC | 00:54 | |
| *** andymccr has joined #openstack-ironic | 00:55 | |
| *** yangyapeng has joined #openstack-ironic | 01:00 | |
| *** yangyapeng has quit IRC | 01:06 | |
| *** rmcall has joined #openstack-ironic | 01:10 | |
| *** yangyapeng has joined #openstack-ironic | 01:13 | |
| *** hieulq has quit IRC | 01:16 | |
| *** hieulq has joined #openstack-ironic | 01:17 | |
| *** yangyapeng has quit IRC | 01:18 | |
| *** yangyapeng has joined #openstack-ironic | 01:22 | |
| *** phuongnh has joined #openstack-ironic | 01:29 | |
| *** zhangfei has joined #openstack-ironic | 01:40 | |
| *** d0ugal has quit IRC | 01:43 | |
| *** penick has joined #openstack-ironic | 01:57 | |
| *** penick_ has joined #openstack-ironic | 02:00 | |
| *** d0ugal has joined #openstack-ironic | 02:01 | |
| *** penick has quit IRC | 02:01 | |
| *** liuzz has quit IRC | 02:09 | |
| *** liuzz has joined #openstack-ironic | 02:10 | |
| *** dsneddon has quit IRC | 02:17 | |
| *** rmcall has quit IRC | 02:21 | |
| *** harlowja has quit IRC | 02:24 | |
| *** penick_ has quit IRC | 02:32 | |
| *** vsaienk0 has quit IRC | 02:33 | |
| *** vsaienk0 has joined #openstack-ironic | 02:39 | |
| *** mordred has quit IRC | 02:42 | |
| *** athomas has quit IRC | 02:42 | |
| *** athomas has joined #openstack-ironic | 02:43 | |
| *** mordred has joined #openstack-ironic | 02:44 | |
| *** pmannidi has quit IRC | 02:44 | |
| *** pmannidi has joined #openstack-ironic | 02:48 | |
| *** lbragstad has quit IRC | 02:54 | |
| *** tuanla____ has joined #openstack-ironic | 02:59 | |
| *** ijw has quit IRC | 03:00 | |
| *** sasha2 has quit IRC | 03:05 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Update documentation related to 'refresh()' https://review.openstack.org/531341 | 03:18 |
| *** rbudden has quit IRC | 03:22 | |
| *** jistr has quit IRC | 03:31 | |
| *** hshiina has quit IRC | 03:36 | |
| *** vsaienk0 has quit IRC | 03:37 | |
| *** vsaienk0 has joined #openstack-ironic | 03:45 | |
| *** ijw has joined #openstack-ironic | 03:52 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 03:52 | |
| *** AlexeyAbashkin has quit IRC | 03:57 | |
| *** charcol has quit IRC | 03:59 | |
| *** udesale has joined #openstack-ironic | 03:59 | |
| *** hshiina has joined #openstack-ironic | 03:59 | |
| *** vsaienk0 has quit IRC | 04:00 | |
| *** charcol has joined #openstack-ironic | 04:01 | |
| *** ccollett has joined #openstack-ironic | 04:02 | |
| *** vsaienk0 has joined #openstack-ironic | 04:13 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 04:25 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds RPC calls for rescue interface https://review.openstack.org/509336 | 04:28 |
| *** AlexeyAbashkin has quit IRC | 04:29 | |
| *** hshiina2 has joined #openstack-ironic | 04:30 | |
| *** hshiina has quit IRC | 04:33 | |
| *** links has joined #openstack-ironic | 04:37 | |
| *** links has quit IRC | 04:40 | |
| *** ijw has quit IRC | 04:42 | |
| *** ijw has joined #openstack-ironic | 04:42 | |
| *** namnh has joined #openstack-ironic | 04:44 | |
| *** ijw has quit IRC | 04:45 | |
| *** links has joined #openstack-ironic | 04:46 | |
| *** hoonetorg has quit IRC | 05:00 | |
| *** karthiks has joined #openstack-ironic | 05:10 | |
| *** stendulker has joined #openstack-ironic | 05:11 | |
| *** hoonetorg has joined #openstack-ironic | 05:14 | |
| *** tzumainn has quit IRC | 05:14 | |
| *** jaganathan has joined #openstack-ironic | 05:19 | |
| *** harlowja has joined #openstack-ironic | 05:24 | |
| *** hshiina3 has joined #openstack-ironic | 05:26 | |
| *** hshiina2 has quit IRC | 05:30 | |
| *** deray has joined #openstack-ironic | 05:33 | |
| deray | hello and good morning Ironic'ers! | 05:34 |
| *** dpawar has joined #openstack-ironic | 05:57 | |
| *** rbartal has joined #openstack-ironic | 05:58 | |
| *** karthiks has quit IRC | 06:04 | |
| *** oanson has joined #openstack-ironic | 06:06 | |
| *** racedo has joined #openstack-ironic | 06:10 | |
| *** racedo has quit IRC | 06:10 | |
| *** oanson has quit IRC | 06:18 | |
| *** oanson has joined #openstack-ironic | 06:22 | |
| *** pcaruana has joined #openstack-ironic | 06:27 | |
| *** pcaruana has quit IRC | 06:33 | |
| *** pcaruana has joined #openstack-ironic | 06:34 | |
| *** karthiks has joined #openstack-ironic | 06:34 | |
| *** mjura has joined #openstack-ironic | 06:36 | |
| *** fragatina has quit IRC | 06:36 | |
| *** marios has joined #openstack-ironic | 06:45 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: [WIP} Fixes issue with UEFI bootable ISO creation https://review.openstack.org/531700 | 06:52 |
| *** hjensas has quit IRC | 06:54 | |
| *** gcb has quit IRC | 06:55 | |
| *** udesale__ has joined #openstack-ironic | 06:57 | |
| *** udesale has quit IRC | 06:59 | |
| *** tuanla____ has quit IRC | 07:04 | |
| *** tuanla____ has joined #openstack-ironic | 07:05 | |
| *** masber has quit IRC | 07:08 | |
| *** absubram has joined #openstack-ironic | 07:13 | |
| *** dpawar has quit IRC | 07:14 | |
| *** rcernin has quit IRC | 07:15 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic-python-agent master: Rescue extension for TinyIPA with DHCP network https://review.openstack.org/445692 | 07:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/532436 | 07:17 |
| *** udesale has joined #openstack-ironic | 07:19 | |
| *** udesale has quit IRC | 07:20 | |
| *** pmannidi has quit IRC | 07:21 | |
| *** udesale__ has quit IRC | 07:21 | |
| *** makowals has joined #openstack-ironic | 07:26 | |
| *** dpawar has joined #openstack-ironic | 07:31 | |
| *** vsaienk0 has quit IRC | 07:33 | |
| *** ccollett has quit IRC | 07:33 | |
| *** charcol has quit IRC | 07:34 | |
| *** threestrands has quit IRC | 07:36 | |
| openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient master: Accept port and portgroup as volume connector types https://review.openstack.org/532442 | 07:43 |
| *** vsaienk0 has joined #openstack-ironic | 07:46 | |
| openstackgerrit | fupingxie proposed openstack/ironic-python-agent master: Modify log infomation to achieve the same format https://review.openstack.org/532444 | 07:48 |
| *** harlowja has quit IRC | 07:49 | |
| *** hoonetorg has quit IRC | 07:50 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:57 | |
| *** hjensas has joined #openstack-ironic | 08:05 | |
| *** hjensas has quit IRC | 08:05 | |
| *** hjensas has joined #openstack-ironic | 08:05 | |
| *** hoonetorg has joined #openstack-ironic | 08:07 | |
| *** jtomasek has joined #openstack-ironic | 08:11 | |
| *** tesseract has joined #openstack-ironic | 08:14 | |
| *** dsneddon has joined #openstack-ironic | 08:17 | |
| *** deray has quit IRC | 08:20 | |
| *** karthiks has quit IRC | 08:20 | |
| *** alex_xu has quit IRC | 08:28 | |
| *** alex_xu has joined #openstack-ironic | 08:29 | |
| *** karthiks has joined #openstack-ironic | 08:32 | |
| *** ianychoi has quit IRC | 08:35 | |
| *** deray has joined #openstack-ironic | 08:39 | |
| *** etingof has quit IRC | 08:41 | |
| *** karthiks has quit IRC | 08:44 | |
| *** racedo has joined #openstack-ironic | 08:51 | |
| *** jistr has joined #openstack-ironic | 08:57 | |
| *** karthiks has joined #openstack-ironic | 08:59 | |
| *** ijw has joined #openstack-ironic | 09:01 | |
| *** hshiina3 has quit IRC | 09:11 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescue related methods to network interface https://review.openstack.org/509342 | 09:13 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 09:13 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Agent rescue implementation https://review.openstack.org/400437 | 09:13 |
| *** etingof has joined #openstack-ironic | 09:15 | |
| *** deray has quit IRC | 09:23 | |
| openstackgerrit | John Haan proposed openstack/ironic-specs master: Ironic currently supports several dhcp provider:neutron,none. We have only one option for deploying ironic node with standalone. We need to configure dhcp manually in this case. New DHCP Provider will be added for standalone deployment. ISCDHCPDriver conf https://review.openstack.org/532468 | 09:25 |
| openstackgerrit | John Haan proposed openstack/ironic-specs master: Support ISCDHCPDriver for Standalone Deployment https://review.openstack.org/532468 | 09:27 |
| *** lucas-afk is now known as lucasagomes | 09:31 | |
| *** e0ne has joined #openstack-ironic | 09:34 | |
| *** stendulker has quit IRC | 09:37 | |
| *** stendulker has joined #openstack-ironic | 09:37 | |
| *** rbartal has quit IRC | 09:40 | |
| *** derekh has joined #openstack-ironic | 09:42 | |
| *** stendulker has quit IRC | 09:42 | |
| *** stendulker has joined #openstack-ironic | 09:43 | |
| *** liuzz has quit IRC | 09:49 | |
| *** liuzz_ has joined #openstack-ironic | 09:49 | |
| *** deray has joined #openstack-ironic | 09:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: TESTING: DNR Revert "Run in superconductor cellsv2 mode for non-grenade jobs" https://review.openstack.org/532474 | 09:56 |
| *** dmellado has quit IRC | 10:06 | |
| *** dmellado has joined #openstack-ironic | 10:07 | |
| *** sambetts|afk is now known as sambetts | 10:13 | |
| *** threestrands has joined #openstack-ironic | 10:21 | |
| *** MattMan_1 has quit IRC | 10:23 | |
| *** MattMan_1 has joined #openstack-ironic | 10:24 | |
| *** dtantsur|afk is now known as dtantsur | 10:24 | |
| dtantsur | morning ironic | 10:24 |
| anupn | Hi Ironic'ers :) | 10:28 |
| anupn | morning dtantsur | 10:29 |
| *** phuongnh has quit IRC | 10:34 | |
| dtantsur | sambetts: morning, up for a quick review? https://review.openstack.org/#/c/531654/ | 10:40 |
| patchbot | patch 531654 - ironic-tempest-plugin - Cleaned up the requirements of Ironic Tempest plugin | 10:40 |
| sambetts | dtantsur: +A | 10:43 |
| dtantsur | thnx | 10:43 |
| pas-ha | morning ironic :) | 10:45 |
| dtantsur | hey pas-ha! how are you feeling today? | 10:46 |
| pas-ha | nice, thank you :) | 10:46 |
| *** dmellado has quit IRC | 10:49 | |
| *** namnh has quit IRC | 10:51 | |
| *** tuanla____ has quit IRC | 10:55 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Implement Redfish Sessions https://review.openstack.org/471942 | 11:00 |
| *** stendulker_ has joined #openstack-ironic | 11:00 | |
| *** stendulker has quit IRC | 11:04 | |
| *** deray has quit IRC | 11:04 | |
| *** zhangfei has quit IRC | 11:06 | |
| *** stendulker_ has quit IRC | 11:07 | |
| *** yangyapeng has quit IRC | 11:12 | |
| *** nicodemos has joined #openstack-ironic | 11:13 | |
| *** gmonteiro has joined #openstack-ironic | 11:13 | |
| gmonteiro | Good Morning Ironic :) | 11:14 |
| *** amarao has joined #openstack-ironic | 11:15 | |
| *** AlexeyAbashkin has quit IRC | 11:18 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:18 | |
| pas-ha | all tox-based jobs are failing | 11:20 |
| pas-ha | `TASK [tox : Install any sibling python packages] MODULE FAILURE` | 11:20 |
| dtantsur | >_< | 11:21 |
| dtantsur | seems to be already known to infra | 11:21 |
| *** dmellado has joined #openstack-ironic | 11:23 | |
| *** yangyapeng has joined #openstack-ironic | 11:25 | |
| *** dmellado has quit IRC | 11:32 | |
| dtantsur | sambetts: if you have some time, mind also reviewing https://review.openstack.org/#/c/528308/ ? | 11:33 |
| patchbot | patch 528308 - ironic-specs - Amend future of classic drivers with automated mig... | 11:33 |
| dtantsur | has 2x +2 now | 11:33 |
| sambetts | dtantsur: sure :) | 11:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch non-vendor parts admin guide to hardware types https://review.openstack.org/528337 | 11:34 |
| *** dmellado has joined #openstack-ironic | 11:34 | |
| gmonteiro | dtantsur: you can reviewing this please https://review.openstack.org/#/c/523943/ ? If you have time :) | 11:36 |
| patchbot | patch 523943 - ironic - Introduce hpOneView and ilorest to OneView | 11:36 |
| dtantsur | on my radar, but busy right now | 11:38 |
| dtantsur | I guess I'd prefer to request an intermediary release before we dive into this patch chain again | 11:38 |
| dtantsur | it will be today or tomorrow anyway | 11:39 |
| *** d0ugal has quit IRC | 11:39 | |
| *** milan_ has joined #openstack-ironic | 11:40 | |
| ricardoas | morning, ironic! | 11:44 |
| *** jkilpatr has quit IRC | 11:44 | |
| *** nicodemos has quit IRC | 11:46 | |
| *** d0ugal has joined #openstack-ironic | 11:57 | |
| *** nicodemos has joined #openstack-ironic | 12:03 | |
| *** absubram has quit IRC | 12:10 | |
| *** jkilpatr has joined #openstack-ironic | 12:17 | |
| *** bfournie has quit IRC | 12:18 | |
| *** bfournie has joined #openstack-ironic | 12:19 | |
| *** bfournie has quit IRC | 12:24 | |
| anupn | Hi dtantsur: I am looking into graphical console access spec, and was wondering how can vendor specific baremetal machines without default VNC server can be accessed, do you have any idea? | 12:25 |
| dtantsur | anupn: the answer will be vendor specific. if they don't have VNC, they probably won't support the graphical console | 12:26 |
| dtantsur | I'll defer to pas-ha for a more detailed answer | 12:26 |
| *** lucasagomes is now known as lucas-hungry | 12:27 | |
| anupn | dtantsur, Thanks. Yes, agree, but then the feature would be a feature very specific to those vendors that support VNC. So, was wondering can there be any way to access without VNC as well. | 12:28 |
| pas-ha | anupn: there was a next spec that proposed running proprietary vnc code in a docker container. (i don't like it that much though) | 12:28 |
| pas-ha | but basically yes, as dtantsur says. | 12:28 |
| pas-ha | the spec proposes a basic framework to add such support | 12:29 |
| anupn | pas-ha, hmm yes. I read spec - https://review.openstack.org/#/c/306078/6/specs/approved/external-display-drivers.rst | 12:29 |
| pas-ha | just start-stop-get methods, the same as for serial consoles. | 12:29 |
| patchbot | patch 306078 - ironic-specs - External display drivers | 12:29 |
| anupn | but looks like it is circled around VNC | 12:29 |
| pas-ha | sort of, use whatever proprietary stuff your vendor does provide you, pack it in a container and export its X11 as vnc.. sort of.. | 12:30 |
| pas-ha | but again, as this is pluggable, the idea is to let users benefit from newer systems that support it easier, like newer Dells https://pshchelo.github.io/vnc-in-ironic.html | 12:31 |
| anupn | pas-ha, ah i see | 12:31 |
| anupn | so vendor specific graphical console interface needs to be run inside x11 server, right? | 12:32 |
| pas-ha | that was a prototype I've done for some dell servers some time ago | 12:32 |
| pas-ha | most probably yes | 12:32 |
| *** jlabarre has joined #openstack-ironic | 12:34 | |
| anupn | pas-ha, so i just looked up, iLO drivers use virtual KVM to remotely access server, i think then i need to figure out how each vendor use it's client to access the console | 12:35 |
| anupn | pas-ha, thanks for pointing | 12:36 |
| openstackgerrit | Merged openstack/networking-generic-switch master: Prepare releasenotes https://review.openstack.org/530946 | 12:36 |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Fix persistent information when getting boot device https://review.openstack.org/524616 | 12:37 |
| anupn | pas-ha, i was just reading this https://pshchelo.github.io/vnc-in-ironic.html, great blog. But, looks like it describes about Dell hardware with built-in VNC suppprt. I understand for such hardware we just need to have VNC client present to access the display. | 12:41 |
| *** gcb has joined #openstack-ironic | 12:43 | |
| *** alineb has quit IRC | 12:54 | |
| *** makowals has quit IRC | 12:57 | |
| *** links has quit IRC | 12:58 | |
| *** bfournie has joined #openstack-ironic | 12:59 | |
| *** makowals has joined #openstack-ironic | 12:59 | |
| *** rbartal has joined #openstack-ironic | 13:05 | |
| *** rbartal has quit IRC | 13:12 | |
| jroll | good morning ironic | 13:12 |
| *** vsaienk0 has quit IRC | 13:14 | |
| *** vsaienk0 has joined #openstack-ironic | 13:14 | |
| dtantsur | morning jroll | 13:15 |
| *** dprince has joined #openstack-ironic | 13:16 | |
| *** alineb has joined #openstack-ironic | 13:17 | |
| *** lucas-hungry is now known as lucasagomes | 13:22 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type https://review.openstack.org/524729 | 13:24 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Fix persistent information when getting boot device https://review.openstack.org/524616 | 13:24 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add a timeout for powering on/off a node on oneview https://review.openstack.org/524624 | 13:24 |
| *** rbartal has joined #openstack-ironic | 13:24 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Devstack - Add ironic-neutron-agent https://review.openstack.org/530117 | 13:30 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 13:30 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 13:30 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type https://review.openstack.org/524729 | 13:31 |
| hjensas | sambetts: placement is not working. https://etherpad.openstack.org/p/ironic-neutron-placement | 13:32 |
| *** hw_wutianwei has joined #openstack-ironic | 13:35 | |
| sambetts | hjensas: can you reach out to the placement API or DB and pull a dump of all the data it has about the compute hosts? The nova ironic driver + nova should be putting all ironics nodes in available state into placement as compute hosts | 13:36 |
| hjensas | sambetts: I did'nt look into that much before, I was focusing on getting the binding and find_subnet based on segment mapping in neutron. | 13:36 |
| hjensas | sambetts: sure | 13:37 |
| * hjensas goes to read the api-ref. | 13:37 | |
| sambetts | hjensas: we know the nova adds the nodes to placement in normal operation, because scheduling relies on it now, however I'm wondering if they generate their own UUIDs instead of keeping the ironic ones, and then set the compute hosts "name" field to the ironic uuid | 13:38 |
| *** SerenaFeng has joined #openstack-ironic | 13:38 | |
| hjensas | sambetts: They do. I saw that last night when looking into it. | 13:38 |
| *** makowals has quit IRC | 13:39 | |
| *** SerenaFeng has quit IRC | 13:40 | |
| sambetts | hjensas: which may mean we need to query placement to get our placement UUID instead of just using the ironic UUID as the compute host identifier, although they use the host name in the neutron agents, which map to the placement name field, so I expected it to just work | 13:42 |
| *** makowals has joined #openstack-ironic | 13:43 | |
| *** rbudden has joined #openstack-ironic | 13:43 | |
| TheJulia | goood morning everyone | 13:44 |
| sambetts | hjensas: the other thing that could be happening is that it takes a few moments for nova + ironic to sync to placements and node's not in avaiable are removed from placement by nova, so we might need to check if the node is in placement (or check its state) and toggle the agent active state on/off based on the node avaiablity | 13:44 |
| hjensas | sambetts: then we would have to get the placement UUID in binding information in neutron.py as well. So two places to have external calls. | 13:44 |
| sambetts | hjensas: I think the UUID thing should be fine, I expect the second case I've described is more liekyl | 13:45 |
| *** trown|outtypewww is now known as trown | 13:45 | |
| hjensas | sambetts: hmm, yes I think I only saw the exception loggen when doing a deployment test. e.g ironic node in clean/provision etc states. | 13:46 |
| sambetts | hjensas: yeah, it won't be in placement at that time because of the way the nova ironic driver works, so we'll have to flag the agent off (or whatever we need to do) to make neutron ignore it | 13:47 |
| nicodemos | morning, TheJulia. | 13:49 |
| hjensas | sambetts: ok, I'll look into it. Seems you are right. As soon as I set the node to maintenance I get one error in neutron logs each time the agent does an update. | 13:51 |
| hjensas | sambetts: thanks! | 13:51 |
| *** dpawar has quit IRC | 13:51 | |
| sambetts | hjensas: thanks for working on it! | 13:55 |
| *** hw_wutianwei has quit IRC | 14:01 | |
| *** tzumainn has joined #openstack-ironic | 14:02 | |
| hjensas | sambetts: oh, spoke too soon. I also see it when the node is available. I'll look a bit more. | 14:03 |
| sambetts | hjensas: it takes some time for things to sync between ironic nova and placement | 14:03 |
| *** sasha2 has joined #openstack-ironic | 14:04 | |
| hjensas | sambetts: how much? I have it 9 times, 1 every minute. | 14:06 |
| sambetts | hjensas: I think it can take up to 2 minutes to sync | 14:06 |
| sambetts | then the node should be placment | 14:06 |
| *** hw_wutianwei has joined #openstack-ironic | 14:08 | |
| hjensas | sambetts: I see it in the placement even if state is not available. But the update, it is trying to add the node to an aggregate. T | 14:09 |
| *** rloo has joined #openstack-ironic | 14:10 | |
| hjensas | sambetts: When I look at the aggregate this has 1 host, hostname of my test node (aio devstac). But this host is not in placement resource_providers. | 14:10 |
| hjensas | sambetts: I'll dig a bit more into the nova server side to see what that that os-aggregate action add-host does. | 14:12 |
| dtantsur | does anyone remember why we have this check https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/iscsi_deploy.py#L99 for *whole-disk* images? | 14:13 |
| sambetts | hjensas: the 1 host your seeing is the neutron L2 agent (OVS) which is running on your host under the hostname of your host, the ironic nodes are separate from that | 14:15 |
| *** Goneri has joined #openstack-ironic | 14:19 | |
| hjensas | sambetts: ack. | 14:24 |
| jroll | dtantsur: interesting question, I would guess we made the assumption that root_gb == the entire disk there, which clearly isn't right but may be typical | 14:28 |
| dtantsur | jroll: yeah, but since root_gb is not used by anything, as we stop using flavor's disk size for scheduling, it makes not much sense to me | 14:29 |
| dtantsur | jroll: we also don't validate it for the direct deploy | 14:29 |
| jroll | dtantsur: I could be wrong ¯\_(ツ)_/¯ | 14:31 |
| jroll | I bet that's from the before times | 14:31 |
| dtantsur | yep, I suspect it too | 14:31 |
| TheJulia | I'm +1 to ripping it out since at a very quick glance it looks like it wouldn't be sane for qcow2 images | 14:31 |
| * dtantsur fixes | 14:31 | |
| *** trown is now known as trown|brb | 14:32 | |
| TheJulia | oh, it gets the qcow image size | 14:32 |
| jroll | https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L330 | 14:32 |
| jroll | yeah | 14:32 |
| TheJulia | when told to... | 14:33 |
| jroll | which it never is. good one. | 14:33 |
| TheJulia | \o/ | 14:33 |
| dtantsur | \o/ | 14:34 |
| jroll | and surely this has been refactored 10 times so good luck with finding the history | 14:34 |
| jroll | :) | 14:34 |
| dtantsur | we start growing enterprisy: we have weird code nobody understands :D | 14:36 |
| *** crushil has joined #openstack-ironic | 14:36 | |
| *** lbragstad has joined #openstack-ironic | 14:38 | |
| *** rloo has quit IRC | 14:40 | |
| *** rloo has joined #openstack-ironic | 14:40 | |
| * TheJulia wonders if/when ironic dresses in gothic or punk clothing | 14:47 | |
| *** trown|brb is now known as trown | 14:48 | |
| sambetts | TheJulia: oh we've already got back eyeliner one ;) | 14:48 |
| sambetts | on* | 14:48 |
| sambetts | black | 14:48 |
| sambetts | got my spelling... | 14:48 |
| sambetts | god* | 14:48 |
| sambetts | ahasuydkjhuyasdkjhads | 14:48 |
| sambetts | I give up | 14:48 |
| * TheJulia knew she should have gotten more black eyeliner yesterday | 14:48 | |
| TheJulia | Hmmmm... maybe that is what I should do.... pixie boots + 50s dress..... | 14:51 |
| * TheJulia has been pondering new profile photos | 14:51 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Do not validate root partition size for whole disk images in iscsi deploy https://review.openstack.org/532548 | 14:53 |
| dtantsur | ^^^ | 14:54 |
| *** dpawar has joined #openstack-ironic | 14:55 | |
| *** dpawar has quit IRC | 14:55 | |
| -openstackstatus- NOTICE: Gerrit is being restarted due to slowness and to apply kernel patches | 14:58 | |
| hjensas | sambetts: Looks like the nova side of things does a lookup in os-services, not in os-hypervisors where the ironic nodes can bee seen. | 14:58 |
| sambetts | hjensas: which nova side? | 14:59 |
| sambetts | the part that adds the node into placement? | 14:59 |
| sambetts | or the part that neutron calls? | 15:00 |
| hjensas | sambetts: when the os-aggregats add_host action is done. This is what is called to place it into the segments aggregate. | 15:00 |
| jroll | dtantsur: hm, I'm wondering if we should check the image against total disk size, though | 15:00 |
| sambetts | hjensas: its a placement aggregate not a nova host aggregate, I believe it calls placements directly | 15:01 |
| *** openstackgerrit has quit IRC | 15:01 | |
| hjensas | sambetts: -X POST http://192.168.122.200/compute/v2.1/os-aggregates/1/action | 15:01 |
| *** absubram has joined #openstack-ironic | 15:02 | |
| jroll | hjensas: sambetts: nova host aggregates can't handle ironic nodes yet. however, jaypipes has code in progress to do that | 15:02 |
| sambetts | jroll: for routed networks it shouldn't have too handle host aggregates | 15:03 |
| sambetts | it should only be calling and creating a placemnt aggregate which is a separate thing | 15:03 |
| jroll | sambetts: I don't have full context, just pointing out why hjensas is having problems | 15:03 |
| hjensas | sambetts: this is whats in that aggregate | 15:03 |
| jroll | ah gotcha | 15:03 |
| hjensas | { | 15:03 |
| hjensas | "aggregate": { | 15:03 |
| hjensas | "name": "Neutron segment id fca8b5c2-1d33-4e01-ae28-793bac323559", | 15:03 |
| hjensas | "availability_zone": null, | 15:03 |
| hjensas | "deleted": false, | 15:03 |
| hjensas | "created_at": "2018-01-10T11:05:43.000000", | 15:03 |
| hjensas | "updated_at": null, | 15:03 |
| hjensas | "hosts": [ | 15:03 |
| hjensas | "routed-devstack.lab.example.com" | 15:03 |
| hjensas | ], | 15:03 |
| jroll | gerrit down? | 15:03 |
| hjensas | "deleted_at": null, | 15:03 |
| sambetts | hjensas: the placement agregate | 15:03 |
| hjensas | "id": 1, | 15:03 |
| hjensas | "metadata": { | 15:03 |
| sambetts | ? | 15:03 |
| hjensas | } | 15:03 |
| hjensas | } | 15:03 |
| hjensas | } | 15:03 |
| sambetts | hjensas: please use paste.openstack.org | 15:03 |
| hjensas | sambetts: no, the os-aggregates/1 | 15:04 |
| * jroll now sees gerrit notice | 15:04 | |
| sambetts | hjensas: ... neutron isn't supposed to be creating nova host-aggregates... it supposed to be creating placement aggregates and grouping resources... thats what I was told it should be doing and thats what the specs describe it doing | 15:06 |
| *** absubram has quit IRC | 15:06 | |
| *** absubram has joined #openstack-ironic | 15:06 | |
| rloo | good morning everyone, sambetts, jroll, hjensas, dtantsur, TheJulia | 15:08 |
| dtantsur | morning rloo | 15:08 |
| hjensas | sambetts: ... great ... it does not seem to do as the spec decribe then - https://www.openstack.org/assets/presentation-media/Scaling-Up-OpenStack-Networking-with-Routed-Networks.pdf <- slide 20. | 15:09 |
| jroll | mornign rloo \o | 15:09 |
| sambetts | hjensas: I'm just looking through the code in neutron now | 15:10 |
| sambetts | hjensas: https://github.com/openstack/neutron/blob/d505e3eeb7e4d3ee211499286fd050ed48fdbf96/neutron/services/segments/plugin.py#L216 | 15:10 |
| sambetts | hjensas: this is the code thats making those calls | 15:10 |
| *** dpawar has joined #openstack-ironic | 15:11 | |
| TheJulia | good morning rloo | 15:12 |
| hjensas | sambetts: yes, 223, 224 ends up in nova running https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/aggregates.py#L141-L156 | 15:13 |
| hjensas | sambetts: and in the end https://github.com/openstack/nova/blob/master/nova/db/api.py#L170-L175 is called to get service entry. afaict ... | 15:14 |
| *** ianychoi has joined #openstack-ironic | 15:14 | |
| jroll | creating host aggregates via nova does create placement aggregates, right? just the problem is that doesn't really work for nova? | 15:14 |
| sambetts | hjensas: I believe they taking advantage of the fact that the nova host aggregates and placement aggregates currently get synced together, so creating a host aggregate in nova results in an aggregate appearing in placement | 15:14 |
| jroll | er, doesn't work for ironic* | 15:14 |
| jroll | sambetts: +1 | 15:14 |
| johnthetubaguy | rloo: not got through all your replies on your spec, but I think I am starting to lean towards where you specify the priority in the API call, like you say, needs more thought | 15:15 |
| sambetts | jroll, hjensas: IMO neutron should be going directly to placemnt to create these aggregates | 15:15 |
| hjensas | sambetts: ok, so neutron code might need patching then. | 15:15 |
| jroll | sambetts: I don't think the scheduler used placement for aggregates until this cycle, that's likely why | 15:16 |
| rloo | johnthetubaguy: thx. it is on my list to think about a way not to have priority in api call. i need to 'gear up' to looking at all that stuff again, hopefully later today :) | 15:16 |
| hjensas | sambetts: ok, I will check with the neutron guys later. Need to go now. Thanks for the help! | 15:16 |
| jroll | sambetts: in which case we probably don't want to move to placement directly until rocky, so we don't need a lockstep upgrade | 15:16 |
| jroll | sambetts: this work fixes it from the opposite direction, btw: https://review.openstack.org/#/c/526753/ https://review.openstack.org/#/c/529135/ | 15:17 |
| patchbot | patch 526753 - nova - allow compute nodes to be associated with host agg | 15:17 |
| patchbot | patch 529135 - nova-specs - Support aggregate affinity filters/weighers | 15:17 |
| sambetts | jroll: the aggregte here isn't actually being used for schduling (other that checking if there are no IPs left in that segment) its just used claim an allocation on an IP address associated to a segment | 15:17 |
| sambetts | jroll: having these as host aggregates in nova actually makes little sense | 15:18 |
| jroll | sambetts: ah | 15:18 |
| sambetts | because the resource its actually aggregated against has no data model in nova | 15:18 |
| * jroll wonders how old of nova neutron attempts to be compatible with | 15:18 | |
| sambetts | this is the use for placement like the shared storage pool idea | 15:18 |
| jroll | right | 15:18 |
| *** rbartal has quit IRC | 15:20 | |
| *** hjensas has quit IRC | 15:22 | |
| johnthetubaguy | jroll: I was musing some on host aggregates too, but placement does make lots of that useless, been looking bits at the SKA scheduling | 15:23 |
| jroll | johnthetubaguy: SKA? | 15:24 |
| johnthetubaguy | jroll: https://www.skatelescope.org | 15:24 |
| jroll | ah | 15:25 |
| *** jaypipes has joined #openstack-ironic | 15:25 | |
| johnthetubaguy | jroll: my worry is with ironic we get too many candidates back from placement, stuff like traits helps reduce that down early | 15:26 |
| johnthetubaguy | aggregates should too I guess | 15:26 |
| johnthetubaguy | I did wonder about generic segmentation between HA ironic nodes, so you shard between compute node groups, and that maps to aggregates, etc | 15:29 |
| *** hw_wutianwei has quit IRC | 15:29 | |
| jroll | yeah, I'd like to do some work on that, same for conductors | 15:29 |
| sambetts | jroll, johnthetubaguy: https://bugs.launchpad.net/ironic/+bug/1739426 most complete thoughts on conductor placement stuff | 15:30 |
| openstack | Launchpad bug 1739426 in Ironic "[RFE] ironic conductor - node affinity placement" [Wishlist,New] - Assigned to Sam Betts (sambetts) | 15:30 |
| jroll | oh I'm well aware of this :P | 15:31 |
| sambetts | I've avoided specific implementation details in it, because I think maybe, *maybe* ironic using placements directly for condcutors/nodes with the information I've described might be right way to go (but I havn't played with it yet) | 15:33 |
| johnthetubaguy | sambetts: nova's affinity is still busted I think? | 15:33 |
| johnthetubaguy | well, works badly rather than non-functional I believe | 15:33 |
| jroll | sambetts: yeah, I'd love to make computes/placement/conductors/aggregates all kinda work together there | 15:34 |
| johnthetubaguy | +1 | 15:34 |
| sambetts | johnthetubaguy: yes, my RFE is directed at making sure that ironic nodes land on the conductor that is most relevent to it (closest etc) | 15:34 |
| sambetts | johnthetubaguy: rather than instance affinity | 15:34 |
| johnthetubaguy | SKA has a "compute islands" concept, which is kinda related I suppose | 15:34 |
| johnthetubaguy | sambetts: ah, got it | 15:34 |
| johnthetubaguy | above probably drops out as AZs | 15:35 |
| jroll | johnthetubaguy: this fixes ironic affinity, btw https://review.openstack.org/#/c/529135/ | 15:35 |
| patchbot | patch 529135 - nova-specs - Support aggregate affinity filters/weighers | 15:36 |
| jroll | or aims to | 15:36 |
| johnthetubaguy | ah, I should catch up with those soon | 15:36 |
| sambetts | jroll, johnthetubaguy: we should add a PTG topic on this maybe or at least have a whiteboard session at some point during, (if your both coming of course) | 15:37 |
| jroll | I'll be there | 15:37 |
| jroll | and agree | 15:37 |
| johnthetubaguy | unsure about me yet, but its feels too close not to go, and +1 | 15:38 |
| sambetts | johnthetubaguy: definatly too close for us not to go ;) | 15:38 |
| *** jkilpatr has quit IRC | 15:39 | |
| * dtantsur is jealous to people who don't need visas.. | 15:48 | |
| *** jkilpatr has joined #openstack-ironic | 15:55 | |
| * johnthetubaguy is annoyed it may be his last trip to Ireland without needing a visa | 15:56 | |
| rloo | johnthetubaguy: OH, brexit means you'd need a visa to Europe? | 16:03 |
| johnthetubaguy | no idea yet, but seems likely | 16:03 |
| sambetts | rloo: seems like we'll have something like the US ESTA program at the moment | 16:03 |
| dtantsur | I don't think it's likely.. I still suspect people with 1st world passport will be able to move around.. | 16:03 |
| dtantsur | oh ESTA-like yes | 16:03 |
| rloo | TheJulia, jroll, mgoddard: I've got one question wrt https://review.openstack.org/#/c/528238/. If we could agree... | 16:03 |
| patchbot | patch 528238 - ironic - Node traits: Add DB API & model | 16:03 |
| rloo | sambetts, dtantsur: well, that seems like a bit of a pain, but I guess the pros outweigh the cons for some people :) | 16:04 |
| johnthetubaguy | yeah, ESTA like I would expect | 16:05 |
| *** rbartal has joined #openstack-ironic | 16:06 | |
| dtantsur | heh yeah | 16:07 |
| dtantsur | ESTA is cool, as far as I know | 16:07 |
| sambetts | the US ESTA its cool for limited travel, but you only get 90 days over 2 years and I came dangerously close to maxing that out a couple of year ago | 16:09 |
| sambetts | when I was attending os summits and ptgs and other meetings in the US | 16:10 |
| jroll | rloo: good question, replied | 16:11 |
| *** mjura has quit IRC | 16:14 | |
| *** jkilpatr has quit IRC | 16:16 | |
| rloo | jroll: good try. see my reply :) | 16:19 |
| jroll | rloo: ok, fair. it needs to get implemented then :P | 16:19 |
| jroll | rloo: at any rate, I think any api requests are going to fail before we get to that exception, I'm fine either way | 16:20 |
| rpioso | Morning, all | 16:21 |
| *** openstackgerrit has joined #openstack-ironic | 16:22 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type https://review.openstack.org/524729 | 16:22 |
| TheJulia | rloo: I'm all for tweaking later once we have an api layer | 16:23 |
| rloo | jroll: i just read that api-wg spec; it sez if false the http status is 404, which is what it will/should be with or w/o the exception. so either way is fine, we just need to decide/be consistent. although i think for methods that are meant to return bool, maybe we should just stick to that. not sure. | 16:24 |
| jroll | rloo: yeah, left another comment, tl;dr I'm not terribly opinionated here | 16:24 |
| dtantsur | sambetts: and when it runs out you need a visa? | 16:25 |
| *** vsaienk0 has quit IRC | 16:25 | |
| sambetts | dtantsur: yeah | 16:25 |
| rloo | jroll: me neither. TheJulia, you good with jroll's suggestion? change node_tag_exists to also raise exception? | 16:25 |
| sambetts | dtantsur: until your 2 years is up, then you can get a new ESTA | 16:25 |
| sambetts | as I understand it anyway | 16:25 |
| dtantsur | sambetts: I see. well, 90 days is better than nothing :) | 16:25 |
| rloo | TheJulia: i just want to make sure we address the inconsistency now, vs 'tweaking' later (if we even remember about tweaking it) | 16:25 |
| rloo | TheJulia: never mind, you +2'd it so I'll assume you're ok with the exception :) | 16:26 |
| *** vsaienk0 has joined #openstack-ironic | 16:27 | |
| TheJulia | we can change it in another patch, the beauty of patches :) | 16:28 |
| rloo | TheJulia: yes! | 16:29 |
| *** jkilpatr has joined #openstack-ironic | 16:29 | |
| edleafe | rloo: the exception should be raised if the resource (node) doesn't exist. If the trait doesn't exist on the node, returning False is fine. | 16:29 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove python-oneviewclient from oneview hardware type https://review.openstack.org/524729 | 16:29 |
| *** dpawar has quit IRC | 16:30 | |
| rloo | edleafe: thx for your feedback! | 16:30 |
| *** dpawar has joined #openstack-ironic | 16:31 | |
| *** rbartal has quit IRC | 16:31 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: WIP: API: Node Traits API https://review.openstack.org/532269 | 16:31 |
| jroll | johnthetubaguy: ^ made that work so I can hack on client things | 16:31 |
| jroll | still probably not perfect but at least it runs and the APIs do things | 16:32 |
| * jroll lunches | 16:32 | |
| johnthetubaguy | jroll: awesome, still hacking on the underneath one | 16:32 |
| jroll | sounds good :) | 16:32 |
| mgoddard | rloo, jroll: thanks for coming to an agreement on node_trait_exists | 16:34 |
| *** amarao has quit IRC | 16:35 | |
| *** dpawar has quit IRC | 16:35 | |
| rloo | mgoddard: :) hopefully the two trait patches land today. hopefully the gate is open today :D | 16:36 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Fix persistent information when getting boot device https://review.openstack.org/524616 | 16:39 |
| johnthetubaguy | I am really struggling with the release_mappings test, anyone got any pointers | 16:42 |
| johnthetubaguy | new object version seems to be changing an old release, unless I am reading things incorrectly? | 16:42 |
| johnthetubaguy | so I think I found it, doh! | 16:44 |
| *** d0ugal has quit IRC | 16:46 | |
| *** zshi has quit IRC | 16:47 | |
| rloo | johnthetubaguy: just saw your ask, glad you figured it out :) | 16:47 |
| *** racedo has quit IRC | 16:48 | |
| rloo | johnthetubaguy: don't know if this helps or not, but ironic deals with objects a bit diff from nova, there's some doc here: https://docs.openstack.org/ironic/latest/contributor/rolling-upgrades.html#when-developing-a-new-feature-or-modifying-an-ironicobject | 16:49 |
| johnthetubaguy | rloo: thanks for the pointer, I was just seeing your comments on there | 16:50 |
| *** zshi has joined #openstack-ironic | 16:50 | |
| *** AlexeyAbashkin has quit IRC | 16:55 | |
| *** marios has quit IRC | 16:56 | |
| *** e0ne has quit IRC | 17:05 | |
| *** vsaienk0 has quit IRC | 17:07 | |
| *** vsaienk0 has joined #openstack-ironic | 17:09 | |
| *** pcaruana has quit IRC | 17:11 | |
| *** harlowja has joined #openstack-ironic | 17:16 | |
| rloo | TheJulia, sambetts, dtantsur: do we want to discuss https://review.openstack.org/#/c/528308/ or are folks good with removing the check in rocky? | 17:17 |
| patchbot | patch 528308 - ironic-specs - Amend future of classic drivers with automated mig... | 17:17 |
| *** gmonteiro has quit IRC | 17:18 | |
| rloo | (I mean after Rocky) | 17:19 |
| *** lucasagomes is now known as lucas-afk | 17:19 | |
| openstackgerrit | John Garbutt proposed openstack/ironic master: Add RPC object for traits https://review.openstack.org/532268 | 17:19 |
| openstackgerrit | John Garbutt proposed openstack/ironic master: WIP: API: Node Traits API https://review.openstack.org/532269 | 17:19 |
| jlvillal | TheJulia: rloo vdrok sambetts dtantsur vsaienk0 mgoddard: FYI: I have removed the -2 from: https://review.openstack.org/#/c/527733/ | 17:20 |
| patchbot | patch 527733 - ironic - Remove ironic_tempest_plugin/ directory | 17:20 |
| *** hjensas has joined #openstack-ironic | 17:20 | |
| rloo | jlvillal: thx for reminding. i remembered earlier and went to whiteboard to find link but it wasn't in the priorities and then i.got.distracted. :) | 17:23 |
| *** vsaienk0 has quit IRC | 17:23 | |
| jlvillal | :) | 17:23 |
| *** bfournie has quit IRC | 17:26 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Fix Py2/Py3 differences in write locking code https://review.openstack.org/532579 | 17:31 |
| *** baha has joined #openstack-ironic | 17:32 | |
| rloo | jlvillal: wrt that PLUGIN_REMOVED file -- why not delete the entire directory? | 17:33 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 17:33 |
| *** vsaienk0 has joined #openstack-ironic | 17:34 | |
| jlvillal | rloo: I figured if someone went looking they would find the README. Basically trying to be helpful. | 17:35 |
| *** jaypipes has quit IRC | 17:35 | |
| jlvillal | rloo: At some point in the future I could see removing it entirely | 17:35 |
| rloo | jlvillal: no one should be looking for that, given the email that has gone out etc (and the final email you'll send out). | 17:36 |
| rloo | jlvillal: just extra stuff to remember to delete in the future. | 17:36 |
| jlvillal | rloo: Well I imagine a lot of our consumers don't read the email list. | 17:36 |
| jlvillal | rloo: Not sure what others think about it though. | 17:37 |
| rloo | jlvillal: it is in the release notes AND in the commit msg. and they can ask on irc. and they can look on mailing list after the fact :) | 17:38 |
| rloo | jlvillal: i don't recall adding that sort of stuff when we've deleted other directories in the past. | 17:38 |
| rloo | jlvillal: really, just want to minimize amount of work for us in the future. we're not good about cleaning things up after the fact. | 17:39 |
| jlvillal | Okay. | 17:39 |
| rloo | jlvillal: if anything, a TODO could be grepped for. but i say just delete it. | 17:39 |
| jlvillal | rloo: Okay. I can do a follow-up patch to delete it. | 17:39 |
| rloo | jlvillal: thx. and update that doc too :) | 17:40 |
| jlvillal | rloo: Which doc? | 17:40 |
| jlvillal | ??? | 17:40 |
| rloo | jlvillal: i linked it in my comment | 17:40 |
| jlvillal | Ah, okay | 17:40 |
| *** penick has joined #openstack-ironic | 17:43 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove ironic_tempest_plugin/ directory https://review.openstack.org/532585 | 17:44 |
| jlvillal | rloo: ^^^ But no doc change. I need to figure out the doc thing as I am not 100% sure how to do it yet. Let me research. | 17:44 |
| rloo | jlvillal: okey dokey | 17:45 |
| dtantsur | rloo: (sorry, was out) I can discuss, but I don't really care | 17:45 |
| rloo | dtantsur: i just want to +A that thing :) | 17:45 |
| dtantsur | I support you in your desire :D | 17:45 |
| rloo | dtantsur: ha ha. consensus! | 17:46 |
| rloo | dtantsur: could reword it to something like, we may deleted that check after rocky if it isn't needed? i hate punting on things but ... | 17:46 |
| dtantsur | rloo: there is actually no point to keeping it after rocky, as there is no way to get rocky running with old drivers | 17:47 |
| dtantsur | but it won't hurt either, so I'm like whatever :) | 17:47 |
| jlvillal | dtantsur: Do you know off the top of your head how to fix this? https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#running-tempest-tests | 17:48 |
| jlvillal | Do we just checkout openstack/ironic-tempest-plugin/ and we are golden? | 17:48 |
| dtantsur | jlvillal: I think it should work, we just need to tell people to pip install ironic-tempest-plugin | 17:48 |
| jlvillal | dtantsur: Ah, pip install. okay | 17:48 |
| dtantsur | jlvillal: or maybe we should update our devstack plugin to install it? | 17:49 |
| dtantsur | then no changes will be needed there | 17:49 |
| jlvillal | dtantsur: Hmmm. That sounds like it could work. | 17:50 |
| *** milan_ has quit IRC | 17:50 | |
| jlvillal | dtantsur: By install it, install it from git I assume. | 17:50 |
| dtantsur | jlvillal: yes, whatever function devstack has for it | 17:51 |
| jlvillal | Ah, I see like virtualpdu does it. If we use it via git, do it via git, otherwise use pip | 17:52 |
| rloo | dtantsur: dang, you beat me to commenting in that spec! | 17:52 |
| dtantsur | jlvillal: since it's not a lib, we can only leave the git way | 17:52 |
| dtantsur | rloo: hehe :) but you're right, our online upgrade procedure does not allow skipping releases entirely | 17:52 |
| jlvillal | dtantsur: Okay, slightly confused. Before you said 'pip' but no you are saying we can't use pip??? | 17:53 |
| jlvillal | FYI: virtualpdu way: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L648-L654 | 17:53 |
| rloo | dtantsur: ok, i'm going to +A... We can update it later if we need to. | 17:53 |
| jlvillal | s/no you are/now you are/ | 17:53 |
| dtantsur | jlvillal: pip install is for end users. in devstack we only use pip for libraries, everything else always uses git. | 17:53 |
| dtantsur | jlvillal: actually, we should talk to QA folks before deciding on anything like that | 17:54 |
| dtantsur | they may have some opinions | 17:54 |
| jlvillal | dtantsur: Okay. Let me look investigate more | 17:54 |
| * jlvillal wonders if mtreinish reads our channel :) | 17:54 | |
| TheJulia | jlvillal: it may also depend on where in the world he is at present | 17:56 |
| jlvillal | TheJulia: Tristan da Cunha? | 17:56 |
| jlvillal | My guess is no ;) | 17:56 |
| * TheJulia feels confused | 17:56 | |
| jlvillal | TheJulia: One of the world's most remote islands in the world | 17:57 |
| jlvillal | TheJulia: I think you have to take a boat from Africa for about 10 days to get to the island. | 17:57 |
| TheJulia | nice | 17:58 |
| * jlvillal would get definite island fever living there | 17:58 | |
| *** derekh has quit IRC | 18:00 | |
| *** bfournie has joined #openstack-ironic | 18:08 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor deploy interface methods https://review.openstack.org/524433 | 18:08 |
| *** pc_m has quit IRC | 18:10 | |
| *** karthiks has quit IRC | 18:12 | |
| *** pc_m has joined #openstack-ironic | 18:12 | |
| *** trown is now known as trown|lunch | 18:17 | |
| openstackgerrit | Merged openstack/ironic-specs master: Amend future of classic drivers with automated migration https://review.openstack.org/528308 | 18:18 |
| *** jkilpatr has quit IRC | 18:19 | |
| *** jkilpatr has joined #openstack-ironic | 18:20 | |
| openstackgerrit | John Garbutt proposed openstack/ironic master: Add RPC object for traits https://review.openstack.org/532268 | 18:25 |
| mtreinish | jlvillal: what's up? | 18:25 |
| mtreinish | TheJulia: still home, start traveling again next week | 18:26 |
| *** openstack has joined #openstack-ironic | 18:31 | |
| *** ChanServ sets mode: +o openstack | 18:31 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:31 | |
| mtreinish | jroll: does the ironic devstack plugin add the ironic tempest plugin to TEMPEST_PLUGINS? | 18:34 |
| jroll | mtreinish: oh, maybe | 18:34 |
| jroll | idk | 18:34 |
| jroll | the problem they want to solve is fixing up this doc now that it's moved: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#running-tempest-tests | 18:35 |
| *** AlexeyAbashkin has quit IRC | 18:35 | |
| mtreinish | I mean the way I think would be best is have the ironic devstack plugin clone the ironic tempest plugin and then add that local path to TEMPEST_PLUGINS | 18:35 |
| jroll | makes sense, then let the tempest devstack plugin decide how to install it | 18:36 |
| *** vsaienk0 has joined #openstack-ironic | 18:36 | |
| *** tesseract has quit IRC | 18:36 | |
| *** harlowja has quit IRC | 18:37 | |
| mtreinish | then that guide would just change tox -eall-plugins to tox -eall | 18:38 |
| TheJulia | Awesome noise to hear: Removing earphones from being on for about 40 minutes and hear rain... where it has not rained since August. | 18:39 |
| *** ijw has quit IRC | 18:39 | |
| jroll | \o/ | 18:40 |
| jroll | mtreinish: thanks, sounds good to me. jlvillal ^^ | 18:40 |
| *** jkilpatr has joined #openstack-ironic | 18:41 | |
| jlvillal | jroll: mtreinish Thanks. I'll work on that today. | 18:41 |
| jroll | jlvillal: bonus points for being able to remove 15 lines of zuul config that set TEMPEST_PLUGINS :P | 18:42 |
| *** vsaienk0 has quit IRC | 18:46 | |
| *** ijw has joined #openstack-ironic | 18:46 | |
| *** e0ne has joined #openstack-ironic | 18:48 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Add a timeout for powering on/off a node on oneview https://review.openstack.org/524624 | 18:50 |
| *** vsaienk0 has joined #openstack-ironic | 18:56 | |
| *** sambetts is now known as sambetts|afk | 19:10 | |
| sambetts|afk | Night all o/ | 19:10 |
| *** harlowja has joined #openstack-ironic | 19:13 | |
| *** ijw has quit IRC | 19:15 | |
| *** harlowja_ has joined #openstack-ironic | 19:16 | |
| *** harlowja has quit IRC | 19:19 | |
| *** vsaienk0 has quit IRC | 19:24 | |
| dtantsur | night sambetts|afk | 19:24 |
| *** vsaienk0 has joined #openstack-ironic | 19:27 | |
| leifmadsen_ | I'm playing around with bifrost again, and wondering if there is a "simple" way to generate an ansible inventory file that contains a list of the machines that were instantiated and installed by bifrost? | 19:27 |
| leifmadsen_ | for example, I'm using bifrost to spin up 5 physical machines that have IPMI. Once those are completed, I want to run some Ansible _against_ those machines, but it'd be nice not to have to manually build an inventory file | 19:28 |
| *** rmcall has joined #openstack-ironic | 19:30 | |
| *** crushil has quit IRC | 19:31 | |
| jroll | either I'm missing something or some of our client methods are broken... like set boot device. has anyone noticed that? | 19:32 |
| rloo | jroll: i hope you are missing something. what is broken? | 19:32 |
| jroll | one example: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/v1/node.py#L538 | 19:33 |
| jroll | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/v1/node.py#L334 | 19:33 |
| jroll | https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/base.py#L182 | 19:34 |
| jroll | oh no it's not, I'm sorry | 19:34 |
| jroll | these argument names are just really confusing | 19:34 |
| rloo | jroll: ++ | 19:35 |
| rloo | jroll: and we haven't taken the time to clean them up :-( just, you know, when/if we do, someone might say we're bike shedding! :D | 19:35 |
| *** trown|lunch is now known as trown | 19:35 | |
| jroll | resource_id=<uuid>/management/boot_device, that's quite the "resource id" | 19:35 |
| * jroll slogs on | 19:35 | |
| * rloo was hoping jroll would update/clean it up | 19:36 | |
| jroll | heh | 19:37 |
| jroll | I might | 19:37 |
| rloo | jroll: i was going to say not to bother, but i see that's the common code we'll still need after we delete ironic CLI | 19:37 |
| jroll | rloo: yeah, I may actually do it, when I have some time | 19:38 |
| rloo | jroll: thx :) | 19:39 |
| TheJulia | leifmadsen_: the whole idea was to continue to use the same data-source, there is also an env var that can be set with bifrost's inventory file that polls the list of machines out of ironic. | 19:40 |
| leifmadsen_ | TheJulia: yea, I figured there might be a way to do it, I just wasn't sure... what env var? | 19:41 |
| leifmadsen_ | I mostly just need an inventory that I can pass to my own ansible with the list of machines | 19:42 |
| TheJulia | leifmadsen_: looking it up now | 19:42 |
| leifmadsen_ | although I suspect it's going to require tweaking somewhere to get the machines to line up with my group names | 19:42 |
| TheJulia | leifmadsen_: see https://github.com/openstack/bifrost/blob/master/playbooks/example-deploy-all-available-nodes.yaml | 19:43 |
| *** penick has quit IRC | 19:43 | |
| leifmadsen_ | ok cool thanks | 19:43 |
| TheJulia | leifmadsen_: no problem :) | 19:43 |
| TheJulia | leifmadsen_: let me know how it goes | 19:43 |
| *** rmcall has quit IRC | 19:43 | |
| leifmadsen_ | will do! | 19:43 |
| leifmadsen_ | oh snap, neato | 19:45 |
| leifmadsen_ | TheJulia: is there a way to somehow set metadata prior to the baremetal deploy to have the machines pulled from ironic be assigned to some sort of group? | 19:45 |
| leifmadsen_ | for example, everything is in "baremetal" group | 19:46 |
| leifmadsen_ | I'd assume you'd have to add the metadata to ironic somehow | 19:46 |
| *** etingof has quit IRC | 19:46 | |
| *** rmcall has joined #openstack-ironic | 19:46 | |
| *** jtomasek has quit IRC | 19:49 | |
| TheJulia | leifmadsen_: that would have to be done in the inventory processing script | 19:49 |
| TheJulia | what ansible actually calls to retrieve the inventory | 19:49 |
| TheJulia | so... really... anything is possible with some code :) | 19:49 |
| leifmadsen_ | right right :) | 19:49 |
| leifmadsen_ | I just wasn't sure if there was something already I could do in a group_vars/ or something to make this all magic like :) | 19:50 |
| leifmadsen_ | this gets me pretty close though, thanks! | 19:50 |
| leifmadsen_ | at least I can steal/tweak the bifrost_inventory.py and make it do what I need | 19:50 |
| leifmadsen_ | if I figure out something not too hacky, I'll make a PR | 19:50 |
| TheJulia | leifmadsen_: there might be some other magic someplace that I'm not aware of | 19:50 |
| leifmadsen_ | yea, all good, I'll look around | 19:50 |
| leifmadsen_ | there might be some stuff I can look at in the ironic roles | 19:51 |
| TheJulia | leifmadsen_: yeah, if you find something useful, feel free to submit a change | 19:51 |
| leifmadsen_ | at the very least this gives me something I can do to instantiate the virthost part | 19:51 |
| leifmadsen_ | ++ | 19:51 |
| TheJulia | \o/ | 19:51 |
| *** dtantsur is now known as dtantsur|afk | 19:53 | |
| dtantsur|afk | o/ | 19:53 |
| *** vsaienk0 has quit IRC | 19:54 | |
| *** vsaienk0 has joined #openstack-ironic | 19:54 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient master: Ignore .eggs from git https://review.openstack.org/532621 | 19:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient master: WIP: traits in client https://review.openstack.org/532622 | 19:57 |
| jroll | that first one is super easy review stats :P | 19:58 |
| * rloo leaves the first one for others then | 19:59 | |
| *** rmcall has quit IRC | 20:25 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Cleaned up the requirements of Ironic Tempest plugin https://review.openstack.org/531654 | 20:33 |
| *** jtomasek has joined #openstack-ironic | 20:43 | |
| *** dprince has quit IRC | 20:55 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 21:00 | |
| *** AlexeyAbashkin has quit IRC | 21:04 | |
| *** e0ne has quit IRC | 21:14 | |
| NobodyCam | Good AfterNoon Ironic'ers | 21:23 |
| 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 | 21:32 |
| *** vsaienk0 has quit IRC | 21:34 | |
| *** vsaienk0 has joined #openstack-ironic | 21:36 | |
| *** trown is now known as trown|outtypewww | 21:38 | |
| *** pcaruana has joined #openstack-ironic | 21:42 | |
| *** pcaruana has quit IRC | 21:43 | |
| *** dr_gogeta86 has quit IRC | 21:47 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add additional context to contribution guide https://review.openstack.org/529681 | 21:47 |
| *** jtomasek has quit IRC | 21:51 | |
| *** threestrands_ has joined #openstack-ironic | 21:52 | |
| *** threestrands_ has quit IRC | 21:53 | |
| *** threestrands_ has joined #openstack-ironic | 21:53 | |
| *** threestrands_ has quit IRC | 21:53 | |
| *** threestrands_ has joined #openstack-ironic | 21:53 | |
| *** threestrands has quit IRC | 21:54 | |
| *** masber has joined #openstack-ironic | 22:02 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector master: Fix Py2/Py3 differences in write locking code https://review.openstack.org/532579 | 22:04 |
| *** ijw has joined #openstack-ironic | 22:06 | |
| *** vsaienk0 has quit IRC | 22:08 | |
| *** rcernin has joined #openstack-ironic | 22:11 | |
| *** ijw has quit IRC | 22:12 | |
| *** baha has quit IRC | 22:19 | |
| -openstackstatus- NOTICE: The zuul system is being restarted to apply security updates and will be offline for several minutes. It will be restarted and changes re-equeued; changes approved during the downtime will need to be rechecked or re-approved. | 22:23 | |
| *** jkilpatr has quit IRC | 22:32 | |
| *** Goneri has quit IRC | 22:34 | |
| *** vsaienk0 has joined #openstack-ironic | 22:36 | |
| *** threestrands_ has quit IRC | 22:41 | |
| *** bfournie has quit IRC | 22:44 | |
| *** bfournie has joined #openstack-ironic | 22:45 | |
| *** vsaienk0 has quit IRC | 22:46 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Abstract away the virtualization management interface https://review.openstack.org/523321 | 22:47 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added https://review.openstack.org/523879 | 22:47 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: added system boot mode support https://review.openstack.org/526101 | 22:48 |
| *** bfournie has quit IRC | 22:49 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: documentation on simulators added https://review.openstack.org/523445 | 22:50 |
| *** threestrands has joined #openstack-ironic | 22:55 | |
| *** threestrands has quit IRC | 22:55 | |
| *** threestrands has joined #openstack-ironic | 22:55 | |
| *** racedo has joined #openstack-ironic | 22:59 | |
| *** charcol has joined #openstack-ironic | 23:02 | |
| *** ijw has joined #openstack-ironic | 23:06 | |
| *** rmcall has joined #openstack-ironic | 23:09 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-lib master: Add wait_for_disk to destroy_disk_metadata function https://review.openstack.org/529325 | 23:10 |
| *** threestrands has quit IRC | 23:11 | |
| *** ijw has quit IRC | 23:12 | |
| *** racedo has quit IRC | 23:14 | |
| *** rama_y has joined #openstack-ironic | 23:15 | |
| *** threestrands has joined #openstack-ironic | 23:20 | |
| *** ijw has joined #openstack-ironic | 23:20 | |
| *** ijw has quit IRC | 23:25 | |
| *** bfournie has joined #openstack-ironic | 23:46 | |
| *** hjensas has quit IRC | 23:52 | |
| *** hjensas has joined #openstack-ironic | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!