| *** r-daneel has quit IRC | 00:00 | |
| *** pmannidi has quit IRC | 00:06 | |
| *** pmannidi has joined #openstack-ironic | 00:22 | |
| *** cjloader has quit IRC | 00:33 | |
| *** hshiina|afk is now known as hshiina | 00:33 | |
| *** cjloader has joined #openstack-ironic | 00:47 | |
| *** namnh has joined #openstack-ironic | 00:51 | |
| *** cjloader has quit IRC | 00:59 | |
| *** tiendc has joined #openstack-ironic | 01:09 | |
| *** rloo has quit IRC | 01:09 | |
| *** sapd has joined #openstack-ironic | 01:35 | |
| *** phuongnh has joined #openstack-ironic | 01:38 | |
| *** yolanda__ has joined #openstack-ironic | 01:40 | |
| *** cjloader_ has joined #openstack-ironic | 01:42 | |
| *** yolanda_ has quit IRC | 01:44 | |
| *** rajinir has quit IRC | 01:48 | |
| *** rpioso|afk_ has joined #openstack-ironic | 02:08 | |
| *** early` has joined #openstack-ironic | 02:09 | |
| *** beekneemech has joined #openstack-ironic | 02:10 | |
| *** csmart1 has joined #openstack-ironic | 02:12 | |
| *** taseer1 has joined #openstack-ironic | 02:12 | |
| *** early has quit IRC | 02:12 | |
| *** rpioso|afk has quit IRC | 02:12 | |
| *** csmart has quit IRC | 02:12 | |
| *** mordred has quit IRC | 02:12 | |
| *** d0ugal has quit IRC | 02:12 | |
| *** Taseer has quit IRC | 02:12 | |
| *** bnemec has quit IRC | 02:12 | |
| *** oanson has quit IRC | 02:12 | |
| *** ianw_pto has quit IRC | 02:12 | |
| *** rpioso|afk_ is now known as rpioso|afk | 02:12 | |
| *** ianw has joined #openstack-ironic | 02:13 | |
| *** oanson has joined #openstack-ironic | 02:13 | |
| *** d0ugal has joined #openstack-ironic | 02:13 | |
| *** mordred has joined #openstack-ironic | 02:14 | |
| *** mkrai has quit IRC | 02:16 | |
| *** retr0h has quit IRC | 02:16 | |
| *** csmart1 has quit IRC | 02:17 | |
| *** jovial[m] has quit IRC | 02:17 | |
| *** openstackgerrit has joined #openstack-ironic | 02:20 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 02:20 |
|---|---|---|
| *** cjloader_ has quit IRC | 02:24 | |
| *** cjloader_ has joined #openstack-ironic | 02:43 | |
| *** fragatina has quit IRC | 02:50 | |
| *** fragatina has joined #openstack-ironic | 02:50 | |
| *** fragatina has quit IRC | 02:51 | |
| *** fragatina has joined #openstack-ironic | 02:51 | |
| *** akhilaki has joined #openstack-ironic | 02:53 | |
| *** fragatina has quit IRC | 02:56 | |
| *** cjloader_ has quit IRC | 03:00 | |
| *** trungnv has joined #openstack-ironic | 03:00 | |
| *** trungnv_ has joined #openstack-ironic | 03:01 | |
| *** trungnv_ has quit IRC | 03:05 | |
| *** namnh_ has joined #openstack-ironic | 03:06 | |
| *** namnh has quit IRC | 03:08 | |
| *** akhilaki has quit IRC | 03:17 | |
| *** fragatina has joined #openstack-ironic | 03:21 | |
| *** liuzz_ has joined #openstack-ironic | 03:29 | |
| *** liuzz has quit IRC | 03:30 | |
| *** srart has quit IRC | 03:35 | |
| *** liuzz_ has quit IRC | 03:36 | |
| *** liuzz has joined #openstack-ironic | 03:38 | |
| *** srart has joined #openstack-ironic | 03:54 | |
| *** zhangfei has joined #openstack-ironic | 03:55 | |
| *** namnh has joined #openstack-ironic | 04:23 | |
| *** namnh_ has quit IRC | 04:23 | |
| *** taseer1 is now known as Taseer | 04:36 | |
| *** jaganathan has quit IRC | 04:49 | |
| *** links has joined #openstack-ironic | 04:53 | |
| *** jaganathan has joined #openstack-ironic | 05:01 | |
| *** e0ne has joined #openstack-ironic | 05:16 | |
| *** jaganathan has quit IRC | 05:24 | |
| *** mjura has joined #openstack-ironic | 05:27 | |
| *** e0ne has quit IRC | 05:43 | |
| *** e0ne has joined #openstack-ironic | 05:44 | |
| *** yolanda__ is now known as yolanda | 05:45 | |
| *** d0ugal has quit IRC | 05:46 | |
| *** marios has joined #openstack-ironic | 05:49 | |
| *** jaganathan has joined #openstack-ironic | 05:49 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 05:50 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 05:51 |
| *** liuzz_ has joined #openstack-ironic | 05:52 | |
| *** mvpnitesh has joined #openstack-ironic | 05:52 | |
| *** liuzz has quit IRC | 05:55 | |
| *** yuvaladar has joined #openstack-ironic | 06:08 | |
| *** yuvaladar has quit IRC | 06:09 | |
| *** d0ugal has joined #openstack-ironic | 06:13 | |
| *** jtomasek has joined #openstack-ironic | 06:18 | |
| *** jaganathan has quit IRC | 06:22 | |
| *** e0ne_ has joined #openstack-ironic | 06:27 | |
| *** e0ne has quit IRC | 06:30 | |
| *** rbartal has joined #openstack-ironic | 06:33 | |
| *** jaganathan has joined #openstack-ironic | 06:38 | |
| *** athomas has joined #openstack-ironic | 06:39 | |
| *** trungnv has quit IRC | 06:43 | |
| *** tiendc has quit IRC | 06:43 | |
| *** trungnv has joined #openstack-ironic | 06:44 | |
| *** tiendc has joined #openstack-ironic | 06:44 | |
| *** xdjiapei_ has joined #openstack-ironic | 06:47 | |
| *** xdjiapei_ has left #openstack-ironic | 06:47 | |
| *** jiapei has joined #openstack-ironic | 06:49 | |
| jiapei | Good afternoon | 06:50 |
| jiapei | Could anyone have a review of my patch? https://review.openstack.org/#/c/561405/ | 06:51 |
| patchbot | patch 561405 - ironic - Fix XClarity parameters discrepancy | 06:51 |
| jiapei | https://review.openstack.org/#/c/559960/ and this one😊 | 06:52 |
| patchbot | patch 559960 - ironic - Add documentatin for XClarity Driver | 06:52 |
| *** tesseract has joined #openstack-ironic | 06:58 | |
| *** tesseract has quit IRC | 07:00 | |
| *** diga has joined #openstack-ironic | 07:01 | |
| *** tesseract has joined #openstack-ironic | 07:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui stable/queens: Imported Translations from Zanata https://review.openstack.org/564402 | 07:04 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/564403 | 07:04 |
| *** jtomasek has quit IRC | 07:05 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 07:08 |
| *** diga has quit IRC | 07:08 | |
| *** yuvaladar has joined #openstack-ironic | 07:16 | |
| *** jiapei has quit IRC | 07:18 | |
| *** jiapei has joined #openstack-ironic | 07:18 | |
| *** jiapei has quit IRC | 07:19 | |
| *** jiapei has joined #openstack-ironic | 07:22 | |
| *** jiapei_ has joined #openstack-ironic | 07:23 | |
| *** jiapei has quit IRC | 07:24 | |
| *** jiapei_ is now known as jiapei | 07:24 | |
| *** jiapei has quit IRC | 07:24 | |
| *** jiapei_ has joined #openstack-ironic | 07:27 | |
| *** milan has joined #openstack-ironic | 07:28 | |
| *** jiapei_ has quit IRC | 07:28 | |
| *** jiapei has joined #openstack-ironic | 07:29 | |
| *** pcaruana has joined #openstack-ironic | 07:31 | |
| *** rcernin has quit IRC | 07:32 | |
| *** pmannidi has quit IRC | 07:35 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:45 | |
| *** pmannidi has joined #openstack-ironic | 07:49 | |
| *** threestrands_ has quit IRC | 07:52 | |
| *** racedo has joined #openstack-ironic | 08:04 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| *** exodusftw has quit IRC | 08:21 | |
| *** mvpnitesh has quit IRC | 08:23 | |
| *** derekh has joined #openstack-ironic | 08:34 | |
| *** yuvaladar has quit IRC | 08:42 | |
| *** mvpnitesh has joined #openstack-ironic | 08:48 | |
| *** hshiina is now known as hshiina|afk | 08:49 | |
| *** jovial[m] has joined #openstack-ironic | 08:52 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 08:57 | |
| *** hoonetorg has quit IRC | 08:58 | |
| *** AlexeyAbashkin has quit IRC | 09:01 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 09:01 | |
| *** sambetts|afk is now known as sambetts | 09:03 | |
| *** hoonetorg has joined #openstack-ironic | 09:15 | |
| *** MattMan has quit IRC | 09:17 | |
| *** MattMan has joined #openstack-ironic | 09:17 | |
| *** milan has quit IRC | 09:18 | |
| *** tiendc has quit IRC | 09:27 | |
| *** trungnv has quit IRC | 09:27 | |
| *** tiendc has joined #openstack-ironic | 09:28 | |
| *** trungnv has joined #openstack-ironic | 09:28 | |
| *** e0ne_ has quit IRC | 09:33 | |
| *** trungnv_ has joined #openstack-ironic | 09:34 | |
| *** derekh_ has joined #openstack-ironic | 09:34 | |
| *** dtantsur|afk is now known as dtantsur | 09:38 | |
| dtantsur | morning ironic | 09:39 |
| *** trungnv has quit IRC | 09:39 | |
| *** derekh has quit IRC | 09:39 | |
| sambetts | o/ dtantsur | 09:39 |
| *** jovial[m] has quit IRC | 09:40 | |
| dtantsur | \o | 09:40 |
| *** mtreinish has quit IRC | 09:41 | |
| *** jtomasek has joined #openstack-ironic | 09:42 | |
| *** mgoddard has joined #openstack-ironic | 09:44 | |
| etingof | o/ dtantsur | 09:44 |
| *** mtreinish has joined #openstack-ironic | 09:47 | |
| *** milan has joined #openstack-ironic | 09:59 | |
| *** trungnv_ has quit IRC | 10:00 | |
| pas-ha | morning dtantsur :-) | 10:03 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 10:14 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 10:14 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 10:14 |
| *** namnh has quit IRC | 10:14 | |
| *** jovial[m] has joined #openstack-ironic | 10:22 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Introduce oslo.messaging and sync rpc call https://review.openstack.org/561915 | 10:29 |
| dtantsur | TheJulia: https://github.com/dtantsur/ironic-bug-report | 10:30 |
| dtantsur | sambetts: mind reviewing https://review.openstack.org/#/c/557295/ and its parents? this is needed to switch inspector's discovery to hw types | 10:31 |
| patchbot | patch 557295 - ironic-tempest-plugin - Rework the ironic-inspector auto-discovery test | 10:31 |
| *** jiapei has quit IRC | 10:47 | |
| *** milan has quit IRC | 10:47 | |
| *** jiapei has joined #openstack-ironic | 10:47 | |
| *** jiapei has joined #openstack-ironic | 10:47 | |
| *** yolanda_ has joined #openstack-ironic | 10:53 | |
| *** yolanda__ has joined #openstack-ironic | 10:54 | |
| *** yolanda has quit IRC | 10:56 | |
| *** yolanda_ has quit IRC | 10:58 | |
| *** threestrands_ has joined #openstack-ironic | 11:01 | |
| *** e0ne has joined #openstack-ironic | 11:11 | |
| *** zhangfei has quit IRC | 11:13 | |
| *** csmart has joined #openstack-ironic | 11:16 | |
| *** retr0h has joined #openstack-ironic | 11:16 | |
| *** mkrai has joined #openstack-ironic | 11:16 | |
| sambetts | dtantsur: lgtm | 11:22 |
| dtantsur | thnx | 11:23 |
| *** lucasagomes is now known as lucas-hungry | 11:24 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:27 | |
| *** phuongnh has quit IRC | 11:33 | |
| *** jiapei has quit IRC | 11:34 | |
| *** e0ne has quit IRC | 11:35 | |
| dtantsur | does anyone remember why we don't wipe the whole metadata from all disks before deployment? | 11:37 |
| *** bfournie has quit IRC | 11:38 | |
| * dtantsur offline for a few mins due to updates | 11:39 | |
| *** dtantsur has quit IRC | 11:39 | |
| *** dtantsur has joined #openstack-ironic | 11:44 | |
| *** dtantsur has quit IRC | 11:44 | |
| *** dtantsur has joined #openstack-ironic | 11:44 | |
| *** dtantsur has quit IRC | 11:44 | |
| *** dprince has joined #openstack-ironic | 11:47 | |
| *** alex_xu has quit IRC | 11:49 | |
| *** dtantsur has joined #openstack-ironic | 11:49 | |
| *** dtantsur has quit IRC | 11:49 | |
| *** alex_xu has joined #openstack-ironic | 11:49 | |
| openstackgerrit | Will Szumski proposed openstack/ironic-python-agent master: Do not abort ata secure erase if security is in enabled state https://review.openstack.org/558533 | 11:49 |
| *** dtantsur has joined #openstack-ironic | 11:49 | |
| *** dtantsur has quit IRC | 11:49 | |
| *** e0ne has joined #openstack-ironic | 11:49 | |
| *** dtantsur has joined #openstack-ironic | 11:54 | |
| *** rbartal has quit IRC | 11:56 | |
| etingof | dtantsur would wiping out all the metadata reliably mean zeroing out the whole media which can be expensive...? fs control structures can be scattered over it | 11:57 |
| *** rbartal has joined #openstack-ironic | 11:59 | |
| *** rbudden has joined #openstack-ironic | 12:06 | |
| sambetts | dtantsur: well before the node even makes it to avaiable it should have gone through cleaning and therefore shouldn't have any meta data on it anyway | 12:06 |
| sambetts | dtantsur: otherwise your just moving cleaning to pre-deployment instead of pre-avaiable | 12:06 |
| *** tiendc has quit IRC | 12:11 | |
| *** mvpnitesh has quit IRC | 12:14 | |
| *** liuzz has joined #openstack-ironic | 12:14 | |
| dtantsur | sambetts: right, but see [tripleo] thread on re-enabling cleaning | 12:17 |
| *** liuzz_ has quit IRC | 12:18 | |
| dtantsur | tl;dr they don't want cleaning because they constantly see people deleting nodes by mistake | 12:18 |
| dtantsur | (yes, "pets" case) | 12:18 |
| *** bfournie has joined #openstack-ironic | 12:18 | |
| sambetts | dtantsur: sure... add if someone deletes a node and redeploys it by mistake your in the same situation | 12:19 |
| dtantsur | sambetts: right, I guess they assume that re-deploying something by mistake is harder (well, with tripleo it is :D ) | 12:20 |
| sambetts | tbh the problem here is the "deleting a node by mistake" because Ironic is doing exactly what its been told to do | 12:22 |
| dtantsur | I don't disagree with that :) | 12:23 |
| * dtantsur thinks of modifying the tripleo workflows to wipe all available nodes.. | 12:24 | |
| sambetts | dtantsur: yeah it might be saner if you don't want to enable auto cleaning, to add a flow which moves the node to manageable and triggers the manual cleaning step to wipe the metadata | 12:25 |
| sambetts | dtantsur: but IMO you really should enable auto cleaning | 12:26 |
| sambetts | and fix the problems around nodes getting deleted for some reason | 12:26 |
| sambetts | dtantsur: tbh deploy steps / the ansible driver would could also give you the flexibility to add a metadata clear to the deploy process | 12:28 |
| *** lucas-hungry is now known as lucasagomes | 12:30 | |
| *** trown|outtypewww is now known as trown | 12:34 | |
| dtantsur | yep | 12:36 |
| dtantsur | anyway, I don't want to fix half of tripleo :) | 12:36 |
| *** e0ne has quit IRC | 12:37 | |
| dtantsur | I'm just wondering if it's useful to try metadata cleaning right before the deployment to make sure we don't fail in crazy cases | 12:37 |
| sambetts | dtantsur: s/*.puppet/ansible/g ;) | 12:37 |
| dtantsur | hehe | 12:37 |
| dtantsur | in addition to disabled cleaning, it may be e.g. a new disk connected to an available node without wiping it (not that I recommend doing that, but...) | 12:38 |
| sambetts | personally not sure its a good idea because of the rebuild scenario too, if we wipe on deployment then a rebuild could remove information that normally wouldn't be removed unless you issue a full clean | 12:40 |
| dtantsur | side note: "the problem around nodes getting deleted" is called Heat :D | 12:40 |
| dtantsur | hmmmmm | 12:40 |
| dtantsur | yes, rebuild may be a problem | 12:41 |
| *** rloo has joined #openstack-ironic | 12:44 | |
| *** threestrands_ has quit IRC | 12:58 | |
| *** Goneri has joined #openstack-ironic | 13:08 | |
| *** jcoufal has joined #openstack-ironic | 13:11 | |
| *** baha has joined #openstack-ironic | 13:11 | |
| *** rbartal has quit IRC | 13:17 | |
| hjensas | dtantsur: and Heat is just doing what you told it when it deletes a node, cause when it happens it's because someone changed a parameter requiring resource to be replaced. Like, let's change this thing in config drive. | 13:18 |
| *** rbartal has joined #openstack-ironic | 13:21 | |
| dtantsur | yeah, not quite an obvious behavior for an installer :) | 13:21 |
| *** cjloader has joined #openstack-ironic | 13:24 | |
| sambetts | perhaps Heat needs/needed a big warning "Due to the nature of this change this will result in a full destructive redeploy of some resources, do you want to continue?" | 13:26 |
| hjensas | dtantsur: sambetts: Yeah, there was talk about some dry-run features in tripleo. Something that would output something with alarming colors and uppercase letters ... not sure what happened to that. | 13:28 |
| *** cjloader has quit IRC | 13:29 | |
| *** cjloader has joined #openstack-ironic | 13:29 | |
| olivierb- | Hello sambetts, concerning the DHCP timeout issue I mentioned yesterday, I tried today after having asked our lab admin to change the switch config to enable spanning tree fast link on the port where the baremetal machines are connected and even though it seems to ease up a little I still have had some issues. So better but not perfect. This is FYI | 13:30 |
| *** cjloader has quit IRC | 13:34 | |
| dtantsur | hjensas: I guess nothing | 13:35 |
| *** mjturek has joined #openstack-ironic | 13:44 | |
| *** e0ne has joined #openstack-ironic | 13:55 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic stable/queens: Implement a function to check the image status https://review.openstack.org/564515 | 13:58 |
| *** rbartal has quit IRC | 14:00 | |
| bfournie | dtantsur: isn't there a unit test on https://review.openstack.org/#/c/559500/? | 14:04 |
| patchbot | patch 559500 - ironic - Implement a function to check the image status (MERGED) | 14:05 |
| TheJulia | good morning everyone | 14:05 |
| *** cjloader has joined #openstack-ironic | 14:05 | |
| *** e0ne has quit IRC | 14:06 | |
| bfournie | dtantsur: actually there really isn't, just a fix so existing test passes | 14:07 |
| *** cjloader has quit IRC | 14:08 | |
| dtantsur | yep | 14:08 |
| dtantsur | morning TheJulia | 14:08 |
| bfournie | TheJulia: good morning | 14:08 |
| *** cjloader has joined #openstack-ironic | 14:08 | |
| * dtantsur has found out that ironic-tempest-plugin is using the 'fake' classic driver... this will never end..... | 14:08 | |
| TheJulia | The whole tripleo cleaning thing makes me just want to turn on cleaning by default deep inside of ironic. If heat does something wrong, then that becomes reason to fix the issues in heat or add features to help become mitigation controls. To simply not have any starting point sanity is not sane. I realize that is also a hardline view, and perhaps I've spent too many hours this week chasing bugs in that arena... | 14:10 |
| TheJulia | dtantsur: revert^4 of classic drivers "we think we go it!" | 14:11 |
| TheJulia | s/go/got/ | 14:11 |
| sambetts | TheJulia: +1000, at least the pretty quick metadata delete should be enabled by default | 14:11 |
| TheJulia | ++ | 14:11 |
| *** e0ne has joined #openstack-ironic | 14:11 | |
| dtantsur | ++ | 14:12 |
| dtantsur | I understand why someone would not want a full cleaning in a single-tenant environment, but at least something... | 14:12 |
| dtantsur | deprecating automated_clean option? :D | 14:12 |
| sambetts | I would support it | 14:12 |
| * TheJulia ponders | 14:13 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Allow using the fake-hardware hardware type for API tests https://review.openstack.org/564517 | 14:13 |
| dtantsur | part one ^^^ | 14:13 |
| TheJulia | I'm kind of liking the idea of making metadata cleaning a hard requirement | 14:14 |
| TheJulia | but easing that into place is the problem such that other overarching bugs are fixed is... yeah | 14:14 |
| *** e0ne has quit IRC | 14:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch to the fake-hardware hardware type for API tests https://review.openstack.org/564519 | 14:15 |
| dtantsur | part two ^^^ | 14:15 |
| *** e0ne has joined #openstack-ironic | 14:20 | |
| *** rajinir has joined #openstack-ironic | 14:22 | |
| *** e0ne has quit IRC | 14:25 | |
| dtantsur | TheJulia: speaking of the driver composition reform, could you review https://review.openstack.org/#/c/557292/ and https://review.openstack.org/#/c/557295/ please? | 14:30 |
| patchbot | patch 557292 - ironic-tempest-plugin - Run ironic-inspector discovery jobs in the CI | 14:30 |
| patchbot | patch 557295 - ironic-tempest-plugin - Rework the ironic-inspector auto-discovery test | 14:30 |
| *** kzaitsev_pi has quit IRC | 14:30 | |
| *** kzaitsev_pi has joined #openstack-ironic | 14:31 | |
| jroll | morning | 14:31 |
| jroll | reminder to keep your management controllers off the internet: https://www.bleepingcomputer.com/news/security/ransomware-hits-hpe-ilo-remote-management-interfaces/ | 14:31 |
| sambetts | -_- | 14:32 |
| TheJulia | dtantsur: sure, once I'm not replyign to emails | 14:32 |
| TheJulia | wow.... | 14:32 |
| dtantsur | jroll: morning :) | 14:32 |
| jroll | dtantsur | does anyone remember why we don't wipe the whole metadata from all disks before deployment? <- I would support this, let's do it once we have deploy steps | 14:33 |
| jroll | can exclude that step in rebuild | 14:33 |
| dtantsur | right | 14:33 |
| * TheJulia posts the link jroll found to facebook and ponders twitter with "Remember that time..." | 14:35 | |
| jroll | :) | 14:35 |
| *** beekneemech is now known as bnemec | 14:37 | |
| *** pcaruana has quit IRC | 14:42 | |
| *** r-daneel has joined #openstack-ironic | 14:45 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: [DNM] Testing Cisco Third Party CI https://review.openstack.org/514312 | 14:49 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: [DNM] Testing Cisco Third Party CI https://review.openstack.org/514312 | 14:49 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: [DNM] Testing Cisco Third Party CI https://review.openstack.org/514312 | 14:52 |
| dtantsur | etingof: are you going to pycon.cz in the end? | 14:52 |
| *** jiapei has joined #openstack-ironic | 14:57 | |
| *** athomas has quit IRC | 15:00 | |
| *** r-daneel has quit IRC | 15:01 | |
| *** mjura has quit IRC | 15:01 | |
| *** ajya has joined #openstack-ironic | 15:02 | |
| *** rpioso|afk is now known as rpioso | 15:03 | |
| *** early` has quit IRC | 15:03 | |
| rpioso | Good morning | 15:03 |
| *** jistr|mtgs is now known as jistr | 15:04 | |
| TheJulia | o/ rpioso | 15:05 |
| rpioso | TheJulia: \o | 15:05 |
| -openstackstatus- NOTICE: We've successfully troubleshooted the issue that prevented paste.openstack.org from loading and it's now back online, thank you for your patience. | 15:05 | |
| *** links has quit IRC | 15:06 | |
| jroll | TheJulia: fyi I won't be making it to the forum, but zer0c00l and penick will be there representing our team | 15:06 |
| TheJulia | jroll: :( Thanks for the heads up | 15:07 |
| jroll | no problem | 15:08 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Introduce oslo.messaging and sync rpc call https://review.openstack.org/561915 | 15:09 |
| *** early has joined #openstack-ironic | 15:15 | |
| *** gyee has joined #openstack-ironic | 15:40 | |
| TheJulia | gyee: o/ | 15:40 |
| gyee | TheJulia, hi | 15:40 |
| TheJulia | gyee: I feel bad I missed something you asked earlier in the week, it is just not coming to me at the moment :( | 15:40 |
| gyee | no worries | 15:40 |
| gyee | I did ended up filing a couple of bugs for Ironic :-) | 15:41 |
| TheJulia | woot! | 15:41 |
| gyee | the 'validate' command wasn't happy, even with storage-interface set to cinder | 15:43 |
| TheJulia | Well, it may never be state wise, do you have a link handy? | 15:46 |
| TheJulia | state wise unless it has all of the information | 15:46 |
| gyee | TheJulia, here are the two that I filed. https://bugs.launchpad.net/nova/+bug/1766661 and https://bugs.launchpad.net/nova/+bug/1766668 | 15:48 |
| openstack | Launchpad bug 1766661 in OpenStack Compute (nova) "'host' property is missing for Cinder volume connector when boot from volume" [Undecided,New] | 15:48 |
| openstack | Launchpad bug 1766668 in OpenStack Compute (nova) "'image_source' should not be required for Ironic boot from volume" [Undecided,New] | 15:48 |
| gyee | the other is the volume connector, we are testing bfv with 3par_iscsi cinder backend | 15:48 |
| TheJulia | oh, neat | 15:48 |
| TheJulia | gyee: do any other other cinder driers expect a host key on the connector data to create the volume attachment? | 15:51 |
| gyee | per my understanding, LVM does not require the 'host' property | 15:52 |
| gyee | that's what devstack has by default | 15:52 |
| gyee | but there are quite a few cinder backends need the 'host' property | 15:53 |
| NobodyCam | Good Morning Ironic'ers | 15:54 |
| *** AlexeyAbashkin has quit IRC | 15:56 | |
| TheJulia | gyee: pondering the first one, replied to the second one. | 15:57 |
| *** hamzy has quit IRC | 15:59 | |
| gyee | TheJulia, yeah, the first one is kinda misleading, the way it is presented | 16:00 |
| TheJulia | yeah, partially this is by design | 16:01 |
| * TheJulia looks at one thing | 16:01 | |
| *** d0ugal has quit IRC | 16:03 | |
| *** d0ugal has joined #openstack-ironic | 16:04 | |
| *** akhilaki has joined #openstack-ironic | 16:06 | |
| *** marios has quit IRC | 16:07 | |
| TheJulia | so I guess the code around https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1851 would need to change | 16:07 |
| TheJulia | or the way we're getting there is just different enough | 16:07 |
| TheJulia | gyee: are you trying to manually boot from volume, or are you trying to use nova? | 16:09 |
| gyee | the properties came from Ironic right? | 16:09 |
| TheJulia | yes and no | 16:10 |
| *** baha has quit IRC | 16:10 | |
| gyee | I am using 'openstack server create --volume <id> ...' | 16:10 |
| TheJulia | it will lookup and ship IP address information about the node based upon ports defined | 16:10 |
| TheJulia | fun! | 16:10 |
| TheJulia | okay | 16:10 |
| gyee | but I have other issues right now, I can see the volume attaching with that hack, but now Nova capabilities filter is returning nothing | 16:13 |
| gyee | time for more code diving :-) | 16:13 |
| *** jtomasek has quit IRC | 16:14 | |
| TheJulia | i see what is going on | 16:14 |
| TheJulia | so partially by design, partial bug | 16:14 |
| TheJulia | I'll update the bug | 16:15 |
| gyee | TheJulia, thanks! I'll continue with the investigation on my end. Will update you if I discover something interesting. | 16:15 |
| *** fragatina has quit IRC | 16:18 | |
| *** fragatina has joined #openstack-ironic | 16:18 | |
| openstackgerrit | Oleksiy Petrenko proposed openstack/ironic master: WIP: Cache min_disk in instance_info https://review.openstack.org/564551 | 16:25 |
| openstackgerrit | Oleksiy Petrenko proposed openstack/ironic master: WIP: Cache min_disk in instance_info https://review.openstack.org/564551 | 16:27 |
| * dtantsur facepalms | 16:27 | |
| dtantsur | the way we set default_deploy_interface in the CI is incompatible with how we use the fake hardware type in the API tests >_< | 16:28 |
| TheJulia | gyee: replied to the first issue, please take a look and let me know if that makes sense and describes what needs to be done. | 16:28 |
| * TheJulia slides whiskey over to dtantsur | 16:29 | |
| dtantsur | TheJulia: thanks, it's much needed. I'm not sure how to unscrew it without moving the API tests to a job of their own.. | 16:29 |
| openstackgerrit | Oleksiy Petrenko proposed openstack/ironic-python-agent master: WIP: Validate disk size according to image.min_disk(if provided) https://review.openstack.org/564553 | 16:29 |
| dtantsur | which is probably a good idea | 16:29 |
| TheJulia | ++ | 16:30 |
| TheJulia | luckily that is a lot of forklifting and not writing new code | 16:30 |
| TheJulia | gyee: absolutely agree that we should better document too, please open that as a separate item for ironic | 16:31 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Run ironic-inspector discovery jobs in the CI https://review.openstack.org/557292 | 16:35 |
| gyee | TheJulia, yes, your assessment make sense. For point #4, adding a 'host' type should work. | 16:36 |
| dtantsur | hjensas: hey, could you please respond on https://review.openstack.org/#/c/561257/ re testing? | 16:36 |
| patchbot | patch 561257 - networking-baremetal - ML2 Agent: Handle SIGHUP mutable config options | 16:37 |
| *** lucasagomes is now known as lucas-afk | 16:38 | |
| hjensas | dtantsur: let me restack and retest, will put some log messages in the comments. | 16:39 |
| gyee | TheJulia, I haven't had a chance to get to the Ironic multi-tenancy part yet so I don't know whether specifying the mac address in the connector will work. | 16:41 |
| dtantsur | thanks hjensas | 16:52 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Move API (functional) tests to separate jobs https://review.openstack.org/564560 | 16:52 |
| dtantsur | TheJulia: naive attempt ^^^ | 16:52 |
| *** racedo has quit IRC | 16:53 | |
| *** jaganathan has quit IRC | 16:53 | |
| *** dprince has quit IRC | 16:54 | |
| *** racedo has joined #openstack-ironic | 16:54 | |
| *** racedo has quit IRC | 16:54 | |
| *** racedo has joined #openstack-ironic | 16:54 | |
| *** racedo has quit IRC | 16:55 | |
| dtantsur | the biggest downside is that we're going to have EVEN MOAR JOBS on ironic-tempest-plugin | 16:55 |
| *** racedo has joined #openstack-ironic | 16:55 | |
| *** racedo has quit IRC | 16:56 | |
| *** racedo has joined #openstack-ironic | 16:56 | |
| *** racedo has quit IRC | 16:56 | |
| *** racedo has joined #openstack-ironic | 16:57 | |
| *** racedo has quit IRC | 16:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Move API (functional) tests to separate jobs https://review.openstack.org/564560 | 17:01 |
| *** milan has joined #openstack-ironic | 17:01 | |
| TheJulia | Well, we should be touching it less often | 17:01 |
| TheJulia | gyee: given that the mac address is information about the host... it should work (i hope) | 17:02 |
| *** derekh_ has quit IRC | 17:03 | |
| gyee | TheJulia, should I move those bugs to storyboard? Please let me know. | 17:04 |
| TheJulia | gyee: the first has legitimate nova related things. The second likely should just be moved over to storyboard | 17:05 |
| TheJulia | OH! | 17:05 |
| TheJulia | I wonder if the reason why your getting that error is were getting unexpected data from the backend storage driver for the target information | 17:05 |
| TheJulia | if you could get us a look at that data, it would really help | 17:06 |
| gyee | you mean the 'host' KeyError? | 17:06 |
| *** lbragstad has quit IRC | 17:07 | |
| TheJulia | I mean the image_source error | 17:07 |
| TheJulia | I'm wondering if were just getting something back in the target information from cinder that doesn't match up with what we expect which might be why were still returning an image source needed for validation | 17:07 |
| *** milan has quit IRC | 17:07 | |
| *** baoli has joined #openstack-ironic | 17:08 | |
| gyee | let me reset the test env to see if we can extract more information on that one | 17:08 |
| TheJulia | gyee: thanks! | 17:09 |
| *** baha has joined #openstack-ironic | 17:10 | |
| *** dtantsur is now known as dtantsur|afk | 17:10 | |
| dtantsur|afk | o/ | 17:10 |
| *** hamzy has joined #openstack-ironic | 17:12 | |
| *** trown is now known as trown|lunch | 17:13 | |
| *** baoli has quit IRC | 17:17 | |
| TheJulia | goodnight dtantsur|afk | 17:23 |
| *** dprince has joined #openstack-ironic | 17:27 | |
| *** e0ne has joined #openstack-ironic | 17:35 | |
| *** bfournie has quit IRC | 17:38 | |
| *** bfournie has joined #openstack-ironic | 17:38 | |
| *** bfournie has quit IRC | 17:39 | |
| *** srart has quit IRC | 17:39 | |
| *** srart has joined #openstack-ironic | 17:40 | |
| *** phuongnh has joined #openstack-ironic | 17:55 | |
| *** e0ne has quit IRC | 18:00 | |
| *** fragatin_ has joined #openstack-ironic | 18:15 | |
| *** trown|lunch is now known as trown | 18:16 | |
| *** fragatina has quit IRC | 18:17 | |
| *** lbragstad has joined #openstack-ironic | 18:22 | |
| *** e0ne has joined #openstack-ironic | 18:22 | |
| *** tesseract has quit IRC | 18:24 | |
| *** phuongnh has quit IRC | 18:24 | |
| *** Phuongnh has joined #openstack-ironic | 18:24 | |
| *** jiapei has quit IRC | 18:26 | |
| sambetts | /nick sambetts|afk | 18:43 |
| sambetts | night all | 18:43 |
| *** akhilaki has quit IRC | 18:45 | |
| *** Phuong_nh has joined #openstack-ironic | 18:46 | |
| *** harlowja has joined #openstack-ironic | 18:46 | |
| *** Phuongnh has quit IRC | 18:48 | |
| *** mgoddard has quit IRC | 18:52 | |
| *** chihhsin has quit IRC | 18:53 | |
| *** chihhsin has joined #openstack-ironic | 18:54 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Deprecate Oneview https://review.openstack.org/564587 | 19:01 |
| *** bfournie has joined #openstack-ironic | 19:05 | |
| *** bfournie has quit IRC | 19:06 | |
| *** bfournie has joined #openstack-ironic | 19:06 | |
| *** Phuong_nh has quit IRC | 19:07 | |
| *** e0ne has quit IRC | 19:14 | |
| *** fragatin_ has quit IRC | 19:19 | |
| *** e0ne has joined #openstack-ironic | 19:21 | |
| *** e0ne has quit IRC | 19:21 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Call shutdown() on SIGTERM - controlled teardown https://review.openstack.org/563335 | 19:29 |
| *** yolanda__ has quit IRC | 20:38 | |
| *** yolanda__ has joined #openstack-ironic | 20:43 | |
| *** hamzy has quit IRC | 20:45 | |
| *** trown is now known as trown|outtypewww | 20:56 | |
| *** cjloader has quit IRC | 21:00 | |
| *** fragatina has joined #openstack-ironic | 21:02 | |
| *** livelace-link has joined #openstack-ironic | 21:03 | |
| *** mgoddard has joined #openstack-ironic | 21:09 | |
| *** Goneri has quit IRC | 21:16 | |
| *** dprince has quit IRC | 21:17 | |
| *** baha has quit IRC | 21:23 | |
| gyee | TheJulia, I updated https://bugs.launchpad.net/nova/+bug/1766668. Doesn't appear 'openstack baremetal validate <uuid>' involve Cinder. | 21:25 |
| openstack | Launchpad bug 1766668 in OpenStack Compute (nova) "'image_source' should not be required for Ironic boot from volume" [Undecided,New] | 21:25 |
| TheJulia | gyee, ti does not | 21:25 |
| TheJulia | it is purely in ironic's code base, meaning there is nothing to do with nova there | 21:25 |
| *** mgoddard has quit IRC | 21:26 | |
| gyee | I see. But for BFV feature, should validate involve Cinder at all, like to make sure the connectors are sane? | 21:27 |
| *** hemna_ has joined #openstack-ironic | 21:29 | |
| *** ajya has quit IRC | 21:29 | |
| gyee | TheJulia, for BFV, is resource_class required on the baremetal node? | 22:01 |
| TheJulia | gyee: With queens, a resource class is required for proper scheduling from nova | 22:02 |
| gyee | what about Pike? | 22:02 |
| TheJulia | gyee: that is when it started but wasn't a hard requirement | 22:02 |
| gyee | so looks like I also need to add iscsi_boot=True in the flavor's extra_specs | 22:03 |
| TheJulia | depending on your scheduling settings, quite possibly | 22:03 |
| gyee | not sure if it was mentioned in the BFV section of the doc | 22:03 |
| gyee | don't remember seeing it | 22:03 |
| TheJulia | gyee: I don't think it is indicated, but it is not always a requirement depending on how scheduling is setup | 22:04 |
| gyee | TheJulia, seems like that's the case for me. If the extra_specs don't match the ironic node capabilities exactly, Nova capabilities filter return nothing. Hence, no valid host found error. | 22:12 |
| gyee | I can try using resource_class next. | 22:13 |
| *** rcernin has joined #openstack-ironic | 22:23 | |
| openstackgerrit | Merged openstack/bifrost master: Fix cross project jobs https://review.openstack.org/564279 | 22:23 |
| *** rbudden has quit IRC | 22:28 | |
| *** grafoo has quit IRC | 22:30 | |
| *** rpioso is now known as rpioso|afk | 22:44 | |
| *** rbudden has joined #openstack-ironic | 22:49 | |
| *** jmccrory has quit IRC | 23:17 | |
| *** jmccrory has joined #openstack-ironic | 23:18 | |
| *** fragatina has quit IRC | 23:20 | |
| *** fragatina has joined #openstack-ironic | 23:20 | |
| *** fragatina has quit IRC | 23:23 | |
| *** fragatina has joined #openstack-ironic | 23:24 | |
| *** pmannidi has quit IRC | 23:27 | |
| *** pmannidi has joined #openstack-ironic | 23:28 | |
| *** gyee has quit IRC | 23:54 | |
| *** anton has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!