| openstackgerrit | melissaml proposed openstack/ironic-specs master: fix typos in documentation https://review.openstack.org/542871 | 00:06 |
|---|---|---|
| openstackgerrit | melissaml proposed openstack/python-ironicclient master: fix a typo in documentation https://review.openstack.org/555511 | 00:09 |
| *** mjturek has quit IRC | 00:12 | |
| openstackgerrit | melissaml proposed openstack/bifrost master: fix typos in documentation https://review.openstack.org/555533 | 00:20 |
| *** gyee has quit IRC | 00:21 | |
| *** masber has quit IRC | 00:22 | |
| *** weshay is now known as weshay_PTO | 00:25 | |
| openstackgerrit | melissaml proposed openstack/ironic master: fix typos in documentation https://review.openstack.org/555542 | 00:28 |
| *** hamzy has joined #openstack-ironic | 00:30 | |
| openstackgerrit | melissaml proposed openstack/ironic master: fix typos in documentation https://review.openstack.org/555542 | 00:34 |
| *** hshiina has joined #openstack-ironic | 00:46 | |
| openstackgerrit | melissaml proposed openstack/ironic-specs master: fix typos in documentation https://review.openstack.org/555556 | 00:56 |
| *** e0ne_ has quit IRC | 01:00 | |
| *** tiendc has joined #openstack-ironic | 01:06 | |
| *** masber has joined #openstack-ironic | 01:12 | |
| *** harlowja_ has quit IRC | 01:13 | |
| openstackgerrit | Doug Hellmann proposed openstack/ironic master: add lower-constraints job https://review.openstack.org/555567 | 01:17 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic-inspector master: add lower-constraints job https://review.openstack.org/555568 | 01:17 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic-lib master: add lower-constraints job https://review.openstack.org/555569 | 01:18 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic-python-agent master: add lower-constraints job https://review.openstack.org/555570 | 01:18 |
| openstackgerrit | Doug Hellmann proposed openstack/ironic-ui master: add lower-constraints job https://review.openstack.org/555572 | 01:18 |
| *** sapd has quit IRC | 01:34 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/551624 | 01:46 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/555585 | 01:46 |
| *** sapd has joined #openstack-ironic | 02:01 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Add BIOSInterface to base driver class https://review.openstack.org/507793 | 02:11 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration https://review.openstack.org/534595 | 02:11 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration https://review.openstack.org/534595 | 02:34 |
| *** masber has quit IRC | 02:41 | |
| *** cjloade__ has joined #openstack-ironic | 02:42 | |
| *** cjloade__ is now known as cjloader__ | 02:43 | |
| *** livelace-link has quit IRC | 02:45 | |
| *** cjloader_ has quit IRC | 02:45 | |
| *** livelace-link has joined #openstack-ironic | 02:46 | |
| *** cjloader__ has quit IRC | 02:47 | |
| *** livelace-link has quit IRC | 02:48 | |
| *** rbudden has quit IRC | 03:04 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state https://review.openstack.org/555610 | 03:10 |
| openstackgerrit | Rushil Chugh proposed openstack/ironic master: Add documentation for XClarity Driver https://review.openstack.org/549659 | 03:10 |
| *** tuanla____ has joined #openstack-ironic | 03:11 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 03:15 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state https://review.openstack.org/555610 | 03:18 |
| *** masber has joined #openstack-ironic | 03:19 | |
| *** AlexeyAbashkin has quit IRC | 03:20 | |
| *** fragatin_ has joined #openstack-ironic | 03:25 | |
| *** fragatin_ has quit IRC | 03:27 | |
| *** fragatin_ has joined #openstack-ironic | 03:28 | |
| *** fragatina has quit IRC | 03:29 | |
| *** fragatin_ has quit IRC | 03:33 | |
| *** hshiina has quit IRC | 03:41 | |
| *** hshiina has joined #openstack-ironic | 03:42 | |
| *** fragatina has joined #openstack-ironic | 03:45 | |
| *** hshiina has quit IRC | 03:48 | |
| *** fragatina has quit IRC | 03:49 | |
| *** fragatina has joined #openstack-ironic | 03:54 | |
| *** itlinux has quit IRC | 03:59 | |
| *** hshiina has joined #openstack-ironic | 04:05 | |
| *** jaganathan has quit IRC | 04:13 | |
| *** early has quit IRC | 04:16 | |
| *** harlowja has joined #openstack-ironic | 04:19 | |
| *** fragatina has quit IRC | 04:19 | |
| *** early has joined #openstack-ironic | 04:19 | |
| *** fragatina has joined #openstack-ironic | 04:20 | |
| *** persia has quit IRC | 04:20 | |
| *** weshay_PTO is now known as weshay | 04:21 | |
| *** persia has joined #openstack-ironic | 04:21 | |
| *** links has joined #openstack-ironic | 04:26 | |
| *** sapd_ has joined #openstack-ironic | 04:51 | |
| *** sapd has quit IRC | 04:51 | |
| *** dpawar has joined #openstack-ironic | 05:02 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state https://review.openstack.org/555610 | 05:06 |
| *** harlowja has quit IRC | 05:12 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 05:15 | |
| *** AlexeyAbashkin has quit IRC | 05:19 | |
| *** jaganathan has joined #openstack-ironic | 05:20 | |
| *** gyankum has joined #openstack-ironic | 05:23 | |
| *** dpawar has quit IRC | 06:02 | |
| openstackgerrit | pippo proposed openstack/ironic master: Fix typos There are two 'the', delete one of them. https://review.openstack.org/555645 | 06:10 |
| *** AlexeyAbashkin has joined #openstack-ironic | 06:17 | |
| *** AlexeyAbashkin has quit IRC | 06:22 | |
| *** fragatina has quit IRC | 06:23 | |
| openstackgerrit | melissaml proposed openstack/python-ironic-inspector-client master: Follow the new PTI for document build https://review.openstack.org/554716 | 06:34 |
| *** dpawar has joined #openstack-ironic | 06:38 | |
| *** marios has joined #openstack-ironic | 06:44 | |
| *** hjensas has quit IRC | 06:45 | |
| *** dpawar has quit IRC | 06:47 | |
| *** fragatina has joined #openstack-ironic | 06:48 | |
| *** dpawar has joined #openstack-ironic | 07:06 | |
| *** mjura has joined #openstack-ironic | 07:08 | |
| openstackgerrit | Anshul Jain proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 07:12 |
| *** jtomasek has joined #openstack-ironic | 07:17 | |
| *** pcaruana has joined #openstack-ironic | 07:21 | |
| *** MattMan has quit IRC | 07:26 | |
| *** MattMan has joined #openstack-ironic | 07:27 | |
| *** cjloader has joined #openstack-ironic | 07:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/555674 | 07:30 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Updated from global requirements https://review.openstack.org/551490 | 07:30 |
| *** cjloader has quit IRC | 07:31 | |
| *** cjloader has joined #openstack-ironic | 07:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/networking-baremetal master: Updated from global requirements https://review.openstack.org/551519 | 07:35 |
| *** arnewiebalck_ has joined #openstack-ironic | 07:38 | |
| *** cjloader has quit IRC | 07:42 | |
| *** cjloader has joined #openstack-ironic | 07:43 | |
| *** cjloader has quit IRC | 07:44 | |
| *** cjloader has joined #openstack-ironic | 07:45 | |
| *** cjloader has quit IRC | 07:48 | |
| *** cjloader has joined #openstack-ironic | 07:48 | |
| *** dpawar has quit IRC | 07:48 | |
| *** cjloader has quit IRC | 07:56 | |
| *** cjloader has joined #openstack-ironic | 07:57 | |
| *** ianychoi has quit IRC | 08:00 | |
| *** ianychoi has joined #openstack-ironic | 08:00 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:02 | |
| *** hjensas has joined #openstack-ironic | 08:10 | |
| *** hjensas has quit IRC | 08:10 | |
| *** hjensas has joined #openstack-ironic | 08:10 | |
| *** cjloader has quit IRC | 08:10 | |
| *** cjloader has joined #openstack-ironic | 08:10 | |
| *** cjloader has quit IRC | 08:11 | |
| *** cjloader has joined #openstack-ironic | 08:12 | |
| *** cjloader has quit IRC | 08:16 | |
| *** tesseract has joined #openstack-ironic | 08:20 | |
| *** dpawar has joined #openstack-ironic | 08:39 | |
| *** milan has joined #openstack-ironic | 08:42 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Add maintenance_type to objects involved https://review.openstack.org/555708 | 09:07 |
| *** cjloader has joined #openstack-ironic | 09:12 | |
| *** lucas-afk is now known as lucasagomes | 09:13 | |
| *** cjloader has quit IRC | 09:17 | |
| *** rcernin has quit IRC | 09:20 | |
| *** mgoddard has joined #openstack-ironic | 09:23 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 09:26 |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 09:26 |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 09:26 |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add RPC object https://review.openstack.org/511714 | 09:26 |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 09:26 |
| openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 09:27 |
| zshi | tuanla____, ^^ | 09:28 |
| tuanla____ | hello zshi | 09:29 |
| zshi | hey | 09:29 |
| zshi | see above, rebase of bios patches | 09:29 |
| *** stendulker has joined #openstack-ironic | 09:29 | |
| tuanla____ | zshi, I got it. | 09:30 |
| zshi | cool | 09:30 |
| *** e0ne has joined #openstack-ironic | 09:37 | |
| *** derekh has joined #openstack-ironic | 09:37 | |
| *** fatema_ has joined #openstack-ironic | 09:40 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state https://review.openstack.org/555610 | 09:41 |
| *** hshiina has quit IRC | 09:43 | |
| *** liuzz has quit IRC | 09:47 | |
| *** tuanla____ has quit IRC | 10:01 | |
| *** rcernin has joined #openstack-ironic | 10:08 | |
| *** milan has quit IRC | 10:09 | |
| *** mjura has quit IRC | 10:30 | |
| *** mjura has joined #openstack-ironic | 10:31 | |
| *** sambetts|afk is now known as sambetts | 10:32 | |
| *** dtantsur|afk is now known as dtantsur | 10:41 | |
| dtantsur | morning ironic, happy Friday | 10:41 |
| etingof | o/ | 10:41 |
| * etingof has yet to encounter an unhappy Friday | 10:42 | |
| dtantsur | this one may count for me, I'm even more sick than yesterday | 10:43 |
| sambetts | dtantsur: :( | 10:43 |
| dtantsur | etingof: we can have some of the outreachy people help with https://review.openstack.org/#/c/463829/ per deray's comment | 11:04 |
| patchbot | patch 463829 - sushy - Add foundation for supporting redfish OEMs | 11:04 |
| openstackgerrit | melissaml proposed openstack/bifrost master: Follow the new PTI for document build https://review.openstack.org/555740 | 11:08 |
| * etingof will keep that in mind | 11:11 | |
| dtantsur | etingof: this is actually a quite important patch to finish IMO | 11:12 |
| *** athomas_ has joined #openstack-ironic | 11:14 | |
| *** dprince has joined #openstack-ironic | 11:14 | |
| *** pcaruana has quit IRC | 11:27 | |
| *** stendulker has quit IRC | 11:30 | |
| *** bfournie has quit IRC | 11:32 | |
| *** bfournie has joined #openstack-ironic | 11:32 | |
| *** bfournie has quit IRC | 11:36 | |
| jroll | morning ironic, happy friday indeed \o/ | 12:01 |
| dtantsur | morning jroll | 12:03 |
| *** jlabarre has joined #openstack-ironic | 12:03 | |
| *** bfournie has joined #openstack-ironic | 12:10 | |
| openstackgerrit | Merged openstack/python-ironicclient master: fix a typo in documentation https://review.openstack.org/555511 | 12:22 |
| *** jaganathan has quit IRC | 12:26 | |
| *** pcaruana has joined #openstack-ironic | 12:30 | |
| *** rcernin has quit IRC | 12:30 | |
| *** tiendc has quit IRC | 12:40 | |
| *** lucasagomes is now known as lucas-hungry | 12:43 | |
| *** edleafe is now known as figleaf | 12:46 | |
| *** trown has joined #openstack-ironic | 12:47 | |
| *** cjloader has joined #openstack-ironic | 13:06 | |
| *** Goneri has joined #openstack-ironic | 13:06 | |
| *** rbudden has joined #openstack-ironic | 13:10 | |
| *** cjloader has quit IRC | 13:11 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface https://review.openstack.org/526773 | 13:13 |
| *** racedo has quit IRC | 13:15 | |
| *** racedo has joined #openstack-ironic | 13:15 | |
| *** racedo has quit IRC | 13:16 | |
| *** racedo has joined #openstack-ironic | 13:16 | |
| *** racedo has quit IRC | 13:16 | |
| *** dtantsur is now known as dtantsur|brb | 13:20 | |
| *** mjturek has joined #openstack-ironic | 13:27 | |
| *** lucas-hungry is now known as lucasagomes | 13:32 | |
| *** milan has joined #openstack-ironic | 13:35 | |
| *** dansmith is now known as superdan | 13:39 | |
| *** hjensas has quit IRC | 13:46 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/499050 | 13:53 |
| *** dpawar has quit IRC | 13:55 | |
| *** d0ugal_ has joined #openstack-ironic | 13:56 | |
| *** d0ugal has quit IRC | 13:57 | |
| *** links has quit IRC | 13:58 | |
| *** marios has quit IRC | 13:59 | |
| *** marios has joined #openstack-ironic | 13:59 | |
| *** diga has joined #openstack-ironic | 14:02 | |
| *** cjloader has joined #openstack-ironic | 14:06 | |
| *** cjloader has quit IRC | 14:11 | |
| *** erolg has joined #openstack-ironic | 14:16 | |
| *** erolg_ has joined #openstack-ironic | 14:16 | |
| *** erolg_ has left #openstack-ironic | 14:17 | |
| TheJulia | good morning everyone! | 14:30 |
| jroll | morning TheJulia \o | 14:32 |
| *** sai-mtg is now known as sai-intrw | 14:36 | |
| *** dtantsur|brb is now known as dtantsur | 14:37 | |
| dtantsur | morning TheJulia | 14:37 |
| *** r-daneel has joined #openstack-ironic | 14:44 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: bifrost-create-vm-nodes: Install virtualbmc to virtualenv if enabled https://review.openstack.org/555818 | 14:48 |
| *** mjturek has quit IRC | 14:48 | |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 14:49 |
| -openstackstatus- NOTICE: zuul.o.o has been restarted to pick up latest code base and clear memory usage. Both check / gate queues were saved, be sure to check your patches and recheck when needed. | 14:51 | |
| *** zhangfei has joined #openstack-ironic | 14:53 | |
| TheJulia | dtantsur: any chance I can get you to take a look at 554725 ? | 14:55 |
| * dtantsur looks | 14:57 | |
| * TheJulia has been having zero luck getting our ci job being triggered in the nova gate to not just pip install python-ironicclient | 14:58 | |
| dtantsur | TheJulia: it looks good. but I feel that we keep inventing workarounds instead of just doing what we should really do... | 14:58 |
| TheJulia | fair point, because nobody wants to review thousands of lines of test changes | 14:59 |
| dtantsur | well, I refer to fixing ironicclient rather than dropping it from nova | 14:59 |
| dtantsur | but I realize that it may be even more lines of code | 14:59 |
| TheJulia | but https://review.openstack.org/#/c/554762/ is also fairly clean | 15:00 |
| patchbot | patch 554762 - nova - WIP: Add microversion to ironic client wrapper call | 15:00 |
| *** d0ugal__ has joined #openstack-ironic | 15:01 | |
| *** d0ugal_ has quit IRC | 15:01 | |
| TheJulia | and aligns with how the virt driver uses python-ironicclient :\ | 15:02 |
| *** d0ugal__ has quit IRC | 15:02 | |
| *** d0ugal has joined #openstack-ironic | 15:02 | |
| dtantsur | TheJulia: yep. the last question probably: are virt drivers single-thread (green thread)? | 15:03 |
| *** r-daneel has quit IRC | 15:04 | |
| dtantsur | meaning, if we're doing operations on two nodes at the same time, are there any chances they'll be racing on reset_microversion? | 15:04 |
| TheJulia | that is a super good question, to which I don't have an answer | 15:04 |
| dtantsur | TheJulia: shouldn't we just cache client per microversion: {'1.37': client1, '1.42': client2} ? | 15:04 |
| dtantsur | if we do that, we don't even need ironicclient changes | 15:04 |
| TheJulia | problem is, then we have to dump the entire client use model of the virt driver because it runs only a singular instance | 15:05 |
| dtantsur | not sure I get this concern, to be honest | 15:05 |
| TheJulia | presently, no matter what, the virt driver will only use a single python-ironicclient instance | 15:05 |
| dtantsur | currently we cache a client that is created from a constant input. so even if two threads race, they both will create a suitable client. | 15:05 |
| dtantsur | and then IIRC we don't modify it, so it should be fine to share it | 15:06 |
| dtantsur | if we do things like: self.cached_clients[microversion] = get_client(api_version=microversion), I think we will retain this property | 15:06 |
| dtantsur | because even if two threads will do the same thing, either of them will end up with a client suitable for both | 15:07 |
| TheJulia | except the initial cached copy already set the microversion to be used moving forward | 15:07 |
| TheJulia | so to us a different microverison, we would have to toss the cached copy and create a new one | 15:07 |
| dtantsur | well, in my suggestion there's no "main" cached copy | 15:07 |
| dtantsur | how many microversions are we practically going to use? I suspect we're talking about 2-3 copies of the client | 15:08 |
| TheJulia | oh, per microversion referenced... that seems like trying to perform too much magic | 15:08 |
| dtantsur | with one of them used constantly, the others only occasionally | 15:08 |
| TheJulia | but if it is threaded execution, we don't have a choice actually | 15:08 |
| * TheJulia will have to look at that | 15:08 | |
| *** hjensas has joined #openstack-ironic | 15:09 | |
| dtantsur | yeah, I don't know nova-compute's threading model | 15:09 |
| TheJulia | I think virt driver interactions end up out of the same thread, but yeah... | 15:09 |
| * TheJulia adds that to today's todo list | 15:11 | |
| * dtantsur leaves comments to ensure we check it | 15:12 | |
| *** zhangfei has quit IRC | 15:14 | |
| jroll | without double-checking, I think the n-cpu threading model is similar to our conductor, spawn a greenthread for each RCP message | 15:19 |
| *** itlinux has joined #openstack-ironic | 15:21 | |
| *** trown has quit IRC | 15:23 | |
| *** trown has joined #openstack-ironic | 15:24 | |
| *** jtomasek has quit IRC | 15:25 | |
| *** melwitt is now known as jgwentworth | 15:26 | |
| *** mjura has quit IRC | 15:28 | |
| NobodyCam | Good Morning Ironic'ers and ofc... | 15:33 |
| NobodyCam | TGIF! | 15:33 |
| TheJulia | good morning NobodyCam | 15:33 |
| NobodyCam | :) Good Morning TheJulia and happy Friday | 15:34 |
| *** links has joined #openstack-ironic | 15:40 | |
| *** gyee has joined #openstack-ironic | 15:41 | |
| radez | bfournie: hey you have a few mins? | 15:41 |
| diga | jroll: Hi | 15:45 |
| *** rpioso|afk is now known as rpioso | 15:46 | |
| bfournie | radez: sure, I have until a meeting at top of the hour, will contact you on separate channel | 15:46 |
| radez | bfournie: ok, I'll type fast :) | 15:46 |
| rpioso | Good Friday morning, ironicers | 15:46 |
| *** links has quit IRC | 15:46 | |
| *** cjloader has joined #openstack-ironic | 15:48 | |
| -openstackstatus- NOTICE: Gerrit will be temporarily unreachable as we restart it to complete the rename of some projects. | 15:49 | |
| *** cjloader has quit IRC | 15:52 | |
| *** diga has quit IRC | 15:52 | |
| *** cjloader has joined #openstack-ironic | 15:52 | |
| *** cjloader has quit IRC | 15:54 | |
| *** cjloader has joined #openstack-ironic | 15:55 | |
| dtantsur | morning NobodyCam, rpioso | 15:59 |
| *** gyankum has quit IRC | 16:00 | |
| NobodyCam | Good Morning dtantsur | 16:01 |
| NobodyCam | happy Friday | 16:01 |
| rpioso | dtantsur, NobodyCam: o/ | 16:02 |
| NobodyCam | hey hey rpioso :) good Morning | 16:02 |
| *** fatema_ has quit IRC | 16:04 | |
| *** jroll has quit IRC | 16:07 | |
| etingof | rpioso, hey, I'd appreciate your quick look at the latest developments here -- https://review.openstack.org/#/c/526773/ | 16:07 |
| patchbot | patch 526773 - ironic - Add boot mode support to ManagementInterface | 16:07 |
| rpioso | etingof: That's on my list for today. Thank you for following up :) | 16:08 |
| etingof | rpioso, awesome, thank you! ;) | 16:08 |
| *** jroll has joined #openstack-ironic | 16:10 | |
| *** AlexeyAbashkin has quit IRC | 16:14 | |
| *** rama_y has joined #openstack-ironic | 16:15 | |
| *** jroll has quit IRC | 16:15 | |
| *** jroll has joined #openstack-ironic | 16:16 | |
| *** harlowja has joined #openstack-ironic | 16:16 | |
| *** penick is now known as JunoMan | 16:22 | |
| dtantsur | TheJulia: I have a few backports https://review.openstack.org/#/q/project:openstack/python-ironic-inspector-client mind checking them please? | 16:23 |
| TheJulia | sure | 16:24 |
| etingof | TheJulia, https://review.openstack.org/#/c/553807/ | 16:24 |
| patchbot | patch 553807 - ironic-lib - Added `get_raw_image_mb()` | 16:24 |
| etingof | TheJulia, sambetts wrt ^ I'm wondering if the cluster compression ratio (which we generally do not know) can skew our heuristic raw image size measurements which is based on cluster count allocation? | 16:26 |
| TheJulia | dtantsur: done | 16:26 |
| * etingof needs to run | 16:26 | |
| dtantsur | thanks! | 16:26 |
| *** etingof is now known as etingof|afk | 16:26 | |
| TheJulia | etingof: thanks for the link, I'll take a look | 16:26 |
| TheJulia | etingof|afk: tl;dr yeah, sparesness pretty much destroys the ability to accurately return information :\ | 16:28 |
| *** cjloader has quit IRC | 16:29 | |
| *** cjloader has joined #openstack-ironic | 16:30 | |
| sambetts | this is the implemenation of qemu-img measure for qcow2 we might be able to learn something from that https://github.com/qemu/qemu/blob/4bdc24fa018901892bb8a5bd1808ebd605f4c64d/block/qcow2.c#L3805 | 16:31 |
| sambetts | particularly https://github.com/qemu/qemu/blob/4bdc24fa018901892bb8a5bd1808ebd605f4c64d/block/qcow2.c#L2660-L2687 | 16:32 |
| sambetts | which seems to be basically doing the cluster calculation that we added to that function | 16:33 |
| sambetts | if I'm understanding it correctly | 16:33 |
| TheJulia | etingof|afk: great work, I put some thoughts in | 16:37 |
| * TheJulia looks | 16:37 | |
| *** cjloader has quit IRC | 16:38 | |
| *** cjloader has joined #openstack-ironic | 16:38 | |
| TheJulia | I kind of feel like the only real way to provide a realistic guard it is to crack open the image and eval the contents since there can be some fun cases that exist, and I think converting the image will help shuffle cluster allocation of an original image which still might not account for what the image actually thinks... because that is actually the post deploy failure point potentially if the filesystem/system | 16:43 |
| TheJulia | thinks the disk is bigger than it really is. ideally with gpt we would still fail during deploy, but I don't think we would with mbr style tables and a whole disk iamge if there is not a config drive sent, or even if there was but a pre-existing partition | 16:43 |
| *** vabada has quit IRC | 16:48 | |
| sambetts | my concern with the whole approuch is that if someone has an image where their filesystem is sized to 500gb but they havn't used much of it so the image is sparse so we can write it into a smaller disk, the image is still expecting a 500gb file system | 16:52 |
| *** bnemec is now known as beekneemech | 16:54 | |
| *** lucasagomes is now known as lucas-afk | 16:57 | |
| *** cjloader has quit IRC | 16:58 | |
| *** cjloader has joined #openstack-ironic | 16:59 | |
| *** cjloader has quit IRC | 17:02 | |
| *** cjloader has joined #openstack-ironic | 17:03 | |
| *** athomas_ has quit IRC | 17:03 | |
| *** masber has quit IRC | 17:06 | |
| TheJulia | sambetts: my eact concern just stated with actual numbers :) | 17:10 |
| TheJulia | exact | 17:10 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| dtantsur|afk | o/ | 17:16 |
| sambetts | dtantsur|afk: have good weekend, rest up and feel better | 17:17 |
| TheJulia | o/ | 17:17 |
| *** cjloader has quit IRC | 17:17 | |
| *** masber has joined #openstack-ironic | 17:18 | |
| *** e0ne has quit IRC | 17:22 | |
| *** pcaruana has quit IRC | 17:23 | |
| *** itlinux has quit IRC | 17:27 | |
| *** beekneemech has quit IRC | 17:29 | |
| *** marios has quit IRC | 17:30 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/queens: Use the regular inspector tests in the CI https://review.openstack.org/555374 | 17:31 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/pike: Use the regular inspector tests in the CI https://review.openstack.org/555375 | 17:31 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/551624 | 17:31 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client stable/ocata: Use the regular inspector tests in the CI https://review.openstack.org/555376 | 17:31 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Update links in README https://review.openstack.org/551707 | 17:31 |
| *** bnemec has joined #openstack-ironic | 17:32 | |
| *** bnemec is now known as beekneemech | 17:33 | |
| *** beekneemech has quit IRC | 17:34 | |
| *** bnemec has joined #openstack-ironic | 17:34 | |
| *** milan has quit IRC | 17:36 | |
| *** dprince has quit IRC | 17:36 | |
| *** masber has quit IRC | 17:37 | |
| *** mgoddard has quit IRC | 17:40 | |
| *** trown has quit IRC | 17:40 | |
| *** derekh has quit IRC | 17:46 | |
| *** harlowja has quit IRC | 17:50 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:53 | |
| *** erolg has quit IRC | 17:54 | |
| *** AlexeyAbashkin has quit IRC | 17:57 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:58 | |
| *** baha has joined #openstack-ironic | 18:00 | |
| *** mjturek has joined #openstack-ironic | 18:01 | |
| *** AlexeyAbashkin has quit IRC | 18:03 | |
| *** cjloader has joined #openstack-ironic | 18:06 | |
| *** cjloader has quit IRC | 18:10 | |
| *** slagle has quit IRC | 18:15 | |
| * sambetts --> home | 18:16 | |
| *** sambetts is now known as sambetts|afk | 18:16 | |
| *** itlinux has joined #openstack-ironic | 18:32 | |
| *** harlowja has joined #openstack-ironic | 18:33 | |
| *** r-daneel has joined #openstack-ironic | 18:37 | |
| *** harlowja has quit IRC | 18:38 | |
| *** itlinux has quit IRC | 18:47 | |
| *** itlinux has joined #openstack-ironic | 18:50 | |
| *** r-daneel_ has joined #openstack-ironic | 18:50 | |
| *** r-daneel has quit IRC | 18:52 | |
| *** r-daneel_ is now known as r-daneel | 18:52 | |
| *** jesusaur has quit IRC | 19:05 | |
| *** itlinux has quit IRC | 19:08 | |
| *** harlowja has joined #openstack-ironic | 19:09 | |
| *** athomas has quit IRC | 19:10 | |
| *** jesusaur has joined #openstack-ironic | 19:13 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:14 | |
| *** itlinux has joined #openstack-ironic | 19:15 | |
| *** itlinux has quit IRC | 19:16 | |
| *** AlexeyAbashkin has quit IRC | 19:19 | |
| *** openstackgerrit has quit IRC | 19:34 | |
| *** etingof|afk is now known as etingof | 19:51 | |
| etingof | TheJulia, sambetts|afk the brute-force approach could be to crack-open the image and write it to /dev/null counting how many bytes/blocks has been written | 19:52 |
| *** fragatina has quit IRC | 19:52 | |
| etingof | may be that should better be some block device of a black hole kind... | 19:54 |
| *** patriciadomin has joined #openstack-ironic | 20:03 | |
| *** openstackgerrit has joined #openstack-ironic | 20:04 | |
| openstackgerrit | Doug Hellmann proposed openstack/networking-baremetal master: add lower-constraints job https://review.openstack.org/555922 | 20:04 |
| TheJulia | etingof: well, even doing that, you can have sections of data that is being tracked as zeroed out in the file, and qemu-ing will seek to later in the target block device. | 20:06 |
| *** itlinux has joined #openstack-ironic | 20:07 | |
| *** itlinux has quit IRC | 20:09 | |
| *** jroll has quit IRC | 20:09 | |
| *** tesseract has quit IRC | 20:09 | |
| *** jroll has joined #openstack-ironic | 20:09 | |
| *** r-daneel has quit IRC | 20:11 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:14 | |
| *** AlexeyAbashkin has quit IRC | 20:18 | |
| *** r-daneel has joined #openstack-ironic | 20:29 | |
| *** bfournie has quit IRC | 20:45 | |
| *** bfournie has joined #openstack-ironic | 20:46 | |
| *** bfournie has quit IRC | 20:50 | |
| *** livelace-link has joined #openstack-ironic | 20:58 | |
| *** jesusaur has quit IRC | 21:05 | |
| *** jesusaur has joined #openstack-ironic | 21:06 | |
| *** cjloader has joined #openstack-ironic | 21:09 | |
| *** fragatina has joined #openstack-ironic | 21:11 | |
| *** cjloader has quit IRC | 21:14 | |
| *** bnemec is now known as beekneemech | 21:32 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/555585 | 21:32 |
| *** jesusaur has quit IRC | 21:36 | |
| *** baha has quit IRC | 21:52 | |
| *** jlabarre has quit IRC | 21:59 | |
| TheJulia | sweet http://logs.openstack.org/50/499050/12/check/ironic-dsvm-standalone/e6f6a90/logs/screen-ir-cond.txt.gz#_Mar_23_15_45_55_460127 | 22:02 |
| openstackgerrit | Merged openstack/bifrost master: add lower-constraints job https://review.openstack.org/555351 | 22:04 |
| openstackgerrit | Merged openstack/bifrost master: Use endpoint rather than url in clouds.yaml with no keystone https://review.openstack.org/550573 | 22:04 |
| *** e0ne has joined #openstack-ironic | 22:05 | |
| TheJulia | looks like we're broken hard against ansible 2.5 | 22:06 |
| TheJulia | I'm going to go grab some dinner and take a deeper log | 22:11 |
| TheJulia | look | 22:11 |
| openstackgerrit | Merged openstack/sushy master: Mark Systems/Managers/SessionService optional https://review.openstack.org/552817 | 22:13 |
| *** srart has quit IRC | 22:16 | |
| openstackgerrit | Merged openstack/sushy master: Fix wrong message of invalid boot "enabled" parameter https://review.openstack.org/549503 | 22:24 |
| openstackgerrit | Merged openstack/sushy master: Add system status field https://review.openstack.org/550669 | 22:24 |
| *** Goneri has quit IRC | 22:25 | |
| *** figleaf is now known as edleafe | 22:34 | |
| *** EmilienM has quit IRC | 22:34 | |
| *** EmilienM has joined #openstack-ironic | 22:36 | |
| *** EmilienM has quit IRC | 22:36 | |
| *** EmilienM has joined #openstack-ironic | 22:36 | |
| *** slagle has joined #openstack-ironic | 22:37 | |
| *** r-daneel has quit IRC | 22:43 | |
| *** rama_y has quit IRC | 22:46 | |
| *** rama_y has joined #openstack-ironic | 22:55 | |
| *** e0ne has quit IRC | 23:02 | |
| *** srart has joined #openstack-ironic | 23:09 | |
| *** srart has quit IRC | 23:12 | |
| *** srart has joined #openstack-ironic | 23:12 | |
| *** rpioso is now known as rpioso|afk | 23:24 | |
| rpioso|afk | Nite all. Have a great weekend! | 23:24 |
| *** jesusaur has joined #openstack-ironic | 23:24 | |
| *** jesusaur has quit IRC | 23:42 | |
| *** jesusaur has joined #openstack-ironic | 23:43 | |
| *** jgwentworth is now known as melwitt | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!