*** rloo has quit IRC | 00:00 | |
*** itlinux has joined #openstack-ironic | 00:23 | |
*** lifeless_ has joined #openstack-ironic | 00:46 | |
*** lifeless has quit IRC | 00:46 | |
*** itlinux has quit IRC | 00:51 | |
*** dpawar has joined #openstack-ironic | 00:52 | |
*** phuongnh has joined #openstack-ironic | 00:54 | |
*** tiendc has joined #openstack-ironic | 01:15 | |
*** jlabarre has quit IRC | 01:16 | |
*** dpawar has quit IRC | 01:32 | |
*** dpawar has joined #openstack-ironic | 01:33 | |
*** fragatina has quit IRC | 02:15 | |
*** fragatina has joined #openstack-ironic | 02:15 | |
*** fragatin_ has joined #openstack-ironic | 02:18 | |
*** fragatina has quit IRC | 02:19 | |
*** fragatin_ has quit IRC | 02:22 | |
*** hshiina has joined #openstack-ironic | 02:23 | |
*** r-daneel has joined #openstack-ironic | 02:32 | |
*** r-daneel_ has joined #openstack-ironic | 02:34 | |
*** r-daneel has quit IRC | 02:36 | |
*** r-daneel_ is now known as r-daneel | 02:36 | |
*** links has joined #openstack-ironic | 02:40 | |
*** itlinux has joined #openstack-ironic | 02:42 | |
*** yolanda has quit IRC | 02:44 | |
*** yolanda has joined #openstack-ironic | 02:45 | |
*** yolanda has quit IRC | 02:53 | |
*** yolanda has joined #openstack-ironic | 02:54 | |
openstackgerrit | Merged openstack/python-ironicclient master: [doc] Add 'openstack create' command to command reference https://review.openstack.org/552932 | 03:00 |
---|---|---|
openstackgerrit | Hironori Shiina proposed openstack/ironic-specs master: Remove unnecessary indents https://review.openstack.org/554406 | 03:02 |
*** dpawar has quit IRC | 03:07 | |
*** skazi has joined #openstack-ironic | 03:08 | |
*** dpawar has joined #openstack-ironic | 03:19 | |
*** stendulker has joined #openstack-ironic | 03:32 | |
*** dpawar has quit IRC | 03:45 | |
*** jaganathan_ has quit IRC | 03:49 | |
*** itlinux has quit IRC | 03:49 | |
*** jaganathan has joined #openstack-ironic | 03:49 | |
*** Nisha_Agarwal has joined #openstack-ironic | 04:04 | |
*** fragatina has joined #openstack-ironic | 04:14 | |
*** fragatina has quit IRC | 04:16 | |
*** fragatina has joined #openstack-ironic | 04:16 | |
*** dpawar has joined #openstack-ironic | 04:27 | |
*** Nisha_Agarwal has quit IRC | 04:47 | |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 04:48 |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 04:53 |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 04:53 |
*** Nisha_Agarwal has joined #openstack-ironic | 04:54 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: [WIP] Implement raid configuration for BM via irmc drive (OOB-1) https://review.openstack.org/512979 | 04:57 |
*** zshi has quit IRC | 04:59 | |
*** zshi has joined #openstack-ironic | 05:00 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Add RedfishInspect Interface https://review.openstack.org/449425 | 05:06 |
*** zshi has quit IRC | 05:09 | |
*** zshi has joined #openstack-ironic | 05:11 | |
*** dpawar has quit IRC | 05:38 | |
*** dpawar has joined #openstack-ironic | 05:39 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-specs master: Support power fault recovery https://review.openstack.org/553308 | 05:41 |
*** dpawar has quit IRC | 05:43 | |
*** OctopusZhang_ has joined #openstack-ironic | 05:44 | |
*** dpawar has joined #openstack-ironic | 05:53 | |
*** dpawar has quit IRC | 05:54 | |
*** fragatina has quit IRC | 05:58 | |
*** dpawar has joined #openstack-ironic | 06:05 | |
*** OctopusZhang has joined #openstack-ironic | 06:08 | |
*** Nisha_Agarwal has quit IRC | 06:09 | |
*** OctopusZhang has quit IRC | 06:09 | |
openstackgerrit | Lenny Verkhovsky proposed openstack/ironic master: Infiniband Port Configuration update[1] https://review.openstack.org/553400 | 06:12 |
*** masber has quit IRC | 06:12 | |
*** OctopusZhang_ has quit IRC | 06:12 | |
*** OctopusZhang has joined #openstack-ironic | 06:15 | |
openstackgerrit | Merged openstack/ironic-tempest-plugin master: Updated from global requirements https://review.openstack.org/553186 | 06:20 |
*** e0ne has joined #openstack-ironic | 06:21 | |
*** Nisha_Agarwal has joined #openstack-ironic | 06:25 | |
openstackgerrit | zenghui.shi proposed openstack/ironic master: BIOS Settings: Add RPC object https://review.openstack.org/511714 | 06:31 |
*** etingof has joined #openstack-ironic | 06:33 | |
*** OctopusZhang has quit IRC | 06:37 | |
*** masber has joined #openstack-ironic | 06:44 | |
*** masber has quit IRC | 06:47 | |
*** masber has joined #openstack-ironic | 06:47 | |
*** OctopusZhang has joined #openstack-ironic | 06:56 | |
*** marios has joined #openstack-ironic | 06:57 | |
*** OctopusZhang has quit IRC | 06:58 | |
*** mjura has joined #openstack-ironic | 07:00 | |
*** d0ugal has joined #openstack-ironic | 07:12 | |
*** d0ugal has quit IRC | 07:12 | |
*** d0ugal has joined #openstack-ironic | 07:12 | |
*** rbartal has joined #openstack-ironic | 07:15 | |
*** d0ugal has quit IRC | 07:21 | |
*** rcernin has quit IRC | 07:23 | |
*** rcernin has joined #openstack-ironic | 07:24 | |
*** rcernin has quit IRC | 07:24 | |
*** dpawar has quit IRC | 07:27 | |
*** dpawar has joined #openstack-ironic | 07:28 | |
*** pcaruana has joined #openstack-ironic | 07:34 | |
*** dpawar has quit IRC | 07:37 | |
openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: multiprocess server, ZMQ-based management cli tool https://review.openstack.org/488874 | 07:37 |
*** dpawar has joined #openstack-ironic | 07:40 | |
*** livelace has joined #openstack-ironic | 07:48 | |
*** d0ugal has joined #openstack-ironic | 07:48 | |
*** zhangfei has joined #openstack-ironic | 07:54 | |
*** tesseract has joined #openstack-ironic | 08:00 | |
*** namnh has joined #openstack-ironic | 08:01 | |
*** AlexeyAbashkin has joined #openstack-ironic | 08:04 | |
*** jtomasek has joined #openstack-ironic | 08:05 | |
pas-ha | morning ironic | 08:07 |
openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: multiprocess server, ZMQ-based management cli tool https://review.openstack.org/488874 | 08:07 |
etingof | o/ | 08:08 |
*** priteau has joined #openstack-ironic | 08:08 | |
*** livelace has quit IRC | 08:08 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: test_failure_to_write fails with root user https://review.openstack.org/548884 | 08:09 |
*** zhangfei has quit IRC | 08:19 | |
pas-ha | dtantsur|afk: please see my last comments to https://review.openstack.org/#/c/553736 , I think we'll miss some useful tests with this change | 08:22 |
patchbot | patch 553736 - ironic - Gate: run ironic tests in the regular multinode job | 08:22 |
*** fbouliane has quit IRC | 08:31 | |
*** fbouliane has joined #openstack-ironic | 08:33 | |
*** e0ne has quit IRC | 08:33 | |
*** zhangfei has joined #openstack-ironic | 08:48 | |
*** lucas-pto is now known as lucasagomes | 09:00 | |
*** d0ugal has quit IRC | 09:02 | |
*** hshiina has quit IRC | 09:05 | |
*** jcoufal has joined #openstack-ironic | 09:06 | |
*** racedo has quit IRC | 09:08 | |
*** racedo has joined #openstack-ironic | 09:11 | |
*** MattMan has quit IRC | 09:16 | |
*** MattMan has joined #openstack-ironic | 09:16 | |
*** dpawar has quit IRC | 09:23 | |
*** dpawar has joined #openstack-ironic | 09:31 | |
*** derekh has joined #openstack-ironic | 09:40 | |
*** mgoddard has joined #openstack-ironic | 09:54 | |
*** zhangfei has quit IRC | 09:54 | |
*** e0ne has joined #openstack-ironic | 09:55 | |
*** milan has joined #openstack-ironic | 09:57 | |
*** grafoo has joined #openstack-ironic | 09:59 | |
*** grafoo has left #openstack-ironic | 09:59 | |
*** namnh has quit IRC | 10:10 | |
*** zhangfei has joined #openstack-ironic | 10:17 | |
*** milan has quit IRC | 10:18 | |
*** zhangfei has quit IRC | 10:21 | |
*** Nisha_Agarwal has quit IRC | 10:31 | |
*** sambetts|afk is now known as sambetts | 10:36 | |
*** jcoufal_ has joined #openstack-ironic | 10:42 | |
*** masber has quit IRC | 10:43 | |
*** serlex has joined #openstack-ironic | 10:44 | |
*** jcoufal has quit IRC | 10:45 | |
vdrok | good morning ironic! | 10:51 |
*** tiendc has quit IRC | 10:53 | |
*** phuongnh has quit IRC | 10:56 | |
*** rcernin has joined #openstack-ironic | 11:02 | |
*** rcernin has quit IRC | 11:06 | |
*** masber has joined #openstack-ironic | 11:08 | |
*** masuberu has joined #openstack-ironic | 11:12 | |
*** stendulker_ has joined #openstack-ironic | 11:12 | |
*** milan has joined #openstack-ironic | 11:13 | |
*** masber has quit IRC | 11:14 | |
*** stendulker has quit IRC | 11:16 | |
*** zshi has quit IRC | 11:17 | |
*** zshi has joined #openstack-ironic | 11:18 | |
*** stendulker_ has quit IRC | 11:20 | |
*** pcaruana has quit IRC | 11:23 | |
*** bfournie has quit IRC | 11:28 | |
*** jcoufal has joined #openstack-ironic | 11:28 | |
*** jcoufal_ has quit IRC | 11:32 | |
*** dpawar has quit IRC | 11:49 | |
*** pcaruana has joined #openstack-ironic | 11:55 | |
*** racedo_ has joined #openstack-ironic | 11:58 | |
*** racedo has quit IRC | 11:58 | |
*** bfournie has joined #openstack-ironic | 12:00 | |
*** liuzz has quit IRC | 12:04 | |
*** liuzz has joined #openstack-ironic | 12:04 | |
*** lucasagomes is now known as lucas-hungry | 12:06 | |
jroll | morning everyone | 12:12 |
*** trown|outtypewww is now known as trown|ruck | 12:12 | |
vdrok | good morning jroll ! | 12:14 |
jroll | \o | 12:15 |
*** parlos has joined #openstack-ironic | 12:21 | |
*** dprince has joined #openstack-ironic | 12:30 | |
*** openstackgerrit has quit IRC | 12:33 | |
*** r-daneel has quit IRC | 12:41 | |
*** d0ugal has joined #openstack-ironic | 12:41 | |
*** d0ugal has quit IRC | 12:41 | |
*** d0ugal has joined #openstack-ironic | 12:41 | |
*** mjturek has joined #openstack-ironic | 12:46 | |
*** AlexeyAbashkin has quit IRC | 12:48 | |
*** cdearborn has joined #openstack-ironic | 12:49 | |
*** rloo has joined #openstack-ironic | 12:55 | |
*** baha has joined #openstack-ironic | 12:57 | |
*** Goneri has joined #openstack-ironic | 12:59 | |
*** rloo has quit IRC | 13:04 | |
*** rloo has joined #openstack-ironic | 13:04 | |
*** openstackgerrit has joined #openstack-ironic | 13:06 | |
openstackgerrit | Merged openstack/ironic-specs master: Follow up of inspect wait state spec https://review.openstack.org/553822 | 13:06 |
*** AlexeyAbashkin has joined #openstack-ironic | 13:10 | |
*** lucas-hungry is now known as lucasagomes | 13:10 | |
*** mjturek has quit IRC | 13:11 | |
*** fragatina has joined #openstack-ironic | 13:17 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added redfish boot mode management https://review.openstack.org/528314 | 13:27 |
*** r-daneel has joined #openstack-ironic | 13:32 | |
TheJulia | good morning everyone | 13:33 |
etingof | o/ | 13:33 |
*** r-daneel has quit IRC | 13:36 | |
*** jlabarre has joined #openstack-ironic | 13:43 | |
rloo | Happy first-day-of-spring morning yee fine folks in ironic-land. That means you, TheJulia, etingof and everyone else! | 13:45 |
etingof | happy Equinox indeed! ;) | 13:46 |
rloo | looks like the gate is happy today too... | 13:47 |
*** links has quit IRC | 13:51 | |
TheJulia | NobodyCam: When you get a chance: https://review.openstack.org/#/c/553036/ | 13:52 |
patchbot | patch 553036 - bifrost - Define ironic_db_password default | 13:52 |
*** rbudden has joined #openstack-ironic | 13:54 | |
*** dmellado has quit IRC | 13:55 | |
rloo | scratch that; i suspect it is still broken for pike; i'll update whiteboard | 13:56 |
yolanda | hi TheJulia , rloo , that's the ordered list for ironic patches about bios: https://etherpad.openstack.org/p/ironic-bios-ordered-patches | 14:00 |
rloo | thx yolanda. would you mind updating the etherpad https://etherpad.openstack.org/p/IronicWhiteBoard with it? | 14:00 |
rloo | yolanda: there's a section there for bios. | 14:00 |
rloo | yolanda: L177 | 14:01 |
yolanda | sure | 14:02 |
rloo | yolanda: are they all ready to be reviewed, or are you still working/coding? | 14:02 |
rloo | yolanda: please indicate status next to them so people know | 14:02 |
*** zhangfei has joined #openstack-ironic | 14:03 | |
yolanda | they are ready | 14:03 |
*** sai_ is now known as sai-mtg | 14:09 | |
rloo | thx yolanda! | 14:10 |
yolanda | you're welcome | 14:13 |
*** pcaruana has quit IRC | 14:21 | |
*** fragatina has quit IRC | 14:23 | |
*** pcaruana has joined #openstack-ironic | 14:28 | |
openstackgerrit | Akira Yoshiyama proposed openstack/virtualbmc master: Add Serial-over-LAN (SOL) support https://review.openstack.org/482853 | 14:34 |
NobodyCam | TheJulia: is there a bug to track the followup on 553036? | 14:36 |
NobodyCam | and Good Morning Ironic'ers | 14:37 |
*** d0ugal has quit IRC | 14:38 | |
TheJulia | NobodyCam: enotime | 14:40 |
NobodyCam | :p | 14:40 |
TheJulia | NobodyCam: and we're migrating to storyboard later this week, so it changes | 14:41 |
NobodyCam | and Good Morning | 14:41 |
vdrok | good morning TheJulia rloo and NobodyCam ! | 14:45 |
rloo | good morning NobodyCam and vdrok! | 14:46 |
NobodyCam | Good morning vdrok and rloo :) | 14:47 |
*** itlinux has joined #openstack-ironic | 14:49 | |
*** dmellado has joined #openstack-ironic | 14:54 | |
*** r-daneel has joined #openstack-ironic | 14:55 | |
openstackgerrit | Lenny Verkhovsky proposed openstack/ironic master: Infiniband Port Configuration update[1] https://review.openstack.org/553400 | 14:56 |
*** dmellado has quit IRC | 14:56 | |
*** mjturek has joined #openstack-ironic | 14:57 | |
*** dtantsur|afk is now known as dtantsur | 15:00 | |
dtantsur | hi all | 15:00 |
dtantsur | pas-ha: thanks, yep, I'll update the regex | 15:01 |
*** dmellado has joined #openstack-ironic | 15:01 | |
*** jistr is now known as jistr|mtg | 15:01 | |
*** dprince has quit IRC | 15:03 | |
rloo | hi dtantsur. oh... dtantsur, TheJulia, jroll: did we still want to talk about nova-ironic-driver & REST API? | 15:03 |
rloo | I know that TheJulia and jroll discussed it yesterday | 15:03 |
*** dmellado has quit IRC | 15:04 | |
jroll | the conclusion was julia was going to investigate adding per-call microversioning to ironicclient and using that today | 15:04 |
* jroll doesn't have anything to add today | 15:05 | |
*** dmellado has joined #openstack-ironic | 15:05 | |
openstackgerrit | Merged openstack/bifrost master: Define ironic_db_password default https://review.openstack.org/553036 | 15:07 |
* dtantsur is fine with the conclusion | 15:09 | |
rloo | jroll: thx. per-call microversioning in client seems like a win, since it'll be useful for others? | 15:09 |
jroll | rloo: I agree | 15:09 |
TheJulia | It is a compromise to avoid bulk test re-writing. On a plus side, those tests can eventually be rewritten still | 15:09 |
*** d0ugal has joined #openstack-ironic | 15:09 | |
rloo | TheJulia: what is the 'right' solution? | 15:10 |
rloo | TheJulia: or 'ideal' maybe is what i mean | 15:10 |
rloo | TheJulia: 'magic' isn't an answer | 15:10 |
TheJulia | There is no solution that will bring universal happiness | 15:10 |
rloo | hmm, spoken like a politician. err PTL. | 15:10 |
TheJulia | I've had to learn fast | 15:10 |
rloo | quick learner :D | 15:11 |
jroll | well, I don't think anyone is against the pure rest thing. just gotta be realistic about what we can get done in one cycle. | 15:11 |
TheJulia | the problem with the rest thing it the way the tests are written, it will be a massive patch | 15:11 |
TheJulia | now, if they are willing to just trust and merge for us and we keep it to unit tests... that is different, but we're likely talking in excess of 4k lines | 15:11 |
rloo | ok, seems like a win to make the change to the client code. others can use it, and we don't have to muck with the tests. | 15:12 |
rloo | would be good to get some 'confirmation' that other openstack services are going in this direction. is it worth checking with api-sig, or maybe this is a bit outside what they do. | 15:12 |
rloo | not that they are unopinionated :) | 15:13 |
TheJulia | The impression I got from some discussions in dublin was more rest based, our tests are just not in great shape in nova | 15:14 |
dtantsur | should I point folks to my proposed guideline? :) | 15:14 |
rloo | did we vote for Stein release or is jroll just hoping... ? | 15:14 |
TheJulia | dtantsur: and the meeting notes where the consensus was to get rid of the client libraries? | 15:14 |
dtantsur | TheJulia: I haven't been a part of such consensus | 15:15 |
rloo | huh? get rid of client libraries? | 15:15 |
jroll | rloo: heh, I am just using it as a placeholder | 15:15 |
TheJulia | Yeah, there has been chatter about that too | 15:15 |
rloo | jroll: heh. i voted for stein... thought maybe i won for once! | 15:15 |
dtantsur | what we ended up talking about is that our client libraries need to decide: are they just language bindings or full-featured SDKs | 15:15 |
dtantsur | if the former, make them as tiny as possible (and here's where people might want to just use keystoneauth instead) | 15:16 |
*** dmellado has quit IRC | 15:16 | |
dtantsur | if the latter, make them less low-level by NOT exposing things like microversions so shamelessly | 15:16 |
rloo | dtantsur: ah. so, what's wrong with two different client libraries! and the universe will be happy :D | 15:16 |
dtantsur | heh | 15:17 |
dtantsur | we only have very short notes at https://etherpad.openstack.org/p/api-sig-ptg-rocky unfortunately | 15:17 |
TheJulia | rloo: see line about no one thing making everyone happy :) | 15:17 |
dtantsur | and ironicclient is much closer to a language binding right now, we should just set the expectations right | 15:17 |
TheJulia | although we also have a lot of logic under the hood to support things like versioning | 15:18 |
rloo | TheJulia: that's ok. what <fill in the blank> will make everyone happy? holidays maybe... | 15:18 |
TheJulia | +1 | 15:18 |
dtantsur | TheJulia: right, this is where we went the full SDK way all of a suddent | 15:18 |
*** zhangfei has quit IRC | 15:18 | |
dtantsur | well, because we realized that using microversions is hard.. | 15:18 |
TheJulia | not just hard, but prior design decisions complicate future directions | 15:19 |
rloo | ain't that the truth... | 15:19 |
dtantsur | yep | 15:20 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/queens: Switch documentation to hardware types https://review.openstack.org/554586 | 15:21 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Gate: run ironic tests in the regular multinode job https://review.openstack.org/553736 | 15:26 |
dtantsur | pas-ha: let's try this ^^^ | 15:26 |
* TheJulia gets out the popcorn | 15:30 | |
*** jistr|mtg is now known as jistr | 15:36 | |
dtantsur | folks, we need moar reviews on https://review.openstack.org/#/c/552817/ it's relatively simple | 15:39 |
patchbot | patch 552817 - sushy - Mark Systems/Managers/SessionService optional | 15:39 |
TheJulia | I just +2'ed it, it was already in my review window | 15:40 |
*** fragatina has joined #openstack-ironic | 15:45 | |
*** jlabarre has quit IRC | 15:51 | |
*** rbartal has quit IRC | 15:59 | |
*** mjura has quit IRC | 16:07 | |
rloo | johnthetubaguy: if you're around, I don't know what you mean by: 'I like the idea of the API being able to say if the IPA ramdisk image copy is going to happen (or has happened) or isn't relevant for that type of build" (wrt https://review.openstack.org/#/c/549493/) | 16:12 |
patchbot | patch 549493 - ironic-specs - Deployment steps framework | 16:12 |
johnthetubaguy | rloo: I was thinking about boot from volume, it would be nice to know which bits of the build process are skipped | 16:12 |
rloo | johnthetubaguy: oh, do you mean via the driver_internal_info['deploy_steps'] ? | 16:12 |
rloo | johnthetubaguy: so if ^^, you can see the list of deploy steps. it won't explicitly tell you which ones are not being done. | 16:13 |
rloo | johnthetubaguy: were you thinking of a fancier API? | 16:14 |
*** serlex has quit IRC | 16:15 | |
*** masuberu has quit IRC | 16:18 | |
johnthetubaguy | rloo: yeah, I was thinking deploy_steps would be missing the ipa_ramdisk pull down image from glance step? | 16:20 |
rloo | johnthetubaguy: got it. yup, if you knew about that deploy step, you'd know it wasn't being done :) | 16:21 |
rloo | johnthetubaguy: in the future, if we ever do an API like 'get-avail-deploy-steps', you could do a diff to see what is avail vs what is actually being done. but don't hold your breath just yet. | 16:22 |
*** gyee has joined #openstack-ironic | 16:26 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: Deployment steps framework https://review.openstack.org/549493 | 16:27 |
johnthetubaguy | rloo: cool, yeah, that bit would be extra | 16:30 |
rloo | johnthetubaguy: :) | 16:30 |
*** marios has quit IRC | 16:34 | |
openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: multiprocess server, ZMQ-based management cli tool https://review.openstack.org/488874 | 16:42 |
TheJulia | etingof: ^^^ preemtive rebase since it was about to merge conflict | 16:42 |
*** trown|ruck is now known as trown|lunch | 16:42 | |
etingof | TheJulia, so it's rebased on master by now | 16:43 |
TheJulia | a requirement changes the cliff line in it reversing the order | 16:43 |
*** dprince has joined #openstack-ironic | 16:44 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic stable/pike: Change pxe dhcp options name to codes. https://review.openstack.org/525248 | 16:47 |
etingof | TheJulia, yep, thanks! not many people have guts to review it, apparently... ;) | 16:47 |
jroll | I just can't bring myself to review it further than "why?" | 16:48 |
jroll | (to be quite honest) | 16:48 |
* rloo doesn't even want to know what you're all talking about :D | 16:48 | |
jroll | etingof: wait, does this add a hard dependency on zmq? | 16:48 |
etingof | jroll, yes, the client-server communication is now being done through zmq | 16:49 |
jroll | that seems... overkill | 16:49 |
* jroll suspects people are now using this in production, this makes me sad | 16:50 | |
etingof | well, in gerrit one could dig up an opposite opinion ;-) | 16:51 |
etingof | dtantsur, ^ | 16:51 |
etingof | jroll, previous take on this has been based on python multiprocessing for IPC | 16:51 |
jroll | I realize I can leave a review, I guess I thought it might be worth discussing the use cases quickly before I do so | 16:52 |
* jroll realizes zmq doesn't require a central broker, interesting | 16:52 | |
* etingof hmm, either jroll is brave or the patch is not reasonably scary yet | 16:53 | |
etingof | zmq is cool indeed | 16:53 |
*** AlexeyAbashkin has quit IRC | 16:53 | |
jroll | don't worry, I still haven't read the code :P | 16:55 |
*** cjloader_ has joined #openstack-ironic | 16:55 | |
*** cjloader_ is now known as cjloader | 16:56 | |
jroll | I just don't see the point ¯\_(ツ)_/¯ | 16:56 |
etingof | jroll, check out the discussion on gerrit | 16:57 |
TheJulia | etingof: looks fairly strait forward, I have two concerns thus far, but still reviewing | 16:58 |
jroll | etingof: that doesn't help, I think the premise is broken | 16:58 |
jroll | it's fine, I won't block it, I just don't think the use case is one we should support :) | 16:59 |
*** masuberu has joined #openstack-ironic | 16:59 | |
etingof | jroll, the original premise I think was that zmq moves the IPC complexities away from the app | 17:00 |
etingof | jroll, the argument can be knocked down though | 17:01 |
dtantsur | jroll: I asked for zmq, because I did not want custom low-level networking code there, to be honest | 17:02 |
*** milan has quit IRC | 17:02 | |
dtantsur | I have quite some argument with zmq, and it does tend to hide a lot of unpleasant details | 17:02 |
etingof | the good new is that we now have both implementations to choose from ;) | 17:02 |
dtantsur | I'm open to other networking libraries | 17:02 |
jroll | I can live with zmq, I'm talking about the need for managing virtualbmc, having it live across reboots, etc | 17:02 |
* dtantsur prepares a loooong rant | 17:02 | |
jroll | this feels like trying to make this production ready, which implies people using it for production | 17:02 |
dtantsur | jroll: *some people* use vbmc in production | 17:03 |
* dtantsur cough-cough | 17:03 | |
*** lucasagomes is now known as lucas-afk | 17:03 | |
jroll | I don't think that's something we should make an effort to support | 17:03 |
* etingof meanwhile hopes not to become an expert in Python IPC libs | 17:03 | |
dtantsur | jroll: well, to be entirely true, the current vbmc is quite fragile, and has problems even with long-running test environments | 17:03 |
etingof | not anymore! ^^ | 17:04 |
dtantsur | we have a bug saying "vbmc sometimes just dies" or something like that | 17:04 |
etingof | (once we merge) | 17:04 |
dtantsur | since it's not a daemon, it's hard even to track why and when it died, to say nothing about restart | 17:04 |
jroll | ok, that's more fair | 17:04 |
dtantsur | and e.g. my testing environments tend to live for several weeks, so not repairing vbmc is a benefit for me | 17:04 |
dtantsur | still, my comment re production holds :( (yes, I know, I know..) | 17:05 |
*** mjturek has quit IRC | 17:05 | |
jroll | well, it isn't the ssh driver :) | 17:05 |
jroll | so, like I said, I won't block it, I'm not sure I can review it sanely | 17:06 |
*** mjturek has joined #openstack-ironic | 17:07 | |
TheJulia | etingof: reviewed | 17:08 |
etingof | thanks! | 17:08 |
openstackgerrit | Merged openstack/ironic-inspector master: test_failure_to_write fails with root user https://review.openstack.org/548884 | 17:11 |
*** bkero- has quit IRC | 17:11 | |
*** fragatina has quit IRC | 17:17 | |
openstackgerrit | Merged openstack/virtualbmc master: Updated from global requirements https://review.openstack.org/552411 | 17:18 |
*** Nisha_Agarwal has joined #openstack-ironic | 17:28 | |
*** priteau has quit IRC | 17:30 | |
*** masuberu has quit IRC | 17:33 | |
*** e0ne has quit IRC | 17:35 | |
openstackgerrit | Merged openstack/ironic master: Rework Bare Metal service overview in the install guide https://review.openstack.org/554284 | 17:36 |
*** baoli has joined #openstack-ironic | 17:39 | |
*** baoli has quit IRC | 17:40 | |
*** trown|lunch is now known as trown|ruck | 17:45 | |
*** jlabarre has joined #openstack-ironic | 17:47 | |
TheJulia | rpioso|afk: w/r/t dell 3rd party ci, looks like you guys still have a job running a classic driver, you may want to update that. It will likely may dtantsur's day. :) | 17:48 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Gate: run ironic tests in the regular multinode job https://review.openstack.org/553736 | 17:49 |
openstackgerrit | Merged openstack/ironic-specs master: Remove unnecessary indents https://review.openstack.org/554406 | 17:50 |
rpioso|afk | TheJulia: I’ll check with rajinir. Thanks for offering us an opportunity to make dtantsur’s day :) | 17:51 |
*** derekh has quit IRC | 17:52 | |
TheJulia | rpioso|afk: rajinir: also looking at that classic driver job, looks like it is only allowing for 600 seconds for the node build which may not be enough. It might be a good idea to check your CI job timeouts and extend them a little. :\ It might clean up the failure rate which seems higher for the classic driver run. | 17:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Prevent overwriting of last_error on cleaning failures https://review.openstack.org/546656 | 18:05 |
dtantsur | hmm, the HPE CI started reporting "merge failed" on seemingly valid patches | 18:08 |
*** mgoddard has quit IRC | 18:09 | |
TheJulia | dtantsur: like with-in the last hour or two? | 18:12 |
dtantsur | yep | 18:12 |
TheJulia | ugh | 18:12 |
* TheJulia just finished updating the whiteboard | 18:13 | |
* sambetts --> home | 18:13 | |
sambetts | Night all o/ | 18:13 |
*** sambetts is now known as sambetts|afk | 18:13 | |
TheJulia | Speaking of HPE, they have a quick spec update at https://review.openstack.org/#/c/530838/4 that seems like a quick review | 18:13 |
patchbot | patch 530838 - ironic-specs - Update OOB RAID for iLO5 based HPE Proliant Servers | 18:13 |
*** AlexeyAbashkin has joined #openstack-ironic | 18:14 | |
dtantsur | I'm a bit exhausted after a 15 minutes doctor visit ended up taking 2 hours | 18:14 |
dtantsur | but will try to get to it tomorrow | 18:14 |
*** AlexeyAbashkin has quit IRC | 18:14 | |
TheJulia | dtantsur: understood, been there as well :( | 18:15 |
TheJulia | Have a wonderful evening | 18:15 |
*** AlexeyAbashkin has joined #openstack-ironic | 18:15 | |
dtantsur | you too | 18:16 |
*** dtantsur is now known as dtantsur|afk | 18:16 | |
openstackgerrit | Merged openstack/ironic master: Infiniband Port Configuration update[1] https://review.openstack.org/553400 | 18:17 |
*** harlowja has joined #openstack-ironic | 18:23 | |
*** gyee has quit IRC | 18:30 | |
*** fragatina has joined #openstack-ironic | 18:35 | |
*** cjloader_ has joined #openstack-ironic | 18:41 | |
*** cjloader has quit IRC | 18:41 | |
*** AlexeyAbashkin has quit IRC | 18:44 | |
*** Nisha_Agarwal has quit IRC | 18:46 | |
* TheJulia steps away to eat some lunch | 18:46 | |
*** livelace has joined #openstack-ironic | 18:57 | |
*** AlexeyAbashkin has joined #openstack-ironic | 19:03 | |
*** cdearborn_ has joined #openstack-ironic | 19:27 | |
*** mjturek has quit IRC | 19:28 | |
*** cdearborn has quit IRC | 19:30 | |
*** etingof has quit IRC | 19:31 | |
*** livelace has quit IRC | 19:35 | |
*** mjturek has joined #openstack-ironic | 19:39 | |
*** AlexeyAbashkin has quit IRC | 19:51 | |
*** e0ne has joined #openstack-ironic | 19:53 | |
*** tesseract has quit IRC | 19:56 | |
*** rama_y has joined #openstack-ironic | 19:58 | |
*** masber has joined #openstack-ironic | 20:02 | |
*** dprince has quit IRC | 20:25 | |
rajinir | TheJuloa: By classic driver do you mean the pxe_drac? | 20:26 |
*** milan has joined #openstack-ironic | 20:26 | |
*** priteau has joined #openstack-ironic | 20:29 | |
*** masber has quit IRC | 20:32 | |
*** openstackgerrit has quit IRC | 20:33 | |
*** dmellado has joined #openstack-ironic | 20:56 | |
*** dmellado has quit IRC | 20:58 | |
*** trown|ruck is now known as trown|outtypewww | 21:00 | |
*** dmellado has joined #openstack-ironic | 21:02 | |
*** baha has quit IRC | 21:08 | |
*** dmellado has quit IRC | 21:08 | |
*** mjturek has quit IRC | 21:08 | |
*** cjloader has joined #openstack-ironic | 21:10 | |
*** jlabarre has quit IRC | 21:11 | |
*** dmellado has joined #openstack-ironic | 21:12 | |
*** dmellado has quit IRC | 21:13 | |
*** cjloader_ has quit IRC | 21:14 | |
*** dmellado has joined #openstack-ironic | 21:15 | |
*** cjloader has quit IRC | 21:15 | |
rloo | rajinir: yes, I think that's what TheJulia meant. classic drivers: see the table: https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html | 21:17 |
TheJulia | rajinir: you guys happen to run a pxe_ipmitool job | 21:17 |
*** AlexeyAbashkin has joined #openstack-ironic | 21:19 | |
*** jcoufal has quit IRC | 21:23 | |
*** AlexeyAbashkin has quit IRC | 21:23 | |
*** jcoufal has joined #openstack-ironic | 21:25 | |
*** cdearborn_ has quit IRC | 21:26 | |
*** itlinux has quit IRC | 21:31 | |
*** dmellado has quit IRC | 21:32 | |
*** e0ne has quit IRC | 21:33 | |
*** bfournie has quit IRC | 21:36 | |
*** milan has quit IRC | 21:49 | |
*** openstackgerrit has joined #openstack-ironic | 21:50 | |
openstackgerrit | melissaml proposed openstack/python-ironic-inspector-client master: Follow the new PTI for document build https://review.openstack.org/554716 | 21:50 |
*** jcoufal has quit IRC | 21:51 | |
*** cjloader has joined #openstack-ironic | 21:58 | |
*** cjloader has quit IRC | 22:02 | |
*** dmellado has joined #openstack-ironic | 22:04 | |
*** rcernin has joined #openstack-ironic | 22:13 | |
*** priteau has quit IRC | 22:15 | |
*** priteau has joined #openstack-ironic | 22:15 | |
*** slagle has quit IRC | 22:16 | |
*** slagle has joined #openstack-ironic | 22:17 | |
*** AlexeyAbashkin has joined #openstack-ironic | 22:19 | |
*** priteau has quit IRC | 22:20 | |
*** AlexeyAbashkin has quit IRC | 22:23 | |
*** ianychoi__ is now known as ianychoi | 22:32 | |
openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Raise exception for unsupported microversion https://review.openstack.org/532979 | 22:51 |
openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Provide a facility to reset the microersion https://review.openstack.org/554725 | 22:51 |
*** pmannidi has joined #openstack-ironic | 22:54 | |
*** harlowja has quit IRC | 22:57 | |
*** masber has joined #openstack-ironic | 23:03 | |
*** masuberu has joined #openstack-ironic | 23:04 | |
*** rloo has quit IRC | 23:07 | |
*** rloo has joined #openstack-ironic | 23:07 | |
*** masber has quit IRC | 23:08 | |
*** khyr0n has joined #openstack-ironic | 23:16 | |
*** AlexeyAbashkin has joined #openstack-ironic | 23:19 | |
*** etingof has joined #openstack-ironic | 23:22 | |
*** AlexeyAbashkin has quit IRC | 23:23 | |
rpioso|afk | rloo, TheJulia: Thank you! | 23:30 |
*** harlowja has joined #openstack-ironic | 23:49 | |
*** jlabarre has joined #openstack-ironic | 23:50 | |
*** rama_y_ has joined #openstack-ironic | 23:59 | |
*** rama_y has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!