*** namnh has joined #openstack-ironic | 00:00 | |
*** namnh has quit IRC | 00:01 | |
*** bfournie has joined #openstack-ironic | 00:08 | |
*** mjturek has quit IRC | 00:09 | |
*** gyee has quit IRC | 00:38 | |
*** zul has joined #openstack-ironic | 00:57 | |
*** phuongnh has joined #openstack-ironic | 01:03 | |
*** phuongnh has quit IRC | 01:59 | |
*** phuongnh has joined #openstack-ironic | 02:00 | |
*** gkadam_ has joined #openstack-ironic | 02:24 | |
*** gkadam__ has joined #openstack-ironic | 02:26 | |
*** gkadam_ has quit IRC | 02:29 | |
*** yolanda has quit IRC | 03:14 | |
*** jaypipes has quit IRC | 04:11 | |
*** jaypipes has joined #openstack-ironic | 04:11 | |
*** galaxyblr has joined #openstack-ironic | 04:43 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix not exist deploy image within irmc-virtual-media booting https://review.openstack.org/589702 | 04:51 |
---|---|---|
*** hoonetorg has quit IRC | 05:31 | |
*** e0ne has joined #openstack-ironic | 05:40 | |
*** gkadam__ has quit IRC | 05:42 | |
*** gkadam has joined #openstack-ironic | 05:44 | |
*** e0ne has quit IRC | 05:52 | |
*** hoonetorg has joined #openstack-ironic | 05:52 | |
*** e0ne has joined #openstack-ironic | 06:03 | |
*** hoonetorg has quit IRC | 06:06 | |
*** yolanda has joined #openstack-ironic | 06:06 | |
*** gkadam has quit IRC | 06:09 | |
*** gkadam has joined #openstack-ironic | 06:24 | |
*** skazi has quit IRC | 06:26 | |
*** pcaruana has joined #openstack-ironic | 06:27 | |
*** trown|outtypewww has quit IRC | 06:42 | |
*** trown has joined #openstack-ironic | 06:43 | |
*** etingof has quit IRC | 06:44 | |
*** e0ne has quit IRC | 06:51 | |
openstackgerrit | Merged openstack/bifrost master: Fix ironic-inspector config https://review.openstack.org/589514 | 07:09 |
openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Dummy commit to trigger complete CI test pass https://review.openstack.org/589493 | 07:23 |
*** rcernin has quit IRC | 07:35 | |
*** jawadahmed has joined #openstack-ironic | 07:40 | |
*** pmannidi has quit IRC | 07:46 | |
*** chason has quit IRC | 07:49 | |
*** w-miller has joined #openstack-ironic | 07:57 | |
*** jawadahmed is now known as Jay_ | 07:59 | |
*** moshele has joined #openstack-ironic | 08:03 | |
openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Create symlink for ironic-rootwrap when using venv https://review.openstack.org/589422 | 08:08 |
openstackgerrit | Merged openstack/ironic master: Update docs on ironic boot mode management https://review.openstack.org/588582 | 08:17 |
*** phuongnh has quit IRC | 08:22 | |
*** ajya has joined #openstack-ironic | 08:26 | |
*** tssurya has joined #openstack-ironic | 08:28 | |
openstackgerrit | paresh sao proposed openstack/ironic master: [WIP]OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 08:30 |
*** serlex has joined #openstack-ironic | 08:42 | |
*** derekh has joined #openstack-ironic | 08:46 | |
*** dougsz_ has joined #openstack-ironic | 08:51 | |
*** ajya has quit IRC | 08:52 | |
*** ajya has joined #openstack-ironic | 08:56 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Wrap Flask into oslo.service https://review.openstack.org/561823 | 09:00 |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix not exist deploy image within irmc-virtual-media booting https://review.openstack.org/589702 | 09:24 |
*** etingof has joined #openstack-ironic | 09:30 | |
*** priteau has quit IRC | 09:45 | |
*** priteau has joined #openstack-ironic | 09:45 | |
*** ajya has quit IRC | 09:46 | |
*** ajya has joined #openstack-ironic | 09:46 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Remove deprecated option node_status_keep_time https://review.openstack.org/589818 | 09:47 |
*** tonyb has quit IRC | 10:16 | |
openstackgerrit | Giulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors https://review.openstack.org/589831 | 10:18 |
*** tonyb has joined #openstack-ironic | 10:23 | |
*** yolanda has quit IRC | 10:36 | |
*** dougsz_ has quit IRC | 10:38 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add support for loading standard registries from filesystem https://review.openstack.org/589837 | 10:52 |
*** dougsz_ has joined #openstack-ironic | 11:17 | |
openstackgerrit | paresh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 11:18 |
openstackgerrit | paresh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers. https://review.openstack.org/555664 | 11:19 |
*** phuongnh has joined #openstack-ironic | 11:20 | |
*** bfournie has quit IRC | 11:21 | |
jroll | \o | 11:32 |
etingof | good morning jroll o/ | 11:36 |
*** phuongnh has quit IRC | 11:46 | |
*** chason has joined #openstack-ironic | 11:52 | |
*** bfournie has joined #openstack-ironic | 11:54 | |
TheJulia | Good morning | 11:55 |
*** yolanda has joined #openstack-ironic | 11:57 | |
etingof | good morning TheJulia ;) | 12:00 |
openstackgerrit | Aleksandra Bezborodova proposed openstack/sushy master: Add a virtual media resource https://review.openstack.org/570810 | 12:08 |
*** MattMan has quit IRC | 12:12 | |
*** MattMan has joined #openstack-ironic | 12:12 | |
*** tssurya has quit IRC | 12:13 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-inspector-specs master: Enumerate Inspector errors https://review.openstack.org/494534 | 12:21 |
*** rh-jelabarre has quit IRC | 12:26 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-specs master: IPA to report node activity https://review.openstack.org/492702 | 12:30 |
*** slagle has joined #openstack-ironic | 12:33 | |
*** e0ne has joined #openstack-ironic | 12:34 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic-lib master: Added `get_raw_image_mb()` https://review.openstack.org/553807 | 12:35 |
*** tssurya has joined #openstack-ironic | 12:36 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Raw image size estimation improved https://review.openstack.org/544839 | 12:36 |
*** slagle has quit IRC | 12:48 | |
*** gfidente has joined #openstack-ironic | 12:52 | |
openstackgerrit | Moshe Levi proposed openstack/ironic-specs master: Add Support for Smart NIC https://review.openstack.org/582767 | 12:53 |
gfidente | dtantsur|afk regarding https://review.openstack.org/#/c/589831/ , how do I "link" to a story from the commit message? | 12:55 |
patchbot | patch 589831 - ironic-python-agent - Clear GPT and MBR headers with dd to avoid sgdisk ... | 12:55 |
jroll | gfidente: just add "Story: 12345" at the end of the commit message | 12:59 |
gfidente | jroll ah thanks | 13:00 |
TheJulia | Folks, reviews on https://review.openstack.org/#/c/589277 https://review.openstack.org/#/c/589702/2 and https://review.openstack.org/#/c/586618/ would be much appreciated | 13:00 |
patchbot | patch 589277 - ironic - Add conductor_group docs | 13:00 |
patchbot | patch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med... | 13:00 |
patchbot | patch 586618 - ironic - Add admin-only provisioner reference arch | 13:00 |
TheJulia | gfidente: also add Task: tasknumber | 13:00 |
TheJulia | gfidente: that way storyboard will track the patch and close it out | 13:00 |
jroll | oh right | 13:00 |
gfidente | TheJulia how do I get the task number? | 13:00 |
TheJulia | gfidente: They are listed in the storyboard page. All items migrated have at least one task. | 13:00 |
gfidente | wait let me see if I can find it | 13:01 |
gfidente | thanks meantime :D | 13:01 |
TheJulia | https://storyboard.openstack.org/#!/story/1737556 | 13:01 |
openstackgerrit | Giulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors https://review.openstack.org/589831 | 13:03 |
gfidente | please, don't take it as a review request but could you please see if the storyboard numbers I put make sense? | 13:03 |
gfidente | ^^ | 13:03 |
*** markguz_ has joined #openstack-ironic | 13:04 | |
TheJulia | gfidente: you didn't need to copy the content and create a new story | 13:05 |
TheJulia | gfidente: when we migrated from launchpad, we copied everything | 13:05 |
TheJulia | minus attachments | 13:05 |
TheJulia | gfidente: replied on the patch | 13:07 |
*** markguz_ has quit IRC | 13:08 | |
*** e0ne has quit IRC | 13:18 | |
gfidente | TheJulia I must be missing something, in https://storyboard.openstack.org/#!/story/2003352 I see a different task number | 13:27 |
TheJulia | gfidente: assigned when you created the duplicate story? | 13:28 |
gfidente | I created the story manually | 13:29 |
gfidente | do you mean there is a process copying story into storyboard automatically? | 13:29 |
gfidente | guess so ;D | 13:30 |
*** e0ne has joined #openstack-ironic | 13:31 | |
TheJulia | yeah | 13:31 |
TheJulia | Reading the bug... I wonder if https://review.openstack.org/#/c/568518/ changes the need | 13:31 |
patchbot | patch 568518 - ironic-lib - Expose GPT partitioning fixing method (MERGED) | 13:31 |
openstackgerrit | Giulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors https://review.openstack.org/589831 | 13:34 |
gfidente | can we close this as dup https://storyboard.openstack.org/#!/story/2003352 ? | 13:34 |
openstackgerrit | Dirk Mueller proposed openstack/ironic-inspector master: Remove unused pydot3 dependency https://review.openstack.org/589901 | 13:39 |
*** mjturek has joined #openstack-ironic | 13:39 | |
TheJulia | gfidente: sure | 13:40 |
gfidente | TheJulia how? | 13:40 |
TheJulia | gfidente: mark invalid please | 13:40 |
TheJulia | and comment of course | 13:40 |
*** baha has joined #openstack-ironic | 13:40 | |
*** Jay_ has quit IRC | 13:47 | |
* TheJulia hopes we can get documents merged today so I can release 11.1 | 13:48 | |
*** mjturek has quit IRC | 13:50 | |
*** mjturek has joined #openstack-ironic | 13:52 | |
*** markguz_ has joined #openstack-ironic | 13:54 | |
*** mjturek has quit IRC | 13:58 | |
*** rh-jelabarre has joined #openstack-ironic | 13:58 | |
*** rh-jelabarre has quit IRC | 13:59 | |
*** rpioso|afk is now known as rpioso | 13:59 | |
rpioso | Good morning, ironicers | 13:59 |
*** markguz_ has quit IRC | 14:00 | |
*** mjturek has joined #openstack-ironic | 14:00 | |
*** markguz_ has joined #openstack-ironic | 14:06 | |
jroll | TIL pip install -e does not install entrypoints | 14:15 |
*** mgoddard has joined #openstack-ironic | 14:16 | |
*** hongbin has joined #openstack-ironic | 14:18 | |
sambetts|afk | jroll: huh i thought it did, but if you add new ones you have to rerun it because it won't do it dynamically? | 14:18 |
jroll | sambetts|afk: seems not, removing -e let my driver load correctly | 14:20 |
jroll | I thought it did too | 14:20 |
jroll | or rather I *know* it did, not sure what happened | 14:20 |
sambetts|afk | oh weird :/ | 14:21 |
TheJulia | NobodyCam: if you have a moment, could you take a look at https://review.openstack.org/#/c/589422/3 | 14:23 |
patchbot | patch 589422 - bifrost - Create symlink for ironic-rootwrap when using venv | 14:23 |
*** galaxyblr has quit IRC | 14:23 | |
*** moshele has quit IRC | 14:29 | |
*** bdodd has joined #openstack-ironic | 14:34 | |
*** gkadam has quit IRC | 14:43 | |
*** priteau has quit IRC | 14:53 | |
*** e0ne has quit IRC | 14:58 | |
*** priteau has joined #openstack-ironic | 15:04 | |
*** chason has quit IRC | 15:06 | |
*** chason has joined #openstack-ironic | 15:07 | |
*** chason has quit IRC | 15:07 | |
*** chason has joined #openstack-ironic | 15:08 | |
*** gkadam has joined #openstack-ironic | 15:24 | |
NobodyCam | Good Morning Ironic'ers | 15:31 |
NobodyCam | Happy Hump day! | 15:31 |
*** pcaruana has quit IRC | 15:34 | |
*** skazi has joined #openstack-ironic | 15:36 | |
*** prometheanfire has joined #openstack-ironic | 15:40 | |
*** jlvillal is now known as jlv-sick | 15:40 | |
prometheanfire | when migrating to hardware types, it looks like agent_ipmitool (or pxe_ipmitool) now requires python-ironic-inspector-client for the inspector (where previously it seemed to work without it) can I get a confirmation that this is intended? | 15:42 |
prometheanfire | https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html | 15:42 |
prometheanfire | this was tested in queens | 15:42 |
*** markguz_ has quit IRC | 15:54 | |
jroll | prometheanfire: where it worked without it, were you using inspection? | 15:54 |
*** mjturek has quit IRC | 15:54 | |
jroll | python-ironic-inspector-client dependency was added a long time ago, to be able to use inspection: https://github.com/openstack/ironic/commit/08e66ef8ecebabf97549a89e0ded8986a844fd76 | 15:55 |
prometheanfire | I don't think so | 15:55 |
prometheanfire | I'm guessing we just got by without it | 15:55 |
prometheanfire | but under the new world order we need it | 15:55 |
jroll | but if you aren't using inspection, you can use the no-inspect interface with the ipmi hardware type | 15:55 |
jroll | I guess it defaults to using inspector though | 15:55 |
* jroll pokes around | 15:56 | |
prometheanfire | yep, no fallback it seems | 15:56 |
jroll | er, inspector is not enabled by default: https://github.com/openstack/ironic/blob/master/ironic/conf/default.py#L120 | 15:57 |
*** tssurya has quit IRC | 15:57 | |
prometheanfire | looks like I'm just enabling it by default now https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html | 15:57 |
jroll | I guess, what are you hitting? are you packaging or is this a deployment? | 15:57 |
prometheanfire | from the ipmitool rows | 15:57 |
*** mjturek has joined #openstack-ironic | 15:57 | |
prometheanfire | https://github.com/openstack/openstack-ansible-os_ironic/blob/master/vars/main.yml#L27-L33 | 15:58 |
jroll | right, so in the past: agent_ipmitool used inspector, but there was a config option to enable it (default disabled) | 15:58 |
jroll | now ipmi hardware type can use inspector, if the inspector interface is enabled in enabled_inspect_interfaces | 15:59 |
prometheanfire | ok, so enabled but still disabled (in the past) | 15:59 |
prometheanfire | yep, which we enable here https://github.com/openstack/openstack-ansible-os_ironic/blob/master/templates/ironic.conf.j2#L22 | 16:00 |
prometheanfire | in any case we still need the client in the venv to support it | 16:00 |
jroll | ah ok | 16:00 |
jroll | yep | 16:00 |
jroll | which was true in the past :) | 16:00 |
prometheanfire | yep, we just got by | 16:00 |
jroll | not sure how, heh | 16:01 |
*** mjturek has quit IRC | 16:01 | |
jroll | prometheanfire: hope that helps | 16:02 |
prometheanfire | we never changed the default disabled to enabled | 16:02 |
*** mjturek has joined #openstack-ironic | 16:02 | |
jroll | ah, I see | 16:02 |
*** sambetts|afk is now known as sambetts | 16:03 | |
jroll | surprised nobody used that | 16:03 |
openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Detection of config errors for netmiko drivers https://review.openstack.org/589960 | 16:04 |
openstackgerrit | Mark Goddard proposed openstack/networking-generic-switch master: Detection of errors for Dell PowerConnect devices https://review.openstack.org/589961 | 16:04 |
prometheanfire | not many used the role, I think I'm jumping on that grenade :P | 16:04 |
prometheanfire | ya, needs to be added here https://github.com/openstack/openstack-ansible-os_ironic/blob/master/defaults/main.yml#L210 | 16:05 |
prometheanfire | anything else we need? | 16:05 |
prometheanfire | stuff from here? https://github.com/openstack/ironic/blob/master/driver-requirements.txt | 16:06 |
jroll | so those are all needed based on the drivers used | 16:06 |
prometheanfire | looks like those are not in requirements (the project) | 16:06 |
jroll | e.g. ilo driver needs proliantutils and the ilorest thing | 16:06 |
jroll | right, we should really use [extras] for them, but nobody has done that | 16:07 |
prometheanfire | something for stein? | 16:07 |
jroll | maybe? | 16:07 |
jroll | I think the biggest hurdle is getting them into g-r | 16:08 |
*** derekh has quit IRC | 16:08 | |
prometheanfire | that's not so bad as long as the requirements that they pull in are compatable (no caps....) | 16:08 |
jroll | yeah, I've not looked into it all | 16:08 |
jroll | so far nobody has really complained about the structure we have today | 16:09 |
prometheanfire | python-scciclient is an openstack project | 16:09 |
*** derekh has joined #openstack-ironic | 16:09 | |
jroll | I don't think it's official | 16:09 |
jroll | just hosted here | 16:09 |
prometheanfire | so is proliant utils? | 16:09 |
prometheanfire | but not updated in a year | 16:09 |
prometheanfire | https://github.com/openstack/proliantutils | 16:10 |
prometheanfire | and it caps some things :| | 16:10 |
jroll | sure, being in the openstack namespace doesn't tell us much about it | 16:10 |
prometheanfire | nothing too hurtful though | 16:10 |
prometheanfire | ya | 16:10 |
*** trown is now known as trown|lunch | 16:13 | |
prometheanfire | are these used anymore or did you migrate? | 16:13 |
prometheanfire | 3 - ironic-oneview-cli | 16:13 |
prometheanfire | 2 - ironic-oneviewd | 16:14 |
jroll | I would assume those moved | 16:14 |
jroll | actually I think the oneview driver was removed | 16:14 |
jroll | bah, maybe not | 16:14 |
jroll | it was supposed to be though: https://github.com/openstack/ironic/blob/master/ironic/drivers/oneview.py#L37 | 16:15 |
jroll | oh stein is next | 16:15 |
prometheanfire | ya, removing oneview will simplify things | 16:17 |
prometheanfire | I need to but osa people about how they want to handle these drivers | 16:17 |
jroll | I feel like I'd have a config for the list of enabled drivers | 16:18 |
jroll | and install things based on that | 16:18 |
jroll | I guess extras would be super helpful there eh | 16:18 |
prometheanfire | :P | 16:19 |
prometheanfire | I don't think we support using extras in OSA, but it is generally good | 16:19 |
jroll | well you could at least do like... for driver in enabled_drivers: pip_command.append('[%s]' % driver) | 16:21 |
prometheanfire | jroll: do you do a co-installability test (install all those drivers with your tests)? | 16:23 |
prometheanfire | as long as you do that then basic co-installability with openstack sounds good | 16:23 |
*** serlex has quit IRC | 16:24 | |
jroll | prometheanfire: I believe so, we had bugs around it before | 16:25 |
* jroll double checks | 16:25 | |
jroll | maybe not? we used to | 16:25 |
*** mjturek has quit IRC | 16:28 | |
jroll | what happened to that >:( | 16:28 |
prometheanfire | lol | 16:28 |
* prometheanfire lobs a grenade | 16:28 | |
*** moshele has joined #openstack-ironic | 16:30 | |
*** mjturek has joined #openstack-ironic | 16:34 | |
jroll | wtaf | 16:35 |
jroll | I think I'm imagining things | 16:35 |
jroll | this bug started the conversation https://storyboard.openstack.org/#!/story/1518999 | 16:35 |
jroll | but no clue where it went from there | 16:35 |
openstackgerrit | Giulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors https://review.openstack.org/589831 | 16:40 |
*** marlinc has quit IRC | 16:45 | |
*** moshele has quit IRC | 16:45 | |
openstackgerrit | Merged openstack/bifrost master: Create symlink for ironic-rootwrap when using venv https://review.openstack.org/589422 | 16:48 |
*** gyee has joined #openstack-ironic | 16:52 | |
*** mgoddard has quit IRC | 16:54 | |
*** w-miller has quit IRC | 16:56 | |
TheJulia | prometheanfire: jroll: we basically did with bifrost, although I'm having to exclude ucs stuffs for python3 reasons | 16:56 |
TheJulia | prometheanfire: jroll: I never put irmc in though, seems like I should do that | 16:56 |
prometheanfire | TheJulia: so I should add 'irmc' to my pip install list? | 16:57 |
TheJulia | prometheanfire: no, irmc driver uses python-scciclient | 16:58 |
jroll | prometheanfire: irmc is the driver, I believe python-scciclient is... yeah | 16:58 |
prometheanfire | ok, scciclient is already in | 16:58 |
*** marlinc has joined #openstack-ironic | 16:59 | |
* TheJulia guesses she should release 11.1 | 17:00 | |
*** gkadam has quit IRC | 17:01 | |
*** bfournie has quit IRC | 17:02 | |
*** skazi has quit IRC | 17:02 | |
* TheJulia was hoping docs would be in the branch, but c'est la vie | 17:05 | |
TheJulia | on a plus side, we can unblock stuff and begin merging new things \o/ | 17:05 |
TheJulia | once I put the release in | 17:05 |
*** skazi has joined #openstack-ironic | 17:06 | |
*** dougsz_ has quit IRC | 17:09 | |
*** e0ne has joined #openstack-ironic | 17:11 | |
*** markguz_ has joined #openstack-ironic | 17:12 | |
*** markguz_ has quit IRC | 17:14 | |
*** ajya has quit IRC | 17:14 | |
*** ajya has joined #openstack-ironic | 17:14 | |
*** markguz_ has joined #openstack-ironic | 17:14 | |
*** trown|lunch is now known as trown | 17:16 | |
*** derekh has quit IRC | 17:18 | |
jroll | \o/ | 17:18 |
*** markguz_ has quit IRC | 17:19 | |
*** skazi has quit IRC | 17:21 | |
*** sambetts is now known as sambetts|afk | 17:29 | |
*** markguz_ has joined #openstack-ironic | 17:35 | |
*** moshele has joined #openstack-ironic | 17:40 | |
*** phuongnh has joined #openstack-ironic | 17:43 | |
*** moshele has quit IRC | 17:44 | |
*** gkadam has joined #openstack-ironic | 17:46 | |
*** bfournie has joined #openstack-ironic | 17:46 | |
*** phuongnh has quit IRC | 17:47 | |
*** markguz_ has quit IRC | 17:51 | |
*** markguz_ has joined #openstack-ironic | 17:52 | |
*** markguz_ has quit IRC | 17:53 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add support for loading resources from archive file https://review.openstack.org/589147 | 17:57 |
*** gfidente has quit IRC | 18:00 | |
*** e0ne has quit IRC | 18:03 | |
mbeierl | another dumb question - how can I enable logging for a unit test? I tried OS_LOG_CAPTURE=True and OS_DEBUG=True, but i'm still not getting log messages when I put them into test code? | 18:05 |
openstackgerrit | Merged openstack/ironic stable/queens: Fix error thrown by logging in common/neutron.py https://review.openstack.org/589471 | 18:15 |
*** moshele has joined #openstack-ironic | 18:20 | |
*** e0ne has joined #openstack-ironic | 18:20 | |
*** moshele has quit IRC | 18:21 | |
*** ajya has quit IRC | 18:23 | |
*** mjturek has quit IRC | 18:25 | |
TheJulia | pas-ha: you around? | 18:38 |
TheJulia | mbeierl: that is a great question. Typically I've just put in debugging prints to try and determine exactly what is occuring in terms of debugging tests... but getting the actual log output I'm not sure. So many tests actually mock log and collect/test the contents, so I'm not sure you really can reliably. | 18:39 |
*** mjturek has joined #openstack-ironic | 18:44 | |
* TheJulia watches her laptop melt from the releases repo tests | 18:45 | |
mbeierl | TheJulia: Ok, thanks. I was hoping to see some of the debug logs in the modules/drac interface, but i'll see what I can do. Unfortunately, from digging a little into the unit test case setup, it looks like it's the FakeLogger that is taking over an suppressing it, which is where I got the env vars from, but they didn't seem to do much. | 18:59 |
*** w-miller has joined #openstack-ironic | 19:00 | |
*** e0ne has quit IRC | 19:11 | |
*** skazi has joined #openstack-ironic | 19:17 | |
*** mjturek has quit IRC | 19:17 | |
*** gkadam has quit IRC | 19:18 | |
*** e0ne has joined #openstack-ironic | 19:19 | |
*** w-miller has quit IRC | 19:21 | |
*** mjturek has joined #openstack-ironic | 19:23 | |
*** e0ne has quit IRC | 19:23 | |
TheJulia | jroll: Any chance I can get a review on https://review.openstack.org/#/c/589702/ ? | 19:32 |
patchbot | patch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med... | 19:32 |
TheJulia | its a quick review | 19:32 |
*** mjturek has quit IRC | 19:32 | |
*** mjturek has joined #openstack-ironic | 19:34 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost master: Begin change over to use python3 https://review.openstack.org/588325 | 19:39 |
*** w-miller has joined #openstack-ironic | 19:39 | |
openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Change jobs for Python3 effort https://review.openstack.org/589295 | 19:50 |
*** w-miller has quit IRC | 19:52 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Changing CI job templates for python3-first https://review.openstack.org/590019 | 19:59 |
*** mjturek has quit IRC | 20:01 | |
*** mjturek has joined #openstack-ironic | 20:04 | |
*** anupn has quit IRC | 20:06 | |
*** luzC has quit IRC | 20:07 | |
*** mjturek has quit IRC | 20:08 | |
*** luzC has joined #openstack-ironic | 20:09 | |
jroll | that looks odd | 20:13 |
jroll | TheJulia: -1 :P | 20:19 |
*** slagle has joined #openstack-ironic | 20:39 | |
JayF | TheJulia: jroll: https://superuser.openstack.org/articles/whats-next-ironic-1-1/ someone needs to s/1.1/11.1/ in the title there... | 20:43 |
jroll | woof | 20:43 |
jroll | thanks JayF | 20:44 |
*** baha has quit IRC | 20:46 | |
jroll | TheJulia: I'm emailing her, jfyi | 20:49 |
TheJulia | JayF: Heh, I suspect it was CMS text parsing weirdness, but ETOOMANYTHINGS | 20:50 |
TheJulia | jroll: thanks | 20:50 |
TheJulia | jroll: evil -1's ;) | 20:50 |
jroll | TheJulia: it's broken! | 20:50 |
TheJulia | :( | 20:50 |
jroll | like, provably | 20:50 |
TheJulia | Something didn't feel right, but I figure in CI they would likely be using local files instead of urls, which would have bypassed it | 20:51 |
* TheJulia goes back to the smartnics spec | 20:51 | |
jroll | I don't think it's covered in CI | 20:51 |
*** markguz_ has joined #openstack-ironic | 20:54 | |
*** markguz_ has quit IRC | 20:58 | |
TheJulia | That was what I was thinking when I looked at it | 20:59 |
*** trown is now known as trown|outtypewww | 21:05 | |
TheJulia | yeah, that is likely broken | 21:07 |
TheJulia | good thing looking further up in the code | 21:07 |
*** pcaruana has joined #openstack-ironic | 21:07 | |
TheJulia | I'll hold on the 11.1 release till tomorrow morning, I need to do the cycle highlights | 21:08 |
TheJulia | which I already posted that I'll do tomorrow | 21:08 |
*** slagle has quit IRC | 21:13 | |
*** slagle has joined #openstack-ironic | 21:14 | |
*** priteau has quit IRC | 21:20 | |
openstackgerrit | Bill Dodd proposed openstack/ironic master: Add BIOS interface to Redfish hardware type https://review.openstack.org/582314 | 21:22 |
rpioso | bdodd: \o/ | 21:23 |
*** slagle has quit IRC | 21:23 | |
TheJulia | neat! | 21:28 |
TheJulia | goodnight folks, I'm going to go have a cidre and cook some dinner | 21:28 |
bdodd | goodnight, TheJulia | 21:30 |
rpioso | TheJulia: Enjoy! | 21:41 |
*** pcaruana has quit IRC | 22:02 | |
*** moshele has joined #openstack-ironic | 22:06 | |
openstackgerrit | Bill Dodd proposed openstack/ironic master: Add BIOS interface to Redfish hardware type https://review.openstack.org/582314 | 22:09 |
* bdodd hopes that rebasing and uploading new patchset resolves merge conflict | 22:15 | |
*** JayF has quit IRC | 22:18 | |
*** rcernin has joined #openstack-ironic | 22:19 | |
*** JayF has joined #openstack-ironic | 22:19 | |
*** moshele has quit IRC | 22:25 | |
*** priteau has joined #openstack-ironic | 22:25 | |
*** hamzy_ has joined #openstack-ironic | 22:40 | |
*** hamzy has quit IRC | 22:42 | |
*** hongbin has quit IRC | 22:54 | |
*** jtomasek_ has joined #openstack-ironic | 22:54 | |
*** gmann_ has joined #openstack-ironic | 22:55 | |
*** r-daneel has joined #openstack-ironic | 22:59 | |
*** rpioso is now known as rpioso|afk | 23:01 | |
*** amotoki_ has joined #openstack-ironic | 23:01 | |
*** ddurst- has joined #openstack-ironic | 23:02 | |
*** jtomasek has quit IRC | 23:02 | |
*** ddurst has quit IRC | 23:02 | |
*** UForgotten has quit IRC | 23:02 | |
*** mtreinish has quit IRC | 23:02 | |
*** penick has quit IRC | 23:02 | |
*** oanson has quit IRC | 23:02 | |
*** pc_m has quit IRC | 23:02 | |
*** honza has quit IRC | 23:02 | |
*** lucasagomes has quit IRC | 23:02 | |
*** lhinds has quit IRC | 23:02 | |
*** amotoki has quit IRC | 23:02 | |
*** gmann has quit IRC | 23:02 | |
*** gmann_ is now known as gmann | 23:02 | |
*** mtreinish has joined #openstack-ironic | 23:04 | |
*** mariojv has joined #openstack-ironic | 23:09 | |
TheJulia | bdodd: when you run git review, it does try to rebase off of the state of master that it knows, but doesn't update master | 23:44 |
TheJulia | so you might want to do ; git checkout master; git pull --ff-only ; git checkout my-current-dev-branch ; git review | 23:44 |
TheJulia | bdodd: worst comes to worst, git rebase -i master | 23:45 |
TheJulia | then git review | 23:45 |
*** rmcall has joined #openstack-ironic | 23:50 | |
*** moshele has joined #openstack-ironic | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!