| *** dsneddon has joined #openstack-ironic | 00:02 | |
| *** dsneddon has quit IRC | 00:06 | |
| *** gyee has quit IRC | 00:16 | |
| *** sthussey has quit IRC | 00:21 | |
| openstackgerrit | Candy Tsai proposed openstack/ironic-python-agent master: Rename build name according to README https://review.openstack.org/642515 | 00:33 |
|---|---|---|
| *** dsneddon has joined #openstack-ironic | 00:35 | |
| *** dsneddon has quit IRC | 00:49 | |
| *** whoami-rajat has joined #openstack-ironic | 01:12 | |
| *** dsneddon has joined #openstack-ironic | 01:15 | |
| *** dsneddon has quit IRC | 01:28 | |
| *** dsneddon has joined #openstack-ironic | 01:39 | |
| *** dsneddon has quit IRC | 01:55 | |
| *** dsneddon has joined #openstack-ironic | 02:00 | |
| *** openstackstatus has quit IRC | 02:22 | |
| *** openstack has joined #openstack-ironic | 02:24 | |
| *** ChanServ sets mode: +o openstack | 02:24 | |
| *** dsneddon has quit IRC | 03:05 | |
| *** dsneddon has joined #openstack-ironic | 03:23 | |
| *** MattMan_ has quit IRC | 03:24 | |
| *** MattMan_ has joined #openstack-ironic | 03:24 | |
| *** dsneddon has quit IRC | 03:28 | |
| *** trungnv has joined #openstack-ironic | 03:32 | |
| *** dsneddon has joined #openstack-ironic | 03:32 | |
| trungnv | TheJulia, Firmware update for iRMC is already to merge --- Please help me review on following patch: https://review.openstack.org/#/c/599992/ | 03:33 |
| patchbot | patch 599992 - ironic - Support firmware upgrade for irmc driver - 11 patch sets | 03:33 |
| *** dsneddon has quit IRC | 03:37 | |
| *** rh-jelabarre has quit IRC | 03:51 | |
| *** dsneddon has joined #openstack-ironic | 03:56 | |
| *** stendulker has joined #openstack-ironic | 04:16 | |
| *** dsneddon has quit IRC | 05:05 | |
| *** dsneddon has joined #openstack-ironic | 05:35 | |
| *** dsneddon has quit IRC | 05:39 | |
| *** dsneddon has joined #openstack-ironic | 05:42 | |
| *** jtomasek has joined #openstack-ironic | 05:58 | |
| *** mgoddard has quit IRC | 06:07 | |
| *** andreaf has quit IRC | 06:09 | |
| *** mgoddard has joined #openstack-ironic | 06:10 | |
| *** andreaf has joined #openstack-ironic | 06:12 | |
| *** e0ne has joined #openstack-ironic | 06:18 | |
| openstackgerrit | Digambar proposed openstack/ironic stable/ocata: Fix OOB introspection to use pxe_enabled flag in idrac driver https://review.openstack.org/641683 | 06:25 |
| openstackgerrit | Merged openstack/ironic master: Check microversions before validations for allocations and deploy templates https://review.openstack.org/642435 | 06:59 |
| *** Qianbiao has joined #openstack-ironic | 07:12 | |
| Qianbiao | Hello everyone. | 07:13 |
| *** Qianbiao is now known as Qianbiao|afk | 07:14 | |
| *** dsneddon has quit IRC | 07:22 | |
| arne_wiebalck | Good morning, ironic! | 07:25 |
| *** pcaruana has joined #openstack-ironic | 07:39 | |
| *** pcaruana has quit IRC | 07:43 | |
| *** pcaruana has joined #openstack-ironic | 07:43 | |
| openstackgerrit | Rachit Kapadia proposed openstack/ironic master: Set boot_mode in node properties during OOB Introspection https://review.openstack.org/639698 | 07:44 |
| *** dsneddon has joined #openstack-ironic | 07:49 | |
| *** dsneddon has quit IRC | 07:54 | |
| *** rpittau|afk is now known as rpittau | 07:59 | |
| rpittau | good morning ironic! o/ | 07:59 |
| arne_wiebalck | hey rpittau o/ | 08:00 |
| rpittau | hi arne_wiebalck :) | 08:01 |
| *** mkrai has joined #openstack-ironic | 08:03 | |
| *** Qianbiao|afk is now known as Qianbiao | 08:03 | |
| mkrai | Hi, can someone please explain the technical difference between flat and Neutron network interface in Ironic? | 08:06 |
| *** tssurya has joined #openstack-ironic | 08:08 | |
| *** e0ne has quit IRC | 08:17 | |
| *** sburette has joined #openstack-ironic | 08:17 | |
| *** e0ne has joined #openstack-ironic | 08:23 | |
| *** mkrai has quit IRC | 08:25 | |
| *** dsneddon has joined #openstack-ironic | 08:29 | |
| *** dougsz has joined #openstack-ironic | 08:31 | |
| *** iurygregory has joined #openstack-ironic | 08:35 | |
| iurygregory | morning o/ | 08:36 |
| *** e0ne has quit IRC | 08:39 | |
| mgoddard | morning ironic | 08:51 |
| *** dtantsur|afk is now known as dtantsur | 08:52 | |
| dtantsur | morning ironic | 08:52 |
| mgoddard | hi dtantsur | 08:52 |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: WIP: Support reapply with supplied introspection data https://review.openstack.org/639039 | 08:54 |
| arne_wiebalck | hey iurygregory mgoddard dtantsur o/ | 08:55 |
| iurygregory | o/ | 08:55 |
| mgoddard | hi arne_wiebalck iurygregory | 08:55 |
| rpittau | hi dtantsur :) | 08:57 |
| rpittau | hey mgoddard :) | 08:57 |
| *** priteau has joined #openstack-ironic | 09:02 | |
| *** hwoarang has quit IRC | 09:04 | |
| *** hwoarang has joined #openstack-ironic | 09:05 | |
| *** iurygregory has quit IRC | 09:06 | |
| *** iurygregory has joined #openstack-ironic | 09:07 | |
| Qianbiao | dtantsur I am applying a static public IP for log server. currently, the url is fixed to another CI log for now. | 09:07 |
| dtantsur | Qianbiao: well, we need to be able to see the logs from a run | 09:07 |
| Qianbiao | sure | 09:08 |
| Qianbiao | when static IP is ready, will link to real log server. | 09:08 |
| Qianbiao | currently, we only run pxe in standalone env. | 09:08 |
| *** mkrai has joined #openstack-ironic | 09:12 | |
| *** hwoarang has quit IRC | 09:13 | |
| *** hwoarang has joined #openstack-ironic | 09:15 | |
| *** lucasagomes has joined #openstack-ironic | 09:16 | |
| dtantsur | rpittau: if you have a second: https://review.openstack.org/#/c/642399/ | 09:18 |
| patchbot | patch 642399 - ironic-tempest-plugin - Fix deploy template tests on Rocky and earlier - 3 patch sets | 09:18 |
| dtantsur | needed to unbreak the stable CI | 09:18 |
| rpittau | unbreak? :D | 09:18 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Fix ansible lint and ubuntu deployment https://review.openstack.org/642423 | 09:19 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: Adding python 3.6 unit test https://review.openstack.org/640807 | 09:19 |
| * iurygregory wonders if python3 requires more time to let nodes reach manageable state http://logs.openstack.org/28/591828/4/check/sushy-tempest-ironic-partition-redfish-src/62a5762/job-output.txt.gz#_2019-03-11_16_55_22_096108 | 09:22 | |
| dtantsur | iurygregory: 2 minutes is more than enough, unless something is broken in the emulator /cc etingof | 09:23 |
| * etingof ran into the emulator bug last night | 09:24 | |
| iurygregory | bug with python3? XD | 09:24 |
| dtantsur | iurygregory: unbelievable, yeah? :) | 09:25 |
| etingof | iurygregory, http://logs.openstack.org/28/591828/4/check/sushy-tempest-ironic-partition-redfish-src/62a5762/controller/logs/screen-ir-cond.txt.gz?level=WARNING#_Mar_11_16_55_21_511892 | 09:25 |
| dtantsur | rpittau: FYI it seems that we'll have to disable stable/queens grenade. Pike is around EM, and it's broken even worse than it used to when you last checked. /cc TheJulia | 09:25 |
| iurygregory | dtantsur, i would say its a bug in devstack =P | 09:25 |
| dtantsur | NO WAY | 09:25 |
| etingof | iurygregory, so the emulator is down | 09:25 |
| iurygregory | bad emulator | 09:26 |
| iurygregory | emulator-- XD | 09:26 |
| etingof | but the bug I encountered is unrelated I think | 09:26 |
| iurygregory | I've put a recheck just to see how it goes | 09:27 |
| etingof | my bug surfaces when the client tries to list the systems - emulator returns HTTP 500 | 09:27 |
| etingof | iurygregory, yeah, we need to understand if the process never went up or crashed or firewalled (?) | 09:28 |
| iurygregory | etingof, i would say its strange the cause be firewall since python2 works.. | 09:29 |
| rpittau | dtantsur, sounds "good", any log I can have a look at in the meantime ? | 09:29 |
| dtantsur | rpittau: see the whiteboard | 09:30 |
| rpittau | dtantsur, ack | 09:30 |
| etingof | iurygregory, but everyone blames the firewall, you can't omit the usual suspect | 09:30 |
| iurygregory | i would blame if python2 fails too XD | 09:31 |
| dtantsur | rpittau: talking to tonyb on #openstack-stable | 09:31 |
| *** derekh has joined #openstack-ironic | 09:34 | |
| *** dsneddon has quit IRC | 09:35 | |
| mkrai | Hi, can someone please explain the technical difference between flat and Neutron network interface in Ironic? | 09:36 |
| dtantsur | mkrai: hi. the neutron one uses an ML2 driver to actually change ports on a switch. the flat one simply uses neutron to give out DHCP options on a flat network. | 09:37 |
| dtantsur | mkrai: details: https://docs.openstack.org/ironic/latest/admin/multitenancy.html | 09:37 |
| mkrai | dtantsur: so in flat network no neutron is involved? | 09:42 |
| dtantsur | mkrai: it is, but just for DHCP | 09:43 |
| mkrai | dtantsur: and how does the port binding works? | 09:44 |
| dtantsur | mkrai: could you please check the link above? it's a lot of information to convey via IRC conversation | 09:44 |
| dtantsur | if you mean the flat network, it's mostly a no-op | 09:44 |
| *** andrein has joined #openstack-ironic | 09:47 | |
| mkrai | dtantsur: I checked it already but didn't get very clear understanding | 09:48 |
| *** dsneddon has joined #openstack-ironic | 10:01 | |
| dtantsur | mgoddard, rpittau, any of you want to check https://review.openstack.org/#/c/635996/ before it gets approved? | 10:05 |
| patchbot | patch 635996 - ironic - fast tracked deployment support - 21 patch sets | 10:05 |
| rpittau | dtantsur, I was going to :) | 10:06 |
| dtantsur | oka | 10:06 |
| dtantsur | y | 10:06 |
| *** dsneddon has quit IRC | 10:06 | |
| rpittau | I need coffee first though :P | 10:07 |
| mgoddard | dtantsur: I'll take a look | 10:07 |
| dtantsur | cool :) then I'll leave it in your hands. | 10:07 |
| *** dougsz has left #openstack-ironic | 10:08 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 10:08 |
| rpittau | if any core has 1 minute, this is really quick: https://review.openstack.org/641080 | 10:15 |
| patchbot | patch 641080 - ironic-ui - Supporting all py3 environments with tox - 1 patch set | 10:15 |
| *** andrein has quit IRC | 10:15 | |
| *** andrein has joined #openstack-ironic | 10:16 | |
| *** e0ne has joined #openstack-ironic | 10:16 | |
| *** iurygregory has quit IRC | 10:29 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 10:29 |
| *** iurygregory has joined #openstack-ironic | 10:29 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/metalsmith master: [WIP] Accept hostname in reserve_node in addition to provision_node https://review.openstack.org/640397 | 10:30 |
| Qianbiao | stendulker thanks for the review, will fix it soon. | 10:32 |
| stendulker | Qianbiao: np | 10:33 |
| *** dsneddon has joined #openstack-ironic | 10:34 | |
| *** Qianbiao is now known as Qianbiao|afk | 10:34 | |
| openstackgerrit | Digambar proposed openstack/ironic stable/ocata: Fix OOB introspection to use pxe_enabled flag in idrac driver https://review.openstack.org/641683 | 10:35 |
| iurygregory | ocata =O | 10:36 |
| rpittau | oO | 10:36 |
| *** dsneddon has quit IRC | 10:39 | |
| dtantsur | heh | 10:43 |
| dtantsur | we keep it alive while we can without putting too much effort in it | 10:43 |
| *** stendulker has quit IRC | 10:45 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Fix deploy template tests on Rocky and earlier https://review.openstack.org/642399 | 10:50 |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Fix formatting issue in doc https://review.openstack.org/642719 | 10:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/rocky: Fix TypeError: __str__ returned non-string (type ImageRefValidationFailed) https://review.openstack.org/641584 | 10:52 |
| *** dsneddon has joined #openstack-ironic | 11:04 | |
| jroll | dtantsur: re http://logs.openstack.org/74/642474/5/check/ironicclient-tempest/a859932/controller/logs/screen-n-cpu.txt.gz#_Mar_11_16_04_40_215579 , I think now "ironic is not up" just looks different | 11:05 |
| dtantsur | ah | 11:06 |
| jroll | dtantsur | if I understood jroll correctly, the UEFI and the multinode jobs don't work on Bionic <- hm, multinode worked on bionic in patch set 1, maybe just a flake? | 11:07 |
| jroll | sorry, am slow with scrollback | 11:07 |
| dtantsur | jroll: or maybe I just misunderstood you :) | 11:07 |
| jroll | hotel computing :| | 11:07 |
| jroll | dtantsur: I guess. everything but uefi worked in PS1, so PS2 made uefi non-voting. | 11:08 |
| *** dsneddon has quit IRC | 11:10 | |
| Qianbiao|afk | Hello | 11:14 |
| Qianbiao|afk | is there any config item like power_state_change_timeout | 11:14 |
| Qianbiao|afk | not timeout but power state change times | 11:15 |
| openstackgerrit | Merged openstack/tenks master: Fix ansible lint and ubuntu deployment https://review.openstack.org/642423 | 11:31 |
| openstackgerrit | Merged openstack/tenks master: Adding python 3.6 unit test https://review.openstack.org/640807 | 11:31 |
| *** dsneddon has joined #openstack-ironic | 11:39 | |
| *** dsneddon has quit IRC | 11:44 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Ensure non-empty libvirt <loader/> tag https://review.openstack.org/642728 | 11:45 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Create libvirt boot loader element if needed https://review.openstack.org/642581 | 11:46 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 11:47 |
| dtantsur | folks, does anyone want a.. ehemmm.. funny side project? ;) | 11:51 |
| dtantsur | wsme is in dying state (and has been like that for time) | 11:51 |
| dtantsur | we need to switch off before it starts falling apart | 11:51 |
| *** rh-jelabarre has joined #openstack-ironic | 11:53 | |
| etingof | what's switch off exactly? | 11:55 |
| rpittau | so convert from wsme to pure wsgi ? | 11:57 |
| etingof | hmm, those are different things, no? | 11:57 |
| etingof | wsme is an early web services library impl in Python while wsgi is basically a calling convention | 11:59 |
| *** mkrai has quit IRC | 11:59 | |
| * etingof is guessing migrating ironic from wsme onto something... | 11:59 | |
| rpittau | yep, my brain did a weird automatic association for some reason :/ | 12:00 |
| * iurygregory saw the thread on ML | 12:00 | |
| iurygregory | maybe will be something we will need to do in the next cycle | 12:00 |
| *** dsneddon has joined #openstack-ironic | 12:00 | |
| *** iurygregory has quit IRC | 12:03 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as driver_info https://review.openstack.org/642402 | 12:03 |
| *** iurygregory has joined #openstack-ironic | 12:03 | |
| *** dsneddon has quit IRC | 12:05 | |
| jroll | dtantsur: I imagine we can just remove our wsme use and use our object layer directly (probably with some serialization code), do you agree? | 12:05 |
| etingof | wsme seems to do some type checking, not sure if that's important though | 12:07 |
| jroll | right, our object layer can do that | 12:08 |
| etingof | ORM you mean? | 12:08 |
| jroll | etingof: ironic/objects/* | 12:09 |
| jroll | the classes that inherit from oslo.versionedobjects | 12:09 |
| jroll | I'm hand-waving a bit, but I'm fairly sure that can handle it all | 12:09 |
| *** trown|outtypewww is now known as trown | 12:10 | |
| etingof | so why wsme appeared in the first place? does it date back prior to versionobjects? | 12:13 |
| jroll | I honestly don't know - I think it was just "the recommended thing" at the time | 12:13 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 12:16 |
| dtantsur | etingof: cargo-culted from nova? | 12:16 |
| dtantsur | jroll: yeah, I think it's possible with some additional checks on ID vs UUID | 12:16 |
| jroll | nova doesn't use wsme | 12:16 |
| jroll | (never did afaik) | 12:16 |
| jroll | dtantsur: yeah, should just need a small layer | 12:17 |
| * etingof could probably take a look unless anyone wants to | 12:18 | |
| jroll | I want to take this opportunity to rewrite the entire API layer, but I know I don't have the time :) | 12:19 |
| jroll | one thing I would like to look at is using the native o.vo remotable methods, rather than update_node etc via our rpc layer | 12:19 |
| jroll | our API should just be able to do node.foo = bar; node.save() and have that routed to the conductor | 12:20 |
| etingof | that seems to align with o.vo model | 12:21 |
| dtantsur | jroll: I wonder how it will work with JSON RPC backend though | 12:21 |
| dtantsur | etingof: feel free to. If I may suggest, I'd start with abstracting away explicit pecan.request.XXX that we use everywhere. | 12:22 |
| jroll | dtantsur: good question | 12:22 |
| dtantsur | like change pecan.request.version to something like api_utils.version() | 12:22 |
| jroll | wait, won't pecan remain? | 12:22 |
| jroll | or is pecan.request a wsme thing? | 12:22 |
| dtantsur | jroll: it may change with wsme gone.. | 12:22 |
| dtantsur | I'd personally kill pecan just as well, but maybe we have reasons to keep it.. | 12:23 |
| *** bfournie has quit IRC | 12:23 | |
| * dtantsur would just rewrite the whole thing in Flask | 12:23 | |
| jroll | my main reason to keep pecan is that it's much more work to remove :) | 12:23 |
| jroll | but yes, I'd much rather flask if we have the bandwidth | 12:23 |
| dtantsur | jroll, etingof, fun fact: I have a few bits for a Flask implementation ready in https://github.com/dtantsur/ironic-proxy/blob/master/ironic_proxy/api.py | 12:24 |
| * rpittau also votes for Flask | 12:24 | |
| jroll | ++ | 12:24 |
| dtantsur | I've added the topic for the PTG, but I wouldn't mind somebody already starting | 12:24 |
| iurygregory | flask++ | 12:24 |
| * etingof has heard that flask 1.0 is substantially different from flask 0.x | 12:25 | |
| dtantsur | dunno about "substantially" | 12:25 |
| dtantsur | I think ironic-inspector has migrated from <1.0 to >=1.0 quite painlessly | 12:26 |
| etingof | so people having their stuff based on flask 0.x have hard times converting to flask 1.x | 12:26 |
| * jroll won't be at PTG - I support replacing pecan/wsme with flask if we have the time. otherwise let's just remove wsme and have a small compatibility layer with our objects layer. | 12:26 | |
| rpittau | is Flask 0.x still maintained ? | 12:26 |
| dtantsur | if we keep pecan, the work seems quite easy | 12:28 |
| dtantsur | no routing changes, etc | 12:28 |
| dtantsur | should be pretty doable as a background task even | 12:28 |
| dtantsur | and we don't have to change everything at once | 12:28 |
| etingof | flask 1.x and 0.x seem to have latest releases in April 2018 | 12:29 |
| dtantsur | ditto for pecan | 12:31 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/ironic master: prevent serial console lockout https://review.openstack.org/642734 | 12:31 |
| dtantsur | pecan seems a one man show recently, but it may be just stable enough | 12:31 |
| larsks | jroll: here's my clever solution to the problem of console lockout: https://review.openstack.org/#/c/642734/ | 12:32 |
| patchbot | patch 642734 - ironic - prevent serial console lockout - 1 patch set | 12:32 |
| jroll | larsks: thanks, heavy meeting load this week but will try to take a look | 12:32 |
| *** dsneddon has joined #openstack-ironic | 12:33 | |
| larsks | Thanks. Sorry for the meetings. | 12:33 |
| jroll | nice idea though | 12:33 |
| jroll | heh, no worries | 12:33 |
| *** e0ne has quit IRC | 12:36 | |
| *** dsneddon has quit IRC | 12:39 | |
| *** iurygregory has quit IRC | 12:39 | |
| *** mjturek has joined #openstack-ironic | 12:40 | |
| *** iurygregory has joined #openstack-ironic | 12:40 | |
| *** mjturek has quit IRC | 12:45 | |
| dtantsur | mgoddard: I think we screwed up this again: http://logs.openstack.org/00/642400/1/check/ironic-tempest-dsvm-functional-python3/7bb4a5a/job-output.txt.gz#_2019-03-12_11_36_07_040683 | 12:47 |
| openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 12:47 |
| dtantsur | I wonder why pecan returns HTTP 405 instead of 404 for unknown routes | 12:47 |
| dtantsur | (speaking of pecan) | 12:47 |
| mgoddard | dtantsur: :( this one keeps coming back to bite | 12:48 |
| mgoddard | dtantsur: any way we can run tempest checks against stable branches? | 12:49 |
| mgoddard | for every tempest plugin patch | 12:49 |
| openstackgerrit | Merged openstack/ironic-ui master: Supporting all py3 environments with tox https://review.openstack.org/641080 | 12:50 |
| dtantsur | mgoddard: we used to, it got broken | 12:54 |
| dtantsur | mgoddard: we need to fix https://review.openstack.org/#/c/636169/ | 12:54 |
| patchbot | patch 636169 - ironic-tempest-plugin - Run all defined jobs defined in check and gate - 15 patch sets | 12:54 |
| *** baha has joined #openstack-ironic | 12:56 | |
| *** bfournie has joined #openstack-ironic | 12:57 | |
| arne_wiebalck | rpittau: I merged the py3 tox patch you asked for earlier | 13:00 |
| *** FlorianFa has quit IRC | 13:03 | |
| rpittau | arne_wiebalck, thanks! | 13:09 |
| openstackgerrit | Nikolay Fedotov proposed openstack/ironic master: Kg key for IPMIv2 authentication https://review.openstack.org/641658 | 13:11 |
| Qianbiao|afk | dtantsure CI report will be ready in this week. | 13:13 |
| dtantsur | cool | 13:13 |
| *** dsneddon has joined #openstack-ironic | 13:15 | |
| iurygregory | if someone find the issue with the standalone-queens and wholedisk we will be able to move =( I've try different configs but no lucky | 13:17 |
| Qianbiao|afk | <iurygregory> i am setting up CI env with standalone+(queue/rocky/stein)+wholedisk | 13:19 |
| Qianbiao|afk | and everything works. | 13:19 |
| Qianbiao|afk | what is your issue | 13:20 |
| *** cdearborn has joined #openstack-ironic | 13:20 | |
| *** dsneddon has quit IRC | 13:20 | |
| iurygregory | Qianbiao|afk, "Timeouts" basically XD | 13:20 |
| *** Qianbiao|afk is now known as Qianbiao | 13:20 | |
| iurygregory | Qianbiao, https://review.openstack.org/#/c/636169/ | 13:20 |
| patchbot | patch 636169 - ironic-tempest-plugin - Run all defined jobs defined in check and gate - 15 patch sets | 13:20 |
| openstackgerrit | Lars Kellogg-Stedman proposed openstack/ironic master: prevent serial console lockout https://review.openstack.org/642734 | 13:21 |
| *** sthussey has joined #openstack-ironic | 13:23 | |
| iurygregory | bbl need to fix things in the flat o/ | 13:23 |
| *** iurygregory has quit IRC | 13:24 | |
| *** cdearborn has quit IRC | 13:24 | |
| *** dsneddon has joined #openstack-ironic | 13:25 | |
| *** irclogbot_1 has quit IRC | 13:26 | |
| *** e0ne has joined #openstack-ironic | 13:28 | |
| *** irclogbot_1 has joined #openstack-ironic | 13:29 | |
| openstackgerrit | Varsha Verma proposed openstack/sushy master: Add `FabricCollection` and `Fabric` classes https://review.openstack.org/642748 | 13:29 |
| openstackgerrit | Varsha Verma proposed openstack/sushy master: Add `FabricCollection` and `Fabric` classes https://review.openstack.org/642748 | 13:31 |
| *** dsneddon has quit IRC | 13:32 | |
| *** Qianbiao has quit IRC | 13:43 | |
| dtantsur | mgoddard: so I wonder if we should fix ironic to return Method Not Allowed or fix the tests to accept both | 13:47 |
| dtantsur | we're quite inconsistent already: /v1/nodes/<>/traits returns 405, port groups - presumably 405/404 | 13:47 |
| dtantsur | god | 13:50 |
| dtantsur | we're returning 404 on GET and 405 on others | 13:50 |
| mgoddard | dtantsur: odd that the unit tests don't catch this | 13:55 |
| *** dsneddon has joined #openstack-ironic | 13:56 | |
| mgoddard | I guess it's a different case | 13:56 |
| dtantsur | mgoddard: unit tests verify our expectations, not how they match the default pecan logic (arguably flowed) | 13:56 |
| mgoddard | dtantsur: yeah, and we're not unit testing deploy templates in old versions | 13:56 |
| dtantsur | mgoddard: even if we did, we would test that it matches how we understand it.. | 13:57 |
| mgoddard | we can't change what old versions return for unknown endpoints (404 vs 405) | 13:57 |
| dtantsur | right | 13:57 |
| dtantsur | (I'm not sure if it's possible to change in pecan at all) | 13:57 |
| mgoddard | so do we need a new ironic with an old microversion to look exactly the same | 13:57 |
| mgoddard | we could probably check the method in _route and do a 405 if necessary? | 13:58 |
| dtantsur | mgoddard: we can try. but we'll need a 3-way dance, because ironic-tempest-plugin won't allow us to change 404 to 405 just easily | 13:58 |
| mgoddard | dtantsur: true | 13:58 |
| *** openstack has joined #openstack-ironic | 15:39 | |
| *** ChanServ sets mode: +o openstack | 15:39 | |
| TheJulia | mgoddard: typically yeah | 15:41 |
| mgoddard | TheJulia: ok, I'll go with that | 15:41 |
| *** dsneddon has joined #openstack-ironic | 15:41 | |
| mgoddard | TheJulia: thanks for responding, I'll take a look shortly | 15:43 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: API reference https://review.openstack.org/642498 | 15:43 |
| TheJulia | nothing really major, hopefully a little more clarity | 15:43 |
| *** iurygregory has joined #openstack-ironic | 15:43 | |
| iurygregory | I'm back o/ | 15:43 |
| TheJulia | I should be able to post a follow-up later today. One meeting and I need to do cycle highlights | 15:43 |
| *** diga has joined #openstack-ironic | 15:44 | |
| * TheJulia goes and does cycle highlights | 15:45 | |
| diga | Hi all | 15:45 |
| diga | I need help on this patch - https://review.openstack.org/#/c/641683/ | 15:45 |
| patchbot | patch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 4 patch sets | 15:45 |
| *** e0ne has quit IRC | 15:45 | |
| diga | This is a cherry pick patch backported to stable/ocata | 15:45 |
| diga | some template test cases are failing which I am not sure how I can address it | 15:46 |
| diga | As this is a cherry pick patch backported successfully for pick, queens and rocky release | 15:46 |
| diga | TheJulia, arne_wiebalck : any solution to resolve the issue ? | 15:47 |
| iurygregory | seems to be trying an API version that is not supported in the release | 15:52 |
| iurygregory | Version 1.54 was requested but the minor version is not supported by this service. The supported version range is: [1.1, 1.31] | 15:52 |
| iurygregory | maybe the patch is not backward compatible | 15:53 |
| diga | iurygregory: Yes | 15:53 |
| diga | iurygregory: what do I do now ? | 15:54 |
| iurygregory | diga, well i would sa | 15:54 |
| diga | iurygregory: is it not allowed to backport ? | 15:54 |
| iurygregory | i would say that the feature need to be re written so you can backport .-. | 15:54 |
| iurygregory | or maybe you need to squash some commits that is missing =) | 15:56 |
| dtantsur | TheJulia: great, thanks! | 15:56 |
| TheJulia | So that patch, at a glance, shouldn't be triggering that issue. I don't see an explicit invocation of version | 15:56 |
| diga | okay | 15:56 |
| dtantsur | diga, iurygregory, the CI is broken, mgoddard and I are fixing it | 15:56 |
| dtantsur | * stable CI | 15:57 |
| dtantsur | TheJulia: ^^ | 15:57 |
| iurygregory | now that makes sense hehe | 15:57 |
| iurygregory | XD | 15:57 |
| TheJulia | dtantsur: I was kind of suspecting that, thanks :) | 15:57 |
| dtantsur | TheJulia: https://review.openstack.org/#/c/642759/ | 15:57 |
| patchbot | patch 642759 - ironic-tempest-plugin - Deploy Templates: Allow old ironic API to return 405 - 1 patch set | 15:57 |
| dtantsur | this should unblock the CI, but we actually need to solve this 404 vs 405 thing | 15:57 |
| diga | dtantsur: :) Thank you | 15:57 |
| *** e0ne has joined #openstack-ironic | 15:58 | |
| diga | dtantsur: you mean I need to resolve templest-api issue failed in zuul | 15:58 |
| diga | ? | 15:59 |
| dtantsur | diga: yes. please wait for the above mentioned patch to merge before any rechecks | 15:59 |
| diga | dtantsur: okay | 15:59 |
| mgoddard | dtantsur: when you say 'convert to a local doctree', you mean put steps and templates in separate files with a toctree in node-deployment.rst linking to them? | 16:00 |
| dtantsur | mgoddard: no, you can create a toctree for local headings | 16:00 |
| dtantsur | I can try to google the required magic | 16:01 |
| openstackgerrit | Candy Tsai proposed openstack/ironic-python-agent master: Correct clean_build command according to Makefile https://review.openstack.org/642515 | 16:01 |
| diga | TheJulia: dtantsur: I will wait in that case | 16:01 |
| diga | Thanks for your help | 16:01 |
| mgoddard | dtantsur: it's ok, I'll try to work it out | 16:01 |
| dtantsur | mgoddard: okay, I was not exactly correct, but http://docutils.sourceforge.net/docs/ref/rst/directives.html#table-of-contents | 16:02 |
| diga | iurygregory: thank you! | 16:02 |
| iurygregory | diga, np | 16:03 |
| *** andrein has quit IRC | 16:05 | |
| mgoddard | dtantsur: I tried adding a pointer page, but I get 'document isn't included in any toctree'. Is there a way to avoid this? | 16:09 |
| dtantsur | mgoddard: yep, you can create a hidden toctree and add it there | 16:10 |
| dtantsur | which is a toctree with :hidden: | 16:10 |
| mgoddard | dtantsur: hmm | 16:11 |
| *** yajushiSri has quit IRC | 16:12 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fixing a bash test in devstack ironic lib https://review.openstack.org/642809 | 16:13 |
| TheJulia | I love it when meetings are quick | 16:16 |
| iurygregory | we all do TheJulia XD | 16:16 |
| *** yajushiSri has joined #openstack-ironic | 16:20 | |
| *** gyee has joined #openstack-ironic | 16:27 | |
| *** baha has quit IRC | 16:28 | |
| *** diga has quit IRC | 16:30 | |
| rpittau | interesting thing, virtual node trying to boot from ipxe even if explicitly set IRONIC_IPXE_ENABLED=False | 16:34 |
| *** tssurya has quit IRC | 16:40 | |
| *** hwoarang has quit IRC | 16:41 | |
| TheJulia | dtantsur: mgoddard: Since you guys are around, do we think http://paste.openstack.org/show/747628/ covers it for cycle highlights? | 16:42 |
| *** hwoarang has joined #openstack-ironic | 16:43 | |
| dtantsur | TheJulia: I'd split deploy templates into its own bullet point and expand on "node protection" | 16:43 |
| TheJulia | I'm trying to keep the list short since it is 2-3 items for executive review | 16:43 |
| dtantsur | s/file downloads/serving images/ | 16:43 |
| gyee | TheJulia, update on the redfish issue, confirmed with HPE that redfish is only supported on Gen10 so I'll give that a try today. | 16:43 |
| TheJulia | s/executive/marketing/ | 16:44 |
| TheJulia | (same difference I guess) | 16:44 |
| dtantsur | TheJulia: I don't suggest to increase the list, just make it differently structured :) | 16:44 |
| TheJulia | Well, it does say 3-4 in the email | 16:44 |
| dtantsur | TheJulia: or maybe we should skip deploy templates until we get at least one in-tree? | 16:44 |
| dtantsur | hmm | 16:44 |
| TheJulia | I have mixed feelings on that | 16:45 |
| * TheJulia tires to remember what she had mixed feelings on last cycle as well | 16:45 | |
| dtantsur | yeah, /me too | 16:45 |
| dtantsur | TheJulia: BIOS? | 16:45 |
| TheJulia | maybe, looking | 16:45 |
| dtantsur | I seem to remember it did not have any implementations | 16:45 |
| TheJulia | nah, we had 2 last cycle | 16:45 |
| dtantsur | wow | 16:45 |
| dtantsur | anyway, I think my biggest comments are 1. s/file downloads/serving images/, 2. s/node protection/deployed node protection/ | 16:46 |
| TheJulia | We mentioned them in Rocky | 16:46 |
| TheJulia | conductor groups I guess | 16:48 |
| TheJulia | Yeah, I think down play deployment templates a little bit. We want to wet appetite but not say "we've solved everything for you" | 16:55 |
| *** e0ne has quit IRC | 16:55 | |
| mgoddard | I think that's sensible :( | 16:57 |
| TheJulia | :( | 16:57 |
| mgoddard | here is your new swiss army knife. it doesn't have any tools yet | 16:58 |
| TheJulia | basically | 16:58 |
| * TheJulia has this vision of a movie that is funny, but chooses not to mention it | 16:58 | |
| TheJulia | dtantsur: mgoddard: https://review.openstack.org/642824 | 16:59 |
| patchbot | patch 642824 - releases - Add stein cycle highlights for ironic - 1 patch set | 16:59 |
| rpittau | good night o/ | 16:59 |
| *** rpittau is now known as rpittau|afk | 16:59 | |
| TheJulia | wait, there is night? | 16:59 |
| rpittau|afk | and it's dark and full of terrors..... | 17:00 |
| TheJulia | Not ironic bare metal bears? | 17:00 |
| *** baha has joined #openstack-ironic | 17:00 | |
| * TheJulia wonders what people in palm springs would say with her putting a pixie boots sticker on her car.... | 17:01 | |
| *** sburette has quit IRC | 17:02 | |
| TheJulia | dtantsur: revised | 17:04 |
| dtantsur | thnx! | 17:04 |
| TheJulia | and commented | 17:04 |
| TheJulia | Has anyone looked at the standalone job? | 17:07 |
| *** priteau has quit IRC | 17:08 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: documentation https://review.openstack.org/642531 | 17:09 |
| dtantsur | TheJulia: again? :( | 17:20 |
| *** trown is now known as trown|lunch | 17:20 | |
| TheJulia | yeah it seems | 17:20 |
| dtantsur | oh | 17:21 |
| dtantsur | TheJulia: on master? do you have a link to an example handy? | 17:21 |
| TheJulia | not handy, I just noticed it had failed on a couple patches and I think ricardo mentioned timeouts earlier | 17:21 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add docs on UEFI boot libvirt-based emulation https://review.openstack.org/642830 | 17:22 |
| dtantsur | TheJulia: it's still broken on Rocky until we merge https://review.openstack.org/642400 | 17:22 |
| patchbot | patch 642400 - ironic (stable/rocky) - devstack: use stable/rocky IPA for stable/rocky ir... - 1 patch set | 17:22 |
| dtantsur | which is blocked on mgoddard's ironic-tempest-plugin fix | 17:22 |
| TheJulia | sigh | 17:28 |
| *** dtantsur is now known as dtantsur|afk | 17:39 | |
| dtantsur|afk | \o | 17:39 |
| mgoddard | TheJulia: here is the aforementioned patch: https://review.openstack.org/#/c/642762/ | 17:43 |
| openstackgerrit | Lin Yang proposed openstack/sushy stable/stein: Fix wrong default JsonDataReader() argument https://review.openstack.org/642837 | 17:43 |
| patchbot | patch 642762 - ironic - Return 405 for old versions in allocation and depl... - 1 patch set | 17:43 |
| TheJulia | mgoddard: tempest fix? | 17:44 |
| mgoddard | TheJulia: https://review.openstack.org/642759 | 17:44 |
| patchbot | patch 642759 - ironic-tempest-plugin - Deploy Templates: Allow old ironic API to return 405 - 1 patch set | 17:44 |
| mgoddard | has +A already | 17:44 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Add fast-track testing capability https://review.openstack.org/637039 | 17:48 |
| TheJulia | mgoddard: dtantsur|afk: re fast track, do you guys want me to just re-rev the same patch or follow-up. There does seem to be a bug in one of the last requested chagnes that breaks it looking at tempest test log in some cases. Should be an easy fix, it is just yet another thing | 17:49 |
| mgoddard | TheJulia: just looking at your comments, I'll get back to you | 17:50 |
| TheJulia | k | 17:50 |
| *** derekh has quit IRC | 17:55 | |
| mgoddard | TheJulia: what was the bug that might need a respin? | 18:00 |
| TheJulia | basically ioerror on trying to switch pxe config | 18:00 |
| TheJulia | because we don't handle that stuff might be missing | 18:00 |
| TheJulia | it seems silly to even handle it actually, if we're fast tracking at least | 18:01 |
| TheJulia | but yeah | 18:01 |
| TheJulia | http://logs.openstack.org/04/641104/1/check/ironic-inspector-tempest-discovery-fast-track/fe1f1e4/controller/logs/screen-ir-cond.txt.gz | 18:01 |
| *** e0ne has joined #openstack-ironic | 18:02 | |
| *** e0ne has quit IRC | 18:06 | |
| mgoddard | TheJulia: we should probably fix that before merging. I'd be happy with a follow up for the rest | 18:09 |
| TheJulia | k, I'm trying to figure out the best way to actually fix that since its a legit issue | 18:09 |
| *** trown|lunch is now known as trown | 18:13 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy templates: conductor and API nits https://review.openstack.org/640446 | 18:17 |
| *** e0ne has joined #openstack-ironic | 18:18 | |
| *** e0ne_ has joined #openstack-ironic | 18:21 | |
| *** e0ne has quit IRC | 18:22 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy templates: conductor and API nits https://review.openstack.org/640446 | 18:23 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Deploy Templates: factor out ironic.conductor.steps https://review.openstack.org/642059 | 18:23 |
| TheJulia | oh jeeze, is it one of those "and to an or" days? | 18:26 |
| *** hwoarang has quit IRC | 18:26 | |
| *** gmann is now known as gmann_afk | 18:26 | |
| *** hwoarang has joined #openstack-ironic | 18:27 | |
| mgoddard | TheJulia: do they mean different things? | 18:28 |
| TheJulia | yeah | 18:28 |
| mgoddard | uh oh | 18:28 |
| TheJulia | indeed | 18:29 |
| openstackgerrit | Merged openstack/ironic-inspector master: Reset cached extension managers in the unit tests https://review.openstack.org/641990 | 18:31 |
| openstackgerrit | Merged openstack/ironic-inspector master: Exclude unrelevant files from tempest job https://review.openstack.org/640912 | 18:31 |
| openstackgerrit | Merged openstack/ironic-inspector master: Clean up release notes and add a prelude https://review.openstack.org/642513 | 18:31 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Deploy Templates: Allow old ironic API to return 405 https://review.openstack.org/642759 | 18:31 |
| *** pcaruana has quit IRC | 18:39 | |
| *** e0ne_ has quit IRC | 18:41 | |
| *** iurygregory has quit IRC | 18:52 | |
| *** betherly has joined #openstack-ironic | 19:32 | |
| *** betherly has quit IRC | 19:37 | |
| *** mbeierl has quit IRC | 19:43 | |
| *** hwoarang has quit IRC | 19:48 | |
| *** hwoarang has joined #openstack-ironic | 19:49 | |
| *** andrein has joined #openstack-ironic | 19:51 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: fast tracked deployment support https://review.openstack.org/635996 | 19:56 |
| TheJulia | mgoddard: dtantsur|afk ^ | 19:57 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add fast-track testing https://review.openstack.org/641104 | 20:07 |
| *** e0ne has joined #openstack-ironic | 20:20 | |
| openstackgerrit | Nisha Brahmankar proposed openstack/python-ironic-inspector-client master: Deprecates "uuid" parameters to Python calls when node is expected https://review.openstack.org/642564 | 20:43 |
| *** gmann_afk is now known as gmann | 20:54 | |
| *** mjturek has quit IRC | 20:57 | |
| *** bfournie has quit IRC | 21:07 | |
| *** baha has quit IRC | 21:19 | |
| *** betherly has joined #openstack-ironic | 21:21 | |
| *** betherly has quit IRC | 21:26 | |
| *** e0ne has quit IRC | 21:26 | |
| *** rcernin has joined #openstack-ironic | 22:01 | |
| *** betherly has joined #openstack-ironic | 22:08 | |
| *** hoonetorg has quit IRC | 22:09 | |
| *** hoonetorg has joined #openstack-ironic | 22:10 | |
| *** whoami-rajat has quit IRC | 22:12 | |
| *** betherly has quit IRC | 22:12 | |
| *** ianychoi_ is now known as ianychoi | 22:47 | |
| *** betherly has joined #openstack-ironic | 23:10 | |
| *** betherly has quit IRC | 23:15 | |
| TheJulia | goodnight folks, stepping away | 23:36 |
| *** betherly has joined #openstack-ironic | 23:42 | |
| *** dsneddon has quit IRC | 23:46 | |
| *** betherly has quit IRC | 23:46 | |
| openstackgerrit | Merged openstack/ironic stable/rocky: devstack: use stable/rocky IPA for stable/rocky ironic https://review.openstack.org/642400 | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!