Wednesday, 2018-08-08

*** namnh has joined #openstack-ironic00:00
*** namnh has quit IRC00:01
*** bfournie has joined #openstack-ironic00:08
*** mjturek has quit IRC00:09
*** gyee has quit IRC00:38
*** zul has joined #openstack-ironic00:57
*** phuongnh has joined #openstack-ironic01:03
*** phuongnh has quit IRC01:59
*** phuongnh has joined #openstack-ironic02:00
*** gkadam_ has joined #openstack-ironic02:24
*** gkadam__ has joined #openstack-ironic02:26
*** gkadam_ has quit IRC02:29
*** yolanda has quit IRC03:14
*** jaypipes has quit IRC04:11
*** jaypipes has joined #openstack-ironic04:11
*** galaxyblr has joined #openstack-ironic04:43
openstackgerritNguyen Van Trung proposed openstack/ironic master: Fix not exist deploy image within irmc-virtual-media booting  https://review.openstack.org/58970204:51
*** hoonetorg has quit IRC05:31
*** e0ne has joined #openstack-ironic05:40
*** gkadam__ has quit IRC05:42
*** gkadam has joined #openstack-ironic05:44
*** e0ne has quit IRC05:52
*** hoonetorg has joined #openstack-ironic05:52
*** e0ne has joined #openstack-ironic06:03
*** hoonetorg has quit IRC06:06
*** yolanda has joined #openstack-ironic06:06
*** gkadam has quit IRC06:09
*** gkadam has joined #openstack-ironic06:24
*** skazi has quit IRC06:26
*** pcaruana has joined #openstack-ironic06:27
*** trown|outtypewww has quit IRC06:42
*** trown has joined #openstack-ironic06:43
*** etingof has quit IRC06:44
*** e0ne has quit IRC06:51
openstackgerritMerged openstack/bifrost master: Fix ironic-inspector config  https://review.openstack.org/58951407:09
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Dummy commit to trigger complete CI test pass  https://review.openstack.org/58949307:23
*** rcernin has quit IRC07:35
*** jawadahmed has joined #openstack-ironic07:40
*** pmannidi has quit IRC07:46
*** chason has quit IRC07:49
*** w-miller has joined #openstack-ironic07:57
*** jawadahmed is now known as Jay_07:59
*** moshele has joined #openstack-ironic08:03
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Create symlink for ironic-rootwrap when using venv  https://review.openstack.org/58942208:08
openstackgerritMerged openstack/ironic master: Update docs on ironic boot mode management  https://review.openstack.org/58858208:17
*** phuongnh has quit IRC08:22
*** ajya has joined #openstack-ironic08:26
*** tssurya has joined #openstack-ironic08:28
openstackgerritparesh sao proposed openstack/ironic master: [WIP]OOB RAID implementation for ilo5 based HPE Proliant servers.  https://review.openstack.org/55566408:30
*** serlex has joined #openstack-ironic08:42
*** derekh has joined #openstack-ironic08:46
*** dougsz_ has joined #openstack-ironic08:51
*** ajya has quit IRC08:52
*** ajya has joined #openstack-ironic08:56
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Wrap Flask into oslo.service  https://review.openstack.org/56182309:00
openstackgerritNguyen Van Trung proposed openstack/ironic master: Fix not exist deploy image within irmc-virtual-media booting  https://review.openstack.org/58970209:24
*** etingof has joined #openstack-ironic09:30
*** priteau has quit IRC09:45
*** priteau has joined #openstack-ironic09:45
*** ajya has quit IRC09:46
*** ajya has joined #openstack-ironic09:46
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Remove deprecated option node_status_keep_time  https://review.openstack.org/58981809:47
*** tonyb has quit IRC10:16
openstackgerritGiulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors  https://review.openstack.org/58983110:18
*** tonyb has joined #openstack-ironic10:23
*** yolanda has quit IRC10:36
*** dougsz_ has quit IRC10:38
openstackgerritAija Jaunteva proposed openstack/sushy master: Add support for loading standard registries from filesystem  https://review.openstack.org/58983710:52
*** dougsz_ has joined #openstack-ironic11:17
openstackgerritparesh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers.  https://review.openstack.org/55566411:18
openstackgerritparesh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers.  https://review.openstack.org/55566411:19
*** phuongnh has joined #openstack-ironic11:20
*** bfournie has quit IRC11:21
jroll\o11:32
etingofgood morning jroll o/11:36
*** phuongnh has quit IRC11:46
*** chason has joined #openstack-ironic11:52
*** bfournie has joined #openstack-ironic11:54
TheJuliaGood morning11:55
*** yolanda has joined #openstack-ironic11:57
etingofgood morning TheJulia ;)12:00
openstackgerritAleksandra Bezborodova proposed openstack/sushy master: Add a virtual media resource  https://review.openstack.org/57081012:08
*** MattMan has quit IRC12:12
*** MattMan has joined #openstack-ironic12:12
*** tssurya has quit IRC12:13
openstackgerritIlya Etingof proposed openstack/ironic-inspector-specs master: Enumerate Inspector errors  https://review.openstack.org/49453412:21
*** rh-jelabarre has quit IRC12:26
openstackgerritIlya Etingof proposed openstack/ironic-specs master: IPA to report node activity  https://review.openstack.org/49270212:30
*** slagle has joined #openstack-ironic12:33
*** e0ne has joined #openstack-ironic12:34
openstackgerritIlya Etingof proposed openstack/ironic-lib master: Added `get_raw_image_mb()`  https://review.openstack.org/55380712:35
*** tssurya has joined #openstack-ironic12:36
openstackgerritIlya Etingof proposed openstack/ironic master: Raw image size estimation improved  https://review.openstack.org/54483912:36
*** slagle has quit IRC12:48
*** gfidente has joined #openstack-ironic12:52
openstackgerritMoshe Levi proposed openstack/ironic-specs master: Add Support for Smart NIC  https://review.openstack.org/58276712:53
gfidentedtantsur|afk regarding https://review.openstack.org/#/c/589831/ , how do I "link" to a story from the commit message?12:55
patchbotpatch 589831 - ironic-python-agent - Clear GPT and MBR headers with dd to avoid sgdisk ...12:55
jrollgfidente: just add "Story: 12345" at the end of the commit message12:59
gfidentejroll ah thanks13:00
TheJuliaFolks, 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 appreciated13:00
patchbotpatch 589277 - ironic - Add conductor_group docs13:00
patchbotpatch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med...13:00
patchbotpatch 586618 - ironic - Add admin-only provisioner reference arch13:00
TheJuliagfidente: also add Task: tasknumber13:00
TheJuliagfidente: that way storyboard will track the patch and close it out13:00
jrolloh right13:00
gfidenteTheJulia how do I get the task number?13:00
TheJuliagfidente: They are listed in the storyboard page. All items migrated have at least one task.13:00
gfidentewait let me see if I can find it13:01
gfidentethanks meantime :D13:01
TheJuliahttps://storyboard.openstack.org/#!/story/173755613:01
openstackgerritGiulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors  https://review.openstack.org/58983113:03
gfidenteplease, 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-ironic13:04
TheJuliagfidente: you didn't need to copy the content and create a new story13:05
TheJuliagfidente: when we migrated from launchpad, we copied everything13:05
TheJuliaminus attachments13:05
TheJuliagfidente: replied on the patch13:07
*** markguz_ has quit IRC13:08
*** e0ne has quit IRC13:18
gfidenteTheJulia I must be missing something, in https://storyboard.openstack.org/#!/story/2003352 I see a different task number13:27
TheJuliagfidente: assigned when you created the duplicate story?13:28
gfidenteI created the story manually13:29
gfidentedo you mean there is a process copying story into storyboard automatically?13:29
gfidenteguess so ;D13:30
*** e0ne has joined #openstack-ironic13:31
TheJuliayeah13:31
TheJuliaReading the bug... I wonder if https://review.openstack.org/#/c/568518/ changes the need13:31
patchbotpatch 568518 - ironic-lib - Expose GPT partitioning fixing method (MERGED)13:31
openstackgerritGiulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors  https://review.openstack.org/58983113:34
gfidentecan we close this as dup https://storyboard.openstack.org/#!/story/2003352 ?13:34
openstackgerritDirk Mueller proposed openstack/ironic-inspector master: Remove unused pydot3 dependency  https://review.openstack.org/58990113:39
*** mjturek has joined #openstack-ironic13:39
TheJuliagfidente: sure13:40
gfidenteTheJulia how?13:40
TheJuliagfidente: mark invalid please13:40
TheJuliaand comment of course13:40
*** baha has joined #openstack-ironic13:40
*** Jay_ has quit IRC13:47
* TheJulia hopes we can get documents merged today so I can release 11.113:48
*** mjturek has quit IRC13:50
*** mjturek has joined #openstack-ironic13:52
*** markguz_ has joined #openstack-ironic13:54
*** mjturek has quit IRC13:58
*** rh-jelabarre has joined #openstack-ironic13:58
*** rh-jelabarre has quit IRC13:59
*** rpioso|afk is now known as rpioso13:59
rpiosoGood morning, ironicers13:59
*** markguz_ has quit IRC14:00
*** mjturek has joined #openstack-ironic14:00
*** markguz_ has joined #openstack-ironic14:06
jrollTIL pip install -e does not install entrypoints14:15
*** mgoddard has joined #openstack-ironic14:16
*** hongbin has joined #openstack-ironic14:18
sambetts|afkjroll: 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
jrollsambetts|afk: seems not, removing -e let my driver load correctly14:20
jrollI thought it did too14:20
jrollor rather I *know* it did, not sure what happened14:20
sambetts|afkoh weird :/14:21
TheJuliaNobodyCam: if you have a moment, could you take a look at https://review.openstack.org/#/c/589422/314:23
patchbotpatch 589422 - bifrost - Create symlink for ironic-rootwrap when using venv14:23
*** galaxyblr has quit IRC14:23
*** moshele has quit IRC14:29
*** bdodd has joined #openstack-ironic14:34
*** gkadam has quit IRC14:43
*** priteau has quit IRC14:53
*** e0ne has quit IRC14:58
*** priteau has joined #openstack-ironic15:04
*** chason has quit IRC15:06
*** chason has joined #openstack-ironic15:07
*** chason has quit IRC15:07
*** chason has joined #openstack-ironic15:08
*** gkadam has joined #openstack-ironic15:24
NobodyCamGood Morning Ironic'ers15:31
NobodyCamHappy Hump day!15:31
*** pcaruana has quit IRC15:34
*** skazi has joined #openstack-ironic15:36
*** prometheanfire has joined #openstack-ironic15:40
*** jlvillal is now known as jlv-sick15:40
prometheanfirewhen 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
prometheanfirehttps://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html15:42
prometheanfirethis was tested in queens15:42
*** markguz_ has quit IRC15:54
jrollprometheanfire: where it worked without it, were you using inspection?15:54
*** mjturek has quit IRC15:54
jrollpython-ironic-inspector-client dependency was added a long time ago, to be able to use inspection: https://github.com/openstack/ironic/commit/08e66ef8ecebabf97549a89e0ded8986a844fd7615:55
prometheanfireI don't think so15:55
prometheanfireI'm guessing we just got by without it15:55
prometheanfirebut under the new world order we need it15:55
jrollbut if you aren't using inspection, you can use the no-inspect interface with the ipmi hardware type15:55
jrollI guess it defaults to using inspector though15:55
* jroll pokes around15:56
prometheanfireyep, no fallback it seems15:56
jroller, inspector is not enabled by default: https://github.com/openstack/ironic/blob/master/ironic/conf/default.py#L12015:57
*** tssurya has quit IRC15:57
prometheanfirelooks like I'm just enabling it by default now https://docs.openstack.org/ironic/latest/admin/upgrade-to-hardware-types.html15:57
jrollI guess, what are you hitting? are you packaging or is this a deployment?15:57
prometheanfirefrom the ipmitool rows15:57
*** mjturek has joined #openstack-ironic15:57
prometheanfirehttps://github.com/openstack/openstack-ansible-os_ironic/blob/master/vars/main.yml#L27-L3315:58
jrollright, so in the past: agent_ipmitool used inspector, but there was a config option to enable it (default disabled)15:58
jrollnow ipmi hardware type can use inspector, if the inspector interface is enabled in enabled_inspect_interfaces15:59
prometheanfireok, so enabled but still disabled (in the past)15:59
prometheanfireyep, which we enable here https://github.com/openstack/openstack-ansible-os_ironic/blob/master/templates/ironic.conf.j2#L2216:00
prometheanfirein any case we still need the client in the venv to support it16:00
jrollah ok16:00
jrollyep16:00
jrollwhich was true in the past :)16:00
prometheanfireyep, we just got by16:00
jrollnot sure how, heh16:01
*** mjturek has quit IRC16:01
jrollprometheanfire: hope that helps16:02
prometheanfirewe never changed the default disabled to enabled16:02
*** mjturek has joined #openstack-ironic16:02
jrollah, I see16:02
*** sambetts|afk is now known as sambetts16:03
jrollsurprised nobody used that16:03
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Detection of config errors for netmiko drivers  https://review.openstack.org/58996016:04
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Detection of errors for Dell PowerConnect devices  https://review.openstack.org/58996116:04
prometheanfirenot many used the role, I think I'm jumping on that grenade :P16:04
prometheanfireya, needs to be added here https://github.com/openstack/openstack-ansible-os_ironic/blob/master/defaults/main.yml#L21016:05
prometheanfireanything else we need?16:05
prometheanfirestuff from here? https://github.com/openstack/ironic/blob/master/driver-requirements.txt16:06
jrollso those are all needed based on the drivers used16:06
prometheanfirelooks like those are not in requirements (the project)16:06
jrolle.g. ilo driver needs proliantutils and the ilorest thing16:06
jrollright, we should really use [extras] for them, but nobody has done that16:07
prometheanfiresomething for stein?16:07
jrollmaybe?16:07
jrollI think the biggest hurdle is getting them into g-r16:08
*** derekh has quit IRC16:08
prometheanfirethat's not so bad as long as the requirements that they pull in are compatable (no caps....)16:08
jrollyeah, I've not looked into it all16:08
jrollso far nobody has really complained about the structure we have today16:09
prometheanfirepython-scciclient is an openstack project16:09
*** derekh has joined #openstack-ironic16:09
jrollI don't think it's official16:09
jrolljust hosted here16:09
prometheanfireso is proliant utils?16:09
prometheanfirebut not updated in a year16:09
prometheanfirehttps://github.com/openstack/proliantutils16:10
prometheanfireand it caps some things :|16:10
jrollsure, being in the openstack namespace doesn't tell us much about it16:10
prometheanfirenothing too hurtful though16:10
prometheanfireya16:10
*** trown is now known as trown|lunch16:13
prometheanfireare these used anymore or did you migrate?16:13
prometheanfire    3   - ironic-oneview-cli16:13
prometheanfire    2   - ironic-oneviewd16:14
jrollI would assume those moved16:14
jrollactually I think the oneview driver was removed16:14
jrollbah, maybe not16:14
jrollit was supposed to be though: https://github.com/openstack/ironic/blob/master/ironic/drivers/oneview.py#L3716:15
jrolloh stein is next16:15
prometheanfireya, removing oneview will simplify things16:17
prometheanfireI need to but osa people about how they want to handle these drivers16:17
jrollI feel like I'd have a config for the list of enabled drivers16:18
jrolland install things based on that16:18
jrollI guess extras would be super helpful there eh16:18
prometheanfire:P16:19
prometheanfireI don't think we support using extras in OSA, but it is generally good16:19
jrollwell you could at least do like... for driver in enabled_drivers: pip_command.append('[%s]' % driver)16:21
prometheanfirejroll: do you do a co-installability test (install all those drivers with your tests)?16:23
prometheanfireas long as you do that then basic co-installability with openstack sounds good16:23
*** serlex has quit IRC16:24
jrollprometheanfire: I believe so, we had bugs around it before16:25
* jroll double checks16:25
jrollmaybe not? we used to16:25
*** mjturek has quit IRC16:28
jrollwhat happened to that >:(16:28
prometheanfirelol16:28
* prometheanfire lobs a grenade16:28
*** moshele has joined #openstack-ironic16:30
*** mjturek has joined #openstack-ironic16:34
jrollwtaf16:35
jrollI think I'm imagining things16:35
jrollthis bug started the conversation https://storyboard.openstack.org/#!/story/151899916:35
jrollbut no clue where it went from there16:35
openstackgerritGiulio Fidente proposed openstack/ironic-python-agent master: Clear GPT and MBR headers with dd to avoid sgdisk CRC errors  https://review.openstack.org/58983116:40
*** marlinc has quit IRC16:45
*** moshele has quit IRC16:45
openstackgerritMerged openstack/bifrost master: Create symlink for ironic-rootwrap when using venv  https://review.openstack.org/58942216:48
*** gyee has joined #openstack-ironic16:52
*** mgoddard has quit IRC16:54
*** w-miller has quit IRC16:56
TheJuliaprometheanfire: jroll: we basically did with bifrost, although I'm having to exclude ucs stuffs for python3 reasons16:56
TheJuliaprometheanfire: jroll: I  never put irmc in though, seems like I should do that16:56
prometheanfireTheJulia: so I should add 'irmc' to my pip install list?16:57
TheJuliaprometheanfire: no, irmc driver uses python-scciclient16:58
jrollprometheanfire: irmc is the driver, I believe python-scciclient is... yeah16:58
prometheanfireok, scciclient is already in16:58
*** marlinc has joined #openstack-ironic16:59
* TheJulia guesses she should release 11.117:00
*** gkadam has quit IRC17:01
*** bfournie has quit IRC17:02
*** skazi has quit IRC17:02
* TheJulia was hoping docs would be in the branch, but c'est la vie17:05
TheJuliaon a plus side, we can unblock stuff and begin merging new things \o/17:05
TheJuliaonce I put the release in17:05
*** skazi has joined #openstack-ironic17:06
*** dougsz_ has quit IRC17:09
*** e0ne has joined #openstack-ironic17:11
*** markguz_ has joined #openstack-ironic17:12
*** markguz_ has quit IRC17:14
*** ajya has quit IRC17:14
*** ajya has joined #openstack-ironic17:14
*** markguz_ has joined #openstack-ironic17:14
*** trown|lunch is now known as trown17:16
*** derekh has quit IRC17:18
jroll\o/17:18
*** markguz_ has quit IRC17:19
*** skazi has quit IRC17:21
*** sambetts is now known as sambetts|afk17:29
*** markguz_ has joined #openstack-ironic17:35
*** moshele has joined #openstack-ironic17:40
*** phuongnh has joined #openstack-ironic17:43
*** moshele has quit IRC17:44
*** gkadam has joined #openstack-ironic17:46
*** bfournie has joined #openstack-ironic17:46
*** phuongnh has quit IRC17:47
*** markguz_ has quit IRC17:51
*** markguz_ has joined #openstack-ironic17:52
*** markguz_ has quit IRC17:53
openstackgerritAija Jaunteva proposed openstack/sushy master: Add support for loading resources from archive file  https://review.openstack.org/58914717:57
*** gfidente has quit IRC18:00
*** e0ne has quit IRC18:03
mbeierlanother 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
openstackgerritMerged openstack/ironic stable/queens: Fix error thrown by logging in common/neutron.py  https://review.openstack.org/58947118:15
*** moshele has joined #openstack-ironic18:20
*** e0ne has joined #openstack-ironic18:20
*** moshele has quit IRC18:21
*** ajya has quit IRC18:23
*** mjturek has quit IRC18:25
TheJuliapas-ha: you around?18:38
TheJuliambeierl: 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-ironic18:44
* TheJulia watches her laptop melt from the releases repo tests18:45
mbeierlTheJulia: 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-ironic19:00
*** e0ne has quit IRC19:11
*** skazi has joined #openstack-ironic19:17
*** mjturek has quit IRC19:17
*** gkadam has quit IRC19:18
*** e0ne has joined #openstack-ironic19:19
*** w-miller has quit IRC19:21
*** mjturek has joined #openstack-ironic19:23
*** e0ne has quit IRC19:23
TheJuliajroll: Any chance I can get a review on https://review.openstack.org/#/c/589702/ ?19:32
patchbotpatch 589702 - ironic - Fix not exist deploy image within irmc-virtual-med...19:32
TheJuliaits a quick review19:32
*** mjturek has quit IRC19:32
*** mjturek has joined #openstack-ironic19:34
openstackgerritJulia Kreger proposed openstack/bifrost master: Begin change over to use python3  https://review.openstack.org/58832519:39
*** w-miller has joined #openstack-ironic19:39
openstackgerritJulia Kreger proposed openstack/virtualbmc master: Change jobs for Python3 effort  https://review.openstack.org/58929519:50
*** w-miller has quit IRC19:52
openstackgerritJulia Kreger proposed openstack/ironic master: Changing CI job templates for python3-first  https://review.openstack.org/59001919:59
*** mjturek has quit IRC20:01
*** mjturek has joined #openstack-ironic20:04
*** anupn has quit IRC20:06
*** luzC has quit IRC20:07
*** mjturek has quit IRC20:08
*** luzC has joined #openstack-ironic20:09
jrollthat looks odd20:13
jrollTheJulia: -1 :P20:19
*** slagle has joined #openstack-ironic20:39
JayFTheJulia: 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
jrollwoof20:43
jrollthanks JayF20:44
*** baha has quit IRC20:46
jrollTheJulia: I'm emailing her, jfyi20:49
TheJuliaJayF: Heh, I suspect it was CMS text parsing weirdness, but ETOOMANYTHINGS20:50
TheJuliajroll: thanks20:50
TheJuliajroll: evil -1's ;)20:50
jrollTheJulia: it's broken!20:50
TheJulia:(20:50
jrolllike, provably20:50
TheJuliaSomething didn't feel right, but I figure in CI they would likely be using local files instead of urls, which would have bypassed it20:51
* TheJulia goes back to the smartnics spec20:51
jrollI don't think it's covered in CI20:51
*** markguz_ has joined #openstack-ironic20:54
*** markguz_ has quit IRC20:58
TheJuliaThat was what I was thinking when I looked at it20:59
*** trown is now known as trown|outtypewww21:05
TheJuliayeah, that is likely broken21:07
TheJuliagood thing looking further up in the code21:07
*** pcaruana has joined #openstack-ironic21:07
TheJuliaI'll hold on the 11.1 release till tomorrow morning, I need to do the cycle highlights21:08
TheJuliawhich I already posted that I'll do tomorrow21:08
*** slagle has quit IRC21:13
*** slagle has joined #openstack-ironic21:14
*** priteau has quit IRC21:20
openstackgerritBill Dodd proposed openstack/ironic master: Add BIOS interface to Redfish hardware type  https://review.openstack.org/58231421:22
rpiosobdodd: \o/21:23
*** slagle has quit IRC21:23
TheJulianeat!21:28
TheJuliagoodnight folks, I'm going to go have a cidre and cook some dinner21:28
bdoddgoodnight, TheJulia21:30
rpiosoTheJulia: Enjoy!21:41
*** pcaruana has quit IRC22:02
*** moshele has joined #openstack-ironic22:06
openstackgerritBill Dodd proposed openstack/ironic master: Add BIOS interface to Redfish hardware type  https://review.openstack.org/58231422:09
* bdodd hopes that rebasing and uploading new patchset resolves merge conflict22:15
*** JayF has quit IRC22:18
*** rcernin has joined #openstack-ironic22:19
*** JayF has joined #openstack-ironic22:19
*** moshele has quit IRC22:25
*** priteau has joined #openstack-ironic22:25
*** hamzy_ has joined #openstack-ironic22:40
*** hamzy has quit IRC22:42
*** hongbin has quit IRC22:54
*** jtomasek_ has joined #openstack-ironic22:54
*** gmann_ has joined #openstack-ironic22:55
*** r-daneel has joined #openstack-ironic22:59
*** rpioso is now known as rpioso|afk23:01
*** amotoki_ has joined #openstack-ironic23:01
*** ddurst- has joined #openstack-ironic23:02
*** jtomasek has quit IRC23:02
*** ddurst has quit IRC23:02
*** UForgotten has quit IRC23:02
*** mtreinish has quit IRC23:02
*** penick has quit IRC23:02
*** oanson has quit IRC23:02
*** pc_m has quit IRC23:02
*** honza has quit IRC23:02
*** lucasagomes has quit IRC23:02
*** lhinds has quit IRC23:02
*** amotoki has quit IRC23:02
*** gmann has quit IRC23:02
*** gmann_ is now known as gmann23:02
*** mtreinish has joined #openstack-ironic23:04
*** mariojv has joined #openstack-ironic23:09
TheJuliabdodd: when you run git review, it does try to rebase off of the state of master that it knows, but doesn't update master23:44
TheJuliaso you might want to do ; git checkout master; git pull --ff-only ; git checkout my-current-dev-branch ; git review23:44
TheJuliabdodd: worst comes to worst, git rebase -i master23:45
TheJuliathen git review23:45
*** rmcall has joined #openstack-ironic23:50
*** moshele has joined #openstack-ironic23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!